Merged
Conversation
bump expo SDK to 54
update async-storage dep version cap
There was a problem hiding this comment.
Pull request overview
Updates the React Native (Expo) SDK and sample app to support Expo SDK 54 while broadening @logto/rn compatibility across Expo SDK 52–54.
Changes:
- Bumped
@logto/rnversion and widened peer dependency range for async storage. - Updated
rn-sampledependencies to Expo SDK 54 / React 19 / RN 0.81. - Adjusted Expo app config and removed the custom Metro config from the sample.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| packages/rn/package.json | Bumps package version, updates dev deps (RN/Expo modules), and broadens peer dependency range for async-storage. |
| packages/rn-sample/package.json | Moves the sample app to Expo SDK 54 + updated React/RN ecosystem versions. |
| packages/rn-sample/metro.config.js | Removes the monorepo Metro override in favor of default Expo/Metro behavior. |
| packages/rn-sample/app.json | Adds Expo plugins and reformats config values for SDK 54 setup. |
Comments suppressed due to low confidence (1)
packages/rn/package.json:67
@logto/rnimports fromreact(hooks/components) but doesn’t declarereactinpeerDependencies. Declaring an explicitreactpeer range (aligned with the supported Expo SDKs / React versions) makes the package’s runtime requirement clear and helps package managers validate consumer installs.
"@react-native-async-storage/async-storage": ">=1.23.1 <3",
"expo-crypto": ">=14.0.2 <16",
"expo-secure-store": ">=14.0.1 <16",
"expo-web-browser": ">=14.0.2 <16",
"react-native": ">=0.76.0 <1"
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
charIeszhao
approved these changes
Jan 29, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
@logto/rnto include SDK 54~.@logto/rnSDK peer dependency ranges to cover Expo SDK 52~ and 54~ while pinning dev deps accordingly.Testing
test locally on ios simulator
Checklist
.changeset