fix: check required command options

cacjs · Nov 26, 2018 · 93e2bae · 93e2bae
1 parent 50395ec
commit 93e2bae
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 0 deletions.
diff --git a/src/Command.ts b/src/Command.ts
@@ -251,6 +251,24 @@ export default class Command {
       }
     }
   }
+
+  /**
+   * Check if the required string-type options exist
+   * @param values
+   * @param globalCommand
+   */
+  checkRequiredOptions(values: { [k: string]: any }, globalCommand: Command) {
+    const requiredOptions = [...globalCommand.options, ...this.options].filter(
+      option => option.required
+    )
+    for (const option of requiredOptions) {
+      const value = values[option.names[0]]
+      if (value === undefined || typeof value === 'boolean') {
+        console.error(`error: option \`${option.rawName}\` argument is missing`)
+        process.exit(1)
+      }
+    }
+  }
 }
 
 export { HelpCallback, CommandExample, CommandConfig }
diff --git a/src/index.ts b/src/index.ts
@@ -229,6 +229,8 @@ class CAC extends EventEmitter {
 
     command.checkUnknownOptions(originalOptions, globalCommand)
 
+    command.checkRequiredOptions(originalOptions, globalCommand)
+
     const minimalArgsCount = command.args.filter(arg => arg.required).length
 
     if (args.length < minimalArgsCount) {