Skip to content

[menu-bar] Upgrade to expo SDK 56#346

Open
gabrieldonadel wants to merge 6 commits into
mainfrom
@gabrieldonadel/sdk-56
Open

[menu-bar] Upgrade to expo SDK 56#346
gabrieldonadel wants to merge 6 commits into
mainfrom
@gabrieldonadel/sdk-56

Conversation

@gabrieldonadel

Copy link
Copy Markdown
Member

Why

We should upgrade to Expo SDK 56 in order to try out the outOfTreePlatforms experiment flag.

How

  • Bump expo dependencies to SDK 56 (canary because of outOfTreePlatforms experiment) and run npx expo install --fix.
  • Remove the two existing expo patches
  • Adopt outOfTreePlatforms experiment
    • Removes the resolveRequest hook in metro.config.js that rewrote react-native to react-native-macos for the macos platform.
    • Removes the getModulesRunBeforeMainModule override that injected both InitializeCore files.
    • Simplifies the Podfile to use the standard reactNativePath instead of the -macos suffixed path, and switches react-native-config to the macos platform with --source-dir set explicitly.
    • Adds patches/react-native-macos+0.81.6.patch to (a) make react-native.config.js tolerate @react-native-community/cli not being installed and (b) let the Cocoapods autolinking script fall back to ios project/platform config when macos keys are absent.
  • Replaced @react-native-clipboard/clipboard with expo-clipboard
  • Replaced SystemIconView local module with expo-symbols

Test Plan

  • Run menu-bar app locally

"eslint-plugin-react-compiler": "^19.1.0-rc.2",
"expo": "~54.0.33",
"expo-web-browser": "~15.0.10",
"expo": "^56.0.6-canary-20260606-4d4666c",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this isn't the latest 😅

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is the latest Canary that includes the outOfTreePlatforms flag

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't we use 56.0.12? Didn't we backport it?

@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/sdk-56 branch from 4a56177 to 2f06228 Compare June 16, 2026 14:49
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/sdk-56 branch from 13ece28 to e629495 Compare June 16, 2026 18:50
@gabrieldonadel gabrieldonadel force-pushed the @gabrieldonadel/sdk-56 branch from e629495 to f9a6bd9 Compare June 16, 2026 20:58
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.

3 participants