[teamleaderleo]

Summary

Private design-reference comparison: done.

Leo’s item D prototype is already present on feat-cmux-next through the merged multi-view work (#23437, #23651, #24067): sidebar.multiView remains off by default; mouse-down does not navigate, mouse-up on the same row does; the window remembers its visible workspace set; workspace rows use the existing pane drop overlay; and a pane-edge drop keeps the current focus.

This first commit adds the missing real-window, per-frame regression:

• hold and drag a sidebar workspace toward a split edge;
• while the drag is held, the focused workspace stays focused;
• every workspace already in the view remains highlighted (focused row strong, other visible rows soft);
• the existing pane drop overlay shows its split indicator;
• releasing adds the workspace to the current view without changing focus or dropping visible-row highlights.

Leo’s words:

“do the Claude Code desktop multi-view better”

and:

“MOUSE DOWN DRAGGING ... SHOULDN'T BE IMMEDIATELY FOCUSING”

Before / after captures

These are the existing item-D dogfood captures for the behavior this test protects:

Before (toggle off)	After (multi-view toggle on)
light · dark	light · dark

Validation

• git diff --check passed.
• Focused fleet test still required at this exact head: SidebarMultiViewDropTests / SidebarMultiViewWindowTests on a GUI-capable Mac.
• Exact-head Astra high/fast review still required at 4726064.
• No production overlay was added; the test asserts the existing LayoutRootView drop overlay.

This is a draft until the focused fleet check and exact-head independent review are recorded.

agent-identity footer --repo manaflow-ai/cmux
Agent session: agent session

---

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

---

Summary by cubic

Adds a real-window regression test proving a multi-view sidebar drag keeps the focused workspace focused and every visible row lit for the whole drag, then joins the dragged workspace into the view on drop without moving focus.

Also fixes the test's highlight assertion: litRows now counts the soft fill of a workspace on screen beside the focused one (via WorkspaceRowView.isOnScreen), which SidebarPointerGestureTests.filledRows alone missed.

^{Written for commit 8305d2b. Summary will update on new commits.}

[blacksmith-sh]

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

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

[github-actions]

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

Review 8305d2b: 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=pr27845 -f ref=lane/sidebar-visible-workspaces -f platform=macos -f backend_mode=local

web and size checks: green

Shipped in:

• cmux NEXT nightly 1.0.0-nightly.4133907849701 (c2ef233b76)
• cmux NEXT nightly 1.0.0-nightly.4133963836301 (48e4448d11)
• cmux NEXT nightly 1.0.0-nightly.4134024448601 (94322a6ccf)
• cmux NEXT nightly 1.0.0-nightly.4134131101101 (1c00c00a4b)
• cmux NEXT nightly 1.0.0-nightly.4134171502501 (a54e300843)

[teamleaderleo]

Owned by an agent session · session

[teamleaderleo]

Review (codex gpt-6.1-sol xhigh, router-review): BLOCK at 4726064

BLOCK at 4726064

Visual verdict: no images; Leo tries the build

1. [P1] [blocking] The fill assertion cannot recognize multi-view highlights — Packages/macOS/CmuxNext/Tests/CmuxNextAppTests/SidebarMultiViewDropTests.swift:41 (also lines 57 and 67). SidebarPointerGestureTests.filledRows checks only isSelected, isSecondarySelected, and isDropTarget; the visible companion row uses isOnScreen for its soft fill. Thus the first assertion sees only k0 and the new test fails before testing the split preview. Include isOnScreen in the helper or assert that property directly.

Follow-ups: none

[teamleaderleo]

Review (codex, router-review): SHIP at f385485

Reviewer: codex gpt-6-astra high, fast tier.

Follow-ups: 1. Each lands as an issue in this repo that links this receipt; the lane that opened the PR fixes it forward.

SHIP at f385485

Visual verdict: no images; Leo tries the build

1. [P2] [follow-up] No after screenshot artifact was provided (review/images/README.txt:1), so visual comparison was unavailable.

Follow-ups:

• [P2] [follow-up] No after screenshots available for visual validation.

[cmux-bot]

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

A newer verdict blocks 5f594b8 (no independent SHIP at the head).

[teamleaderleo]

Review (carry): SHIP at 5f594b8 (base merge only since f385485; patch-id unchanged)

[teamleaderleo]

Review (next-review patch-id; carried): SHIP at 7c6960b

The independent SHIP at f385485 carries to this head: the PR's own change (git diff $(git merge-base feat-cmux-next HEAD) HEAD) has the same git patch-id --verbatim (7c61b1de559ef676) at both heads, and no merge between them was resolved by hand. Nothing new to review. A newer BLOCK, or any change to the PR's own diff, ends the carry.

[teamleaderleo]

Review (next-review patch-id; carried): SHIP at 95a4a76

The independent SHIP at f385485 carries to this head: the PR's own change (git diff $(git merge-base feat-cmux-next HEAD) HEAD) has the same git patch-id --verbatim (7c61b1de559ef676) at both heads, and no merge between them was resolved by hand. Nothing new to review. A newer BLOCK, or any change to the PR's own diff, ends the carry.

[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 (next-review patch-id; carried): SHIP at fc2ef0d

The independent SHIP at f385485 carries to this head: the PR's own change (git diff $(git merge-base feat-cmux-next HEAD) HEAD) has the same git patch-id --verbatim (7c61b1de559ef676) at both heads, and no merge between them was resolved by hand. Nothing new to review. A newer BLOCK, or any change to the PR's own diff, ends the carry.

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