@curdx/flow 2.0.0-beta.14 → 2.0.0-beta.15

package/.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "Claude Code Discipline Layer — spec-driven workflow + goal-backward verification + Karpathy 4 principles enforced via gates. Stops Claude from faking \"done\" on non-trivial features.",
-    "version": "2.0.0-beta.14"
+    "version": "2.0.0-beta.15"
   },
   "plugins": [
     {

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "curdx-flow",
-  "version": "2.0.0-beta.14",
+  "version": "2.0.0-beta.15",
   "description": "Claude Code Discipline Layer — spec-driven workflow + goal-backward verification + Karpathy 4 principles enforced via gates. Stops Claude from faking \"done\" on non-trivial features.",
   "author": {
     "name": "wdx",

package/cli/install.js

@@ -58,6 +58,15 @@ export async function install(args = []) {
   }
   log.ok(`claude CLI found (${ver})`);
 
+  // Snapshot curdx-flow's pre-install version BEFORE Step 2 touches the
+  // marketplace. Step 2 does `claude plugin marketplace remove` + `add` to
+  // rebind to the current source, and the remove side-effect also drops
+  // any plugins installed from that marketplace. If we only call
+  // listPlugins() in Step 3, curdx-flow is already gone from the list and
+  // we can't tell a fresh install apart from an upgrade — the Step 3
+  // output then incorrectly says "installed" for both cases.
+  const prevCurdxFlow = listPlugins().find((p) => p.name === "curdx-flow");
+
   // ---------- Step 2: Add marketplace ----------
   log.blank();
   const marketplaceSource = useOffline ? PKG_ROOT : "curdx/curdx-flow";
@@ -105,25 +114,51 @@ export async function install(args = []) {
     // marketplace not local (online install) or unreadable — fall through
   }
 
-  const installed = listPlugins();
-  const already = installed.find((p) => p.name === "curdx-flow");
-  if (already && shippedVersion && already.version !== shippedVersion) {
+  // Use the pre-Step-2 snapshot — by this point `claude plugin marketplace
+  // remove` has already evicted the plugin, so listPlugins() here would
+  // always return undefined for curdx-flow and we'd mis-report "installed"
+  // when we actually upgraded (the bug reported by @wdx's beta.14 log).
+  const already = prevCurdxFlow;
+
+  if (already && shippedVersion && already.version === shippedVersion) {
+    // Already on the exact version the tarball ships — no re-install needed.
+    // `claude plugin install` would succeed too but the fresh-install output
+    // is misleading here.
+    log.ok(
+      `curdx-flow already at v${already.version} ${color.dim("(no change)")}`
+    );
+  } else if (already && shippedVersion) {
+    // Existing install, different version — frame as an upgrade.
     log.info(
-      `curdx-flow installed at v${already.version}, marketplace ships v${shippedVersion} — updating...`
+      `curdx-flow v${already.version} → v${shippedVersion}, installing...`
     );
     const r = await run(
       "claude",
-      ["plugin", "update", "curdx-flow@curdx-flow-marketplace"],
+      ["plugin", "install", "curdx-flow@curdx-flow-marketplace"],
       { silent: true }
     );
     if (r.code !== 0) {
-      log.warn(`Update returned non-zero: ${r.stderr.trim() || r.stdout.trim()}`);
-      log.info(`If the version stays on v${already.version}, run: claude plugin uninstall curdx-flow@curdx-flow-marketplace && retry`);
-    } else {
-      log.ok(`curdx-flow updated to v${shippedVersion}`);
+      log.err(`Install failed: ${r.stderr.trim() || r.stdout.trim()}`);
+      process.exit(1);
     }
+    log.ok(`curdx-flow upgraded to v${shippedVersion}`);
   } else if (already) {
-    log.ok(`curdx-flow already installed (v${already.version}, ${already.status})`);
+    // shippedVersion unknown (e.g. online install) — best we can do is report
+    // the previous version and let `claude plugin install` idempotently
+    // re-register.
+    log.info(
+      `curdx-flow v${already.version} detected, re-registering...`
+    );
+    const r = await run(
+      "claude",
+      ["plugin", "install", "curdx-flow@curdx-flow-marketplace"],
+      { silent: true }
+    );
+    if (r.code !== 0) {
+      log.err(`Install failed: ${r.stderr.trim() || r.stdout.trim()}`);
+      process.exit(1);
+    }
+    log.ok(`curdx-flow re-registered`);
   } else {
     const r = await run(
       "claude",
@@ -134,7 +169,11 @@ export async function install(args = []) {
       log.err(`Install failed: ${r.stderr.trim() || r.stdout.trim()}`);
       process.exit(1);
     }
-    log.ok("curdx-flow installed");
+    if (shippedVersion) {
+      log.ok(`curdx-flow v${shippedVersion} installed`);
+    } else {
+      log.ok("curdx-flow installed");
+    }
   }
 
   // ---------- Step 3.5: Register user-level MCPs (context7, sequential-thinking) ----------

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@curdx/flow",
-  "version": "2.0.0-beta.14",
+  "version": "2.0.0-beta.15",
   "description": "CLI installer for CurDX-Flow — AI engineering workflow meta-framework for Claude Code",
   "type": "module",
   "bin": {