cmux-next: clear two base reds in cmux-next checks (Home shortcut open, cmux host dir)
Closed
TE
18d ago+2+2+0123456789-1-1-0123456789
Two guards in "cmux-next checks" fail on the feat-cmux-next tip (41cd51f8), so every PR inherits them:
- check-concurrency:
HomeHostView.swift:159(79b0ab2d, first-run Switching card) opens the Keyboard Shortcuts settings URL with a synchronousNSWorkspace.shared.openon the main thread. It now usesopenInBackground, the sanctioned wrapper (cx-050n). - check-state-roots:
cmux-tui/src/remote_host/config.rs:74(cmux host) defaults to<home>/.cmux/host. That is one listener config per machine, not a per-build root, andserve --dirandCMUX_HOST_DIRalready override it. I added it as a reviewedkeeprow. @lawrencecchen, if the host should be per build, the row becomes amove.
Evidence: both scripts fail at 41cd51f8 and pass on this head. The third red on the tip, the error journal's try! (check-crash-safety), and the stale ci-target-graph.json are fixed in #22021.
Changelog: none
🤖 Generated with Claude Code
Need help on this PR? Tag @codesmith-bot with what you need.
Summary by cubic
Clears two failures in the cmux-next checks that redline every PR built on the feat-cmux-next tip.
- The Keyboard Shortcuts settings URL is now opened with
openInBackgroundinstead of a synchronousNSWorkspace.openon the main thread, satisfying check-concurrency. ~/.cmux/hostis now a reviewed keep in the state-roots baseline: it is a machine-wide cmux host config (one listener per machine), not a per-build root, andserve --dirandCMUX_HOST_DIRalready override it.