Skip to content

Delete some unused code - #21613

Open
NullVoxPopuli wants to merge 1 commit into
mainfrom
nvp/debug-cleanup
Open

Delete some unused code#21613
NullVoxPopuli wants to merge 1 commit into
mainfrom
nvp/debug-cleanup

Conversation

@NullVoxPopuli

@NullVoxPopuli NullVoxPopuli commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

Deleting some of the debug VM code -- it was cute, but only showed up in errors when certain flags were set -- which were not exposed to our users (or ourselves by default)

@NullVoxPopuli
NullVoxPopuli marked this pull request as ready for review September 10, 2026 22:32
@github-actions

Copy link
Copy Markdown
Contributor

📊 Size report

Tarball size — 1.2 MB → 1.2 MB

dist/dev   No changes

File Before (Size / Brotli) After (Size / Brotli)
./packages/shared-chunks/api-{hash}.js 26.3 kB / 5.9 kB -61.6%↓10.1 kB / -61.5%↓2.3 kB
./packages/shared-chunks/template-{hash}.js 1.1 kB / 401 B -54%↓491 B / -49.4%↓203 B
Total (Includes all files) 2.1 MB / 494 kB 2.1 MB / 494 kB

dist/prod   No changes

File Before (Size / Brotli) After (Size / Brotli)
./packages/shared-chunks/cache-{hash}.js 697 B / 258 B 1,310%↑9.9 kB / 972%↑2.8 kB
Total (Includes all files) 1.9 MB / 451.2 kB 1.9 MB / 451.2 kB

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

File Before (Size / Brotli) After (Size / Brotli)
Total (Includes all files) 344.2 kB / 95.4 kB 344.2 kB / 95.4 kB

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

File Before (Size / Brotli) After (Size / Brotli)
Total (Includes all files) 135.3 kB / 37.8 kB 135.3 kB / 37.8 kB

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

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