refactor(mobile): split native menus by platform

[juliusmarminge]

The shared menu wrapper loads and initializes the iOS themed MenuView wrapper on Android before choosing its Android branch. Use a base iOS ControlPillMenu and an .android.tsx override and preserve its existing export from ControlPill, so all callers keep the same API. Keep iOS gesture suppression and menu lifecycle handlers with the native menu, and Android anchoring and haptics with the Android implementation.

Verification:

• Existing menu action color tests: 6 passed; mobile workspace typecheck passed.
• Targeted lint passed with a refs warning on the preserved iOS props forwarding code.
• Both release exports succeeded. Source maps resolve only the matching ControlPillMenu implementation.

The existing menu branches and handlers are preserved. No visual or interaction change is intended; evidence is release module resolution and existing tests.
Based on #12381.

Runtime verification on September 18, 2026: the integrated stack at 47a3dcb was tested on an iPhone 17 Pro simulator (iOS 27.0) and T3Code_API_36_1 emulator (Android 16.0), one at a time. Compatible development clients were built and installed from this checkout, then connected to a disposable environment with seeded Git projects. Coverage included Home/list rendering, thread navigation, file search and word wrap, terminal menus and session selection, review diff selection, archive menus, and Settings Add Environment / scanner open and close. Temporary connections were removed and the devices and test servers were stopped. Tablet layouts, QR decoding, and Live Activity start/update/stop were not exercised. Screenshots use seeded demo content; any test counts in demo messages or terminal output are fixture text, not validation results.

The iOS native menus and Android nested menus were exercised through file word wrap, terminal text size/session selection, and review diff selection. Runtime evidence is attached to #12391.

The pre-existing iOS archive keyboard issue and stale-connection loading limitation are recorded with comparison evidence in #12399.

Prepared with Codex.

[github-actions]

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

ℹ️ No successful main baseline artifact is available yet. This run establishes the initial measurement.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	—	13.5 KiB	—	15.1 KiB	✅
Codex	Thread snapshot wire	—	7.1 KiB	—	7.3 KiB	✅
Codex	Live turn WebSocket wire	—	6.5 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	56.3 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	10	—	21	✅
Claude	Total thread wire	—	13.5 KiB	—	15.1 KiB	✅
Claude	Thread snapshot wire	—	7.1 KiB	—	7.3 KiB	✅
Claude	Live turn WebSocket wire	—	6.4 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	57.0 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	9	—	21	✅

Baseline: unavailable · PR result: 7828c61 · 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.9 KiB
• Claude decoded thread snapshot: 114.6 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.yaml

Review profile: CHILL

Plan: Team

Run ID: ced1971e-ad7e-451f-b986-2f3483bdc81c

📥 Commits

Reviewing files that changed from the base of the PR and between f23d3fa and 7828c61.

📒 Files selected for processing (1)

• apps/mobile/src/components/ControlPillMenu.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 2 remain after this review.

---

📝 Walkthrough

Walkthrough

The change extracts ControlPillMenu from ControlPill.tsx into shared type, Android, and iOS modules. ControlPill.tsx now re-exports the component.

Changes

ControlPillMenu extraction

Layer / File(s)	Summary
Menu contract and exports apps/mobile/src/components/ControlPillMenu.types.ts, apps/mobile/src/components/ControlPill.tsx	Adds ControlPillMenuProps and re-exports ControlPillMenu from the existing component module.
Android menu implementation apps/mobile/src/components/ControlPillMenu.android.tsx	Adds anchored-menu rendering and conditional long-press handling with haptic feedback.
iOS menu implementation apps/mobile/src/components/ControlPillMenu.tsx	Adds themed native menu rendering and coordinates long-press, touch, accessibility, and menu lifecycle events.
ControlPill integration cleanup apps/mobile/src/components/ControlPill.tsx	Removes the embedded menu implementation and its menu-specific dependencies.

Priority: ⬇️ Low

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to 7828c

The menu split retains existing interaction and callback behavior, with no concrete merge-blocking risk identified.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 25.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 4 functions across 5 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 and concisely describes the main change: splitting native mobile menus by platform.
Description check	✅ Passed	The description explains what changed, why the refactor was needed, the intended lack of UI changes, and verification results. It does not use the template headings or include the checklist, but the r…

• Fix all pre-merge checks with AI

✨ 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

---

_{Comment @coderabbitai help to get the list of available commands.}

[macroscopeapp]

Approvability

Verdict: Approved at 7828c61

Macroscope's review found this PR approvable — This refactor separates the existing iOS and Android menu implementations without changing callers, APIs, or interaction logic. Runtime impact is limited to loading the platform-specific implementation, with no new user-facing capability or product-default change.

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

Dismissing prior approval to re-evaluate 7828c61