claude-codex-pair 0.4.0 → 0.4.1

package/DESIGN.md

@@ -86,7 +86,9 @@ at most once a day, cache the answer under the state directory, and stay quiet o
 to stderr after the command's own output, so `pair wait`'s first line is still the delivery or
 timeout line agents read. `pair upgrade` installs with npm, then runs the `init` of the package it
 just installed (found with `npm root --global`), not the first `pair` on PATH, because both this
-process and an older copy on PATH would write the old templates.
+process and an older copy on PATH would write the old templates. It asks `npm view` first, so it uses
+the same registry as the install. Already on the latest version, it only runs the in-process
+refresh. If the check fails, it goes ahead with the install, which reports the actual npm error.
 
 Channel operations report `EACCES` and `EPERM` with the affected path and the sandbox
 configuration printed by `init`. Either ordinary permissions or sandbox restrictions can

package/README.md

@@ -203,8 +203,10 @@ offline or sandboxed.
 ## Upgrading
 
 Run `pair upgrade` in your own terminal, not through an agent: agent sandboxes block the network and
-global installs. It runs `npm install -g claude-codex-pair@latest`, then the new `pair init` in the
-current worktree if pair is set up there. Other worktrees refresh their instructions on the next
+global installs. If npm has a newer version, it runs `npm install -g claude-codex-pair@latest`, then
+the new `pair init` in the current worktree if pair is set up there, and prints only the files that
+`init` wrote or kept. Already on the latest version, it installs nothing and only refreshes this
+worktree's stale instructions. Other worktrees refresh their instructions on the next
 `pair send`, `wait`, `history` or `clean`. pair owns these files, so edits inside them are
 overwritten. pair, including `pair init`, never rewrites instructions that a newer pair wrote, so
 worktrees shared by different versions don't flip back and forth. To downgrade, delete them first. Restart Claude Code and Codex afterwards. Running

package/dist/pair.js

@@ -892,24 +892,31 @@ function status(channel, root) {
  * Returns a note for after the command's output: agents read `pair wait`'s first line, with stderr merged.
  */
 function refreshInstructions(root) {
-    const skillPath = join(root, ".claude", "skills", "pair", "SKILL.md");
-    const agentsPath = join(root, "AGENTS.md");
-    const refreshed = [];
     try {
-        if (instructionState(skillPath, claudeSkill()) === "stale") {
-            writeFileSync(skillPath, claudeSkill());
-            refreshed.push(skillPath);
-        }
-        if (instructionState(agentsPath, codexSection(), true) === "stale") {
-            writeFileSync(agentsPath, readFileSync(agentsPath, "utf8").replace(PAIR_SECTION, () => codexSection()));
-            refreshed.push(`${agentsPath} (pair section)`);
-        }
+        return refreshedNote(refreshStale(root));
     }
     catch (error) {
         // A read-only sandbox: message delivery must still work, so report and go on.
         const reason = error instanceof Error ? error.message : String(error);
         return `pair: couldn't refresh instructions (${reason}); run \`pair init\` in your own terminal`;
     }
+}
+/** The stale instruction files it rewrote. A write failure throws. */
+function refreshStale(root) {
+    const skillPath = join(root, ".claude", "skills", "pair", "SKILL.md");
+    const agentsPath = join(root, "AGENTS.md");
+    const refreshed = [];
+    if (instructionState(skillPath, claudeSkill()) === "stale") {
+        writeFileSync(skillPath, claudeSkill());
+        refreshed.push(skillPath);
+    }
+    if (instructionState(agentsPath, codexSection(), true) === "stale") {
+        writeFileSync(agentsPath, readFileSync(agentsPath, "utf8").replace(PAIR_SECTION, () => codexSection()));
+        refreshed.push(`${agentsPath} (pair section)`);
+    }
+    return refreshed;
+}
+function refreshedNote(refreshed) {
     return refreshed.length > 0
         ? `pair: refreshed ${refreshed.join(", ")} to pair ${PACKAGE.version}; reload agent sessions to load them`
         : undefined;
@@ -964,6 +971,21 @@ async function updateNotice() {
 }
 /** Installs the latest pair, then lets the new version refresh this worktree's instructions. */
 function upgrade(root) {
+    // npm's own answer, from the registry it installs from; if it fails, the install reports why.
+    const view = spawnSync("npm", ["view", PACKAGE.name, "version"], { encoding: "utf8" });
+    const latest = view.status === 0 ? view.stdout.trim() : undefined;
+    if (latest && !isNewer(latest, PACKAGE.version)) {
+        let note;
+        try {
+            note = refreshedNote(refreshStale(root)); // refreshing is all that is left to do, so a failure fails upgrade
+        }
+        catch (error) {
+            const reason = error instanceof Error ? error.message : String(error);
+            throw new PairError(`already up to date, but couldn't refresh instructions (${reason})`);
+        }
+        console.log(`already up to date: ${PACKAGE.name} ${PACKAGE.version}${note ? `\n${note}` : ""}`);
+        return 0;
+    }
     const install = spawnSync("npm", ["install", "--global", `${PACKAGE.name}@latest`], { stdio: "inherit" });
     if (install.error) {
         throw new PairError(`couldn't run npm: ${install.error.message}`);
@@ -982,12 +1004,14 @@ function upgrade(root) {
             throw new PairError("installed, but `npm root --global` failed; run `pair init` yourself");
         }
         const installed = join(globalRoot.stdout.trim(), PACKAGE.name, "dist", "pair.js");
-        const init = spawnSync(process.execPath, [installed, "init"], { cwd: root, stdio: "inherit" });
+        const init = spawnSync(process.execPath, [installed, "init"], { cwd: root, encoding: "utf8", stdio: ["ignore", "pipe", "inherit"] });
         if (init.error || init.status !== 0) {
             throw new PairError(`installed, but the new \`pair init\` failed${init.error ? `: ${init.error.message}` : ""}; run it yourself`);
         }
+        // Only what it did to the files: the sandbox and `codex mcp add` setup it also prints is already done.
+        console.log(init.stdout.split("\n").filter((line) => /^(?:wrote|kept): /.test(line)).join("\n"));
     }
-    console.log("Restart Claude Code and Codex sessions to load the new version. Other worktrees refresh on their next pair send or wait.");
+    console.log(`upgraded: ${PACKAGE.name} ${PACKAGE.version} → ${latest ?? "latest"}. Restart Claude Code and Codex sessions to load it; other worktrees refresh on their next pair send or wait.`);
     return 0;
 }
 /** Finished threads, most recently finished first. Entries that aren't archived threads are ignored. */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-codex-pair",
-  "version": "0.4.0",
+  "version": "0.4.1",
   "description": "Claude Code and Codex review each other's work over a file-system chat channel",
   "keywords": [
     "claude-code",