Skip to content

Make acknowledged MCP frame navigation atomic - #157

Merged
CameronFoxly merged 2 commits into
mainfrom
cameronfoxly-atomic-browser-navigation
Aug 11, 2026
Merged

Make acknowledged MCP frame navigation atomic#157
CameronFoxly merged 2 commits into
mainfrom
cameronfoxly-atomic-browser-navigation

Conversation

@CameronFoxly

@CameronFoxly CameronFoxly commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a hook-owned, awaitable frame navigation operation that flushes the installed canvas into its old content frame before moving the playhead
  • install populated or empty target-frame data into canvasStore and advance synchronization refs before returning applied.currentFrameIndex
  • reuse the serialized browser command dispatcher from Acknowledge serialized MCP browser commands #156 so immediately following current-frame batches cannot outrun navigation
  • preserve explicit set_cells_batch.frameIndex active/background targeting

Regression coverage

  • flush / playhead / load / acknowledgement ordering
  • populated and empty targets
  • invalid indices and failure propagation
  • pending layer switches
  • variable-duration go_to_frame(5) immediately followed by a targetless mutation, with both acknowledgements reporting timeline frame 5 while content ordinal 1 is mutated

Validation

  • focused MCP/navigation tests (26 tests)
  • adjacent synchronization suite (211 tests)
  • targeted ESLint
  • production build

Stacked on #156. Addresses #154.

CameronFoxly and others added 2 commits August 10, 2026 17:46
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>

Copilot-Session: c6779fac-ec14-40a7-8ce6-56fad219bbb6
@CameronFoxly
CameronFoxly force-pushed the cameronfoxly-atomic-browser-navigation branch from 39a0ca2 to b4393de Compare August 11, 2026 00:48
@CameronFoxly
CameronFoxly changed the base branch from cameronfoxly-browser-mcp-acknowledgements to main August 11, 2026 15:18
@CameronFoxly
CameronFoxly merged commit 79e0117 into main Aug 11, 2026
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