Repository navigation
chore!: ship ESM-only package output - #475
Merged
Merged
Conversation
Drop the CommonJS build target. The package now publishes one ES module build in lib/module and one set of type definitions in lib/typescript, and the export conditions point both import and require resolution at them. BREAKING CHANGE: The package no longer ships a CommonJS build. Jest setups that load react-native-nitro-sqlite/mock must add the package to transformIgnorePatterns so Jest transforms its ES modules. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This was referenced Nov 10, 2026
This branch was successfully deployed
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.
The core package builds a CommonJS copy of every file and a second set of type definitions next to its ES module build. React Native apps don't use it, because Metro reads the source through the
react-nativecondition, and Node 22.12 and later canrequire()the ES module build. This PR drops the CommonJS target, so the package ships one build inlib/moduleand one set of types inlib/typescript, and everyexportsentry points at them.This breaks Jest setups that load
react-native-nitro-sqlite/mock. Jest doesn't transform files innode_modulesby default, so it fails on the ES module syntax until the package is added totransformIgnorePatterns. The mock guide now includes that step:🤖 Generated with Claude Code