@cannbot-ai/install-helper 1.2.1-beta.0 → 1.2.1-beta.2

package/dist/index.js

@@ -2913,7 +2913,7 @@ function createRepositoryManager(customPath) {
 
 // src/core/cannbot-delegate.ts
 import { existsSync as existsSync17, readFileSync as readFileSync12, writeFileSync as writeFileSync5, copyFileSync as copyFileSync3, unlinkSync as unlinkSync6, readdirSync as readdirSync8 } from "fs";
-import { join as join16, resolve as resolve2 } from "path";
+import { join as join16, resolve as resolve3 } from "path";
 import { execa as execa6 } from "execa";
 
 // src/core/cannbot-installer.ts
@@ -3154,7 +3154,7 @@ async function ensureCannbotDiscovery() {
 
 // src/core/cannbot-registry.ts
 import { existsSync as existsSync13, readFileSync as readFileSync9, readdirSync as readdirSync6, rmSync as rmSync3, rmdirSync, unlinkSync as unlinkSync4, lstatSync as lstatSync2, writeFileSync as writeFileSync3 } from "fs";
-import { join as join12, resolve, sep as sep2, dirname as dirname6 } from "path";
+import { join as join12, resolve as resolve2, sep as sep2, dirname as dirname6 } from "path";
 
 // src/core/record.ts
 init_paths();
@@ -3163,38 +3163,88 @@ init_fs_helpers();
 init_logger();
 init_i18n();
 import { existsSync as existsSync12, mkdirSync as mkdirSync5, readFileSync as readFileSync8, unlinkSync as unlinkSync3, readdirSync as readdirSync5 } from "fs";
-import { join as join11 } from "path";
+import { join as join11, resolve } from "path";
+var LEGACY_LOCATION_KEY = "__legacy__";
 function getInstallsDir() {
   return join11(getCannbotConfigDir(), "installs");
 }
 function getRecordPath(pluginId) {
   return join11(getInstallsDir(), `${pluginId}.json`);
 }
-function readRecord(pluginId) {
+function locationKey(loc) {
+  return `${loc.tool}:${loc.level}:${resolve(loc.installPath)}`;
+}
+function readStore(pluginId) {
   const recordPath = getRecordPath(pluginId);
   if (!existsSync12(recordPath)) {
     return null;
   }
+  let parsed;
   try {
-    const content = readFileSync8(recordPath, "utf-8");
-    return JSON.parse(content);
+    parsed = JSON.parse(readFileSync8(recordPath, "utf-8"));
   } catch {
     logger.warn(t("record_corrupted").replace("{file}", recordPath));
     return null;
   }
+  const store = parsed;
+  if (store && typeof store === "object" && "locations" in store && store.locations) {
+    return { version: 2, locations: store.locations };
+  }
+  const legacy = parsed;
+  if (legacy && typeof legacy === "object" && typeof legacy.pluginId === "string") {
+    const key = legacy.tool && legacy.level && legacy.installPath ? locationKey(legacy) : LEGACY_LOCATION_KEY;
+    return { version: 2, locations: { [key]: legacy } };
+  }
+  logger.warn(t("record_corrupted").replace("{file}", recordPath));
+  return null;
 }
-function writeRecord(record) {
+function writeStore(pluginId, store) {
   const installsDir = getInstallsDir();
   if (!existsSync12(installsDir)) {
     mkdirSync5(installsDir, { recursive: true });
   }
-  const recordPath = getRecordPath(record.pluginId);
-  atomicWriteFileSync(recordPath, JSON.stringify(record, null, 2));
+  atomicWriteFileSync(getRecordPath(pluginId), JSON.stringify(store, null, 2));
 }
-function deleteRecord(pluginId) {
+function readRecords(pluginId) {
+  const store = readStore(pluginId);
+  return store ? Object.values(store.locations) : [];
+}
+var byLatest = (a, b) => (b.installTime || "").localeCompare(a.installTime || "");
+function readRecord(pluginId) {
+  const records = readRecords(pluginId);
+  if (records.length === 0) {
+    return null;
+  }
+  const cwd = resolve(process.cwd());
+  const localMatch = records.filter((r) => r.level === "project" && resolve(r.installPath) === cwd).sort(byLatest);
+  if (localMatch.length > 0) return localMatch[0];
+  const globalMatch = records.filter((r) => r.level === "global").sort(byLatest);
+  if (globalMatch.length > 0) return globalMatch[0];
+  return [...records].sort(byLatest)[0];
+}
+function writeRecord(record) {
+  const store = readStore(record.pluginId) ?? { version: 2, locations: {} };
+  store.locations[locationKey(record)] = record;
+  writeStore(record.pluginId, store);
+}
+function deleteRecord(pluginId, where) {
   const recordPath = getRecordPath(pluginId);
-  if (existsSync12(recordPath)) {
+  if (!existsSync12(recordPath)) {
+    return;
+  }
+  if (!where) {
+    unlinkSync3(recordPath);
+    return;
+  }
+  const store = readStore(pluginId);
+  if (!store) {
+    return;
+  }
+  delete store.locations[locationKey(where)];
+  if (Object.keys(store.locations).length === 0) {
     unlinkSync3(recordPath);
+  } else {
+    writeStore(pluginId, store);
   }
 }
 function scanInstalledFiles(pluginId, displayName, tool, level, installPath, configRoot, manifest, externalRepoNames, configRootConfigLink) {
@@ -3445,7 +3495,7 @@ function toInstalledEntry(target, registryPath, record) {
   };
 }
 function scanCannbotInstalled(target) {
-  const root = resolve(target || process.cwd());
+  const root = resolve2(target || process.cwd());
   const installed = [];
   for (const tool of CANNBOT_SUPPORTED_TOOLS) {
     const result = readCannbotRegistry(root, tool);
@@ -3471,7 +3521,7 @@ function countMarkerBlocks(content, pluginName) {
   return content.split(start).length - 1;
 }
 function removeIfSafe(pathToRemove, allowedBase) {
-  const resolved = resolve(pathToRemove);
+  const resolved = resolve2(pathToRemove);
   if (resolved !== allowedBase && !resolved.startsWith(allowedBase + sep2)) {
     return "skipped";
   }
@@ -3488,8 +3538,8 @@ function removeIfSafe(pathToRemove, allowedBase) {
   }
 }
 function removeEmptyDirUpward(dirPath, stopAt) {
-  let current = resolve(dirPath);
-  const stop = resolve(stopAt);
+  let current = resolve2(dirPath);
+  const stop = resolve2(stopAt);
   let removedAny = false;
   while (current.startsWith(stop + sep2)) {
     try {
@@ -3505,7 +3555,7 @@ function removeEmptyDirUpward(dirPath, stopAt) {
   return removedAny;
 }
 function cleanHookSettings(hookSettingsPath, target, pluginId) {
-  const resolved = resolve(target, hookSettingsPath);
+  const resolved = resolve2(target, hookSettingsPath);
   if (!existsSync13(resolved)) return;
   let settings;
   try {
@@ -3618,7 +3668,7 @@ function uninstallCannbotPlugin(pluginId, target) {
     removedDirs: 0,
     errors: []
   };
-  const root = resolve(target || process.cwd());
+  const root = resolve2(target || process.cwd());
   let read = null;
   let record;
   for (const tool of CANNBOT_SUPPORTED_TOOLS) {
@@ -3652,7 +3702,7 @@ function uninstallCannbotPlugin(pluginId, target) {
   }
   if (record.instructions) {
     const instructionsPath = join12(root, record.instructions);
-    const resolved = resolve(instructionsPath);
+    const resolved = resolve2(instructionsPath);
     if (resolved.startsWith(allowedBase + sep2) && existsSync13(resolved)) {
       try {
         const content = readFileSync9(resolved, "utf-8");
@@ -3736,16 +3786,21 @@ function uninstallCannbotPlugin(pluginId, target) {
 }
 function findOrphanedDelegatedRecords(target) {
   const orphans = [];
-  const installed = new Set(scanCannbotInstalled(target).map((e) => e.id));
   const installsDir = join12(getCannbotConfigDir(), "installs");
   if (!existsSync13(installsDir)) return orphans;
   for (const file of readdirSync6(installsDir)) {
     if (!file.endsWith(".json") || file === "skills.json") continue;
     const pluginId = file.replace(/\.json$/, "");
     try {
-      const rec = JSON.parse(readFileSync9(join12(installsDir, file), "utf-8"));
-      if (rec?.kind !== "delegated") continue;
-      if (!installed.has(pluginId)) {
+      const records = readRecords(pluginId);
+      const delegated = records.filter((r) => r?.kind === "delegated");
+      if (delegated.length === 0) continue;
+      const isOrphan = delegated.some((rec) => {
+        const recTarget = rec.installPath || target;
+        const installedThere = new Set(scanCannbotInstalled(recTarget).map((e) => e.id));
+        return !installedThere.has(pluginId);
+      });
+      if (isOrphan) {
         orphans.push(pluginId);
       }
     } catch {
@@ -3886,10 +3941,7 @@ function readAllRecords() {
   const records = [];
   const plugins = getAllPlugins();
   for (const plugin of plugins) {
-    const record = readRecord(plugin.id);
-    if (record) {
-      records.push(record);
-    }
+    records.push(...readRecords(plugin.id));
   }
   return records;
 }
@@ -4598,7 +4650,7 @@ function installerSupportsBundledFlag() {
 }
 async function delegateInstall(options) {
   const { pluginId, tool, level } = options;
-  const target = resolve2(options.target || process.cwd());
+  const target = resolve3(options.target || process.cwd());
   const displayName = options.displayName || pluginId;
   const result = {
     success: false,
@@ -5331,7 +5383,9 @@ function printEnhancedSummary(results, tool, configRoot) {
     ""
   ];
   if (successResults.length === 1) {
-    const plugin = findPlugin(successResults[0].pluginId);
+    const plugin = findPlugin(successResults[0].pluginId, {
+      source: successResults[0].source
+    });
     const docsPath = plugin ? `${plugin.dir}/quickstart.md` : `${successResults[0].pluginId}/quickstart.md`;
     lines.push(
       `  ${chalk6.dim(t("install_enhanced_docs"))}: ${chalk6.dim(docsPath)}`
@@ -5499,7 +5553,8 @@ async function pluginInstallFlow() {
     displayName: selected[index]?.plugin?.displayName || result.pluginId,
     success: result.success,
     skillsCount: result.skillsCount,
-    agentsCount: result.agentsCount
+    agentsCount: result.agentsCount,
+    source: selected[index]?.source
   }));
   printInstallSummary(summary);
   const configRoot = getConfigRoot(answers.tool, answers.level);
@@ -6099,6 +6154,9 @@ async function installCommand(names, options) {
           logger.info(`${t("install_to")}: ${chalk10.cyan(getSkillsRoot(tool2, level2))}`);
           logger.info(`${t("start_to_use").replace("{tool}", chalk10.green(tool2))}`);
           logger.blank();
+          if (failCount > 0) {
+            process.exitCode = 1;
+          }
           return;
         }
       }
@@ -6135,6 +6193,7 @@ async function installCommand(names, options) {
       logger.step(`  ${p.id}${badge} (${p.aliases.join(", ")})`);
     }
     logger.info(`${t("available_skills")}: ${chalk10.cyan("install-helper install --list")}`);
+    process.exitCode = 1;
     return;
   }
   let tool;
@@ -6188,7 +6247,6 @@ async function installCommand(names, options) {
         spinner.fail(t("repo_clone_failed").replace("{error}", error instanceof Error ? error.message : t("error_unknown")).replace("{url}", "").replace("{dir}", ""));
         return;
       }
-      const allPlugins = getAllPlugins();
       const results = await installPlugins(
         pluginsToInstall,
         tool,
@@ -6199,17 +6257,24 @@ async function installCommand(names, options) {
         source
       );
       const summary = results.map((result) => {
-        const plugin = allPlugins.find((p) => p.id === result.pluginId);
+        const plugin = findPlugin(result.pluginId, { source });
         return {
           pluginId: result.pluginId,
           displayName: plugin?.displayName || result.pluginId,
           success: result.success,
           skillsCount: result.skillsCount,
-          agentsCount: result.agentsCount
+          agentsCount: result.agentsCount,
+          // the command-line source is the user intent — legacy entries
+          // carry no `source` field, so plugin?.source alone would lose the
+          // "skills" scope and the docs hint would resolve cannbot-first
+          source: source ?? plugin?.source
         };
       });
       printInstallSummary(summary);
       printEnhancedSummary(summary, tool, configRoot);
+      if (results.some((r) => !r.success)) {
+        process.exitCode = 1;
+      }
     }
   }
   if (skills.length > 0) {
@@ -6246,6 +6311,9 @@ async function installCommand(names, options) {
     logger.info(`${t("install_to")}: ${chalk10.cyan(getSkillsRoot(tool, level))}`);
     logger.info(`${t("start_to_use").replace("{tool}", chalk10.green(tool))}`);
     logger.blank();
+    if (failCount > 0) {
+      process.exitCode = 1;
+    }
   }
 }
 async function installPlugins(pluginIds, tool, level, repoPath, installPath, yes, source) {
@@ -6286,7 +6354,7 @@ async function installPlugins(pluginIds, tool, level, repoPath, installPath, yes
 // src/commands/uninstall.ts
 init_registry();
 import { existsSync as existsSync21, unlinkSync as unlinkSync8, rmdirSync as rmdirSync2, readdirSync as readdirSync11, readFileSync as readFileSync15 } from "fs";
-import { resolve as resolve3, basename as basename4, sep as sep3, join as join20 } from "path";
+import { resolve as resolve4, basename as basename4, sep as sep3, join as join20 } from "path";
 import chalk11 from "chalk";
 import { select as select5, checkbox as checkbox2, Separator as Separator4, confirm as confirm3 } from "@inquirer/prompts";
 init_config();
@@ -6686,13 +6754,13 @@ async function uninstallPluginById(pluginId, batchMode) {
     return;
   }
   logger.info(`${t("uninstall_in_progress")} ${plugin.displayName}...`);
-  const configRoot = getConfigRoot(record.tool, record.level);
-  const allowedBases = [resolve3(configRoot), resolve3(record.installPath)];
+  const configRoot = record.configRoot;
+  const allowedBases = [resolve4(configRoot), resolve4(record.installPath)];
   if (record.tool === "codex") {
-    allowedBases.push(resolve3(getSkillsRoot(record.tool, record.level, record.installPath)));
+    allowedBases.push(resolve4(getSkillsRoot(record.tool, record.level, record.installPath)));
   }
   const isSafePath = (p) => {
-    const resolved = resolve3(p);
+    const resolved = resolve4(p);
     return allowedBases.some((base) => resolved === base || resolved.startsWith(base + sep3));
   };
   let removedFiles = 0;
@@ -6720,7 +6788,7 @@ async function uninstallPluginById(pluginId, batchMode) {
   const sortedDirs = [...record.directories].filter(isSafePath).sort((a, b) => b.length - a.length);
   for (const dirPath of sortedDirs) {
     try {
-      if (resolve3(dirPath) === resolve3(configRoot)) {
+      if (resolve4(dirPath) === resolve4(configRoot)) {
         continue;
       }
       if (existsSync21(dirPath)) {
@@ -6737,7 +6805,7 @@ async function uninstallPluginById(pluginId, batchMode) {
   }
   if (record.tool === "codex") {
     try {
-      const agentsRoot = resolve3(getSkillsRoot(record.tool, record.level, record.installPath), "..");
+      const agentsRoot = resolve4(getSkillsRoot(record.tool, record.level, record.installPath), "..");
       if (existsSync21(agentsRoot) && readdirSync11(agentsRoot).length === 0) {
         rmdirSync2(agentsRoot);
         logger.step(`  ${t("uninstall_clean_empty_dir")}: ${basename4(agentsRoot)}/`);
@@ -6746,7 +6814,11 @@ async function uninstallPluginById(pluginId, batchMode) {
     } catch {
     }
   }
-  deleteRecord(plugin.id);
+  deleteRecord(plugin.id, {
+    tool: record.tool,
+    level: record.level,
+    installPath: record.installPath
+  });
   removeInstalledPlugin(plugin.id);
   const backups = findBackups(configRoot);
   const otherBackups = backups.filter((b) => b.pluginId !== pluginId);