refactor(mobile): git sheets use uniwind platform variants instead of className ternaries

[juliusmarminge]

The problem

The Git sheets used Platform.OS ternaries over className strings for styling-only divergence. #13161 migrated them to ios:/android: Uniwind class variants and was reverted by #13169, because Uniwind 1.11.0 dropped the platform guard for every utility past the first of each @media ios/@media android block \u2014 Android classes kept applying on iOS and vice versa. The revert explicitly gated re-landing on the compiler being fixed and re-proved on devices.

How this fixes it

Stacked on #13172, which patches that compiler bug (media queries now outlive the reset between a block's sibling rules) and pins it with compiled-output tests. On top of that, this re-lands the original migration (ec28eefa0d0, reverting its revert 6dcde218ad6):

• Styling-only divergence -> ios:/android: class variants ("android:shrink android:grow-0 ios:flex-1", ios:rounded-[22px] vs android:rounded-[20px], font-weight families, etc.) \u2014 ~165 fewer lines of className ternary machinery across five files.
• Behavior/structure divergence stays in JS: Android native-sheet variants, runtime maxHeight: windowHeight * 0.92 inline styles, conditional icon rendering, numeric size props.

Base note: targets agent/mobile-audit-uniwind-platform-variants; retarget to main once #13172 merges. Merging this before #13172 would ship unguarded variants \u2014 exactly what #13169 reverted.

Verification

• tsc --noEmit, mobile src/lib suite (43 files / 613 tests), dependency-graph.test.ts all pass.
• Real compiled bundles (expo export, this branch): every migrated class is platform-guarded \u2014 the iOS stylesheet contains none of android:shrink, android:grow-0, android:gap-1/gap-2, android:p-2, android:rounded-[20px], android:font-t3-medium and the Android stylesheet contains none of ios:flex-1, ios:flex-row, ios:border, ios:gap-3, ios:rounded-[22px], ios:font-t3-bold, ios:font-[family-name:Menlo], ios:pt-[72px]. (The same grep on revert(mobile): git sheets back to Platform.OS ternaries (un-guarded uniwind variants broke both platforms) #13169's head showed the leak.)
• Device re-proof of the Git sheets specifically (the check revert(mobile): git sheets back to Platform.OS ternaries (un-guarded uniwind variants broke both platforms) #13169 asked for) is being run now; evidence will be added to this PR.

Co-Authored-By: Apex noreply@callstack.com

---

Summary by CodeRabbit

• Style
  • Updated Git-related sheets in the mobile app with platform-specific styling for Android and iOS.
  • Preserved existing layouts, spacing, colors, typography, controls, and interaction states across branch, commit, confirmation, and overview screens.
  • Maintained existing commit selection, editing, validation, reset, and action behavior.

[macroscopeapp]

Approvability

Verdict: Approved at 25d82da

Macroscope's review found this PR approvable — This change only rewrites existing iOS/Android styling conditionals into guarded Uniwind platform variants across the Git sheets. Existing actions, layouts, and platform-specific behavior remain intact, with compiler tests and bundle checks covering the platform isolation.

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

[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.4 KiB	—	7.8 KiB	✅
Codex	Live turn WebSocket decoded	—	56.2 KiB	—	66.4 KiB	✅
Codex	Live turn messages	—	9	—	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.5 KiB	—	7.8 KiB	✅
Claude	Live turn WebSocket decoded	—	57.1 KiB	—	66.4 KiB	✅
Claude	Live turn messages	—	10	—	21	✅

Baseline: unavailable · PR result: 25d82da · 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.7 KiB

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

[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: Team

Run ID: 09660dad-48d7-416c-8f45-dc2574ecb4d5

📥 Commits

Reviewing files that changed from the base of the PR and between 343aa08 and 25d82da.

📒 Files selected for processing (5)

• apps/mobile/src/features/threads/git/GitBranchesSheet.tsx
• apps/mobile/src/features/threads/git/GitCommitSheet.tsx
• apps/mobile/src/features/threads/git/GitConfirmSheet.tsx
• apps/mobile/src/features/threads/git/GitOverviewSheet.tsx
• apps/mobile/src/features/threads/git/gitSheetComponents.tsx

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

---

📝 Walkthrough

Walkthrough

Changes

Git sheet styling

Layer / File(s)	Summary
Shared sheet component styling apps/mobile/src/features/threads/git/gitSheetComponents.tsx	MetaCard and SheetListRow replace runtime platform styling with NativeWind modifiers while retaining platform-specific layout, state, shape, and typography.
Branch and commit sheet styling apps/mobile/src/features/threads/git/GitBranchesSheet.tsx, apps/mobile/src/features/threads/git/GitCommitSheet.tsx	Platform-specific classes now define sheet, card, row, input, text, and action styling. Branch and commit behavior remains unchanged.
Confirmation and overview sheet styling apps/mobile/src/features/threads/git/GitConfirmSheet.tsx, apps/mobile/src/features/threads/git/GitOverviewSheet.tsx	Confirmation and overview layouts now use platform-specific utility classes. Existing Android and iOS visual distinctions remain represented.

Priority: ➖ Normal

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

Change: Refactor

Suggested reviewers: pixpmusic

Merge Risk: ⚪ Minimal · up to 25d82

The Git sheet styling migration does not show an actionable compatibility or runtime regression in the reviewed change set.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 6 functions across 5 files.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the main change: replacing className ternaries with Uniwind platform variants in the mobile Git sheets.
Description check	✅ Passed	The description clearly explains the problem, solution, dependency on #13172, verification steps, and target branch. It does not use the template headings, include the required checklist, or provide b…
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.

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