Merge pull request #2244 from eabrouwer3/eb/2242-json-cli-option

fix: Prevent need for double escaping strings
nestjs · Sep 21, 2023 · e2e046f · e2e046f
2 parents 484d20a + 519030c
commit e2e046f
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/actions/start.action.ts b/actions/start.action.ts
@@ -157,7 +157,13 @@ export class StartAction extends BuildAction {
     let childProcessArgs: string[] = [];
     const argsStartIndex = process.argv.indexOf('--');
     if (argsStartIndex >= 0) {
-      childProcessArgs = process.argv.slice(argsStartIndex + 1);
+      // Prevents the need for users to double escape strings
+      // i.e. I can run the more natural
+      //   nest start -- '{"foo": "bar"}'
+      // instead of
+      //   nest start -- '\'{"foo": "bar"}\''
+      childProcessArgs = process.argv.slice(argsStartIndex + 1)
+        .map(arg => JSON.stringify(arg));
     }
     outputFilePath =
       outputFilePath.indexOf(' ') >= 0 ? `"${outputFilePath}"` : outputFilePath;