fix(web): keep the branch toolbar in sync with the real checkout

[denniskasper]

What Changed

Two related branch-toolbar reconciliation fixes:

1. Refresh VCS status (not just the ref list) when the branch combobox opens and after branch actions. Previously, opening the picker reloaded the refs but left the trigger label driven by stale status — so the list could update while the label didn't.
2. When git authoritatively reports a detached HEAD (on no branch — e.g. the thread's branch was deleted out from under the checkout), stop falling back to the now-dangling thread branch and show "Select ref" instead of a branch name that isn't actually checked out.

Related to #2926.

Why

The toolbar could display a branch you weren't on: after deleting/switching the underlying branch externally, the label kept showing the old/deleted branch (e.g. throwaway/repro-error) even though HEAD was detached. These changes make the toolbar reflect the real checkout.

UI Changes

No layout/styling change — behavioral. In detached HEAD (e.g. the thread's branch was deleted), the trigger label previously showed the stale/deleted branch name; it now shows "Select ref". The combobox also re-syncs its status when opened, so the label can't lag behind the ref list.

Checklist

• This PR is small and focused
• I explained what changed and why
• I included before/after screenshots for any UI changes (N/A — no visual/layout change; behavioral, described above)
• I included a video for animation/interaction changes (N/A — covered by the watcher demo in fix(vcs): reflect external git changes in status via .git watcher #2987)

[coderabbitai]

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: f1aad669-5990-45ef-817a-9ca35907848f

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

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 and usage tips.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit edec0b7. Configure here.}

[cursor]

Stale refs defeat detached toolbar label

Medium Severity

Detached checkout is detected from loaded VCS status (refName is null), but currentGitBranch still substitutes the refs list when status returns that null via ??. The detached branch of resolveBranchToolbarValue then returns that ref-derived name, so the toolbar can keep showing a deleted or stale branch instead of Select ref.

Additional Locations (1)

• apps/web/src/components/BranchToolbar.logic.ts#L104-L107

 

^{Reviewed by Cursor Bugbot for commit edec0b7. Configure here.}

[macroscopeapp]

Approvability

Verdict: Needs human review

An unresolved review comment identifies a potential logic bug where the detached checkout detection may not correctly display 'Select ref' due to how currentGitBranch is derived. This substantive concern about the fix's correctness warrants human review.

^{You can customize Macroscope's approvability policy. Learn more.}