chore(react-router): Use snake_case for span op names

Author: onurtemizkan

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

@@ -153,14 +153,14 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.origin': 'auto.http.react_router.loader',
-        'sentry.op': 'function.react-router.loader',
+        'sentry.op': 'function.react_router.loader',
       },
       description: 'Executing Server Loader',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       timestamp: expect.any(Number),
       status: 'ok',
-      op: 'function.react-router.loader',
+      op: 'function.react_router.loader',
       origin: 'auto.http.react_router.loader',
     });
   });
@@ -213,14 +213,14 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.origin': 'auto.http.react_router.action',
-        'sentry.op': 'function.react-router.action',
+        'sentry.op': 'function.react_router.action',
       },
       description: 'Executing Server Action',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       timestamp: expect.any(Number),
       status: 'ok',
-      op: 'function.react-router.action',
+      op: 'function.react_router.action',
       origin: 'auto.http.react_router.action',
     });
   });

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

@@ -153,11 +153,11 @@ test.describe('server - performance', () => {
       span_id: expect.any(String),
       trace_id: expect.any(String),
       data: {
-        'sentry.op': 'function.react-router.loader',
+        'sentry.op': 'function.react_router.loader',
         'sentry.origin': 'auto.http.react_router.server',
       },
       description: 'Executing Server Loader',
-      op: 'function.react-router.loader',
+      op: 'function.react_router.loader',
       origin: 'auto.http.react_router.server',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
@@ -213,11 +213,11 @@ test.describe('server - performance', () => {
       span_id: expect.any(String),
       trace_id: expect.any(String),
       data: {
-        'sentry.op': 'function.react-router.action',
+        'sentry.op': 'function.react_router.action',
         'sentry.origin': 'auto.http.react_router.server',
       },
       description: 'Executing Server Action',
-      op: 'function.react-router.action',
+      op: 'function.react_router.action',
       origin: 'auto.http.react_router.server',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),

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

@@ -122,14 +122,14 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.origin': 'auto.http.react_router',
-        'sentry.op': 'function.react-router.loader',
+        'sentry.op': 'function.react_router.loader',
       },
       description: 'Executing Server Loader',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       timestamp: expect.any(Number),
       status: 'ok',
-      op: 'function.react-router.loader',
+      op: 'function.react_router.loader',
       origin: 'auto.http.react_router',
     });
   });
@@ -150,14 +150,14 @@ test.describe('server - performance', () => {
       trace_id: expect.any(String),
       data: {
         'sentry.origin': 'auto.http.react_router',
-        'sentry.op': 'function.react-router.action',
+        'sentry.op': 'function.react_router.action',
       },
       description: 'Executing Server Action',
       parent_span_id: expect.any(String),
       start_timestamp: expect.any(Number),
       timestamp: expect.any(Number),
       status: 'ok',
-      op: 'function.react-router.action',
+      op: 'function.react_router.action',
       origin: 'auto.http.react_router',
     });
   });

packages/react-router/src/server/instrumentation/util.ts

@@ -5,10 +5,10 @@
  */
 export function getOpName(pathName: string, requestMethod: string): string {
   return isLoaderRequest(pathName, requestMethod)
-    ? 'function.react-router.loader'
+    ? 'function.react_router.loader'
     : isActionRequest(pathName, requestMethod)
-      ? 'function.react-router.action'
-      : 'function.react-router';
+      ? 'function.react_router.action'
+      : 'function.react_router';
 }
 
 /**

packages/react-router/src/server/wrapServerAction.ts

@@ -67,7 +67,7 @@ export function wrapServerAction<T>(options: SpanOptions = {}, actionFn: (args:
           ...options,
           attributes: {
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.action',
-            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.action',
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.action',
             ...options.attributes,
           },
         },

packages/react-router/src/server/wrapServerLoader.ts

@@ -67,7 +67,7 @@ export function wrapServerLoader<T>(options: SpanOptions = {}, loaderFn: (args:
           ...options,
           attributes: {
             [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.loader',
-            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.loader',
+            [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.loader',
             ...options.attributes,
           },
         },

packages/react-router/test/server/wrapServerAction.test.ts

@@ -31,7 +31,7 @@ describe('wrapServerAction', () => {
         name: 'Executing Server Action',
         attributes: {
           [core.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.action',
-          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.action',
+          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.action',
         },
       },
       expect.any(Function),
@@ -61,7 +61,7 @@ describe('wrapServerAction', () => {
         name: 'Custom Action',
         attributes: {
           [core.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.action',
-          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.action',
+          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.action',
           'sentry.custom': 'value',
         },
       },

packages/react-router/test/server/wrapServerLoader.test.ts

@@ -31,7 +31,7 @@ describe('wrapServerLoader', () => {
         name: 'Executing Server Loader',
         attributes: {
           [core.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.loader',
-          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.loader',
+          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.loader',
         },
       },
       expect.any(Function),
@@ -61,7 +61,7 @@ describe('wrapServerLoader', () => {
         name: 'Custom Loader',
         attributes: {
           [core.SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react_router.loader',
-          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.loader',
+          [core.SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react_router.loader',
           'sentry.custom': 'value',
         },
       },