refactor(code): reorganize test structure to test/ directory

- Moved all tests from src/__tests__/ to test/unit/ and test/integration/
- Created new utils package with extracted utilities:
  - parseArgs function with bug fix for flag value parsing
  - Platform detection utilities
- Fixed CLI -v/--version and -h/--help flag handling
- Updated package.json test script to bun test ./test
- Reorganized test structure: unit tests in test/unit/, integration tests in test/integration/
- All 87 tests passing with new structure

Author: amondnet

packages/code/package.json

@@ -28,7 +28,7 @@
     "dev": "bun --watch run src/cli.ts",
     "build": "echo 'Build handled by root build:npm'",
     "typecheck": "tsc -p tsconfig.json --noEmit",
-    "test": "bun test ./src"
+    "test": "bun test ./test"
   },
   "dependencies": {
     "@pleaseai/code-format": "workspace:*",

packages/code/src/__tests__/cli.test.ts

@@ -17,6 +17,7 @@ import process from 'node:process'
 import { Format } from '@pleaseai/code-format'
 import pkg from '../package.json'
 import { runLSPDiagnostics } from './hooks/lsp'
+import { parseArgs } from './utils'
 
 const VERSION = pkg.version
 
@@ -31,38 +32,6 @@ interface HookInput {
   tool_use_id: string
 }
 
-interface ParsedArgs {
-  command: string
-  args: string[]
-  flags: Record<string, string | boolean>
-}
-
-function parseArgs(argv: string[]): ParsedArgs {
-  const args = argv.slice(2)
-  const flags: Record<string, string | boolean> = {}
-  const positional: string[] = []
-
-  for (let i = 0; i < args.length; i++) {
-    const arg = args[i]!
-    if (arg.startsWith('--')) {
-      const [key, value] = arg.slice(2).split('=')
-      flags[key!] = value ?? true
-    }
-    else if (arg.startsWith('-')) {
-      flags[arg.slice(1)] = true
-    }
-    else {
-      positional.push(arg)
-    }
-  }
-
-  return {
-    command: positional[0] ?? 'help',
-    args: positional.slice(1),
-    flags,
-  }
-}
-
 async function readStdinJson(): Promise<HookInput> {
   const chunks: Buffer[] = []
   for await (const chunk of Bun.stdin.stream()) {
@@ -157,6 +126,16 @@ Environment:
 async function main(): Promise<void> {
   const { command, args, flags } = parseArgs(process.argv)
 
+  // Check for version/help flags first (before command routing)
+  if (flags.v || flags.version) {
+    versionCommand()
+    return
+  }
+  if (flags.h || flags.help) {
+    helpCommand()
+    return
+  }
+
   const projectDir
     = (flags.project as string)
       ?? process.env.CODE_PROJECT_PATH
@@ -208,14 +187,10 @@ async function main(): Promise<void> {
     }
 
     case 'version':
-    case '-v':
-    case '--version':
       versionCommand()
       break
 
     case 'help':
-    case '-h':
-    case '--help':
       helpCommand()
       break
 

packages/code/src/cli.ts

@@ -0,0 +1,54 @@
+/**
+ * CLI argument parsing utilities
+ */
+
+export interface ParsedArgs {
+  command: string
+  args: string[]
+  flags: Record<string, string | boolean>
+}
+
+/**
+ * Parse command-line arguments into structured format.
+ *
+ * Supports:
+ * - --flag=value (equals syntax)
+ * - --flag (boolean flag)
+ * - -f (short boolean flag)
+ * - Positional arguments
+ *
+ * @param argv - Process argv array (first 2 elements are node/bun path and script path)
+ * @returns Parsed arguments with command, positional args, and flags
+ */
+export function parseArgs(argv: string[]): ParsedArgs {
+  const args = argv.slice(2)
+  const flags: Record<string, string | boolean> = {}
+  const positional: string[] = []
+
+  for (let i = 0; i < args.length; i++) {
+    const arg = args[i]!
+    if (arg.startsWith('--')) {
+      const eqIndex = arg.indexOf('=')
+      if (eqIndex > -1) {
+        const key = arg.slice(2, eqIndex)
+        const value = arg.slice(eqIndex + 1)
+        flags[key] = value
+      }
+      else {
+        flags[arg.slice(2)] = true
+      }
+    }
+    else if (arg.startsWith('-')) {
+      flags[arg.slice(1)] = true
+    }
+    else {
+      positional.push(arg)
+    }
+  }
+
+  return {
+    command: positional[0] ?? 'help',
+    args: positional.slice(1),
+    flags,
+  }
+}

packages/code/src/utils/args.ts

@@ -0,0 +1,6 @@
+/**
+ * Utility modules for @pleaseai/code
+ */
+
+export { parseArgs, type ParsedArgs } from './args'
+export { getPotentialBinaryPaths, getTarget, isMusl } from './platform'