Skip to content

feat(browser): Export getTraceData from the browser sdks #16433

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
May 30, 2025

Conversation

AbhiPrasad
Copy link
Member

resolves #16418

@AbhiPrasad AbhiPrasad requested a review from a team May 30, 2025 13:19
@AbhiPrasad AbhiPrasad self-assigned this May 30, 2025
@AbhiPrasad AbhiPrasad requested review from Lms24 and andreiborza and removed request for a team May 30, 2025 13:19
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.

Should we move this to exports.ts rather than index? Given we export stuff like ?continueTrace from CDN bundles, I think we should also export this one in the bundles

@AbhiPrasad
Copy link
Member Author

Yeah good point! I moved to exports.ts in 5832141

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.

Thanks for fixing this!

@AbhiPrasad AbhiPrasad enabled auto-merge (squash) May 30, 2025 13:32
Copy link
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 23.99 kB - -
@sentry/browser - with treeshaking flags 23.76 kB - -
@sentry/browser (incl. Tracing) 38.29 kB - -
@sentry/browser (incl. Tracing, Replay) 76.43 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 69.55 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 81.19 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 93.27 kB - -
@sentry/browser (incl. Feedback) 40.73 kB - -
@sentry/browser (incl. sendFeedback) 28.7 kB - -
@sentry/browser (incl. FeedbackAsync) 33.59 kB - -
@sentry/react 25.76 kB - -
@sentry/react (incl. Tracing) 40.29 kB - -
@sentry/vue 28.34 kB - -
@sentry/vue (incl. Tracing) 40.13 kB - -
@sentry/svelte 24.01 kB - -
CDN Bundle 25.48 kB +0.7% +175 B 🔺
CDN Bundle (incl. Tracing) 38.48 kB +0.03% +9 B 🔺
CDN Bundle (incl. Tracing, Replay) 74.33 kB +0.02% +8 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) 79.75 kB +0.02% +8 B 🔺
CDN Bundle - uncompressed 74.41 kB +0.7% +512 B 🔺
CDN Bundle (incl. Tracing) - uncompressed 113.89 kB +0.02% +18 B 🔺
CDN Bundle (incl. Tracing, Replay) - uncompressed 227.86 kB +0.01% +18 B 🔺
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 240.69 kB +0.01% +18 B 🔺
@sentry/nextjs (client) 41.96 kB - -
@sentry/sveltekit (client) 38.79 kB - -
@sentry/node 149.87 kB +0.01% +1 B 🔺
@sentry/node - without tracing 98.13 kB +0.01% +1 B 🔺
@sentry/aws-serverless 123.88 kB - -

View base workflow run

@AbhiPrasad AbhiPrasad merged commit bf3b102 into develop May 30, 2025
124 checks passed
@AbhiPrasad AbhiPrasad deleted the abhi-getTraceData-browser branch May 30, 2025 13:46
@nbbeeken
Copy link

nbbeeken commented Jun 2, 2025

Happy Monday, thanks @AbhiPrasad for the quick turnaround here, is it possible this could be published soon? TIA! appreciate the help 🙂

@AbhiPrasad
Copy link
Member Author

Hey @nbbeeken - we'll get a release cut today! There's some other folks waiting on this as well.

To unblock yourself, you can import getTraceData from @sentry/core. The core package is what all of our SDKs depend on under the hood (browser, node, edge, etc.)

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Sentry.getTraceData is not exported from @sentry/react
3 participants