claude-autopm 1.13.4 → 1.13.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-autopm",
-  "version": "1.13.4",
+  "version": "1.13.6",
   "description": "Autonomous Project Management Framework for Claude Code - Advanced AI-powered development automation",
   "main": "bin/autopm.js",
   "bin": {

package/scripts/mcp-handler.js

@@ -228,42 +228,49 @@ class MCPHandler {
     const config = this.loadConfig();
     const activeServers = config.mcp?.activeServers || [];
 
-    // Ensure .claude directory exists even if no servers
+    // Ensure .claude directory exists
     this.ensureClaudeDir();
 
+    // Read existing mcp-servers.json to preserve all servers
+    let existingMcpConfig = { mcpServers: {}, contextPools: {}, documentationSources: {} };
+    if (fs.existsSync(this.mcpServersPath)) {
+      try {
+        const content = fs.readFileSync(this.mcpServersPath, 'utf8');
+        existingMcpConfig = JSON.parse(content);
+      } catch (error) {
+        console.log('⚠️  Could not read existing mcp-servers.json, creating new');
+      }
+    }
+
     if (activeServers.length === 0) {
-      console.log('ℹ️ No active servers to sync');
-      // Still create empty mcp-servers.json
-      const emptyConfig = {
-        mcpServers: {},
-        contextPools: config.mcp?.contextPools || {},
-        documentationSources: config.mcp?.documentationSources || {}
-      };
-      fs.writeFileSync(
-        this.mcpServersPath,
-        JSON.stringify(emptyConfig, null, 2)
-      );
+      console.log('ℹ️ No active servers in config.json');
+      console.log('💡 Preserving existing servers in mcp-servers.json');
+      console.log(`📊 Existing servers: ${Object.keys(existingMcpConfig.mcpServers).length}`);
       return;
     }
 
+    // Start with existing configuration
     const mcpConfig = {
-      mcpServers: {},
-      contextPools: config.mcp?.contextPools || {},
-      documentationSources: config.mcp?.documentationSources || {}
+      mcpServers: existingMcpConfig.mcpServers || {},
+      contextPools: existingMcpConfig.contextPools || {},
+      documentationSources: existingMcpConfig.documentationSources || {}
     };
 
-    // Process each active server
+    // Update active servers from registry
     activeServers.forEach(serverName => {
       const server = this.getServer(serverName);
       if (!server) {
-        console.log(`  ⚠️ Server '${serverName}' not found, skipping`);
+        console.log(`  ⚠️ Server '${serverName}' not found in registry, skipping`);
         return;
       }
 
+      // Convert env metadata to simple strings for Claude Code compatibility
+      const envVars = this._convertEnvMetadataToStrings(server.metadata.env);
+
       mcpConfig.mcpServers[serverName] = {
         command: server.metadata.command,
         args: server.metadata.args,
-        env: server.metadata.env || {},
+        env: envVars,
         envFile: server.metadata.envFile
       };
 
@@ -278,6 +285,7 @@ class MCPHandler {
 
     console.log(`\n✅ Configuration synced to ${this.mcpServersPath}`);
     console.log(`📊 Active servers: ${activeServers.length}`);
+    console.log(`📦 Total servers in file: ${Object.keys(mcpConfig.mcpServers).length}`);
   }
 
   /**
@@ -1244,6 +1252,42 @@ This server can be integrated with various agents and context pools.
     return defaultValue !== '';
   }
 
+  /**
+   * Convert env metadata objects to simple string format for Claude Code
+   * @private
+   * @param {Object} envObj - Environment variables object (may contain metadata)
+   * @returns {Object} Environment variables as simple strings
+   */
+  _convertEnvMetadataToStrings(envObj) {
+    if (!envObj) return {};
+
+    const converted = {};
+
+    Object.entries(envObj).forEach(([key, value]) => {
+      // If value is already a string, keep it
+      if (typeof value === 'string') {
+        converted[key] = value;
+      }
+      // If value is metadata object, convert to string format
+      else if (typeof value === 'object' && value !== null) {
+        const defaultVal = value.default || '';
+        // If it's a literal value (not empty string), use it directly
+        if (defaultVal && !defaultVal.startsWith('${')) {
+          converted[key] = defaultVal;
+        }
+        // Otherwise use variable substitution format
+        else {
+          converted[key] = `\${${key}:-${defaultVal}}`;
+        }
+      }
+      else {
+        converted[key] = String(value);
+      }
+    });
+
+    return converted;
+  }
+
   /**
    * Run comprehensive MCP diagnostics
    * @returns {Object} Diagnostic results