baychat 0.17.0 → 0.17.1

package/dist/connect.js

@@ -32,8 +32,10 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
 };
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.renderConnectMenu = renderConnectMenu;
+exports.isClaudeConnectTarget = isClaudeConnectTarget;
 exports.parseConnectClient = parseConnectClient;
 exports.writeClientConfig = writeClientConfig;
+exports.cmdConnectClaude = cmdConnectClaude;
 exports.cmdConnect = cmdConnect;
 const node_fs_1 = __importDefault(require("node:fs"));
 const node_path_1 = __importDefault(require("node:path"));
@@ -61,14 +63,32 @@ function renderConnectMenu() {
         "Connect this laptop to BayChat, once, and configure your client:",
         "",
         ...rows,
+        "  npx baychat connect claude  → Claude Code (refreshes its skill)",
         "",
         "You approve a QR on your phone. After that, in any coding session:",
         "",
         "  /baychat <name>        join BayChat as a session called <name>",
         "",
-        "Claude Code needs no config — `npx baychat login` sets it up for you.",
+        "First-time Claude Code setup is `npx baychat login` — it registers the MCP",
+        "server AND writes the skill. After that, `connect claude` refreshes the skill",
+        "on its own: no QR, no re-pairing. Run it after every CLI upgrade.",
     ].join("\n");
 }
+/** Spellings of Claude Code accepted by `connect`. */
+const CLAUDE_CONNECT_ALIASES = new Set(["claude", "claude-code", "claudecode"]);
+/**
+ * Is this `connect` argument Claude Code?
+ *
+ * Claude Code is deliberately NOT in `MCP_CLIENTS`: that list is clients whose
+ * MCP config is a file we write, and Claude Code's is registered by
+ * `claude mcp add` instead. But refusing the word entirely made the runtime this
+ * product is mostly used from the only one you could not `connect` — and sent a
+ * user whose credential was perfectly valid to `login`, which opens a pairing QR,
+ * when all they needed was the skill file refreshed after a CLI upgrade.
+ */
+function isClaudeConnectTarget(value) {
+    return CLAUDE_CONNECT_ALIASES.has(value.trim().toLowerCase());
+}
 /**
  * Narrow a client argument.
  *
@@ -159,11 +179,38 @@ const realIo = {
  *
  * @returns 0 on success, 2 when the laptop login expired without approval.
  */
+/**
+ * `baychat connect claude` — refresh Claude Code's skill.
+ *
+ * Deliberately does NOT touch the credential or the MCP registration. Both are
+ * `login`'s job, and re-doing them would put a device-pairing QR in front of
+ * somebody whose login is fine. Writing the skill needs no auth at all: it is
+ * generated locally from this CLI's own version, which is exactly why it goes
+ * stale on upgrade and exactly why this command has to exist.
+ */
+function cmdConnectClaude() {
+    try {
+        for (const line of (0, runtime_install_1.describeInstall)((0, runtime_install_1.installRuntimeCommand)("claude")))
+            console.log(`  ${line}`);
+        console.log("");
+        console.log("  Restart Claude Code once so it picks the skill up.");
+        console.log("  Not logged in yet? `npx baychat login` — that registers the MCP server too.");
+        return 0;
+    }
+    catch (err) {
+        console.log(`Could not install the Claude Code skill: ${err instanceof Error ? err.message : String(err)}`);
+        return 1;
+    }
+}
 async function cmdConnect(clientArg, opts = {}) {
     if (clientArg === undefined) {
         console.log(renderConnectMenu());
         return 0;
     }
+    // Claude Code first: it is not an MCP-config client, so it must not reach
+    // `parseConnectClient`, which would reject it.
+    if (isClaudeConnectTarget(clientArg))
+        return cmdConnectClaude();
     const client = parseConnectClient(clientArg);
     const base = (opts.base || process.env.BAYCHAT_API_URL || config_1.DEFAULT_API_URL).replace(/\/$/, "");
     const steps = (0, connect_plan_1.planConnect)({

package/dist/relay/commands.js

@@ -746,7 +746,7 @@ async function cmdRelayAttach(opts) {
  * bare name, which is exactly the behaviour that shipped before this existed — a
  * machine where PATH is fine keeps working, and one where it is not now works too.
  */
-function resolveRuntimeBin(runtime) {
+function resolveRuntimeBin(runtime, env = process.env, execPath = process.execPath) {
     // Delegates to the prover rather than walking PATH itself. The hand-rolled
     // walk this replaces had two Windows faults, and both produced a recorded
     // path the daemon could never spawn:
@@ -763,7 +763,7 @@ function resolveRuntimeBin(runtime) {
     // `resolveRuntimeBinary` tries the platform's real extension order and PROVES
     // each candidate by running it, which is the same question this function was
     // always asking — just answered correctly.
-    const decided = decideRuntimeBin(runtime);
+    const decided = decideRuntimeBin(runtime, env, execPath);
     return decided.ok ? decided.bin : undefined;
 }
 /**
@@ -775,16 +775,16 @@ function resolveRuntimeBin(runtime) {
  * then re-probed could still register `runtimeBin: undefined` — the exact state
  * the guard exists to prevent, reached by way of the guard.
  */
-function decideRuntimeBin(runtime) {
-    const override = process.env[`BAYCHAT_${runtime.toUpperCase()}_BIN`];
-    const resolved = (0, runtime_binary_1.resolveRuntimeBinary)(runtime, (0, runtime_binary_1.currentBinaryEnv)(override));
+function decideRuntimeBin(runtime, env = process.env, execPath = process.execPath) {
+    const override = env[`BAYCHAT_${runtime.toUpperCase()}_BIN`];
+    const resolved = (0, runtime_binary_1.resolveRuntimeBinary)(runtime, (0, runtime_binary_1.currentBinaryEnv)(override, env, execPath));
     if (!resolved.ok) {
         // An explicit override is a user ASSERTION. If it cannot be proven, stop —
         // see the guard in `cmdRelayAttach`. Absence of one is not an assertion, so
         // a plain failed probe keeps the best-effort fallback.
         if (override)
             return { ok: false, reason: (0, runtime_binary_1.summarizeResolutionFailure)(resolved) };
-        return { ok: true, bin: managedRuntimeBin(runtime) };
+        return { ok: true, bin: managedRuntimeBin(runtime, env) };
     }
     try {
         // Resolve symlinks: ~/.local/bin/claude is typically a link into a versioned
@@ -797,10 +797,10 @@ function decideRuntimeBin(runtime) {
         return { ok: true, bin: resolved.path };
     }
 }
-function managedRuntimeBin(runtime) {
+function managedRuntimeBin(runtime, env = process.env) {
     if (runtime !== "codex")
         return undefined;
-    const root = process.env.CODEX_MANAGED_PACKAGE_ROOT?.trim();
+    const root = env.CODEX_MANAGED_PACKAGE_ROOT?.trim();
     if (!root)
         return undefined;
     const candidate = path.join(root, "bin", "codex.js");

package/dist/runtime-binary.js

@@ -273,6 +273,24 @@ function currentBinaryEnv(override, env = process.env, execPath = process.execPa
                 timeout: PROBE_TIMEOUT_MS,
                 encoding: "utf8",
                 shell: false,
+                // THE PROBE MUST BE ABLE TO START WHAT IT IS PROBING.
+                //
+                // An npm runtime is a `#!/usr/bin/env node` script, and a spawn inherits
+                // THIS process's environment — the daemon's, under systemd, with no node
+                // on it. So probing the real binary died with
+                // `/usr/bin/env: 'node': No such file or directory`, the candidate was
+                // rejected as broken rather than unstartable, and resolution fell through
+                // to `/snap/bin/codex`.
+                //
+                // Measured 2026-09-01 18:00 from the daemon's log, after the PATH search
+                // was already fixed: `codex resolved to /snap/bin/codex (codex-cli
+                // 0.114.0)`, then `queue did not deliver ... this codex build has no
+                // \`queue\` subcommand`. The snap is 0.114.0; `queue` landed in 0.149.0.
+                // So a probe that could not start a shebang script cost Codex the one
+                // rung a live session can answer on.
+                //
+                // Same environment the spawn will use, for the same reason.
+                env: (0, spawn_env_1.headlessSpawnEnv)(env, execPath),
             });
             return summarizeProbe({
                 status: run.status,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "baychat",
-  "version": "0.17.0",
+  "version": "0.17.1",
   "description": "BayChat connector CLI — pair an agent session (Claude Code, Codex) with BayChat and chat in groups",
   "bin": {
     "baychat": "dist/index.js"