team-skills 1.3.5 → 1.3.7

package/CHANGELOG.md

@@ -7,12 +7,30 @@
 
 ## [Unreleased]
 
+## [1.3.7] - 2026-06-26
+
+### 修复
+
+- 安装时自检：目标与源为同一文件时跳过删除/复制，防止误删源文件（`createSymlinkSafe` + `installSkillsProject` + `verifyGlobalSymlinks`）
+
+## [1.3.6] - 2026-06-26
+
+### 修复
+
+- D12：移除 4 个 SKILL.md 中 `###` 标题后的 bare `>` 空行（using-team-skills×3、team-brainstorm×6、team-finish×6、team-verify×4），统一为标准空行格式
+- D4：`verify_cmd` RESOLVE 链一致性修复——team-impl 和 team-test 补齐 `Cargo.toml`（与 team-verify/team-feedback 对齐）
+
 ## [1.3.5] - 2026-06-26
 
 ### 变更
 
 - `team-refine` 每轮新增 Step 4：team-score 满分校验（硬门槛覆盖 + 评分维度覆盖 + 缺口修复），收敛条件同步强化
 - 6 个 SKILL.md 质量修复：移除 bare `>` 空行（brainstorm）、`*none*` → `*default*` 兜底修正（review/security）、GATE 自检补充对抗性自问（brainstorm/orchestrator/score）
+- CLI 模块化重构：提取 `installers.js` 共享模块（5 个函数），命令文件总行数 -278 行
+
+### 修复
+
+- 项目路径与全局路径重叠时保留 symlinks，跳过文件复制（避免 `update`/`init` 覆盖全局 symlinks）
 
 ## [1.3.4] - 2026-06-26
 

package/package.json

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

package/skills/team-brainstorm/SKILL.md

@@ -67,7 +67,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 ## 执行步骤
 
 ### Phase 1：探索
->
+
 > 理解用户要解决的真实问题和项目现状。不要急于构思方案——先确认"问题是什么"比"答案是什么"更重要。
 
 1. **READ** 用户需求，提取核心目标和关键词
@@ -86,7 +86,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 8. **EXEC** 创建 `docs/tasks/{slug}/` 目录（**IF** 已存在 → 跳过）→ **ASSERT** `exit_code == 0`
 
 ### Phase 2：需求澄清（一次性提问）
->
+
 > 挖出用户未说出的假设和隐性约束。好问题比好答案更有价值——问错问题意味着后续全部方向偏移。
 
 > TRAP：你会倾向于接受用户的初始框架，不质疑其前提假设。
@@ -109,7 +109,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 - 整合用户回复，进入 Phase 3
 
 ### Phase 3：方案设计
->
+
 > 探索根本不同的解决路径，不是同一个想法的三种措辞。每个方案应从不同的设计取舍出发（如性能 vs 简洁、侵入式 vs 非侵入式）。
 
 > TRAP：锚定偏差——你会倾向于围绕第一个想到的方案展开变体，而非探索本质不同的路径。
@@ -136,7 +136,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 ```
 
 ### Phase 4：展示设计
->
+
 > 逐段确认而非一次倾倒，每段确认后再展示下一段。目标是让用户在每个维度上做出知情决策，而非被信息量压垮后草率同意（FP-1）。
 
 逐段展示设计，每段后等待用户确认：
@@ -162,7 +162,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 - *default* → 向用户澄清确认意图
 
 ### Phase 5：产出 00-design-brief.md
->
+
 > 将讨论共识固化为结构化文档。这是下游 Skill 的唯一输入——口头讨论不算数，写下来的才算数。
 
 > SIGNAL：方案缺少具体下一步行动 → brainstorm 停留在空想层面，补充可执行的交付物定义。
@@ -221,7 +221,7 @@ NO IMPLEMENTATION WITHOUT USER APPROVED DESIGN FIRST
 - 残留占位符 → 替换为实际值后重新 **WRITE**
 
 ### Phase 6：Handoff
->
+
 > 确保 brainstorm 成果顺滑传递给下游 Skill，不丢失上下文。
 
 **WRITE**（对话中）slug 目录路径 `docs/tasks/{slug}/`。

package/skills/team-finish/SKILL.md

@@ -61,7 +61,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 ## 执行步骤
 
 ### Step 1：验证测试
->
+
 > 用新鲜执行结果确认代码可交付。"上次跑过了"不是证据——只有当次输出才是。
 
 > TRAP：你会倾向于引用上一轮的测试结果来跳过重新执行。Iron Law 不允许——每次进入 finish 都必须重新运行。
@@ -78,7 +78,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 > 不可忽略失败继续展示选项（FP-4）。
 
 ### Step 1.5：凭证泄露扫描
->
+
 > 推送前最后一道安全防线。凭证泄露一旦进入远程仓库，撤回成本极高。
 
 **EXEC** `grep -rn -E '(AK|SK|access[_-]?key|secret[_-]?key|api[_-]?key|token|password|passwd|credential)\s*[:=]' .` — 推送前凭证扫描（RL-2）
@@ -87,7 +87,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 - **ELSE** → **GOTO** Step 2
 
 ### Step 2：确定基准分支
->
+
 > 精确找到合并目标。基准错误 = 合并到错误分支，后果比不合并更糟。
 
 **RESOLVE** `base_branch`（首个命中即停）：
@@ -105,7 +105,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 - 失败（分支无公共祖先）→ **BLOCKED**，触发 **H3**
 
 ### Step 3：展示选项
->
+
 > 让用户在完整信息下做选择。选项列表必须覆盖所有合理路径，不替用户预判。
 
 **WRITE**（对话中）选项列表：
@@ -122,7 +122,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 ```
 
 ### Step 4：执行选择
->
+
 > 严格按用户选择执行，不添加未要求的操作。不可逆操作必须二次确认。
 
 > TRAP：合并成功后你会倾向于跳过重新测试（"刚才不是通过了吗"）。合并引入的代码交互可能导致回归——必须重新验证。
@@ -181,7 +181,7 @@ NO BRANCH COMPLETION WITHOUT TEST VERIFICATION FIRST
 - *default* → **WRITE**（对话中）"请选择 A/B/C"
 
 ### Step 5：清理工作目录
->
+
 > 操作完成后工作区必须干净。残留的 worktree 或未提交变更是下次操作的隐患。
 
 > SIGNAL：`git status` 显示未提交变更 → 提交纪律不完整，有文件遗漏在 commit 之外。

package/skills/team-impl/SKILL.md

@@ -277,7 +277,7 @@ NO PRODUCTION CODE WITHOUT A FAILING TEST FIRST
 1. **RESOLVE** `verify_cmd`（首个命中即停）：
    1. `READ("05-risk.md", "§一验证计划")`
    2. `READ("CLAUDE.md").verify_cmd` / `READ(".cursor/rules/")`
-   3. `READ("package.json").scripts.test` / `READ("Makefile")` / `READ("CI 配置")`
+   3. `READ("package.json").scripts.test` / `READ("Makefile")` / `READ("Cargo.toml")` / `READ("CI 配置")`
    4. *none* → **NEEDS_CONTEXT**：请用户提供验证命令，记录到 `06-tdd-log.md`
 
 2. **EXEC** `verify_cmd`（测试）→ **ASSERT** `exit_code == 0` && `failures == 0`

package/skills/team-test/SKILL.md

@@ -124,7 +124,7 @@ Phase 1 只分析，不写测试代码。
 
 1. `READ("05-risk.md", "§一验证计划")`（精简模式下不存在属于正常）
 2. `READ("CLAUDE.md").verify_cmd` / `READ(".cursor/rules/")`
-3. `READ("package.json").scripts.test` / `READ("Makefile")`
+3. `READ("package.json").scripts.test` / `READ("Makefile")` / `READ("Cargo.toml")`
 4. *none* → **NEEDS_CONTEXT**：请用户提供测试命令
 
 ### Phase 4：补充测试（填补缺口）

package/skills/team-verify/SKILL.md

@@ -53,7 +53,7 @@ NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE FIRST
 ## 执行步骤
 
 ### Step 1：确定验证命令
->
+
 > 找到当前项目真正有效的验证命令。"上次用的命令"不等于"现在正确的命令"——项目配置可能已变更。
 
 **RESOLVE** `verify_cmd`（首个命中即停）：
@@ -66,7 +66,7 @@ NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE FIRST
    - *default* → **NEEDS_CONTEXT**：请用户提供验证命令
 
 ### Step 2：执行验证
->
+
 > 亲手执行，亲眼看到。"刚才跑过了"是最危险的借口——上一次结果不是当前事实。
 
 > TRAP：你会倾向于引用上一轮输出而非重新执行。每次到这一步，默认假设之前的结果已失效。
@@ -81,7 +81,7 @@ NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE FIRST
    - `exit_code != 0` || `failures > 0` → 记录失败详情 → fix → **GOTO** Step 2
 
 ### Step 3：报告结果
->
+
 > 报告是验证的证据——不是总结感想，而是呈现可追溯的事实。缺少 `exit_code` 或输出摘要的报告等于没有报告。
 
 > TRAP："Tests pass" 不等于 "changed code is tested"。如果测试数量比上次减少了，可能是删除了失败测试而非修复了 bug。
@@ -123,7 +123,7 @@ NO COMPLETION CLAIMS WITHOUT FRESH VERIFICATION EVIDENCE FIRST
 > ```
 
 ### Step 4：工具失败恢复
->
+
 > 区分"验证不通过"与"验证命令本身失败"。前者是代码问题，后者是环境问题。修复环境不等于验证通过。
 
 > 验证命令本身执行失败（超时、进程崩溃、环境错误），不同于验证不通过。

package/skills/using-team-skills/SKILL.md

@@ -70,7 +70,7 @@ NO SKILL RECOMMENDATION WITHOUT SCENE ANALYSIS FIRST
 ## 执行步骤
 
 ### Step 1：分析用户场景
->
+
 > 精准识别用户当前工程阶段，而非表面关键词匹配。错误分诊比慢分诊代价更高。
 
 > TRAP：你会倾向于推荐 `team-orchestrator`（最完整的流水线），即使用户只需要一个单点 Skill。
@@ -101,7 +101,7 @@ NO SKILL RECOMMENDATION WITHOUT SCENE ANALYSIS FIRST
 13. *none* → **NEEDS_CONTEXT**：请用户描述当前阶段和目标
 
 ### Step 2：推荐并说明理由
->
+
 > 推荐的核心是"最短路径"——从用户当前状态到目标状态经过最少的 Skill。过度推荐等于浪费用户时间。
 
 > GOOD：`用户报告登录接口 500 错误。当前阶段：调试。推荐：team-debug。理由：已有明确错误现象，需先定位根因再决定是否修改规格或实现。启动方式：/team-debug`
@@ -115,7 +115,7 @@ NO SKILL RECOMMENDATION WITHOUT SCENE ANALYSIS FIRST
 **WRITE**（对话中）推荐结果：推荐 Skill + 推荐理由 + 启动方式
 
 ### Step 3：可选 — 展示流程图
->
+
 > 仅在用户主动需要全貌时展示，不主动展开——信息过载是另一种误导。
 
 **IF** 用户需要了解全貌 → 展示 Mermaid 流程图

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++;
     }