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

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

@@ -52,6 +52,35 @@ const ANR_EVENT = {
   },
 };
 
+const ANR_EVENT_WITHOUT_STACKTRACE = {
+  // Ensure we have context
+  contexts: {
+    device: {
+      arch: expect.any(String),
+    },
+    app: {
+      app_start_time: expect.any(String),
+    },
+    os: {
+      name: expect.any(String),
+    },
+    culture: {
+      timezone: expect.any(String),
+    },
+  },
+  // and an exception that is our ANR
+  exception: {
+    values: [
+      {
+        type: 'ApplicationNotResponding',
+        value: 'Application Not Responding for at least 100 ms',
+        mechanism: { type: 'ANR' },
+        stacktrace: {},
+      },
+    ],
+  },
+};
+
 const ANR_EVENT_WITH_SCOPE = {
   ...ANR_EVENT,
   user: {
@@ -98,11 +127,11 @@ conditionalTest({ min: 16 })('should report ANR when event loop blocked', () =>
     createRunner(__dirname, 'indefinite.mjs').withMockSentryServer().expect({ event: ANR_EVENT }).start(done);
   });
 
-  test('With --inspect', done => {
+  test("With --inspect the debugger isn't used", done => {
     createRunner(__dirname, 'basic.mjs')
       .withMockSentryServer()
       .withFlags('--inspect')
-      .expect({ event: ANR_EVENT_WITH_SCOPE })
+      .expect({ event: ANR_EVENT_WITHOUT_STACKTRACE })
       .start(done);
   });
 

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

@@ -14,6 +14,7 @@ import {
 } from '@sentry/core';
 import { NODE_VERSION } from '../../nodeVersion';
 import type { NodeClient } from '../../sdk/client';
+import { isDebuggerEnabled } from '../../utils/debug';
 import type { AnrIntegrationOptions, WorkerStartData } from './common';
 
 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

@@ -2,6 +2,7 @@ import { Worker } from 'node:worker_threads';
 import type { Event, EventHint, Exception, IntegrationFn } from '@sentry/core';
 import { defineIntegration, logger } from '@sentry/core';
 import type { NodeClient } from '../../sdk/client';
+import { isDebuggerEnabled } from '../../utils/debug';
 import type { FrameVariables, LocalVariablesIntegrationOptions, LocalVariablesWorkerArgs } from './common';
 import { LOCAL_VARIABLES_KEY, functionNamesMatch } from './common';
 
@@ -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

@@ -3,6 +3,7 @@ import type { Event, Exception, IntegrationFn, StackFrame, StackParser } from '@
 import { LRUMap, defineIntegration, getClient, logger } from '@sentry/core';
 import { NODE_MAJOR } from '../../nodeVersion';
 import type { NodeClient } from '../../sdk/client';
+import { isDebuggerEnabled } from '../../utils/debug';
 import type {
   FrameVariables,
   LocalVariablesIntegrationOptions,
@@ -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'));
+}