Merge pull request #336 from MatrixAI/feature-env-command-parsing

Use a single `--` instead of needing double `--` to separate commands

Author: aryanjassal

src/errors.ts

@@ -84,8 +84,8 @@ class ErrorPolykeyCLIUnexpectedError<T> extends ErrorPolykeyCLI<T> {
   exitCode = sysexits.SOFTWARE;
 }
 
-class ErrorPolykeyCLISubprocessFailure<T> extends ErrorPolykeyCLI<T> {
-  static description = 'A subprocess failed to exit gracefully';
+class ErrorPolykeyCLIChildProcessFailure<T> extends ErrorPolykeyCLI<T> {
+  static description = 'A child process failed to exit gracefully';
   exitCode = sysexits.UNKNOWN;
 }
 
@@ -196,7 +196,7 @@ export {
   ErrorPolykeyCLIUncaughtException,
   ErrorPolykeyCLIUnhandledRejection,
   ErrorPolykeyCLIUnexpectedError,
-  ErrorPolykeyCLISubprocessFailure,
+  ErrorPolykeyCLIChildProcessFailure,
   ErrorPolykeyCLIAsynchronousDeadlock,
   ErrorPolykeyCLINodePath,
   ErrorPolykeyCLIClientOptions,

src/secrets/CommandEdit.ts

@@ -120,7 +120,7 @@ class CommandEdit extends CommandPolykey {
           };
           const onError = (e: Error) => {
             cleanup();
-            const error = new errors.ErrorPolykeyCLISubprocessFailure(
+            const error = new errors.ErrorPolykeyCLIChildProcessFailure(
               `Failed to run command '${process.env.EDITOR}'`,
               { cause: e },
             );
@@ -129,7 +129,7 @@ class CommandEdit extends CommandPolykey {
           const onClose = (code: number | null) => {
             cleanup();
             if (code !== 0) {
-              const error = new errors.ErrorPolykeyCLISubprocessFailure(
+              const error = new errors.ErrorPolykeyCLIChildProcessFailure(
                 `Editor exited with code ${code}`,
               );
               reject(error);