chore: upgrade dependencies across the repository - #10210
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe PR updates React Native dependencies across workspace packages and example applications. It adds ChangesDependency and example alignment
Estimated code review effort: 3 (Moderate) | ~25 minutes Possibly related PRs
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@apps/tvos-example/package.json`:
- Around line 30-34: Remove the unused `@types/react-test-renderer` and
react-test-renderer entries from the tvOS example’s package.json dependencies,
leaving the remaining packages unchanged; no testing-library migration is
needed.
In `@apps/web-example/package.json`:
- Around line 20-23: Align the dependency versions in the web-example package
manifest by replacing react-native 0.87.0-rc.4 with the React Native 0.86.x
version supported by Expo 57, or upgrade Expo packages to a release supporting
0.87. Afterward, run Expo dependency checks to verify the package matrix is
consistent.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 74b56722-7eeb-4cb5-af3e-9a03c86e217c
⛔ Files ignored due to path filters (15)
.yarn/patches/@gorhom-bottom-sheet-npm-5.2.8-1ad4b8f529.patchis excluded by!**/.yarn/**.yarn/patches/fuse.js-npm-7.1.0-5dcae892a6.patchis excluded by!**/.yarn/**.yarn/patches/metro-npm-0.87.0-1c5c005545.patchis excluded by!**/.yarn/**.yarn/patches/metro-runtime-npm-0.87.0-2e4e60a54a.patchis excluded by!**/.yarn/**.yarn/patches/react-native-gesture-handler-npm-3.0.2-893975050e.patchis excluded by!**/.yarn/**.yarn/patches/react-native-macos-npm-0.81.9-aa583a0648.patchis excluded by!**/.yarn/**.yarn/patches/react-native-mmkv-npm-4.3.2-24e071ac53.patchis excluded by!**/.yarn/**.yarn/patches/react-native-nitro-modules-npm-0.36.1-32b64d0393.patchis excluded by!**/.yarn/**.yarn/patches/react-native-safe-area-context-npm-5.8.0-98d3ae1299.patchis excluded by!**/.yarn/**.yarn/patches/react-native-screens-npm-4.26.0-54f969441f.patchis excluded by!**/.yarn/**.yarn/patches/react-native-tvos-npm-0.86.0-1-2baf7632d6.patchis excluded by!**/.yarn/**apps/fabric-example/ios/Podfile.lockis excluded by!**/*.lockapps/macos-example/macos/Podfile.lockis excluded by!**/*.lockapps/tvos-example/ios/Podfile.lockis excluded by!**/*.lockyarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (15)
.yarnrc.ymlapps/common-app/package.jsonapps/fabric-example/android/app/build.gradleapps/fabric-example/android/gradle.propertiesapps/fabric-example/package.jsonapps/macos-example/metro.config.jsapps/macos-example/package.jsonapps/next-example/package.jsonapps/tvos-example/package.jsonapps/web-example/package.jsonpackage.jsonpackages/eslint-plugin-reanimated/package.jsonpackages/react-native-reanimated/package.jsonpackages/react-native-worklets/package.jsonscripts/patches/resolutions.patch
Add a macOS-local helper to turn bundle mode off when working on the macOS example.
tjzel
left a comment
There was a problem hiding this comment.
approved with szpont machen
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@scripts/toggle-bundle-mode.sh`:
- Around line 19-22: Update the bundle-mode disable flow around the PATCHES
reversal loop and yarn install so failures from git apply --reverse or
YARN_ENABLE_IMMUTABLE_INSTALLS=false yarn install are detected and propagated
instead of printing success or exiting 0. Handle already-reversed patches by
retrying installation or restoring the prior patch state, ensuring a failed
install does not leave the script reporting bundle mode as successfully
disabled.
🪄 Autofix
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Repository UI
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 35ac5d1b-77fe-45a2-81e6-84e5984dcdda
📒 Files selected for processing (2)
apps/macos-example/package.jsonscripts/toggle-bundle-mode.sh
Summary
This PR upgrades various dependencies in the repository and adds minor adjustments, through that achieving the following:
Dep upgrade
Having example app opt-out from built-in kotlin and new DSL
The Upgrade Helper recommends to opt-out from AGP 9's built-in kotlin and new DSL. In our packages we do handle both configurations for those options both enabled and disabled. It is not the same in the case of other libraries we are using, therefore if we wanted to keep those properties set to true, we'd have to keep manual patches for each of those libraries in the repository which makes maintenance more troublesome and messy. A cleaner approach is to opt-out of those in our example app.
Patches cleanup
The dep upgrade and the above opt-out both help us reduce the number of patches the repo needed to make example apps work. Now only the necessary ones are left.
RN Screens as peer dep across example apps
Previously RN Screens was just a dep in the
common-appthat the actual example apps import. This had to change now because:macos-exampledue to some import mismatchesAs a result we both have the newest RN Screens in normal example apps, and also a working one in the macOS example
Other remarks
0.87.0, otherwise they were also applied to the macOS example that should be using older metroTest plan
Tested with all the example apps. All build and are working.