fix(clients): a dev server left running no longer says the thread is waiting

[t3dotgg]

When an agent ends its turn and leaves a dev server running, the composer strip says "Waiting on command Start the shared dev server" and its dot pulses. The agent is done, and nothing will wake it. On one maintainer machine in the last 10 days, every dev server left at the end of a turn was a hand-off to the user ("Tell me when to stop it"). Those servers ran for 2 to 24 hours, and the dot pulsed the whole time.

Now the strip uses the same rule that alerts use since #14213 (backgroundWorkHoldsCompletion):

• Only commands left: "Running: Start the shared dev server", with a still dot.
• Work that wakes the agent (subagents, monitors): "Waiting on …" with the pulse, as before.

The Stop button stays. Mobile shows the same title, so it follows without a code change. #14910 applies the same rule to the sidebar status.

Before	After

Proof: a real Claude Opus 5.5 turn in a dev build left a 30-minute sleep running in the background. The strip showed the new title, and the dot's computed animation was none. Stop still ended the command. The new unit test fails on main and passes here. Typecheck passes for client-runtime, web, and mobile.

Made with Claude Code (Claude Opus 5.5).

🤖 Generated with Claude Code

[macroscopeapp]

Approvability

Verdict: Approved at cb864a1

Macroscope's review found this PR approvable — This is a focused UI bug fix: command-only background work is labeled as running and no longer pulses, while work that can wake the agent retains its waiting presentation. The change is self-contained, covered by tests, and does not affect schemas, deployment, security, billing, or static-analysis configuration.

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

[coderabbitai]

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

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: 931b9b94-9c6e-41cc-9a9f-8f05af087eba

📥 Commits

Reviewing files that changed from the base of the PR and between 5691412 and cb864a1.

📒 Files selected for processing (3)

• apps/web/src/components/ChatView.tsx
• packages/client-runtime/src/state/threadExecution.test.ts
• packages/client-runtime/src/state/threadExecution.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 5 remain after this review.

---

📝 Walkthrough

Walkthrough

Pending background work is classified as waiting or running. Titles reflect that classification, and the chat banner pulses its status indicator only when work is waiting.

Changes

Background work status

Layer / File(s)	Summary
Classify and present pending work packages/client-runtime/src/state/threadExecution.ts, packages/client-runtime/src/state/threadExecution.test.ts	Presentations now include a waiting flag. Titles distinguish waiting work from running commands. Tests cover subagents, monitors, command-only work, and mixed work.
Render waiting status apps/web/src/components/ChatView.tsx	The banner description includes dev servers. Its status indicator pulses only when presentation.waiting is true.

Priority: ⬇️ Low

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

Change: Bug fix

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to cb864

Command-only work is presented as running, while work that can wake the agent remains waiting. No issue identified here prevents merge.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 33.33% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 3 functions across 3 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: a dev server left running no longer makes the thread appear to be waiting.
Description check	✅ Passed	The description explains the problem, the change, and focused verification. It also includes before-and-after UI evidence and identifies the agent used. It does not explicitly link a triaged issue or …
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

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

---

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

[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	4.9 KiB	4.9 KiB	−40 B (−0.8%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.7 KiB	3.7 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.1 KiB	−40 B (−3.3%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.4 KiB	20.4 KiB	−41 B (−0.2%)	29.3 KiB	✅
Codex	Live turn messages	2	1	−1 (−50.0%)	8	✅
Claude	Total thread wire	4.9 KiB	4.9 KiB	+41 B (+0.8%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.7 KiB	3.7 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	+41 B (+3.5%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	20.7 KiB	20.8 KiB	+41 B (+0.2%)	29.3 KiB	✅
Claude	Live turn messages	1	2	+1 (+100.0%)	8	✅

Baseline: 5691412 · PR result: cb864a1 · 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: 106.1 KiB
• Claude decoded thread snapshot: 106.4 KiB

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