fix: Add NODE_OPTIONS for debugging in next-dev

vercel · May 16, 2023 · 4ec16e9 · 4ec16e9
1 parent e5ad96e
commit 4ec16e9
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 6 deletions.
diff --git a/packages/next/src/server/lib/server-ipc/index.ts b/packages/next/src/server/lib/server-ipc/index.ts
@@ -4,6 +4,7 @@ import { genExecArgv, getNodeOptionsWithoutInspect } from '../utils'
 import { deserializeErr, errorToJSON } from '../../render'
 import crypto from 'crypto'
 import isError from '../../../lib/is-error'
+import { genRenderExecArgv } from '../worker-utils'
 
 // we can't use process.send as jest-worker relies on
 // it already and can cause unexpected message errors
@@ -115,10 +116,13 @@ export const createWorker = (
             }
           : {}),
       },
-      execArgv: genExecArgv(
-        isNodeDebugging === undefined ? false : isNodeDebugging,
-        (serverPort || 0) + 1
-      ),
+      execArgv:
+        isNodeDebugging !== undefined
+          ? genExecArgv(
+              isNodeDebugging === undefined ? false : isNodeDebugging,
+              (serverPort || 0) + 1
+            )
+          : genRenderExecArgv(),
     },
     exposedMethods: [
       'initialize',

diff --git a/packages/next/src/server/lib/start-server.ts b/packages/next/src/server/lib/start-server.ts
@@ -1,7 +1,6 @@
 import http from 'http'
 import { isIPv6 } from 'net'
 import * as Log from '../../build/output/log'
-import { getNodeOptionsWithoutInspect } from './utils'
 import type { IncomingMessage, ServerResponse } from 'http'
 import type { ChildProcess } from 'child_process'
 import { normalizeRepeatedSlashes } from '../../shared/lib/utils'
@@ -187,7 +186,8 @@ export async function startServer({
           env: {
             FORCE_COLOR: '1',
             ...((initialEnv || process.env) as typeof process.env),
-            NODE_OPTIONS: getNodeOptionsWithoutInspect().trim(),
+            // We should allow NodeOptions to be passed down for render-server for debugging
+            NODE_OPTIONS: process.env.NODE_OPTIONS,
           },
         },
         exposedMethods: ['initialize'],