Skip to content

fix(flowchat): guard submission placement reentry - #3358

Merged
wsp1911 merged 2 commits into
GCWing:mainfrom
wsp1911:main
Oct 12, 2026
Merged

wsp1911 merged 2 commits into
GCWing:mainfrom
wsp1911:main

Conversation

@wsp1911

@wsp1911 wsp1911 commented Oct 12, 2026

Copy link
Copy Markdown
Collaborator

Prevent synchronous virtualizer measurement callbacks from re-entering
submission placement and consuming the pending intent twice.

Revalidate session, surface, and intent ownership before applying geometry.
Make placement state updates idempotent and clear inactive pending sends.

Cover cancellation, replacement, and measurement failure paths with
regression tests.

Prevent synchronous virtualizer measurement callbacks from re-entering
submission placement and consuming the pending intent twice.

Revalidate session, surface, and intent ownership before applying geometry.
Make placement state updates idempotent and clear inactive pending sends.

Cover cancellation, replacement, and measurement failure paths with
regression tests.
Mock the unified API boundary before transport initialization so pet
unit tests cannot leave WebSocket retries running during teardown.
Flush React updates on unmount and assert resize requests against the
API mock actually used by the component.

Keep floating menu changes from triggering window resize requests while
preserving layout measurement when the inline composer changes.
@wsp1911
wsp1911 merged commit 78f692b into GCWing:main Oct 12, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant