#382: classic mode to auto complete relative paths outside of project root

Author: dwash96

cecli/commands/core.py

@@ -165,12 +165,12 @@ def get_raw_completions(self, cmd):
         raw_completer = getattr(self, f"completions_raw_{cmd}", None)
         return raw_completer
 
-    def get_completions(self, cmd):
+    def get_completions(self, cmd, args=""):
         assert cmd.startswith("/")
         cmd = cmd[1:]
         command_class = CommandRegistry.get_command(cmd)
         if command_class:
-            return command_class.get_completions(self.io, self.coder, "")
+            return command_class.get_completions(self.io, self.coder, args)
         return []
 
     def get_commands(self):

cecli/commands/read_only.py

@@ -215,9 +215,14 @@ def get_completions(cls, io, coder, args) -> List[str]:
         if "/" in args:
             # Has directory component
             dir_part, file_part = args.rsplit("/", 1)
-            search_dir = root / dir_part
+            if dir_part == "":
+                search_dir = Path("/")
+                path_prefix = "/"
+            else:
+                # Use os.path.expanduser for ~ support if needed, but Path handles it mostly
+                search_dir = (root / dir_part).resolve()
+                path_prefix = dir_part + "/"
             search_prefix = file_part.lower()
-            path_prefix = dir_part + "/"
         else:
             search_dir = root
             search_prefix = args.lower()
@@ -228,8 +233,9 @@ def get_completions(cls, io, coder, args) -> List[str]:
             if search_dir.exists() and search_dir.is_dir():
                 for entry in search_dir.iterdir():
                     name = entry.name
-                    if search_prefix and search_prefix not in name.lower():
+                    if search_prefix and not name.lower().startswith(search_prefix):
                         continue
+
                     # Add trailing slash for directories
                     if entry.is_dir():
                         completions.append(path_prefix + name + "/")
@@ -238,6 +244,7 @@ def get_completions(cls, io, coder, args) -> List[str]:
         except (PermissionError, OSError):
             pass
 
+        # Also include files already in the chat that match
         add_completions = coder.commands.get_completions("/add")
         for c in add_completions:
             if args.lower() in str(c).lower() and str(c) not in completions:

cecli/commands/read_only_stub.py

@@ -206,7 +206,7 @@ def _add_read_only_directory(
 
     @classmethod
     def get_completions(cls, io, coder, args) -> List[str]:
-        """Get completion options for read-only command."""
+        """Get completion options for read-only-stub command."""
         from pathlib import Path
 
         root = Path(coder.root) if hasattr(coder, "root") else Path.cwd()
@@ -215,9 +215,13 @@ def get_completions(cls, io, coder, args) -> List[str]:
         if "/" in args:
             # Has directory component
             dir_part, file_part = args.rsplit("/", 1)
-            search_dir = root / dir_part
+            if dir_part == "":
+                search_dir = Path("/")
+                path_prefix = "/"
+            else:
+                search_dir = (root / dir_part).resolve()
+                path_prefix = dir_part + "/"
             search_prefix = file_part.lower()
-            path_prefix = dir_part + "/"
         else:
             search_dir = root
             search_prefix = args.lower()
@@ -228,8 +232,9 @@ def get_completions(cls, io, coder, args) -> List[str]:
             if search_dir.exists() and search_dir.is_dir():
                 for entry in search_dir.iterdir():
                     name = entry.name
-                    if search_prefix and search_prefix not in name.lower():
+                    if search_prefix and not name.lower().startswith(search_prefix):
                         continue
+
                     # Add trailing slash for directories
                     if entry.is_dir():
                         completions.append(path_prefix + name + "/")
@@ -238,6 +243,7 @@ def get_completions(cls, io, coder, args) -> List[str]:
         except (PermissionError, OSError):
             pass
 
+        # Also include files already in the chat that match
         add_completions = coder.commands.get_completions("/add")
         for c in add_completions:
             if args.lower() in str(c).lower() and str(c) not in completions:

cecli/io.py

@@ -236,16 +236,10 @@ def get_command_completions(self, document, complete_event, text, words):
             yield from raw_completer(document, complete_event)
             return
 
-        if cmd not in self.command_completions:
-            candidates = self.commands.get_completions(cmd)
-            self.command_completions[cmd] = candidates
-        else:
-            candidates = self.command_completions[cmd]
+        candidates = self.commands.get_completions(cmd, partial)
 
         if candidates is None:
             return
-
-        candidates = [word for word in candidates if partial in word.lower()]
         for candidate in sorted(candidates):
             yield Completion(candidate, start_position=-len(words[-1]))