feat(sveltekit): Set url.template, url.path and url.full on pageload and navigation spans

Lms24 · cursoragent · Lms24 · commit 5b4c45f42022 · 2026-07-08T16:52:35.000+02:00
Adds the url.template attribute to SvelteKit pageload and navigation root spans (using the shared @sentry/conventions attribute keys), and passes the absolute navigation URL through to startBrowserTracingNavigationSpan so that url.full and url.path are set as well. part of #21921 Co-Authored-By: Cursor Agent <cursoragent@cursor.com> Co-authored-by: Cursor <cursoragent@cursor.com>
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2-kit-tracing/tests/tracing.client.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2-kit-tracing/tests/tracing.client.test.ts
@@ -36,6 +36,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav1',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav1$/),
+            'url.template': '/nav1',
+          },
         },
       },
     });
@@ -49,6 +54,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+            'url.template': '/',
+          },
         },
       },
     });
@@ -62,6 +72,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav2',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav2$/),
+            'url.template': '/nav2',
+          },
         },
       },
     });
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2-kit-tracing/tests/tracing.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2-kit-tracing/tests/tracing.test.ts
@@ -26,6 +26,11 @@ test('capture a distributed pageload trace', async ({ page }) => {
       trace: {
         op: 'pageload',
         origin: 'auto.pageload.sveltekit',
+        data: {
+          'url.path': '/users/123xyz',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123xyz$/),
+          'url.template': '/users/[id]',
+        },
       },
     },
   });
@@ -88,6 +93,11 @@ test('capture a distributed navigation trace', async ({ page }) => {
       trace: {
         op: 'navigation',
         origin: 'auto.navigation.sveltekit',
+        data: {
+          'url.path': '/users',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users$/),
+          'url.template': '/users',
+        },
       },
     },
   });
@@ -139,6 +149,11 @@ test('record client-side universal load fetch span and trace', async ({ page })
       trace: {
         op: 'navigation',
         origin: 'auto.navigation.sveltekit',
+        data: {
+          'url.path': '/universal-load-fetch',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/universal-load-fetch$/),
+          'url.template': '/universal-load-fetch',
+        },
       },
     },
   });
@@ -212,6 +227,11 @@ test('captures a navigation transaction directly after pageload', async ({ page
       trace: {
         op: 'pageload',
         origin: 'auto.pageload.sveltekit',
+        data: {
+          'url.path': '/',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+          'url.template': '/',
+        },
       },
     },
   });
@@ -228,6 +248,9 @@ test('captures a navigation transaction directly after pageload', async ({ page
           'sentry.sveltekit.navigation.from': '/',
           'sentry.sveltekit.navigation.to': '/users/[id]',
           'sentry.sveltekit.navigation.type': 'link',
+          'url.path': '/users/123abc',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123abc$/),
+          'url.template': '/users/[id]',
         },
       },
     },
@@ -290,6 +313,9 @@ test('captures one navigation transaction per redirect', async ({ page }) => {
           'sentry.sveltekit.navigation.from': '/',
           'sentry.sveltekit.navigation.to': '/redirect1',
           'sentry.sample_rate': 1,
+          'url.path': '/redirect1',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect1$/),
+          'url.template': '/redirect1',
         },
       },
     },
@@ -327,6 +353,9 @@ test('captures one navigation transaction per redirect', async ({ page }) => {
           'sentry.sveltekit.navigation.from': '/',
           'sentry.sveltekit.navigation.to': '/redirect2',
           'sentry.sample_rate': 1,
+          'url.path': '/redirect2',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect2$/),
+          'url.template': '/redirect2',
         },
       },
     },
@@ -364,6 +393,9 @@ test('captures one navigation transaction per redirect', async ({ page }) => {
           'sentry.sveltekit.navigation.from': '/',
           'sentry.sveltekit.navigation.to': '/users/[id]',
           'sentry.sample_rate': 1,
+          'url.path': '/users/789',
+          'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/789$/),
+          'url.template': '/users/[id]',
         },
       },
     },
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2-svelte-5/tests/performance.client.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2-svelte-5/tests/performance.client.test.ts
@@ -36,6 +36,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav1',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav1$/),
+            'url.template': '/nav1',
+          },
         },
       },
     });
@@ -49,6 +54,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+            'url.template': '/',
+          },
         },
       },
     });
@@ -62,6 +72,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav2',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav2$/),
+            'url.template': '/nav2',
+          },
         },
       },
     });
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2-svelte-5/tests/performance.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2-svelte-5/tests/performance.test.ts
@@ -27,6 +27,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'pageload',
           origin: 'auto.pageload.sveltekit',
+          data: {
+            'url.path': '/users/123xyz',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123xyz$/),
+            'url.template': '/users/[id]',
+          },
         },
       },
     });
@@ -81,6 +86,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
+          data: {
+            'url.path': '/users',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users$/),
+            'url.template': '/users',
+          },
         },
       },
     });
@@ -132,6 +142,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
+          data: {
+            'url.path': '/universal-load-fetch',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/universal-load-fetch$/),
+            'url.template': '/universal-load-fetch',
+          },
         },
       },
     });
@@ -205,6 +220,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'pageload',
           origin: 'auto.pageload.sveltekit',
+          data: {
+            'url.path': '/',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+            'url.template': '/',
+          },
         },
       },
     });
@@ -221,6 +241,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/users/[id]',
             'sentry.sveltekit.navigation.type': 'link',
+            'url.path': '/users/123abc',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123abc$/),
+            'url.template': '/users/[id]',
           },
         },
       },
@@ -283,6 +306,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/redirect1',
             'sentry.sample_rate': 1,
+            'url.path': '/redirect1',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect1$/),
+            'url.template': '/redirect1',
           },
         },
       },
@@ -320,6 +346,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/redirect2',
             'sentry.sample_rate': 1,
+            'url.path': '/redirect2',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect2$/),
+            'url.template': '/redirect2',
           },
         },
       },
@@ -357,6 +386,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/users/[id]',
             'sentry.sample_rate': 1,
+            'url.path': '/users/789',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/789$/),
+            'url.template': '/users/[id]',
           },
         },
       },
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2/tests/performance.client.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2/tests/performance.client.test.ts
@@ -31,6 +31,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav1',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav1$/),
+            'url.template': '/nav1',
+          },
         },
       },
     });
@@ -44,6 +49,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+            'url.template': '/',
+          },
         },
       },
     });
@@ -57,6 +67,11 @@ test.describe('client-specific performance events', () => {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
           trace_id: expect.stringMatching(/[a-f0-9]{32}/),
+          data: {
+            'url.path': '/nav2',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/nav2$/),
+            'url.template': '/nav2',
+          },
         },
       },
     });
diff --git a/dev-packages/e2e-tests/test-applications/sveltekit-2/tests/performance.test.ts b/dev-packages/e2e-tests/test-applications/sveltekit-2/tests/performance.test.ts
@@ -27,6 +27,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'pageload',
           origin: 'auto.pageload.sveltekit',
+          data: {
+            'url.path': '/users/123xyz',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123xyz$/),
+            'url.template': '/users/[id]',
+          },
         },
       },
     });
@@ -81,6 +86,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
+          data: {
+            'url.path': '/users',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users$/),
+            'url.template': '/users',
+          },
         },
       },
     });
@@ -132,6 +142,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'navigation',
           origin: 'auto.navigation.sveltekit',
+          data: {
+            'url.path': '/universal-load-fetch',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/universal-load-fetch$/),
+            'url.template': '/universal-load-fetch',
+          },
         },
       },
     });
@@ -205,6 +220,11 @@ test.describe('performance events', () => {
         trace: {
           op: 'pageload',
           origin: 'auto.pageload.sveltekit',
+          data: {
+            'url.path': '/',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/$/),
+            'url.template': '/',
+          },
         },
       },
     });
@@ -221,6 +241,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/users/[id]',
             'sentry.sveltekit.navigation.type': 'link',
+            'url.path': '/users/123abc',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/123abc$/),
+            'url.template': '/users/[id]',
           },
         },
       },
@@ -283,6 +306,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/redirect1',
             'sentry.sample_rate': 1,
+            'url.path': '/redirect1',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect1$/),
+            'url.template': '/redirect1',
           },
         },
       },
@@ -320,6 +346,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/redirect2',
             'sentry.sample_rate': 1,
+            'url.path': '/redirect2',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/redirect2$/),
+            'url.template': '/redirect2',
           },
         },
       },
@@ -357,6 +386,9 @@ test.describe('performance events', () => {
             'sentry.sveltekit.navigation.from': '/',
             'sentry.sveltekit.navigation.to': '/users/[id]',
             'sentry.sample_rate': 1,
+            'url.path': '/users/789',
+            'url.full': expect.stringMatching(/^https?:\/\/localhost:\d+\/users\/789$/),
+            'url.template': '/users/[id]',
           },
         },
       },
diff --git a/packages/sveltekit/package.json b/packages/sveltekit/package.json
@@ -53,6 +53,7 @@
   "dependencies": {
     "@sentry/cloudflare": "10.64.0",
     "@sentry/core": "10.64.0",
+    "@sentry/conventions": "^0.15.1",
     "@sentry/node": "10.64.0",
     "@sentry/svelte": "10.64.0",
     "@sentry/vite-plugin": "^5.3.0",
diff --git a/packages/sveltekit/src/client/browserTracingIntegration.ts b/packages/sveltekit/src/client/browserTracingIntegration.ts
diff --git a/packages/sveltekit/test/client/browserTracingIntegration.test.ts b/packages/sveltekit/test/client/browserTracingIntegration.test.ts
