Skip to content

ref(browser): Use conventions constants for browser navigation attributes - #24322

Open
logaretm wants to merge 1 commit into
developfrom
awad/browser-navigation-attributes
Open

ref(browser): Use conventions constants for browser navigation attributes#24322
logaretm wants to merge 1 commit into
developfrom
awad/browser-navigation-attributes

Conversation

@logaretm

@logaretm logaretm commented Sep 10, 2026

Copy link
Copy Markdown
Member

Swaps our two locally-defined browser navigation attribute constants for the ones now published in @sentry/conventions 0.23.0.

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

size-limit report 📦

⚠️ Warning: Base artifact is not the latest one, because the latest workflow run is not done yet. This may lead to incorrect results. Try to re-run all tests to get up to date results.

Path Size % Change Change
@sentry/browser 28.92 kB - -
@sentry/browser - with treeshaking flags 27.21 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.11 kB - -
@sentry/browser (incl. Tracing) 50.42 kB -0.02% -6 B 🔽
@sentry/browser (incl. Tracing + Span Streaming) 50.43 kB -0.02% -9 B 🔽
@sentry/browser (incl. Tracing, Profiling) 53.41 kB -0.02% -9 B 🔽
@sentry/browser (incl. Tracing, Replay) 89.97 kB +0.04% +27 B 🔺
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 79.06 kB +0.01% +3 B 🔺
@sentry/browser (incl. Tracing, Replay with Canvas) 94.67 kB +0.03% +26 B 🔺
@sentry/browser (incl. Tracing, Replay, Feedback) 107.65 kB -0.01% -9 B 🔽
@sentry/browser (incl. Feedback) 46.41 kB - -
@sentry/browser (incl. sendFeedback) 33.97 kB - -
@sentry/browser (incl. FeedbackAsync) 39.08 kB - -
@sentry/browser (incl. Metrics) 29.93 kB - -
@sentry/browser (incl. Logs) 30.2 kB - -
@sentry/browser (incl. Metrics & Logs) 30.87 kB - -
@sentry/react 30.67 kB - -
@sentry/react (incl. Tracing) 52.73 kB -0.01% -5 B 🔽
@sentry/vue 36.16 kB - -
@sentry/vue (incl. Tracing) 52.66 kB -0.02% -10 B 🔽
@sentry/svelte 28.93 kB - -
CDN Bundle 30.66 kB - -
CDN Bundle (incl. Tracing) 50.92 kB -0.01% -5 B 🔽
CDN Bundle (incl. Logs, Metrics) 32.93 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 52.91 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 73.62 kB - -
CDN Bundle (incl. Tracing, Replay) 88.46 kB -0.03% -21 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 90.43 kB -0.01% -5 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) 94.53 kB -0.02% -16 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 96.53 kB -0.02% -19 B 🔽
CDN Bundle - uncompressed 90.73 kB - -
CDN Bundle (incl. Tracing) - uncompressed 152.16 kB -0.01% -11 B 🔽
CDN Bundle (incl. Logs, Metrics) - uncompressed 97.31 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 158.12 kB -0.01% -11 B 🔽
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 226.72 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 271.73 kB -0.01% -11 B 🔽
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 277.68 kB -0.01% -11 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 285.43 kB -0.01% -11 B 🔽
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 291.37 kB -0.01% -11 B 🔽
@sentry/nextjs (client) 55.05 kB -0.02% -11 B 🔽
@sentry/sveltekit (client) 50.84 kB +0.01% +1 B 🔺
@sentry/core/server 37.08 kB - -
@sentry/core/browser 13.66 kB - -
@sentry/node 128.59 kB +0.01% +5 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 81.82 kB - -
@sentry/node - without tracing 89.55 kB +0.04% +30 B 🔺
@sentry/node - without channel injection 107.52 kB +0.02% +12 B 🔺
@sentry/aws-serverless 97.77 kB +0.03% +24 B 🔺
@sentry/cloudflare (withSentry) - minified 203.02 kB +0.01% +4 B 🔺
@sentry/cloudflare (withSentry) 505.43 kB -0.01% -4 B 🔽

View base workflow run

Base automatically changed from lms/ref-bump-conventions-0.23.0 to develop September 10, 2026 18:13
@logaretm
logaretm marked this pull request as ready for review September 10, 2026 18:16
@logaretm
logaretm requested review from a team as code owners September 10, 2026 18:16
@logaretm
logaretm requested review from JPeer264, msonnb, mydea and s1gr1d and removed request for a team September 10, 2026 18:16
…utes

Replaces the locally-defined `browser.navigation.type` placeholder and the
`browser.soft_navigation.id` attribute with `BROWSER_NAVIGATION_TYPE` and
`BROWSER_NAVIGATION_ID` from `@sentry/conventions`.
@logaretm
logaretm force-pushed the awad/browser-navigation-attributes branch from 969f6b8 to 9ebbdb9 Compare September 10, 2026 18:16
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