Use inspector.url()

Author: timfish

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

@@ -74,11 +74,6 @@ 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();
@@ -104,9 +99,14 @@ const _anrIntegration = ((options: Partial<AnrIntegrationOptions> = {}) => {
         });
       }
     },
-    setup(initClient: NodeClient) {
+    async setup(initClient: NodeClient) {
       client = initClient;
 
+      if (options.captureStackTrace && (await isDebuggerEnabled())) {
+        logger.warn('ANR captureStackTrace has been disabled because the debugger was already enabled');
+        options.captureStackTrace = false;
+      }
+
       // setImmediate is used to ensure that all other integrations have had their setup called first.
       // This allows us to call into all integrations to fetch the full context
       setImmediate(() => this.startWorker());

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

@@ -102,15 +102,16 @@ export const localVariablesAsyncIntegration = defineIntegration(((
 
   return {
     name: 'LocalVariablesAsync',
-    setup(client: NodeClient) {
+    async setup(client: NodeClient) {
       const clientOptions = client.getOptions();
 
       if (!clientOptions.includeLocalVariables) {
         return;
       }
 
-      if (isDebuggerEnabled()) {
-        logger.warn('Local variables capture has been disabled because the debugger is enabled');
+      if (await isDebuggerEnabled()) {
+        logger.warn('Local variables capture has been disabled because the debugger was already enabled');
+        return;
       }
 
       const options: LocalVariablesWorkerArgs = {

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

@@ -290,7 +290,7 @@ const _localVariablesSyncIntegration = ((
 
   return {
     name: INTEGRATION_NAME,
-    setupOnce() {
+    async setupOnce() {
       const client = getClient<NodeClient>();
       const clientOptions = client?.getOptions();
 
@@ -307,8 +307,9 @@ const _localVariablesSyncIntegration = ((
         return;
       }
 
-      if (isDebuggerEnabled()) {
-        logger.warn('Local variables capture has been disabled because the debugger is enabled');
+      if (await isDebuggerEnabled()) {
+        logger.warn('Local variables capture has been disabled because the debugger was already enabled');
+        return;
       }
 
       AsyncSession.create(sessionOverride).then(

packages/node/src/utils/debug.ts

@@ -1,6 +1,18 @@
+let cachedDebuggerEnabled: boolean | undefined;
+
 /**
- * Has the debugger been enabled via the command line?
+ * Was the debugger enabled when this function was first called?
  */
-export function isDebuggerEnabled(): boolean {
-  return process.execArgv.some(arg => arg.startsWith('--inspect'));
+export async function isDebuggerEnabled(): Promise<boolean> {
+  if (cachedDebuggerEnabled === undefined) {
+    try {
+      // Node can be built without inspector support
+      const inspector = await import('node:inspector');
+      cachedDebuggerEnabled = !!inspector.url();
+    } catch (_) {
+      cachedDebuggerEnabled = false;
+    }
+  }
+
+  return cachedDebuggerEnabled;
 }