fix: ship only dist and raise the node floor to 22.12#56
Merged
Conversation
Drop src from the files allowlist. The exports map only exposes the package entry and the factory subpath, both resolved from dist, so the TypeScript source no longer needs to ship. Smaller install.
Raise the engines floor from >=22 to >=22.12.0, where require() of a synchronous ES module is unflagged, the path a CommonJS consumer uses to load this ESM-only package.
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
1 participant
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.
Two user-facing packaging fixes. This is the first change release-please will release.
Changes
distin the published package. The exports map exposes just the entry and thefactorysubpath, both resolved fromdist, so the TypeScript source no longer ships. Smaller install. (README, LICENSE, and package.json are included by npm regardless.)>=22to>=22.12.0, whererequire()of a synchronous ES module is unflagged — the path a CommonJS consumer uses to load this ESM-only package.Verification
npm test,npm run coverage(100%), andnpm run test:packpass locally.npm pack --dry-runconfirms the tarball isdist/*+ README + LICENSE + package.json, nosrc.Both are
fix:, so on merge release-please opens a release PR for 3.0.2. Merging that release PR is what tags, releases, and publishes to npm via OIDC.