fix(mobile): restore QR scanning in iPad pairing sheet

[arhammahajan]

Problem

On iPad, New Task → Add Environment opens a pairing sheet without its title or QR scanning action. Adding an environment through Settings shows both.

Fixes #15985.

Change

Initialize the root ConnectionsNew route with the existing iOS sheet header options and the Add Environment title. This lets React Native Screens mount the sheet's native header stack on the first render, restoring the existing QR action and avoiding a later header-visibility change that remounts the pairing form.

Scope and approval

This qualifies for the very small, focused obvious-bug exception in CONTRIBUTING.md: it restores an existing pairing control that is missing from one navigation entry point. The four added lines are confined to the root route options in apps/mobile/src/Stack.tsx; the pairing form and scanner logic are unchanged. The iOS header preset is conditional, and web/desktop, provider adapters, and wire contracts are untouched.

Issue triage identifies the root sheet header as the likely cause. That comment is context for the problem, rather than approval of this specific implementation.

Verification

• iOS development and standalone Release Preview builds passed.
• Tested New Task → Add Environment on an iPad Air 11-inch (M2), running iPadOS 27. The QR button appears and the flow works.
• Confirmed the missing header before the fix and the restored title and QR button afterward on an iPad Pro 13-inch (M5) simulator.
• Checked camera permission denial and opening/closing the scanner.
• vp test run apps/mobile/src/features/connection/pairing.test.ts: 7 tests passed.
• Mobile typecheck, lint, formatting, and git diff --check passed.

iPhone runtime behavior and Android runtime behavior were not exercised.

Before

After

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

[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: Advanced
• Run ID: 08cae951-7805-48f0-b8d5-dd3103c15bff

📥 Commits

Reviewing files that changed from the base of the PR and between 7812230 and c03635b.

📒 Files selected for processing (1)

• apps/mobile/src/Stack.tsx

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

---

📝 Walkthrough

Walkthrough

The ConnectionsNew route applies SHEET_GLASS_HEADER_OPTIONS on iOS. Its title and form-sheet presentation settings remain unchanged.

Changes

Add Environment header

Layer / File(s)	Summary
Apply iOS sheet header options apps/mobile/src/Stack.tsx	On iOS, the route applies SHEET_GLASS_HEADER_OPTIONS to keep the native header visible from the first render. Other platforms do not receive these options.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~5 minutes

Change: Bug fix · Severity of issue fixed: Low

Suggested reviewers: juliusmarminge

Merge Risk: ⚪ Minimal · up to c0363

The iOS Add Environment sheet now receives its header and title from the initial route options, with its form-sheet behavior preserved. No actionable merge risk is evident.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	[#15985] requires QR scanning in the initial Add Environment flow on iPad. The root ConnectionsNew route now applies SHEET_GLASS_HEADER_OPTIONS on iOS and sets the title. `ConnectionsNewRouteScree…
Out of Scope Changes check	✅ Passed	The diff only adds iOS sheet-header options and the Add Environment title to the root ConnectionsNew route in apps/mobile/src/Stack.tsx. These changes directly support [#15985]. No unrelated chang…
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Title check	✅ Passed	The title clearly and concisely identifies the main change: restoring QR scanning in the iPad pairing sheet.
Description check	✅ Passed	The description covers the problem, change, scope and approval rationale, and verification. It includes focused test results, platform limitations, and before-and-after screenshots.

✨ Finishing Touches

🧪 Generate unit tests (beta)

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

[arhammahajan]

Hey @juliusmarminge just following up. Can this be merged?