fix: check default commands (#152)

* Add tests

* Fix #151

* merge

---------

Co-authored-by: Kevin Deng <sxzz@sxzz.moe>

Author: domdomegg

examples/default-command-inverted.ts

@@ -0,0 +1,11 @@
+import { cac } from '../src/index.ts'
+const cli = cac()
+
+cli
+  .command('something', 'Do something')
+  .alias('!')
+  .action(() => {
+    console.info('Did something!')
+  })
+
+cli.parse()

examples/default-command.ts

@@ -0,0 +1,11 @@
+import { cac } from '../src/index.ts'
+const cli = cac()
+
+cli
+  .command('', 'Do something')
+  .alias('something')
+  .action(() => {
+    console.info('Did something!')
+  })
+
+cli.parse()

src/cac.ts

@@ -217,7 +217,7 @@ export class CAC extends EventTarget {
     if (shouldParse) {
       // Search the default command
       for (const command of this.commands) {
-        if (command.name === '') {
+        if (command.isDefaultCommand) {
           shouldParse = false
           const parsed = this.mri(argv.slice(2), command)
           this.setParsedInfo(parsed, command)

tests/index.test.ts

@@ -186,3 +186,26 @@ describe('--version in help message', () => {
     expect(output).toContain(`--version`)
   })
 })
+
+describe('default commands', () => {
+  test('simple: empty call', async () => {
+    const output = await getOutput('default-command.ts', [])
+    expect(output).toContain('Did something!')
+  })
+
+  test('simple: name alias call', async () => {
+    const output = await getOutput('default-command.ts', ['something'])
+    expect(output).toContain('Did something!')
+  })
+
+  // See https://github.com/cacjs/cac/issues/151
+  test('inverted: empty call', async () => {
+    const output = await getOutput('default-command-inverted.ts', [])
+    expect(output).toContain('Did something!')
+  })
+
+  test('inverted: name alias call', async () => {
+    const output = await getOutput('default-command-inverted.ts', ['something'])
+    expect(output).toContain('Did something!')
+  })
+})