Merge pull request #2630 from micalevisk/fix-issue-2629

feat: replace 'source-map-support' with nodejs native source mapping binding
nestjs · Jul 5, 2024 · 682e7dd · 682e7dd
2 parents d87405a + b5dba00
commit 682e7dd
Show file tree

Hide file tree

Showing 3 changed files with 321 additions and 68 deletions.
diff --git a/actions/start.action.ts b/actions/start.action.ts
@@ -175,21 +175,11 @@ export class StartAction extends BuildAction {
         typeof debug === 'string' ? `--inspect=${debug}` : '--inspect';
       processArgs.unshift(inspectFlag);
     }
-    const sourceMapsRegisterPath = this.getSourceMapSupportPkg();
-    if (sourceMapsRegisterPath !== undefined) {
-      processArgs.unshift(`-r "${sourceMapsRegisterPath}"`);
-    }
+    processArgs.unshift('--enable-source-maps');
+
     return spawn(binaryToRun, processArgs, {
       stdio: 'inherit',
       shell: true,
     });
   }
-
-  private getSourceMapSupportPkg() {
-    try {
-      return require.resolve('source-map-support/register');
-    } catch {
-      return undefined;
-    }
-  }
 }