Skip to content

feat: Add @sentry-internal/browser-utils #11381

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

Merged
merged 2 commits into from
Apr 5, 2024
Merged

Conversation

AbhiPrasad
Copy link
Member

@AbhiPrasad AbhiPrasad commented Apr 2, 2024

This PR creates a browser-utils package that contains browser specific utils that can be shared between the browser, replay, and feedback packages.

Eventually we will move browserTracingIntegration outside of this package, but the instrumentation handlers will remain here. We will also move the browser related utils in @sentry/utils to this package.

Given this removes @sentry-internal/tracing entirely, it is a part of #9885

ref #9832

@AbhiPrasad AbhiPrasad requested review from s1gr1d, a team and mydea and removed request for a team April 2, 2024 17:19
@AbhiPrasad AbhiPrasad self-assigned this Apr 2, 2024
Copy link
Contributor

github-actions bot commented Apr 2, 2024

size-limit report 📦

Path Size
@sentry/browser (incl. Tracing, Replay, Feedback) 76.26 KB (0%)
@sentry/browser (incl. Tracing, Replay) 67.53 KB (0%)
@sentry/browser (incl. Tracing, Replay with Canvas) 71.35 KB (0%)
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 61.08 KB (0%)
@sentry/browser (incl. Tracing) 32.19 KB (0%)
@sentry/browser (incl. browserTracingIntegration) 32.19 KB (0%)
@sentry/browser (incl. feedbackIntegration) 31.3 KB (0%)
@sentry/browser (incl. feedbackModalIntegration) 31.43 KB (0%)
@sentry/browser (incl. feedbackScreenshotIntegration) 31.43 KB (0%)
@sentry/browser (incl. sendFeedback) 27.37 KB (0%)
@sentry/browser 22.52 KB (0%)
CDN Bundle (incl. Tracing, Replay, Feedback) 71.24 KB (-0.06% 🔽)
CDN Bundle (incl. Tracing, Replay) 65.99 KB (+0.01% 🔺)
CDN Bundle (incl. Tracing) 32.31 KB (+0.02% 🔺)
CDN Bundle 23.74 KB (0%)
CDN Bundle (incl. Tracing, Replay) - uncompressed 205.57 KB (-0.01% 🔽)
CDN Bundle (incl. Tracing) - uncompressed 96.28 KB (-0.01% 🔽)
CDN Bundle - uncompressed 70.05 KB (0%)
@sentry/react (incl. Tracing, Replay) 67.52 KB (0%)
@sentry/react 22.56 KB (0%)

@AbhiPrasad AbhiPrasad force-pushed the abhi-browser-utils branch from 5c0b74c to c6128f1 Compare April 3, 2024 15:43
@getsentry getsentry deleted a comment from codecov bot Apr 3, 2024
@AbhiPrasad AbhiPrasad force-pushed the abhi-browser-utils branch from c6128f1 to 4cb9912 Compare April 3, 2024 20:05
@AbhiPrasad AbhiPrasad force-pushed the abhi-browser-utils branch from e10f027 to 6058f06 Compare April 4, 2024 19:06
Copy link
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Nice, looking forward to when browserTracing becomes part of browser. Makes much more sense this way!
Looks fine publish config-wise 🤞

@AbhiPrasad AbhiPrasad merged commit 3f9cd43 into develop Apr 5, 2024
@AbhiPrasad AbhiPrasad deleted the abhi-browser-utils branch April 5, 2024 15:22
AbhiPrasad added a commit that referenced this pull request Apr 8, 2024
ref #9832

Building off #11381,
this moves browser related utils into the browser utils package.

Next step is to move `browserTracingIntegration` into browser!
cadesalaberry pushed a commit to cadesalaberry/sentry-javascript that referenced this pull request Apr 19, 2024
This PR creates a `browser-utils` package that contains browser specific
utils that can be shared between the browser, replay, and feedback
packages.

Eventually we will move `browserTracingIntegration` outside of this
package, but the instrumentation handlers will remain here. We will also
move the browser related utils in `@sentry/utils` to this package.

Given this removes `@sentry-internal/tracing` entirely, it is a part of
getsentry#9885

ref getsentry#9832
cadesalaberry pushed a commit to cadesalaberry/sentry-javascript that referenced this pull request Apr 19, 2024
…#11451)

ref getsentry#9832

Building off getsentry#11381,
this moves browser related utils into the browser utils package.

Next step is to move `browserTracingIntegration` into browser!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants