fix: correct extractPositionalArg to not swallow path after boolean --no-ui flag

The arg parser incorrectly set skipNext=true for ALL --flag patterns, causing
code-iq serve --no-ui /repo to silently drop /repo (treating it as the value
of --no-ui). Boolean flags that take no value must not consume the next token.

- Added BOOLEAN_FLAGS set (--no-ui, --help, -h, --version) to CodeIqApplication
- extractPositionalArg now skips skipNext for flags in BOOLEAN_FLAGS
- Added pathNotSwallowedWhenNoUiPrecedesPath test to ServeCommandTest
- Added CodeIqApplicationArgParsingTest with 5 reflection-based unit tests
  for extractPositionalArg edge cases

Co-Authored-By: Paperclip <noreply@paperclip.ing>

Author: aksOps

src/main/java/io/github/randomcodespace/iq/CodeIqApplication.java

@@ -134,9 +134,18 @@ private static String extractFlag(String[] args, String flagName) {
         return null;
     }
 
+    /**
+     * Boolean (no-value) flags for the serve command.
+     * These must NOT consume the next token as their value.
+     */
+    private static final java.util.Set<String> BOOLEAN_FLAGS = java.util.Set.of(
+            "--no-ui", "--help", "-h", "--version"
+    );
+
     /**
      * Extract the first positional argument after the command name.
      * Skips flags (--name value pairs) to find positional args.
+     * Boolean flags (no value) are not allowed to consume the next token.
      */
     private static String extractPositionalArg(String[] args, String command) {
         boolean foundCommand = false;
@@ -151,17 +160,17 @@ private static String extractPositionalArg(String[] args, String command) {
                 continue;
             }
             if (foundCommand) {
-                // Skip --flag value pairs
-                if (arg.startsWith("--") && !arg.contains("=")) {
+                // Skip --flag value pairs, but not boolean flags that take no value
+                if (arg.startsWith("--") && !arg.contains("=") && !BOOLEAN_FLAGS.contains(arg)) {
                     skipNext = true;
                     continue;
                 }
-                if (arg.startsWith("-") && arg.length() == 2) {
+                if (arg.startsWith("-") && arg.length() == 2 && !BOOLEAN_FLAGS.contains(arg)) {
                     skipNext = true; // short flag like -p 8080
                     continue;
                 }
                 if (arg.startsWith("-")) {
-                    continue; // --flag=value or -flag
+                    continue; // --flag=value, boolean flag, or unknown short flag
                 }
                 return arg;
             }

src/test/java/io/github/randomcodespace/iq/CodeIqApplicationArgParsingTest.java

@@ -0,0 +1,52 @@
+package io.github.randomcodespace.iq;
+
+import org.junit.jupiter.api.Test;
+
+import java.lang.reflect.Method;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertNull;
+
+/**
+ * Unit tests for CodeIqApplication argument parsing helper methods.
+ * These are called in main() before the Spring context starts.
+ */
+class CodeIqApplicationArgParsingTest {
+
+    private static String extractPositionalArg(String[] args, String command) throws Exception {
+        Method m = CodeIqApplication.class.getDeclaredMethod("extractPositionalArg", String[].class, String.class);
+        m.setAccessible(true);
+        return (String) m.invoke(null, args, command);
+    }
+
+    @Test
+    void extractsPathAfterCommand() throws Exception {
+        String result = extractPositionalArg(new String[]{"serve", "/my/repo"}, "serve");
+        assertEquals("/my/repo", result);
+    }
+
+    @Test
+    void pathNotSwallowedByBooleanNoUiFlag() throws Exception {
+        // Regression: --no-ui is boolean; must not consume /repo as its value.
+        String result = extractPositionalArg(new String[]{"serve", "--no-ui", "/my/repo"}, "serve");
+        assertEquals("/my/repo", result);
+    }
+
+    @Test
+    void pathStillExtractedWhenPortFlagPrecedes() throws Exception {
+        String result = extractPositionalArg(new String[]{"serve", "--port", "9090", "/my/repo"}, "serve");
+        assertEquals("/my/repo", result);
+    }
+
+    @Test
+    void pathStillExtractedWithNoUiAndPort() throws Exception {
+        String result = extractPositionalArg(new String[]{"serve", "--no-ui", "--port", "9090", "/my/repo"}, "serve");
+        assertEquals("/my/repo", result);
+    }
+
+    @Test
+    void returnsNullWhenNoPath() throws Exception {
+        String result = extractPositionalArg(new String[]{"serve", "--no-ui"}, "serve");
+        assertNull(result);
+    }
+}

src/test/java/io/github/randomcodespace/iq/cli/ServeCommandTest.java

@@ -64,4 +64,14 @@ void noUiFlagIsRecognized() {
         cmdLine.parseArgs("--no-ui");
         assertEquals(true, cmd.isNoUi());
     }
+
+    @Test
+    void pathNotSwallowedWhenNoUiPrecedesPath() {
+        // Regression: --no-ui is boolean and must not consume the next positional arg.
+        var cmd = new ServeCommand();
+        var cmdLine = new CommandLine(cmd);
+        cmdLine.parseArgs("--no-ui", "/some/repo");
+        assertEquals(true, cmd.isNoUi());
+        assertEquals("/some/repo", cmd.getPath().toString());
+    }
 }