Skip to content

ref(tanstackstart-react): Use @sentry/conventions#21498

Merged
mydea merged 1 commit into
developfrom
feat/use-sentry-conventions-tanstackstart-react
Jun 19, 2026
Merged

ref(tanstackstart-react): Use @sentry/conventions#21498
mydea merged 1 commit into
developfrom
feat/use-sentry-conventions-tanstackstart-react

Conversation

@mydea

@mydea mydea commented Jun 12, 2026

Copy link
Copy Markdown
Member

Switches @sentry/tanstackstart-react to source span/attribute keys from @sentry/conventions, added as a regular runtime dependency and externalized at build time (resolved at runtime by the consumer's installed copy). No functional change — the attribute values are identical.

Part of splitting the larger @sentry/conventions migration into per-package PRs.

Ref: #20982

@mydea mydea force-pushed the feat/use-sentry-conventions-tanstackstart-react branch from 8bc39c1 to bcbaee6 Compare June 12, 2026 11:24
@mydea mydea changed the base branch from develop to feat/use-sentry-conventions-build-infra June 12, 2026 11:26
@mydea mydea force-pushed the feat/use-sentry-conventions-build-infra branch from 625467a to 2a83a9f Compare June 12, 2026 11:33
@mydea mydea force-pushed the feat/use-sentry-conventions-tanstackstart-react branch from bcbaee6 to 4389650 Compare June 15, 2026 08:11
@mydea mydea changed the base branch from feat/use-sentry-conventions-build-infra to develop June 15, 2026 08:11
@github-actions

github-actions Bot commented Jun 15, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 27.45 kB - -
@sentry/browser - with treeshaking flags 25.88 kB - -
@sentry/browser (incl. Tracing) 45.88 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.11 kB - -
@sentry/browser (incl. Tracing, Profiling) 50.65 kB - -
@sentry/browser (incl. Tracing, Replay) 85.08 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 74.68 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 89.78 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 102.44 kB - -
@sentry/browser (incl. Feedback) 44.62 kB - -
@sentry/browser (incl. sendFeedback) 32.25 kB - -
@sentry/browser (incl. FeedbackAsync) 37.38 kB - -
@sentry/browser (incl. Metrics) 28.52 kB - -
@sentry/browser (incl. Logs) 28.76 kB - -
@sentry/browser (incl. Metrics & Logs) 29.45 kB - -
@sentry/react 29.25 kB - -
@sentry/react (incl. Tracing) 48.17 kB - -
@sentry/vue 32.56 kB - -
@sentry/vue (incl. Tracing) 47.74 kB - -
@sentry/svelte 27.48 kB - -
CDN Bundle 29.86 kB - -
CDN Bundle (incl. Tracing) 48.28 kB - -
CDN Bundle (incl. Logs, Metrics) 31.4 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 49.58 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 70.71 kB - -
CDN Bundle (incl. Tracing, Replay) 85.61 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 86.88 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 91.46 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.71 kB - -
CDN Bundle - uncompressed 88.8 kB - -
CDN Bundle (incl. Tracing) - uncompressed 146.04 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 93.5 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 150.02 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 218.33 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 264.91 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 268.87 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 278.61 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 282.56 kB - -
@sentry/nextjs (client) 50.58 kB - -
@sentry/sveltekit (client) 46.27 kB - -
@sentry/core/server 76.16 kB - -
@sentry/core/browser 63.31 kB - -
@sentry/node-core 61.84 kB - -
@sentry/node 127.65 kB - -
@sentry/node - without tracing 74.22 kB +0.01% +1 B 🔺
@sentry/aws-serverless 85.33 kB -0.01% -1 B 🔽
@sentry/cloudflare (withSentry) - minified 174.48 kB - -
@sentry/cloudflare (withSentry) 436.52 kB - -

View base workflow run

@mydea mydea force-pushed the feat/use-sentry-conventions-tanstackstart-react branch 3 times, most recently from 19eb7a1 to 87b30f7 Compare June 17, 2026 09:09
@mydea mydea marked this pull request as ready for review June 17, 2026 09:13
@mydea mydea requested a review from a team as a code owner June 17, 2026 09:13
@mydea mydea requested review from chargome and s1gr1d and removed request for a team June 17, 2026 09:13
Source span/attribute keys from `@sentry/conventions` as a regular dependency
(externalized at build time, resolved at runtime). No functional change.

Maps the `@sentry/conventions/attributes` subpath in the shared TS config for the
repo's `node` moduleResolution type builds.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@mydea mydea force-pushed the feat/use-sentry-conventions-tanstackstart-react branch from 87b30f7 to 3bd7d9d Compare June 18, 2026 09:53
@mydea mydea requested review from isaacs and nicohrubec June 18, 2026 13:56

@nicohrubec nicohrubec left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no lock file changes?

@mydea

mydea commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

no lock file changes?

nope, the sentry conventions are already in the lockfile!

@mydea mydea merged commit a520447 into develop Jun 19, 2026
46 checks passed
@mydea mydea deleted the feat/use-sentry-conventions-tanstackstart-react branch June 19, 2026 07:51
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