add-skill 1.0.22 → 1.0.23

package/dist/index.js

@@ -447,7 +447,13 @@ async function createSymlink(target, linkPath) {
       } else {
         await rm2(linkPath, { recursive: true });
       }
-    } catch {
+    } catch (err) {
+      if (err && typeof err === "object" && "code" in err && err.code === "ELOOP") {
+        try {
+          await rm2(linkPath, { force: true });
+        } catch {
+        }
+      }
     }
     const linkDir = join4(linkPath, "..");
     await mkdir(linkDir, { recursive: true });
@@ -469,10 +475,12 @@ async function installSkillForAgent(skill, agentType, options = {}) {
   const canonicalDir = join4(canonicalBase, skillName);
   const agentBase = isGlobal ? agent.globalSkillsDir : join4(cwd, agent.skillsDir);
   const agentDir = join4(agentBase, skillName);
+  const installMode = options.mode ?? "symlink";
   if (!isPathSafe(canonicalBase, canonicalDir)) {
     return {
       success: false,
       path: agentDir,
+      mode: installMode,
       error: "Invalid skill name: potential path traversal detected"
     };
   }
@@ -480,32 +488,49 @@ async function installSkillForAgent(skill, agentType, options = {}) {
     return {
       success: false,
       path: agentDir,
+      mode: installMode,
       error: "Invalid skill name: potential path traversal detected"
     };
   }
   try {
+    if (installMode === "copy") {
+      await mkdir(agentDir, { recursive: true });
+      await copyDirectory(skill.path, agentDir);
+      return {
+        success: true,
+        path: agentDir,
+        mode: "copy"
+      };
+    }
     await mkdir(canonicalDir, { recursive: true });
     await copyDirectory(skill.path, canonicalDir);
     const symlinkCreated = await createSymlink(canonicalDir, agentDir);
     if (!symlinkCreated) {
+      try {
+        await rm2(agentDir, { recursive: true, force: true });
+      } catch {
+      }
       await mkdir(agentDir, { recursive: true });
       await copyDirectory(skill.path, agentDir);
       return {
         success: true,
         path: agentDir,
         canonicalPath: canonicalDir,
+        mode: "symlink",
         symlinkFailed: true
       };
     }
     return {
       success: true,
       path: agentDir,
-      canonicalPath: canonicalDir
+      canonicalPath: canonicalDir,
+      mode: "symlink"
     };
   } catch (error) {
     return {
       success: false,
       path: agentDir,
+      mode: installMode,
       error: error instanceof Error ? error.message : "Unknown error"
     };
   }
@@ -599,7 +624,7 @@ function track(data) {
 // package.json
 var package_default = {
   name: "add-skill",
-  version: "1.0.22",
+  version: "1.0.23",
   description: "Install agent skills onto coding agents (OpenCode, Claude Code, Codex, Cursor)",
   type: "module",
   bin: {
@@ -886,6 +911,22 @@ async function main(source, options) {
       }
       installGlobally = scope;
     }
+    let installMode = "symlink";
+    if (!options.yes) {
+      const modeChoice = await p.select({
+        message: "Installation method",
+        options: [
+          { value: "symlink", label: "Symlink (Recommended)", hint: "Single source of truth, easy updates" },
+          { value: "copy", label: "Copy to all agents", hint: "Independent copies for each agent" }
+        ]
+      });
+      if (p.isCancel(modeChoice)) {
+        p.cancel("Installation cancelled");
+        await cleanup(tempDir);
+        process.exit(0);
+      }
+      installMode = modeChoice;
+    }
     const cwd = process.cwd();
     const summaryLines = [];
     const overwriteStatus = /* @__PURE__ */ new Map();
@@ -902,12 +943,17 @@ async function main(source, options) {
     );
     for (const skill of selectedSkills) {
       if (summaryLines.length > 0) summaryLines.push("");
-      const canonicalPath = getCanonicalPath(skill.name, { global: installGlobally });
-      const shortCanonical = shortenPath(canonicalPath, cwd);
-      summaryLines.push(`${chalk.cyan(shortCanonical)}`);
+      if (installMode === "symlink") {
+        const canonicalPath = getCanonicalPath(skill.name, { global: installGlobally });
+        const shortCanonical = shortenPath(canonicalPath, cwd);
+        summaryLines.push(`${chalk.cyan(shortCanonical)}`);
+        summaryLines.push(`  ${chalk.dim("symlink \u2192")} ${formatList(agentNames)}`);
+      } else {
+        summaryLines.push(`${chalk.cyan(getSkillDisplayName(skill))}`);
+        summaryLines.push(`  ${chalk.dim("copy \u2192")} ${formatList(agentNames)}`);
+      }
       const skillOverwrites = overwriteStatus.get(skill.name);
       const overwriteAgents = targetAgents.filter((a) => skillOverwrites?.get(a)).map((a) => agents[a].displayName);
-      summaryLines.push(`  ${chalk.dim("\u2192")} ${formatList(agentNames)}`);
       if (overwriteAgents.length > 0) {
         summaryLines.push(`  ${chalk.yellow("overwrites:")} ${formatList(overwriteAgents)}`);
       }
@@ -926,7 +972,7 @@ async function main(source, options) {
     const results = [];
     for (const skill of selectedSkills) {
       for (const agent of targetAgents) {
-        const result = await installSkillForAgent(skill, agent, { global: installGlobally });
+        const result = await installSkillForAgent(skill, agent, { global: installGlobally, mode: installMode });
         results.push({
           skill: getSkillDisplayName(skill),
           agent: agents[agent].displayName,
@@ -970,22 +1016,30 @@ async function main(source, options) {
       }
       const skillCount = bySkill.size;
       const agentCount = new Set(successful.map((r) => r.agent)).size;
-      const symlinkFailures = successful.filter((r) => r.symlinkFailed);
+      const symlinkFailures = successful.filter((r) => r.mode === "symlink" && r.symlinkFailed);
       const copiedAgents = symlinkFailures.map((r) => r.agent);
       const resultLines = [];
-      for (const [, skillResults] of bySkill) {
+      for (const [skillName, skillResults] of bySkill) {
         const firstResult = skillResults[0];
-        if (firstResult.canonicalPath) {
-          const shortPath = shortenPath(firstResult.canonicalPath, cwd);
-          resultLines.push(`${chalk.green("\u2713")} ${shortPath}`);
-        }
-        const symlinked = skillResults.filter((r) => !r.symlinkFailed).map((r) => r.agent);
-        const copied = skillResults.filter((r) => r.symlinkFailed).map((r) => r.agent);
-        if (symlinked.length > 0) {
-          resultLines.push(`  ${chalk.dim("\u2192")} ${formatList(symlinked)}`);
-        }
-        if (copied.length > 0) {
-          resultLines.push(`  ${chalk.yellow("copied \u2192")} ${formatList(copied)}`);
+        if (firstResult.mode === "copy") {
+          resultLines.push(`${chalk.green("\u2713")} ${skillName} ${chalk.dim("(copied)")}`);
+          for (const r of skillResults) {
+            const shortPath = shortenPath(r.path, cwd);
+            resultLines.push(`  ${chalk.dim("\u2192")} ${shortPath}`);
+          }
+        } else {
+          if (firstResult.canonicalPath) {
+            const shortPath = shortenPath(firstResult.canonicalPath, cwd);
+            resultLines.push(`${chalk.green("\u2713")} ${shortPath}`);
+          }
+          const symlinked = skillResults.filter((r) => !r.symlinkFailed).map((r) => r.agent);
+          const copied = skillResults.filter((r) => r.symlinkFailed).map((r) => r.agent);
+          if (symlinked.length > 0) {
+            resultLines.push(`  ${chalk.dim("symlink \u2192")} ${formatList(symlinked)}`);
+          }
+          if (copied.length > 0) {
+            resultLines.push(`  ${chalk.yellow("copied \u2192")} ${formatList(copied)}`);
+          }
         }
       }
       const title = chalk.green(`Installed ${skillCount} skill${skillCount !== 1 ? "s" : ""} to ${agentCount} agent${agentCount !== 1 ? "s" : ""}`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "add-skill",
-  "version": "1.0.22",
+  "version": "1.0.23",
   "description": "Install agent skills onto coding agents (OpenCode, Claude Code, Codex, Cursor)",
   "type": "module",
   "bin": {