Remove Transactor.isInProgress

[noencke]

Cleanup PR that replaces the now-redundant isInProgress with checks against size. Transactor.size has also been made a property rather than a method.

[Copilot]

Pull request overview

This PR removes the now-redundant isInProgress() method from the Transactor interface and converts size() from a method to a property. Since isInProgress() was semantically equivalent to size > 0, all usages have been replaced with direct size comparisons.

Changes:

• Removed Transactor.isInProgress() method from the interface
• Converted Transactor.size() from a method to a property getter
• Updated all call sites to check size === 0 or size > 0 instead of calling isInProgress()
• Updated documentation comments to reference Transactor.size instead of Transactor.isInProgress()

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
packages/dds/tree/src/shared-tree-core/transaction.ts	Removed isInProgress() method, converted size() to property getter, updated documentation comments
packages/dds/tree/src/shared-tree/treeCheckout.ts	Replaced isInProgress() calls with size === 0 or size > 0 checks
packages/dds/tree/src/shared-tree/sharedTree.ts	Replaced isInProgress() calls with size === 0 or size > 0 checks
packages/dds/tree/src/shared-tree/schematizingTreeView.ts	Replaced isInProgress() call with size > 0 check
packages/dds/tree/src/test/shared-tree/treeCheckout.spec.ts	Updated test assertions to check size property values directly
packages/dds/tree/src/test/shared-tree-core/transaction.spec.ts	Updated test assertions to use size property instead of isInProgress() and size() method
packages/dds/tree/src/test/shared-tree/fuzz/fuzzEditReducers.ts	Replaced isInProgress() calls with size property checks
packages/dds/tree/src/test/shared-tree/fuzz/fuzzEditGenerators.ts	Replaced isInProgress() calls with size > 0 checks