@curdx/flow 3.3.0 → 3.3.2

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All notable changes to `@curdx/flow` are documented here. Format follows [Keep a Changelog](https://keepachangelog.com/) and the project follows [Semantic Versioning](https://semver.org/).
 
+## 3.3.2 — 2026-04-27
+
+### Fixed
+
+- **CLAUDE.md sync no longer skipped on the "nothing selected" path** — when a user upgraded flow with all tools already installed and ran `install`, the multiselect would show nothing pre-checked; pressing enter without a selection caused the flow to early-return before reaching the sync step, so the managed block was never added to CLAUDE.md. Each of `install` / `update` / `uninstall` now wraps its body in `try / finally` and runs the sync at the end of any non-cancelled exit (including "nothing to do" paths). User-cancelled flows (Ctrl+C, multiselect cancel, uninstall confirm "no") still skip the sync to respect intent.
+
+## 3.3.1 — 2026-04-27
+
+### Fixed
+
+- **Silent stalls between phases** — added spinners to the previously-silent windows where flow shells out to `claude plugin list --json` and `claude mcp list` (the latter performs an MCP server health check and can take 5-15 seconds). Affected sites: `install` (state-derivation between marketplace refresh and the multiselect), `update` and `uninstall` (state-derivation at flow entry), and the post-flow CLAUDE.md sync (after install/update/uninstall busts the cache, sync re-queries state). Each now shows `Checking installed state… (claude plugin list / mcp list)` with a result line so the run no longer feels frozen.
+- **CLAUDE.md sync feedback** — replaced the post-hoc `p.log.info` line with a live spinner that converts to a final status line on completion, matching the marketplace-refresh and per-item install UX.
+
 ## 3.3.0 — 2026-04-27
 
 ### Added

package/dist/index.mjs

@@ -67,6 +67,9 @@ var messages = {
   "chrome.prereqChrome": "\u9700\u8981\u672C\u673A\u5DF2\u5B89\u88C5 Chrome\uFF08chrome-devtools-mcp \u4F1A\u8C03\u7528\u672C\u5730\u6D4F\u89C8\u5668\uFF09",
   "reinstall.uninstalling": "\u5148\u5378\u8F7D\u65E7\u7248\u672C\u2026",
   "reinstall.installing": "\u5B89\u88C5\u65B0\u7248\u672C\u2026",
+  "state.checking": "\u68C0\u67E5\u5DF2\u5B89\u88C5\u72B6\u6001\u2026\uFF08claude plugin list / mcp list\uFF09",
+  "state.checked": "\u5DF2\u68C0\u67E5 {count} \u9879",
+  "claudeMd.syncing": "\u540C\u6B65 ~/.claude/CLAUDE.md \u2026",
   "claudeMd.synced": "CLAUDE.md \u5DF2\u66F4\u65B0\uFF08{path}\uFF09",
   "claudeMd.unchanged": "CLAUDE.md \u5DF2\u662F\u6700\u65B0",
   "claudeMd.removed": "\u5DF2\u4ECE CLAUDE.md \u79FB\u9664 @curdx/flow \u533A\u5757",
@@ -135,6 +138,9 @@ var messages2 = {
   "chrome.prereqChrome": "Requires Chrome installed locally (chrome-devtools-mcp drives the local browser)",
   "reinstall.uninstalling": "Uninstalling old version\u2026",
   "reinstall.installing": "Installing new version\u2026",
+  "state.checking": "Checking installed state\u2026 (claude plugin list / mcp list)",
+  "state.checked": "Checked {count} item(s)",
+  "claudeMd.syncing": "Syncing ~/.claude/CLAUDE.md\u2026",
   "claudeMd.synced": "CLAUDE.md updated ({path})",
   "claudeMd.unchanged": "CLAUDE.md already up to date",
   "claudeMd.removed": "Removed @curdx/flow block from CLAUDE.md",
@@ -769,23 +775,29 @@ async function syncClaudeMd(opts) {
   }
 }
 async function syncFromState(opts) {
-  const r = await syncClaudeMd(opts);
+  if (opts?.skip) {
+    p3.log.info(t("claudeMd.skipped"));
+    return;
+  }
+  const sp = p3.spinner();
+  sp.start(t("claudeMd.syncing"));
+  const r = await syncClaudeMd();
   switch (r.status) {
     case "skipped":
-      p3.log.info(t("claudeMd.skipped"));
+      sp.stop(t("claudeMd.skipped"));
       return;
     case "unchanged":
-      p3.log.info(t("claudeMd.unchanged"));
+      sp.stop(t("claudeMd.unchanged"));
       return;
     case "created":
     case "updated":
-      p3.log.info(t("claudeMd.synced", { path: r.path }));
+      sp.stop(t("claudeMd.synced", { path: r.path }));
       return;
     case "removed":
-      p3.log.info(t("claudeMd.removed"));
+      sp.stop(t("claudeMd.removed"));
       return;
     case "failed":
-      p3.log.warn(t("claudeMd.failed", { error: r.error ?? "unknown" }));
+      sp.stop(t("claudeMd.failed", { error: r.error ?? "unknown" }));
       return;
   }
 }
@@ -940,41 +952,55 @@ async function maybeRefreshMarketplaces(opts) {
   );
 }
 async function installFlow(opts = {}) {
-  await maybeRefreshMarketplaces(opts);
-  const explicit = opts.all || opts.ids && opts.ids.length > 0;
-  const candidates = explicit ? selectFromIds(opts) : [...PKGS];
-  if (candidates.length === 0) {
-    p4.log.info(t("install.nothingSelected"));
-    return;
-  }
-  const stateMap = /* @__PURE__ */ new Map();
-  await Promise.all(
-    candidates.map(async (pkg) => {
-      stateMap.set(pkg.id, await deriveState(pkg));
-    })
-  );
-  let targets;
-  if (explicit) {
-    targets = candidates;
-  } else {
-    const picked = await selectInteractive(stateMap);
-    if (picked === null) {
-      p4.cancel(t("app.cancelled"));
+  let userCancelled = false;
+  try {
+    await maybeRefreshMarketplaces(opts);
+    const explicit = opts.all || opts.ids && opts.ids.length > 0;
+    const candidates = explicit ? selectFromIds(opts) : [...PKGS];
+    if (candidates.length === 0) {
+      p4.log.info(t("install.nothingSelected"));
       return;
     }
-    targets = picked;
-  }
-  if (targets.length === 0) {
-    p4.log.info(t("install.nothingSelected"));
-    return;
-  }
-  const results = [];
-  for (const pkg of targets) {
-    const state = stateMap.get(pkg.id) ?? { kind: "not_installed" };
-    results.push(await runOne(pkg, state, opts));
+    const stateMap = /* @__PURE__ */ new Map();
+    const sp = p4.spinner();
+    sp.start(t("state.checking"));
+    try {
+      await Promise.all([listPlugins(), listMcp()]);
+      await Promise.all(
+        candidates.map(async (pkg) => {
+          stateMap.set(pkg.id, await deriveState(pkg));
+        })
+      );
+    } finally {
+      sp.stop(t("state.checked", { count: candidates.length }));
+    }
+    let targets;
+    if (explicit) {
+      targets = candidates;
+    } else {
+      const picked = await selectInteractive(stateMap);
+      if (picked === null) {
+        userCancelled = true;
+        p4.cancel(t("app.cancelled"));
+        return;
+      }
+      targets = picked;
+    }
+    if (targets.length === 0) {
+      p4.log.info(t("install.nothingSelected"));
+      return;
+    }
+    const results = [];
+    for (const pkg of targets) {
+      const state = stateMap.get(pkg.id) ?? { kind: "not_installed" };
+      results.push(await runOne(pkg, state, opts));
+    }
+    summarize(results);
+  } finally {
+    if (!userCancelled) {
+      await syncFromState({ skip: opts.noClaudeMd });
+    }
   }
-  summarize(results);
-  await syncFromState({ skip: opts.noClaudeMd });
 }
 
 // src/flows/uninstall.ts
@@ -984,81 +1010,102 @@ async function getInstalled() {
   const states = await Promise.all(PKGS.map(async (pkg) => ({ pkg, installed: await pkg.isInstalled() })));
   return states.filter((s) => s.installed).map((s) => s.pkg);
 }
+async function probeInstalled() {
+  const sp = p5.spinner();
+  sp.start(t("state.checking"));
+  try {
+    await Promise.all([listPlugins(), listMcp()]);
+    const installed = await getInstalled();
+    sp.stop(t("state.checked", { count: installed.length }));
+    return installed;
+  } catch (err) {
+    sp.stop(t("state.checked", { count: 0 }));
+    throw err;
+  }
+}
 async function uninstallFlow(opts = {}) {
-  const installed = await getInstalled();
-  let targets;
-  if (opts.ids && opts.ids.length > 0) {
-    targets = [];
-    for (const id of opts.ids) {
-      const pkg = findPkg(id);
-      if (!pkg) {
-        p5.log.warn(`Unknown id: ${id}`);
-        continue;
+  let userCancelled = false;
+  try {
+    const installed = await probeInstalled();
+    let targets;
+    if (opts.ids && opts.ids.length > 0) {
+      targets = [];
+      for (const id of opts.ids) {
+        const pkg = findPkg(id);
+        if (!pkg) {
+          p5.log.warn(`Unknown id: ${id}`);
+          continue;
+        }
+        if (!installed.some((x2) => x2.id === pkg.id)) {
+          p5.log.warn(`${pkg.name}: ${t("pkg.notInstalled")}`);
+          continue;
+        }
+        targets.push(pkg);
       }
-      if (!installed.some((x2) => x2.id === pkg.id)) {
-        p5.log.warn(`${pkg.name}: ${t("pkg.notInstalled")}`);
-        continue;
+    } else {
+      if (installed.length === 0) {
+        p5.log.info(t("uninstall.noneInstalled"));
+        return;
       }
-      targets.push(pkg);
-    }
-  } else {
-    if (installed.length === 0) {
-      p5.log.info(t("uninstall.noneInstalled"));
-      return;
+      const picked = await p5.multiselect({
+        message: t("uninstall.selectPrompt"),
+        options: installed.map((pkg) => ({
+          value: pkg.id,
+          label: `${pkg.name} ${pc2.dim(`(${pkg.type})`)}`,
+          hint: pkg.description
+        })),
+        required: false
+      });
+      if (p5.isCancel(picked)) {
+        userCancelled = true;
+        p5.cancel(t("app.cancelled"));
+        return;
+      }
+      targets = picked.map((id) => findPkg(id)).filter((x2) => Boolean(x2));
     }
-    const picked = await p5.multiselect({
-      message: t("uninstall.selectPrompt"),
-      options: installed.map((pkg) => ({
-        value: pkg.id,
-        label: `${pkg.name} ${pc2.dim(`(${pkg.type})`)}`,
-        hint: pkg.description
-      })),
-      required: false
-    });
-    if (p5.isCancel(picked)) {
-      p5.cancel(t("app.cancelled"));
+    if (targets.length === 0) {
+      p5.log.info(t("install.nothingSelected"));
       return;
     }
-    targets = picked.map((id) => findPkg(id)).filter((x2) => Boolean(x2));
-  }
-  if (targets.length === 0) {
-    p5.log.info(t("install.nothingSelected"));
-    return;
-  }
-  if (!opts.yes) {
-    const ok2 = await p5.confirm({
-      message: t("uninstall.confirm", { count: targets.length }),
-      initialValue: false
-    });
-    if (p5.isCancel(ok2) || ok2 === false) {
-      p5.cancel(t("app.cancelled"));
-      return;
+    if (!opts.yes) {
+      const ok2 = await p5.confirm({
+        message: t("uninstall.confirm", { count: targets.length }),
+        initialValue: false
+      });
+      if (p5.isCancel(ok2) || ok2 === false) {
+        userCancelled = true;
+        p5.cancel(t("app.cancelled"));
+        return;
+      }
     }
-  }
-  const results = [];
-  for (const pkg of targets) {
-    const log5 = p5.taskLog({ title: t("uninstall.starting", { name: pkg.name }) });
-    try {
-      await pkg.uninstall({ log: log5, config: {}, t });
-      log5.success(t("uninstall.success", { name: pkg.name }));
-      results.push({ id: pkg.id, status: "ok" });
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      log5.error(`${t("uninstall.failed", { name: pkg.name })}
+    const results = [];
+    for (const pkg of targets) {
+      const log5 = p5.taskLog({ title: t("uninstall.starting", { name: pkg.name }) });
+      try {
+        await pkg.uninstall({ log: log5, config: {}, t });
+        log5.success(t("uninstall.success", { name: pkg.name }));
+        results.push({ id: pkg.id, status: "ok" });
+      } catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        log5.error(`${t("uninstall.failed", { name: pkg.name })}
 ${msg}`);
-      results.push({ id: pkg.id, status: "fail", message: msg });
+        results.push({ id: pkg.id, status: "fail", message: msg });
+      }
+    }
+    const ok = results.filter((r) => r.status === "ok").length;
+    const fail = results.filter((r) => r.status === "fail").length;
+    p5.note(
+      [
+        pc2.green(t("install.summaryOk", { count: ok })),
+        pc2.red(t("install.summaryFail", { count: fail }))
+      ].join("\n"),
+      t("install.summaryTitle")
+    );
+  } finally {
+    if (!userCancelled) {
+      await syncFromState({ skip: opts.noClaudeMd });
     }
   }
-  const ok = results.filter((r) => r.status === "ok").length;
-  const fail = results.filter((r) => r.status === "fail").length;
-  p5.note(
-    [
-      pc2.green(t("install.summaryOk", { count: ok })),
-      pc2.red(t("install.summaryFail", { count: fail }))
-    ].join("\n"),
-    t("install.summaryTitle")
-  );
-  await syncFromState({ skip: opts.noClaudeMd });
 }
 
 // src/flows/update.ts
@@ -1068,90 +1115,110 @@ async function getInstalled2() {
   const states = await Promise.all(PKGS.map(async (pkg) => ({ pkg, installed: await pkg.isInstalled() })));
   return states.filter((s) => s.installed).map((s) => s.pkg);
 }
-async function updateFlow(opts = {}) {
-  const installed = await getInstalled2();
-  if (installed.length === 0) {
-    p6.log.info(t("update.noneInstalled"));
-    return;
+async function probeInstalled2() {
+  const sp = p6.spinner();
+  sp.start(t("state.checking"));
+  try {
+    await Promise.all([listPlugins(), listMcp()]);
+    const installed = await getInstalled2();
+    sp.stop(t("state.checked", { count: installed.length }));
+    return installed;
+  } catch (err) {
+    sp.stop(t("state.checked", { count: 0 }));
+    throw err;
   }
-  let targets;
-  if (opts.all) {
-    targets = installed;
-  } else if (opts.ids && opts.ids.length > 0) {
-    targets = [];
-    for (const id of opts.ids) {
-      const pkg = findPkg(id);
-      if (!pkg) {
-        p6.log.warn(`Unknown id: ${id}`);
-        continue;
+}
+async function updateFlow(opts = {}) {
+  let userCancelled = false;
+  try {
+    const installed = await probeInstalled2();
+    if (installed.length === 0) {
+      p6.log.info(t("update.noneInstalled"));
+      return;
+    }
+    let targets;
+    if (opts.all) {
+      targets = installed;
+    } else if (opts.ids && opts.ids.length > 0) {
+      targets = [];
+      for (const id of opts.ids) {
+        const pkg = findPkg(id);
+        if (!pkg) {
+          p6.log.warn(`Unknown id: ${id}`);
+          continue;
+        }
+        if (!installed.some((x2) => x2.id === pkg.id)) {
+          p6.log.warn(`${pkg.name}: ${t("pkg.notInstalled")}`);
+          continue;
+        }
+        targets.push(pkg);
       }
-      if (!installed.some((x2) => x2.id === pkg.id)) {
-        p6.log.warn(`${pkg.name}: ${t("pkg.notInstalled")}`);
-        continue;
+    } else {
+      const picked = await p6.multiselect({
+        message: t("update.selectPrompt"),
+        options: installed.map((pkg) => ({
+          value: pkg.id,
+          label: `${pkg.name} ${pc3.dim(`(${pkg.type})`)}`,
+          hint: pkg.description
+        })),
+        required: false
+      });
+      if (p6.isCancel(picked)) {
+        userCancelled = true;
+        p6.cancel(t("app.cancelled"));
+        return;
       }
-      targets.push(pkg);
+      targets = picked.map((id) => findPkg(id)).filter((x2) => Boolean(x2));
     }
-  } else {
-    const picked = await p6.multiselect({
-      message: t("update.selectPrompt"),
-      options: installed.map((pkg) => ({
-        value: pkg.id,
-        label: `${pkg.name} ${pc3.dim(`(${pkg.type})`)}`,
-        hint: pkg.description
-      })),
-      required: false
-    });
-    if (p6.isCancel(picked)) {
-      p6.cancel(t("app.cancelled"));
+    if (targets.length === 0) {
+      p6.log.info(t("install.nothingSelected"));
       return;
     }
-    targets = picked.map((id) => findPkg(id)).filter((x2) => Boolean(x2));
-  }
-  if (targets.length === 0) {
-    p6.log.info(t("install.nothingSelected"));
-    return;
-  }
-  const results = [];
-  for (const pkg of targets) {
-    if (pkg.id === "sequential-thinking") {
-      p6.log.info(t("update.mcpAutoNote", { name: pkg.name }));
-      results.push({ id: pkg.id, status: "noop" });
-      continue;
-    }
-    if (pkg.id === "context7") {
-      p6.log.info(t("update.context7Note"));
-      results.push({ id: pkg.id, status: "noop" });
-      continue;
-    }
-    const log5 = p6.taskLog({ title: t("update.starting", { name: pkg.name }) });
-    try {
-      if (pkg.update) {
-        await pkg.update({ log: log5, config: {}, t });
-      } else {
-        await pkg.uninstall({ log: log5, config: {}, t });
-        await pkg.install({ log: log5, config: {}, t });
+    const results = [];
+    for (const pkg of targets) {
+      if (pkg.id === "sequential-thinking") {
+        p6.log.info(t("update.mcpAutoNote", { name: pkg.name }));
+        results.push({ id: pkg.id, status: "noop" });
+        continue;
       }
-      log5.success(t("update.success", { name: pkg.name }));
-      results.push({ id: pkg.id, status: "ok" });
-    } catch (err) {
-      const msg = err instanceof Error ? err.message : String(err);
-      log5.error(`${t("update.failed", { name: pkg.name })}
+      if (pkg.id === "context7") {
+        p6.log.info(t("update.context7Note"));
+        results.push({ id: pkg.id, status: "noop" });
+        continue;
+      }
+      const log5 = p6.taskLog({ title: t("update.starting", { name: pkg.name }) });
+      try {
+        if (pkg.update) {
+          await pkg.update({ log: log5, config: {}, t });
+        } else {
+          await pkg.uninstall({ log: log5, config: {}, t });
+          await pkg.install({ log: log5, config: {}, t });
+        }
+        log5.success(t("update.success", { name: pkg.name }));
+        results.push({ id: pkg.id, status: "ok" });
+      } catch (err) {
+        const msg = err instanceof Error ? err.message : String(err);
+        log5.error(`${t("update.failed", { name: pkg.name })}
 ${msg}`);
-      results.push({ id: pkg.id, status: "fail", message: msg });
+        results.push({ id: pkg.id, status: "fail", message: msg });
+      }
+    }
+    const ok = results.filter((r) => r.status === "ok").length;
+    const fail = results.filter((r) => r.status === "fail").length;
+    const noop = results.filter((r) => r.status === "noop").length;
+    p6.note(
+      [
+        pc3.green(t("install.summaryOk", { count: ok })),
+        pc3.red(t("install.summaryFail", { count: fail })),
+        pc3.dim(`noop: ${noop}`)
+      ].join("\n"),
+      t("install.summaryTitle")
+    );
+  } finally {
+    if (!userCancelled) {
+      await syncFromState({ skip: opts.noClaudeMd });
     }
   }
-  const ok = results.filter((r) => r.status === "ok").length;
-  const fail = results.filter((r) => r.status === "fail").length;
-  const noop = results.filter((r) => r.status === "noop").length;
-  p6.note(
-    [
-      pc3.green(t("install.summaryOk", { count: ok })),
-      pc3.red(t("install.summaryFail", { count: fail })),
-      pc3.dim(`noop: ${noop}`)
-    ].join("\n"),
-    t("install.summaryTitle")
-  );
-  await syncFromState({ skip: opts.noClaudeMd });
 }
 
 // src/flows/status.ts
@@ -1316,7 +1383,7 @@ var SUBCOMMANDS = /* @__PURE__ */ new Set(["install", "uninstall", "update", "st
 var root = defineCommand({
   meta: {
     name: "@curdx/flow",
-    version: "3.3.0",
+    version: "3.3.2",
     description: "Interactive installer for Claude Code plugins and MCP servers"
   },
   args: sharedArgs,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@curdx/flow",
-  "version": "3.3.0",
+  "version": "3.3.2",
   "description": "Interactive installer for Claude Code plugins and MCP servers",
   "type": "module",
   "bin": "./dist/index.mjs",