Make browser controls visible in chat and the Computer sidebar - #646
Merged
Merged
Conversation
davidmckayv
requested review from
MikeRyanDev,
guidovizoso,
mxmzb and
tylerslaton
as code owners
September 26, 2026 23:58
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
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.
Computer access was hard to find: channel chat used an icon-only toggle, standalone Bot chat had no live sidebar, and takeover controls appeared only in the expanded viewer or a pending-help prompt.
Expose a labeled Computer toggle and state-aware Take control / Hand back controls in chat and in the computer sidebar. Reuse the existing request-aware ownership protocol and live viewer so every surface reflects the same control state, including loading and failures. Standalone Bot chat retains its selected Bot, history, and New chat behavior while gaining live computer access.
Validation:
Test limitation: a new simulated narrow-sheet close case triggered a Bun native SIGTRAP, so the standalone route regression uses explicit desktop geometry. Existing narrow DetailPanel tests and the full app suite pass. No mobile hands-on result is claimed.