fix: argument parsing infinite loop and duplicate handling in claude-yolo

- Fix missing shift statements for --inspect and --ps causing infinite loops
- Remove duplicate argument handling between parse_args() and main case statement
- Add exit 0 for --ps to prevent incorrect continuation to claude.sh
- Fix claude-trace --run-with syntax (remove unnecessary "claude" argument)
- Consolidate all argument parsing through single parse_args() function

Closes #8

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: lroolle

DEV-LOGS.md

@@ -5,6 +5,47 @@
 
 ## Issue Analysis: 2025-06-22
 
+### [bug-critical] Argument parsing infinite loop in claude-yolo
+
+**Problem**: Cursor bot detected critical bugs in claude-yolo argument parsing.
+
+**Root Cause Analysis**:
+
+**Bug 1 - Infinite Loop**: Lines 84-89 in parse_args() missing `shift` statements:
+```bash
+--inspect)
+    inspect_container  # ❌ Missing shift - infinite loop
+;;
+--ps)
+    list_containers   # ❌ Missing shift - infinite loop
+;;
+```
+
+**Bug 2 - Duplicate Handling**: Lines 122-137 duplicate parse_args() logic:
+```bash
+# Main script also handles --inspect/--ps directly
+case "$1" in
+--inspect) inspect_container ;;  # ❌ Duplicate of parse_args
+--ps) list_containers ;;         # ❌ Duplicate + no exit
+```
+
+**Impact**:
+- Infinite loop when using `--inspect` or `--ps`
+- `--ps` shows containers but continues to exec claude.sh
+- Mixed options like `claude-yolo --inspect -v ~/foo:/bar` silently ignore -v
+- Inconsistent behavior between direct calls and mixed arguments
+
+**Technical Details**:
+- **Flow Issue**: parse_args() calls inspect_container() → exits, but missing shift causes loop
+- **Design Flaw**: Two separate parsing paths with different behaviors
+- **Silent Failures**: Some argument combinations work, others don't
+
+**Status**: Critical - requires immediate fix
+
+---
+
+## Issue Analysis: 2025-06-22
+
 ### [problem-discovered] GitHub CLI auth fails in containers
 
 **Problem**: Mounting `~/.config/gh/` doesn't work for GitHub CLI authentication in containers.

claude-yolo

@@ -83,9 +83,12 @@ parse_args() {
             ;;
         --inspect)
             inspect_container
+            # inspect_container calls exec, but add shift for safety
+            shift
             ;;
         --ps)
             list_containers
+            exit 0
             ;;
         --help | -h)
             echo "Claude YOLO - Container Shortcuts"
@@ -119,19 +122,5 @@ parse_args() {
     exec "$CLAUDE_SH" --yolo "${claude_args[@]}"
 }
 
-# Handle special shortcuts first
-case "$1" in
---inspect)
-    inspect_container
-    ;;
---ps)
-    list_containers
-    ;;
---help | -h)
-    # Will be handled by parse_args
-    parse_args "$@"
-    ;;
-*)
-    parse_args "$@"
-    ;;
-esac
+# All argument parsing handled by parse_args
+parse_args "$@"

claude.sh

@@ -645,7 +645,7 @@ if [ "$OPEN_SHELL" = true ]; then
     DOCKER_ARGS+=("/bin/zsh")
 elif [ "$USE_TRACE" = true ]; then
     echo "Using claude-trace for logging"
-    DOCKER_ARGS+=("claude-trace" "--include-all-requests" "--run-with" "claude" "${CLAUDE_ARGS[@]}")
+    DOCKER_ARGS+=("claude-trace" "--include-all-requests" "--run-with" "${CLAUDE_ARGS[@]}")
 else
     DOCKER_ARGS+=("claude" "${CLAUDE_ARGS[@]}")
 fi

docker-entrypoint.sh

@@ -224,19 +224,24 @@ main() {
                 build_gosu_env_cmd "$CLAUDE_USER" "$cmd" "$@" --dangerously-skip-permissions
             elif [ "$cmd" = "claude-trace" ]; then
                 echo "Executing Claude-trace as $CLAUDE_USER with YOLO powers: $cmd $@"
-                # claude-trace --include-all-requests --run-with claude [args]
-                # We need to inject --dangerously-skip-permissions into the claude command
+                # claude-trace --include-all-requests --run-with [args]
+                # We need to inject --dangerously-skip-permissions into the claude arguments
                 args=("$@")
                 new_args=()
                 i=0
+                inject_next=false
                 while [ $i -lt ${#args[@]} ]; do
-                    if [ "${args[$i]}" = "--run-with" ] && [ $((i + 1)) -lt ${#args[@]} ] && [ "${args[$((i + 1))]}" = "claude" ]; then
-                        new_args+=("--run-with" "claude" "--dangerously-skip-permissions")
-                        i=$((i + 2))
+                    if [ "${args[$i]}" = "--run-with" ]; then
+                        new_args+=("--run-with")
+                        inject_next=true
+                    elif [ "$inject_next" = true ]; then
+                        # First argument after --run-with gets --dangerously-skip-permissions added
+                        new_args+=("${args[$i]}" "--dangerously-skip-permissions")
+                        inject_next=false
                     else
                         new_args+=("${args[$i]}")
-                        i=$((i + 1))
                     fi
+                    i=$((i + 1))
                 done
                 build_gosu_env_cmd "$CLAUDE_USER" "$cmd" "${new_args[@]}"
             else
@@ -255,19 +260,24 @@ main() {
                 exec "$@" --dangerously-skip-permissions
             elif [ "$cmd" = "claude-trace" ]; then
                 echo "Executing Claude-trace in root mode with YOLO powers: $@"
-                # claude-trace --include-all-requests --run-with claude [args]
-                # We need to inject --dangerously-skip-permissions into the claude command
+                # claude-trace --include-all-requests --run-with [args]
+                # We need to inject --dangerously-skip-permissions into the claude arguments
                 args=("$@")
                 new_args=()
                 i=1 # Skip first arg (claude-trace)
+                inject_next=false
                 while [ $i -lt ${#args[@]} ]; do
-                    if [ "${args[$i]}" = "--run-with" ] && [ $((i + 1)) -lt ${#args[@]} ] && [ "${args[$((i + 1))]}" = "claude" ]; then
-                        new_args+=("--run-with" "claude" "--dangerously-skip-permissions")
-                        i=$((i + 2))
+                    if [ "${args[$i]}" = "--run-with" ]; then
+                        new_args+=("--run-with")
+                        inject_next=true
+                    elif [ "$inject_next" = true ]; then
+                        # First argument after --run-with gets --dangerously-skip-permissions added
+                        new_args+=("${args[$i]}" "--dangerously-skip-permissions")
+                        inject_next=false
                     else
                         new_args+=("${args[$i]}")
-                        i=$((i + 1))
                     fi
+                    i=$((i + 1))
                 done
                 exec "$cmd" "${new_args[@]}"
             else