fix: default option values with same names as args would not parse correctly

Author: chenasraf

src/index.ts

@@ -204,16 +204,18 @@ export class Massarg<Options> {
         const value = option.parse!(tempValue, this.data)
         this._addOptionToData(option, value)
 
-        continue
+        // continue
       }
 
       const command = this._commands.find((o) => o.name === arg || o.aliases?.includes(arg))
 
       if (command) {
         if (this._runCommand) {
-          continue
+          this.data.extras.push(arg)
+          // continue
+        } else {
+          this._runCommand = command
         }
-        this._runCommand = command
       }
 
       if (!option && !command) {

tests/commands.test.ts

@@ -23,4 +23,28 @@ describe("Commands", () => {
       .parse(["c"])
     expect(cmd).toHaveBeenCalledTimes(1)
   })
+
+  test("should consume the first command, then treat the rest as extras", () => {
+    const cmd = jest.fn()
+    const opt = jest.fn()
+    const def = massarg()
+      .command({
+        name: "cmd",
+        aliases: ["c"],
+        run: cmd,
+      })
+      .command({
+        name: "opt",
+        aliases: ["o"],
+        run: opt,
+      })
+
+    const options = def.parseArgs(["cmd", "opt"])
+
+    def.parse(["cmd", "opt"])
+
+    expect(cmd).toHaveBeenCalledTimes(1)
+    expect(opt).not.toHaveBeenCalled()
+    expect(options.extras).toContain("opt")
+  })
 })