chore: apply PR feedback

Author: calvinbrewer

packages/wizard/src/__tests__/parse-args.test.ts

@@ -43,6 +43,15 @@ describe('wizard parseArgs — mode resolution', () => {
     expect(result.modeError).toMatch(/Unknown --mode value/)
   })
 
+  it('rejects --mode at the end of argv with a clear error (regression: silent fall-through)', () => {
+    // Without the trailing-value guard, `--mode` as the final arg falls
+    // through to the default `implement` with no diagnostic — strictly
+    // worse than `--mode=` (which already errors). Both forms now surface
+    // the same "requires a value" error.
+    const result = parseArgs(argv('--mode'))
+    expect(result.modeError).toMatch(/--mode requires a value/)
+  })
+
   it('lets the last mode flag win when multiple are passed', () => {
     // Useful for wrappers that always append a mode flag — they don't have
     // to detect and remove an earlier one.

packages/wizard/src/bin/parse-args.ts

@@ -33,8 +33,12 @@ export function parseArgs(argv: string[]): ParsedArgs {
       mode = 'implement'
       continue
     }
-    if (arg === '--mode' && i + 1 < args.length) {
-      const next = args[i + 1] ?? ''
+    if (arg === '--mode') {
+      if (i + 1 >= args.length) {
+        modeError = "--mode requires a value. Expected 'plan' or 'implement'."
+        break
+      }
+      const next = args[i + 1]
       if (next === 'plan' || next === 'implement') {
         mode = next
         i++