-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
@sentry/react & @sentry/tracing 6.19.6 -> 6.19.7 adding 10kB to bundle #5003
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
Comments
Hey @ericgrigorian thanks for bringing this up. I made a quick search w.r.t what might be causing this. The only thing that changed for reacht was #4966 which is a small change was #4966 which is a small change. |
Hey @Lms24, that's correct I'm using both @sentry/tracing and @sentry/react and upgraded both to keep the versions in parity. I'm not sure which one caused the increase but let upgrade them individually and get back to you later today. Looking at the two commits you referenced, I think you're right that tracing may be the issue here. |
Could you tell me how you're bundling your app? Are you using Rollup, Webpack or something else? |
Same issue here, using the same versions, and create-react-app 5 (so webpack) to build |
We'll look into this but currently, our focus is to get v7 out (first beta will be out soon). If going back to 6.19.6 works for you, I suggest doing that for now (or giving our v7 alphas/betas a try). Meanwhile, if you happen to find a fix, PRs are always welcome and appreciated :) |
We shipped v7 with treeshaking, should be resolved. |
Problem Statement
I just updated from 6.19.6 to 6.19.7 and sentry added over 10kB to my entrypoint!! That is incredibly big difference in size. So unfortunately for the sake of performance I downgraded back to 6.19.6.
Solution Brainstorm
Not sure what you can do about it and I know v7 is supposed to help with it but please be a bit more courteous, I work on some apps where 10kB would increase the total size of the app by 10%-15%.
The text was updated successfully, but these errors were encountered: