sticky-note-cli 3.1.29 → 3.1.31

package/bin/cli.js

@@ -442,6 +442,35 @@ function detectMcpServers() {
     } catch (err) { debugLog("detectMcpServers copilot mcp-config: " + (err.message || err)); }
   }
 
+  // Also detect from ~/.claude/settings.json (global Claude Code settings).
+  // This captures servers like Render that users configure globally rather than per-project.
+  const globalClaudeSettingsPath = path.join(os.homedir(), ".claude", "settings.json");
+  if (fs.existsSync(globalClaudeSettingsPath)) {
+    try {
+      const globalSettings = JSON.parse(fs.readFileSync(globalClaudeSettingsPath, "utf-8"));
+      const globalMcp = globalSettings.mcpServers || {};
+      for (const [name, config] of Object.entries(globalMcp)) {
+        if (servers.has(name)) continue; // project-level takes precedence
+        if (config.type === "permission-detected") continue;
+        const entry = {
+          name,
+          type: config.type || config.transport || "stdio",
+          source: "~/.claude/settings.json",
+        };
+        if (config.command) entry.command = config.command;
+        if (config.args) entry.args = config.args;
+        if (config.env) entry.env = config.env;
+        if (config.url) entry.url = config.url;
+        servers.set(name, entry);
+      }
+    } catch (err) { debugLog("detectMcpServers global settings: " + (err.message || err)); }
+  }
+
+  // Detect servers known only from allow-list permissions (settings.local.json).
+  // These are servers Claude Code has permission to call but whose config isn't in
+  // any local file — typically account-level cloud MCPs (Figma, Sentry via claude.ai).
+  // We mark them "account-mcp" so the manifest records them without treating them
+  // as locally-provisionable servers.
   const localSettingsPath = path.join(process.cwd(), ".claude", "settings.local.json");
   if (fs.existsSync(localSettingsPath)) {
     try {
@@ -456,7 +485,7 @@ function detectMcpServers() {
           if (serverName && !servers.has(serverName)) {
             servers.set(serverName, {
               name: serverName,
-              type: "permission-detected",
+              type: "account-mcp",
               source: "settings.local.json",
             });
           }
@@ -1121,8 +1150,8 @@ async function cmdInit() {
   mkdirSafe(path.join(envDir, "commands"));
   if (!fs.existsSync(envManifestDest)) {
     // 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.
+    // Skip permission-detected (legacy marker with no real config).
+    // account-mcp entries are included — they're real cloud MCPs the team uses.
     const manifestData = { version: "1", mcp_servers: {}, permissions: [], env_vars: {} };
     for (const server of mcpServers.filter((s) => s.type !== "permission-detected")) {
       const entry = { type: server.type || "stdio", description: server.name, required: false };
@@ -1462,8 +1491,8 @@ function cmdUpdate() {
   mkdirSafe(path.join(envDir, "agents"));
   mkdirSafe(path.join(envDir, "commands"));
   if (!fs.existsSync(envManifestDest)) {
-    // 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.
+    // Create manifest from scratch, skipping permission-detected (legacy marker with no real config).
+    // account-mcp entries are included — they're real cloud MCPs the team uses.
     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) {
@@ -1495,6 +1524,22 @@ 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) {
+      // Convert permission-detected → account-mcp rather than deleting.
+      // These are real servers the team uses (e.g. Figma, Sentry via claude.ai account);
+      // they can't be locally provisioned but should still be documented in the manifest.
+      for (const k of stale) servers[k].type = "account-mcp";
+      manifest.mcp_servers = servers;
+      fs.writeFileSync(envManifestDest, JSON.stringify(manifest, null, 2) + "\n");
+      print(`  [OK] environment/manifest.json: upgraded ${stale.length} entry(s) from permission-detected to account-mcp`);
+    }
   }
 
   print(`\n  ✨ Scripts updated to v${VERSION}`);

package/bin/mcp-server.js

@@ -529,10 +529,18 @@ async function toolGetEnvironmentStatus() {
 
   const provisioned = { mcp_servers: [], skills: [], agents: [], commands: [] };
   const missing = { mcp_servers: [] };
+  const accountMcp = []; // account-level cloud MCPs — connect via claude.ai, not locally provisioned
 
   // Check MCP servers
   if (manifest.mcp_servers) {
     for (const [name, config] of Object.entries(manifest.mcp_servers)) {
+      // Account-level cloud MCPs (Figma, Sentry, etc.) are managed by the claude.ai
+      // account, not local config. Report them separately — not as missing.
+      if (config.type === "account-mcp") {
+        accountMcp.push({ name, description: config.description || name });
+        continue;
+      }
+
       if (registeredServers.includes(name)) {
         provisioned.mcp_servers.push(name);
       } else {
@@ -598,6 +606,11 @@ async function toolGetEnvironmentStatus() {
     action: hasMissing ? "inform_user_of_missing" : "none",
   };
 
+  if (accountMcp.length > 0) {
+    result.account_mcp = accountMcp;
+    result.account_mcp_note = "These MCP servers are managed by the claude.ai account. Each team member connects them via their own claude.ai account — no local provisioning needed.";
+  }
+
   if (hasMissing) {
     result.missing = missing;
     const requiredMissing = missing.mcp_servers.filter((s) => s.required);

package/package.json

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