Skip to content

chore: upgrade dependencies and update documentation for Expo SDK 57 - #2

Merged
jkasprzyk17 merged 1 commit into
mainfrom
chore/expo-57-support
Jul 19, 2026
Merged

chore: upgrade dependencies and update documentation for Expo SDK 57#2
jkasprzyk17 merged 1 commit into
mainfrom
chore/expo-57-support

Conversation

@jkasprzyk17

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Jul 7, 2026

Copy link
Copy Markdown

Review Change Stack

📝 Walkthrough

Summary by CodeRabbit

  • Breaking Changes

    • Updated the minimum supported Nitro Modules version to 0.36.0+.
    • Upgraded the example app and library to newer Expo, React, and React Native versions.
  • Documentation

    • Refreshed installation, quick-start, compatibility, and changelog notes with the latest supported versions and migration steps.
  • Chores

    • Bumped package and test tooling versions, including the Jest preset update.

Walkthrough

This PR bumps react-native-nitro-modules peer dependency to >=0.36.0, upgrades the example app to Expo SDK 57 (React Native 0.86, React 19.2.3), updates root/package dependencies and devDependencies, switches the Jest preset, and updates documentation and changelog entries accordingly.

Changes

Version Upgrade and Documentation

Layer / File(s) Summary
Package manifest and dependency version bumps
package.json, package/package.json, package/jest.config.js, example/package.json
Bumps package version to 0.0.16, raises react-native-nitro-modules peer dependency to >=0.36.0, updates devDependencies, switches Jest preset to @react-native/jest-preset, and upgrades example app dependencies to Expo ~57, React 19.2.3, React Native 0.86.0.
Changelog and documentation updates
CHANGELOG.md, README.md, docs/docs/compatibility.md, docs/docs/quick-start.md, docs/docs/setup/installation.md
Adds a 0.0.16 changelog entry and updates README/docs prerequisites to reflect Nitro Modules 0.36.0+, Expo 53–57 support, Node.js 22.13+ for SDK 57, and Android minSdk prefab notes.

Estimated code review effort: 2 (Simple) | ~10 minutes

Nothing groundbreaking here, just a pile of version bumps you could've done half-asleep — dependencies nudged up, docs patched to match, Jest preset swapped. Utterly mechanical, but at least it's consistent, unlike some PRs I've seen.

🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Description check ❓ Inconclusive No pull request description was provided, so there is no meaningful summary to assess. Add a brief description explaining the dependency bumps and documentation updates for Expo SDK 57.
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly matches the changes: dependency upgrades and Expo SDK 57 documentation updates.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Security Check ✅ Passed No medium/high security issues: the PR only updates docs, dependency versions, and a Jest preset; it adds no attacker-controlled runtime sink or auth boundary change.

Comment @coderabbitai help to get the list of available commands.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

♻️ Duplicate comments (1)
example/package.json (1)

34-37: 🗄️ Data Integrity & Integration | 🟠 Major | ⚡ Quick win

Same TypeScript split flagged in package/package.json.

typescript here is ~6.0.3 while the library pins ^5.8.3. See the comment on package/package.json (Lines 12-32) — root-cause the misalignment there rather than patching it in isolation here.

🤖 Prompt for 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.

In `@example/package.json` around lines 34 - 37, The TypeScript version in this
package is out of sync with the library package, so fix the shared version
source rather than changing this entry alone. Update the dependency alignment in
the main package configuration that also governs `typescript` (the one
referenced in `package/package.json`), then ensure this
`devDependencies.typescript` entry matches that single source of truth across
the repo.
🤖 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.

Duplicate comments:
In `@example/package.json`:
- Around line 34-37: The TypeScript version in this package is out of sync with
the library package, so fix the shared version source rather than changing this
entry alone. Update the dependency alignment in the main package configuration
that also governs `typescript` (the one referenced in `package/package.json`),
then ensure this `devDependencies.typescript` entry matches that single source
of truth across the repo.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7b463bf0-59f6-4a5c-8d50-ebdec7b62278

📥 Commits

Reviewing files that changed from the base of the PR and between 3f673da and 5d5293f.

⛔ Files ignored due to path filters (1)
  • bun.lock is excluded by !**/*.lock
📒 Files selected for processing (10)
  • CHANGELOG.md
  • README.md
  • docs/docs/compatibility.md
  • docs/docs/quick-start.md
  • docs/docs/setup/installation.md
  • example/package.json
  • package.json
  • package/jest.config.js
  • package/package.json
  • package/tsconfig.tsbuildinfo

@jkasprzyk17
jkasprzyk17 merged commit 994625a into main Jul 19, 2026
4 checks passed
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