Skip to content

cmux-next: New Tab tabs draw the new-tab icon, not the agent chat bubble - #17630

Merged
teamleaderleo merged 7 commits into
feat-cmux-nextfrom
op-next-icons-8
Oct 7, 2026
Merged

teamleaderleo merged 7 commits into
feat-cmux-nextfrom
op-next-icons-8

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 6, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

Fixes item 15 of the cmux-next sweep (cmuxterm-hq#1421): Agent tabs and New Tab tabs shared the generic chat bubble. Both ride a conversation record, and TabItemMapping gave every conversation tab agent.chat.

The pane already knows which tabs are still on the New Tab page (AgentTabs.pageTabs), since it titles them "New Tab". It now passes that to the mapping, and those tabs draw the registry's tab.new.

Tab Was Now
New Tab page agent.chat tab.new
Agent chat with a known harness harness mark harness mark (unchanged)
Agent chat without a harness mark agent.chat agent.chat (unchanged)

Converted call sites:

  • TabItemMapping.item(_:fallbackTitle:isNewTabPage:): new isNewTabPage flag, default false.
  • PaneController.snapshot(): computes isNewTabPage once and uses it for both the title and the icon.

When a New Tab page becomes a chat, its title and icon change in the same snapshot.

Not changed:

  • Workspace-row kind icons. They are pending Leo's call.
  • The sidebar's tab list. It doesn't know which tabs are New Tab pages (it also titles them from the record), so it keeps agent.chat for them. Follow-up: give SidebarTabKind a new-tab case once the sidebar gets that state.

Testing

Red commit d6ea400 adds ConversationTabItemTests.aNewTabPageTabDrawsTheNewTabIcon; the fix follows. The same conversation record maps to tab.new as a New Tab page and to agent.chat as a chat.

Changelog

Fixed: New Tab tabs show a new-tab icon instead of the agent chat bubble

Demo Video

Fleet builds of base (ff267c4) and the fix, on a capture mini (1x display), opening a workspace's first New Tab.

New Tab tab at 6x, before (top) and after (bottom):

tab zoom

Before After
before after

Checklist

  • Behavior changes have added or updated tests, or Testing says why not
  • UI change with no new or changed user-facing strings; localization not affected

🤖 Generated with Claude Code

teamleaderleo and others added 2 commits October 6, 2026 12:14
…ubble

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A tab still on the New Tab page and an Agent chat tab both rode a
conversation record and drew the agent chat bubble. The pane knows which
tabs are New Tab pages; it now tells the mapping, which draws tab.new.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@cursor

cursor Bot commented Oct 6, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@coderabbitai

coderabbitai Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 53fa86f5-d0c2-45c2-b7b9-e9d0ef2d12a8

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · 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.

@github-actions

github-actions Bot commented Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Not verified: macOS jobs did not run: compile and app tests were skipped on 9be9a2eae5; ci-status is green without them.

CI passes on 9be9a2eae5 (run 37549559618 attempt 1).
macOS jobs did not run: compile and app tests were skipped: Fast static checks skipped (the macos job needs both to succeed). Re-run CI once they pass, before merging.

Written by scripts/ci/classify_failures.py (ci-failure-attribution.yml); signatures are its SIGNATURES table. A machine verdict is the runner's fault, not this PR's; yours means the failing file is one this PR changes, also red on main that main's latest full suite fails the same way, seen on other PRs that it failed on another pull request's run lately.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Batch queue: green. Batch stack 1a07d268cf62 on feat-cmux-next 423bd8a882c8 with #17584 #17602 #17609 #17611 #17625 #17630 #17633. Fleet production build: job 07b1c1720d6973c1d000950b (cmux-ci artifact 07b1c1720d6973c1d000950b (job 07b1c1720d6973c1d000950b, tag nb-local-20261006-192713-1)). heavy tier did not run on this branch (run); your PR's own checks are the tests. Validated this PR at 4527a78193a3.

Next: land it yourself with gh-merge-green on this receipt.

@cursor

cursor Bot commented Oct 6, 2026

Copy link
Copy Markdown

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Batch queue: green. Batch stack 302099401834 on feat-cmux-next ebac81f0c771 with #17609 #17620 #17630. Fleet production build: job 5283f103448a81da175c7814 (cmux-ci artifact 5283f103448a81da175c7814 (job 5283f103448a81da175c7814, tag nb-local-20261006-223112-1)). heavy tier did not run on this branch (run); your PR's own checks are the tests. Validated this PR at 4171758ff58e.

Next: land it yourself with gh-merge-green on this receipt.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Batch queue: green. Batch stack b2ae9b9ef0f0 on feat-cmux-next 044508d44ee8 with #17609 #17611 #17620 #17623 #17630 #17633 #17635 #17636 #17648. Fleet production build: job d26b5daf716cd175c2f38aa0 (cmux-ci artifact d26b5daf716cd175c2f38aa0 (job d26b5daf716cd175c2f38aa0, tag nb-local-20261006-225524-1)). heavy tier did not run on this branch (run); your PR's own checks are the tests. Validated this PR at fd235a9e615c.

Next: land it yourself with gh-merge-green on this receipt.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Batch queue: green. Batch stack c2b7744fd7cc on feat-cmux-next ac66021d2193 with #17551 #17601 #17620 #17623 #17630 #17635 #17643 #17644. Fleet production build: job 267908eced53b1284e673af5 (cmux-ci artifact 267908eced53b1284e673af5 (job 267908eced53b1284e673af5, tag nb-local-20261007-000439-1)). heavy tier did not run on this branch (run); your PR's own checks are the tests. Validated this PR at 9be9a2eae561.

Next: land it yourself with gh-merge-green on this receipt.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

merge-override: Base red: StaticSidebarToggleTests.withTheSidebarHiddenTheStripStartsAfterTheToggle fails identically on #17620/#17630/#17635 (unrelated surfaces), own tests pass, other reds are runner refusals; owner approved override

@teamleaderleo
teamleaderleo merged commit db1a87d into feat-cmux-next Oct 7, 2026
61 of 74 checks passed
@teamleaderleo
teamleaderleo deleted the op-next-icons-8 branch October 7, 2026 01:02
@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

The feat-cmux-next push run https://github.com/manaflow-ai/cmux/actions/runs/37555112268 at 2df79b5 failed: cmux-next checks (god files, concurrency, crash safety, l10n).
Those jobs last passed at d7167cb. This pull request is one of 3 merged since: #18115, #17630, #17635.
If the failure is in your change, fix forward on feat-cmux-next. Pull requests run only the tiers their paths reach (docs/ci/cmux-next-tiers.md); label a batch PR full-ci to run them all before merging.

@github-actions

github-actions Bot commented Oct 7, 2026

Copy link
Copy Markdown
Contributor

The feat-cmux-next push run https://github.com/manaflow-ai/cmux/actions/runs/37555118221 at ea3879d failed: cmux-next checks (god files, concurrency, crash safety, l10n).
Those jobs last passed at d7167cb. This pull request is one of 4 merged since: #18115, #17630, #17635, #17535.
If the failure is in your change, fix forward on feat-cmux-next. Pull requests run only the tiers their paths reach (docs/ci/cmux-next-tiers.md); label a batch PR full-ci to run them all before merging.

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