@artyfacts/claude 1.3.17 → 1.3.19

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 };
   }
@@ -902,26 +911,32 @@ var McpHandler = class {
       if (oauthServer) {
         const result = addMcpServer({
           name: serverName,
-          transport: "stdio",
-          command: "npx",
-          args: ["-y", "mcp-remote", oauthServer.url]
+          transport: "http",
+          url: oauthServer.url
         });
         if (!result.success) {
           throw new Error(`Failed to add ${platform} MCP: ${result.error}`);
         }
-        console.log(`[MCP] Added ${serverName} via mcp-remote`);
-        console.log(`[MCP] Triggering ${platform} OAuth...`);
-        const oauthProcess = (0, import_child_process2.spawnSync)("npx", ["-y", "mcp-remote", oauthServer.url], {
+        console.log(`[MCP] Added ${serverName} \u2192 ${oauthServer.url}`);
+        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.`,
+          "--allowedTools",
+          `mcp__${serverName}__*`,
+          "--permission-mode",
+          "default"
+        ], {
           encoding: "utf-8",
           stdio: "inherit",
-          // Show output and allow browser to open
+          // Show output and allow browser interaction
           timeout: 12e4
           // 2 minute timeout for user to complete OAuth
         });
         if (oauthProcess.status === 0) {
           console.log(`[MCP] ${platform} OAuth completed!`);
         } else {
-          console.log(`[MCP] OAuth process ended - check browser for login`);
+          console.log(`[MCP] OAuth may need completion - try using ${platform} tools in Claude`);
         }
       } else {
         const configBuilder = CREDENTIAL_MCP_CONFIGS[platform];

package/dist/cli.mjs

@@ -7,7 +7,7 @@ import {
   getCredentials,
   loadCredentials,
   promptForApiKey
-} from "./chunk-GZQ2C7RT.mjs";
+} from "./chunk-4BEMUW57.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 };
   }
@@ -6095,26 +6104,32 @@ var McpHandler = class {
       if (oauthServer) {
         const result = addMcpServer({
           name: serverName,
-          transport: "stdio",
-          command: "npx",
-          args: ["-y", "mcp-remote", oauthServer.url]
+          transport: "http",
+          url: oauthServer.url
         });
         if (!result.success) {
           throw new Error(`Failed to add ${platform} MCP: ${result.error}`);
         }
-        console.log(`[MCP] Added ${serverName} via mcp-remote`);
-        console.log(`[MCP] Triggering ${platform} OAuth...`);
-        const oauthProcess = (0, import_child_process2.spawnSync)("npx", ["-y", "mcp-remote", oauthServer.url], {
+        console.log(`[MCP] Added ${serverName} \u2192 ${oauthServer.url}`);
+        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.`,
+          "--allowedTools",
+          `mcp__${serverName}__*`,
+          "--permission-mode",
+          "default"
+        ], {
           encoding: "utf-8",
           stdio: "inherit",
-          // Show output and allow browser to open
+          // Show output and allow browser interaction
           timeout: 12e4
           // 2 minute timeout for user to complete OAuth
         });
         if (oauthProcess.status === 0) {
           console.log(`[MCP] ${platform} OAuth completed!`);
         } else {
-          console.log(`[MCP] OAuth process ended - check browser for login`);
+          console.log(`[MCP] OAuth may need completion - try using ${platform} tools in Claude`);
         }
       } else {
         const configBuilder = CREDENTIAL_MCP_CONFIGS[platform];

package/dist/index.mjs

@@ -14,7 +14,7 @@ import {
   promptForApiKey,
   runDeviceAuth,
   saveCredentials
-} from "./chunk-GZQ2C7RT.mjs";
+} from "./chunk-4BEMUW57.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.17",
+  "version": "1.3.19",
   "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 };
@@ -173,32 +185,36 @@ export class McpHandler {
       // Check if this is an OAuth-based server
       const oauthServer = OAUTH_MCP_SERVERS[platform];
       if (oauthServer) {
-        // Use mcp-remote for OAuth handling - it opens browser automatically
+        // Use http transport - Claude handles OAuth natively
         const result = addMcpServer({
           name: serverName,
-          transport: 'stdio',
-          command: 'npx',
-          args: ['-y', 'mcp-remote', oauthServer.url],
+          transport: 'http',
+          url: oauthServer.url,
         });
         
         if (!result.success) {
           throw new Error(`Failed to add ${platform} MCP: ${result.error}`);
         }
         
-        console.log(`[MCP] Added ${serverName} via mcp-remote`);
+        console.log(`[MCP] Added ${serverName} → ${oauthServer.url}`);
         
-        // Trigger OAuth by running the MCP server - mcp-remote will open browser
-        console.log(`[MCP] Triggering ${platform} OAuth...`);
-        const oauthProcess = spawnSync('npx', ['-y', 'mcp-remote', oauthServer.url], {
+        // Trigger OAuth by asking Claude to use the MCP - this opens browser!
+        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.`,
+          '--allowedTools', `mcp__${serverName}__*`,
+          '--permission-mode', 'default',
+        ], {
           encoding: 'utf-8',
-          stdio: 'inherit', // Show output and allow browser to open
+          stdio: 'inherit', // Show output and allow browser interaction
           timeout: 120000, // 2 minute timeout for user to complete OAuth
         });
         
         if (oauthProcess.status === 0) {
           console.log(`[MCP] ${platform} OAuth completed!`);
         } else {
-          console.log(`[MCP] OAuth process ended - check browser for login`);
+          console.log(`[MCP] OAuth may need completion - try using ${platform} tools in Claude`);
         }
       } else {
         // Command-based MCP server - needs credentials