Give every Cloud sidebar row reachable hover text

[teamleaderleo]

What

The Cloud tree hosts its row content in CloudTreePassthroughHostingView, whose hitTest returns nil so the NSOutlineView owns every pointer event. Several rows put their secondary information in a SwiftUI .help() inside that host, which means no pointer can ever reach it. Terminal rows (directory, agent state, the detached explanation), display rows (transport and screen), machine-resource rows and port rows all had hover text that never appeared. The same strings were already mirrored into cell-level accessibility labels, so VoiceOver read them and the mouse did not.

Workspace rows had a second, narrower version of the bug. configureDisplayHost is the only code that has resolved a workspace's presence heads, so it set the cell tooltip there. configure then ran its own if case chain whose else reset toolTip to nil, and a second chain that overwrote the accessibility label. Only the trailing updatePresenceSubscription() put them back, and only when the cell was already in a window with a superview. A freshly created cell kept neither.

Separately, CloudTreeMachineRowContent takes an injected now and passes it to its metrics, but subtitle measured the machine's age against Date(), so the two halves of one row could be read against different clocks.

How

CloudTreeRowToolTip.describe(node:style:presenceHeads:) computes the hover text and the accessibility label for every row kind in one exhaustive switch, and CloudTreeCellView applies it in exactly one place. That removes the double write entirely rather than reordering it.

Row kinds that gained hover text they never had:

Row	Hover text
Workspace	name, working directory, terminal count, collaborators
Terminal	the text CloudTreeTerminalRowContent.toolTip already built
Display	title and transport
Port	full link, title, detail
Browser	title, URL, host or owning workspace
Placeholder	its own line, which truncates at narrow widths

Section headers deliberately keep no tooltip: their labels are fixed strings that never truncate, and CloudPortsVPNAffordanceTests asserts that.

An untitled browser row's searchableTitle is the empty resource title, so its accessibility label was empty; it now falls back to the same placeholder the row draws.

subtitle reads now.

Verification

The tests are committed before the fix (e05a8bc) and fail there; the fix (6d120a1) turns them green. Focused CI runs for both SHAs are linked in a comment below.

Also adds dogfood/scenarios/cloud-sidebar-audit-tour.json, the reusable tour that walks Files, Vault and Cloud in the right sidebar and opens the Cloud Sidebar Spacing Lab. It is a look, not a test.

Part of the Cloud right sidebar audit (manaflow-ai/cmuxterm-hq#853).

Changelog

Fixed: Cloud sidebar rows show their details on hover again, and a workspace row names its directory, terminal count and collaborators.

🤖 Generated with Claude Code

---

Summary by cubic

Fixes Cloud sidebar hover text so every row shows its details on hover, and a workspace row names its directory, terminal count, and collaborators.

Row tooltips lived in SwiftUI .help() inside a host that never hit-tests, so no pointer could reach them. Hover text and the accessibility label are now computed in one exhaustive switch and applied to the cell in exactly one place. Workspace rows previously had their tooltip written in configureDisplayHost and then reset by configure; they now keep it regardless of whether the cell is in a window.

Tooltips drop when they would only repeat the row's own text, and a workspace with zero terminals shows no count line. Section headers keep no tooltip since their labels are fixed and never truncate. An untitled browser row no longer has an empty accessibility label. The machine row's age reads the injected clock instead of Date(), making it pinnable by tests.

Adds CloudTreeRowToolTipTests covering each row kind. Merges main, which brings a reworded fork-detach message.

^{Written for commit 2416da9. Summary will update on new commits.}

[coderabbitai]

Warning

Review limit reached

Next included review available in 2 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 10 included reviews currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Repository: manaflow-ai/cmux/.coderabbit.yaml

Review profile: ASSERTIVE

Plan: Advanced

Run ID: 0c4bb723-f27c-4380-9051-9c73464d905b

📥 Commits

Reviewing files that changed from the base of the PR and between 5cfc6a6 and 2416da9.

📒 Files selected for processing (1)

• Resources/Localizable.xcstrings

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[github-actions]

All contributors have signed the CLA ✍️ ✅
_{Posted by the CLA Assistant Lite bot.}

[teamleaderleo]

Review

A review subagent went over e05a8bc00ff..6d120a1923c read-only. It found one break and three smaller things. Fixed in 7d06ae9e6da.

Fixed:

• It broke an existing test, and the reason it broke was a product bug. CloudWorkspacePresenceHeadsTests pins that a workspace with no presence, no detail and no terminals has no hover text. My describe gave it "Workspace b\n0 terminals", because count(0) is a non-optional never-empty string. So every workspace row in the Cloud sidebar would have popped a tooltip repeating its own visible title plus a zero. That is the rule the new type's own doc comment states, broken by the type itself. joined now takes a beyond: title and returns nil when everything left is the row's own title, and the zero count is no longer a line at all.
• Three more arms had gained a tooltip byte-identical to the row's visible text: .placeholder (its searchableTitle is placeholder.text), .localWorkspace (row.title) and .resource. Same beyond: guard, so they go back to no hover text unless they have something to add. None of these were in the "lost their .help()" set this PR is about, so this was scope I had not earned.
• The now comment overstated the fix. Both shipping constructors leave now at its default, so no user ever saw the two halves of a machine row disagree. The comment now says what is true: it makes the age pinnable by a test.
• .devicesEmpty in the section-label arm is unreachable (outlineView(_:viewFor:item:) routes it to CloudTreeDevicesEmptyCell). Kept for exhaustiveness, now says so.
• dogfood/scenarios/cloud-sidebar-audit-tour.json did not belong in this PR. Screenshots cannot capture tooltips, and the tour needs a runner fix to work at all. It moved to Resolve a dogfood menu path against the direct children of each open menu #15248.

Left:

• prepareForReuse still clears toolTip, which the review called redundant now that configureDisplayHost is the single writer. Kept: clearing on reuse is the defensive half of a cell lifecycle, and it costs nothing.
• terminalRowHasToolTip compares against the same production expression the fix wires up, so it checks wiring rather than content. The review is right that it is mildly tautological. It is still red at base, and the content of a terminal tooltip is already covered by CloudTreeTerminalRowContent's own tests.
• sectionHeadersHaveNoToolTip is green before and after. It is a regression guard, not red-before evidence, and I am not counting it as one.
• searchableTitle for an untitled browser is still the empty string. Only the label and tooltip get the fallback. Changing searchableTitle changes what sidebar search matches, which is a separate call.

Two new tests pin the rule directly: a bare workspace row and a placeholder row both have no hover text.

[teamleaderleo]

Red and green

Same focused command both times: python3 scripts/ci/dispatch-focused-test.py cmuxTests/CloudTreeRowToolTipTests --ref <sha>.

Red at e05a8bc00ff, tests only: https://github.com/manaflow-ai/cmux/actions/runs/36396054301 — failed.

Seven of the eight failed, each on its own assertion:

✘ A workspace row off-window still describes itself
    Expectation failed: (cell).toolTip → nil → nil
✘ A workspace row lists its collaborators without dropping its own name
✘ A terminal row's directory and agent reach the pointer
✘ A display row names its transport on hover
✘ A port row's full link survives a truncated title
✘ An untitled browser row is still labelled for assistive technology
✘ A machine row's age is measured against the clock it was given

Section headers keep their bare label passed at both ends. It is a regression guard, not red-before evidence, and I am not counting it as one.

Green at 6d120a1923c, with the fix: https://github.com/manaflow-ai/cmux/actions/runs/36396100321 — success.

After the review fix at 7d06ae9e6da, widened to the two suites the review said the change could reach: https://github.com/manaflow-ai/cmux/actions/runs/36399535425 (CloudTreeRowToolTipTests, CloudWorkspacePresenceHeadsTests, CloudPortsVPNAffordanceTests). Outcome posted when it lands.

Verification

python3 scripts/verify-local.py was not run: the sandbox declined it, and I did not split it into sub-checks after the denial. Everything here rests on the CI runs above.

[teamleaderleo]

CI receipt, as promised.

Run 36399535425 at 7d06ae9e6da, green:

cmuxTests/CloudTreeRowToolTipTests: 10 test(s) executed
cmuxTests/CloudWorkspacePresenceHeadsTests: 2 test(s) executed
cmuxTests/CloudPortsVPNAffordanceTests: 2 test(s) executed
✔ Test run with 14 tests in 3 suites passed

The test job reads as skipped on that run; that is the normal path, not a coverage hole. The build job runs the selected tests itself and sets tested=true, which is exactly what the test job's condition checks for (.github/workflows/test-e2e.yml:1158). The executed counts above are from the build job's own run.

python3 scripts/verify-local.py is blocked in this session's sandbox, so CI is the only check that has run on this branch.

[cursor]

Bugbot is paused — on-demand spend limit reached

Bugbot uses usage-based billing for this team and has hit its on-demand spend limit.

A team admin can raise the spend limit in the Cursor dashboard, or wait for the next billing cycle to continue.

[teamleaderleo]

Closing this: the work already landed on main in #15326, so there is nothing left here to merge.

#15326 ("say a machine's id and age in its accessibility label", defccda0e53) carries the same row hover-text work, including cmuxTests/CloudTreeRowToolTipTests.swift with the same test names and several cases this branch never had. After merging main, this branch's own diff against the main-side parent is one file, Resources/Localizable.xcstrings, holding a single reverted fork-error string. That revert is an artifact of the catch-up, not intended work.

The catch-up that produced it ran while the local clone still had a shallow graft, which made git merge-base --all return two unrelated bases and the merge report conflicts in files this branch never touched. The clone is fully unshallowed now, so later catch-ups on the other Cloud sidebar branches are sound; this one is simply redundant.

No behavior is lost by closing it. The tooltip coverage is on main.