Skip to content

cmux-next: UI batch 1 (diff count colors, gallery fit) - #18335

Closed
teamleaderleo wants to merge 11 commits into
feat-cmux-nextfrom
next/ui-batch-1
Closed

teamleaderleo wants to merge 11 commits into
feat-cmux-nextfrom
next/ui-batch-1

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

UI batch 1: two ready web UI PRs merged once on current feat-cmux-next, so they land together with one gallery preview and one dev build.

They merged with no conflicts. On top of the merges, one commit regenerates the agent pane and page bundles with scripts/cmux-next/regenerate-bundles-remote.sh. The constituent PRs stay open until this lands; they then close as landed via this PR.

#18224 and #18277 were in the first cut and are dropped: they landed on their own as daf0d36 and 7fac2bf.

Testing

  • Each constituent's own tests come along unchanged, and this PR's hosted CI runs them together on the merged head.
  • The pane strings were regenerated with gen-strings.mjs agentPane, which produced no changes.
  • The batch's gallery preview is built at its head (link below).

Changelog

Changed: File rows in the agent pane's changes tree color additions and deletions by diff side
Changed: Component gallery entries render at their natural size

Proof

The gallery preview and the dev build link follow in comments.

🤖 Generated with Claude Code


Note

Low Risk
Presentation-only UI in the agent changes tree and gallery previews; no auth, data, or API behavior changes.

Overview
UI batch landing two web changes together on feat-cmux-next, plus regenerated agent pane/page bundles.

In the agent pane changes tree, per-file +N / -N decorations now use the same diff-side colors as the rest of the pane (--acpmux-add / --acpmux-del and related count tokens via treeUnsafeCSS and sprite SVG fills on --trees-status-added / --trees-status-deleted).

In the component gallery, component entries are no longer laid out like a 16:9 window preview: the stage uses each entry’s natural width/height (stageHeight), and fit zoom scales to that frame instead of the window pane size (aligned with compare view behavior for components).

Constituent PRs #18325 and #18321 are merged here; bundle regen is from scripts/cmux-next/regenerate-bundles-remote.sh.

Reviewed by Cursor Bugbot for commit ebf99e7. Bugbot is set up for automated code reviews on this repo. Configure here.

@teamleaderleo teamleaderleo added the dev-build Build a fleet dogfood build of each push (newest head under load) label Oct 7, 2026
@coderabbitai

coderabbitai Bot commented Oct 7, 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: 62537756-0f5f-4eab-a0b1-e287cf9895b3

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 7, 2026 •

Copy link
Copy Markdown
Contributor

Unclear: web / react-apps-check failed: no known signature; failed step: Check diff viewer eager JS budget.

CI failed on ebf99e78f9 (run 37604938390 attempt 1): 1 unknown.
macOS jobs did not run: compile and app tests were skipped: Fast static checks skipped.

Job Verdict Why
web / react-apps-check unknown no known signature; failed step: Check diff viewer eager JS budget

Not re-run automatically: web / react-apps-check is not a machine failure.

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.

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit bd8e4ac. Configure here.

teamleaderleo and others added 3 commits October 7, 2026 02:58
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@teamleaderleo teamleaderleo changed the title cmux-next: UI batch 1 (diff count colors, model catalog refresh, gallery fit) cmux-next: UI batch 1 (diff count colors, gallery fit) Oct 7, 2026
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Gallery preview at ebf99e7, built from this branch:

@cursor

cursor Bot commented Oct 7, 2026

Copy link
Copy Markdown

This PR is too large for Bugbot to review. It changes 85 lines and 4,177,163 characters. Split the change into smaller pull requests to get a review.

@teamleaderleo teamleaderleo removed the dev-build Build a fleet dogfood build of each push (newest head under load) label Oct 7, 2026
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Closing: #18224, #18277 and #18325 landed on their own, so this batch carries only #18321, which lands alone.

@teamleaderleo
teamleaderleo deleted the next/ui-batch-1 branch October 7, 2026 10:11
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.

1 participant