Skip to content

feat(remix): Set url.template on pageload and navigation spans#22025

Open
Lms24 wants to merge 2 commits into
feat/pageload-url-attrs-react-routerfrom
feat/pageload-url-attrs-remix
Open

feat(remix): Set url.template on pageload and navigation spans#22025
Lms24 wants to merge 2 commits into
feat/pageload-url-attrs-react-routerfrom
feat/pageload-url-attrs-remix

Conversation

@Lms24

@Lms24 Lms24 commented Jul 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Applies url.template to pageload and navigation root spans in the Remix client integration, and adds e2e coverage for url.full, url.path and url.template across the Remix test applications.

part of #21921

@Lms24 Lms24 changed the title feat(remix): Set url.template on pageload and navigation spans feat(remix): Set url.template on pageload and navigation spans Jul 7, 2026
@Lms24 Lms24 self-assigned this Jul 7, 2026
@github-actions

github-actions Bot commented Jul 7, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 27.59 kB - -
@sentry/browser - with treeshaking flags 26.03 kB - -
@sentry/browser (incl. Tracing) 46.34 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 48.13 kB - -
@sentry/browser (incl. Tracing, Profiling) 51.12 kB - -
@sentry/browser (incl. Tracing, Replay) 85.61 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 75.26 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 90.32 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 102.97 kB - -
@sentry/browser (incl. Feedback) 44.76 kB - -
@sentry/browser (incl. sendFeedback) 32.38 kB - -
@sentry/browser (incl. FeedbackAsync) 37.51 kB - -
@sentry/browser (incl. Metrics) 28.67 kB - -
@sentry/browser (incl. Logs) 28.91 kB - -
@sentry/browser (incl. Metrics & Logs) 29.59 kB - -
@sentry/react 29.38 kB - -
@sentry/react (incl. Tracing) 48.61 kB - -
@sentry/vue 33.03 kB - -
@sentry/vue (incl. Tracing) 48.24 kB - -
@sentry/svelte 27.61 kB - -
CDN Bundle 30 kB - -
CDN Bundle (incl. Tracing) 48.32 kB - -
CDN Bundle (incl. Logs, Metrics) 31.57 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 49.64 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 70.81 kB - -
CDN Bundle (incl. Tracing, Replay) 85.84 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 87.14 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 91.64 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 92.92 kB - -
CDN Bundle - uncompressed 89.35 kB - -
CDN Bundle (incl. Tracing) - uncompressed 146.1 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 94.05 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 150.07 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 218.75 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 265.27 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 269.23 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 278.97 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 282.92 kB - -
@sentry/nextjs (client) 51.16 kB - -
@sentry/sveltekit (client) 46.75 kB - -
@sentry/core/server 78.42 kB - -
@sentry/core/browser 64.74 kB - -
@sentry/node-core 62.72 kB -0.01% -1 B 🔽
@sentry/node 124.6 kB +0.01% +1 B 🔺
@sentry/node (incl. diagnostics channel injection) 134.02 kB - -
@sentry/node/import (ESM hook with diagnostics-channel injection) 69.95 kB - -
@sentry/node/light 50.72 kB - -
@sentry/node - without tracing 74.05 kB -0.01% -1 B 🔽
@sentry/aws-serverless 85.5 kB - -
@sentry/cloudflare (withSentry) - minified 181.69 kB - -
@sentry/cloudflare (withSentry) 449.12 kB - -

View base workflow run

@Lms24 Lms24 marked this pull request as ready for review July 8, 2026 07:35
@Lms24 Lms24 requested a review from a team as a code owner July 8, 2026 07:35
@Lms24 Lms24 requested review from chargome, mydea and nicohrubec and removed request for a team July 8, 2026 07:35
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from d451d27 to 0a8c24f Compare July 8, 2026 07:36
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 30325b3 to 481776a Compare July 8, 2026 07:36
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from 0a8c24f to e084ddd Compare July 8, 2026 09:29
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 481776a to 03a93e9 Compare July 8, 2026 09:29
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from e084ddd to de83057 Compare July 8, 2026 12:29
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch 2 times, most recently from 49a362b to bf7c7d2 Compare July 8, 2026 13:11
@Lms24 Lms24 requested a review from a team as a code owner July 8, 2026 13:11
@Lms24 Lms24 requested review from logaretm and removed request for a team July 8, 2026 13:11
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch 2 times, most recently from 4c71cd0 to a573d41 Compare July 8, 2026 14:22
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch 2 times, most recently from 85266e1 to 2265752 Compare July 8, 2026 14:41
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from a573d41 to 2b630c1 Compare July 8, 2026 14:41
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 2265752 to 85266e1 Compare July 8, 2026 14:42
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from 2b630c1 to a573d41 Compare July 8, 2026 14:42
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 85266e1 to 2265752 Compare July 8, 2026 14:43
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from a573d41 to 2b630c1 Compare July 8, 2026 14:43
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 2265752 to 85266e1 Compare July 8, 2026 14:46
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch 2 times, most recently from a573d41 to 9fa0ae8 Compare July 8, 2026 14:52
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 85266e1 to 02a2e89 Compare July 8, 2026 14:52
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from 9fa0ae8 to 8103d6f Compare July 8, 2026 14:57
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 02a2e89 to 3142788 Compare July 8, 2026 14:57
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from 8103d6f to d5cc776 Compare July 8, 2026 15:00
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 3142788 to 3b539ca Compare July 8, 2026 15:00
Lms24 and others added 2 commits July 8, 2026 18:07
@Lms24 @cursoragent
feat(react-router): Set url.template, url.path and url.full on pagelo…
5f175c1 
…ad and navigation spans

Extracted from the framework-wide url.template PR into its own PR since
the react-router framework integration needed additional navigation span
and trailing-slash fixes beyond the other framework SDKs.

part of #21921

Co-Authored-By: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@Lms24 @cursoragent
feat(remix): Set url.template on pageload and navigation spans
bb2b5af 
Applies `url.template` to pageload and navigation root spans in the Remix
client integration, and adds e2e coverage across the Remix test
applications.

part of #21921

Co-Authored-By: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Cursor <cursoragent@cursor.com>
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-react-router branch from 3b539ca to f841e71 Compare July 8, 2026 16:08
@Lms24 Lms24 force-pushed the feat/pageload-url-attrs-remix branch from d5cc776 to bb2b5af Compare July 8, 2026 16:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chargome chargome chargome approved these changes
@nicohrubec nicohrubec nicohrubec approved these changes
@s1gr1d s1gr1d s1gr1d approved these changes
@mydea mydea Awaiting requested review from mydea mydea is a code owner automatically assigned from getsentry/team-javascript-sdks-framework
@logaretm logaretm Awaiting requested review from logaretm logaretm was automatically assigned from getsentry/team-javascript-sdks-browser

Assignees

@Lms24 Lms24

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Lms24 @chargome @nicohrubec @s1gr1d