team-skills 1.3.6 → 1.3.7

package/CHANGELOG.md

@@ -7,6 +7,12 @@
 
 ## [Unreleased]
 
+## [1.3.7] - 2026-06-26
+
+### 修复
+
+- 安装时自检：目标与源为同一文件时跳过删除/复制，防止误删源文件（`createSymlinkSafe` + `installSkillsProject` + `verifyGlobalSymlinks`）
+
 ## [1.3.6] - 2026-06-26
 
 ### 修复

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "team-skills",
-  "version": "1.3.6",
+  "version": "1.3.7",
   "description": "AI Agent Skills framework — Spec-Driven development with directed-graph rollback and quality gates",
   "type": "module",
   "bin": {

package/src/lib/fs-utils.js

@@ -1,7 +1,7 @@
 import {
   mkdirSync, symlinkSync, unlinkSync, readlinkSync,
   lstatSync, existsSync, readdirSync, statSync,
-  copyFileSync, chmodSync, rmSync,
+  copyFileSync, chmodSync, rmSync, realpathSync,
 } from 'node:fs';
 import { join, dirname } from 'node:path';
 
@@ -25,6 +25,12 @@ export function createSymlinkSafe(source, target, { force = false, dryRun = fals
     if (!force) return 'conflict';
     unlinkSync(target);
   } else if (existsSync(target)) {
+    // 目标存在但不是软连接 — 解析真实路径，如果是同一文件则视为已安装
+    try {
+      if (realpathSync(target) === realpathSync(source)) return 'exists';
+    } catch {
+      // 任一无法解析，继续走 force 逻辑
+    }
     if (!force) return 'conflict';
     rmSync(target, { recursive: true });
   }

package/src/lib/installers.js

@@ -1,5 +1,5 @@
 import { join } from 'node:path';
-import { existsSync, readdirSync, rmSync, copyFileSync } from 'node:fs';
+import { existsSync, readdirSync, rmSync, copyFileSync, realpathSync } from 'node:fs';
 import { createSymlinkSafe, ensureDir, isSymlink, copyRecursive } from './fs-utils.js';
 import { GLOBAL_TARGETS, PROJECT_IDE_DIRS } from './constants.js';
 import * as log from './logger.js';
@@ -56,21 +56,44 @@ export function verifyGlobalSymlinks(targets, skills, rules) {
   log.heading('验证安装');
   let errors = 0;
 
-  const verify = (label, dest) => {
-    if (isSymlink(dest)) {
-      log.success(label);
-    } else {
-      log.error(`${label} 未正确安装`);
-      errors++;
-    }
-  };
-
   for (const t of targets) {
     for (const skill of skills) {
-      verify(`${t.label} Skill: ${skill.name}`, join(t.dir, skill.name));
+      const dest = join(t.dir, skill.name);
+      const label = `${t.label} Skill: ${skill.name}`;
+      if (isSymlink(dest)) {
+        log.success(label);
+      } else if (existsSync(dest)) {
+        try {
+          if (realpathSync(dest) === realpathSync(skill.path)) {
+            log.skip(`${label}（已存在，跳过）`);
+            continue;
+          }
+        } catch { /* 忽略解析失败 */ }
+        log.error(`${label} 未正确安装`);
+        errors++;
+      } else {
+        log.error(`${label} 未正确安装`);
+        errors++;
+      }
     }
     for (const rule of rules) {
-      verify(`${t.label} Rule: ${rule.name}`, join(t.dir, '_team-rules', rule.name));
+      const dest = join(t.dir, '_team-rules', rule.name);
+      const label = `${t.label} Rule: ${rule.name}`;
+      if (isSymlink(dest)) {
+        log.success(label);
+      } else if (existsSync(dest)) {
+        try {
+          if (realpathSync(dest) === realpathSync(rule.path)) {
+            log.skip(`${label}（已存在，跳过）`);
+            continue;
+          }
+        } catch { /* 忽略解析失败 */ }
+        log.error(`${label} 未正确安装`);
+        errors++;
+      } else {
+        log.error(`${label} 未正确安装`);
+        errors++;
+      }
     }
   }
 
@@ -98,7 +121,16 @@ export function installSkillsProject(projectDir, ides, skills, rules, { dryRun,
     for (const skill of skills) {
       const dest = join(skillsDst, skill.name);
       if (!dryRun) {
-        if (existsSync(dest)) rmSync(dest, { recursive: true });
+        // 如果目标就是源文件本身，跳过删除+复制（否则删了就复制不了）
+        if (existsSync(dest)) {
+          try {
+            if (realpathSync(dest) === realpathSync(skill.path)) {
+              log.skip(`${tag}Skill: ${skill.name}（自身，跳过）`);
+              continue;
+            }
+          } catch { /* 忽略解析失败 */ }
+          rmSync(dest, { recursive: true });
+        }
         copyRecursive(skill.path, dest);
       }
       log.success(`${tag}Skill: ${skill.name}`);
@@ -108,7 +140,19 @@ export function installSkillsProject(projectDir, ides, skills, rules, { dryRun,
     const rulesDst = join(skillsDst, '_team-rules');
     if (!dryRun) ensureDir(rulesDst);
     for (const r of rules) {
-      if (!dryRun) copyFileSync(r.path, join(rulesDst, r.name));
+      if (!dryRun) {
+        const ruleDest = join(rulesDst, r.name);
+        // 如果目标就是源文件本身，跳过复制
+        if (existsSync(ruleDest)) {
+          try {
+            if (realpathSync(ruleDest) === realpathSync(r.path)) {
+              log.skip(`${tag}Rule: ${r.name}（自身，跳过）`);
+              continue;
+            }
+          } catch { /* 忽略解析失败 */ }
+        }
+        copyFileSync(r.path, ruleDest);
+      }
       log.success(`${tag}Rule: ${r.name}`);
       count++;
     }