get-claudia 1.53.2 → 1.53.3

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to Claudia will be documented in this file.
 
+## 1.53.3 (2026-03-04)
+
+### Fix: Actually restore disabled MCP servers on upgrade
+
+v1.53.2 added restore logic for `_disabled_`-prefixed keys in `mcpServers`, but an early return (`if (!config._disabled_mcpServers) return`) prevented it from running. The function now handles both migration paths independently: the `_disabled_mcpServers` stash (Path 1) and `_disabled_*` prefixed keys directly in `mcpServers` (Path 2). Path 2 is now generic and renames any `_disabled_*` key, not just gmail/google-calendar.
+
 ## 1.53.2 (2026-03-04)
 
 ### Re-enable Gmail and Calendar MCPs

package/bin/index.js

@@ -1014,31 +1014,38 @@ function restoreMcpServers(targetPath) {
   try {
     const raw = readFileSync(mcpPath, 'utf-8');
     const config = JSON.parse(raw);
-    if (!config._disabled_mcpServers) return;
     if (!config.mcpServers) config.mcpServers = {};
 
-    // Restore all previously disabled servers (memory, gmail, google-calendar)
-    const toRestore = ['claudia-memory', 'claudia_memory', 'gmail', 'google-calendar'];
     let changed = false;
     const restored = [];
 
-    for (const key of toRestore) {
-      if (config._disabled_mcpServers[key] && !config.mcpServers[key]) {
-        const serverConfig = { ...config._disabled_mcpServers[key] };
-        delete serverConfig._replaced_by;
-        delete serverConfig._warning;
-        config.mcpServers[key] = serverConfig;
-        delete config._disabled_mcpServers[key];
-        changed = true;
-        restored.push(key);
+    // Path 1: Restore from _disabled_mcpServers stash (older migration format)
+    if (config._disabled_mcpServers) {
+      const toRestore = ['claudia-memory', 'claudia_memory', 'gmail', 'google-calendar'];
+      for (const key of toRestore) {
+        if (config._disabled_mcpServers[key] && !config.mcpServers[key]) {
+          const serverConfig = { ...config._disabled_mcpServers[key] };
+          delete serverConfig._replaced_by;
+          delete serverConfig._warning;
+          config.mcpServers[key] = serverConfig;
+          delete config._disabled_mcpServers[key];
+          changed = true;
+          restored.push(key);
+        }
+      }
+
+      // Clean up _disabled_mcpServers if it's now empty
+      if (Object.keys(config._disabled_mcpServers).length === 0) {
+        delete config._disabled_mcpServers;
       }
     }
 
-    // Also rename _disabled_ prefixed keys in mcpServers itself
+    // Path 2: Rename _disabled_ prefixed keys in mcpServers itself
+    // This handles the case where keys like "_disabled_gmail" exist directly in mcpServers
     for (const key of Object.keys(config.mcpServers)) {
       if (key.startsWith('_disabled_')) {
         const realKey = key.replace('_disabled_', '');
-        if (['gmail', 'google-calendar'].includes(realKey) && !config.mcpServers[realKey]) {
+        if (!config.mcpServers[realKey]) {
           const serverConfig = { ...config.mcpServers[key] };
           delete serverConfig._warning;
           config.mcpServers[realKey] = serverConfig;
@@ -1049,14 +1056,9 @@ function restoreMcpServers(targetPath) {
       }
     }
 
-    // Clean up _disabled_mcpServers if it's now empty
-    if (config._disabled_mcpServers && Object.keys(config._disabled_mcpServers).length === 0) {
-      delete config._disabled_mcpServers;
-    }
-
     if (changed) {
       writeFileSync(mcpPath, JSON.stringify(config, null, 2) + '\n');
-      console.log(` ${colors.green}✓${colors.reset} Restored MCP servers: ${restored.join(', ')} (moved back from _disabled_mcpServers)`);
+      console.log(` ${colors.green}✓${colors.reset} Restored MCP servers: ${restored.join(', ')}`);
     }
   } catch {
     // Not valid JSON or can't read -- skip silently

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "get-claudia",
-  "version": "1.53.2",
+  "version": "1.53.3",
   "description": "An AI assistant who learns how you work.",
   "keywords": [
     "claudia",