@super-hands/connect 0.1.4 → 0.1.6

package/client.mjs

@@ -32,7 +32,7 @@ function codexConfigBlock(args) {
   return [
     `[mcp_servers.${MCP_SERVER_KEY}]`,
     `url = "${args.endpoint}"`,
-    `bearer_token = "${args.token}"`
+    `http_headers = { "Authorization" = "${mcpAuthorizationHeader(args.token)}" }`
   ].join("\n");
 }
 
@@ -94,6 +94,12 @@ function stop(message) {
 `);
   process.exit(1);
 }
+function cursorServerEntry(args) {
+  return {
+    url: args.endpoint,
+    headers: { Authorization: mcpAuthorizationHeader(args.token) }
+  };
+}
 function mergedCursorConfig(existing, args) {
   let config = {};
   if (existing !== null && existing.trim() !== "") {
@@ -111,21 +117,28 @@ function mergedCursorConfig(existing, args) {
   const serversRaw = config.mcpServers;
   const servers = typeof serversRaw === "object" && serversRaw !== null && !Array.isArray(serversRaw) ? serversRaw : {};
   const replaced = MCP_SERVER_KEY in servers;
-  servers[MCP_SERVER_KEY] = {
-    url: args.endpoint,
-    headers: { Authorization: mcpAuthorizationHeader(args.token) }
-  };
+  servers[MCP_SERVER_KEY] = cursorServerEntry(args);
   config.mcpServers = servers;
   return { text: `${JSON.stringify(config, null, 2)}
 `, replaced };
 }
 function appendedCodexConfig(existing, args) {
+  const staleOwnBlock = new RegExp(
+    `\\[mcp_servers\\.${MCP_SERVER_KEY}\\]\\nurl = "[^"\\n]*"\\nbearer_token = "[^"\\n]*"`
+  );
+  if (staleOwnBlock.test(existing)) {
+    return {
+      text: existing.replace(staleOwnBlock, codexConfigBlock(args)),
+      alreadyPresent: false,
+      repaired: true
+    };
+  }
   if (existing.includes(`[mcp_servers.${MCP_SERVER_KEY}]`)) {
-    return { text: existing, alreadyPresent: true };
+    return { text: existing, alreadyPresent: true, repaired: false };
   }
   const sep = existing === "" || existing.endsWith("\n\n") ? "" : existing.endsWith("\n") ? "\n" : "\n\n";
   return { text: `${existing}${sep}${codexConfigBlock(args)}
-`, alreadyPresent: false };
+`, alreadyPresent: false, repaired: false };
 }
 function writeSkill(clientDir) {
   const skillPath = join(clientDir, "skills", CONNECT_SKILL_DIR, CONNECT_SKILL_FILENAME);
@@ -188,26 +201,40 @@ async function main() {
     } catch {
       existing = null;
     }
+    let merged = null;
     try {
-      const merged = mergedCursorConfig(existing, { endpoint, token });
-      mkdirSync(dirname(configPath), { recursive: true });
-      writeFileSync(configPath, merged.text);
-      say(
-        merged.replaced ? `Cursor \u2014 updated the ${MCP_SERVER_KEY} server in ${configPath}.` : `Cursor \u2014 added the ${MCP_SERVER_KEY} server to ${configPath}.`
-      );
-      if (openCursor()) {
-        say(
-          `  Cursor is opening \u2014 enable ${MCP_SERVER_KEY} under Settings \u2192 MCP. If it was already running, restart it first.`
-        );
-      } else {
-        say(`  Open Cursor (restart it if it was running), then enable ${MCP_SERVER_KEY} under Settings \u2192 MCP.`);
-      }
-      writeSkill(cursorDir);
-      connected += 1;
+      merged = mergedCursorConfig(existing, { endpoint, token });
     } catch {
       say(`Cursor \u2014 ${configPath} is not valid JSON, so it was left untouched.`);
       say("  Fix or remove that file, then run this command again.");
     }
+    if (merged) {
+      try {
+        mkdirSync(dirname(configPath), { recursive: true });
+        writeFileSync(configPath, merged.text);
+        say(
+          merged.replaced ? `Cursor \u2014 updated the ${MCP_SERVER_KEY} server in ${configPath}.` : `Cursor \u2014 added the ${MCP_SERVER_KEY} server to ${configPath}.`
+        );
+        if (openCursor()) {
+          say(
+            `  Cursor is opening \u2014 enable ${MCP_SERVER_KEY} under Settings \u2192 MCP. If it was already running, restart it first.`
+          );
+        } else {
+          say(`  Open Cursor (restart it if it was running), then enable ${MCP_SERVER_KEY} under Settings \u2192 MCP.`);
+        }
+        writeSkill(cursorDir);
+        connected += 1;
+      } catch {
+        say(`Cursor \u2014 ${configPath} could not be written, so it was left as it was.`);
+        say(
+          '  A sandboxed agent usually cannot write outside its workspace. Run this same command in a regular terminal, or add this under "mcpServers" in that file (or in .cursor/mcp.json inside the project, which connects this project alone):'
+        );
+        process.stdout.write(
+          `${JSON.stringify({ [MCP_SERVER_KEY]: cursorServerEntry({ endpoint, token }) }, null, 2)}
+`
+        );
+      }
+    }
   }
   if (onlyClaude || autodetect && hasCli("claude")) {
     attempted += 1;
@@ -255,16 +282,27 @@ async function main() {
     const result = appendedCodexConfig(existing, { endpoint, token });
     if (result.alreadyPresent) {
       say(`Codex \u2014 ${configPath} already names a ${MCP_SERVER_KEY} server, so it was left as it is.`);
-      say("  If that connection is stale, update bearer_token there by hand.");
+      say("  If that connection is stale, update the http_headers Authorization value there by hand.");
       writeSkill(codexDir);
       connected += 1;
     } else {
-      mkdirSync(dirname(configPath), { recursive: true });
-      writeFileSync(configPath, result.text);
-      say(`Codex \u2014 added the ${MCP_SERVER_KEY} server to ${configPath}.`);
-      say("  The app and the CLI both read this config \u2014 open either and it connects when a session starts.");
-      writeSkill(codexDir);
-      connected += 1;
+      try {
+        mkdirSync(dirname(configPath), { recursive: true });
+        writeFileSync(configPath, result.text);
+        say(
+          result.repaired ? `Codex \u2014 replaced the ${MCP_SERVER_KEY} entry in ${configPath}: its old bearer_token spelling makes current Codex reject the whole config.` : `Codex \u2014 added the ${MCP_SERVER_KEY} server to ${configPath}.`
+        );
+        say("  The app and the CLI both read this config \u2014 open either and it connects when a session starts.");
+        writeSkill(codexDir);
+        connected += 1;
+      } catch {
+        say(`Codex \u2014 ${configPath} could not be written, so it was left as it was.`);
+        say(
+          "  A sandboxed agent usually cannot write outside its workspace. Run this same command in a regular terminal, or append this block to that file yourself:"
+        );
+        process.stdout.write(`${codexConfigBlock({ endpoint, token })}
+`);
+      }
     }
   }
   if (connected === 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@super-hands/connect",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "Connect the coding agents on this machine to your team's Superhands MCP server. Writes each client's own config; reads no repository, uploads nothing.",
   "bin": {
     "superhands-connect": "client.mjs"