skill-linker 4.1.1 → 4.1.2

package/README.md

@@ -21,7 +21,7 @@
 
 ```bash
 # 安裝技能（需要 --skill 或 --from）
-npx /app/workspace/projects/skill-linker install --skill <路徑> --agent opencode --scope both --yes
+npx skill-linker install --skill <路徑> --agent opencode --scope both --yes
 npx skill-linker install --from https://github.com/anthropics/skills --agent claude --scope both
 
 # 列出已安裝的 Repos
@@ -159,6 +159,8 @@ npx skill-linker install --from https://github.com/obra/superpowers --agent clau
 
 1. **權限問題**：在建立 Symlink 時，請確保您有對應目錄的寫入權限。
 2. **環境需求**：需安裝 Node.js 18.0.0 以上版本。
+3. **Windows**：建立 Symlink 需開啟「開發者模式」或以系統管理員權限執行，否則 `fs.symlinkSync` 會失敗。
+4. **覆寫保護**：`--yes` 只會覆寫既有的 Symlink；若目標是「真實目錄／檔案」，工具會拒絕刪除以保護資料。
 
 ## 授權
 

package/package.json

@@ -1,13 +1,13 @@
 {
   "name": "skill-linker",
-  "version": "4.1.1",
+  "version": "4.1.2",
   "description": "CLI to link AI Agent Skills to various agents (Claude, Copilot, Antigravity, Cursor, etc.)",
   "main": "bin/cli.js",
   "bin": {
     "skill-linker": "bin/cli.js"
   },
   "scripts": {
-    "test": "echo \"No tests yet\""
+    "test": "node --test"
   },
   "keywords": [
     "ai",

package/src/cli.js

@@ -19,10 +19,7 @@ program
 program
   .command("install")
   .description("Install a skill to specified agents")
-  .requiredOption(
-    "--skill <path>",
-    "Path to skill directory or --from clone URL",
-  )
+  .option("--skill <path>", "Path to a local skill directory")
   .option("--from <github-url>", "Clone skill from GitHub URL first, then link")
   .option(
     "-a, --agent <names...>",

package/src/commands/install.js

@@ -75,6 +75,15 @@ async function install(options) {
     skillPaths = [options.skill];
   }
 
+  // Require at least one source: --skill or --from
+  if (skillPaths.length === 0) {
+    console.error(
+      chalk.red("[ERROR]"),
+      "No skill source provided. Use --skill <path> or --from <github-url>.",
+    );
+    process.exit(1);
+  }
+
   // Validate skill paths
   for (const p of skillPaths) {
     if (!dirExists(p)) {
@@ -147,6 +156,9 @@ async function install(options) {
   }
   console.log(chalk.blue("[INFO]"), `Scope: ${scope}`);
 
+  let linkedCount = 0;
+  let failedCount = 0;
+
   // Process each selected agent
   for (const agentIndex of selectedAgents) {
     const agent = agents[agentIndex];
@@ -189,15 +201,27 @@ async function install(options) {
 
         if (createSymlink(sPath, targetLink)) {
           console.log(chalk.green("[SUCCESS]"), `Linked ${sName}`);
+          linkedCount++;
         } else {
           console.error(chalk.red("[ERROR]"), `Failed to link ${sName}`);
+          failedCount++;
         }
       }
     }
   }
 
   console.log("");
-  console.log(chalk.green("[SUCCESS]"), "All operations completed.");
+  if (failedCount > 0) {
+    console.error(
+      chalk.red("[ERROR]"),
+      `Completed with errors: ${linkedCount} linked, ${failedCount} failed.`,
+    );
+    process.exit(1);
+  }
+  console.log(
+    chalk.green("[SUCCESS]"),
+    `All operations completed (${linkedCount} linked).`,
+  );
 }
 
 module.exports = install;

package/src/utils/file-system.js

@@ -35,9 +35,27 @@ function createSymlink(source, target) {
         // Ensure parent directory exists
         ensureDir(path.dirname(target));
 
-        // Remove existing link/file/directory if present
-        // force: true makes it ignore the error if file doesn't exist
-        fs.rmSync(target, { recursive: true, force: true });
+        // Inspect the target itself without following symlinks.
+        let stat = null;
+        try {
+            stat = fs.lstatSync(target);
+        } catch {
+            // Target does not exist — nothing to remove.
+        }
+
+        if (stat) {
+            if (stat.isSymbolicLink()) {
+                // Safe: we are replacing a link, not real content.
+                fs.rmSync(target, { force: true });
+            } else {
+                // A real file or directory lives here. Refuse to delete it so
+                // we never destroy user data that we did not create.
+                console.error(
+                    `Refusing to overwrite non-symlink target: ${target}`,
+                );
+                return false;
+            }
+        }
 
         fs.symlinkSync(source, target, 'dir');
         return true;