sticky-note-cli 3.1.28 → 3.1.30

package/bin/cli.js

@@ -1120,9 +1120,11 @@ async function cmdInit() {
   mkdirSafe(path.join(envDir, "agents"));
   mkdirSafe(path.join(envDir, "commands"));
   if (!fs.existsSync(envManifestDest)) {
-    // Build manifest from detected MCP servers and skills
+    // Build manifest from detected MCP servers and skills.
+    // Skip permission-detected entries — they are not real server configs and would
+    // be synced back into .mcp.json by session-start.js, breaking Claude Code.
     const manifestData = { version: "1", mcp_servers: {}, permissions: [], env_vars: {} };
-    for (const server of mcpServers) {
+    for (const server of mcpServers.filter((s) => s.type !== "permission-detected")) {
       const entry = { type: server.type || "stdio", description: server.name, required: false };
       if (server.command) entry.command = server.command;
       if (server.args) entry.args = server.args;
@@ -1460,7 +1462,8 @@ function cmdUpdate() {
   mkdirSafe(path.join(envDir, "agents"));
   mkdirSafe(path.join(envDir, "commands"));
   if (!fs.existsSync(envManifestDest)) {
-    const detectedServers = Array.from(detectMcpServers().values());
+    // Create manifest from scratch, skipping permission-detected entries.
+    const detectedServers = Array.from(detectMcpServers().values()).filter((s) => s.type !== "permission-detected");
     const manifestData = { version: "1", mcp_servers: {}, permissions: [], env_vars: {} };
     for (const server of detectedServers) {
       const entry = { type: server.type || "stdio", description: server.name, required: false };
@@ -1491,6 +1494,19 @@ function cmdUpdate() {
     }
     fs.writeFileSync(envManifestDest, JSON.stringify(manifestData, null, 2) + "\n");
     print(`  [OK] Created environment/ in ${envDir} (${detectedServers.length} MCP server(s) detected)`);
+  } else {
+    // Manifest already exists — strip any permission-detected entries that may have
+    // been written by an older version of sticky-note. These are not real server
+    // configs and cause get_environment_status to report false "missing" servers.
+    const manifest = readJsonSafe(envManifestDest, {});
+    const servers = manifest.mcp_servers || {};
+    const stale = Object.keys(servers).filter((k) => servers[k].type === "permission-detected");
+    if (stale.length > 0) {
+      for (const k of stale) delete servers[k];
+      manifest.mcp_servers = servers;
+      fs.writeFileSync(envManifestDest, JSON.stringify(manifest, null, 2) + "\n");
+      print(`  [OK] environment/manifest.json: removed ${stale.length} stale permission-detected entry(s)`);
+    }
   }
 
   print(`\n  ✨ Scripts updated to v${VERSION}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sticky-note-cli",
-  "version": "3.1.28",
+  "version": "3.1.30",
   "description": "Human-to-human handoff for AI coding assistants. Git-backed shared memory for Claude Code and Copilot CLI.",
   "license": "MIT",
   "bin": {

package/templates/hooks/session-start.js

@@ -745,6 +745,7 @@ function ensureEnvironmentProvisioned() {
     let mcpChanged = false;
     for (const [name, serverDef] of Object.entries(mcpServers)) {
       if (mcpConfig.mcpServers[name]) continue; // already present
+      if (serverDef.type === "permission-detected") continue; // not a real server config
       if (hasEnvPlaceholders(serverDef)) {
         if (debug) process.stderr.write(`[sticky-note] skipping MCP server "${name}" (has placeholders)\n`);
         continue;