🤖 feat: add MCP server edit functionality

- Add /mcp edit <name> <command> slash command
- Add inline edit button in Settings UI
- Enter to save, Esc to cancel

Author: ammar-agent

src/browser/components/ChatInput/index.tsx

@@ -740,7 +740,11 @@ export const ChatInput: React.FC<ChatInputProps> = (props) => {
           return;
         }
 
-        if (parsed.type === "mcp-add" || parsed.type === "mcp-remove") {
+        if (
+          parsed.type === "mcp-add" ||
+          parsed.type === "mcp-edit" ||
+          parsed.type === "mcp-remove"
+        ) {
           if (!api) {
             setToast({
               id: Date.now().toString(),
@@ -763,7 +767,7 @@ export const ChatInput: React.FC<ChatInputProps> = (props) => {
           try {
             const projectPath = selectedWorkspace.projectPath;
             const result =
-              parsed.type === "mcp-add"
+              parsed.type === "mcp-add" || parsed.type === "mcp-edit"
                 ? await api.projects.mcp.add({
                     projectPath,
                     name: parsed.name,
@@ -779,13 +783,16 @@ export const ChatInput: React.FC<ChatInputProps> = (props) => {
               });
               setInput(messageText);
             } else {
+              const successMessage =
+                parsed.type === "mcp-add"
+                  ? `Added MCP server ${parsed.name}`
+                  : parsed.type === "mcp-edit"
+                    ? `Updated MCP server ${parsed.name}`
+                    : `Removed MCP server ${parsed.name}`;
               setToast({
                 id: Date.now().toString(),
                 type: "success",
-                message:
-                  parsed.type === "mcp-add"
-                    ? `Added MCP server ${parsed.name}`
-                    : `Removed MCP server ${parsed.name}`,
+                message: successMessage,
               });
             }
           } catch (error) {

src/browser/components/Settings/sections/ProjectSettingsSection.tsx

@@ -10,6 +10,9 @@ import {
   Plus,
   ChevronDown,
   Server,
+  Pencil,
+  Check,
+  X,
 } from "lucide-react";
 
 type TestResult = { success: true; tools: string[] } | { success: false; error: string };
@@ -33,6 +36,11 @@ export const ProjectSettingsSection: React.FC = () => {
   const [testingNewCommand, setTestingNewCommand] = useState(false);
   const [newCommandTestResult, setNewCommandTestResult] = useState<TestResult | null>(null);
 
+  // Edit server state
+  const [editingServer, setEditingServer] = useState<string | null>(null);
+  const [editCommand, setEditCommand] = useState("");
+  const [savingEdit, setSavingEdit] = useState(false);
+
   // Set default project when projects load
   useEffect(() => {
     if (projectList.length > 0 && !selectedProject) {
@@ -155,6 +163,46 @@ export const ProjectSettingsSection: React.FC = () => {
     }
   }, [api, selectedProject, newServerName, newServerCommand, refresh]);
 
+  const handleStartEdit = useCallback((name: string, command: string) => {
+    setEditingServer(name);
+    setEditCommand(command);
+  }, []);
+
+  const handleCancelEdit = useCallback(() => {
+    setEditingServer(null);
+    setEditCommand("");
+  }, []);
+
+  const handleSaveEdit = useCallback(async () => {
+    if (!api || !selectedProject || !editingServer || !editCommand.trim()) return;
+    setSavingEdit(true);
+    setError(null);
+    try {
+      const result = await api.projects.mcp.add({
+        projectPath: selectedProject,
+        name: editingServer,
+        command: editCommand.trim(),
+      });
+      if (!result.success) {
+        setError(result.error ?? "Failed to update MCP server");
+      } else {
+        setEditingServer(null);
+        setEditCommand("");
+        // Clear test result for this server since command changed
+        setTestResults((prev) => {
+          const next = new Map(prev);
+          next.delete(editingServer);
+          return next;
+        });
+        await refresh();
+      }
+    } catch (err) {
+      setError(err instanceof Error ? err.message : "Failed to update MCP server");
+    } finally {
+      setSavingEdit(false);
+    }
+  }, [api, selectedProject, editingServer, editCommand, refresh]);
+
   if (projectList.length === 0) {
     return (
       <div className="flex flex-col items-center justify-center py-12 text-center">
@@ -227,52 +275,107 @@ export const ProjectSettingsSection: React.FC = () => {
           {Object.entries(servers).map(([name, command]) => {
             const isTesting = testingServer === name;
             const testResult = testResults.get(name);
+            const isEditing = editingServer === name;
             return (
               <li key={name} className="border-border-medium bg-secondary/20 rounded-lg border p-3">
                 <div className="flex items-start justify-between gap-3">
                   <div className="min-w-0 flex-1">
                     <div className="flex items-center gap-2">
                       <span className="font-medium">{name}</span>
-                      {testResult?.success && (
+                      {testResult?.success && !isEditing && (
                         <span className="rounded bg-green-500/10 px-1.5 py-0.5 text-xs text-green-500">
                           {testResult.tools.length} tools
                         </span>
                       )}
                     </div>
-                    <p className="text-muted-foreground mt-0.5 text-xs break-all">{command}</p>
+                    {isEditing ? (
+                      <input
+                        type="text"
+                        value={editCommand}
+                        onChange={(e) => setEditCommand(e.target.value)}
+                        className="border-border-medium bg-secondary/30 text-foreground placeholder:text-muted-foreground focus:ring-accent mt-1 w-full rounded-md border px-2 py-1 text-xs focus:ring-1 focus:outline-none"
+                        autoFocus
+                        onKeyDown={(e) => {
+                          if (e.key === "Enter") {
+                            void handleSaveEdit();
+                          } else if (e.key === "Escape") {
+                            handleCancelEdit();
+                          }
+                        }}
+                      />
+                    ) : (
+                      <p className="text-muted-foreground mt-0.5 text-xs break-all">{command}</p>
+                    )}
                   </div>
                   <div className="flex shrink-0 gap-1">
-                    <button
-                      type="button"
-                      onClick={() => void handleTest(name)}
-                      disabled={isTesting}
-                      className="text-muted-foreground hover:bg-secondary hover:text-accent rounded p-1.5 transition-colors disabled:opacity-50"
-                      title="Test connection"
-                    >
-                      {isTesting ? (
-                        <Loader2 className="h-4 w-4 animate-spin" />
-                      ) : (
-                        <Play className="h-4 w-4" />
-                      )}
-                    </button>
-                    <button
-                      type="button"
-                      onClick={() => void handleRemove(name)}
-                      disabled={loading}
-                      className="text-muted-foreground hover:bg-destructive/10 hover:text-destructive rounded p-1.5 transition-colors"
-                      title="Remove server"
-                    >
-                      <Trash2 className="h-4 w-4" />
-                    </button>
+                    {isEditing ? (
+                      <>
+                        <button
+                          type="button"
+                          onClick={() => void handleSaveEdit()}
+                          disabled={savingEdit || !editCommand.trim()}
+                          className="text-muted-foreground rounded p-1.5 transition-colors hover:bg-green-500/10 hover:text-green-500 disabled:opacity-50"
+                          title="Save (Enter)"
+                        >
+                          {savingEdit ? (
+                            <Loader2 className="h-4 w-4 animate-spin" />
+                          ) : (
+                            <Check className="h-4 w-4" />
+                          )}
+                        </button>
+                        <button
+                          type="button"
+                          onClick={handleCancelEdit}
+                          disabled={savingEdit}
+                          className="text-muted-foreground hover:bg-destructive/10 hover:text-destructive rounded p-1.5 transition-colors"
+                          title="Cancel (Esc)"
+                        >
+                          <X className="h-4 w-4" />
+                        </button>
+                      </>
+                    ) : (
+                      <>
+                        <button
+                          type="button"
+                          onClick={() => void handleTest(name)}
+                          disabled={isTesting}
+                          className="text-muted-foreground hover:bg-secondary hover:text-accent rounded p-1.5 transition-colors disabled:opacity-50"
+                          title="Test connection"
+                        >
+                          {isTesting ? (
+                            <Loader2 className="h-4 w-4 animate-spin" />
+                          ) : (
+                            <Play className="h-4 w-4" />
+                          )}
+                        </button>
+                        <button
+                          type="button"
+                          onClick={() => handleStartEdit(name, command)}
+                          className="text-muted-foreground hover:bg-secondary hover:text-accent rounded p-1.5 transition-colors"
+                          title="Edit command"
+                        >
+                          <Pencil className="h-4 w-4" />
+                        </button>
+                        <button
+                          type="button"
+                          onClick={() => void handleRemove(name)}
+                          disabled={loading}
+                          className="text-muted-foreground hover:bg-destructive/10 hover:text-destructive rounded p-1.5 transition-colors"
+                          title="Remove server"
+                        >
+                          <Trash2 className="h-4 w-4" />
+                        </button>
+                      </>
+                    )}
                   </div>
                 </div>
-                {testResult && !testResult.success && (
+                {testResult && !testResult.success && !isEditing && (
                   <div className="text-destructive mt-2 flex items-start gap-1.5 text-xs">
                     <XCircle className="mt-0.5 h-3 w-3 shrink-0" />
                     <span>{testResult.error}</span>
                   </div>
                 )}
-                {testResult?.success && testResult.tools.length > 0 && (
+                {testResult?.success && testResult.tools.length > 0 && !isEditing && (
                   <p className="text-muted-foreground mt-2 text-xs">
                     Tools: {testResult.tools.join(", ")}
                   </p>

src/browser/utils/slashCommands/registry.ts

@@ -606,6 +606,18 @@ const mcpCommandDefinition: SlashCommandDefinition = {
       return { type: "mcp-add", name, command: commandText };
     }
 
+    if (sub === "edit") {
+      const name = cleanRemainingTokens[1];
+      const commandText = rawInput
+        .trim()
+        .replace(/^edit\s+[^\s]+\s*/i, "")
+        .trim();
+      if (!name || !commandText) {
+        return { type: "unknown-command", command: "mcp", subcommand: "edit" };
+      }
+      return { type: "mcp-edit", name, command: commandText };
+    }
+
     if (sub === "remove") {
       const name = cleanRemainingTokens[1];
       if (!name) {

src/browser/utils/slashCommands/types.ts

@@ -30,6 +30,7 @@ export type ParsedCommand =
     }
   | { type: "vim-toggle" }
   | { type: "mcp-add"; name: string; command: string }
+  | { type: "mcp-edit"; name: string; command: string }
   | { type: "mcp-remove"; name: string }
   | { type: "mcp-open" }
   | { type: "unknown-command"; command: string; subcommand?: string }