fix: Disable ANR and Local Variables if debugger is enabled via CLI args

Author: timfish

dev-packages/node-integration-tests/suites/anr/test.ts

@@ -99,6 +99,11 @@ conditionalTest({ min: 16 })('should report ANR when event loop blocked', () =>
   });
 
   test('With --inspect', done => {
+    const ANR_EVENT_NO_STACKTRACE = { ...ANR_EVENT };
+    if (ANR_EVENT_NO_STACKTRACE?.exception?.values?.[0]?.stacktrace) {
+      (ANR_EVENT_NO_STACKTRACE.exception.values[0].stacktrace as any) = {};
+    }
+
     createRunner(__dirname, 'basic.mjs')
       .withMockSentryServer()
       .withFlags('--inspect')

packages/node/src/integrations/anr/index.ts

@@ -15,6 +15,7 @@ import {
 import { NODE_VERSION } from '../../nodeVersion';
 import type { NodeClient } from '../../sdk/client';
 import type { AnrIntegrationOptions, WorkerStartData } from './common';
+import { isDebuggerEnabled } from '../../utils/debug';
 
 const { isPromise } = types;
 
@@ -73,6 +74,11 @@ const _anrIntegration = ((options: Partial<AnrIntegrationOptions> = {}) => {
   let worker: Promise<() => void> | undefined;
   let client: NodeClient | undefined;
 
+  if (isDebuggerEnabled() && options.captureStackTrace) {
+    logger.warn('ANR captureStackTrace has been disabled because the debugger is enabled');
+    options.captureStackTrace = false;
+  }
+
   // Hookup the scope fetch function to the global object so that it can be called from the worker thread via the
   // debugger when it pauses
   const gbl = globalWithScopeFetchFn();

packages/node/src/integrations/local-variables/local-variables-async.ts

@@ -4,6 +4,7 @@ import { defineIntegration, logger } from '@sentry/core';
 import type { NodeClient } from '../../sdk/client';
 import type { FrameVariables, LocalVariablesIntegrationOptions, LocalVariablesWorkerArgs } from './common';
 import { LOCAL_VARIABLES_KEY, functionNamesMatch } from './common';
+import { isDebuggerEnabled } from '../../utils/debug';
 
 // This string is a placeholder that gets overwritten with the worker code.
 export const base64WorkerScript = '###LocalVariablesWorkerScript###';
@@ -108,6 +109,10 @@ export const localVariablesAsyncIntegration = defineIntegration(((
         return;
       }
 
+      if (isDebuggerEnabled()) {
+        logger.warn('Local variables capture has been disabled because the debugger is enabled');
+      }
+
       const options: LocalVariablesWorkerArgs = {
         ...integrationOptions,
         debug: logger.isEnabled(),

packages/node/src/integrations/local-variables/local-variables-sync.ts

@@ -11,6 +11,7 @@ import type {
   Variables,
 } from './common';
 import { createRateLimiter, functionNamesMatch } from './common';
+import { isDebuggerEnabled } from '../../utils/debug';
 
 /** Creates a unique hash from stack frames */
 export function hashFrames(frames: StackFrame[] | undefined): string | undefined {
@@ -306,6 +307,10 @@ const _localVariablesSyncIntegration = ((
         return;
       }
 
+      if (isDebuggerEnabled()) {
+        logger.warn('Local variables capture has been disabled because the debugger is enabled');
+      }
+
       AsyncSession.create(sessionOverride).then(
         session => {
           function handlePaused(

packages/node/src/utils/debug.ts

@@ -0,0 +1,6 @@
+/**
+ * Has the debugger been enabled via the command line?
+ */
+export function isDebuggerEnabled(): boolean {
+  return process.execArgv.some(arg => arg.startsWith('--inspect'));
+}