probe-research 0.10.0 → 0.10.2

package/bin/probe-research.js

@@ -11,7 +11,7 @@
  * there is nothing here that can drift from the CLI's behaviour.
  *
  * Resolution order, best first:
- *   1. `probe` already on PATH        -> exec it (the common re-run case)
+ *   1. `probe` on PATH AND new enough -> exec it (the common re-run case)
  *   2. `uv`                           -> uvx, no install, no state left behind
  *   3. `pipx`                         -> isolated install
  *   4. bootstrap uv, then uvx         -> the from-zero path
@@ -26,8 +26,48 @@ const { spawnSync } = require("node:child_process");
 const os = require("node:os");
 
 const DIST = "probe-research";
+
+/**
+ * The minimum CLI version this launcher needs — NOT this package's own version.
+ *
+ * npm and PyPI release INDEPENDENTLY. This package can take a launcher-only
+ * patch (as 0.10.1 did) with no corresponding CLI release, so pinning
+ * `probe-research==<our version>` resolves to a PyPI version that does not
+ * exist. That is exactly how 0.10.1 shipped broken.
+ *
+ * Bump this only when the launcher starts depending on a NEW CLI feature.
+ * 0.10.0 is the release that introduced `probe wizard`.
+ */
+const MIN_CLI = "0.10.0";
 const UV_INSTALL = "curl -LsSf https://astral.sh/uv/install.sh | sh";
 
+/** Compare dotted numeric versions. Returns true when `a` >= `b`. */
+function atLeast(a, b) {
+  const pa = String(a).split(".").map((n) => parseInt(n, 10) || 0);
+  const pb = String(b).split(".").map((n) => parseInt(n, 10) || 0);
+  for (let i = 0; i < Math.max(pa.length, pb.length); i++) {
+    const x = pa[i] || 0;
+    const y = pb[i] || 0;
+    if (x !== y) return x > y;
+  }
+  return true;
+}
+
+/**
+ * The version of an existing `probe`, or null if it cannot be determined.
+ *
+ * Existence is NOT enough. Every current user already has some `probe`, and
+ * they are exactly the people most likely to run this command — handing off to
+ * whatever ancient build happens to be on PATH gives them a CLI with none of
+ * the wizard in it, which is precisely the bug this check exists to stop.
+ */
+function installedVersion() {
+  const r = spawnSync("probe", ["--version"], { encoding: "utf8", shell: false });
+  if (r.status !== 0 || !r.stdout) return null;
+  const m = r.stdout.match(/(\d+\.\d+\.\d+(?:\.\d+)?)/);
+  return m ? m[1] : null;
+}
+
 function has(cmd) {
   const probe = process.platform === "win32" ? "where" : "command";
   const args = process.platform === "win32" ? [cmd] : ["-v", cmd];
@@ -51,10 +91,24 @@ function main() {
   // `npx probe-research` with no arguments runs the wizard — that is the entire
   // reason this package exists, so it should not require remembering a verb.
   const forwarded = args.length ? args : ["wizard"];
+  const wanted = MIN_CLI;
 
-  if (has("probe")) return run("probe", forwarded);
-  if (has("uv")) return run("uv", ["tool", "run", "--from", DIST, "probe", ...forwarded]);
-  if (has("pipx")) return run("pipx", ["run", "--spec", DIST, "probe", ...forwarded]);
+  // Only hand off to an existing install if it is at least as new as this
+  // launcher. Otherwise fall through and let uv/pipx fetch a matching CLI.
+  if (has("probe")) {
+    const found = installedVersion();
+    if (found && atLeast(found, wanted)) return run("probe", forwarded);
+    console.error(
+      `probe-research: installed probe ${found || "(unknown version)"} is older than ` +
+        `${wanted}; fetching ${wanted}…`,
+    );
+  }
+  // A FLOOR, not an exact pin. Unpinned, `uv tool run --from probe-research`
+  // reuses an already-installed 0.8.2 tool; pinned exactly, it demands a PyPI
+  // version that may not exist. `>=` fixes both.
+  const spec = `${DIST}>=${wanted}`;
+  if (has("uv")) return run("uv", ["tool", "run", "--from", spec, "probe", ...forwarded]);
+  if (has("pipx")) return run("pipx", ["run", "--spec", spec, "probe", ...forwarded]);
 
   if (process.platform === "win32") {
     console.error(
@@ -75,7 +129,7 @@ function main() {
   // The installer drops uv in ~/.local/bin, which is not on THIS process's PATH
   // because it was resolved before the install ran.
   const uv = `${os.homedir()}/.local/bin/uv`;
-  return run(uv, ["tool", "run", "--from", DIST, "probe", ...forwarded]);
+  return run(uv, ["tool", "run", "--from", `${DIST}>=${wanted}`, "probe", ...forwarded]);
 }
 
 main();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "probe-research",
-  "version": "0.10.0",
+  "version": "0.10.2",
   "description": "Setup wizard for Probe Research in Claude Code. Runs from zero with npx.",
   "bin": {
     "probe-research": "bin/probe-research.js"
@@ -22,5 +22,8 @@
     "mlops",
     "experiment-tracking",
     "probe-research"
-  ]
+  ],
+  "scripts": {
+    "test": "node --test test/resolve.test.js"
+  }
 }