Repository navigation
feat(client): pair other Computers from the browser and add All/Computer tabs to the channel list - #2505
Merged
Conversation
… All/Computer tabs A Host's browser pairs another Computer through that Computer's device gateway (cross-origin pairing plus a one-time stream ticket), lists it as a tab beside All and This computer, and chats with its Bots and channels in the Messages panel. Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 52 minutes. View limit details
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
This was referenced Oct 21, 2026
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.
Requested by Vain · project thread
Why the change
The web channel list only showed this computer, so a person with BotHarness on two machines could not reach the second one from the browser / 网页频道列表只能看到本机,无法从浏览器接入另一台电脑。
Issue: Refs #2500 (web part of the unification; mobile tabs live in #2499)
Special things to note
pair,unpair,stream-ticketand/api/*. The browser opens the Remote mux with a one-time, 30 s ticket bound to the device and stored as a hash. Review focus:packages/core/src/devices/gateway.ts. Reverting this PR removes the browser path and leaves mobile pairing untouched. ADR-0242 records the decision / 网关新增 CORS 与一次性流票据,评审重点在 gateway.ts;回退即可撤掉网页路径,不影响手机。apps/mobile/src/host/*client, conversation and pairing modules through a browser carrier.device-gateway.test.ts(2 new tests: cross-origin pair/call, cookie refused,/web/refused, ticket single use, evil origin and revoked device refused),computer-sources.test.ts(5 new),bot-sidebar.test.tsgreen. CI-selected related run: 330 files pass;workspace-grant-roundtrip.test.tsfails the same way on main (node:sqlite bundling).pnpm typecheck,pnpm lint(incl. CLI parity) andpnpm format:checkpass.laptop,studio-mac). The laptop browser paired studio-mac from a pairing link, opened Ada on studio-mac, sent a message, and Ada replied live through the ticketed mux withdeepseek-v4-pro.affected. No new config file or test directory.configurable: falseexemption group inscripts/cli-ui-inventory.json; pairing itself already hasdeepseekbot devicescommands.Change outline
+to add one. Right-click a computer tab to remove it (also revokes this browser there).Generated by Claude Code