remove unnecessary checks, add guard for default isolation scope

Author: Lms24

packages/node/src/integrations/tracing/koa.ts

@@ -1,8 +1,16 @@
 import { registerInstrumentations } from '@opentelemetry/instrumentation';
 import { KoaInstrumentation } from '@opentelemetry/instrumentation-koa';
 import { SemanticAttributes } from '@opentelemetry/semantic-conventions';
-import { captureException, defineIntegration, getCurrentScope, spanToJSON } from '@sentry/core';
+import {
+  captureException,
+  defineIntegration,
+  getDefaultIsolationScope,
+  getIsolationScope,
+  spanToJSON,
+} from '@sentry/core';
 import type { IntegrationFn } from '@sentry/types';
+import { logger } from '@sentry/utils';
+import { DEBUG_BUILD } from '../../debug-build';
 
 const _koaIntegration = (() => {
   return {
@@ -12,13 +20,16 @@ const _koaIntegration = (() => {
         instrumentations: [
           new KoaInstrumentation({
             requestHook(span, info) {
-              if (span.isRecording() && info.layerType === 'router') {
-                const attributes = spanToJSON(span).data;
-                const route = attributes && attributes[SemanticAttributes.HTTP_ROUTE];
-                const method = info.context.request.method.toUpperCase() || 'GET';
-                if (route && method) {
-                  getCurrentScope().setTransactionName(`${method} ${route}`);
-                }
+              if (getIsolationScope() === getDefaultIsolationScope()) {
+                DEBUG_BUILD &&
+                  logger.warn('Isolation scope is default isolation scope - skipping setting transactionName');
+                return;
+              }
+              const attributes = spanToJSON(span).data;
+              const route = attributes && attributes[SemanticAttributes.HTTP_ROUTE];
+              const method = info.context.request.method.toUpperCase() || 'GET';
+              if (route) {
+                getIsolationScope().setTransactionName(`${method} ${route}`);
               }
             },
           }),