@synity/bitrix-skills 1.3.7 → 1.3.8

package/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## 1.3.8
+
+### Patch Changes
+
+- fix(list): global features now show correct [installed] status by checking ~/.claude/skills/<name>/ on disk instead of project manifest
+
+  fix(install): add --all flag + generic install handler for bx, bx-crm, bx-calendar global skills
+
 ## 1.3.7
 
 ### Patch Changes

package/dist/cli.js

@@ -944,11 +944,15 @@ import { createRequire } from "module";
 
 // src/commands/install.ts
 init_esm_shims();
+init_fs_safety();
 import { Command, Option } from "clipanion";
 import chalk from "chalk";
 import { existsSync as existsSync4 } from "fs";
+import { access as access4, copyFile as copyFile3, mkdir as mkdir4, readdir as readdir2 } from "fs/promises";
 import { createInterface } from "readline";
-import { join as join4, relative as relative2 } from "path";
+import { dirname as dirname3, join as join4, relative as relative2, resolve as resolve2 } from "path";
+import { homedir as homedir2 } from "os";
+import { fileURLToPath as fileURLToPath5 } from "url";
 
 // src/lib/feature-registry.ts
 init_esm_shims();
@@ -1046,11 +1050,11 @@ function computeChecksum(filepath) {
 
 // src/commands/install.ts
 function promptKey() {
-  return new Promise((resolve2) => {
+  return new Promise((resolve4) => {
     const rl = createInterface({ input: process.stdin, output: process.stdout });
     rl.question(chalk.cyan("License key (Enter to skip \u2014 free tier only): "), (answer) => {
       rl.close();
-      resolve2(answer.trim());
+      resolve4(answer.trim());
     });
   });
 }
@@ -1096,6 +1100,63 @@ async function installBxTask(cwd) {
     return { ok: false, message: `bx-task: ${err.message}` };
   }
 }
+async function getFeatureAssetsDir(featureName) {
+  const here = dirname3(fileURLToPath5(import.meta.url));
+  const candidates = [
+    // dist/cli.js context (inlined): ../src/features/<name>/assets
+    resolve2(here, `../src/features/${featureName}/assets`),
+    // dist/features/<name>/ context: ../../../src/features/<name>/assets
+    resolve2(here, `../../../src/features/${featureName}/assets`),
+    // dev (running from src/)
+    resolve2(here, `../features/${featureName}/assets`)
+  ];
+  for (const c of candidates) {
+    try {
+      await access4(c);
+      return c;
+    } catch {
+    }
+  }
+  return candidates[0];
+}
+async function copyDirRecursive(srcDir, destDir, skillBase, installedFiles, onConflict) {
+  await mkdir4(destDir, { recursive: true });
+  const entries = await readdir2(srcDir, { withFileTypes: true });
+  for (const entry of entries) {
+    const src = join4(srcDir, entry.name);
+    const dest = join4(destDir, entry.name);
+    if (entry.isDirectory()) {
+      await copyDirRecursive(src, dest, skillBase, installedFiles, onConflict);
+      continue;
+    }
+    assertContainedIn(dest, skillBase, dest);
+    await assertNotSymlink(dest);
+    try {
+      await access4(dest);
+      if (onConflict === "skip") continue;
+    } catch {
+    }
+    await copyFile3(src, dest);
+    installedFiles.push(dest);
+  }
+}
+async function installGlobalSkill(featureName) {
+  try {
+    const skillBase = resolve2(homedir2(), ".claude", "skills");
+    const dest = resolve2(skillBase, featureName);
+    const assetsDir = await getFeatureAssetsDir(featureName);
+    const installedFiles = [];
+    await copyDirRecursive(assetsDir, dest, skillBase, installedFiles, "overwrite");
+    return {
+      ok: true,
+      message: `${featureName}: ${installedFiles.length} files installed`,
+      installPath: dest,
+      installedAbsPaths: installedFiles
+    };
+  } catch (err) {
+    return { ok: false, message: `${featureName}: ${err.message}` };
+  }
+}
 function upsertFeature(manifest, entry) {
   const idx = manifest.features.findIndex((f) => f.name === entry.name);
   const features = [...manifest.features];
@@ -1122,6 +1183,7 @@ var InstallCommand = class extends Command {
     ]
   });
   key = Option.String("--key", { description: "License key to unlock paid tier features" });
+  all = Option.Boolean("--all", false, { description: "Install all available features" });
   featuresFlag = Option.String("--features", { description: "Comma-separated feature names" });
   featureArgs = Option.Rest({ required: 0 });
   async execute() {
@@ -1207,10 +1269,15 @@ Installing ${chalk.bold(name)}...
         ok = r.ok;
         message = r.message;
         if (r.installPath) {
-          const { homedir: homedir2 } = await import("os");
           installPath = join4(homedir2(), ".claude", "skills", "bx-task");
         }
         installedAbsPaths = r.installedAbsPaths ?? [];
+      } else if (featureInfo.target === "global") {
+        const r = await installGlobalSkill(name);
+        ok = r.ok;
+        message = r.message;
+        if (r.installPath) installPath = r.installPath;
+        installedAbsPaths = r.installedAbsPaths ?? [];
       } else {
         this.context.stderr.write(chalk.yellow(`  ! No install handler for feature: ${name}
 `));
@@ -1321,6 +1388,15 @@ Uninstalling ${chalk2.bold(name)}...
 init_esm_shims();
 import { Command as Command3 } from "clipanion";
 import chalk3 from "chalk";
+import { existsSync as existsSync5 } from "fs";
+import { resolve as resolve3 } from "path";
+import { homedir as homedir3 } from "os";
+function isInstalled(f, manifest) {
+  if (f.target === "global") {
+    return existsSync5(resolve3(homedir3(), ".claude", "skills", f.name));
+  }
+  return !!manifest?.features.find((e) => e.name === f.name);
+}
 var ListCommand = class extends Command3 {
   static paths = [["list"]];
   static usage = Command3.Usage({
@@ -1329,20 +1405,19 @@ var ListCommand = class extends Command3 {
   async execute() {
     const features = listFeatures();
     const manifest = readManifest(process.cwd());
-    const installed = new Map(manifest?.features.map((f) => [f.name, f]) ?? []);
     if (features.length === 0) {
       this.context.stdout.write(chalk3.yellow("No features found.\n"));
       return 0;
     }
     this.context.stdout.write("\nAvailable features:\n");
     for (const f of features) {
-      const entry = installed.get(f.name);
-      const check = entry ? chalk3.green("\u2713") : chalk3.gray("\u25CB");
+      const installed = isInstalled(f, manifest);
+      const check = installed ? chalk3.green("\u2713") : chalk3.gray("\u25CB");
       const name = chalk3.bold(f.name.padEnd(12));
       const ver = chalk3.cyan(f.version.padEnd(14));
       const target = chalk3.gray(f.target.padEnd(8));
       const desc = f.description;
-      const status = entry ? chalk3.green("[installed]") : chalk3.gray("[not installed]");
+      const status = installed ? chalk3.green("[installed]") : chalk3.gray("[not installed]");
       this.context.stdout.write(`  ${check}  ${name} ${ver} ${target} ${desc}  ${status}
 `);
     }
@@ -1354,7 +1429,7 @@ var ListCommand = class extends Command3 {
 init_esm_shims();
 import { Command as Command4 } from "clipanion";
 import chalk4 from "chalk";
-import { existsSync as existsSync5 } from "fs";
+import { existsSync as existsSync6 } from "fs";
 import { join as join5 } from "path";
 var VerifyCommand = class extends Command4 {
   static paths = [["verify"]];
@@ -1378,7 +1453,7 @@ var VerifyCommand = class extends Command4 {
 `);
       this.context.stdout.write(chalk4.gray(`    install path: ${feature.installPath}
 `));
-      if (!existsSync5(feature.installPath)) {
+      if (!existsSync6(feature.installPath)) {
         this.context.stderr.write(chalk4.red(`    \u2717 install path missing: ${feature.installPath}
 `));
         anyMismatch = true;
@@ -1391,7 +1466,7 @@ var VerifyCommand = class extends Command4 {
       }
       for (const [rel, expectedHash] of Object.entries(stored)) {
         const absPath = join5(feature.installPath, rel);
-        if (!existsSync5(absPath)) {
+        if (!existsSync6(absPath)) {
           this.context.stderr.write(chalk4.red(`    \u2717 missing: ${rel}
 `));
           anyMismatch = true;
@@ -1430,7 +1505,7 @@ var VerifyCommand = class extends Command4 {
 init_esm_shims();
 import { Command as Command5 } from "clipanion";
 import chalk5 from "chalk";
-import { existsSync as existsSync6 } from "fs";
+import { existsSync as existsSync7 } from "fs";
 import { relative as relative3 } from "path";
 var UpdateCommand = class extends Command5 {
   static paths = [["update"]];
@@ -1478,7 +1553,7 @@ var UpdateCommand = class extends Command5 {
           const { buildDestMap: buildDestMap2 } = await Promise.resolve().then(() => (init_dest_map(), dest_map_exports));
           const assetManifest = await loadManifest2();
           const dests = buildDestMap2(assetManifest, cwd);
-          installedAbsPaths = dests.map((d) => d.destAbs).filter((p) => existsSync6(p));
+          installedAbsPaths = dests.map((d) => d.destAbs).filter((p) => existsSync7(p));
         } catch {
         }
       } else if (feature.name === "bx-task") {
@@ -1507,7 +1582,7 @@ var UpdateCommand = class extends Command5 {
       const checksums = {};
       for (const absPath of installedAbsPaths) {
         try {
-          if (existsSync6(absPath)) {
+          if (existsSync7(absPath)) {
             const rel = relative3(feature.installPath, absPath);
             checksums[rel] = computeChecksum(absPath);
           }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@synity/bitrix-skills",
-  "version": "1.3.7",
+  "version": "1.3.8",
   "description": "Multi-feature Bitrix24 tooling CLI for Synity projects",
   "type": "module",
   "bin": {