Skip to content

Use semantic colours for ACP Inspector errors and menu fallbacks - #27613

Merged
teamleaderleo merged 2 commits into
feat-cmux-nextfrom
lane/aiforui-semantic-colours-1029
Oct 29, 2026
Merged

teamleaderleo merged 2 commits into
feat-cmux-nextfrom
lane/aiforui-semantic-colours-1029

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

ACP Inspector errors now follow the shared failure palette independently of the diff colour setting. Its hover fills use the required base theme token, and destructive menu items use host or agent danger colours with a readable foreground fallback before a theme arrives.

Leo (2026-10-29): “I think you guys should just keep going indefinitely. Like just, there's so much random nonsense that you can probably associate every single one of his pointers to one of our situations.” Earlier: “https://aiforui.dev/learn ... you guys should be taking this course and then reading into how it goes.”

Course item 14: use semantic colours so light, dark and custom themes share the same status meaning.

File:line Rule Before After
webviews/src/agent-session/acpmux/Inspector.css:50,153 Semantic error status Diff deletion palette plus a local hex fallback Shared failure palette
webviews/src/agent-session/acpmux/Inspector.css:84,120 Semantic hover Shared token plus duplicate white RGBA fallbacks Shared control-hover token
webviews/src/ui/ui.css:122 Semantic destructive action Unthemed foreground mixed with a local red literal Host danger, agent danger, then semantic foreground

Overlap checked across 667 open PRs: #27343 touches a disjoint toolbar hunk in ui.css; Inspector.css has no overlap.

Risk: Inspector error colour now follows status rather than diff preferences. There are no new timers or animations.

Observed / Not run

Observed: full webviews lint/type/protocol checks and static pre-push gate pass. Regression and gallery fixture were committed before the fix at 9f280fd.

Remote Chromium red/green validation passed at d25d1a4 (run): all six light/dark tests pass on head. Restoring the original production CSS produces four expected failures (status palette and unthemed destructive colour in both themes); both hover-preservation tests pass on both versions. Before/after captures, visual UI review and independent review are pending.

Changelog

Fixed: ACP Inspector errors follow the theme's failure colour independently of diff colours.

Agent session: agent session

@github-actions github-actions Bot added the risk:normal Risk tier (docs/ci/risk-tiers.md) label Oct 29, 2026
@blacksmith-sh

blacksmith-sh Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Found 25 test failures on Blacksmith runners:

Failures

Test View Logs
/tmp/tmp2c5owuqw/summary-fail.json (live):/summary-fail.json (live): View Logs
/tmp/tmpbb_1trwo/summary-missing.json (live):/summary-missing.json (live): View Logs
[new_workspaces] debug.new_workspace timed 0 of 50 new workspace rows/
[new_workspaces] debug.new_workspace timed 0 of 50 new workspace rows
View Logs
[new_workspaces] the window draws no frames/[new_workspaces] the window draws no frames View Logs
[tabs] debug.tab_switch timed 0 of 40 switches/
[tabs] debug.tab_switch timed 0 of 40 switches
View Logs
[tabs] the window draws no frames/[tabs] the window draws no frames View Logs
src/pages/settings/chiefLook.test.tsx/chiefLook.test.tsx View Logs
src/pages/settings/chiefLook.test.tsx/chiefLook.test.tsx View Logs
src/pages/settings/themePane.test.tsx/themePane.test.tsx View Logs
src/pages/settings/themePane.test.tsx/themePane.test.tsx View Logs

...and 15 more test failures. View all on Blacksmith: run 40958674138, run 40958674065

Fix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@teamleaderleo
teamleaderleo marked this pull request as ready for review October 29, 2026 21:43
@github-actions

github-actions Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

cmux status (one comment, edited in place; each line is updated by the job that owns it)

Review d25d1a4: HQ feed card · gallery before/after runs · checks

App of this head (app.zip, kept 1 day)

gh workflow run reload-build.yml -R manaflow-ai/cmux --ref feat-cmux-next -f tag=pr27613 -f ref=lane/aiforui-semantic-colours-1029 -f platform=macos -f backend_mode=local

Unclear: web / react-apps-check failed: no known signature; failed step: Check webview test shards.

CI failed on d25d1a4aea (run 40958674138 attempt 1): 1 code, 1 unknown.

Job Verdict Why
web / react-apps-base-check (2) code a web test failed
web / react-apps-check unknown no known signature; failed step: Check webview test shards

Failures:

  • seen on other PRs chiefLook.test.tsx search renders Bubble Style in the Look card: failed (job)
  • seen on other PRs themePane.test.tsx the Theme page draws each theme key once, in one card, with no second App Theme group: failed (job)
  • seen on other PRs bare-ref-repo.test.tsx an agent's message: its own address names the repository of its bare #N: failed (job)
  • seen on other PRs classic-new-tab-trust.test.tsx a start refused after a while keeps the prompt ahead of what was typed in the composer meanwhile: failed (job)
  • seen on other PRs clickable-cursor.test.ts no stylesheet sets the arrow or auto on a clickable element: failed (job)
  • seen on other PRs markdown-table-obsidian.test.ts markdown tables (webkit) > a drag across cells selects text from the press to the pointer, not whole...: failed (job)
  • seen on other PRs motion-scale.test.ts motion scale > no webview CSS, TS or TSX animates off the scale: failed (job)
  • seen on other PRs reasoning-own-control.test.ts New Tab at 760 px: the model button opens its menu, whose Effort row lists live levels: failed (job)
Matched log lines
web / react-apps-base-check (2): (fail) search renders Bubble Style in the Look card [4014.81ms]

Not re-run automatically: web / react-apps-base-check (2), web / react-apps-check are not machine failures.

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.

red: cmux-next checks (god files, concurrency, crash safety, l10n); base red: the same failure on the base 4d84096; does not block

Gallery before/after for d25d1a4: 16 states rendered, 4 images differ, 1938 more touched states not rendered. Images are in the run's artifact, not on this public PR.

gh run download 40960214528 -R manaflow-ai/cmux -n gallery-advisory-27613-d25d1a4aea59d4015cd9b07617b1f1e5fb95dce0 (then read gallery-review.json)

States
State Theme Status Diff Picked by
agent-pane.inspector/chat-menu Apple System Colors Light unchanged 0% own Inspector.css
agent-pane.inspector/chat-menu Apple System Colors unchanged 0% own Inspector.css
agent-pane.inspector/wire-log Apple System Colors changed 0.0306% own Inspector.css
agent-pane.inspector/wire-log Apple System Colors Light changed 0.0211% own Inspector.css
agent-pane.inspector-colours/errors Apple System Colors Light new 0% own Inspector.css
agent-pane.inspector-colours/errors Apple System Colors new 0% own Inspector.css
pages.agent-activity/empty Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/empty Apple System Colors unchanged 0% own ui.css
pages.agent-activity/split Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/split Apple System Colors unchanged 0% own ui.css
pages.agent-activity/split-earlier-frame Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/split-earlier-frame Apple System Colors unchanged 0% own ui.css
pages.agent-activity/split-no-frames Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/split-no-frames Apple System Colors unchanged 0% own ui.css
pages.agent-activity/timeline Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/timeline Apple System Colors unchanged 0% own ui.css
pages.agent-activity/grid Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/grid Apple System Colors unchanged 0% own ui.css
pages.agent-activity/grid-viewer Apple System Colors Light unchanged 0% own ui.css
pages.agent-activity/grid-viewer Apple System Colors unchanged 0% own ui.css
pages.home-channels/normal Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/normal Apple System Colors unchanged 0% own ui.css
pages.home-channels/channel Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/channel Apple System Colors unchanged 0% own ui.css
pages.home-channels/thread Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/thread Apple System Colors unchanged 0% own ui.css
pages.home-channels/thread-long Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/thread-long Apple System Colors unchanged 0% own ui.css
pages.home-channels/switcher Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/switcher Apple System Colors unchanged 0% own ui.css
pages.home-channels/switcher-filtered Apple System Colors Light unchanged 0% own ui.css
pages.home-channels/switcher-filtered Apple System Colors unchanged 0% own ui.css

Pick other states with a Gallery: entry.id, entry.id/variant line in the PR body.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Review (codex-review-box/gpt-6-astra high, lane-place): SHIP at d25d1a4

SHIP

No findings.

@cmux-bot cmux-bot 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.

cmux-bot approves d25d1a4: an independent review passed this exact head (review box review-27613-semantic-colours receipt by teamleaderleo). A push needs a new review; a later BLOCK at this head dismisses this approval.

@teamleaderleo
teamleaderleo merged commit 6f9161c into feat-cmux-next Oct 29, 2026
190 of 200 checks passed
@teamleaderleo
teamleaderleo deleted the lane/aiforui-semantic-colours-1029 branch October 29, 2026 22:45
@github-actions

github-actions Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
Contributor

This branch was previously deployed

1 inactive deployment
ci-routing-agent — d25d1a4a Deployed Oct 29, 2026 by teamleaderleo via wait for same-key tree publisher #78420
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk:normal Risk tier (docs/ci/risk-tiers.md)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant