Use the host-binary CLI to connect/disconnect claude

Signed-off-by: David Gageot <david.gageot@docker.com>

Author: dgageot

src/extension/ui/src/MCPClients.ts

@@ -18,8 +18,7 @@ export const getMCPClientStates = async (ddClient: v1.DockerDesktopClient) => {
             mcpClientStates[mcpClient.name] = { exists: false, configured: false, path, client: mcpClient };
         } else if (content === undefined) {
             mcpClientStates[mcpClient.name] = { exists: true, configured: false, path, client: mcpClient };
-        }
-        else {
+        } else {
             mcpClientStates[mcpClient.name] = { exists: true, configured: mcpClient.validateConfig(content), path: path, client: mcpClient };
         }
     }

src/extension/ui/src/mcp-clients/ClaudeDesktop.ts

@@ -1,6 +1,6 @@
 import { v1 } from "@docker/extension-api-client-types";
 import { BUSYBOX } from "../Constants";
-import { getUser, writeToMount } from "../utils/Files";
+import { getUser } from "../utils/Files";
 import { MCPClient, SAMPLE_MCP_CONFIG } from "./MCPTypes";
 
 class ClaudeDesktopClient implements MCPClient {
@@ -12,9 +12,9 @@ class ClaudeDesktopClient implements MCPClient {
     "Click on the <strong>Developer</strong> tab",
     "Click on the <strong>Edit Config</strong> button",
     "Add MCP_DOCKER to <code>mcpServers</code> section:" +
-      '<pre style="font-family: monospace; overflow: auto; width: 80%; background-color: grey.200; padding: 1; border-radius: 1; font-size: 12px;">' +
-      JSON.stringify(SAMPLE_MCP_CONFIG, null, 2) +
-      "</pre>",
+    '<pre style="font-family: monospace; overflow: auto; width: 80%; background-color: grey.200; padding: 1; border-radius: 1; font-size: 12px;">' +
+    JSON.stringify(SAMPLE_MCP_CONFIG, null, 2) +
+    "</pre>",
   ];
   expectedConfigPath = {
     darwin:
@@ -56,101 +56,17 @@ class ClaudeDesktopClient implements MCPClient {
     }
   };
   connect = async (client: v1.DockerDesktopClient) => {
-    const platform = client.host.platform;
-    let path = "";
-    switch (platform) {
-      case "darwin":
-        path = "/Users/$USER/Library/Application Support/Claude/";
-        break;
-      case "linux":
-        path = "/home/$USER/.config/claude/";
-        break;
-      case "win32":
-        path = "%APPDATA%\\Claude\\";
-        break;
-      default:
-        throw new Error("Unsupported platform: " + platform);
-    }
-    const user = await getUser(client);
-    path = path.replace("$USER", user);
-
-    let payload: Record<string, any> = {};
     try {
-      const result = await client.docker.cli.exec("run", [
-        "--rm",
-        "--mount",
-        `type=bind,source="${path}",target=/claude_desktop_config`,
-        BUSYBOX,
-        "/bin/cat",
-        "/claude_desktop_config/claude_desktop_config.json",
-      ]);
-      if (result.stdout) {
-        payload = JSON.parse(result.stdout);
-      }
+      await client.extension.host?.cli.exec("host-binary", ["client", "connect", "--global", "claude-desktop"]);
     } catch (e) {
-      // No config or malformed config found, overwrite it
-    }
-
-    if (!payload.mcpServers) {
-      payload.mcpServers = {};
-    }
-    payload.mcpServers.MCP_DOCKER = SAMPLE_MCP_CONFIG.mcpServers.MCP_DOCKER;
-
-    try {
-      await writeToMount(
-        client,
-        `type=bind,source="${path}",target=/claude_desktop_config`,
-        "/claude_desktop_config/claude_desktop_config.json",
-        JSON.stringify(payload, null, 2)
-      );
-    } catch (e) {
-      client.desktopUI.toast.error((e as any).stderr);
+      client.desktopUI.toast.error("Unable to connect Claude Desktop");
     }
   };
   disconnect = async (client: v1.DockerDesktopClient) => {
-    const platform = client.host.platform;
-    let path = "";
-    switch (platform) {
-      case "darwin":
-        path = "/Users/$USER/Library/Application Support/Claude/";
-        break;
-      case "linux":
-        path = "/home/$USER/.config/claude/";
-        break;
-      case "win32":
-        path = "%APPDATA%\\Claude\\";
-        break;
-      default:
-        throw new Error("Unsupported platform: " + platform);
-    }
-    const user = await getUser(client);
-    path = path.replace("$USER", user);
     try {
-      // This method is only called after the config has been validated, so we can safely assume it's a valid config.
-      const previousConfig = JSON.parse(
-        (
-          await client.docker.cli.exec("run", [
-            "--rm",
-            "--mount",
-            `type=bind,source="${path}",target=/claude_desktop_config`,
-            "-w",
-            "/claude_desktop_config",
-            BUSYBOX,
-            "/bin/cat",
-            "/claude_desktop_config/claude_desktop_config.json",
-          ])
-        ).stdout || "{}"
-      );
-      const newConfig = { ...previousConfig };
-      delete newConfig.mcpServers.MCP_DOCKER;
-      await writeToMount(
-        client,
-        `type=bind,source="${path}",target=/claude_desktop_config`,
-        "/claude_desktop_config/claude_desktop_config.json",
-        JSON.stringify(newConfig, null, 2)
-      );
+      await client.extension.host?.cli.exec("host-binary", ["client", "disconnect", "--global", "claude-desktop"]);
     } catch (e) {
-      client.desktopUI.toast.error((e as any).stderr);
+      client.desktopUI.toast.error("Unable to disconnect Claude Desktop");
     }
   };
   validateConfig = (content: string) => {