@i18n-agent/mcp-client 1.8.463 → 1.9.1

package/install.js

@@ -153,21 +153,40 @@ function checkExistingApiKey(configPath) {
   }
 }
 
-async function checkExistingApiKeys(availableIDEs) {
+async function checkExistingApiKeys(availableIDEs, claudeCodeCLIAvailable = false, codexCLIAvailable = false) {
   const withKeys = [];
   const withoutKeys = [];
 
   for (const ide of availableIDEs) {
-    // Also check ~/.claude.json for Claude Code CLI
-    if (ide.key === 'claude-code') {
-      const claudeJsonPath = path.join(os.homedir(), '.claude.json');
-      if (checkExistingApiKey(claudeJsonPath)) {
-        withKeys.push(ide);
-        continue;
+    let hasKey = false;
+
+    // Check CLI registrations first for native CLI tools
+    if (ide.key === 'claude-code' && claudeCodeCLIAvailable) {
+      const cliKey = getClaudeCodeExistingApiKey('i18n-agent');
+      if (cliKey) {
+        hasKey = true;
+      }
+    } else if (ide.key === 'codex' && codexCLIAvailable) {
+      const cliKey = getCodexExistingApiKey('i18n-agent');
+      if (cliKey) {
+        hasKey = true;
+      }
+    }
+
+    // Fall back to config file check
+    if (!hasKey) {
+      // Also check ~/.claude.json for Claude Code CLI
+      if (ide.key === 'claude-code') {
+        const claudeJsonPath = path.join(os.homedir(), '.claude.json');
+        if (checkExistingApiKey(claudeJsonPath)) {
+          hasKey = true;
+        }
+      } else if (checkExistingApiKey(ide.configPath)) {
+        hasKey = true;
       }
     }
 
-    if (checkExistingApiKey(ide.configPath)) {
+    if (hasKey) {
       withKeys.push(ide);
     } else {
       withoutKeys.push(ide);
@@ -201,7 +220,7 @@ function detectNodeEnvironment() {
   const nvmDir = process.env.NVM_DIR || path.join(os.homedir(), '.nvm');
   const nodeVersion = process.version;
   const nodePath = process.execPath;
-  
+
   return {
     isNvm: nodePath.includes('.nvm') || nodePath.includes('nvm'),
     nodePath,
@@ -209,6 +228,195 @@ function detectNodeEnvironment() {
   };
 }
 
+// Detect if Codex CLI is available
+function isCodexCLIAvailable() {
+  try {
+    execSync('codex --version', { stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Check if MCP server is already registered in Codex
+function isCodexMCPRegistered(serverName) {
+  try {
+    const result = execSync(`codex mcp get ${serverName}`, { stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Install MCP server via Codex CLI native command
+function installViaCodexCLI(existingApiKey = '') {
+  const { mcpClientPath, packageDir } = getMcpClientPaths();
+  const nodeEnv = detectNodeEnvironment();
+
+  // Determine node command - use absolute path for nvm
+  const nodeCmd = nodeEnv.isNvm ? nodeEnv.nodePath : 'node';
+
+  // Get existing API key from CLI registration BEFORE removing
+  if (!existingApiKey) {
+    existingApiKey = getCodexExistingApiKey('i18n-agent');
+    if (existingApiKey) {
+      console.log('   🔑 Preserving existing API key from CLI registration');
+    }
+  }
+
+  // Remove existing registration if present
+  if (isCodexMCPRegistered('i18n-agent')) {
+    try {
+      execSync('codex mcp remove i18n-agent', { stdio: 'pipe' });
+      console.log('   🔄 Removed existing i18n-agent registration');
+    } catch {
+      // Ignore if removal fails
+    }
+  }
+
+  // Build the command with env vars
+  const envArgs = [
+    '--env', `MCP_SERVER_URL=https://mcp.i18nagent.ai`
+  ];
+
+  if (existingApiKey) {
+    envArgs.push('--env', `API_KEY=${existingApiKey}`);
+  } else {
+    envArgs.push('--env', 'API_KEY=');
+  }
+
+  // Build the full command
+  // codex mcp add [--env KEY=VALUE]... <name> <command> [args...]
+  const cmdParts = [
+    'codex', 'mcp', 'add',
+    ...envArgs,
+    'i18n-agent',
+    nodeCmd,
+    mcpClientPath
+  ];
+
+  // Execute the command
+  execSync(cmdParts.join(' '), {
+    cwd: packageDir,
+    stdio: 'pipe'
+  });
+
+  return true;
+}
+
+// Detect if Claude Code CLI is available
+function isClaudeCodeCLIAvailable() {
+  try {
+    execSync('claude mcp list', { stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Check if MCP server is already registered in Claude Code CLI
+function isClaudeCodeMCPRegistered(serverName) {
+  try {
+    execSync(`claude mcp get ${serverName}`, { stdio: 'pipe' });
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Get existing API key from Claude Code CLI registration
+function getClaudeCodeExistingApiKey(serverName) {
+  try {
+    const output = execSync(`claude mcp get ${serverName}`, { stdio: 'pipe', encoding: 'utf8' });
+    // Parse output like: API_KEY=i18n_xxx
+    const match = output.match(/API_KEY=([^\s,\n]+)/);
+    if (match && match[1] && match[1] !== '') {
+      return match[1];
+    }
+  } catch {
+    // Server not registered
+  }
+  return '';
+}
+
+// Get existing API key from Codex CLI registration
+function getCodexExistingApiKey(serverName) {
+  try {
+    const output = execSync(`codex mcp list`, { stdio: 'pipe', encoding: 'utf8' });
+    // Parse output - Codex shows: API_KEY=i18n_xxx, MCP_SERVER_URL=...
+    const lines = output.split('\n');
+    for (const line of lines) {
+      if (line.includes(serverName)) {
+        const match = line.match(/API_KEY=([^\s,]+)/);
+        if (match && match[1] && match[1] !== '') {
+          return match[1];
+        }
+      }
+    }
+  } catch {
+    // Server not registered
+  }
+  return '';
+}
+
+// Install MCP server via Claude Code CLI native command
+function installViaClaudeCodeCLI(existingApiKey = '', scope = 'user') {
+  const { mcpClientPath } = getMcpClientPaths();
+  const nodeEnv = detectNodeEnvironment();
+
+  // Determine node command - use absolute path for nvm
+  const nodeCmd = nodeEnv.isNvm ? nodeEnv.nodePath : 'node';
+
+  // Get existing API key from CLI registration BEFORE removing
+  if (!existingApiKey) {
+    existingApiKey = getClaudeCodeExistingApiKey('i18n-agent');
+    if (existingApiKey) {
+      console.log('   🔑 Preserving existing API key from CLI registration');
+    }
+  }
+
+  // Remove existing registration if present
+  if (isClaudeCodeMCPRegistered('i18n-agent')) {
+    try {
+      execSync(`claude mcp remove --scope ${scope} i18n-agent`, { stdio: 'pipe' });
+      console.log('   🔄 Removed existing i18n-agent registration');
+    } catch {
+      // Ignore if removal fails
+    }
+  }
+
+  // Build env args using -e format (Claude uses -e, not --env)
+  // Note: -e args must come AFTER the server name
+  const envArgs = [
+    '-e', `MCP_SERVER_URL=https://mcp.i18nagent.ai`
+  ];
+
+  if (existingApiKey) {
+    envArgs.push('-e', `API_KEY=${existingApiKey}`);
+  } else {
+    envArgs.push('-e', 'API_KEY=');
+  }
+
+  // Build the full command
+  // claude mcp add --transport stdio --scope user <name> -e KEY=VALUE -- <command> [args...]
+  // Note: <name> must come BEFORE -e options
+  const cmdParts = [
+    'claude', 'mcp', 'add',
+    '--transport', 'stdio',
+    '--scope', scope,
+    'i18n-agent',  // Name must come before -e options
+    ...envArgs,
+    '--',  // Separator for command arguments
+    nodeCmd,
+    mcpClientPath
+  ];
+
+  // Execute the command
+  execSync(cmdParts.join(' '), { stdio: 'pipe' });
+
+  return true;
+}
+
 function createWrapperScript(targetDir) {
   const nodeEnv = detectNodeEnvironment();
   const wrapperPath = path.join(targetDir, 'run-mcp.sh');
@@ -407,8 +615,12 @@ For manual setup instructions, visit: https://docs.i18nagent.ai/setup
     });
     console.log('');
 
+    // Check if native CLIs are available (needed for API key detection)
+    const codexCLIAvailable = isCodexCLIAvailable();
+    const claudeCodeCLIAvailable = isClaudeCodeCLIAvailable();
+
     // Check for existing API keys BEFORE installation
-    const { withKeys, withoutKeys } = await checkExistingApiKeys(availableIDEs);
+    const { withKeys, withoutKeys } = await checkExistingApiKeys(availableIDEs, claudeCodeCLIAvailable, codexCLIAvailable);
 
     if (withKeys.length > 0 && withoutKeys.length === 0) {
       console.log(`✅ API Keys Already Configured:`);
@@ -448,11 +660,92 @@ For manual setup instructions, visit: https://docs.i18nagent.ai/setup
     const idesWithApiKey = [];
     const idesNeedingApiKey = [];
 
+    // Show native CLI detection message (CLIs already detected above)
+    if (codexCLIAvailable || claudeCodeCLIAvailable) {
+      console.log('🔧 Native CLI support detected:');
+      if (claudeCodeCLIAvailable) {
+        console.log('   - Claude Code CLI (`claude mcp add`)');
+      }
+      if (codexCLIAvailable) {
+        console.log('   - Codex CLI (`codex mcp add`)');
+      }
+      console.log('');
+    }
+
     for (const ide of availableIDEs) {
       try {
         console.log(`⚙️  Configuring ${ide.name}...`);
 
         let result;
+
+        // Special handling for Claude Code CLI - use native CLI if available
+        if (ide.key === 'claude-code' && claudeCodeCLIAvailable) {
+          // Get existing API key - check CLI registration first, then config file
+          let existingApiKey = getClaudeCodeExistingApiKey('i18n-agent');
+          if (!existingApiKey && fs.existsSync(ide.configPath)) {
+            try {
+              const content = fs.readFileSync(ide.configPath, 'utf8');
+              const config = JSON.parse(content);
+              existingApiKey = config.mcpServers?.["i18n-agent"]?.env?.API_KEY || '';
+            } catch {
+              // Ignore parse errors
+            }
+          }
+
+          try {
+            installViaClaudeCodeCLI(existingApiKey);
+            console.log(`✅ ${ide.name} configured via native CLI!`);
+            console.log(`   Run 'claude mcp list' to verify\n`);
+            installCount++;
+            installedIDEs.push(ide);
+
+            // Track API key status
+            if (existingApiKey) {
+              idesWithApiKey.push(ide);
+            } else {
+              idesNeedingApiKey.push(ide);
+            }
+            continue;
+          } catch (cliError) {
+            console.log(`   ⚠️  Native CLI failed, falling back to config file...`);
+            // Fall through to config file method
+          }
+        }
+
+        // Special handling for Codex - use native CLI if available
+        if (ide.key === 'codex' && codexCLIAvailable) {
+          // Get existing API key - check CLI registration first, then config file
+          let existingApiKey = getCodexExistingApiKey('i18n-agent');
+          if (!existingApiKey && fs.existsSync(ide.configPath)) {
+            try {
+              const content = fs.readFileSync(ide.configPath, 'utf8');
+              const config = JSON.parse(content);
+              existingApiKey = config.mcpServers?.["i18n-agent"]?.env?.API_KEY || '';
+            } catch {
+              // Ignore parse errors
+            }
+          }
+
+          try {
+            installViaCodexCLI(existingApiKey);
+            console.log(`✅ ${ide.name} configured via native CLI!`);
+            console.log(`   Run 'codex mcp list' to verify\n`);
+            installCount++;
+            installedIDEs.push(ide);
+
+            // Track API key status
+            if (existingApiKey) {
+              idesWithApiKey.push(ide);
+            } else {
+              idesNeedingApiKey.push(ide);
+            }
+            continue;
+          } catch (cliError) {
+            console.log(`   ⚠️  Native CLI failed, falling back to config file...`);
+            // Fall through to config file method
+          }
+        }
+
         if (ide.key === 'claude' || ide.key === 'claude-code') {
           result = updateClaudeConfig(ide.configPath, ide.key);
         } else {
@@ -495,6 +788,12 @@ For manual setup instructions, visit: https://docs.i18nagent.ai/setup
         });
         console.log('');
 
+        // Check if CLIs need API key
+        const codexNeedsKey = idesNeedingApiKey.some(ide => ide.key === 'codex');
+        const claudeCodeNeedsKey = idesNeedingApiKey.some(ide => ide.key === 'claude-code');
+        const showCodexInstructions = codexNeedsKey && codexCLIAvailable;
+        const showClaudeCodeInstructions = claudeCodeNeedsKey && claudeCodeCLIAvailable;
+
         // Show setup instructions only for IDEs that need them
         console.log(`🔑 Setup Instructions
 ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━
@@ -503,7 +802,25 @@ Step 1: Get your API key
    👉 Sign up or log in
    👉 Copy your API key (starts with "i18n_")
 
-Step 2: Add API key to the config file
+Step 2: Add API key to your IDE`);
+
+        if (showClaudeCodeInstructions) {
+          console.log(`
+   For Claude Code CLI (recommended):
+   claude mcp remove --scope user i18n-agent
+   claude mcp add --transport stdio --scope user i18n-agent -e MCP_SERVER_URL=https://mcp.i18nagent.ai -e API_KEY=your_key_here -- node ~/.claude/mcp-servers/i18n-agent/mcp-client.js
+`);
+        }
+
+        if (showCodexInstructions) {
+          console.log(`
+   For Codex CLI (recommended):
+   codex mcp remove i18n-agent
+   codex mcp add --env MCP_SERVER_URL=https://mcp.i18nagent.ai --env API_KEY=your_key_here i18n-agent node ~/.claude/mcp-servers/i18n-agent/mcp-client.js
+`);
+        }
+
+        console.log(`   For config file method:
    Open the config file and edit the "API_KEY" field:
 
    "mcpServers": {

package/mcp-client.js

@@ -5,7 +5,7 @@
  * Integrates with Claude Code CLI to provide translation capabilities
  */
 
-const MCP_CLIENT_VERSION = '1.8.461';
+const MCP_CLIENT_VERSION = '1.9.1';
 
 import { Server } from '@modelcontextprotocol/sdk/server/index.js';
 import { StdioServerTransport } from '@modelcontextprotocol/sdk/server/stdio.js';
@@ -49,7 +49,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
     tools: [
       {
         name: 'translate_text',
-        description: '⚠️ CRITICAL: For multi-language translation, use targetLanguages parameter (not targetLanguage). Translate text content with cultural adaptation using AI subagents. Supports both single and multi-language translation. For large requests (>100 texts or >50,000 characters), returns a jobId for async processing. Use check_translation_status to monitor progress and download results. Set pseudoTranslation=true for testing i18n implementations without AI cost.',
+        description: 'Translate text content with cultural adaptation using AI subagents. Supports single or multi-language translation via targetLanguages parameter (string for single, array for multiple). For large requests (>100 texts or >50,000 characters), returns a jobId for async processing. Use check_translation_status to monitor progress and download results. Set pseudoTranslation=true for testing i18n implementations without AI cost.',
         inputSchema: {
           type: 'object',
           properties: {
@@ -59,7 +59,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               description: 'Array of source texts to translate (any language)',
             },
             targetLanguages: {
-              description: '⚠️ REQUIRED: Target language(s) - can be a single string (e.g., "es") OR an array of strings (e.g., ["es", "fr", "zh-CN"]) for multi-language translation',
+              description: 'Target language(s) - provide a single string (e.g., "es") OR an array (e.g., ["es", "fr", "zh-CN"]) for multi-language translation. Use specific locale codes like "en-US" or "en-GB" instead of generic "en".',
               oneOf: [
                 { type: 'string' },
                 { type: 'array', items: { type: 'string' } }
@@ -144,7 +144,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
       },
       {
         name: 'translate_file',
-        description: '⚠️ CRITICAL: For multi-language translation, use targetLanguages parameter (not targetLanguage). Translate file content while preserving structure and format. Supports both single and multi-language translation. Supports JSON, YAML, XML, CSV, TXT, MD, and other text files. For large files (>100KB), returns a jobId for async processing. Use check_translation_status to monitor progress and download results. Set pseudoTranslation=true for testing i18n implementations without AI cost.',
+        description: 'Translate file content while preserving structure and format. Supports single or multi-language translation via targetLanguages parameter (string for single, array for multiple). Supports JSON, YAML, XML, CSV, TXT, MD, and other text files. Always returns a jobId for async processing - use check_translation_status to monitor progress and download_translations to get results. Set pseudoTranslation=true for testing i18n implementations without AI cost.',
         inputSchema: {
           type: 'object',
           properties: {
@@ -163,7 +163,7 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
               default: 'auto',
             },
             targetLanguages: {
-              description: '⚠️ REQUIRED: Target language(s) - can be a single string (e.g., "es") OR an array of strings (e.g., ["es", "fr", "zh-CN"]) for multi-language translation',
+              description: 'Target language(s) - provide a single string (e.g., "es") OR an array (e.g., ["es", "fr", "zh-CN"]) for multi-language translation. Use specific locale codes like "en-US" or "en-GB" instead of generic "en".',
               oneOf: [
                 { type: 'string' },
                 { type: 'array', items: { type: 'string' } }
@@ -555,17 +555,9 @@ async function handleTranslateText(args) {
   // Namespace is optional for text translation, but recommended for organizational tracking
 
   // Normalize targetLanguages - accept both string and array
-  let targetLanguages = rawTargetLanguages;
-  let targetLanguage = undefined;
-
-  if (typeof rawTargetLanguages === 'string') {
-    // Single language provided as string - convert to array for internal processing
-    targetLanguages = [rawTargetLanguages];
-    targetLanguage = rawTargetLanguages;
-  } else if (Array.isArray(rawTargetLanguages) && rawTargetLanguages.length === 1) {
-    // Single language provided as array - extract for backward compatibility
-    targetLanguage = rawTargetLanguages[0];
-  }
+  const targetLanguages = typeof rawTargetLanguages === 'string'
+    ? [rawTargetLanguages]
+    : rawTargetLanguages;
 
   if (!targetLanguages?.length) {
     throw new Error('targetLanguages parameter is required (can be a string for single language or array for multiple languages)');
@@ -585,7 +577,6 @@ async function handleTranslateText(args) {
       arguments: {
         apiKey: API_KEY,
         texts: texts,
-        targetLanguage: targetLanguage,
         targetLanguages: targetLanguages,
         sourceLanguage: sourceLanguage && sourceLanguage !== 'auto' ? sourceLanguage : undefined,
         targetAudience: targetAudience,
@@ -640,12 +631,12 @@ async function handleTranslateText(args) {
           // Extract the actual translation result from the job result
           if (jobResult && jobResult.content && jobResult.content[0]) {
             const translationData = JSON.parse(jobResult.content[0].text);
-            return formatTranslationResult(translationData, texts, targetLanguage, sourceLanguage, targetAudience, industry, region);
+            return formatTranslationResult(translationData, texts, targetLanguages, sourceLanguage, targetAudience, industry, region);
           }
           return jobResult;
         } else {
           // Regular synchronous result
-          return formatTranslationResult(parsed, texts, targetLanguage, sourceLanguage, targetAudience, industry, region);
+          return formatTranslationResult(parsed, texts, targetLanguages, sourceLanguage, targetAudience, industry, region);
         }
       } catch {
         // Not JSON or error parsing - return as-is
@@ -894,17 +885,9 @@ async function handleTranslateFile(args) {
   }
 
   // Normalize targetLanguages - accept both string and array
-  let targetLanguages = rawTargetLanguages;
-  let targetLanguage = undefined;
-
-  if (typeof rawTargetLanguages === 'string') {
-    // Single language provided as string - convert to array for internal processing
-    targetLanguages = [rawTargetLanguages];
-    targetLanguage = rawTargetLanguages;
-  } else if (Array.isArray(rawTargetLanguages) && rawTargetLanguages.length === 1) {
-    // Single language provided as array - extract for backward compatibility
-    targetLanguage = rawTargetLanguages[0];
-  }
+  const targetLanguages = typeof rawTargetLanguages === 'string'
+    ? [rawTargetLanguages]
+    : rawTargetLanguages;
 
   if (!targetLanguages?.length) {
     throw new Error('targetLanguages parameter is required (can be a string for single language or array for multiple languages)');
@@ -939,7 +922,6 @@ async function handleTranslateFile(args) {
   };
 
   // Add optional parameters only if defined
-  if (targetLanguage !== undefined) requestArgs.targetLanguage = targetLanguage;
   if (targetLanguages !== undefined) requestArgs.targetLanguages = targetLanguages;
   if (region !== undefined) requestArgs.region = region;
   if (context !== undefined) requestArgs.context = context;
@@ -1078,14 +1060,15 @@ async function handleTranslateFile(args) {
 }
 
 // Format translation result for consistent output
-function formatTranslationResult(parsedResult, texts, targetLanguage, sourceLanguage, targetAudience, industry, region) {
+function formatTranslationResult(parsedResult, texts, targetLanguages, sourceLanguage, targetAudience, industry, region) {
+  const targetLangsDisplay = Array.isArray(targetLanguages) ? targetLanguages.join(', ') : targetLanguages;
   return {
     translatedTexts: parsedResult?.translatedTexts || [],
     content: [
       {
         type: 'text',
         text: `Translation Results:\n\n` +
-              `🌍 ${parsedResult?.sourceLanguage || sourceLanguage || 'Auto-detected'} → ${parsedResult?.targetLanguage || targetLanguage}\n` +
+              `🌍 ${parsedResult?.sourceLanguage || sourceLanguage || 'Auto-detected'} → ${parsedResult?.targetLanguages?.join(', ') || targetLangsDisplay}\n` +
               `👥 Audience: ${parsedResult?.targetAudience || targetAudience}\n` +
               `🏭 Industry: ${parsedResult?.industry || industry}\n` +
               `${parsedResult?.region || region ? `📍 Region: ${parsedResult?.region || region}\n` : ''}` +

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@i18n-agent/mcp-client",
-  "version": "1.8.463",
+  "version": "1.9.1",
   "description": "🌍 i18n-agent MCP Client - 48 languages, AI-powered translation for Claude, Claude Code, Cursor, VS Code, Codex. Get API key at https://app.i18nagent.ai",
   "main": "mcp-client.js",
   "bin": {