Skip to content

fix(mobile): align SDK 58 patches with Expo Doctor - #3246

Merged
DoodleBears merged 1 commit into
mainfrom
codex/3244-expo-doctor-patches
Oct 31, 2026
Merged

DoodleBears merged 1 commit into
mainfrom
codex/3244-expo-doctor-patches

Conversation

@DoodleBears

@DoodleBears DoodleBears commented Oct 31, 2026 •

Copy link
Copy Markdown
Contributor

Closes #3244 | Failed #3239 job

Why the change

Align the seven SDK 58 dependency patches required by online Expo Doctor so the mobile verification and export gates can run after #3239.

Special things to note

  • Merge risk: two-way door — revert this commit to restore the prior manifest and lock; medium blast radius — mobile builds consume the updated Expo packages, so review exact package versions and native compatibility. A new native development build is required; device/runtime acceptance remains unverified.
  • Fresh online Expo Doctor passes 20/20 and fresh mobile typecheck passes on the exact head; the prior task also recorded the red Doctor baseline and iOS/Android export passing; its broader Windows mobile test run had two locale-dependent failures, so it is not a full-suite pass. Current CI remains a separate gate. No UI or Host behavior changes; a dependency-only refresh does not need UI screenshots or an HTML explainer.
  • Implementation task: codex/local/01a19259-8c5c-7bb0-a512-c61f833c1793; interrupted delivery resumed by codex/local/01a1929c-e6ca-7cb1-a0ae-848d0dcd0803. Claims and detailed evidence are in the issue.

Change outline

- @expo/metro-runtime 58.0.15
+ @expo/metro-runtime 58.0.16
- @expo/ui 58.0.25
+ @expo/ui 58.0.26
- expo 58.0.21
+ expo 58.0.22
- expo-dev-client 58.0.17
+ expo-dev-client 58.0.18
- expo-router 58.0.28
+ expo-router 58.0.29
- expo-widgets 58.0.25
+ expo-widgets 58.0.26
- react-native-screens 4.30.0
+ react-native-screens 4.30.1
apps/mobile/package.json  → seven compatible patch selections
pnpm-lock.yaml            → mobile importer and Expo dependency resolution
pnpm-workspace.yaml       → exact-version release-age exceptions for 18 Expo patches

The lock replaces 19 package records, including 12 Expo transitive patches; peer-context changes account for most changed lines. Doctor checks remain enabled, with no broad release-age wildcard.

Issue: #3244
Agent-Task: codex/local/01a19259-8c5c-7bb0-a512-c61f833c1793
@coderabbitai

coderabbitai Bot commented Oct 31, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@DoodleBears

Copy link
Copy Markdown
Contributor Author

CI verified on exact head

Head 0662aafca55df9bc4380b37f64671447a47751fb: verify run 41216342388 completed with every executed check successful, including the original failing mobile verification job, all three test shards, Android native tests, static/build checks, docs, isolated Host smoke, timeline budget and aggregate gate. Intentionally skipped manual/unused jobs are not counted as passes. Draft remains open; no merge, release or native-device acceptance claimed.

@DoodleBears
DoodleBears marked this pull request as ready for review October 31, 2026 16:10
@DoodleBears
DoodleBears merged commit 2253987 into main Oct 31, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant