roger-roger 0.1.2 → 0.1.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "roger-roger",
-  "version": "0.1.2",
+  "version": "0.1.3",
   "description": "Bring yourself back to the conversation: your coding agents reach you by Slack, a sound or a spoken line when they finish, get stuck, or need a decision, and you answer from your phone.",
   "type": "module",
   "bin": {

package/skills/roger-roger/herdr-plugin.toml

@@ -6,7 +6,7 @@
 
 id = "roger-roger"
 name = "roger-roger"
-version = "0.1.2"
+version = "0.1.3"
 min_herdr_version = "0.9.0"
 description = "Slack, a sound or a spoken line from the agents in your panes, and questions you answer from your phone"
 platforms = ["linux", "macos", "windows"]

package/skills/roger-roger/scripts/help.mjs

@@ -180,7 +180,7 @@ export function helpText(command = "", { columns = process.stdout.columns || 100
   for (const line of wrap(TAGLINE, cols - 2)) out.push(`  ${c.dim(line)}`);
   out.push("");
   out.push(`  ${c.bold("Usage")}`);
-  out.push(...table([["roger-roger <command> [flags]", "every command prints one JSON object"], ["roger-roger help <command>", "that command's flags"]], {
+  out.push(...table([["roger-roger <command> [flags]", "every command prints one JSON object"], ["roger-roger help <command>", "that command's flags"], ["roger-roger --version", "which copy this is"]], {
     indent: 4, columns: cols, c, maxLeft: 30, left: (l) => c.cyan("roger-roger") + " " + c.dim(l.slice("roger-roger ".length)),
   }));
   const sections = [...new Set(COMMANDS.map((x) => x.section))];

package/skills/roger-roger/scripts/install.mjs

@@ -115,23 +115,37 @@ export function globalCopy(env = process.env) {
     const root = (r.stdout ?? "").trim();
     if (r.status !== 0 || !root) return null;
     const pkg = path.join(root, "roger-roger");
-    return { root, dir: path.join(pkg, "skills", "roger-roger"), present: fs.existsSync(path.join(pkg, "package.json")) };
+    let version = "";
+    try {
+      version = JSON.parse(fs.readFileSync(path.join(pkg, "package.json"), "utf8")).version ?? "";
+    } catch {}
+    return { root, dir: path.join(pkg, "skills", "roger-roger"), present: Boolean(version), version };
   } catch {
     return null;
   }
 }
 
+/** a < b for dotted versions; anything unparseable counts as older. */
+export function olderThan(a, b) {
+  const parse = (v) => String(v ?? "").split(".").map((n) => parseInt(n, 10) || 0);
+  const [x, y] = [parse(a), parse(b)];
+  for (let i = 0; i < 3; i++) if ((x[i] ?? 0) !== (y[i] ?? 0)) return (x[i] ?? 0) < (y[i] ?? 0);
+  return false;
+}
+
 /**
  * Which copy of the skill everything should point at, and whether to offer a global install:
  * the copy this runs from, unless that is a passing one (npx's cache, say) and there is no global
- * copy yet. A global copy that exists beside a checkout or a skills.sh copy is left alone: whoever
+ * copy yet, or the global copy is older than this one (`npx roger-roger@latest install` after an
+ * update). A global copy that is as new as a checkout or a skills.sh copy is left alone: whoever
  * runs this copy meant this copy.
  */
-export function pickTarget(running, global) {
+export function pickTarget(running, global, runningVersion = VERSION) {
   if (!global) return { target: running, offer: false, reason: "npm was not found" };
   if (norm(running) === norm(global.dir)) return { target: running, offer: false, reason: "global" };
-  if (global.present) return { target: running, offer: false, reason: "a global copy exists, but this one was run" };
-  return { target: running, offer: true, reason: "no global copy yet" };
+  if (!global.present) return { target: running, offer: true, reason: "no global copy yet" };
+  if (runningVersion && olderThan(global.version, runningVersion)) return { target: running, offer: true, reason: "update", from: global.version, to: runningVersion };
+  return { target: running, offer: false, reason: "a global copy exists, but this one was run" };
 }
 
 /** `npm install -g roger-roger@latest`. Throws with npm's last words when it fails. */
@@ -422,9 +436,11 @@ export async function runInstall(args, { script, hooks, input = process.stdin, o
     if (pick.reason === "global") {
       ui.done("The roger-roger command is on your PATH", global.dir);
     } else if (pick.offer) {
-      const yes = await ui.confirm({ message: "Put the roger-roger command on your PATH?", hint: "npm install -g roger-roger, so you and your agents can run it from anywhere" });
+      const yes = await ui.confirm(pick.reason === "update"
+        ? { message: `Update the roger-roger command from ${pick.from} to ${pick.to}?`, hint: "npm install -g roger-roger@latest; the old one would keep answering" }
+        : { message: "Put the roger-roger command on your PATH?", hint: "npm install -g roger-roger, so you and your agents can run it from anywhere" });
       if (yes) {
-        const done = await attempt(ui, "Installing globally", () => ui.run("Installing roger-roger with npm", () => installGlobally(env), { done: () => ({ text: "The roger-roger command is on your PATH", detail: global.dir }) }), { skipLabel: "Carry on with this copy" });
+        const done = await attempt(ui, pick.reason === "update" ? "Updating" : "Installing globally", () => ui.run(`${pick.reason === "update" ? "Updating" : "Installing"} roger-roger with npm`, () => installGlobally(env), { done: () => ({ text: `The roger-roger command is on your PATH${pick.to ? `, at ${pick.to}` : ""}`, detail: global.dir }) }), { skipLabel: "Carry on with this copy" });
         if (done !== null && fs.existsSync(path.join(global.dir, "scripts", "roger-roger.mjs"))) {
           target = global.dir;
           hookScript = path.join(global.dir, "scripts", "roger-roger.mjs");

package/skills/roger-roger/scripts/roger-roger.mjs

@@ -10,7 +10,7 @@
 import fs from "node:fs";
 import path from "node:path";
 import {
-  DEFAULTS, DUCK_MODES, KINDS, METHODS, ON_EXPIRE, SPEECH_MODES, WHEN_MODES,
+  DEFAULTS, DUCK_MODES, KINDS, METHODS, ON_EXPIRE, SPEECH_MODES, VERSION, WHEN_MODES,
   applySetup, rogerRogerHome, configPath, listArg, listSounds, loadConfig, migrateLegacyHome, parseArgs,
   readSlackCredentials, saveConfig, slackAppToken, slackToken,
 } from "./lib.mjs";
@@ -324,6 +324,10 @@ const AGENTS = {
 
 /** `help`, `help <command>`, `--help`, or a command that doesn't exist: the overview, or one command's page. */
 function help(command, args) {
+  if (["--version", "-v", "-V", "version"].includes(command)) {
+    process.stdout.write(`${VERSION || "unversioned copy"}\n`);
+    return;
+  }
   const wanted = command === "help" ? args._[0] : "";
   const unknown = command && !["help", "--help", "-h", undefined].includes(command);
   if (unknown) process.stderr.write(`roger-roger: unknown command "${command}"\n`);