Clear NODE_OPTIONS to avoid node/electron interception issues

Without this, intercepting old versions of node while using a new
desktop app would break, as the server's own options would be used
by the subprocess, which would shutdown if they weren't understood.

Author: pimterry

src/interceptors/terminal/terminal-env-overrides.ts

@@ -67,6 +67,10 @@ export function getTerminalEnvVars(
                 ? `${OVERRIDE_PYTHONPATH}:$PYTHONPATH`
             : currentEnv.PYTHONPATH
                 ? `${OVERRIDE_PYTHONPATH}:${currentEnv.PYTHONPATH}`
-            : OVERRIDE_PYTHONPATH
+            : OVERRIDE_PYTHONPATH,
+
+        // Clear NODE_OPTIONS - it's meant for _us_, not subprocesses.
+        // Otherwise e.g. --max-http-header-size can break old Node/Electron
+        NODE_OPTIONS: ""
     };
 }