-
Notifications
You must be signed in to change notification settings - Fork 28
Update deps for August 2025 #2851
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
897772e to
f1459fc
Compare
| |------|-----:|----------| | ||
| | esbuild (js) | 123,964 bytes | dist/js/index.js | | ||
| | esbuild (ts) | 123,964 bytes | dist/ts/index.js | | ||
| | esbuild (js) | 135,295 bytes | dist/js/index.js | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, i wonder what caused an additional 10k+ increase to bundle size 🤔
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, and even weirder Parcel didn't change. I'm wondering if it's something in our configuration for these bundlers? @timostamm any ideas what may be the culprit here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Protobuf-ES 2.6.1 includes an update to google-protobuf v31.1 with a small bump in bundle size (see PR), but that should be much smaller than 10kb.
Would be good to know what's going on here. Can you diff the bundled output against the previous one?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just updated the connect bundle size benchmark, bump was small, as expected: connectrpc/connect-es#1533
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After some investigation, the bump in size is predominantly related to two PRs:
- Fix closed enum parsing bufbuild/protobuf-es#1170 which imports
BinaryWriterfrombinary-encoding.js - Update to google protobuf v31.1 bufbuild/protobuf-es#1168 which updates google protobuf, causing a slight bump in a generated descriptor file.
We have mitigated the effect of 1170 with PR bufbuild/protobuf-es#1197, which, once released, will be pulled in with next month's dependabot updates.
paul-sachs
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No blockers, just curious about the root cause of the bundle size increase
Signed-off-by: Steve Ayers <sayers@buf.build>
Signed-off-by: Steve Ayers <sayers@buf.build>
3ec7f45 to
4b5e113
Compare
Signed-off-by: Timo Stamm <ts@timostamm.de>
SSIA --------- Signed-off-by: Steve Ayers <sayers@buf.build> Signed-off-by: Timo Stamm <ts@timostamm.de> Co-authored-by: Timo Stamm <ts@timostamm.de> Signed-off-by: Steve Ayers <sayers@buf.build>
SSIA