openkrew 0.5.16 → 0.5.18

package/core-bootstrap.mjs

@@ -73,9 +73,10 @@ function getGatewayName() {
 }
 
 /**
- * Checks if the binary is downloaded and matches the npm package version.
- * When npm install -g openkrew@X.Y.Z runs, PACKAGE_VERSION updates to X.Y.Z.
- * If the installed binary version doesn't match, we re-download from S3.
+ * Checks if the binary is downloaded and satisfies the npm package version.
+ * Uses >= rather than strict equality so the self-updater (which bumps the
+ * binary ahead of the npm wrapper) isn't un-done on the next CLI invocation.
+ * The wrapper's version is the minimum required; newer binaries still satisfy.
  */
 function isInstalled() {
   const binaryPath = join(BIN_DIR, getBinaryName());
@@ -83,12 +84,28 @@ function isInstalled() {
 
   try {
     const installedVersion = readFileSync(VERSION_FILE, "utf8").trim();
-    return installedVersion === PACKAGE_VERSION;
+    return compareSemver(installedVersion, PACKAGE_VERSION) >= 0;
   } catch {
     return false;
   }
 }
 
+/**
+ * Numeric semver compare. Returns <0 if a<b, 0 if equal, >0 if a>b. Tolerates
+ * missing segments (treats them as 0) and ignores prerelease suffixes —
+ * good enough for our 0.X.Y release cadence.
+ */
+function compareSemver(a, b) {
+  const pa = String(a).split("-")[0].split(".").map((n) => parseInt(n, 10) || 0);
+  const pb = String(b).split("-")[0].split(".").map((n) => parseInt(n, 10) || 0);
+  const len = Math.max(pa.length, pb.length);
+  for (let i = 0; i < len; i++) {
+    const diff = (pa[i] ?? 0) - (pb[i] ?? 0);
+    if (diff !== 0) return diff;
+  }
+  return 0;
+}
+
 /**
  * Downloads the self-contained binary from S3.
  */

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "openkrew",
-  "version": "0.5.16",
+  "version": "0.5.18",
   "description": "Distributed multi-machine AI agent team platform",
   "type": "module",
   "license": "BUSL-1.1",

package/scripts/postinstall.mjs

@@ -53,12 +53,26 @@ function isInstalled() {
   const binaryPath = join(BIN_DIR, getBinaryName());
   if (!existsSync(binaryPath)) return false;
   try {
-    return readFileSync(VERSION_FILE, "utf8").trim() === PACKAGE_VERSION;
+    // Use >= rather than strict equality so the Hub's self-updater (which
+    // bumps the binary ahead of the npm wrapper) isn't reversed on the next
+    // `npm install -g openkrew@<older>` run triggered by this wrapper.
+    return compareSemver(readFileSync(VERSION_FILE, "utf8").trim(), PACKAGE_VERSION) >= 0;
   } catch {
     return false;
   }
 }
 
+function compareSemver(a, b) {
+  const pa = String(a).split("-")[0].split(".").map((n) => parseInt(n, 10) || 0);
+  const pb = String(b).split("-")[0].split(".").map((n) => parseInt(n, 10) || 0);
+  const len = Math.max(pa.length, pb.length);
+  for (let i = 0; i < len; i++) {
+    const diff = (pa[i] ?? 0) - (pb[i] ?? 0);
+    if (diff !== 0) return diff;
+  }
+  return 0;
+}
+
 async function downloadBinary() {
   const rid = getRuntimeId();
   if (!rid) return; // Unsupported platform — bootstrapper will handle error on first run