@artyfacts/claude 1.3.18 → 1.3.20

package/dist/cli.js

@@ -876,8 +876,17 @@ function addMcpServer(options) {
   } else {
     const error = result.stderr || result.stdout || `Exit code ${result.status}`;
     if (error.includes("already exists")) {
-      console.log(`[MCP] Server ${name} already configured - treating as success`);
-      return { success: true, alreadyExists: true };
+      console.log(`[MCP] Server ${name} exists - removing old config...`);
+      (0, import_child_process2.spawnSync)("claude", ["mcp", "remove", name], { encoding: "utf-8" });
+      const retryResult = (0, import_child_process2.spawnSync)("claude", cliArgs, {
+        encoding: "utf-8",
+        stdio: ["pipe", "pipe", "pipe"]
+      });
+      if (retryResult.status === 0) {
+        return { success: true, replaced: true };
+      } else {
+        return { success: false, error: retryResult.stderr || retryResult.stdout || "Failed after removing old config" };
+      }
     }
     return { success: false, error };
   }
@@ -912,11 +921,11 @@ var McpHandler = class {
         console.log(`[MCP] Triggering ${platform} OAuth via Claude...`);
         const oauthProcess = (0, import_child_process2.spawnSync)("claude", [
           "-p",
-          `Please list the available tools from the ${platform} MCP server. Just list them briefly.`,
+          `Please authenticate with the ${platform} MCP server. If there's an authenticate tool, call it. Otherwise list what tools are available.`,
           "--allowedTools",
           `mcp__${serverName}__*`,
           "--permission-mode",
-          "default"
+          "bypassPermissions"
         ], {
           encoding: "utf-8",
           stdio: "inherit",

package/dist/cli.mjs

@@ -7,7 +7,7 @@ import {
   getCredentials,
   loadCredentials,
   promptForApiKey
-} from "./chunk-SKROECYN.mjs";
+} from "./chunk-KXV4I4WJ.mjs";
 
 // src/cli.ts
 import { Command } from "commander";

package/dist/index.js

@@ -6069,8 +6069,17 @@ function addMcpServer(options) {
   } else {
     const error = result.stderr || result.stdout || `Exit code ${result.status}`;
     if (error.includes("already exists")) {
-      console.log(`[MCP] Server ${name} already configured - treating as success`);
-      return { success: true, alreadyExists: true };
+      console.log(`[MCP] Server ${name} exists - removing old config...`);
+      (0, import_child_process2.spawnSync)("claude", ["mcp", "remove", name], { encoding: "utf-8" });
+      const retryResult = (0, import_child_process2.spawnSync)("claude", cliArgs, {
+        encoding: "utf-8",
+        stdio: ["pipe", "pipe", "pipe"]
+      });
+      if (retryResult.status === 0) {
+        return { success: true, replaced: true };
+      } else {
+        return { success: false, error: retryResult.stderr || retryResult.stdout || "Failed after removing old config" };
+      }
     }
     return { success: false, error };
   }
@@ -6105,11 +6114,11 @@ var McpHandler = class {
         console.log(`[MCP] Triggering ${platform} OAuth via Claude...`);
         const oauthProcess = (0, import_child_process2.spawnSync)("claude", [
           "-p",
-          `Please list the available tools from the ${platform} MCP server. Just list them briefly.`,
+          `Please authenticate with the ${platform} MCP server. If there's an authenticate tool, call it. Otherwise list what tools are available.`,
           "--allowedTools",
           `mcp__${serverName}__*`,
           "--permission-mode",
-          "default"
+          "bypassPermissions"
         ], {
           encoding: "utf-8",
           stdio: "inherit",

package/dist/index.mjs

@@ -14,7 +14,7 @@ import {
   promptForApiKey,
   runDeviceAuth,
   saveCredentials
-} from "./chunk-SKROECYN.mjs";
+} from "./chunk-KXV4I4WJ.mjs";
 
 // node_modules/@anthropic-ai/sdk/internal/tslib.mjs
 function __classPrivateFieldSet(receiver, state, value, kind, f) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@artyfacts/claude",
-  "version": "1.3.18",
+  "version": "1.3.20",
   "description": "Claude adapter for Artyfacts - Execute tasks using Claude Code CLI",
   "main": "dist/index.js",
   "module": "dist/index.mjs",

package/src/mcp.ts

@@ -101,7 +101,7 @@ function addMcpServer(options: {
   args?: string[];
   env?: Record<string, string>;
   scope?: 'local' | 'user' | 'project';
-}): { success: boolean; error?: string; alreadyExists?: boolean } {
+}): { success: boolean; error?: string; replaced?: boolean } {
   const { name, transport, url, command, args = [], env = {}, scope = 'user' } = options;
   
   const cliArgs = ['mcp', 'add', '-s', scope, '-t', transport];
@@ -136,10 +136,22 @@ function addMcpServer(options: {
   } else {
     const error = result.stderr || result.stdout || `Exit code ${result.status}`;
     
-    // "Already exists" is not an error - the server is configured
+    // "Already exists" - remove and retry to ensure fresh config
     if (error.includes('already exists')) {
-      console.log(`[MCP] Server ${name} already configured - treating as success`);
-      return { success: true, alreadyExists: true };
+      console.log(`[MCP] Server ${name} exists - removing old config...`);
+      spawnSync('claude', ['mcp', 'remove', name], { encoding: 'utf-8' });
+      
+      // Retry the add
+      const retryResult = spawnSync('claude', cliArgs, {
+        encoding: 'utf-8',
+        stdio: ['pipe', 'pipe', 'pipe'],
+      });
+      
+      if (retryResult.status === 0) {
+        return { success: true, replaced: true };
+      } else {
+        return { success: false, error: retryResult.stderr || retryResult.stdout || 'Failed after removing old config' };
+      }
     }
     
     return { success: false, error };
@@ -186,13 +198,13 @@ export class McpHandler {
         
         console.log(`[MCP] Added ${serverName} → ${oauthServer.url}`);
         
-        // Trigger OAuth by asking Claude to use the MCP - this opens browser!
+        // Trigger OAuth by asking Claude to authenticate with the MCP
         console.log(`[MCP] Triggering ${platform} OAuth via Claude...`);
         const oauthProcess = spawnSync('claude', [
           '-p',
-          `Please list the available tools from the ${platform} MCP server. Just list them briefly.`,
+          `Please authenticate with the ${platform} MCP server. If there's an authenticate tool, call it. Otherwise list what tools are available.`,
           '--allowedTools', `mcp__${serverName}__*`,
-          '--permission-mode', 'default',
+          '--permission-mode', 'bypassPermissions',
         ], {
           encoding: 'utf-8',
           stdio: 'inherit', // Show output and allow browser interaction