Skip to content

cmux-next sidebar: Priority row's label no longer overrides NSView.tag (base compile red from #22778) - #22853

Merged
teamleaderleo merged 3 commits into
feat-cmux-nextfrom
cmux/priority-row-tag-red
Oct 21, 2026
Merged

teamleaderleo merged 3 commits into
feat-cmux-nextfrom
cmux/priority-row-tag-red

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 21, 2026 •

Copy link
Copy Markdown
Collaborator

What changed

SidebarPriorityRowView (#22778) declared private let tag: ChromeLabel, which overrides NSView.tag: Int. CmuxNextSidebar doesn't compile at feat-cmux-next bfa73a4, so neither does the app:

SidebarPriorityRowView.swift:12:17: error: overriding property must be as accessible as its enclosing type
SidebarPriorityRowView.swift:12:17: error: property 'tag' with type 'ChromeLabel' cannot override a property with type 'Int'

The label (a chat's project or machine) is renamed placeLabel. With that fixed, the app target fails next: AppActions.swift:51: cannot find 'SidebarPriorityFeed' in scope. The type lives in CmuxNextAgentLanes, which AppActions.swift didn't import, so the second commit adds the import. The CmuxNextApp size cap refuses any new source line, so the third commit joins the two-line Logger(...).fault(...) call in AppActions.swift into one line. Nothing else changes.

Changelog

none

Observed / Not run

  • Observed: fleet cmux-ci compile-check --app fails at dc3bb42 (a merge of base 3a240e4) with the errors above, in job c8edb1f5.
  • Observed: compile-check --app at d4b22cb (the rename alone) fails on the missing import (job 4d577874). At the head 5fa7538 it passes (job 52e968e4).
  • Observed: at 5fa7538, CI's CmuxNextApp size cap failed on the one-line growth. At 991a42c, check-app-size.sh --base bfa73a4 passes locally (no net new line), and prepush-gate.py reports no new violations.
  • Pending: compile-check --app at 991a42c.
  • Not run: Swift tests. Nothing in this PR changes behavior.

Risk: low (a compile fix; it renames one private property).

Agent session: cosmic-gray

🤖 Generated with Claude Code

… NSView.tag (base compile red)

SidebarPriorityRowView declared `private let tag: ChromeLabel`, which
overrides NSView's `tag: Int`; the module fails to compile ("overriding
property must be as accessible as its enclosing type", "property 'tag'
with type 'ChromeLabel' cannot override a property with type 'Int'"),
so feat-cmux-next does not build since #22778. Rename it placeLabel.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Agent-Session: 01a15ba2-c7ad-7850-9e71-5b51f1c9ed9f
@cursor

cursor Bot commented Oct 21, 2026

Copy link
Copy Markdown

Bugbot needs on-demand usage enabled

Bugbot uses usage-based billing for this team and requires on-demand usage to be enabled.

A team admin can enable on-demand usage in the Cursor dashboard.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

owner: 01a15ba2-c7ad-7850-9e71-5b51f1c9ed9f

@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

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

Review 5fa7538: 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=pr22853 -f ref=cmux/priority-row-tag-red -f platform=macos -f backend_mode=local

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

CI passes on 991a42cfb7 (run 39539397343 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.

web and size checks: green

@blacksmith-sh

blacksmith-sh Bot commented Oct 21, 2026

Copy link
Copy Markdown

Found 1 test failure on Blacksmith runners:

Failure

Test View Logs
test_the_repository_highlights_validate (main.DigestTests.test_the_repository_highl
ights_validate)/
test_the_repository_highlights_validate (main.DigestTests.test_the_repository_highl
ights_validate)
View Logs

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

…eed (base compile red)

AppActions.bind calls SidebarPriorityFeed.bind (#22778), a type of
CmuxNextAgentLanes, without importing it: "cannot find
'SidebarPriorityFeed' in scope".

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Agent-Session: 01a15ba2-c7ad-7850-9e71-5b51f1c9ed9f
@cursor

cursor Bot commented Oct 21, 2026

Copy link
Copy Markdown

Bugbot needs on-demand usage enabled

Bugbot uses usage-based billing for this team and requires on-demand usage to be enabled.

A team admin can enable on-demand usage in the Cursor dashboard.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Bead cx-2cjy6 created for this PR by the cmux coordinator.

The CmuxNextApp size cap refuses any source-line growth, and the import
added one line. The Logger(...).fault(...) call becomes one line, so the
fix adds no line to the app target.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Agent-Session: 01a15ba2-c7ad-7850-9e71-5b51f1c9ed9f
teamleaderleo added a commit that referenced this pull request Oct 21, 2026
@teamleaderleo
teamleaderleo merged commit da0403c into feat-cmux-next Oct 21, 2026
@teamleaderleo
teamleaderleo deleted the cmux/priority-row-tag-red branch October 21, 2026 03:52
teamleaderleo added a commit that referenced this pull request Oct 21, 2026
teamleaderleo added a commit that referenced this pull request Oct 21, 2026
teamleaderleo added a commit that referenced this pull request Oct 21, 2026
…w-reel

Agent-Session: 01a15ba2-c7ad-7850-9e71-5b51f1c9ed9f
teamleaderleo added a commit that referenced this pull request Oct 21, 2026
Picks up the base fixes (#22877 inspector page, #22853 compile, #22856, #22876).
settings.gallery.ts: kept both new covers entries (ChiefPetCard, FontPicker).
ci-target-graph.json and the settings exports take the base side; PR autofix regenerates them.

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

Copy link
Copy Markdown
Contributor

Shipped in:

@github-actions

Copy link
Copy Markdown
Contributor

Shipped in:

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