Fix -C suggestion.

Author: simi

crates/rb-cli/src/commands/shell_integration.rs

@@ -68,10 +68,15 @@ _rb_completion() {{
     # Call rb to get context-aware completions
     local completions
     completions=$(rb __bash_complete "${{COMP_LINE}}" "${{COMP_POINT}}" 2>/dev/null)
-    
+
     if [ -n "$completions" ]; then
+        # Only use nospace when actively navigating through a directory path
+        # (i.e., when current word ends with /)
+        if [[ "$cur" =~ /$ ]]; then
+            compopt -o nospace
+        fi
+
         COMPREPLY=($(compgen -W "$completions" -- "$cur"))
-        # Bash will automatically add space for single completion
     else
         # No rb completions, fall back to default bash completion (files/dirs)
         compopt -o default

spec/behaviour/directory_completion_nospace_spec.sh

@@ -0,0 +1,45 @@
+#!/bin/bash
+# ShellSpec tests for directory completion nospace behavior
+# Tests that bash completion for directories behaves correctly with trailing slashes
+
+Describe "Ruby Butler Directory Completion Nospace"
+  Include spec/support/helpers.sh
+
+  Describe "bash completion script behavior"
+    It "should add space after completing a partial directory name (allows next argument)"
+      # When completing "rb -C sp<TAB>" -> "rb -C spec/ "
+      # Bash SHOULD add a space because user might want to continue with next arg
+      # The completion function detects $cur doesn't end with / yet
+
+      Skip "Manual test: source completion, type 'rb -C sp' then TAB"
+
+      # Expected: "rb -C spec/ " (with space)
+      # This allows: "rb -C spec/ run" or other commands
+    End
+
+    It "should NOT add space when navigating within directory path (allows subdirectory completion)"
+      # When completing "rb -C spec/<TAB>" -> suggests subdirs
+      # Bash should NOT add space because $cur ends with /
+      # This allows continued navigation: "rb -C spec/commands/<TAB>"
+
+      Skip "Manual test: source completion, type 'rb -C spec/' then TAB"
+
+      # Expected: suggests "spec/behaviour/", "spec/commands/", "spec/support/"
+      # Then "rb -C spec/commands/" (no space) allows further TAB completion
+    End
+  End
+
+  Describe "completion output correctness"
+    It "outputs directory names with trailing slash"
+      When run rb __bash_complete "rb -C sp" 9
+      The output should include "spec/"
+    End
+
+    It "outputs subdirectories with full path and trailing slash"
+      When run rb __bash_complete "rb -C spec/" 13
+      The output should include "spec/behaviour/"
+      The output should include "spec/commands/"
+      The output should include "spec/support/"
+    End
+  End
+End

spec/commands/completion/path_completion_spec.sh

@@ -1,6 +1,10 @@
 #!/bin/bash
 # ShellSpec tests for path-based completion
 # Tests directory and file completion for path-based flags
+#
+# Note: These tests verify the completion OUTPUT (what rb __bash_complete returns).
+# The bash completion function behavior (adding/not adding space) is controlled by
+# the generated bash script which uses compopt -o nospace when $cur ends with /
 
 Describe "Ruby Butler Path Completion"
   Include spec/support/helpers.sh
@@ -29,6 +33,20 @@ Describe "Ruby Butler Path Completion"
         The status should equal 0
         The first line of output should not equal "runtime"
       End
+
+      It "completes partial directory path and suggests subdirectories"
+        When run rb __bash_complete "rb -C sp" 9
+        The status should equal 0
+        The output should include "spec/"
+      End
+
+      It "suggests subdirectories after completing a directory"
+        When run rb __bash_complete "rb -C spec/" 13
+        The status should equal 0
+        The output should include "spec/behaviour/"
+        The output should include "spec/commands/"
+        The output should include "spec/support/"
+      End
     End
 
     Context "-G flag (gem-home) completion"