fix(mobile): restore chat image previews in the v5 stack

[juliusmarminge]

Chat image taps stopped opening the native viewer on iPhone after the v5 stack migration. The legacy presentation container reported its first or synthetic route as unfocused while a real child route was active, so FilePreviewModal immediately cleared the selected image.

Let the v5 card and iPad workspace presentation containers preserve their real route descriptors' navigation and focus contexts. The native-stack patch adds an opt-in presentationEnvelope prop; ordinary native-stack screens keep the existing provider behavior. The preview's dismiss-on-blur guard stays intact.

Verification

• Reproduced before the fix: the image tap handler fired, but useIsFocused() was false while navigation.isFocused() was true, and no native viewer appeared.
• After the fix, tapping a chat image opens Quick Look on iPhone and iPad. On iPhone, closing returns to chat; the hold menu opens, its file-viewer action navigates correctly, and the file preview opens full-screen. The reported hold-menu failure was not reproduced with the workspace-image fixture.
• 25 focused workspace projection and scroll-edge tests passed.
• Mobile typecheck, targeted lint, and React Doctor passed.

After: native iPhone viewer with Close, Markup, and Share controls.

Model: GPT-6.1 Sol. Harness: Codex in T3 Code.

---

[macroscopeapp]

Approvability

Verdict: Approved at 7f779fe

Macroscope's review found this PR approvable — This is a narrowly scoped iOS navigation fix that preserves the real route’s focus context inside existing V5 presentation envelopes, restoring image previews while leaving ordinary native-stack behavior unchanged. The dependency patch, type declarations, and lockfile are synchronized, with no product-default or static-analysis changes.

^{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	4.9 KiB	5.0 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.8 KiB	20.9 KiB	+41 B (+0.2%)	29.3 KiB	✅
Codex	Live turn messages	1	2	+1 (+100.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	−7 B (−0.1%)	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	−7 B (−0.6%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	0 B (0.0%)	29.3 KiB	✅
Claude	Live turn messages	2	2	0 (0.0%)	8	✅

Baseline: 5b78b34 · PR result: 7f779fe · 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]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

• Configuration used: Path: .coderabbit.config.ts
• Review profile: CHILL
• Plan: Team
• Run ID: 4cac0f1c-8550-41cd-9cf9-8b6444fd0274

📥 Commits

Reviewing files that changed from the base of the PR and between 5b78b34 and 7f779fe.

⛔ Files ignored due to path filters (1)

• pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml

📒 Files selected for processing (3)

• apps/mobile/src/features/layout/createWorkspaceStackNavigator.ios.tsx
• apps/mobile/src/native/createV5StackNavigator.ios.tsx
• patches/@react-navigation%2Fnative-stack@7.17.6.patch

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

---

📝 Walkthrough

Walkthrough

The native-stack patch adds presentationEnvelope handling, a navigator-level transition completion event, and header item support. The workspace and V5 iOS stack navigators now pass presentationEnvelope to NativeStackView.

Changes

Native stack updates

Layer / File(s)	Summary
Native-stack view and header support patches/@react-navigation%2Fnative-stack@7.17.6.patch	NativeStackView accepts and passes presentationEnvelope. When true, the scene-level NavigationProvider is skipped. The patch also adds a navigator-level finishTransitioning event and header item configuration support.
iOS navigator wiring apps/mobile/src/features/layout/createWorkspaceStackNavigator.ios.tsx, apps/mobile/src/native/createV5StackNavigator.ios.tsx	Both iOS stack navigators pass presentationEnvelope to NativeStackView.

Priority: ⬇️ Low

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

Change: Bug fix

Merge Risk: ⚪ Minimal · up to 7f779

No merge-blocking issue is established for the image-preview fix; it is ready for normal merge checks.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check		The title clearly and concisely describes the main change: restoring chat image previews in the mobile v5 stack.
Description check		The description explains the problem, the fix, and verification results. It includes UI evidence and agent details. It does not use the template headings and does not provide a scope or approval link,…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

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