fix(web): new worktree threads no longer say "checkout" during setup

[t3dotgg]

When you start a thread in a new worktree, the bar under the composer said "Local checkout" (sometimes "Current checkout") while the worktree was still being set up. That made it look like the agent would run in the main checkout.

The server creates the thread before the worktree exists, so the thread has no worktree path during checkout. The toolbar read "no path" as the project checkout. Now a thread that is still preparing its worktree counts as worktree mode, and the locked label says "New worktree". It switches to "Worktree" when checkout finishes.

ChatView also passes its env mode to the toolbar now, so the toolbar no longer works out its own. That covers the short window before the thread detail loads, where the toolbar showed "Current checkout".

Before	After

Proof: I used a repo with a slow post-checkout hook to hold the checkout step open. In the web dev client, the bar went "New worktree" → "Local checkout / main" before this change. With the change it went "New worktree / From main" → "Worktree / t3code/…". Unit tests, typecheck, and lint pass.

🤖 Generated with Claude Code using Claude Opus 5.5

Summary by CodeRabbit

• Bug Fixes
  • Workspace indicators now reflect the selected mode, showing the worktree icon and “New worktree” label when a worktree is being prepared, even before it is attached.
  • Locked workspace labels now distinguish between a new worktree and a local checkout, with icons matching the active workspace mode.

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.5 KiB	13.5 KiB	−33 B (−0.2%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.1 KiB	7.1 KiB	−5 B (−0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.5 KiB	6.4 KiB	−28 B (−0.4%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	56.3 KiB	56.2 KiB	−44 B (−0.1%)	66.4 KiB	✅
Codex	Live turn messages	10	9	−1 (−10.0%)	21	✅
Claude	Total thread wire	13.5 KiB	13.5 KiB	−6 B (−0.0%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.1 KiB	7.1 KiB	+9 B (+0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.4 KiB	6.4 KiB	−15 B (−0.2%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.0 KiB	57.0 KiB	0 B (0.0%)	66.4 KiB	✅
Claude	Live turn messages	9	9	0 (0.0%)	21	✅

Baseline: b3de243 · PR result: e8f9a37 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

• Codex decoded thread snapshot: 114.0 KiB
• Claude decoded thread snapshot: 114.7 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Repository: pingdotgg/t3code/.coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 460a4663-89b3-4dea-8af4-201ef4cf43d9

📥 Commits

Reviewing files that changed from the base of the PR and between d10bd13 and d4d4e4bbab8681af6ba3dcfd76f204a3e06d6396.

📒 Files selected for processing (5)

• apps/web/src/components/BranchToolbar.logic.test.ts
• apps/web/src/components/BranchToolbar.logic.ts
• apps/web/src/components/BranchToolbar.tsx
• apps/web/src/components/BranchToolbarEnvModeSelector.tsx
• apps/web/src/components/ChatView.tsx

Included review availability: 6 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 10 reviews per hour.

---

📝 Walkthrough

Walkthrough

The toolbar now receives a resolved environment mode that accounts for worktree preparation. Its locked display uses the mode and active worktree path to choose the workspace label and icon.

Changes

Workspace mode

Layer / File(s)	Summary
Resolve mode during worktree preparation apps/web/src/components/BranchToolbar.logic.ts, apps/web/src/components/ChatView.tsx, apps/web/src/components/BranchToolbar.logic.test.ts	For server threads, worktree preparation now selects worktree mode when no worktree path is attached. ChatView passes the preparation state to the resolver. Tests cover preparation in progress and the omitted state.
Pass the resolved mode through the toolbar apps/web/src/components/BranchToolbar.tsx, apps/web/src/components/ChatView.tsx	BranchToolbar now requires envMode and derives effective mode from it or forceNewWorktree. ChatView always passes envMode, and the branch selector always receives the derived mode.
Render the locked workspace label and icon apps/web/src/components/BranchToolbar.logic.ts, apps/web/src/components/BranchToolbarEnvModeSelector.tsx, apps/web/src/components/BranchToolbar.tsx, apps/web/src/components/BranchToolbar.logic.test.ts	The locked workspace label now depends on the effective mode when no worktree is attached. The selector uses the active worktree path and effective mode to choose the label and icon. Tests cover the “New worktree” label.

Priority: ⚪ Not assessed

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

Change: Bug fix

Suggested reviewers: chrisdeeming

Merge Risk: ⚪ Minimal · up to d4d4e

The toolbar should show “New worktree” during setup and “Worktree” once the path is available. No issue requiring a fix before merge is established.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 20.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 5 functions across 5 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: correcting the worktree label during setup.
Description check	✅ Passed	The description explains what changed, why it changed, the UI impact, before-and-after screenshots, testing performed, and the observed behavior. It does not use the template headings or include the c…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

• Fix all pre-merge checks with AI

✨ Finishing Touches 💡 1

📝 Generate docstrings 💡

• Commit to this branch
• Create a new PR

🧪 Generate unit tests (beta)

• Commit to this branch
• Create a new PR

---

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

[macroscopeapp]

Approvability

Verdict: Approved at d4d4e4b

Macroscope's review found this PR approvable — This is a narrowly scoped web bug fix that corrects workspace labeling during worktree creation and adds focused unit tests. It does not introduce a new capability, alter product defaults, change schemas, or affect the underlying worktree execution flow.

No code changes detected at e8f9a37. Prior analysis still applies.

^{You can add or adjust custom eligibility rules. Learn more.}