fix(mobile): stop refreshing Git status on streamed thread updates

[juliusmarminge]

Scope: maintainer-requested mobile performance audit; this layer fixes one measured redundant Git-refresh loop.

Streamed thread-shell updates replaced selectedThread, retriggering the mobile Git-status refresh effect. Each completed refresh also invalidated the branch-list cache, causing extra server requests, SQLite work, queries, and renders throughout a turn.

Key the initial refresh on environment, thread, project availability, and working directory. useEffectEvent keeps the callback current. Explicit Git-action and sheet refreshes remain; the existing server status subscription publishes the final working-tree status when a turn finishes.

Release simulator proof: iPhone 17 Pro / iOS 27, optimized Hermes, 1,210 thread shells, 30 seconds of 20 updates/sec in the selected chat plus four background chats at 2 updates/sec. Both runs delivered all 840 generated updates. Restored the same server database snapshot before the after run.

Sampled stack	Before	After
Git-ref cache invalidation	108	0

This is removal of observed redundant work, not a measured end-to-end latency or battery percentage. One pair; capture windows differ (42.94s / 52.09s), and the after app was relaunched. Synthetic text seeds vary slightly with dispatch timing. Counts are samples, not RPC invocation counts. No overall speedup claim.

Evidence: measurement details, before profile, after profile. Profiles contain normalized process IDs/timestamps and source stack names only. No visual layout changes.

Behavior trade-off: header Git status no longer refreshes on every shell update during a turn; mid-turn edits can appear at turn completion or explicit refresh. Opening the branch sheet mounts this hook and refreshes its branch list, including branches created by an agent. This does not add a cache, timer, or dependency.

Validation: mobile TypeScript, focused lint and format checks; 26 tests across RunFinalizationService.test.ts and VcsStatusBroadcaster.test.ts. Verified the fixed release app loaded the fixture and streamed the complete workload. Native performance was tested on iOS Simulator only; Android and physical-device power measurements remain outside this proof.

Implemented by Claude Opus 5.5 through T3 Code. Reviewed and release-profiled by GPT-6 Astra through the Codex harness.

[macroscopeapp]

Approvability

Verdict: Approved at 1403761

Macroscope's review found this PR approvable — This is a narrowly scoped mobile bug fix that prevents redundant Git-status refreshes during streamed thread updates while preserving refreshes when the selected thread or working directory changes. It introduces no schema, configuration-default, infrastructure, security, or static-analysis changes.

Notes:

• All code in this push has already been reviewed. Approvability was decided on eligibility alone.

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

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

[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	5.0 KiB	4.9 KiB	−23 B (−0.5%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	−23 B (−1.9%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.9 KiB	20.8 KiB	−41 B (−0.2%)	29.3 KiB	✅
Codex	Live turn messages	2	1	−1 (−50.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	−24 B (−0.5%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	−24 B (−2.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	−41 B (−0.2%)	29.3 KiB	✅
Claude	Live turn messages	2	1	−1 (−50.0%)	8	✅

Baseline: e803242 · PR result: ef182b3 · 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: 108.5 KiB
• Claude decoded thread snapshot: 108.8 KiB

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

[coderabbitai]

Warning

Review limit reached

CodeRabbit couldn't verify your organization's usage billing state, so no usage-priced review was started. Retry after the billing service recovers.

Next included review available in 18 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: 3026acb7-af25-42ab-b367-fb25a10a131e

📥 Commits

Reviewing files that changed from the base of the PR and between c6f4eae and ef182b3.

📒 Files selected for processing (2)

• apps/mobile/src/state/use-selected-thread-git-actions.test.ts
• apps/mobile/src/state/use-selected-thread-git-actions.ts

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: c39223de-3d23-4541-825c-032add222c74

📥 Commits

Reviewing files that changed from the base of the PR and between 7b0dc7b and f623cad.

📒 Files selected for processing (1)

• apps/mobile/src/state/use-selected-thread-git-actions.ts

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

---

📝 Walkthrough

Walkthrough

The status-refresh effect now tracks primitive selection identifiers and the selected cwd. It refreshes quietly only when the required selection values are present, using useEffectEvent to call the latest refresh handler.

Changes

Selected-thread status refresh

Layer / File(s)	Summary
Guarded status refresh apps/mobile/src/state/use-selected-thread-git-actions.ts	The effect uses primitive selection values instead of thread and project objects. It checks that the environment, thread, project, and cwd are present before a quiet refresh. useEffectEvent supplies the latest refresh handler without making it an effect dependency.

Priority: ⬇️ Low

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to f623c

Git status refreshes are no longer retriggered by streamed thread updates. The inspected selection and refresh paths support this change without establishing a concrete stale-status case.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

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

✅ Passed checks (4 passed)

Check name	Status	Explanation
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.
Title check	✅ Passed	The title clearly identifies the mobile fix: Git status no longer refreshes on streamed thread updates.
Description check	✅ Passed	The description explains the problem, change, measured results, behavior trade-off, and focused validation. It does not include a link to the maintainer approval or discussion requested by the scope t…

✨ 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.}

Dismissing prior approval to re-evaluate d5079a9

Dismissing prior approval to re-evaluate 1403761