claudekit-cli 3.32.3-dev.1 → 3.33.0-dev.1

package/dist/index.js

@@ -23196,7 +23196,7 @@ function getPagerArgs(pagerCmd) {
   return [];
 }
 async function trySystemPager(content) {
-  return new Promise((resolve13) => {
+  return new Promise((resolve14) => {
     const pagerCmd = process.env.PAGER || "less";
     const pagerArgs = getPagerArgs(pagerCmd);
     try {
@@ -23206,20 +23206,20 @@ async function trySystemPager(content) {
       });
       const timeout = setTimeout(() => {
         pager.kill();
-        resolve13(false);
+        resolve14(false);
       }, 30000);
       pager.stdin.write(content);
       pager.stdin.end();
       pager.on("close", (code2) => {
         clearTimeout(timeout);
-        resolve13(code2 === 0);
+        resolve14(code2 === 0);
       });
       pager.on("error", () => {
         clearTimeout(timeout);
-        resolve13(false);
+        resolve14(false);
       });
     } catch {
-      resolve13(false);
+      resolve14(false);
     }
   });
 }
@@ -23246,16 +23246,16 @@ async function basicPager(content) {
       break;
     }
     const remaining = lines.length - currentLine;
-    await new Promise((resolve13) => {
+    await new Promise((resolve14) => {
       rl.question(`-- More (${remaining} lines) [Enter/q] --`, (answer) => {
         if (answer.toLowerCase() === "q") {
           rl.close();
           process.exitCode = 0;
-          resolve13();
+          resolve14();
           return;
         }
         process.stdout.write("\x1B[1A\x1B[2K");
-        resolve13();
+        resolve14();
       });
     });
   }
@@ -48637,7 +48637,7 @@ async function findSkillByName(name2, sourcePath) {
 // src/commands/skills/skills-installer.ts
 import { existsSync as existsSync24 } from "node:fs";
 import { cp as cp2, mkdir as mkdir23, stat as stat12 } from "node:fs/promises";
-import { dirname as dirname14 } from "node:path";
+import { dirname as dirname14, resolve as resolve12 } from "node:path";
 
 // src/commands/skills/skills-registry.ts
 init_zod();
@@ -48750,6 +48750,13 @@ async function syncRegistry() {
 }
 
 // src/commands/skills/skills-installer.ts
+function isSamePath(path1, path22) {
+  try {
+    return resolve12(path1) === resolve12(path22);
+  } catch {
+    return false;
+  }
+}
 function getErrorMessage(error, targetPath) {
   if (error instanceof Error && "code" in error) {
     const code2 = error.code;
@@ -48773,6 +48780,16 @@ async function installSkillForAgent(skill, agent, options2) {
   const agentConfig = agents[agent];
   const targetPath = getInstallPath(skill.name, agent, options2);
   const alreadyExists = isSkillInstalled(skill.name, agent, options2);
+  if (isSamePath(skill.path, targetPath)) {
+    return {
+      agent,
+      agentDisplayName: agentConfig.displayName,
+      success: true,
+      path: targetPath,
+      skipped: true,
+      skipReason: "Skill already exists at source location"
+    };
+  }
   try {
     const parentDir = dirname14(targetPath);
     if (!existsSync24(parentDir)) {
@@ -49318,19 +49335,22 @@ async function skillsCommand(options2) {
     spinner.start(`Installing ${ctx.selectedSkills.length} skill(s)...`);
     let totalSuccessful = 0;
     let totalFailed = 0;
+    let totalSkipped = 0;
     const allResults = [];
     for (const skill of ctx.selectedSkills) {
       const results = await installSkillToAgents(skill, ctx.selectedAgents, {
         global: ctx.installGlobally
       });
       allResults.push({ skill: skill.name, results });
-      totalSuccessful += results.filter((r2) => r2.success).length;
+      totalSuccessful += results.filter((r2) => r2.success && !r2.skipped).length;
       totalFailed += results.filter((r2) => !r2.success).length;
+      totalSkipped += results.filter((r2) => r2.skipped).length;
     }
     spinner.stop("Installation complete");
     console.log();
     for (const { skill, results } of allResults) {
-      const successful = results.filter((r2) => r2.success);
+      const successful = results.filter((r2) => r2.success && !r2.skipped);
+      const skipped = results.filter((r2) => r2.skipped);
       const failed = results.filter((r2) => !r2.success);
       if (successful.length > 0) {
         f2.success(`${import_picocolors21.default.cyan(skill)} → ${successful.length} agent(s)`);
@@ -49338,6 +49358,12 @@ async function skillsCommand(options2) {
           f2.message(`  ${import_picocolors21.default.green("✓")} ${r2.agentDisplayName}`);
         }
       }
+      if (skipped.length > 0) {
+        f2.info(`${import_picocolors21.default.cyan(skill)} → ${skipped.length} skipped`);
+        for (const r2 of skipped) {
+          f2.message(`  ${import_picocolors21.default.yellow("○")} ${r2.agentDisplayName}: ${import_picocolors21.default.dim(r2.skipReason || "Already at source location")}`);
+        }
+      }
       if (failed.length > 0) {
         f2.error(`${import_picocolors21.default.cyan(skill)} failed: ${failed.length} agent(s)`);
         for (const r2 of failed) {
@@ -49346,13 +49372,20 @@ async function skillsCommand(options2) {
       }
     }
     console.log();
-    if (totalSuccessful === 0 && totalFailed === 0) {
+    if (totalSuccessful === 0 && totalFailed === 0 && totalSkipped === 0) {
       $e(import_picocolors21.default.yellow("No installations performed"));
     } else if (totalFailed > 0 && totalSuccessful === 0) {
       $e(import_picocolors21.default.red("Installation failed"));
       process.exit(1);
     } else {
-      $e(import_picocolors21.default.green(`Done! ${totalSuccessful} installed, ${totalFailed} failed`));
+      const parts = [];
+      if (totalSuccessful > 0)
+        parts.push(`${totalSuccessful} installed`);
+      if (totalSkipped > 0)
+        parts.push(`${totalSkipped} skipped`);
+      if (totalFailed > 0)
+        parts.push(`${totalFailed} failed`);
+      $e(import_picocolors21.default.green(`Done! ${parts.join(", ")}`));
     }
   } catch (error) {
     logger.error(error instanceof Error ? error.message : "Unknown error");
@@ -49407,7 +49440,7 @@ async function detectInstallations() {
 
 // src/commands/uninstall/removal-handler.ts
 import { readdirSync as readdirSync4, rmSync as rmSync5 } from "node:fs";
-import { join as join81, resolve as resolve12, sep as sep3 } from "node:path";
+import { join as join81, resolve as resolve13, sep as sep3 } from "node:path";
 init_logger();
 var import_fs_extra36 = __toESM(require_lib(), 1);
 
@@ -49544,8 +49577,8 @@ async function isDirectory(filePath) {
 }
 async function isPathSafeToRemove(filePath, baseDir) {
   try {
-    const resolvedPath = resolve12(filePath);
-    const resolvedBase = resolve12(baseDir);
+    const resolvedPath = resolve13(filePath);
+    const resolvedBase = resolve13(baseDir);
     if (!resolvedPath.startsWith(resolvedBase + sep3) && resolvedPath !== resolvedBase) {
       logger.debug(`Path outside installation directory: ${filePath}`);
       return false;
@@ -49553,7 +49586,7 @@ async function isPathSafeToRemove(filePath, baseDir) {
     const stats = await import_fs_extra36.lstat(filePath);
     if (stats.isSymbolicLink()) {
       const realPath = await import_fs_extra36.realpath(filePath);
-      const resolvedReal = resolve12(realPath);
+      const resolvedReal = resolve13(realPath);
       if (!resolvedReal.startsWith(resolvedBase + sep3) && resolvedReal !== resolvedBase) {
         logger.debug(`Symlink points outside installation directory: ${filePath} -> ${realPath}`);
         return false;
@@ -49938,7 +49971,7 @@ var import_fs_extra37 = __toESM(require_lib(), 1);
 // package.json
 var package_default = {
   name: "claudekit-cli",
-  version: "3.32.3-dev.1",
+  version: "3.33.0-dev.1",
   description: "CLI tool for bootstrapping and updating ClaudeKit projects",
   type: "module",
   repository: {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "claudekit-cli",
-	"version": "3.32.3-dev.1",
+	"version": "3.33.0-dev.1",
 	"description": "CLI tool for bootstrapping and updating ClaudeKit projects",
 	"type": "module",
 	"repository": {