Skip to content

[wrangler] Stop publishing build metafile - #15398

Merged
james-elicx merged 2 commits into
mainfrom
codex/wrangler-exclude-build-metafile
Sep 1, 2026
Merged

[wrangler] Stop publishing build metafile#15398
james-elicx merged 2 commits into
mainfrom
codex/wrangler-exclude-build-metafile

Conversation

@james-elicx

@james-elicx james-elicx commented Aug 28, 2026

Copy link
Copy Markdown
Member

Wrangler's tsup build generated wrangler-dist/metafile-cjs.json, even though no repository tooling consumes it. Because the entire wrangler-dist directory is published, this build-analysis artifact was included in every npm package.

Disable the unused metafile output so it is never generated in the publish directory.

Measured with pnpm pack before and after the change:

Metric Before After Saving
Unpacked package 64.214 MiB 61.101 MiB 3.113 MiB (4.85%)
npm tarball 10.550 MiB 10.404 MiB 149.7 KiB (1.39%)
Published files 48 47 1 file

A patch changeset is included for the reduction in Wrangler's published package size.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
      • Built Wrangler with pnpm run build --filter wrangler.
      • Packed Wrangler before and after with pnpm pack --json and inspected the tarball contents.
      • Ran node packages/wrangler/bin/wrangler.js --version as a smoke test.
      • Ran oxfmt and oxlint against packages/wrangler/tsup.config.ts.
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: this only removes an unused build-analysis artifact from the npm package.

A picture of a cute animal (not mandatory, but encouraged)

@changeset-bot

changeset-bot Bot commented Aug 28, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: ac12bc2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
wrangler Patch
@cloudflare/vite-plugin Patch
@cloudflare/vitest-plugin Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@ask-bonk

ask-bonk Bot commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

LGTM

github run

@pkg-pr-new

pkg-pr-new Bot commented Aug 28, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@15398

@cloudflare/build-output-utils

npm i https://pkg.pr.new/@cloudflare/build-output-utils@15398

@cloudflare/codemods

npm i https://pkg.pr.new/@cloudflare/codemods@15398

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@15398

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@15398

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@15398

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@15398

miniflare

npm i https://pkg.pr.new/miniflare@15398

@cloudflare/pages-functions

npm i https://pkg.pr.new/@cloudflare/pages-functions@15398

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@15398

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@15398

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@15398

@cloudflare/vitest-plugin

npm i https://pkg.pr.new/@cloudflare/vitest-plugin@15398

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@15398

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@15398

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@15398

wrangler

npm i https://pkg.pr.new/wrangler@15398

commit: ac12bc2

@james-elicx
james-elicx marked this pull request as ready for review August 28, 2026 11:24
@workers-devprod
workers-devprod requested review from a team and emily-shen and removed request for a team August 28, 2026 11:25
@workers-devprod

workers-devprod commented Aug 28, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no bugs or issues to report.

Devin Review
@james-elicx james-elicx added the ci:no-changeset-required Skip pull request checks for a valid changeset label Aug 28, 2026
@james-elicx james-elicx removed the ci:no-changeset-required Skip pull request checks for a valid changeset label Aug 28, 2026
@james-elicx
james-elicx enabled auto-merge (squash) September 1, 2026 15:21

@workers-devprod workers-devprod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codeowners reviews satisfied

@github-project-automation github-project-automation Bot moved this from Untriaged to Approved in workers-sdk Sep 1, 2026
@james-elicx
james-elicx merged commit 1809c5e into main Sep 1, 2026
62 of 67 checks passed
@james-elicx
james-elicx deleted the codex/wrangler-exclude-build-metafile branch September 1, 2026 15:23
@github-project-automation github-project-automation Bot moved this from Approved to Done in workers-sdk Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants