use file-path completion as fallback

Author: kkashilk

crates/q_cli/src/cli/chat/prompt.rs

@@ -62,71 +62,21 @@ pub fn generate_prompt(current_profile: Option<&str>) -> String {
     "> ".to_string()
 }
 
-/// Categorized path context patterns for better organization and maintainability
-pub struct PathContextPatterns {
-    /// File operation commands (cat, ls, etc.)
-    pub file_commands: &'static [&'static str],
-    /// Words indicating file operations (open, read, etc.)
-    pub file_operations: &'static [&'static str],
-    /// Path-related terms (file, path, directory, etc.)
-    pub path_terms: &'static [&'static str],
-    /// Special commands that work with files
-    pub special_commands: &'static [&'static str],
-}
-
-impl Default for PathContextPatterns {
-    fn default() -> Self {
-        Self {
-            file_commands: &["cat ", "ls ", "cd ", "vim ", "nano ", "less ", "more ", "grep "],
-            file_operations: &["open ", "read ", "write ", "edit ", "create ", "delete ", "remove "],
-            path_terms: &["file ", "path ", "directory ", "folder ", "location "],
-            special_commands: &["/context add", "/context rm"],
-        }
-    }
-}
-
 /// A wrapper around FilenameCompleter that provides enhanced path detection
 /// and completion capabilities for the chat interface.
 pub struct PathCompleter {
     /// The underlying filename completer from rustyline
     filename_completer: FilenameCompleter,
-    path_patterns: PathContextPatterns,
 }
 
 impl PathCompleter {
     /// Creates a new PathCompleter instance
     pub fn new() -> Self {
         Self {
             filename_completer: FilenameCompleter::new(),
-            path_patterns: PathContextPatterns::default(),
         }
     }
 
-    /// Check if the input might be referring to a file path based on context clues
-    fn is_path_context(&self, line: &str) -> bool {
-        // Check for file commands
-        let has_file_command = self.path_patterns.file_commands
-            .iter()
-            .any(|&cmd| line.contains(cmd));
-
-        // Check for file operations
-        let has_file_operation = self.path_patterns.file_operations
-            .iter()
-            .any(|&op| line.contains(op));
-
-        // Check for path terms
-        let has_path_term = self.path_patterns.path_terms
-            .iter()
-            .any(|&term| line.contains(term));
-
-        // Check for special commands
-        let has_special_command = self.path_patterns.special_commands
-            .iter()
-            .any(|&cmd| line.contains(cmd));
-
-        has_file_command || has_file_operation || has_path_term || has_special_command
-    }
-
     /// Attempts to complete a file path at the given position in the line
     pub fn complete_path(
         &self,
@@ -149,11 +99,6 @@ impl PathCompleter {
             Err(err) => Err(err),
         }
     }
-
-    /// Checks if the word appears to be a file path based on its syntax
-    pub fn is_path_syntax(&self, word: &str) -> bool {
-        word.contains('/') || word.starts_with('~') || word.starts_with('.')
-    }
 }
 
 pub struct ChatCompleter {
@@ -197,12 +142,10 @@ impl Completer for ChatCompleter {
             return Ok(self.complete_command(word, start));
         }
 
-        // Handle file path completion if the word contains path separators or context suggests file paths
-        if self.path_completer.is_path_syntax(word) || self.path_completer.is_path_context(line) {
-            if let Ok((pos, completions)) = self.path_completer.complete_path(line, pos, _ctx) {
-                if !completions.is_empty() {
-                    return Ok((pos, completions));
-                }
+        // Handle file path completion as fallback
+        if let Ok((pos, completions)) = self.path_completer.complete_path(line, pos, _ctx) {
+            if !completions.is_empty() {
+                return Ok((pos, completions));
             }
         }
 
@@ -322,26 +265,4 @@ mod tests {
         // Verify no completions are returned for regular text
         assert!(completions.is_empty());
     }
-
-    #[test]
-    fn test_is_path_context() {
-        let completer = PathCompleter::new();
-
-        // Test positive cases
-        assert!(completer.is_path_context("Please read file test.txt"));
-        assert!(completer.is_path_context("Show me the path to config"));
-        assert!(completer.is_path_context("List the directory contents"));
-        assert!(completer.is_path_context("/context add ./myfile.txt"));
-        assert!(completer.is_path_context("/context rm ./myfile.txt"));
-        assert!(completer.is_path_context("Can you open this file?"));
-        assert!(completer.is_path_context("Please read this for me"));
-        assert!(completer.is_path_context("cat my_file.txt"));
-        assert!(completer.is_path_context("ls -la"));
-        assert!(completer.is_path_context("cd /usr/local"));
-
-        // Test negative cases
-        assert!(!completer.is_path_context("Hello world"));
-        assert!(!completer.is_path_context("What is the weather today?"));
-        assert!(!completer.is_path_context("How do I upload data using s3 cli"));
-    }
 }