[ZmeiGorynych]

• The Hubs menu now closes on a pointer press anywhere outside it and the Hubs button, as the worktree menu and the preset picker already do. Escape and the button's toggle are unchanged.
• Right-clicking the main worktree opened a menu whose only item, "Delete worktree", was always disabled there. It now opens no menu, and closes one that is open. Other blocked cases (locked, host not connected, gone) keep the disabled item with its reason.
• Test-only: the fake SSH in src/hub/links/hub.test.ts wrote its argument file one printf at a time, so a loaded run could read a partial list (forwardToHub failed once locally). It now writes through a rename.

Tests

• hubs.spec.ts: a press on a repo tab closes the open menu; the button opens and closes it.
• sidebar.spec.ts: right-clicking the main worktree shows no menu and closes an open one; Escape still closes the worktree menu (split from the old main-worktree test).
• The two new-behaviour tests fail on main. Full suite: vitest 2422/2422, Playwright 182/182.

Summary by CodeRabbit

• Bug Fixes
  • Hubs menus now close when clicking or pressing outside them.
  • The main worktree no longer opens a context menu.
  • Escape and the Hubs button reliably close open menus.
• Tests
  • Added end-to-end coverage for menu dismissal and main worktree behavior.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Repository: MotleyAI/worktree-term/.coderabbit.yaml
• Review profile: CHILL
• Plan: Essentials
• Run ID: 0295b6ea-4d4f-462e-9bc3-608887a19392

📥 Commits

Reviewing files that changed from the base of the PR and between 2fd1730 and 70c1758.

📒 Files selected for processing: 6

• src/hub/links/hub.test.ts
• src/web/ui/app.tsx
• src/web/ui/view.ts
• test/e2e/contract.ts
• test/e2e/hubs.spec.ts
• test/e2e/sidebar.spec.ts

Included review availability: This review used your included allowance. 2 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📝 Walkthrough

The change updates Hubs menu dismissal and main worktree context-menu behavior. It adds end-to-end coverage and updates test contract documentation. The fake SSH test now writes captured arguments through a temporary file before renaming it.

Changes: Menu Behavior

Layer / File(s)	Summary
Main worktree menu behavior src/web/ui/view.ts, test/e2e/contract.ts, test/e2e/sidebar.spec.ts	Opening a context menu for the main worktree now leaves the menu closed. End-to-end coverage checks that it does not open a menu and closes an existing worktree menu. The previous disabled-delete control and title assertions were removed.
Hubs menu dismissal src/web/ui/app.tsx, test/e2e/contract.ts, test/e2e/hubs.spec.ts	The Hubs menu closes on outside pointer presses. End-to-end coverage checks outside-click dismissal and toggling the menu with the Hubs button.

Fake SSH Test Output

Layer / File(s)	Summary
Captured argument file update src/hub/links/hub.test.ts	The fake SSH test writes captured arguments to args.tmp and renames it to args after writing.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 70c17

The change is mergeable with no identified correctness, availability, security, or data-integrity risk.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the two primary changes: closing the Hubs menu on outside presses and preventing a worktree menu on the main checkout.
Docstring Coverage		No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 6…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 6 files.

✨ Finishing Touches

📝 Generate docstrings

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud