fix(mobile): restore live tool shimmer and add a Thinking row #10173 · pingdotgg/t3code

Problem

Two mobile regressions against the web chat view:

The running tool row stopped shimmering. #9106 narrowed the shimmer from "the trailing live run" to "a call whose lifecycle is still inProgress". Codex only reports inProgress on item.started, which the work log drops, so in practice the mobile summary row lost its shine the moment a call completed while the turn kept going. Web keeps shining until the latest call fails, is declined, or is stopped (#9098, restored again in #9777 after the perf pass).

No "Thinking" row. Web shows a shimmering Thinking live activity whenever a turn is working with no shimmering tool row (before the first call, and after a failed one, per #9165). Mobile never had it, so a working turn with no tools rendered nothing above the composer.

Fix

Mirror web's live-row rules in deriveThreadFeedPresentation:

The trailing tool group shimmers while its latest call is running or succeeded. A failed, declined, or stopped call turns the shine off.

When a turn is working and no tool row shimmers, append one thinking feed entry. It renders through the existing ShimmeringWorkContent with a brain icon (SF brain, Tabler IconBrain on Android) and uses the same fixed row height as the group toggle, so the list's size estimates stay exact.

The thinking row object is reused across derivations while createdAt/turnId are unchanged, so LegendList keeps the row instance during streaming.

Verification

vp test run apps/mobile/src/lib/threadActivity.test.ts — 90 passed. Updated the completed/failed lifecycle expectations to the new rules and added a test for the standalone Thinking row (present while working, stable identity, absent when idle).

tsc --noEmit for apps/mobile, targeted vp lint and vp fmt --check on the changed files (only pre-existing warnings in ThreadFeed.tsx).

iOS Simulator (iPhone 17 Pro, iOS 26.5) against a disposable base dir seeded with three running turns. Before is main at 62ed748, after is this branch, same dev client, same seeded data.

Working turn, no tool calls yet

Latest tool call completed, turn still running

The label is identical in both; the difference is motion, so here is the recording (before on the left, after on the right):

Latest tool call failed, turn still running

Android was not exercised (no emulator on the verification host); the only Android-specific change is the Tabler icon mapping.

Implemented by Claude Fable 5 in the Claude Code harness.

Note

Restore live tool shimmer and add a `thinking` row to mobile thread feed

Adds a thinking entry variant to the thread feed type union and renders it as a new ThreadThinkingRow component with a brain icon and shimmering label.
The presentation builder appends one thinking row at the tail when a turn is actively working and no work-toggle row is already shimmering; existing thinking rows are filtered out before rebuild to avoid duplicates.
A cached thinking entry keeps object identity stable across repeated derivations unless the active-work timestamp or turn id changes.
Updates work-group shimmer logic in appendToolGroupRows so a trailing work group shimmers when it has an active activity or when its latest activity succeeded, restoring the live-tool shimmer.
Maps the brain SF Symbol to the Tabler brain icon in the Android fallback so ThreadThinkingRow renders correctly on Android.
Risk: appendPresentedFeedEntry and appendToolGroupRows in threadActivity.ts now exclude and recompute shimmer for thinking entries; any out-of-tree callers of these helpers will need to handle the new thinking variant.

^{Macroscope summarized 51cc1ec.}

Added 1 commitSep 5, 2026

fix(mobile): restore live tool shimmer and add a Thinking rowcfd143c

Ggithub-actionsBotSep 5, 2026

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	13.6 KiB	13.6 KiB	+15 B (+0.1%)	15.1 KiB	✅
Codex	Thread snapshot wire	7.0 KiB	7.0 KiB	+4 B (+0.1%)	7.3 KiB	✅
Codex	Live turn WebSocket wire	6.6 KiB	6.6 KiB	+11 B (+0.2%)	7.8 KiB	✅
Codex	Live turn WebSocket decoded	57.1 KiB	57.1 KiB	0 B (0.0%)	66.4 KiB	✅
Codex	Live turn messages	10	10	0 (0.0%)	21	✅
Claude	Total thread wire	13.6 KiB	13.6 KiB	+21 B (+0.2%)	15.1 KiB	✅
Claude	Thread snapshot wire	7.0 KiB	7.0 KiB	−4 B (−0.1%)	7.3 KiB	✅
Claude	Live turn WebSocket wire	6.5 KiB	6.6 KiB	+25 B (+0.4%)	7.8 KiB	✅
Claude	Live turn WebSocket decoded	57.8 KiB	57.9 KiB	+44 B (+0.1%)	66.4 KiB	✅
Claude	Live turn messages	9	10	+1 (+11.1%)	21	✅

Baseline: 62ed748 · PR result: 51cc1ec · 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: 113.8 KiB
Claude decoded thread snapshot: 114.5 KiB

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

MmacroscopeappBotreviewed · review dismissedSep 5, 2026

MmacroscopeappBotSep 5, 2026

Approvability

Verdict: Approved at 51cc1ec

Macroscope's review found this PR approvable — This is a focused mobile thread-feed bug fix that restores live tool shimmer and adds a small Thinking indicator during active turns. Its impact is confined to presentation logic with targeted state and identity coverage, and it introduces no schema, default, deployment, or sensitive-data changes.

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