@sentry/browser size

[vkrol]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• other:

Version:

4.0.2

Description

The size of the @sentry/browser is more than twice as large as the size of raven-js: 86 kB vs 39 kB (minified). In my opinion, this is definitely a regression and the serious reason not to update to the new version.

[HazAT]

Hey, thanks for bringing this up.
While we understand and generally agree with your concerns about bundle size I think it's fair to first compare the gzip bundle sizes instead of the uncompressed minified file size:

@sentry/browser is 21.3799 KB
raven-js 13.44 KB

Also, while this might not be applicable to everyone, we provide and usually guide people to use our CDN Loader which will set up the SDK for you on your website.

see : https://docs.sentry.io/quickstart/?platform=browser

The footprint and impact on your pageload time of this script it <1KB gzipped while keeping the same functionality.

so tl;dr:
We are aware of it, we know there is room for improvement but it's not top priority right now.

[klaemo]

I think it's fair to first compare the gzip bundle sizes instead of the uncompressed minified file size:

I would argue it is also fair to compare minified sizes as perfomance issues not only arise from downloading javascript, but also from parsing and execution. ~92kb is quite hefty and could add up to 1s of parse time on low-end devices (just for this one library!).

I'm not sure where you take the number of < 1KB for the CDN'ed script from. Could you elaborate? When I open https://browser.sentry-cdn.com/4.0.4/bundle.min.js I see a gzipped size of 22KB.

You should be aware that sentry's sdk is just one of many libraries developers include.

PS: I love sentry, it's been super helpful for us. Web performance is just something I'm passionate about. ;)

[HazAT]

@klaemo
Hehe, no worries 😅

Like I said, we are aware and it's not like we don't want it to be smaller.
Highest prio for us was to ship the new SDK, we will work to improve the bundle size over time.
There are many more steps we can take, e.g.: use tslib, combine strings ...
So there is a lot of space for improvements.

Sorry, the link I posted before is already outdated 🙃
I was referring to the Loader: https://docs.sentry.io/platforms/javascript/loader/
While the Loader also has it's limitations due it's async nature and in the end, it still fetches and injects the SDK (even if it's async) it's an alternative we offer because people asked for it.

[rayzru]

@HazAT Sorry, guys, but can you provide next version release date?
I mean, there are some changes already at the #master branch but not released yet. That one is deciding is 4x version is usable for Angular5+ projects though.

[HazAT]

@rayzru Just released 4.0.5, but please keep posts related to the topic.

[oliviertassinari]

In my opinion, this is definitely a regression and the serious reason not to update to the new version.

💯 I was just about to upgrade up until I noticed that:

• before: https://bundlephobia.com/result?p=raven-js@3.27.0
• after: https://bundlephobia.com/result?p=@sentry/browser@4.0.1

At least the package size is smaller, but I think that ⚠️ +10 KB of gzipped JavaScript in a bundle is significant. We will wait, keep up the good work :)

[hlehmann]

@HazAT Could it be possible to generate esm files. It would allow webpack to have better result with concatenation and tree shaking.

[jacobrask]

You might want to add some CI tool to track the bundle size of the package for each merge request. Since this issue it has actually grown to 100 kB, see https://bundlephobia.com/result?p=@sentry/browser@4.3.0

Try for instance https://github.com/siddharthkp/bundlesize

The default performance budget for an entry point in Webpack is 250 kb to make sure you get decent performance on any device and network. 100 kb of Sentry takes up quite a lot of that budget.

[jacobrask]

Please keep us updated if fixing this regression is on the roadmap at all, or if the library will just keep growing without a size budget.

We are paying customers and heavily invested in Sentry on both backend, native and web, but an upgrade to over 3x the size of the library (raven-js@3.23.1 is 31 kB) is not something we can justify.

[oliviertassinari]

@jacobrask You can stick to the older version of the library, it's what we do at https://www.onepixel.com/, it's working fine 👌.

[HazAT]

@jacobrask It's definitely on our list and we also think there are some low hanging fruits where we can easily reduce our bundle size.
More and more people asking for this so we will probably tackle this sooner than expected.

[gaterking]

@HazAT
Sentry browser SDK rollup bundle can be optimized. In bundle min js file, there are many tslib code repeatd. Such as __generator, __assign. In Browser env, it's best to share one code. Butthe browser project use another packages dist file. Maybe reduce the gzip size from 23K to 16K.