Conversation
Thread transfer impact✅ Thread transfer remains within every enforced ceiling.
Baseline: unavailable · PR result: Scenario and decoded snapshot size10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.
Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed. |
ApprovabilityVerdict: Not approved Macroscope's review found this PR not approvable — This PR adds user-facing navigation across queued messages and changes composer shortcut behavior, including protection for unsaved edits. It also adds a new shortcut to the product’s default keybindings, requiring human review. You can add or adjust custom eligibility rules. Learn more. |
Stacked on #12599. Its shortcut opens only the last queued message. This follow-up lets Option/Alt+Up walk backward through the queue and Option/Alt+Down walk forward to the original composer draft.
Unsaved text, attachment, and context changes require Save or Cancel before switching. Keyboard navigation never saves automatically, and clicking another message's pencil uses the same guard. Navigation follows current queue order and leaves recovery of a message that starts or disappears to the existing edit flow. Both shortcuts are configurable.
The base branch
stack/pr-12599-baseis an exact snapshot of #12599 at8ff7cb4b2095880866469c433951c33a2247297d, needed because that PR's head is on a contributor fork. Merge #12599 first, then retarget this PR to its destination and rerun required checks. Do not merge into the snapshot branch.Validation: 172 focused tests pass, web typecheck passes, and targeted lint passes with existing warnings. Real Linux Chromium keyboard/mouse verification covers backward/forward navigation, queue boundaries, draft restoration, save/cancel, empty edits, collapsed queue, saved attachment removal, custom Next binding, key-repeat handling, single/empty queues, and removal of neighboring/current messages from another tab. Reordering is covered by the navigation unit tests, not an integrated drag test. CI passes at
02b518717f. Macroscope correctness, UI consistency, and Effect checks pass; approvability requires human review because this adds a default shortcut.Scope: shared web/desktop UI. Tests used held queue fixtures in an isolated database; successful provider execution was not verified. Native mobile keyboard navigation is not implemented. macOS, Windows, Electron shell, and relay/tunnel connections have not been exercised.
Full screen recording. The GIF is a cropped version at the original playback speed.
Model: GPT-6. Harness: Codex.