Implement delete file (mark3labs#21)

Author: ezynda3

README.md

@@ -28,6 +28,10 @@ This MCP server provides secure access to the local filesystem via the Model Con
   - Move or rename files and directories
   - Parameters: `source` (required): Source path of the file or directory, `destination` (required): Destination path
 
+- **delete_file**
+  - Delete a file or directory from the file system
+  - Parameters: `path` (required): Path to the file or directory to delete, `recursive` (optional): Whether to recursively delete directories (default: false)
+
 #### Directory Operations
 
 - **list_directory**

filesystemserver/handler.go

@@ -1937,6 +1937,138 @@ func (fs *FilesystemHandler) handleReadMultipleFiles(
 	}, nil
 }
 
+func (fs *FilesystemHandler) handleDeleteFile(
+	ctx context.Context,
+	request mcp.CallToolRequest,
+) (*mcp.CallToolResult, error) {
+	path, ok := request.Params.Arguments["path"].(string)
+	if !ok {
+		return nil, fmt.Errorf("path must be a string")
+	}
+
+	// Handle empty or relative paths like "." or "./" by converting to absolute path
+	if path == "." || path == "./" {
+		// Get current working directory
+		cwd, err := os.Getwd()
+		if err != nil {
+			return &mcp.CallToolResult{
+				Content: []mcp.Content{
+					mcp.TextContent{
+						Type: "text",
+						Text: fmt.Sprintf("Error resolving current directory: %v", err),
+					},
+				},
+				IsError: true,
+			}, nil
+		}
+		path = cwd
+	}
+
+	validPath, err := fs.validatePath(path)
+	if err != nil {
+		return &mcp.CallToolResult{
+			Content: []mcp.Content{
+				mcp.TextContent{
+					Type: "text",
+					Text: fmt.Sprintf("Error: %v", err),
+				},
+			},
+			IsError: true,
+		}, nil
+	}
+
+	// Check if path exists
+	info, err := os.Stat(validPath)
+	if os.IsNotExist(err) {
+		return &mcp.CallToolResult{
+			Content: []mcp.Content{
+				mcp.TextContent{
+					Type: "text",
+					Text: fmt.Sprintf("Error: Path does not exist: %s", path),
+				},
+			},
+			IsError: true,
+		}, nil
+	} else if err != nil {
+		return &mcp.CallToolResult{
+			Content: []mcp.Content{
+				mcp.TextContent{
+					Type: "text",
+					Text: fmt.Sprintf("Error accessing path: %v", err),
+				},
+			},
+			IsError: true,
+		}, nil
+	}
+
+	// Extract recursive parameter (optional, default: false)
+	recursive := false
+	if recursiveParam, ok := request.Params.Arguments["recursive"]; ok {
+		if r, ok := recursiveParam.(bool); ok {
+			recursive = r
+		}
+	}
+
+	// Check if it's a directory and handle accordingly
+	if info.IsDir() {
+		if !recursive {
+			return &mcp.CallToolResult{
+				Content: []mcp.Content{
+					mcp.TextContent{
+						Type: "text",
+						Text: fmt.Sprintf("Error: %s is a directory. Use recursive=true to delete directories.", path),
+					},
+				},
+				IsError: true,
+			}, nil
+		}
+
+		// It's a directory and recursive is true, so remove it
+		if err := os.RemoveAll(validPath); err != nil {
+			return &mcp.CallToolResult{
+				Content: []mcp.Content{
+					mcp.TextContent{
+						Type: "text",
+						Text: fmt.Sprintf("Error deleting directory: %v", err),
+					},
+				},
+				IsError: true,
+			}, nil
+		}
+
+		return &mcp.CallToolResult{
+			Content: []mcp.Content{
+				mcp.TextContent{
+					Type: "text",
+					Text: fmt.Sprintf("Successfully deleted directory %s", path),
+				},
+			},
+		}, nil
+	}
+
+	// It's a file, delete it
+	if err := os.Remove(validPath); err != nil {
+		return &mcp.CallToolResult{
+			Content: []mcp.Content{
+				mcp.TextContent{
+					Type: "text",
+					Text: fmt.Sprintf("Error deleting file: %v", err),
+				},
+			},
+			IsError: true,
+		}, nil
+	}
+
+	return &mcp.CallToolResult{
+		Content: []mcp.Content{
+			mcp.TextContent{
+				Type: "text",
+				Text: fmt.Sprintf("Successfully deleted file %s", path),
+			},
+		},
+	}, nil
+}
+
 func (fs *FilesystemHandler) handleListAllowedDirectories(
 	ctx context.Context,
 	request mcp.CallToolRequest,

filesystemserver/server.go

@@ -145,5 +145,17 @@ func NewFilesystemServer(allowedDirs []string) (*server.MCPServer, error) {
 		),
 	), h.handleTree)
 
+	s.AddTool(mcp.NewTool(
+		"delete_file",
+		mcp.WithDescription("Delete a file or directory from the file system."),
+		mcp.WithString("path",
+			mcp.Description("Path to the file or directory to delete"),
+			mcp.Required(),
+		),
+		mcp.WithBoolean("recursive",
+			mcp.Description("Whether to recursively delete directories (default: false)"),
+		),
+	), h.handleDeleteFile)
+
 	return s, nil
 }