@curdx/flow 3.3.1 → 3.3.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 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

package/dist/index.mjs

@@ -952,48 +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();
-  const sp = p4.spinner();
-  sp.start(t("state.checking"));
+  let userCancelled = false;
   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) {
-      p4.cancel(t("app.cancelled"));
+    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
@@ -1017,80 +1024,88 @@ async function probeInstalled() {
   }
 }
 async function uninstallFlow(opts = {}) {
-  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;
+  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
@@ -1114,89 +1129,96 @@ async function probeInstalled2() {
   }
 }
 async function updateFlow(opts = {}) {
-  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;
+  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
@@ -1361,7 +1383,7 @@ var SUBCOMMANDS = /* @__PURE__ */ new Set(["install", "uninstall", "update", "st
 var root = defineCommand({
   meta: {
     name: "@curdx/flow",
-    version: "3.3.1",
+    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.1",
+  "version": "3.3.2",
   "description": "Interactive installer for Claude Code plugins and MCP servers",
   "type": "module",
   "bin": "./dist/index.mjs",