Skip to content

Declare cjs as cjs (part 1)#21414

Merged
NullVoxPopuli merged 10 commits into
mainfrom
nvp/make-cjs-cjs
May 20, 2026
Merged

Declare cjs as cjs (part 1)#21414
NullVoxPopuli merged 10 commits into
mainfrom
nvp/make-cjs-cjs

Conversation

@NullVoxPopuli
Copy link
Copy Markdown
Contributor

There are a lot of cjs files we have in here. This PR is part 1 of 3(?) that actually declares them all as cjs.

This is prereq for unblocking more serious exploration in to #21371

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 19, 2026

📊 Size report

Tarball size1.2 MB1.2 MB

dist/dev   No changes

File Before (Size / Brotli) After (Size / Brotli)
./packages/shared-chunks/template-{hash}.js 1.1 kB / 401 B -54%↓491 B / -49.4%↓203 B
Total (Includes all files) 2 MB / 478.1 kB 2 MB / 478.1 kB

dist/prod   No changes

File Before (Size / Brotli) After (Size / Brotli)
Total (Includes all files) 1.8 MB / 436 kB 1.8 MB / 436 kB

smoke-tests/v2-app-hello-world-template/dist   No changes

File Before (Size / Brotli) After (Size / Brotli)
Total (Includes all files) 235 kB / 64.4 kB 235 kB / 64.4 kB

🤖 This report was automatically generated by wyvox/pkg-size

@NullVoxPopuli NullVoxPopuli marked this pull request as ready for review May 20, 2026 01:12
@NullVoxPopuli NullVoxPopuli requested a review from kategengler May 20, 2026 01:12
Comment thread bin/build-for-publishing.cjs
Comment thread bin/changelog.cjs
Comment thread bin/next-alpha-version.cjs
@NullVoxPopuli NullVoxPopuli requested a review from kategengler May 20, 2026 03:59
@NullVoxPopuli NullVoxPopuli merged commit ee67811 into main May 20, 2026
39 checks passed
@NullVoxPopuli NullVoxPopuli deleted the nvp/make-cjs-cjs branch May 20, 2026 20:32
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