fix(react-router): Use underscores in trace origin values (#18351)

Resolves: #18350

Fixes server validation errors for `contexts.trace.origin` values in the
React Router SDK.

---------

Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>

Author: onurtemizkan

dev-packages/e2e-tests/test-applications/react-router-7-framework-custom/tests/performance/navigation.client.test.ts

@@ -20,12 +20,12 @@ test.describe('client - navigation performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.navigation.react-router',
+            'sentry.origin': 'auto.navigation.react_router',
             'sentry.op': 'navigation',
             'sentry.source': 'route',
           },
           op: 'navigation',
-          origin: 'auto.navigation.react-router',
+          origin: 'auto.navigation.react_router',
         },
       },
       spans: expect.any(Array),
@@ -71,12 +71,12 @@ test.describe('client - navigation performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.navigation.react-router',
+            'sentry.origin': 'auto.navigation.react_router',
             'sentry.op': 'navigation',
             'sentry.source': 'route',
           },
           op: 'navigation',
-          origin: 'auto.navigation.react-router',
+          origin: 'auto.navigation.react_router',
         },
       },
       spans: expect.any(Array),

dev-packages/e2e-tests/test-applications/react-router-7-framework-custom/tests/performance/pageload.client.test.ts

@@ -18,12 +18,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),
@@ -68,12 +68,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),
@@ -119,12 +119,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
     });

dev-packages/e2e-tests/test-applications/react-router-7-framework-custom/tests/performance/performance.server.test.ts

@@ -19,11 +19,11 @@ test.describe('server - performance', () => {
           trace_id: expect.any(String),
           data: {
             'sentry.op': 'http.server',
-            'sentry.origin': 'auto.http.react-router.request-handler',
+            'sentry.origin': 'auto.http.react_router.request_handler',
             'sentry.source': 'route',
           },
           op: 'http.server',
-          origin: 'auto.http.react-router.request-handler',
+          origin: 'auto.http.react_router.request_handler',
         },
       },
       spans: expect.any(Array),
@@ -70,11 +70,11 @@ test.describe('server - performance', () => {
           trace_id: expect.any(String),
           data: {
             'sentry.op': 'http.server',
-            'sentry.origin': 'auto.http.react-router.request-handler',
+            'sentry.origin': 'auto.http.react_router.request_handler',
             'sentry.source': 'route',
           },
           op: 'http.server',
-          origin: 'auto.http.react-router.request-handler',
+          origin: 'auto.http.react_router.request_handler',
         },
       },
       spans: expect.any(Array),
@@ -122,7 +122,7 @@ test.describe('server - performance', () => {
             span_id: expect.any(String),
             trace_id: expect.any(String),
             op: 'http.server',
-            origin: 'auto.http.react-router.loader',
+            origin: 'auto.http.react_router.loader',
             parent_span_id: expect.any(String),
             status: 'ok',
             data: expect.objectContaining({
@@ -133,7 +133,7 @@ test.describe('server - performance', () => {
               'http.target': '/performance/server-loader.data',
               'http.url': 'http://localhost:3030/performance/server-loader.data',
               'sentry.op': 'http.server',
-              'sentry.origin': 'auto.http.react-router.loader',
+              'sentry.origin': 'auto.http.react_router.loader',
               'sentry.source': 'url',
               url: 'http://localhost:3030/performance/server-loader.data',
             }),
@@ -152,7 +152,7 @@ test.describe('server - performance', () => {
       span_id: expect.any(String),
       trace_id: expect.any(String),
       data: {
-        'sentry.origin': 'auto.http.react-router.loader',
+        'sentry.origin': 'auto.http.react_router.loader',
         'sentry.op': 'function.react-router.loader',
       },
       description: 'Executing Server Loader',
@@ -161,7 +161,7 @@ test.describe('server - performance', () => {
       timestamp: expect.any(Number),
       status: 'ok',
       op: 'function.react-router.loader',
-      origin: 'auto.http.react-router.loader',
+      origin: 'auto.http.react_router.loader',
     });
   });
 
@@ -182,7 +182,7 @@ test.describe('server - performance', () => {
             span_id: expect.any(String),
             trace_id: expect.any(String),
             op: 'http.server',
-            origin: 'auto.http.react-router.action',
+            origin: 'auto.http.react_router.action',
             parent_span_id: expect.any(String),
             status: 'ok',
             data: expect.objectContaining({
@@ -193,7 +193,7 @@ test.describe('server - performance', () => {
               'http.target': '/performance/server-action.data',
               'http.url': 'http://localhost:3030/performance/server-action.data',
               'sentry.op': 'http.server',
-              'sentry.origin': 'auto.http.react-router.action',
+              'sentry.origin': 'auto.http.react_router.action',
               'sentry.source': 'url',
               url: 'http://localhost:3030/performance/server-action.data',
             }),
@@ -212,7 +212,7 @@ test.describe('server - performance', () => {
       span_id: expect.any(String),
       trace_id: expect.any(String),
       data: {
-        'sentry.origin': 'auto.http.react-router.action',
+        'sentry.origin': 'auto.http.react_router.action',
         'sentry.op': 'function.react-router.action',
       },
       description: 'Executing Server Action',
@@ -221,7 +221,7 @@ test.describe('server - performance', () => {
       timestamp: expect.any(Number),
       status: 'ok',
       op: 'function.react-router.action',
-      origin: 'auto.http.react-router.action',
+      origin: 'auto.http.react_router.action',
     });
   });
 });

dev-packages/e2e-tests/test-applications/react-router-7-framework-node-20-18/tests/performance/navigation.client.test.ts

@@ -20,12 +20,12 @@ test.describe('client - navigation performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.navigation.react-router',
+            'sentry.origin': 'auto.navigation.react_router',
             'sentry.op': 'navigation',
             'sentry.source': 'route',
           },
           op: 'navigation',
-          origin: 'auto.navigation.react-router',
+          origin: 'auto.navigation.react_router',
         },
       },
       spans: expect.any(Array),
@@ -71,12 +71,12 @@ test.describe('client - navigation performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.navigation.react-router',
+            'sentry.origin': 'auto.navigation.react_router',
             'sentry.op': 'navigation',
             'sentry.source': 'route',
           },
           op: 'navigation',
-          origin: 'auto.navigation.react-router',
+          origin: 'auto.navigation.react_router',
         },
       },
       spans: expect.any(Array),

dev-packages/e2e-tests/test-applications/react-router-7-framework-node-20-18/tests/performance/pageload.client.test.ts

@@ -18,12 +18,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),
@@ -68,12 +68,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),
@@ -119,12 +119,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
     });

dev-packages/e2e-tests/test-applications/react-router-7-framework-node-20-18/tests/performance/performance.server.test.ts

@@ -19,11 +19,11 @@ test.describe('server - performance', () => {
           trace_id: expect.any(String),
           data: {
             'sentry.op': 'http.server',
-            'sentry.origin': 'auto.http.react-router.request-handler',
+            'sentry.origin': 'auto.http.react_router.request_handler',
             'sentry.source': 'route',
           },
           op: 'http.server',
-          origin: 'auto.http.react-router.request-handler',
+          origin: 'auto.http.react_router.request_handler',
         },
       },
       spans: expect.any(Array),
@@ -70,11 +70,11 @@ test.describe('server - performance', () => {
           trace_id: expect.any(String),
           data: {
             'sentry.op': 'http.server',
-            'sentry.origin': 'auto.http.react-router.request-handler',
+            'sentry.origin': 'auto.http.react_router.request_handler',
             'sentry.source': 'route',
           },
           op: 'http.server',
-          origin: 'auto.http.react-router.request-handler',
+          origin: 'auto.http.react_router.request_handler',
         },
       },
       spans: expect.any(Array),
@@ -122,7 +122,7 @@ test.describe('server - performance', () => {
             span_id: expect.any(String),
             trace_id: expect.any(String),
             op: 'http.server',
-            origin: 'auto.http.react-router.server',
+            origin: 'auto.http.react_router.server',
             parent_span_id: expect.any(String),
             status: 'ok',
             data: expect.objectContaining({
@@ -133,7 +133,7 @@ test.describe('server - performance', () => {
               'http.target': '/performance/server-loader.data',
               'http.url': 'http://localhost:3030/performance/server-loader.data',
               'sentry.op': 'http.server',
-              'sentry.origin': 'auto.http.react-router.server',
+              'sentry.origin': 'auto.http.react_router.server',
               'sentry.source': 'url',
               url: 'http://localhost:3030/performance/server-loader.data',
             }),
@@ -154,11 +154,11 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.op': 'function.react-router.loader',
-        'sentry.origin': 'auto.http.react-router.server',
+        'sentry.origin': 'auto.http.react_router.server',
       },
       description: 'Executing Server Loader',
       op: 'function.react-router.loader',
-      origin: 'auto.http.react-router.server',
+      origin: 'auto.http.react_router.server',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       status: 'ok',
@@ -183,7 +183,7 @@ test.describe('server - performance', () => {
             span_id: expect.any(String),
             trace_id: expect.any(String),
             op: 'http.server',
-            origin: 'auto.http.react-router.server',
+            origin: 'auto.http.react_router.server',
             parent_span_id: expect.any(String),
             status: 'ok',
             data: expect.objectContaining({
@@ -194,7 +194,7 @@ test.describe('server - performance', () => {
               'http.target': '/performance/server-action.data',
               'http.url': 'http://localhost:3030/performance/server-action.data',
               'sentry.op': 'http.server',
-              'sentry.origin': 'auto.http.react-router.server',
+              'sentry.origin': 'auto.http.react_router.server',
               'sentry.source': 'url',
               url: 'http://localhost:3030/performance/server-action.data',
             }),
@@ -214,11 +214,11 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.op': 'function.react-router.action',
-        'sentry.origin': 'auto.http.react-router.server',
+        'sentry.origin': 'auto.http.react_router.server',
       },
       description: 'Executing Server Action',
       op: 'function.react-router.action',
-      origin: 'auto.http.react-router.server',
+      origin: 'auto.http.react_router.server',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       status: 'ok',

dev-packages/e2e-tests/test-applications/react-router-7-framework-spa-node-20-18/tests/performance/navigation.client.test.ts

@@ -20,12 +20,12 @@ test.describe('client - navigation performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.navigation.react-router',
+            'sentry.origin': 'auto.navigation.react_router',
             'sentry.op': 'navigation',
             'sentry.source': 'route',
           },
           op: 'navigation',
-          origin: 'auto.navigation.react-router',
+          origin: 'auto.navigation.react_router',
         },
       },
       spans: expect.any(Array),

dev-packages/e2e-tests/test-applications/react-router-7-framework-spa-node-20-18/tests/performance/pageload.client.test.ts

@@ -18,12 +18,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),
@@ -68,12 +68,12 @@ test.describe('client - pageload performance', () => {
           span_id: expect.any(String),
           trace_id: expect.any(String),
           data: {
-            'sentry.origin': 'auto.pageload.react-router',
+            'sentry.origin': 'auto.pageload.react_router',
             'sentry.op': 'pageload',
             'sentry.source': 'route',
           },
           op: 'pageload',
-          origin: 'auto.pageload.react-router',
+          origin: 'auto.pageload.react_router',
         },
       },
       spans: expect.any(Array),