skillbox 0.3.0 → 0.3.1

package/dist/cli.js

@@ -12,7 +12,7 @@ import { registerRemove } from "./commands/remove.js";
 import { registerStatus } from "./commands/status.js";
 import { registerUpdate } from "./commands/update.js";
 const program = new Command();
-program.name("skillbox").description("Local-first, agent-agnostic skills manager").version("0.3.0");
+program.name("skillbox").description("Local-first, agent-agnostic skills manager").version("0.3.1");
 registerAdd(program);
 registerAgent(program);
 registerConfig(program);

package/dist/commands/add-repo.js

@@ -41,8 +41,8 @@ async function installSkillTargets(skillName, options, installs) {
         const written = results
             .filter((result) => result.mode !== "skipped")
             .map((result) => result.path);
-        const warning = buildSymlinkWarning(agent, results);
-        if (warning) {
+        const warnings = buildSymlinkWarning(agent, results);
+        for (const warning of warnings) {
             printInfo(warning);
         }
         for (const target of written) {

package/dist/commands/add.js

@@ -74,8 +74,8 @@ export function registerAdd(program) {
                 const written = results
                     .filter((result) => result.mode !== "skipped")
                     .map((result) => result.path);
-                const warning = buildSymlinkWarning(agent, results);
-                if (warning) {
+                const warnings = buildSymlinkWarning(agent, results);
+                for (const warning of warnings) {
                     printInfo(warning);
                 }
                 if (written.length > 0) {

package/dist/lib/sync.js

@@ -17,18 +17,43 @@ async function copyFiles(sourceDir, targetDir) {
         await fs.copyFile(sourcePath, destPath);
     }
 }
+async function isSymlinkTo(targetDir, expectedSource) {
+    try {
+        const stat = await fs.lstat(targetDir);
+        if (!stat.isSymbolicLink()) {
+            return false;
+        }
+        const linkTarget = await fs.readlink(targetDir);
+        return linkTarget === expectedSource;
+    }
+    catch {
+        return false;
+    }
+}
 async function createSymlink(sourceDir, targetDir) {
+    if (await isSymlinkTo(targetDir, sourceDir)) {
+        return "exists";
+    }
     await fs.symlink(sourceDir, targetDir, "dir");
+    return "created";
 }
 export function buildSymlinkWarning(agent, results) {
     const skipped = results.filter((result) => result.mode === "skipped");
     if (skipped.length === 0) {
-        return null;
+        return [];
+    }
+    const warnings = [];
+    for (const result of skipped) {
+        const skillName = path.basename(result.path);
+        const isExists = result.error?.includes("EEXIST");
+        if (isExists) {
+            warnings.push(`  ⚠ ${skillName} (${agent}): already exists at target, remove manually or use --install-mode copy`);
+        }
+        else {
+            warnings.push(`  ⚠ ${skillName} (${agent}): ${result.error ?? "unknown error"}`);
+        }
     }
-    const details = skipped
-        .map((result) => `${result.path}: ${result.error ?? "unknown error"}`)
-        .join("; ");
-    return `Warning: symlink failed for ${agent}. ${details}. Remove the existing target or run "skillbox config set --install-mode copy" to use file copies.`;
+    return warnings;
 }
 export async function installSkillToTargets(skillName, targets, config) {
     const sourceDir = skillDir(skillName);
@@ -38,8 +63,8 @@ export async function installSkillToTargets(skillName, targets, config) {
         await ensureDir(targetRoot);
         if (config.installMode === "symlink") {
             try {
-                await createSymlink(sourceDir, targetDir);
-                results.push({ path: targetDir, mode: "symlink" });
+                const status = await createSymlink(sourceDir, targetDir);
+                results.push({ path: targetDir, mode: status === "exists" ? "symlink" : "symlink" });
             }
             catch (error) {
                 const message = getErrorMessage(error, "unknown error");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "skillbox",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Local-first, agent-agnostic skills manager",
   "license": "MIT",
   "author": "Christian Anagnostou",