[wrangler] Add bundle sizes to structured output - #14915
Conversation
🦋 Changeset detectedLatest commit: b51a5d6 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
Codeowners approval required for this PR:
Show detailed file reviewers
|
dario-piotrowicz
left a comment
There was a problem hiding this comment.
Thanks for the nice improvement @longlho 🙂
I just left a small comment, please let me know what you think (besides this the PR looks good to me)
|
yup makes sense, I did another pass lmk if this looks better |
|
oh wow the merge conflicts 😓 @longlho could you have a look? 🙏 |
18b6ef2 to
b51a5d6
Compare
|
Codeowners approval required for this PR:
Show detailed file reviewers
|
|
yup done :) |
@cloudflare/autoconfig
@cloudflare/build-output-utils
@cloudflare/codemods
@cloudflare/config
create-cloudflare
@cloudflare/deploy-helpers
@cloudflare/kv-asset-handler
miniflare
@cloudflare/pages-functions
@cloudflare/pages-shared
@cloudflare/unenv-preset
@cloudflare/vite-plugin
@cloudflare/vitest-plugin
@cloudflare/workers-auth
@cloudflare/workers-editor-shared
@cloudflare/workers-utils
wrangler
commit: |
dario-piotrowicz
left a comment
There was a problem hiding this comment.
Looks good to me, thanks for the change @longlho 😄
dario-piotrowicz
left a comment
There was a problem hiding this comment.
LGTM 🙂
Thanks again for the PR @longlho 😄 🥳
|
Codeowners approval required for this PR:
Show detailed file reviewers |
workers-devprod
left a comment
There was a problem hiding this comment.
Codeowners reviews satisfied
|
thanks a lot for the review! im looking forward to the new version and getting rid of our local patch 😄 |
wrangler deploy --dry-runcurrently reports bundle size only through human-formatted console output. CI size gates must regex that text, which is brittle.This adds exact raw and gzip-compressed byte counts to the existing structured
deployandversion-uploadoutput records, including dry runs:{ "bundle_size": { "raw_bytes": 1234, "gzip_bytes": 567 } }The bundle reporter now returns the measurements it already calculates, and deploy helpers pass them to Wrangler's output writer.
Note
This is a contribution from an AI agent: Codex, GPT-5.