Auto complete works on empty input

Author: dwash96

cecli/io.py

@@ -148,7 +148,6 @@ def __init__(
         self.rel_fnames = rel_fnames
         self.encoding = encoding
         self.abs_read_only_fnames = abs_read_only_fnames or []
-        self.post_filter_commands = ["/add"]
 
         fname_to_rel_fnames = defaultdict(list)
         for rel_fname in addable_rel_fnames:
@@ -214,22 +213,47 @@ def tokenize(self):
 
     def get_command_completions(self, document, complete_event, text, words):
         if len(words) == 1 and not text[-1].isspace():
+            # Handle command completion (e.g., typing "/ad" should complete to "/add")
             partial = words[0].lower()
+            # Strip leading '/' if present for comparison with command names
+            if partial.startswith("/"):
+                partial = partial[1:]
             candidates = [cmd for cmd in self.command_names if cmd.startswith(partial)]
             for candidate in sorted(candidates):
-                yield Completion(candidate, start_position=-len(words[-1]))
+                # Add back the leading '/' for the completion
+                yield Completion("/" + candidate, start_position=-len(words[-1]))
             return
 
-        if len(words) <= 1 or text[-1].isspace():
-            return
+        # Handle command followed by space: trigger auto-completion with empty partial
+        if text[-1].isspace():
+            # We have a command followed by space, trigger auto-completion with empty string
+            if len(words) == 1:
+                # Command with no arguments yet, just a trailing space
+                partial = ""
+                # We need to get the command name without the trailing space
+                # The command is words[0] but might have leading '/'
+                cmd_text = words[0]
+            else:
+                # Command with arguments and trailing space
+                partial = ""
+                cmd_text = text.rstrip()  # Remove trailing space for matching
+        else:
+            # No trailing space
+            if len(words) <= 1:
+                return
+            partial = words[-1].lower()
+            cmd_text = text
 
-        cmd = words[0]
-        partial = words[-1].lower()
+        # Pass the text (without trailing space if present) to matching_commands
+        matches, matched_cmd, _ = self.commands.matching_commands(cmd_text.rstrip())
+        if not matches:
+            return
 
-        matches, _, _ = self.commands.matching_commands(cmd)
         if len(matches) == 1:
             cmd = matches[0]
-        elif cmd not in matches:
+        elif matched_cmd in matches:
+            cmd = matched_cmd
+        else:
             return
 
         raw_completer = self.commands.get_raw_completions(cmd)
@@ -242,11 +266,14 @@ def get_command_completions(self, document, complete_event, text, words):
         if candidates is None:
             return
 
-        if cmd in self.post_filter_commands:
-            candidates = [word for word in candidates if partial in word.lower()]
+        candidates = [word for word in candidates if partial in word.lower()]
 
         for candidate in sorted(candidates):
-            yield Completion(candidate, start_position=-len(words[-1]))
+            # Calculate start position based on partial, not words[-1]
+            # When partial is empty (trailing space), start_position should be 0
+            # When partial is not empty, replace that many characters
+            start_position = -len(partial) if partial else 0
+            yield Completion(candidate, start_position=start_position)
 
     def get_completions(self, document, complete_event):
         self.tokenize()
@@ -256,8 +283,9 @@ def get_completions(self, document, complete_event):
         if not words:
             return
 
-        if text and text[-1].isspace():
-            # don't keep completing after a space
+        if text and text[-1].isspace() and not text.startswith("/"):
+            # don't keep completing after a space for non-commands
+            # For commands, we want to allow completion with empty string partial
             return
 
         if text[0] == "/":

cecli/tui/app.py

@@ -884,7 +884,9 @@ def _get_suggestions(self, text: str) -> list[str]:
         suggestions = []
         commands = self.worker.coder.commands
 
-        if len(text) and text[-1] == " ":
+        # Only return early for non-commands ending with space
+        # For commands, we want to allow completion with empty string partial
+        if len(text) and text[-1] == " " and not text.startswith("/"):
             return
 
         if "@" in text:
@@ -905,12 +907,21 @@ def _get_suggestions(self, text: str) -> list[str]:
                     suggestions = all_commands
                 else:
                     suggestions = [c for c in all_commands if c.startswith(cmd_part)]
-            elif len(parts) > 1:
+            else:
                 # Complete command argument
+                # This handles both:
+                # 1. len(parts) > 1: command with arguments
+                # 2. len(parts) == 1 and text.endswith(" "): command with trailing space
                 cmd_name = cmd_part
-                end_lookup = text.rsplit(maxsplit=1)
 
-                arg_prefix = end_lookup[-1]
+                if text.endswith(" "):
+                    # Command with trailing space, empty argument prefix
+                    arg_prefix = ""
+                else:
+                    # Get the last word as argument prefix
+                    end_lookup = text.rsplit(maxsplit=1)
+                    arg_prefix = end_lookup[-1]
+
                 arg_prefix_lower = arg_prefix.lower()
 
                 # Check if this command needs path-based completion
@@ -955,8 +966,14 @@ def _get_completed_text(self, current_text: str, completion: str) -> str:
         """Calculate the new text after applying completion."""
         if current_text.startswith("/"):
             parts = current_text.rsplit(maxsplit=1)
-            if len(parts) == 1:
-                # Replace entire command
+
+            # Check if we have a command with trailing space
+            # This is when we want to insert argument completions after the space
+            if len(parts) == 1 and current_text.endswith(" "):
+                # Command with trailing space, insert completion after space
+                return current_text + completion
+            elif len(parts) == 1:
+                # Replace entire command (command name completion)
                 # Only add space if command takes arguments
                 commands = self.worker.coder.commands
                 try: