Skip to content

fix: include Nitrogen generated files in package#7

Merged
jingjing2222 merged 1 commit into
mainfrom
fix-packaging
Apr 25, 2026
Merged

fix: include Nitrogen generated files in package#7
jingjing2222 merged 1 commit into
mainfrom
fix-packaging

Conversation

@jingjing2222

@jingjing2222 jingjing2222 commented Apr 25, 2026

Copy link
Copy Markdown
Owner

Summary

  • Commit nitrogen/generated outputs so installs from GitHub include the Nitro bridge without consumer-side generation.
  • Stop ignoring nitrogen/ and update contributor docs to match Nitro's packaging guidance.
  • Add verify:package-contents to assert the npm tarball includes built JS and key Nitrogen generated files.
  • Add CI checks for generated Nitrogen drift and package contents.

Root Cause

The npm tarball could include generated files because prepack runs yarn build, but the Git repository did not track nitrogen/. Direct GitHub installs therefore received native build files that reference generated Nitrogen paths without those files being present.

@jingjing2222 jingjing2222 marked this pull request as ready for review April 25, 2026 01:04
@jingjing2222 jingjing2222 self-assigned this Apr 25, 2026
@jingjing2222 jingjing2222 merged commit 5d0861f into main Apr 25, 2026
7 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