patchcord 0.6.19 → 0.6.21

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "patchcord",
   "description": "Cross-machine agent messaging. Messages from other agents land in the inbox and wake the agent to reply.",
-  "version": "0.6.19",
+  "version": "0.6.21",
   "author": {
     "name": "ppravdin"
   },

package/bin/patchcord.mjs

@@ -43,71 +43,6 @@ function isSafeId(s) {
   return /^[A-Za-z0-9_\-]+$/.test(s) && s.length < 100;
 }
 
-/** cursor-agent --yolo often spawns `npx patchcord` without CURSOR_AGENT in env. */
-function _runningUnderCursorAgent() {
-  if (process.env.CURSOR_AGENT) return true;
-  if (process.platform === "linux") {
-    try {
-      const cmdline = readFileSync(`/proc/${process.ppid}/cmdline`, "utf-8").replace(/\0/g, " ");
-      if (/cursor-agent|\bcursor agent\b/i.test(cmdline)) return true;
-    } catch {}
-  }
-  return false;
-}
-
-/** Undo 0.6.14 mistake: Claude Code uses /mcp (not /mcp/bearer). */
-function revertClaudeMcpBearerUrl(mcpPath) {
-  if (!existsSync(mcpPath)) return false;
-  try {
-    const obj = JSON.parse(readFileSync(mcpPath, "utf-8"));
-    const pt = obj?.mcpServers?.patchcord;
-    const url = pt?.url;
-    if (!url || !url.endsWith("/mcp/bearer")) return false;
-    pt.url = url.replace(/\/mcp\/bearer$/, "/mcp");
-    writeFileSync(mcpPath, JSON.stringify(obj, null, 2) + "\n");
-    return true;
-  } catch {
-    return false;
-  }
-}
-
-/** Patchcord uses project .mcp.json only — drop stale local-scope duplicates. */
-function dedupeClaudePatchcordMcp(cwd) {
-  if (run("which claude")) {
-    run("claude mcp remove patchcord -s local");
-  }
-  const claudeJson = join(HOME, ".claude.json");
-  if (!existsSync(claudeJson)) return false;
-  try {
-    const obj = JSON.parse(readFileSync(claudeJson, "utf-8"));
-    let changed = false;
-    if (obj.mcpServers?.patchcord) {
-      delete obj.mcpServers.patchcord;
-      if (Object.keys(obj.mcpServers).length === 0) delete obj.mcpServers;
-      changed = true;
-    }
-    const proj = obj.projects?.[cwd];
-    if (proj?.mcpServers?.patchcord) {
-      delete proj.mcpServers.patchcord;
-      if (Object.keys(proj.mcpServers).length === 0) delete proj.mcpServers;
-      changed = true;
-    }
-    if (changed) {
-      writeFileSync(claudeJson, JSON.stringify(obj, null, 2) + "\n");
-    }
-    return changed;
-  } catch {
-    return false;
-  }
-}
-
-function fixClaudePatchcordMcp(cwd) {
-  const mcpPath = join(cwd, ".mcp.json");
-  const reverted = revertClaudeMcpBearerUrl(mcpPath);
-  const deduped = dedupeClaudePatchcordMcp(cwd);
-  return { reverted, deduped };
-}
-
 // A trailing --help/-h on ANY subcommand must show help and exit — never run the
 // command. (patchcord orchestrator --help used to PROVISION an orchestrator;
 // patchcord login --help hung on the browser login. Side-effecting --help is a
@@ -773,8 +708,6 @@ if (cmd === "subscribe") {
   const forceKimi = args.includes("--kimi");
   const intervalArg = args.find((a) => a !== "--kimi") || "30";
 
-  fixClaudePatchcordMcp(process.cwd());
-
   // Kimi Code uses polling instead of WebSocket realtime.
   // Force Kimi mode with --kimi flag, or auto-detect from bearer config.
   let isKimi = forceKimi;
@@ -1683,11 +1616,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
   const flags = cmd?.startsWith("--") ? process.argv.slice(2) : process.argv.slice(3);
   const fullStatusline = flags.includes("--full");
   const updateOnly = flags.includes("--update-only");
-  const toolFlagEarly = flags.find((f) => f.startsWith("--tool="))?.split("=")[1]
-    || (flags.includes("--tool") ? flags[flags.indexOf("--tool") + 1] : "");
-  const skipClaudePluginForCursor =
-    _runningUnderCursorAgent()
-    || toolFlagEarly.replace(/-/g, "_") === "cursor";
   let wasPluginInstalled = false;
   const { readFileSync, writeFileSync, unlinkSync, rmSync, chmodSync, copyFileSync } = await import("fs");
 
@@ -1733,8 +1661,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
     return out.replace(/,(\s*[}\]])/g, "$1");
   }
 
-  // dedupeClaudePatchcordMcp + revertClaudeMcpBearerUrl live at module scope.
-
   // Read+merge+write a JSON config file. If the file exists but its contents
   // can't be parsed, REFUSE to write — silently overwriting would erase
   // unrelated MCP servers, settings, or hand-edits the user has in there.
@@ -1793,15 +1719,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
   const bold = "\x1b[1m";
   const r = "\x1b[0m";
 
-  const _installCwd = process.cwd();
-  const _mcpFix = fixClaudePatchcordMcp(_installCwd);
-  if (_mcpFix.reverted) {
-    console.log(`  ${green}✓${r} Claude Code MCP URL reverted to /mcp (0.6.14 /mcp/bearer was wrong for CC)`);
-  }
-  if (_mcpFix.deduped) {
-    console.log(`  ${green}✓${r} Removed stale local-scope patchcord MCP (use project .mcp.json only)`);
-  }
-
   // ── Purge stale npx cache entries ────────────────────────────
   // npx never auto-updates a cached package. Users with an old `npx patchcord`
   // entry get the stale binary indefinitely. On every install we sweep
@@ -1867,11 +1784,9 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
   let globalChanges = [];
   if (globalCliInstalled) globalChanges.push("Patchcord CLI installed globally");
 
-  // Claude Code — skip when install runs inside cursor-agent: Claude's plugin
-  // MCP registers as "plugin-patchcord-marketplace-patchcord", which cursor-agent
-  // tries instead of project .cursor/mcp.json → "MCP server not found".
+  // Claude Code
   const hasClaude = run("which claude");
-  if (hasClaude && !skipClaudePluginForCursor) {
+  if (hasClaude) {
     // Remove legacy npm-cache install (pre-marketplace era) — causes duplicate /patchcord commands.
     const npmCachePatchcord = join(HOME, ".claude", "plugins", "npm-cache", "node_modules", "patchcord");
     if (existsSync(npmCachePatchcord)) {
@@ -1985,30 +1900,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
       } else if (allowlistChanged) {
         globalChanges.push("Cursor MCP allowlist configured");
       }
-
-      // cursor-agent v2026.07+ also reads ~/.cursor/cli-config.json permissions.allow
-      // for MCP tool approval — permissions.json alone leaves patchcord at
-      // "not loaded (needs approval)" even with --yolo.
-      const cliConfigPath = join(HOME, ".cursor", "cli-config.json");
-      let cliAllowChanged = false;
-      const cliOk = updateJsonConfig(cliConfigPath, (obj) => {
-        obj.permissions = obj.permissions || {};
-        if (!obj.permissions.allow) obj.permissions.allow = [];
-        if (!Array.isArray(obj.permissions.allow)) {
-          console.log(`\n  ${yellow}⚠${r} Skipped Cursor CLI MCP allow — ${cliConfigPath} permissions.allow is not an array.`);
-          return;
-        }
-        const pat = "Mcp(patchcord:*)";
-        if (!obj.permissions.allow.includes(pat)) {
-          obj.permissions.allow.push(pat);
-          cliAllowChanged = true;
-        }
-      });
-      if (!cliOk) {
-        globalChanges.push("✗ Cursor CLI permissions error");
-      } else if (cliAllowChanged) {
-        globalChanges.push("Cursor CLI MCP allowlist configured");
-      }
     }
   }
 
@@ -2446,13 +2337,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
     }
     toolSlug = toolFlag;  // preserved as-is for the URL param
   }
-  // cursor-agent --yolo runs `npx patchcord` without --tool=cursor. Without
-  // this, browser connect defaults to Claude and writes .mcp.json instead of
-  // .cursor/mcp.json — MCP never loads in cursor-agent.
-  if (!choice && process.env.CURSOR_AGENT) {
-    choice = CLIENT_TYPE_MAP.cursor;
-    toolSlug = toolSlug || "cursor";
-  }
 
   // --token bypass for power users / CI / self-hosters
   const tokenFlag = flags.find(f => f.startsWith("--token="))?.split("=")[1]
@@ -2503,14 +2387,12 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
     let existingToken = "";
     let existingConfigFile = "";
     const mcpJsonPath = join(cwd, ".mcp.json");
-    const cursorMcpPath = join(cwd, ".cursor", "mcp.json");
     const codexTomlPath = join(cwd, ".codex", "config.toml");
     const grokTomlPath = join(cwd, ".grok", "config.toml");
     const kimiJsonPath = join(cwd, ".kimi", "mcp.json");
 
-    const slugForCheck = toolSlug ? toolSlug.replace(/-/g, "_") : (process.env.CURSOR_AGENT ? "cursor" : "");
+    const slugForCheck = toolSlug ? toolSlug.replace(/-/g, "_") : "";
     const checkMcpJson = !slugForCheck || slugForCheck === "claude_code";
-    const checkCursorJson = !slugForCheck || slugForCheck === "cursor";
     const checkCodexToml = !slugForCheck || slugForCheck === "codex";
     const checkGrokToml = !slugForCheck || ["grok", "grok_cli", "grok_build"].includes(slugForCheck);
     const checkKimiJson = !slugForCheck || slugForCheck === "kimi";
@@ -2525,16 +2407,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
         }
       } catch {}
     }
-    if (!existingToken && checkCursorJson && existsSync(cursorMcpPath)) {
-      try {
-        const existing = JSON.parse(readFileSync(cursorMcpPath, "utf-8"));
-        const pt = existing?.mcpServers?.patchcord;
-        if (pt?.headers?.Authorization) {
-          existingToken = pt.headers.Authorization.replace(/^Bearer\s+/i, "");
-          existingConfigFile = cursorMcpPath;
-        }
-      } catch {}
-    }
     if (!existingToken && checkCodexToml && existsSync(codexTomlPath)) {
       try {
         const content = readFileSync(codexTomlPath, "utf-8");
@@ -2857,18 +2729,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
       console.log(`\n  ${green}✓${r} Cursor configured: ${dim}${cursorPath}${r}`);
       console.log(`  ${dim}Per-project only — other projects won't see this agent.${r}`);
     }
-    // Project-layer CLI permissions (cursor-agent merges .cursor/cli.json from git root).
-    const cursorCliPath = join(cursorDir, "cli.json");
-    const cursorCliOk = updateJsonConfig(cursorCliPath, (obj) => {
-      obj.permissions = obj.permissions || {};
-      if (!obj.permissions.allow) obj.permissions.allow = [];
-      if (!Array.isArray(obj.permissions.allow)) return;
-      const pat = "Mcp(patchcord:*)";
-      if (!obj.permissions.allow.includes(pat)) obj.permissions.allow.push(pat);
-    });
-    if (cursorCliOk) {
-      console.log(`  ${green}✓${r} Cursor CLI permissions: ${dim}${cursorCliPath}${r}`);
-    }
   } else if (isGrok) {
     // Grok CLI: project-scoped .grok/config.toml. Grok loads this file from
     // the current directory up through the git root, so the bearer remains
@@ -3410,7 +3270,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
       // their JSON and re-runs.
       process.exit(1);
     }
-    dedupeClaudePatchcordMcp(cwd);
     console.log(`\n  ${green}✓${r} Claude Code configured: ${dim}${mcpPath}${r}`);
 
     // Statusline: ask whether to enable full mode (unless --full flag passed or already configured).
@@ -3526,9 +3385,6 @@ if (!cmd || cmd === "install" || cmd === "agent" || cmd?.startsWith("--")) {
     console.log(`\n  ${green}→${r} ${bold}Restart ${cyan}kimi${r}${bold} in this project (or run ${cyan}/reload${r}${bold}), then say: ${cyan}${bold}check inbox${r}`);
   } else if (isKimi) {
     console.log(`\n  ${green}→${r} ${bold}Restart your ${toolName} session with ${cyan}kimi-pc${r}${bold}, then say: ${cyan}${bold}check inbox${r}`);
-  } else if (isCursor && hasCursorAgent) {
-    console.log(`\n  ${green}→${r} ${bold}Exit and start a new ${cyan}cursor-agent${r}${bold} session${r} ${dim}(MCP loads at session start — not mid-session)${r}`);
-    console.log(`  ${dim}Then:${r} ${cyan}${bold}check inbox${r} ${dim}before${r} ${cyan}${bold}/patchcord-subscribe${r}`);
   } else {
     console.log(`\n  ${green}→${r} ${bold}Restart your ${toolName} session${r}, then say: ${cyan}${bold}check inbox${r}`);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchcord",
-  "version": "0.6.19",
+  "version": "0.6.21",
   "description": "Cross-machine agent messaging for Claude Code and Codex",
   "scripts": {
     "version": "node scripts/sync-plugin-version.mjs && git add .claude-plugin/plugin.json"