feat(remix): Add parameterized transaction naming for routes (#17951)

This PR adds parameterized routes for Remix SDK, similarly to the NextJS
SDK

Resolves: #16690
_Needs docs PR for the Vite plugin_

Adds automatic parameterization of Remix route IDs for transaction
naming. Converts internal Remix route file patterns to parameterized
paths similar to the other SDKs (`routes/user.$id` -> `/user/:id`).

- Created `sentryRemixVitePlugin` that extracts route information from
Remix builds and generates a manifest file containing route ID to
parameterized route mappings.
- Added `remixRouteParameterization.ts` with regex-based route
conversion that handles dynamic segments (`$param` → `:param`), splat
routes (`$` → `:*`), and nested routes.
- Updated both client and server instrumentation in `performance.tsx`
and `instrumentServer.ts` to use parameterized transaction names.
- It also works with Hydrogen

Vite plugin is used like:

```javascript
// vite.config.ts
import { defineConfig } from 'vite';
import { vitePlugin as remix } from '@remix-run/dev';
import { sentryRemixVitePlugin } from '@sentry/remix';

export default defineConfig({
  plugins: [
    remix(),
    sentryRemixVitePlugin({
      appDirPath: './app', // Default is 'app' in project root
    }),
  ],
});
```

Updated all Remix E2E test applications to expect parameterized route
names. Added new E2E test application `create-remix-app-v2-non-vite` for
testing backward compatibility for non-Vite builds - these continue to
use `routes/user.$id` format.

  ## Summary

| Route File | Example URL | Before | After (Server - All Apps) | After
(Client - Vite Plugin) |

|------------|-------------|--------|---------------------------|------------------------------|
| `_index.tsx` | `/` | `routes/_index` | `/` | `/` |
| `user.$id.tsx` | `/user/123` | `routes/user.$id` | `/user/:id` |
`/user/:id` |
| `users.$userId.posts.$postId.tsx` | `/users/123/posts/456` |
`routes/users.$userId.posts.$postId` | `/users/:userId/posts/:postId` |
`/users/:userId/posts/:postId` |
| `$.tsx` (splat) | `/docs/guide/intro` | `routes/$` | `/:*` | `/:*` |
| `docs.$.tsx` (scoped splat) | `/docs/guide/intro` | `routes/docs.$` |
`/docs/:*` | `/docs/:*` |

  ### Notes
- **Server-side**: Parameterization works automatically for all Remix
apps (Vite, non-Vite, Hydrogen, Express).
- **Client-side**: Requires the Vite plugin. Without it, client
transactions continue using route IDs (e.g., `routes/user.$id`).

Author: onurtemizkan

.github/workflows/build.yml

@@ -810,7 +810,7 @@ jobs:
     needs: [job_get_metadata, job_build]
     if: needs.job_build.outputs.changed_remix == 'true' || github.event_name != 'pull_request'
     runs-on: ubuntu-24.04
-    timeout-minutes: 10
+    timeout-minutes: 15
     strategy:
       fail-fast: false
       matrix:

dev-packages/e2e-tests/test-applications/create-remix-app-express-vite-dev/tests/client-transactions.test.ts

@@ -3,7 +3,7 @@ import { waitForTransaction } from '@sentry-internal/test-utils';
 
 test('Sends a pageload transaction to Sentry', async ({ page }) => {
   const transactionPromise = waitForTransaction('create-remix-app-express-vite-dev', transactionEvent => {
-    return transactionEvent.contexts?.trace?.op === 'pageload' && transactionEvent.transaction === 'routes/_index';
+    return transactionEvent.contexts?.trace?.op === 'pageload' && transactionEvent.transaction === '/';
   });
 
   await page.goto('/');
@@ -15,7 +15,7 @@ test('Sends a pageload transaction to Sentry', async ({ page }) => {
 
 test('Sends a navigation transaction to Sentry', async ({ page }) => {
   const transactionPromise = waitForTransaction('create-remix-app-express-vite-dev', transactionEvent => {
-    return transactionEvent.contexts?.trace?.op === 'navigation' && transactionEvent.transaction === 'routes/user.$id';
+    return transactionEvent.contexts?.trace?.op === 'navigation' && transactionEvent.transaction === '/user/:id';
   });
 
   await page.goto('/');
@@ -28,6 +28,22 @@ test('Sends a navigation transaction to Sentry', async ({ page }) => {
   expect(transactionEvent).toBeDefined();
 });
 
+test('Sends a navigation transaction with parameterized route to Sentry', async ({ page }) => {
+  const transactionPromise = waitForTransaction('create-remix-app-express-vite-dev', transactionEvent => {
+    return transactionEvent.contexts?.trace?.op === 'navigation';
+  });
+
+  await page.goto('/');
+
+  const linkElement = page.locator('id=navigation');
+  await linkElement.click();
+
+  const transactionEvent = await transactionPromise;
+
+  expect(transactionEvent).toBeDefined();
+  expect(transactionEvent.transaction).toBeTruthy();
+});
+
 test('Renders `sentry-trace` and `baggage` meta tags for the root route', async ({ page }) => {
   await page.goto('/');
 

dev-packages/e2e-tests/test-applications/create-remix-app-express-vite-dev/tests/server-transactions.test.ts

@@ -48,7 +48,7 @@ test('Sends two linked transactions (server & client) to Sentry', async ({ page
   const pageLoadParentSpanId = pageloadTransaction.contexts?.trace?.parent_span_id;
 
   expect(httpServerTransaction.transaction).toBe('GET http://localhost:3030/');
-  expect(pageloadTransaction.transaction).toBe('routes/_index');
+  expect(pageloadTransaction.transaction).toBe('/');
 
   expect(httpServerTraceId).toBeDefined();
   expect(httpServerSpanId).toBeDefined();

dev-packages/e2e-tests/test-applications/create-remix-app-express-vite-dev/vite.config.ts

@@ -1,14 +1,15 @@
+import { installGlobals } from '@remix-run/node';
 import { vitePlugin as remix } from '@remix-run/dev';
+import { sentryRemixVitePlugin } from '@sentry/remix';
 import { defineConfig } from 'vite';
 import tsconfigPaths from 'vite-tsconfig-paths';
 
-import { installGlobals } from '@remix-run/node';
-
 installGlobals();
 
 export default defineConfig({
   plugins: [
     remix(),
+    sentryRemixVitePlugin(),
     tsconfigPaths({
       // The dev server config errors are not relevant to this test app
       // https://github.com/aleclarson/vite-tsconfig-paths?tab=readme-ov-file#options

dev-packages/e2e-tests/test-applications/create-remix-app-express/tests/client-transactions.test.ts

@@ -3,7 +3,7 @@ import { waitForTransaction } from '@sentry-internal/test-utils';
 
 test('Sends a pageload transaction to Sentry', async ({ page }) => {
   const transactionPromise = waitForTransaction('create-remix-app-express', transactionEvent => {
-    return transactionEvent.contexts?.trace?.op === 'pageload' && transactionEvent.transaction === 'routes/_index';
+    return transactionEvent.contexts?.trace?.op === 'pageload' && transactionEvent.transaction === '/';
   });
 
   await page.goto('/');
@@ -15,7 +15,7 @@ test('Sends a pageload transaction to Sentry', async ({ page }) => {
 
 test('Sends a navigation transaction to Sentry', async ({ page }) => {
   const transactionPromise = waitForTransaction('create-remix-app-express', transactionEvent => {
-    return transactionEvent.contexts?.trace?.op === 'navigation' && transactionEvent.transaction === 'routes/user.$id';
+    return transactionEvent.contexts?.trace?.op === 'navigation' && transactionEvent.transaction === '/user/:id';
   });
 
   await page.goto('/');

dev-packages/e2e-tests/test-applications/create-remix-app-express/tests/server-transactions.test.ts

@@ -90,7 +90,7 @@ test('Propagates trace when ErrorBoundary is triggered', async ({ page }) => {
   const pageLoadParentSpanId = pageloadTransaction.contexts?.trace?.parent_span_id;
 
   expect(httpServerTransaction.transaction).toBe('GET client-error');
-  expect(pageloadTransaction.transaction).toBe('routes/client-error');
+  expect(pageloadTransaction.transaction).toBe('/client-error');
 
   expect(httpServerTraceId).toBeDefined();
   expect(httpServerSpanId).toBeDefined();
@@ -132,7 +132,7 @@ test('Sends two linked transactions (server & client) to Sentry', async ({ page
   const pageLoadParentSpanId = pageloadTransaction.contexts?.trace?.parent_span_id;
 
   expect(httpServerTransaction.transaction).toBe('GET http://localhost:3030/');
-  expect(pageloadTransaction.transaction).toBe('routes/_index');
+  expect(pageloadTransaction.transaction).toBe('/');
 
   expect(httpServerTraceId).toBeDefined();
   expect(httpServerSpanId).toBeDefined();

dev-packages/e2e-tests/test-applications/create-remix-app-express/vite.config.ts

@@ -1,14 +1,15 @@
+import { installGlobals } from '@remix-run/node';
 import { vitePlugin as remix } from '@remix-run/dev';
+import { sentryRemixVitePlugin } from '@sentry/remix';
 import { defineConfig } from 'vite';
 import tsconfigPaths from 'vite-tsconfig-paths';
 
-import { installGlobals } from '@remix-run/node';
-
 installGlobals();
 
 export default defineConfig({
   plugins: [
     remix(),
+    sentryRemixVitePlugin(),
     tsconfigPaths({
       // The dev server config errors are not relevant to this test app
       // https://github.com/aleclarson/vite-tsconfig-paths?tab=readme-ov-file#options

dev-packages/e2e-tests/test-applications/create-remix-app-v2-non-vite/.eslintrc.js

@@ -0,0 +1,4 @@
+/** @type {import('eslint').Linter.Config} */
+module.exports = {
+  extends: ['@remix-run/eslint-config', '@remix-run/eslint-config/node'],
+};

dev-packages/e2e-tests/test-applications/create-remix-app-v2-non-vite/.gitignore

@@ -0,0 +1,6 @@
+node_modules
+
+/.cache
+/build
+/public/build
+.env

dev-packages/e2e-tests/test-applications/create-remix-app-v2-non-vite/.npmrc

@@ -0,0 +1,2 @@
+@sentry:registry=http://127.0.0.1:4873
+@sentry-internal:registry=http://127.0.0.1:4873