Update to Babylon.js 9.0 and modernize React Native stack - #16
Open
bghgary wants to merge 8 commits into
Open
Conversation
bghgary
commented
Mar 26, 2026
bghgary
commented
Contributor
- Babylon.js packages: 5.x -> 9.0.0
- React: 18.0.0 -> 19.0.0
- React Native: 0.69.4 -> 0.79.4
- Remove deprecated @babylonjs/react-native-iosandroid-0-69
- Update @babylonjs/react-native to 2.0.1
- Modernize babel, metro, eslint, and TypeScript configs
- Babylon.js packages: 5.x -> 9.0.0 - React: 18.0.0 -> 19.0.0 - React Native: 0.69.4 -> 0.79.4 - Remove deprecated @babylonjs/react-native-iosandroid-0-69 - Update @babylonjs/react-native to 2.0.1 - Modernize babel, metro, eslint, and TypeScript configs - Update all dev dependencies to current versions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Adds lint/typecheck, Android build, and iOS build jobs. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Android: Update Gradle 7.3 -> 8.13, AGP, SDK 35, Kotlin 2.0, Hermes - Android: Remove Flipper, legacy architecture, BUCK support - Android: Use RN Gradle Plugin with autolinking - iOS: Update Podfile for RN 0.79 pod resolution - iOS: Remove Flipper, M1 workarounds, legacy flags Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Required by RN 0.79 for both Android and iOS native module autolinking. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Podfile.lock from RN 0.69 conflicts with RN 0.79 pod versions. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Remove Flipper, old architecture, JNI, and BUCK files - Add Kotlin MainActivity and MainApplication matching RN 0.79 template - Update AndroidManifest.xml (remove package attr, use namespace) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
The published package references include/ and source/ (lowercase) but actual directories are Include/ and Source/. This breaks on Linux (CI). Adds a postinstall script to patch the paths. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Also patch target_include_directories 'include' -> 'Include'. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
bghgary
marked this pull request as ready for review
March 26, 2026 18:13
CedricGuillemet
approved these changes
Mar 27, 2026
This comment was marked as off-topic.
This comment was marked as off-topic.
Sorry, something went wrong.
RaananW
previously requested changes
Sep 11, 2026
RaananW
left a comment
RaananW
left a comment
Member
There was a problem hiding this comment.
🤖 This review comment was created by an AI agent.
Thanks for modernizing this old sample and adding CI. The job named Lint & Typecheck only runs TypeScript at .github/workflows/ci.yml:10-20. It never runs the existing npm run lint, and no job runs the existing test script. Please add lint and test steps so the new stack is checked before merge.
RaananW
dismissed
their stale review
September 11, 2026 22:38
Automated monitor correction: this repository is excluded from auto-review because its name contains Native.
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
3 participants
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.