fix: Add NODE_OPTIONS for debugging in next-dev

vercel · May 14, 2023 · 800348a · 800348a
1 parent afddb6e
commit 800348a
Show file tree

Hide file tree

Showing 2 changed files with 11 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,11 +1,11 @@
 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'
 import { initialEnv } from '@next/env'
+import { getNodeOptionsWithoutInspect } from './utils'
 
 export interface StartServerOptions {
   dir: string
@@ -187,7 +187,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
+            // NODE_OPTIONS: getNodeOptionsWithoutInspect().trim(),
           },
         },
         exposedMethods: ['initialize'],