Skip to content

Publish npm package as fourjs - #6

Merged
tnb1j merged 1 commit into
mainfrom
release/npm-fourjs-preview
Aug 7, 2026
Merged

Publish npm package as fourjs#6
tnb1j merged 1 commit into
mainfrom
release/npm-fourjs-preview

Conversation

@tnb1j

@tnb1j tnb1j commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Summary

  • Rename the npm package identity from the unavailable @fourjs/core scope to fourjs.
  • Update native package imports across source, add-ons, examples, manuals, generated docs, tests, and migration tooling.
  • Preserve the FOUR namespace and all legacy Three.js compatibility entry points.
  • Add npm repository, issue tracker, homepage, public-access, preview-tag, and CLI metadata.

Release target

  • Package: fourjs@0.185.1-four.0
  • Initial dist-tag: preview
  • Install after publication: npm install fourjs@preview

Verification

  • Rollup build
  • Core lint
  • Native/legacy entry-point parity
  • Migration tests
  • Core browser unit suite: 1326 passed, 1 todo, 0 failed
  • Add-on browser unit suite: 15 passed, 0 failed
  • Example coverage manifest check
  • Tree-shaking builds
  • Clean npm tarball install
  • ESM, CommonJS, add-on, WebGPU, TSL, legacy, and CLI consumer smoke tests
  • LICENSE and NOTICE included in the tarball
  • No stale @fourjs/core specifiers in the tarball

Known limitations

  • The project remains an experimental preview.
  • The screenshot comparison suite is not claimed as passing.
  • Existing CodeQL findings remain open for separate review.
  • Optional full add-on/example lint reports inherited upstream issues on unrelated lines; the protected core lint gate passes.

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

📦 Bundle size

Full ESM build, minified and gzipped.

Before After Diff
WebGL 365.64
86.89
365.64
86.89
+0 B
+0 B
WebGPU 679.61
188.59
679.61
188.59
+0 B
+0 B
WebGPU Nodes 677.65
188.27
677.65
188.27
+0 B
+0 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

Before After Diff
WebGL 507.69
123.44
507.69
123.44
+0 B
+0 B
WebGPU 747.13
201.45
747.13
201.45
+0 B
+0 B
WebGPU Nodes 697.48
189.04
697.48
189.04
+0 B
+0 B

@tnb1j

tnb1j commented Aug 7, 2026

Copy link
Copy Markdown
Owner Author

CodeQL result review before merge:

  • The CodeQL analysis workflow completed successfully.
  • The separate PR result gate reports 7 alerts because this mechanical package-name migration touches many files.
  • The annotated alerts correspond to existing open alert IDs #5, #16-#20, and #45, all created at 2026-08-07T14:34:10Z, before PR Publish npm package as fourjs #6 was created at 2026-08-07T21:56:25Z.
  • Diff inspection confirms the annotated files change only package import/documentation specifiers from @fourjs/core to fourjs; the flagged regex, message-handler, assignment, and unused-variable lines are unchanged.
  • These alerts remain open and are not dismissed. They require separate security remediation.

Required protected checks passed:

  • Lint, Unit, Unit addons, Circular dependencies & Examples testing
  • Node 26 build & entrypoint parity

@tnb1j
tnb1j merged commit a6ac092 into main Aug 7, 2026
6 of 7 checks passed
@tnb1j
tnb1j deleted the release/npm-fourjs-preview branch August 7, 2026 22:02
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