fix: Use NODE_OPTIONS=--enable-source-maps only localy

Author: ServerlessLife

src/infraDeploy.ts

@@ -559,19 +559,9 @@ function getEnvironmentVarablesForDebugger(
   functionId: string,
   timeout: number | undefined,
 ): Record<string, string> {
-  const baseNodeOptions = '--enable-source-maps';
-  const additionalNodeOptions =
-    Configuration.config.additionalNodeOptions || '';
-  const nodeOptions = [baseNodeOptions, additionalNodeOptions]
-    .filter(Boolean)
-    .join(' ');
-
-  console.log('nodeOptions', nodeOptions);
-
   return {
     LLD_FUNCTION_ID: functionId,
     AWS_LAMBDA_EXEC_WRAPPER: '/opt/lld-wrapper',
-    NODE_OPTIONS: nodeOptions,
     LLD_DEBUGGER_ID: Configuration.config.debuggerId,
     LLD_INITIAL_TIMEOUT: timeout ? timeout.toString() : '-1', // should never be negative
     LLD_OBSERVABLE_MODE: Configuration.config.observable ? 'true' : 'false',

src/nodeWorker.ts

@@ -34,12 +34,15 @@ async function runInWorker(input: {
     );
 
     if (!worker) {
+      const environment = input.environment;
+      addEnableSourceMapsToEnv(environment);
+
       worker = startWorker({
         handler: func.handler ?? 'handler',
         artifactFile: input.artifactFile,
         workerId: input.fuctionRequest.workerId,
         functionId: input.fuctionRequest.functionId,
-        environment: input.environment,
+        environment,
         verbose: Configuration.config.verbose,
       });
       worker.used = false;
@@ -85,6 +88,20 @@ async function runInWorker(input: {
   });
 }
 
+/**
+ * Add NODE_OPTIONS: --enable-source-maps to the environment variables
+ * @param environment
+ */
+function addEnableSourceMapsToEnv(environment: {
+  [key: string]: string | undefined;
+}) {
+  const nodeOptions = environment.NODE_OPTIONS || '';
+  if (!nodeOptions.includes('--enable-source-maps')) {
+    environment.NODE_OPTIONS =
+      nodeOptions + (nodeOptions ? ' ' : '') + '--enable-source-maps';
+  }
+}
+
 type WorkerRequest = {
   handler: string;
   artifactFile: string;

src/types/lldConfig.ts

@@ -68,11 +68,6 @@ export type LldConfigBase = {
    * Monorepo subfolder
    */
   subfolder?: string;
-
-  /**
-   * Additional Node options
-   */
-  additionalNodeOptions?: string;
 } & AwsConfiguration;
 
 export type LldConfigCliArgs = {

test/utils/expectInfraDeployed.ts

@@ -15,7 +15,6 @@ export async function expectInfraDeployed(lambdaName: any) {
       LLD_DEBUGGER_ID: expect.any(String),
       LLD_FUNCTION_ID: expect.any(String),
       LLD_INITIAL_TIMEOUT: expect.any(String),
-      NODE_OPTIONS: '--enable-source-maps',
     });
 
     const initialTimeout = parseInt(