agentic-relay 3.8.3 → 3.8.5

package/bin/install.mjs

@@ -29,7 +29,7 @@
  *   deeplink         Print URL the agent registers (tome://, raycast://) — user clicks Install
  */
 
-import { readFileSync, writeFileSync, mkdirSync, existsSync } from "fs";
+import { readFileSync, writeFileSync, mkdirSync, existsSync, realpathSync } from "fs";
 import { homedir } from "os";
 import { join, dirname } from "path";
 import { fileURLToPath } from "url";
@@ -610,24 +610,125 @@ function getExistingKey() {
 
 /**
  * Link a bare global `agentrelay` command so users skip `npx` after install.
- * Installs the exact package copy being run (works whether invoked via npx or a
- * checkout). No-ops if `agentrelay` already resolves; warns + falls back to a
- * manual instruction if the global prefix isn't writable (EACCES etc.).
+ * No-ops only when a GENUINE install of `agentrelay` already matches this
+ * package's version — a stale global install gets upgraded, not skipped.
+ *
+ * npx trap: `npx agentic-relay` prepends its ephemeral cache's .bin to this
+ * process's PATH, so `agentrelay` ALWAYS resolves here (to the very copy being
+ * run) and disappears when npx exits. A naive "already on your PATH" check
+ * therefore never links anything and the user gets `command not found` in
+ * their next shell. We resolve the command's real location and ignore it when
+ * it lives in an npx/npm-exec cache or inside this package itself.
+ *
+ * Prefers a registry install (`agentic-relay@<version>`) over the local path
+ * (the npx cache dir is ephemeral; npm would link/copy from it). Falls back to
+ * the path for unpublished checkouts. Surfaces npm's actual error (EACCES
+ * etc.) instead of swallowing it, and verifies the command resolves afterwards
+ * so a PATH mismatch is reported, not discovered later as `command not found`.
  */
-function linkGlobalCli() {
+function resolveCommandPath(cmd) {
   try {
-    execSync("agentrelay version", { stdio: "ignore" });
-    success("`agentrelay` already on your PATH");
-    return;
+    const probe = process.platform === "win32" ? `where ${cmd}` : `command -v ${cmd}`;
+    const out = execSync(probe, { encoding: "utf-8", stdio: ["ignore", "pipe", "ignore"] });
+    return out.split("\n")[0].trim() || null;
+  } catch {
+    return null;
+  }
+}
+
+/** True when a resolved bin path is an ephemeral npx/npm-exec shim (or this package's own copy), not a real install. */
+function isEphemeralBin(binPath, pkgRoot) {
+  if (!binPath) return false;
+  let real = binPath;
+  try {
+    real = realpathSync(binPath);
+  } catch {
+    // dangling symlink — definitely not a usable install
+    return true;
+  }
+  const sep = process.platform === "win32" ? "\\" : "/";
+  return (
+    real.includes(`${sep}_npx${sep}`) ||
+    binPath.includes(`${sep}_npx${sep}`) ||
+    real.startsWith(realpathSafe(pkgRoot))
+  );
+}
+
+function realpathSafe(p) {
+  try {
+    return realpathSync(p);
   } catch {
-    // not linked yet — proceed to install it
+    return p;
   }
+}
+
+function linkGlobalCli() {
   const pkgRoot = join(__dirname, "..");
+  let version = null;
   try {
-    execSync(`npm install -g "${pkgRoot}"`, { stdio: "ignore" });
-    success("linked global `agentrelay` CLI");
+    version = JSON.parse(readFileSync(join(pkgRoot, "package.json"), "utf-8")).version;
   } catch {
-    warn("could not auto-link the global CLI (permissions?). Run once:  npm i -g agentic-relay");
+    // unknown version — fall through to the path install
+  }
+
+  const binPath = resolveCommandPath("agentrelay");
+  if (binPath && !isEphemeralBin(binPath, pkgRoot)) {
+    try {
+      const out = execSync("agentrelay version", { encoding: "utf-8", stdio: ["ignore", "pipe", "ignore"] });
+      const installed = (out.trim().split(/\s+/).pop() || "").trim();
+      if (version && installed === version) {
+        success(`\`agentrelay\` ${version} already on your PATH`);
+        return;
+      }
+      log(`global \`agentrelay\` is ${installed || "an unknown version"} — updating to ${version}…`);
+    } catch {
+      // resolves but doesn't run — reinstall it
+    }
+  }
+
+  const attempts = version ? [`agentic-relay@${version}`, `"${pkgRoot}"`] : [`"${pkgRoot}"`];
+  let lastErr = null;
+  let installedFrom = null;
+  for (const spec of attempts) {
+    try {
+      execSync(`npm install -g ${spec}`, { stdio: ["ignore", "ignore", "pipe"], encoding: "utf-8" });
+      installedFrom = spec;
+      break;
+    } catch (err) {
+      lastErr = err;
+    }
+  }
+  if (!installedFrom) {
+    const stderr = String((lastErr && lastErr.stderr) || "");
+    const detail =
+      stderr.split("\n").find((l) => /EACCES|EEXIST|E404|ERR!|error/i.test(l)) || "";
+    warn(
+      `could not auto-link the global CLI${detail ? ` — ${detail.trim()}` : ""}.` +
+        ` Run once:  npm i -g agentic-relay` +
+        (/EACCES/.test(stderr) ? "  (npm's global prefix isn't writable — fix permissions or use a user-level prefix)" : ""),
+    );
+    return;
+  }
+
+  // Verify the global bin actually landed. Don't probe the bare command here:
+  // in this process it resolves to the ephemeral npx shim (which shadows the
+  // global install), so check the file in npm's global prefix directly.
+  let binDir = null;
+  let globalBin = null;
+  try {
+    const prefix = execSync("npm prefix -g", { encoding: "utf-8", stdio: ["ignore", "pipe", "ignore"] }).trim();
+    binDir = process.platform === "win32" ? prefix : join(prefix, "bin");
+    const cand = join(binDir, process.platform === "win32" ? "agentrelay.cmd" : "agentrelay");
+    if (existsSync(cand)) globalBin = cand;
+  } catch {
+    // can't resolve the prefix — fall through to the warn
+  }
+  if (globalBin) {
+    success(`linked global \`agentrelay\` CLI (${installedFrom})`);
+    const onPath = (process.env.PATH || "").split(process.platform === "win32" ? ";" : ":").includes(binDir);
+    if (!onPath) warn(`note: ${binDir} doesn't appear to be on your PATH — add it to use \`agentrelay\` directly`);
+  } else {
+    warn(`installed, but no global \`agentrelay\` found in ${binDir || "npm's global bin directory"} — run once:  npm i -g agentic-relay`);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agentic-relay",
-  "version": "3.8.3",
+  "version": "3.8.5",
   "description": "Install Agent Relay AI search across 17+ agents — Claude Code, Codex, Cursor, Gemini CLI, Goose, Windsurf, Cline, BoltAI, Claude Desktop, VS Code, Amazon Q, Roo Code, Witsy, LibreChat, OpenClaw, Tome, Raycast — plus the `agentrelay` CLI for token-cheap parallel agent consulting",
   "bin": {
     "agentrelay": "bin/cli.mjs"