fix(mobile): long approval commands scroll inside the approval card#27332

Problem

On mobile, the pending approval card above the composer had no height cap and no scroll. A long command, such as a 40-line heredoc, made the card taller than the screen. Its top slid under the navigation header, and dragging the card did nothing, so the command could not be read.

Reproduced on main (b42adff3) on an iPhone 17 Pro simulator and a Pixel 9 emulator. The thread was a Supervised Claude Haiku 5.5 thread, asked to run a 40-line heredoc through Bash.

Change

• PendingApprovalCard takes an animated maxHeight and clips to it, the way PendingUserInputCard already does.
  • The heading, title, command, reason and full-access explanation scroll inside the card.
  • The option warning, the "provider no longer available" notice and the decision buttons stay pinned at the bottom.
• ThreadDetailScreen derives the cap from the existing derivePendingUserInputMaxHeight budget, on the UI thread. The budget subtracts:
  • the screen height minus the navigation header;
  • the composer's measured height plus the slot's pb-3;
  • the keyboard lift that the sticky composer applies.
• The card is keyed by request id, so the next approval opens scrolled to the top with its explanation collapsed.
• When the content fits, the card looks the same as before.

Scope and approval

Fixes pingdotgg/t3code#27291

Mobile client only. Web, desktop, server and contracts are unchanged. Approval decisions, options and permission checks are unchanged. Only the layout of the card changes.

Verification

• vp run check:changed passes on the merged head: format, lint, knip, typecheck of 2 packages, and React Compiler on both files.
• No unit test was added. This is a layout bug, so device captures are the proof.

Captures. Real native dev clients on pooled devices. Each side ran from its own checkout, with an isolated server and Metro on a fresh database (T3CODE_DEV_SEED=0).

• Fixtures: the long approval is an identical synthetic Replay (dev) ACP permission request on both sides, a cat <<'EOF' with 40 lines. The short control is the stock Replay "Write approval" scenario.
• Devices: iPhone 17 Pro on iOS 27.2 (402×874 pt, scale 3) and Pixel 9 on Android 16, API 36 (1080×2424 px, scale 2.625). Font scale 1.0. Status bar clock normalized on stills. Captured with agent-device screenshot and native screen recording.
• SHAs: before is b42adff3. After is 0b485f7b, whose two changed files are identical at the merged head.

M01: long command, light, keyboard closed

M01 · iPhone 17 Pro · light

Before	After	Diff0 (0%)	DiffT (2%)
b42adff3	0b485f7b	b42adff3 → 0b485f7b	b42adff3 → 0b485f7b

M01 · Pixel 9 · light

Before	After	Diff0 (0%)	DiffT (2%)
b42adff3	0b485f7b	b42adff3 → 0b485f7b	b42adff3 → 0b485f7b

M02: short approval (control)

M02 · iPhone 17 Pro · light — card and composer positions are identical, and no pixels differ at 2%.

Before	After
b42adff3	0b485f7b

M02 · Pixel 9 · light. The card outline, buttons and composer match exactly. The title and detail text sit one physical pixel (0.38 dp) higher inside the new scroll view.

Before	After	Diff0 (0%)	DiffT (2%)
b42adff3	0b485f7b	b42adff3 → 0b485f7b	b42adff3 → 0b485f7b

M03: long command, dark

M03 · iPhone 17 Pro · dark

Before	After
b42adff3	0b485f7b

M04: scroll and keyboard (recordings)

Both sides ran the same script. Times are seconds from the start of the recording, before / after:

1. Drag inside the command toward the bottom ten times (1.0–16.1 / 1.0–16.1), then capture a checkpoint (17.7 / 17.7).
2. Drag toward the top ten times (18.9–33.9 / 19.1–33.9), then capture a checkpoint (35.5 / 35.7).
3. Focus the composer (36.8 / 37.2).
4. Type a four-line draft (37.9 / 38.6).
5. Capture a keyboard checkpoint (42.1 / 43.8).

Before: the drags move nothing, and the heading stays under the header. After: the command scrolls to line 40 and EOF, then back to the heading, while the buttons stay put. When the keyboard opens, the card shrinks to stay below the header.

Before · iPhone 17 Pro	After · iPhone 17 Pro

Before · Pixel 9	After · Pixel 9

What else changed:

• The heading and title now scroll with the command; only the warning, the unavailable-provider notice and the buttons are pinned.
• With a long card, the scroll indicator shows while dragging.
• On Pixel, the long-card buttons and composer sit about one physical pixel lower. On the short card, the title and detail sit one physical pixel higher (see M02).
• iOS glass and shadow rendering differs faintly in the zero-threshold diffs. None of it survives at 2%.
• While a multi-line draft grows the composer, the card overlaps the header for about half a second, until the composer's measured height settles. You can see this around 42 s in the iPhone recording and 89 s in the Pixel recording.

Not checked:

• An option warning on a long command, such as Antigravity's "allow always" warning. Replay cannot produce option warnings.
• The unavailable-provider and read-only notices.
• An approval card and a question card shown together.
• Tablets and larger system text.

The seed and driver steps follow the repo's test-t3-mobile flow. The Replay fixture is a session/request_permission frame whose execute tool call carries the 40-line heredoc; it is not committed.

Lead, implementation and review B: Claude Opus 5.5 in Claude Code via T3 Code. Review A, proof audit and captures: GPT-6.1-Sol in Codex via T3 Code.

🤖 Generated with Claude Code