fix(tree): forking during changed event (2.116 backport) - #28074
Merged
Jenn (jenn-le) merged 1 commit intoAug 26, 2026
Merged
Conversation
## Description [Forking](https://fluidframework.com/docs/api/fluid-framework/treeviewbeta-interface#fork-methodsignature) (beta) a view during the callback for the ["changed" event](https://fluidframework.com/docs/api/fluid-framework/treebranchevents-interface#changed-methodsignature) (alpha) emitted when a transaction is committed would create a fork with malformed change data. This could result in asserts being triggered when utilizing the fork (including, but not limited to, error code `0x7ce`). This PR also changes usages of `activeBranch` to `branch` wherever appropriate in `treeCheckout.ts`. I have added a PR comment explaining my reasoning for each change. ## Breaking Changes None
github-actions
Bot
added
base: release
Contributor
|
Warning WARNING: This PR is targeting a release branch! All changes must first be merged into Changes to release branches require approval from the Patch Triage group before merging. For more details, see our internal documentation for the patch policy and processes for |
Contributor
|
Hi! Thank you for opening this PR. Want me to review it? Based on the diff (40 lines, 3 files), I've queued these reviewers:
How this works
|
Navin Agarwal (agarwal-navin)
approved these changes
Aug 25, 2026
Contributor
Bundle size comparisonBase commit: Notable changesNo bundles changed by ≥ 500 bytes parsed. Per-bundle deltas
|
Jenn (jenn-le)
enabled auto-merge (squash)
August 25, 2026 22:13
Joshua Smithrud (Josmithr)
approved these changes
Aug 25, 2026
Taylor Williams (taylorsw04)
approved these changes
Aug 26, 2026
Jenn (jenn-le)
merged commit Aug 26, 2026
ca9458a
into
microsoft:release/client/2.116
36 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Cherry-picks commit
53eb97dcb06a9563caf37275f89f20b28104461dfrom PR #28008 intorelease/client/2.116.This fixes malformed change data when a tree view is forked during the
changedevent emitted after a transaction commits. The malformed fork could trigger assertions when it was later used.Reviewer Guidance
The review process is outlined in the pull request guidelines.
Please verify that this is an exact cherry-pick of the source commit.