implement delete_file, delete_directory, move_file, copy_file

Author: alexylon

README.md

@@ -90,6 +90,10 @@ Claude can automatically use these tools:
 - `morph_edit_file` - Ultra-fast code editing (requires MORPH_API_KEY)
 - `list_directory` - List directory contents
 - `create_directory` - Create directories
+- `delete_file` - Delete files (asks for confirmation)
+- `delete_directory` - Delete directories recursively (asks for confirmation)
+- `move_file` - Move or rename files/directories
+- `copy_file` - Copy files
 
 **Code & Search:**
 - `search_code` - Fast regex-based code search (requires `ripgrep`)

src/conversation.rs

@@ -45,6 +45,11 @@ When helping users:
 - Execute bash commands to test code and verify functionality (read-only, no sudo or file modifications)
 - Explain your reasoning when using tools
 
+File deletion safety:
+- ALWAYS ask the user for explicit confirmation before using delete_file or delete_directory
+- List the files/directories that will be deleted and wait for user approval
+- Never delete files without prior user confirmation
+
 Testing after code changes:
 - After editing code files (not comments, README, or documentation), ALWAYS test the changes using execute_bash
 - Run appropriate build/test commands based on the project type:

src/tools/filesystem.rs

@@ -157,7 +157,7 @@ impl FileSystemTool {
         Ok(full_path.exists())
     }
 
-    pub fn _delete_file(&self, path: &str) -> Result<()> {
+    pub fn delete_file(&self, path: &str) -> Result<()> {
         let full_path = self.validate_path(path)?;
 
         if !full_path.exists() {
@@ -175,6 +175,67 @@ impl FileSystemTool {
         Ok(())
     }
 
+    pub fn delete_directory(&self, path: &str) -> Result<()> {
+        let full_path = self.validate_path(path)?;
+
+        if !full_path.exists() {
+            return Err(SofosError::FileNotFound(path.to_string()));
+        }
+
+        if !full_path.is_dir() {
+            return Err(SofosError::InvalidPath(format!(
+                "'{}' is not a directory",
+                path
+            )));
+        }
+
+        fs::remove_dir_all(&full_path)?;
+        Ok(())
+    }
+
+    pub fn move_file(&self, source: &str, destination: &str) -> Result<()> {
+        let source_path = self.validate_path(source)?;
+        let dest_path = self.validate_path(destination)?;
+
+        if !source_path.exists() {
+            return Err(SofosError::FileNotFound(source.to_string()));
+        }
+
+        if let Some(parent) = dest_path.parent() {
+            if !parent.exists() {
+                fs::create_dir_all(parent)?;
+            }
+        }
+
+        fs::rename(&source_path, &dest_path)?;
+        Ok(())
+    }
+
+    pub fn copy_file(&self, source: &str, destination: &str) -> Result<()> {
+        let source_path = self.validate_path(source)?;
+        let dest_path = self.validate_path(destination)?;
+
+        if !source_path.exists() {
+            return Err(SofosError::FileNotFound(source.to_string()));
+        }
+
+        if !source_path.is_file() {
+            return Err(SofosError::InvalidPath(format!(
+                "'{}' is not a file",
+                source
+            )));
+        }
+
+        if let Some(parent) = dest_path.parent() {
+            if !parent.exists() {
+                fs::create_dir_all(parent)?;
+            }
+        }
+
+        fs::copy(&source_path, &dest_path)?;
+        Ok(())
+    }
+
     pub fn _workspace(&self) -> &Path {
         &self.workspace
     }

src/tools/mod.rs

@@ -150,6 +150,44 @@ impl ToolExecutor {
                 self.fs_tool.write_file(path, &merged_code)?;
                 Ok(format!("Successfully applied Morph edit to '{}'", path))
             }
+            "delete_file" => {
+                let path = input["path"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'path' parameter".to_string()))?;
+
+                self.fs_tool.delete_file(path)?;
+                Ok(format!("Successfully deleted file '{}'", path))
+            }
+            "delete_directory" => {
+                let path = input["path"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'path' parameter".to_string()))?;
+
+                self.fs_tool.delete_directory(path)?;
+                Ok(format!("Successfully deleted directory '{}'", path))
+            }
+            "move_file" => {
+                let source = input["source"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'source' parameter".to_string()))?;
+                let destination = input["destination"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'destination' parameter".to_string()))?;
+
+                self.fs_tool.move_file(source, destination)?;
+                Ok(format!("Successfully moved '{}' to '{}'", source, destination))
+            }
+            "copy_file" => {
+                let source = input["source"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'source' parameter".to_string()))?;
+                let destination = input["destination"]
+                    .as_str()
+                    .ok_or_else(|| SofosError::ToolExecution("Missing 'destination' parameter".to_string()))?;
+
+                self.fs_tool.copy_file(source, destination)?;
+                Ok(format!("Successfully copied '{}' to '{}'", source, destination))
+            }
             "execute_bash" => {
                 let command = input["command"]
                     .as_str()

src/tools/types.rs

@@ -118,6 +118,82 @@ fn execute_bash_tool() -> Tool {
     }
 }
 
+fn delete_file_tool() -> Tool {
+    Tool {
+        name: "delete_file".to_string(),
+        description: "Delete a file in the current workspace. IMPORTANT: Always ask the user for confirmation before deleting files. Use this only after explicit user approval.".to_string(),
+        input_schema: json!({
+            "type": "object",
+            "properties": {
+                "path": {
+                    "type": "string",
+                    "description": "The relative path to the file to delete (e.g., 'src/old_file.rs')"
+                }
+            },
+            "required": ["path"]
+        }),
+    }
+}
+
+fn delete_directory_tool() -> Tool {
+    Tool {
+        name: "delete_directory".to_string(),
+        description: "Delete a directory and all its contents in the current workspace. IMPORTANT: Always ask the user for confirmation before deleting directories. Use this only after explicit user approval.".to_string(),
+        input_schema: json!({
+            "type": "object",
+            "properties": {
+                "path": {
+                    "type": "string",
+                    "description": "The relative path to the directory to delete (e.g., 'src/old_module')"
+                }
+            },
+            "required": ["path"]
+        }),
+    }
+}
+
+fn move_file_tool() -> Tool {
+    Tool {
+        name: "move_file".to_string(),
+        description: "Move or rename a file or directory within the workspace. Creates parent directories if needed.".to_string(),
+        input_schema: json!({
+            "type": "object",
+            "properties": {
+                "source": {
+                    "type": "string",
+                    "description": "The relative path to the source file/directory (e.g., 'src/old_name.rs')"
+                },
+                "destination": {
+                    "type": "string",
+                    "description": "The relative path to the destination (e.g., 'src/new_name.rs' or 'new_folder/file.rs')"
+                }
+            },
+            "required": ["source", "destination"]
+        }),
+    }
+}
+
+fn copy_file_tool() -> Tool {
+    Tool {
+        name: "copy_file".to_string(),
+        description: "Copy a file to a new location within the workspace. Creates parent directories if needed.".to_string(),
+        input_schema: json!({
+            "type": "object",
+            "properties": {
+                "source": {
+                    "type": "string",
+                    "description": "The relative path to the source file (e.g., 'src/template.rs')"
+                },
+                "destination": {
+                    "type": "string",
+                    "description": "The relative path to the destination (e.g., 'src/new_file.rs')"
+                }
+            },
+            "required": ["source", "destination"]
+        }),
+    }
+}
+
 fn morph_edit_file_tool() -> Tool {
     Tool {
         name: "morph_edit_file".to_string(),
@@ -150,6 +226,10 @@ pub fn get_tools() -> Vec<Tool> {
         write_file_tool(false),
         list_directory_tool(),
         create_directory_tool(),
+        delete_file_tool(),
+        delete_directory_tool(),
+        move_file_tool(),
+        copy_file_tool(),
         web_search_tool(),
         execute_bash_tool(),
     ]
@@ -162,6 +242,10 @@ pub fn get_tools_with_morph() -> Vec<Tool> {
         write_file_tool(true),
         list_directory_tool(),
         create_directory_tool(),
+        delete_file_tool(),
+        delete_directory_tool(),
+        move_file_tool(),
+        copy_file_tool(),
         web_search_tool(),
         execute_bash_tool(),
         morph_edit_file_tool(),