[teamleaderleo]

Fixes the base's red fast tier. #24002 (cc027d8) added HomeChiefLinkInkTests with #selector(NSView.display). AppKit has both display() and display(_:), so the selector is ambiguous and CmuxNextAppTests no longer builds. Every PR's cmux-next fast tier has been red since, for example jobs 119640111188, 119641860268 and 119642938745 at 4f51657.

Change

One line: #selector(NSView.display as (NSView) -> () -> Void). Only the test changes; no app code.

Tests

• cmux-ci run --class light --script scripts/ci/package-test-lane.sh --ref a88c965e4f52b14b92dfc4553e4333a01c6a808b --arg=suite --arg=Packages/macOS/CmuxNext --arg=HomeChiefLinkInkTests builds CmuxNextAppTests and runs the test. The result goes in a comment.
• Red on base: the fast tier jobs above fail at HomeChiefLinkInkTests.swift:23:116: error: ambiguous use of 'display'.

Observed / Not run

Observed: the base's fast-tier errors. Fleet results are pending at the time of writing. Not run: nothing else; this is test-only.

Changelog

none

No capture: test-only change.

Agent session: cosmic-gray

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith-bot with what you need.}

---

Summary by cubic

Fixes the red cmux-next fast tier by disambiguating #selector(NSView.display) in HomeChiefLinkInkTests. AppKit has both display() and display(_:), so the bare selector is ambiguous and CmuxNextAppTests no longer compiles — every PR's fast tier has been red since #24002.

The fix casts the selector as (NSView) -> () -> Void. Test-only change; no app code is modified.

^{Written for commit 42c6b04. Summary will update on new commits.}

[cursor]

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.

[github-actions]

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

Review 42c6b04: 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=pr24219 -f ref=chief-ui-fix-link-ink -f platform=macos -f backend_mode=local

red: cmux-next checks (god files, concurrency, crash safety, l10n); NEW: red check: Checks job structure (+1 more) is not red on the base 1aee113; do not merge until green; see lane-rules

[teamleaderleo]

Owned by chief-ui on cosmic-gray · session

[blacksmith-sh]

Found 12 test failures on Blacksmith runners:

Failures

Test	View Logs
/tmp/tmp0ona9nal/summary-fail.json (live):/summary-fail.json (live):	View Logs
/tmp/tmpmu2wsl_a/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
convention violations found (ERROR lines above)./ convention violations found (ERROR lines above).	View Logs
nightly cmux-tui resolve/nightly cmux-tui resolve	View Logs
test_every_submodule_free_checkout_on_an_owned_runner_resets_submodules (main.Reuse dWorkspaceSubmodules.test_every_submodule_free_checkout_on_an_owned_runner_resets_submo dules)/ test_every_submodule_free_checkout_on_an_owned_runner_resets_submodules (main.Reuse dWorkspaceSubmodules.test_every_submodule_free_checkout_on_an_owned_runner_resets_submo dules)	View Logs
test_every_submodule_free_checkout_on_an_owned_runner_resets_submodules (main.Reuse dWorkspaceSubmodules.test_every_submodule_free_checkout_on_an_owned_runner_resets_submo dules) (job='web-bundles')/ test_every_submodule_free_checkout_on_an_owned_runner_resets_submodules (main.Reuse dWorkspaceSubmodules.test_every_submodule_free_checkout_on_an_owned_runner_resets_submo dules) (job='web-bundles')	View Logs

...and 2 more test failures. View all on Blacksmith: run 39729550499, run 39729550523

^{Need help on this PR? Tag @codesmith-bot with what you need.}

[teamleaderleo]

Review (claude-reviewer-24219, sonnet): SHIP at a88c965

Read-only review of the one-line change at HomeChiefLinkInkTests.swift:23. I did not build or run it.

1. Valid Swift: yes. #selector(Type.method as (Type) -> (Args) -> Ret) is the documented way to pick one overload. NSView's display() is an instance method with no arguments and a Void return, so its unapplied reference has type (NSView) -> () -> Void. The other overload, display(_ rect: NSRect), has type (NSView) -> (NSRect) -> Void, so the cast excludes it.
2. Selector: it resolves to display. The display(_:) overload is displayRect:, and the cast removes it.
3. Line 25: those two errors cascade from line 23. link had no resolved type, so link.attributedTitle had no base type and the .foregroundColor and nil arguments had no contextual type. Once line 23 type-checks, link is HomeChiefLinkButton, an NSButton subclass. attributedTitle is then NSAttributedString, .foregroundColor infers as NSAttributedString.Key, and effectiveRange: nil infers as NSRangePointer?. Line 25 has nothing else that needs a fix.
4. Other ambiguity: none. #selector appears once in the file, and the HomeChiefLinkButton.init(title:target:action: Selector) signature accepts the result. The rest of the file (the ThemeScope, Palette and ThemeTokens calls, and #expect lines 28-32) is unchanged, and none of it is among the reported errors.

The fix is minimal and test-only. It touches no app source, so there is no runtime risk. The only unverified part is that CI has not yet compiled it, so the CmuxNextAppTests build on this head is the final confirmation.

[teamleaderleo]

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

[teamleaderleo]

Review (claude-reviewer-24219, sonnet): SHIP at 42c6b04

Read-only re-check of the base merge. I did not build or run anything.

• 42c6b04 is Merge branch 'feat-cmux-next' into chief-ui-fix-link-ink, with parents a88c965 (the previously reviewed head) and 1aee113 (base).
• git diff --stat 1aee1139 42c6b046 lists one file, HomeChiefLinkInkTests.swift, 1 insertion and 1 deletion. The change is exactly the line-23 swap from #selector(NSView.display) to #selector(NSView.display as (NSView) -> () -> Void). The merge adds nothing beyond base.
• git diff a88c965e 42c6b046 for that file is empty, so the file is identical to the version I reviewed in my earlier SHIP comment.
• Base at 1aee113 still has the ambiguous #selector(NSView.display) on line 23, so this line is still the fix.

My earlier analysis stands: the cast selects display() (selector display), the line-25 errors cascade from line 23, and the file has no other #selector. CI's CmuxNextAppTests build on this head is the remaining confirmation.

[github-actions]

Shipped in:

• cmux NEXT nightly 1.0.0-nightly.3973389583201 (48a69e9f66)