patchcord 0.3.43 → 0.3.46

package/bin/patchcord.mjs

@@ -20,6 +20,10 @@ function run(cmd) {
   }
 }
 
+function isSafeToken(t) {
+  return /^[A-Za-z0-9_\-=+/.]+$/.test(t) && t.length < 200;
+}
+
 if (cmd === "help" || cmd === "--help" || cmd === "-h") {
   console.log(`patchcord — agent messaging for AI coding agents
 
@@ -404,6 +408,22 @@ if (!cmd || cmd === "install" || cmd === "agent") {
       } catch {}
     }
   } else if (isCodex) {
+    // Check global config for stale patchcord MCP — user may have run installer in ~ by mistake
+    const globalCodexConfig = join(HOME, ".codex", "config.toml");
+    if (existsSync(globalCodexConfig)) {
+      const globalContent = readFileSync(globalCodexConfig, "utf-8");
+      if (globalContent.includes("[mcp_servers.patchcord]")) {
+        console.log(`\n  ${red}⚠ Patchcord is in your GLOBAL Codex config!${r}`);
+        console.log(`  ${dim}${globalCodexConfig}${r}`);
+        console.log(`  ${yellow}This overrides per-project config and causes conflicts.${r}`);
+        const cleanGlobal = (await ask(`  ${dim}Remove patchcord from global config? (Y/n):${r} `)).trim().toLowerCase();
+        if (cleanGlobal !== "n" && cleanGlobal !== "no") {
+          const cleaned = globalContent.replace(/\[mcp_servers\.patchcord\][^\[]*/s, "").replace(/\n{3,}/g, "\n\n").trim();
+          writeFileSync(globalCodexConfig, cleaned + "\n");
+          console.log(`  ${green}✓${r} Removed from global config`);
+        }
+      }
+    }
     const configPath = join(cwd, ".codex", "config.toml");
     if (existsSync(configPath)) {
       const content = readFileSync(configPath, "utf-8");
@@ -436,6 +456,12 @@ if (!cmd || cmd === "install" || cmd === "agent") {
       process.exit(1);
     }
 
+    if (!isSafeToken(token)) {
+      console.log(`  ${red}✗${r} Invalid token format`);
+      rl.close();
+      process.exit(1);
+    }
+
     console.log("Validating...");
     const validateResp = run(`curl -sf --max-time 5 -H "Authorization: Bearer ${token}" "${serverUrl}/api/inbox?limit=0"`);
     if (validateResp) {
@@ -459,19 +485,6 @@ if (!cmd || cmd === "install" || cmd === "agent") {
   if (customUrl === "y" || customUrl === "yes") {
     const url = (await ask("Server URL: ")).trim();
     if (url) serverUrl = url;
-
-    // Re-validate against custom server if identity wasn't found
-    if (!identity) {
-      console.log("Validating token...");
-      const resp2 = run(`curl -sf --max-time 5 -H "Authorization: Bearer ${token}" "${serverUrl}/api/inbox?limit=0"`);
-      if (resp2) {
-        try {
-          const data = JSON.parse(resp2);
-          identity = `${data.agent_id}@${data.namespace_id}`;
-          console.log(`  ${green}✓${r} ${bold}${identity}${r}`);
-        } catch {}
-      }
-    }
   }
 
   rl.close();
@@ -489,7 +502,7 @@ if (!cmd || cmd === "install" || cmd === "agent") {
           command: "npx",
           args: [
             "-y", "mcp-remote",
-            serverUrl,
+            `${serverUrl}/mcp`,
             "--header",
             `Authorization: Bearer ${token}`,
             "--header",
@@ -522,7 +535,7 @@ if (!cmd || cmd === "install" || cmd === "agent") {
           command: "npx",
           args: [
             "-y", "mcp-remote",
-            serverUrl,
+            `${serverUrl}/mcp`,
             "--header",
             `Authorization: Bearer ${token}`,
             "--header",
@@ -546,14 +559,8 @@ if (!cmd || cmd === "install" || cmd === "agent") {
       mkdirSync(join(HOME, ".codeium", "windsurf"), { recursive: true });
       writeFileSync(wsPath, JSON.stringify(wsConfig, null, 2) + "\n");
     }
-    // Install workflows as slash commands (.windsurf/workflows/) — per-project
-    const wsWorkflowDir = join(cwd, ".windsurf", "workflows");
-    mkdirSync(wsWorkflowDir, { recursive: true });
-    cpSync(join(pluginRoot, "skills", "inbox", "SKILL.md"), join(wsWorkflowDir, "patchcord.md"));
-    cpSync(join(pluginRoot, "skills", "wait", "SKILL.md"), join(wsWorkflowDir, "patchcord-wait.md"));
     console.log(`\n  ${green}✓${r} Windsurf configured: ${dim}${wsPath}${r}`);
-    console.log(`  ${green}✓${r} Workflows installed: ${dim}/patchcord${r}, ${dim}/patchcord-wait${r}`);
-    console.log(`  ${yellow}MCP config is global — all Windsurf projects share this agent.${r}`);
+    console.log(`  ${yellow}Global config — all Windsurf projects share this agent.${r}`);
   } else if (isGemini) {
     // Gemini CLI: global only (~/.gemini/settings.json)
     const geminiPath = join(HOME, ".gemini", "settings.json");
@@ -670,19 +677,18 @@ if (!cmd || cmd === "install" || cmd === "agent") {
     let existing = existsSync(configPath) ? readFileSync(configPath, "utf-8") : "";
     // Remove old patchcord config block if present
     existing = existing.replace(/\[mcp_servers\.patchcord\][^\[]*/s, "").replace(/\n{3,}/g, "\n\n").trim();
-    // Codex requires bearer_token via env var — http_headers not supported for auth
-    const envName = "PATCHCORD_TOKEN";
+    existing = existing.trimEnd() + `\n\n[mcp_servers.patchcord]\nurl = "${serverUrl}/mcp/bearer"\nhttp_headers = { "Authorization" = "Bearer ${token}", "X-Patchcord-Machine" = "${hostname}" }\n`;
+    writeFileSync(configPath, existing);
+    // Clean up any PATCHCORD_TOKEN we previously wrote to .env
     const envPath = join(cwd, ".env");
-    let envContent = existsSync(envPath) ? readFileSync(envPath, "utf-8") : "";
-    // Replace or add token in .env
-    if (envContent.includes(envName)) {
-      envContent = envContent.replace(new RegExp(`${envName}=.*`), `${envName}=${token}`);
-    } else {
-      envContent = envContent.trimEnd() + `\n${envName}=${token}\n`;
+    if (existsSync(envPath)) {
+      const envContent = readFileSync(envPath, "utf-8");
+      if (envContent.includes("PATCHCORD_TOKEN=")) {
+        const cleaned = envContent.replace(/^PATCHCORD_TOKEN=.*\n?/gm, "").replace(/\n{3,}/g, "\n\n").trim();
+        writeFileSync(envPath, cleaned ? cleaned + "\n" : "");
+        console.log(`  ${green}✓${r} Cleaned PATCHCORD_TOKEN from .env`);
+      }
     }
-    writeFileSync(envPath, envContent);
-    existing = existing.trimEnd() + `\n\n[mcp_servers.patchcord]\nurl = "${serverUrl}/mcp/bearer"\nbearer_token_env_var = "${envName}"\n`;
-    writeFileSync(configPath, existing);
     // Slash commands (.codex/prompts/)
     const codexPromptsDir = join(codexDir, "prompts");
     mkdirSync(codexPromptsDir, { recursive: true });
@@ -721,6 +727,22 @@ if (!cmd || cmd === "install" || cmd === "agent") {
     console.log(`\n  ${green}✓${r} Claude Code configured: ${dim}${mcpPath}${r}`);
   }
 
+  // Warn about gitignore for per-project configs with tokens
+  if (!isWindsurf && !isGemini && !isZed) {
+    const gitignorePath = join(cwd, ".gitignore");
+    const configFile = isCodex ? ".codex/config.toml" : isCursor ? ".cursor/mcp.json" : isVSCode ? ".vscode/mcp.json" : isOpenCode ? "opencode.json" : ".mcp.json";
+    let needsWarning = true;
+    if (existsSync(gitignorePath)) {
+      const gi = readFileSync(gitignorePath, "utf-8");
+      if (gi.includes(configFile) || gi.includes(".mcp.json") || gi.includes(".codex/") || gi.includes(".cursor/")) {
+        needsWarning = false;
+      }
+    }
+    if (needsWarning) {
+      console.log(`\n  ${yellow}⚠ Add ${configFile} to .gitignore — it contains your token${r}`);
+    }
+  }
+
   const toolName = isOpenCode ? "OpenCode" : isZed ? "Zed" : isVSCode ? "VS Code" : isGemini ? "Gemini CLI" : isWindsurf ? "Windsurf" : isCursor ? "Cursor" : isCodex ? "Codex" : "Claude Code";
   console.log(`\n${dim}Restart your ${toolName} session, then run:${r} ${bold}inbox()${r}`);
   process.exit(0);
@@ -761,7 +783,7 @@ if (cmd === "skill") {
   const baseUrl = mcpUrl.replace(/\/mcp(\/bearer)?$/, "");
   const token = auth.replace(/^Bearer\s+/, "");
 
-  if (!baseUrl || !token) {
+  if (!baseUrl || !token || !isSafeToken(token)) {
     console.error("Cannot read patchcord URL/token from .mcp.json");
     process.exit(1);
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "patchcord",
-  "version": "0.3.43",
+  "version": "0.3.46",
   "description": "Cross-machine agent messaging for Claude Code and Codex",
   "author": "ppravdin",
   "license": "MIT",