update op

Author: chargome

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

@@ -12,7 +12,7 @@ import {
 } from '@sentry/core';
 import type * as reactRouter from 'react-router';
 import { DEBUG_BUILD } from '../../common/debug-build';
-import { getSpanName, isDataRequest } from './util';
+import { getOpName, getSpanName, isDataRequest } from './util';
 
 type ReactRouterModuleExports = typeof reactRouter;
 
@@ -87,7 +87,7 @@ export class ReactRouterInstrumentation extends InstrumentationBase<Instrumentat
                   attributes: {
                     [SEMANTIC_ATTRIBUTE_SENTRY_SOURCE]: 'url',
                     [SEMANTIC_ATTRIBUTE_SENTRY_ORIGIN]: 'auto.http.react-router',
-                    [SEMANTIC_ATTRIBUTE_SENTRY_OP]: 'function.react-router.loader',
+                    [SEMANTIC_ATTRIBUTE_SENTRY_OP]: getOpName(url.pathname, request.method),
                     url: url.pathname,
                     method: request.method,
                   },

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

@@ -1,3 +1,16 @@
+/**
+ * Gets the op name for a request based on whether it's a loader or action request.
+ * @param pathName The URL pathname to check
+ * @param requestMethod The HTTP request method
+ */
+export function getOpName(pathName: string, requestMethod: string): string {
+  return isLoaderRequest(pathName, requestMethod)
+    ? 'function.react-router.loader'
+    : isActionRequest(pathName, requestMethod)
+      ? 'function.react-router.action'
+      : 'function.react-router';
+}
+
 /**
  * Gets the span name for a request based on whether it's a loader or action request.
  * @param pathName The URL pathname to check