mcpick 0.0.12 → 0.0.13

package/CHANGELOG.md

@@ -1,5 +1,12 @@
 # mcpick
 
+## 0.0.13
+
+### Patch Changes
+
+- c985c28: Headers lost on disable/enable: pass -H flags in CLI, sync
+  config→registry before disable
+
 ## 0.0.12
 
 ### Patch Changes

package/dist/cli/commands/disable.js

@@ -1,4 +1,5 @@
 import { defineCommand } from 'citty';
+import { get_all_available_servers } from '../../core/registry.js';
 import { remove_mcp_via_cli } from '../../utils/claude-cli.js';
 import { error } from '../output.js';
 export default defineCommand({
@@ -23,6 +24,8 @@ export default defineCommand({
         if (!['local', 'project', 'user'].includes(scope)) {
             error(`Invalid scope: ${scope}. Use local, project, or user.`);
         }
+        // Sync config→registry before removing so headers/env are preserved
+        await get_all_available_servers();
         const result = await remove_mcp_via_cli(args.server);
         if (!result.success) {
             error(result.error || 'Failed to disable server');

package/dist/core/registry.js

@@ -44,13 +44,29 @@ export async function get_all_available_servers() {
     const registry = await read_server_registry();
     const config = await read_claude_config();
     const config_servers = get_enabled_servers(config);
-    // Merge: registry is base, config servers fill in any missing
-    const known_names = new Set(registry.servers.map((s) => s.name));
+    // Merge: config is the live truth, so update registry entries with config data
+    const config_by_name = new Map(config_servers.map((s) => [s.name, s]));
+    const known_names = new Set();
+    let registry_updated = false;
+    for (let i = 0; i < registry.servers.length; i++) {
+        const name = registry.servers[i].name;
+        known_names.add(name);
+        const config_server = config_by_name.get(name);
+        if (config_server) {
+            registry.servers[i] = config_server;
+            registry_updated = true;
+        }
+    }
     for (const server of config_servers) {
         if (!known_names.has(server.name)) {
             registry.servers.push(server);
+            registry_updated = true;
         }
     }
+    // Persist updated data back to registry so it survives disable/enable cycles
+    if (registry_updated) {
+        await write_server_registry(registry);
+    }
     return registry.servers;
 }
 export async function sync_servers_to_registry(servers) {

package/dist/utils/claude-cli.js

@@ -59,10 +59,15 @@ function build_add_command(server, scope) {
         }
     }
     else {
-        // HTTP or SSE transport
+        // HTTP or SSE transport — URL must come before header flags
         if ('url' in server && server.url) {
             parts.push(shell_escape(server.url));
         }
+        if ('headers' in server && server.headers) {
+            for (const [key, value] of Object.entries(server.headers)) {
+                parts.push('-H', shell_escape(`${key}: ${value}`));
+            }
+        }
     }
     return parts.join(' ');
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcpick",
-  "version": "0.0.12",
+  "version": "0.0.13",
   "description": "Dynamic MCP server and plugin configuration manager for Claude Code",
   "type": "module",
   "main": "./dist/index.js",