Skip to content

fix(ios): patch expo-modules-jsi Swift 6 abs() overload ambiguity - #417

Closed
RonenMars wants to merge 1 commit into
integration-merge-354-355-376from
fix/expo-modules-jsi-swift6-abs-ambiguity
Closed

fix(ios): patch expo-modules-jsi Swift 6 abs() overload ambiguity#417
RonenMars wants to merge 1 commit into
integration-merge-354-355-376from
fix/expo-modules-jsi-swift6-abs-ambiguity

Conversation

@RonenMars

Copy link
Copy Markdown
Owner

Summary

  • Xcode 26.3's Swift 6 compiler fails on abs(milliseconds) in expo-modules-jsi's Date decoding with "type of expression is ambiguous without a type annotation", breaking the Ship iOS job (run 30127420057).
  • Expo's fix for this is bumping the minimum Xcode to 26.4 (confirmed via upstream issue Swift 6.2 build failure in expo-modules-jsi: ambiguous abs() in dateFromMilliseconds expo/expo#47875), but Xcode 26.4 isn't preinstalled on GitHub's macos-15 runner image yet, so this can't be resolved by a workflow change today.
  • Applied via patch-package as a stopgap: disambiguates the call as Swift.abs(milliseconds). Registered patches/ as a CI-significant path so future patch-only changes aren't mis-tagged [skip-ci].
  • Left a comment in deploy.yml next to DEVELOPER_DIR noting to bump to Xcode 26.4 and drop the patch once the runner image has it.

Test plan

  • npm install applies the patch via postinstall (verified locally)
  • Ship iOS workflow archives successfully with this branch

Xcode 26.3's Swift 6 compiler fails on abs(milliseconds) in
expo-modules-jsi's Date decoding with "type of expression is
ambiguous without a type annotation", breaking the Ship iOS job.
Expo's real fix (Xcode 26.4) isn't on GitHub's macos-15 runner yet,
so patch-package disambiguates the call as a stopgap.
@RonenMars
RonenMars deleted the branch integration-merge-354-355-376 July 26, 2026 08:09
@RonenMars RonenMars closed this Jul 26, 2026
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