[teamleaderleo]

Leo (2026-10-29): "could u guys also now work on the per space pins and groups? rn anything u make in teh default space goes across to new ones... there's also other ergo wrt new windows being like new spaces, incognito spaces, etc... this could legit make our thing more useful than chrome by having windows be switchable in this way. think of the potential perf savings~"

This PR covers the windows part of that ask. Per-space pins and groups are not in it.

What changed

• space.bringHere "Move Space to This Window": the reverse of Open Space in New Window. Every workspace of the space that another normal window holds (open or closed) moves into the acting window. That window then shows the space and selects the workspace the space last showed (the one a window showing the space had selected, else the one remembered for the space). If no other window holds any of the space's workspaces, it is a plain space switch. Workspaces only change owner, so terminals keep running and nothing restarts. A source window left with no workspace closes through the existing WindowRegistry.move / closeEmptied path. A source window that still holds other spaces' workspaces stays open. Moving an incognito space, or moving into an incognito window, is refused with the existing RefusalStrings.incognitoMismatch HUD. Surfaces: the palette, the space's right-click menu ("Move to This Window", next to Open in New Window), cmux space bring-here, and MCP (spaces-overhaul-DESIGN.md rule 7).
• window.mergeAll "Merge All Windows": every other open normal window's workspaces move into the acting window, and each workspace keeps its space. The other windows close. The acting window stays on its space with the same selection. For spaces it never showed, it takes each merged window's last shown workspace, so switching there shows what that window showed. Incognito windows are left alone, and running it from an incognito window is refused. Surfaces: the Window menu (after New Incognito Window), the palette, cmux app merge-all-windows, and MCP. It has a context-menu exemption, as Close Other Windows does.
• Titles are in ProfileActions.xcstrings and ActionCatalog.xcstrings (en and ja translated; the other locales are needs_review, as in neighbouring entries).
• plans/cmux-next/action-surfaces.json and actions.md are left to the base autofix (regenerate-action-exports.sh).

Risk

• Additive. No existing default, component or menu row is changed or removed.
• The Window menu gains one row, so MainMenuPlacementTests now expects 10 Window rows instead of 9.
• Both actions run one registry transition (transition { $0.move(...) }), the same path switchProfile adoption and Move to Window use. Incognito windows are filtered out of the sources before the move, so crossesIncognito never drops the batch.

Tests

• New GatherWindowsTests (.closesItsWindows):
  • Bring-here closes the window it emptied and keeps the space's last shown workspace selected.
  • Only that space's workspaces move, and a source window that keeps other spaces stays open.
  • Incognito space or window is refused and nothing changes.
  • Merge-all keeps each workspace's space, closes the merged windows and leaves the incognito window alone.
• SpaceSurfacesTests names space.bringHere and checks it is in the space menu.
• Focused fleet Mac run (package-test-lane.sh suite GatherWindowsTests, SpaceSurfacesTests, MainMenuPlacementTests, ActionSurfaceParityTests, ShowWorkspaceOtherRoomTests, IncognitoSpaceTests): all 6 suites passed (ActionSurfaceParityTests: 2 export-freshness tests skipped as usual). cmux-ci step 0ca0e2da7b669522ed80b23f, at 62fed54: this head (69b56f9) with test(next): BrowserEntryRetirementTests compiles again (base red from #27166) #27301 merged in, on a CI-only branch. At this head alone, CmuxNextAppTests does not compile because of the base red from fix(next): retire browser pages before chrome teardown #27166 (BrowserEntryRetirementTests.swift:43:57, "returning main actor-isolated ... as a 'sending' result"; step c559ae7de547b95c91d859b5). This PR does not touch that file, and test(next): BrowserEntryRetirementTests compiles again (base red from #27166) #27301 fixes it.

No capture: the only visible change is new rows in the Window menu, the space's right-click menu and the command palette, and the main window is unchanged. A fleet capture (steps-6d58e8bbea at 59c45ae: open the palette, type "merge all", then "to this window") keeps one still of the main window, and the palette panel and menus draw in their own windows outside it, so the still shows nothing new. SpaceSurfacesTests and MainMenuPlacementTests pin the new rows instead.

Performance

No perf numbers: the word "perf" here is Leo's quote ("think of the potential perf savings"). This PR adds two window actions and changes no hot path, timer or observer; it measures nothing.

Agent session: agent session

🤖 Generated with Claude Code

[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 4f27413: 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=pr27327 -f ref=spaces-bring-here -f platform=macos -f backend_mode=local

red: cmux-next checks (god files, concurrency, crash safety, l10n); no finished base run on 0ce3c31 to compare; do not merge until green; see lane-rules

[teamleaderleo]

Owned by an agent session · session

[github-actions]

Default changes

This pull request changes 5 defaults at 36e595d698c0 (what everyone gets without configuring anything):

• tunable-source.CmuxNextActions/ActionSurfaceCatalog+Menus.swift:layoutPlacements: "087aba83afdc9cea1daf878ab7433f5e25c39f4d73a9decef89e8f89feddf9d3" -> "da5024a3c17acf9ede3281f1ce7a6b8a06b5364292f94cf6dfacffddec6a01af"
• tunable-source.CmuxNextActions/Catalog/SpaceMenuLabels.swift:bringHere: (none) -> "ea049e589ff08f82de59248e676cfe573dae4dbf884d3069e1d0a96c6c85db28"
• tunable-source.CmuxNextApp/Handlers/RoomHandlers.swift:bind: "af4e8c3897c0daf31dd3e700c2e7494ed5af8a9d9115b0a8fe8fe8a96e9cdcd5" -> "c431958c6656780e7234bdc99c830a66df647ae5cd907581cf6fe10d38028a32"
• tunable-source.CmuxNextApp/Handlers/WorkspaceSetVerbs.swift:bind: "e6313469de7e4319c7bf699c377a5dbf1735f928002d7fdfbba9fd3543b3795a" -> "e5d9201bbbf19a20b1caab05228aa69cf30a864c0254eddc50098600509383ab"
• tunable-source.CmuxNextApp/MainMenu.swift:compactActionIDs: "0d85ad8a7a5d2ed06c73a03da9ef6e6f8b28acb3a7eb85ce556a30b8a356435e" -> "73b640987050077767f5063b6eeba83e458b04e04f580b890000fff1d767c27b"

Signed off on this commit by review (codex gpt-6-astra high fast, independent review host).

Regenerate the snapshot with python3 scripts/ci/defaults_guard.py snapshot.

[teamleaderleo]

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

[blacksmith-sh]

Found 7 test failures on Blacksmith runners:

Failures

Test	View Logs
--from must install a complete prebuilt tree/ --from must install a complete prebuilt tree	View Logs
/tmp/tmp81yumw1b/summary-fail.json (live):/summary-fail.json (live):	View Logs
/tmp/tmpq6fsx0sl/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

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

[teamleaderleo]

Review (codex gpt-6-astra high, next-review): SHIP at 69b56f9

No blocking findings.

[cmux-bot]

cmux-bot approves 69b56f9: an independent review passed this exact head (next-review receipt by teamleaderleo). A push needs a new review; a later BLOCK at this head dismisses this approval.

[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.

A newer verdict blocks 59c45ae (no independent SHIP at the head).

[teamleaderleo]

Review (codex-review-box/gpt-6-astra high, lane-place): BLOCK at 59c45ae

BLOCK

1. [P2] WindowManager+Gather.swift:27 — space.bringHere changes the destination’s space and selected workspace even for CLI/MCP calls without --focus. The action is not marked focuses, and neither this branch nor the switchProfile fallback checks ActionRunScope.viewChangeAllowed(). Automation therefore unexpectedly replaces the user’s current view. Gate the profile switch and preferred selection on that permission, including the no-move fallback.

[teamleaderleo]

Review (codex gpt-6-astra high fast, independent review host): BLOCK at 59c45ae

• P1 — Generated action exports are stale. The new space.bringHere and window.mergeAll descriptors in ProfileActionCatalog.swift:45-50 and WindowActionCatalog.swift:49-53 are not reflected in plans/cmux-next/action-surfaces.json or the generated catalog block in plans/cmux-next/actions.md. check-action-surfaces.sh enables freshness checks, so exportIsFresh, reportIsFresh, and countsByDomainMatchInventory will fail for this PR.

VERDICT: BLOCK

[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.

[teamleaderleo]

Review (codex gpt-6-astra high fast, independent review host): SHIP at 36e595d

No actionable correctness findings in the supplied diff at 36e595d698c0.

Static review only; builds and tests were not run.

VERDICT: SHIP

[teamleaderleo]

Carried onto #28049 with its commits and authorship, plus a fix for its own cmux-next checks red (action-wait and app-size guardrails). Its review is running there. This PR's branch can't be updated by the drain lane, so the work continues there.