Skip to content

Desktop: icon-only Host strip, Add remote in the + menu; ADR-0243 drops environment tabs - #2508

Merged
DeepSeekBots merged 6 commits into
mainfrom
claude/project-thread-fst88w
Oct 21, 2026
Merged

DeepSeekBots merged 6 commits into
mainfrom
claude/project-thread-fst88w

Conversation

@DoodleBears

@DoodleBears DoodleBears commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Requested by Vain · project thread

Why the change

The #2505 text tabs ("All / This computer / …") crowd the channel list, and "computer" clashes with the PersonaBots' Computer. Vain decided to drop environment tabs: icons only on the desktop, the left drawer on the phone / 去掉文字环境 tabs,桌面端改为纯图标条,「添加远程」放进频道列表的 + 菜单。

Refs #2500. Desktop 1.5.0 is held on this change.

Special things to note

  • Desktop web
    • Icon strip: All, This host, one icon per Remote Host. Each icon has a tooltip (name, unread count, offline), an unread badge, and is greyed when offline.
    • Right-click: Change icon… (symbols, emoji, colour) or Remove (with confirm).
    • The strip is hidden until a Remote Host is paired.
    • Add remote is now an item in the channel panel header's + menu.
    • A newly paired Host gets an unused symbol and colour.
    • Looks are device-local (botharness.hostLooks.v1, exempted in cli-ui-inventory.json, per ADR-0243).
  • Rename: computer-sources/computer-tabs → remote-hosts/host-tabs; zh UI says 本机/远程. The scaffold also adds each Remote Host's inbox snapshot and an avatar loader for later tracks.
  • Docs: ADR-0243 gains parity, no full-client fallback, device-local looks, no environment tabs (drawer on phone, strip on desktop), distinct default looks.
  • Not in this PR: the mobile drawer change (removing the phone's pager). It needs native iOS/Android QA and follows separately.
  • Merge risk: web channel list header only. Revert is a plain revert.
  • Verification:
    • tsc --noEmit, pnpm lint (CLI parity, ADR numbers), oxfmt.
    • vitest: remote-hosts, host-looks (new), bot-sidebar, channel-sidebar-tabs.
    • Live: two isolated Hosts (laptop :47011, studio-mac :47012), paired through + → Add remote, checked the strip, tooltip, right-click menu, icon dialog and remote rows in light and dark.
  • CI test plan: expected affected.

Change outline

Light Dark
+ menu
Strip, All, hover
Right-click
Change icon

🤖 Generated with Claude Code

https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r

…ice-local

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
@DoodleBears DoodleBears self-assigned this Oct 21, 2026
@coderabbitai

coderabbitai Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You'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 10 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 2 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 33444c76-842f-4d0e-845e-01d74e68ef91
📥 Commits

Reviewing files that changed from the base of the PR and between 11174d7 and 187e8ad.

⛔ Files ignored due to path filters: 10
  • .github/pr-screenshots/2500-host-strip/add-dark.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/add-light.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/look-dark.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/look-light.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/menu-dark.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/menu-light.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/strip-dark.png is excluded by !**/*.png
  • .github/pr-screenshots/2500-host-strip/strip-light.png is excluded by !**/*.png
  • packages/browser/lib/client.js.map is excluded by !**/*.map
  • packages/computer/lib/client.js.map is excluded by !**/*.map
📒 Files selected for processing: 13
  • changes/2500-computer-tabs.md
  • changes/2500-host-feed.md
  • changes/2500-remote-hosts.md
  • docs/adr/0243-every-client-reads-every-host-through-one-host-feed.md
  • packages/browser/lib/client.js
  • packages/client/src/client/bot-sidebar.tsx
  • packages/client/src/client/channel-sidebar-icon.tsx
  • packages/client/src/client/host-looks.ts
  • packages/client/src/client/host-tabs.tsx
  • packages/client/src/client/locale.ts
  • packages/client/test/host-looks.test.ts
  • packages/computer/lib/client.js
  • scripts/cli-ui-inventory.json
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

…n the desktop

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
Replace the text Computer tabs with an icon strip (All, This host, one per
Remote Host). Each icon has a tooltip, unread badge and offline style, and
right-click changes its device-local look or removes the Remote Host. The
strip stays hidden until a Remote Host is paired; Add remote now lives in
the channel list's + menu. A newly paired Host gets an unused symbol and
colour. Renames Computer to Remote Host / This host per ADR-0243.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
@DoodleBears DoodleBears changed the title docs(adr-0243): Remote Host parity follows the phone; Host looks are device-local Desktop: icon-only Host strip, Add remote in the + menu; ADR-0243 drops environment tabs Oct 21, 2026
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Asbot1EHttjnKA6hQuCC6r
…-fst88w

# Conflicts:
#	packages/client/src/client/bot-sidebar.tsx
#	packages/client/src/client/host-tabs.tsx
#	packages/client/src/client/locale.ts
#	packages/client/src/client/remote-hosts.ts
#	packages/client/test/remote-hosts.test.ts
#	scripts/cli-ui-inventory.json
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.

2 participants