@super-hands/connect 0.1.5 → 0.1.7

package/client.mjs

@@ -37,7 +37,7 @@ function codexConfigBlock(args) {
 }
 
 // lib/connect-skill.ts
-var CONNECT_SKILL_VERSION = 2;
+var CONNECT_SKILL_VERSION = 3;
 var CONNECT_SKILL_DIR = MCP_SERVER_KEY;
 var CONNECT_SKILL_FILENAME = "SKILL.md";
 var VERSION_MARKER = `[//]: # (superhands-skill-version: ${CONNECT_SKILL_VERSION})`;
@@ -60,8 +60,10 @@ Before you build, change, or restyle any UI:
 
 1. Call the \`superhands_get_guidance\` MCP tool with \`intent\` set to what
    you have been asked to build, in plain language \u2014 for example "a sign-in
-   page with email and password and an error state". The decisions that govern
-   that work come back in full, ahead of the rest.
+   page with email and password and an error state", and
+   \`skill_version: ${CONNECT_SKILL_VERSION}\` \u2014 the version of this file, so
+   the server can tell you when this file is out of date. The decisions that
+   govern that work come back in full, ahead of the rest.
 2. Build to what comes back. Where a decision names a case it deliberately
    does not govern, your own judgment resumes there.
 3. When you have finished \u2014 not part-way through \u2014 send the files you wrote or
@@ -94,6 +96,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,10 +119,7 @@ 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 };
@@ -198,26 +203,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;
@@ -269,14 +288,23 @@ async function main() {
       writeSkill(codexDir);
       connected += 1;
     } else {
-      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;
+      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.5",
+  "version": "0.1.7",
   "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"