fix: option with brackets should be set to string type

fix #34

Author: egoist

src/Option.ts

@@ -7,8 +7,9 @@ interface OptionConfig {
 
 export default class Option {
   names: string[]
-  isBoolean: boolean
-  required: boolean
+  isBoolean?: boolean
+  // `required` will be a boolean for options with brackets
+  required?: boolean
   config: OptionConfig
 
   constructor(

src/__test__/index.test.ts

@@ -1,5 +1,6 @@
 import path from 'path'
 import execa from 'execa'
+import cac from '..'
 
 function fixture(file: string) {
   return path.relative(
@@ -46,3 +47,18 @@ snapshotOutput({
   file: 'ignore-default-value.js',
   args: ['build']
 })
+
+test('negated option', () => {
+  const cli = cac()
+
+  cli.option('--foo [foo]', 'Set foo').option('--no-foo', 'Disable foo')
+
+  cli.option('--no-bar', 'Disable bar')
+
+  const { options } = cli.parse(['node', 'bin', '--foo', 'foo'])
+  expect(options).toEqual({
+    '--': [],
+    foo: 'foo',
+    bar: true
+  })
+})

src/utils.ts

@@ -65,7 +65,12 @@ export const getMriOptions = (globalCommand: Command, subCommand?: Command) => {
         res[option.names[0]] = option.names.slice(1)
       }
       return res
-    }, {})
+    }, {}),
+    string: options
+      .filter(option => typeof option.required === 'boolean')
+      .reduce((res: string[], option) => {
+        return res.concat(option.names)
+      }, [])
   }
 }