Skip to content

chore: upgrade dependencies for React Native 0.84#50

Merged
mattinannt merged 1 commit intomainfrom
codex/deps-upgrade-rn-0.84
Feb 14, 2026
Merged

chore: upgrade dependencies for React Native 0.84#50
mattinannt merged 1 commit intomainfrom
codex/deps-upgrade-rn-0.84

Conversation

@mattinannt
Copy link
Member

@mattinannt mattinannt commented Feb 14, 2026

Summary

  • apply dependency upgrades generated via ncu -u --deep across the workspace
  • update response schemas for Zod v4 by using explicit key/value z.record signatures
  • externalize @react-native-community/netinfo in the library build to avoid broken bundled NativeRNCNetInfo imports
  • update playground export script to expo export --no-bytecode so builds do not require a local hermesc binary

Verification

  • pnpm build
  • pnpm test

@sonarqubecloud
Copy link

@coderabbitai
Copy link

coderabbitai bot commented Feb 14, 2026

Walkthrough

This pull request updates dependencies across the monorepo, including Expo and React ecosystem packages in the playground application, development tools in the root package configuration, and packages in the react-native workspace. Additionally, zod schema definitions for response types are modified to explicitly specify string keys for record objects, and the Vite build configuration is updated to treat the netinfo community package as an external dependency.

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly summarizes the main objective of the PR: upgrading dependencies for React Native 0.84, which is the primary theme across all changed files.
Description check ✅ Passed The description is directly related to the changeset, providing specific details about dependency upgrades, schema updates for Zod v4, externalizing a dependency, and updating the export script.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


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.

❤️ Share

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

Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
packages/react-native/src/types/response.ts (1)

3-3: ⚠️ Potential issue | 🟡 Minor

Pre-existing: TResponseData allows Record<string, string> but ZResponseData does not.

The TypeScript type on Line 3 includes Record<string, string> in its union, but the Zod schema on Line 23 only validates z.string() | z.number() | z.array(z.string()). Data containing nested Record<string, string> values would pass the TS type check but fail Zod validation at runtime. This predates this PR but is worth tracking.

Also applies to: 23-23

🤖 Fix all issues with AI agents
In `@packages/react-native/package.json`:
- Around line 44-45: You bumped zod from 3.x to 4.x which is a breaking change
for consumers importing exported Zod schemas (e.g., ZResponseData,
ZResponseTtc); update package.json to reflect a breaking release (bump the
package version to 3.0.0 or the next major), and add zod as a peerDependency
("zod": "^4.0.0") (or at minimum keep it as a devDependency and document the
change) plus add a concise migration note in CHANGELOG/README pointing out the
Zod v3→v4 migration for consumers importing those schemas.
🧹 Nitpick comments (1)
packages/react-native/package.json (1)

58-64: Peer dependency for eslint is pinned to an exact old version.

"eslint": "8.57.0" as a peer dependency is unusually restrictive and pins consumers to ESLint 8. This appears pre-existing but worth noting — if you're upgrading everything else, consider relaxing this to ">=8.0.0" or adding ESLint 9 support.

@mattinannt mattinannt added this pull request to the merge queue Feb 14, 2026
Merged via the queue into main with commit 38c68b0 Feb 14, 2026
9 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.

2 participants