team-skills 1.3.2 → 1.3.3

package/CHANGELOG.md

@@ -7,6 +7,21 @@
 
 ## [Unreleased]
 
+## [1.3.3] - 2026-06-26
+
+### 变更
+
+- `update` 命令新增全局安装阶段：同步更新 `~/.agents/skills/` 和 `~/.claude/skills/`（含 team-score）
+- `setup` 全局安装始终包含 team-score，移除 `--with-score` 参数
+- `setup`、`init`、`update` 统一为默认覆盖模式，移除 `--force` 参数
+- `init` 覆盖行为与 `update` 对齐：先清理再复制（`rmSync` + `copyRecursive`）
+
+### 修复
+
+- `update` 不再误删项目中已安装的 team-score（`cleanStaleSkills` 跳过被排除的 skill）
+- `setup` 验证阶段补全 Claude Code 共享规则检查（之前仅验证 Cursor 路径）
+- `update` 全局安装新增 symlink 验证阶段
+
 ## [1.3.2] - 2026-06-26
 
 ### 新增

package/package.json

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

package/src/commands/init.js

@@ -1,5 +1,5 @@
 import { join, resolve } from 'node:path';
-import { copyFileSync as fsCopyFile } from 'node:fs';
+import { copyFileSync as fsCopyFile, existsSync, rmSync } from 'node:fs';
 import { PACKAGE_ROOT } from '../lib/constants.js';
 import { discoverSkills, discoverSharedRules } from '../lib/inventory.js';
 import { copyRecursive, ensureDir } from '../lib/fs-utils.js';
@@ -41,7 +41,10 @@ function runInit(dir, opts) {
     if (!dryRun) ensureDir(skillsDst);
     for (const skill of skills) {
       const dest = join(skillsDst, skill.name);
-      if (!dryRun) copyRecursive(skill.path, dest);
+      if (!dryRun) {
+        if (existsSync(dest)) rmSync(dest, { recursive: true });
+        copyRecursive(skill.path, dest);
+      }
       log.success(`${tag}Skill: ${skill.name}`);
       count++;
     }
@@ -63,7 +66,10 @@ function runInit(dir, opts) {
     if (!dryRun) ensureDir(skillsDst);
     for (const skill of skills) {
       const dest = join(skillsDst, skill.name);
-      if (!dryRun) copyRecursive(skill.path, dest);
+      if (!dryRun) {
+        if (existsSync(dest)) rmSync(dest, { recursive: true });
+        copyRecursive(skill.path, dest);
+      }
       log.success(`${tag}Skill: ${skill.name}`);
       count++;
     }

package/src/commands/setup.js

@@ -11,25 +11,22 @@ export function registerSetup(program) {
     .command('setup')
     .description('Install skills via symlinks to global directories')
     .argument('[target]', 'Target skills directory', DEFAULT_SKILLS_TARGET)
-    .option('--with-score', 'Include team-score skill (hidden by default)', false)
-    .option('--force', 'Overwrite existing symlinks', false)
     .option('--dry-run', 'Show what would be done without doing it', false)
     .action(runSetup);
 }
 
 function runSetup(target, opts) {
-  const { withScore, force, dryRun } = opts;
+  const { dryRun } = opts;
   const tag = dryRun ? '[dry-run] ' : '';
-  const exclude = withScore ? [] : ['team-score'];
   let count = 0;
 
   // Skills → ~/.agents/skills/ (Cursor auto-discovers from here)
   log.heading('安装 Skills → Cursor');
-  const skills = discoverSkills(PACKAGE_ROOT, { exclude });
+  const skills = discoverSkills(PACKAGE_ROOT);
   for (const skill of skills) {
     const dest = join(target, skill.name);
-    const result = createSymlinkSafe(skill.path, dest, { force, dryRun });
-    logResult(`${tag}Skill: ${skill.name}`, result, dest);
+    const result = createSymlinkSafe(skill.path, dest, { force: true, dryRun });
+    logResult(`${tag}Skill: ${skill.name}`, result);
     if (result === 'created' || result === 'dry-run') count++;
   }
 
@@ -40,8 +37,8 @@ function runSetup(target, opts) {
   if (!dryRun) ensureDir(rulesTarget);
   for (const rule of rules) {
     const dest = join(rulesTarget, rule.name);
-    const result = createSymlinkSafe(rule.path, dest, { force, dryRun });
-    logResult(`${tag}Rule: ${rule.name}`, result, dest);
+    const result = createSymlinkSafe(rule.path, dest, { force: true, dryRun });
+    logResult(`${tag}Rule: ${rule.name}`, result);
     if (result === 'created' || result === 'dry-run') count++;
   }
 
@@ -51,8 +48,8 @@ function runSetup(target, opts) {
   if (!dryRun) ensureDir(claudeSkillsTarget);
   for (const skill of skills) {
     const dest = join(claudeSkillsTarget, skill.name);
-    const result = createSymlinkSafe(skill.path, dest, { force, dryRun });
-    logResult(`${tag}Skill: ${skill.name}`, result, dest);
+    const result = createSymlinkSafe(skill.path, dest, { force: true, dryRun });
+    logResult(`${tag}Skill: ${skill.name}`, result);
     if (result === 'created' || result === 'dry-run') count++;
   }
 
@@ -62,8 +59,8 @@ function runSetup(target, opts) {
   if (!dryRun) ensureDir(claudeRulesTarget);
   for (const rule of rules) {
     const dest = join(claudeRulesTarget, rule.name);
-    const result = createSymlinkSafe(rule.path, dest, { force, dryRun });
-    logResult(`${tag}Rule: ${rule.name}`, result, dest);
+    const result = createSymlinkSafe(rule.path, dest, { force: true, dryRun });
+    logResult(`${tag}Rule: ${rule.name}`, result);
     if (result === 'created' || result === 'dry-run') count++;
   }
 
@@ -82,7 +79,10 @@ function runSetup(target, opts) {
       verify(`Cursor Skill: ${skill.name}`, join(target, skill.name));
       verify(`Claude Skill: ${skill.name}`, join(claudeSkillsTarget, skill.name));
     }
-    for (const rule of rules) verify(`Rule: ${rule.name}`, join(rulesTarget, rule.name));
+    for (const rule of rules) {
+      verify(`Rule: ${rule.name}`, join(rulesTarget, rule.name));
+      verify(`Claude Rule: ${rule.name}`, join(claudeRulesTarget, rule.name));
+    }
     if (errors > 0) {
       log.error(`有 ${errors} 个组件安装异常，请检查。`);
       process.exit(1);
@@ -95,7 +95,7 @@ function runSetup(target, opts) {
   }
 }
 
-function logResult(label, result, dest) {
+function logResult(label, result) {
   switch (result) {
     case 'created':
     case 'dry-run':
@@ -104,9 +104,6 @@ function logResult(label, result, dest) {
     case 'exists':
       log.skip(`${label}（已存在，跳过）`);
       break;
-    case 'conflict':
-      log.warn(`${label}（已存在非 symlink，使用 --force 覆盖）`);
-      break;
     default:
       log.info(label);
   }

package/src/commands/update.js

@@ -1,20 +1,22 @@
 import { join, resolve } from 'node:path';
 import { existsSync, copyFileSync as fsCopyFile, rmSync, readdirSync, readFileSync } from 'node:fs';
 import { execSync } from 'node:child_process';
-import { PACKAGE_ROOT } from '../lib/constants.js';
+import {
+  PACKAGE_ROOT, DEFAULT_SKILLS_TARGET, DEFAULT_CLAUDE_SKILLS_TARGET,
+} from '../lib/constants.js';
 import { discoverSkills, discoverSharedRules } from '../lib/inventory.js';
-import { copyRecursive, ensureDir } from '../lib/fs-utils.js';
+import { copyRecursive, ensureDir, createSymlinkSafe, isSymlink } from '../lib/fs-utils.js';
 import { detectIDE } from '../lib/detect-ide.js';
 import * as log from '../lib/logger.js';
 
 export function registerUpdate(program) {
   program
     .command('update')
-    .description('Upgrade team-skills package and update project installations')
+    .description('Upgrade team-skills package and update global & project installations')
     .argument('[dir]', 'Project directory', '.')
     .option('--ide <type>', 'Force IDE type: claude, cursor, or both')
-    .option('--with-score', 'Include team-score skill (hidden by default)', false)
-    .option('--skip-self', 'Skip self-upgrade, only update project', false)
+    .option('--with-score', 'Include team-score skill in project (hidden by default)', false)
+    .option('--skip-self', 'Skip self-upgrade, only update installations', false)
     .option('--dry-run', 'Show what would change', false)
     .action(runUpdate);
 }
@@ -45,13 +47,13 @@ function upgradeSelf(dryRun) {
   }
 }
 
-function cleanStaleSkills(targetDir, currentNames, dryRun) {
+function cleanStaleSkills(targetDir, currentNames, dryRun, exclude = []) {
   if (!existsSync(targetDir)) return;
   const existing = readdirSync(targetDir).filter(
     name => !name.startsWith('_') && name !== 'CLAUDE.md' && name.startsWith('team-'),
   );
   for (const name of existing) {
-    if (!currentNames.has(name)) {
+    if (!currentNames.has(name) && !exclude.includes(name)) {
       const tag = dryRun ? '[dry-run] ' : '';
       if (!dryRun) rmSync(join(targetDir, name), { recursive: true });
       log.warn(`${tag}移除旧 Skill: ${name}`);
@@ -66,33 +68,105 @@ function runUpdate(dir, opts) {
   if (!skipSelf) upgradeSelf(dryRun);
 
   const exclude = withScore ? [] : ['team-score'];
+  const tag = dryRun ? '[dry-run] ' : '';
+  let count = 0;
+
+  // ── Global: always include team-score ──
+  const allSkills = discoverSkills(PACKAGE_ROOT);
+  const rules = discoverSharedRules();
+  const allSkillNames = new Set(allSkills.map(s => s.name));
+
+  log.heading('更新全局 Skills');
+
+  // Cursor global: ~/.agents/skills/
+  const cursorGlobal = DEFAULT_SKILLS_TARGET;
+  log.heading(`更新 Skills → ${cursorGlobal}`);
+  cleanStaleSkills(cursorGlobal, allSkillNames, dryRun);
+  if (!dryRun) ensureDir(cursorGlobal);
+  for (const skill of allSkills) {
+    const dest = join(cursorGlobal, skill.name);
+    const result = createSymlinkSafe(skill.path, dest, { force: true, dryRun });
+    logGlobalResult(`${tag}Skill: ${skill.name}`, result);
+    if (result === 'created' || result === 'dry-run') count++;
+  }
+  const cursorRulesDst = join(cursorGlobal, '_team-rules');
+  if (!dryRun) ensureDir(cursorRulesDst);
+  for (const r of rules) {
+    const dest = join(cursorRulesDst, r.name);
+    const result = createSymlinkSafe(r.path, dest, { force: true, dryRun });
+    logGlobalResult(`${tag}Rule: ${r.name}`, result);
+    if (result === 'created' || result === 'dry-run') count++;
+  }
+
+  // Claude Code global: ~/.claude/skills/
+  const claudeGlobal = DEFAULT_CLAUDE_SKILLS_TARGET;
+  log.heading(`更新 Skills → ${claudeGlobal}`);
+  cleanStaleSkills(claudeGlobal, allSkillNames, dryRun);
+  if (!dryRun) ensureDir(claudeGlobal);
+  for (const skill of allSkills) {
+    const dest = join(claudeGlobal, skill.name);
+    const result = createSymlinkSafe(skill.path, dest, { force: true, dryRun });
+    logGlobalResult(`${tag}Skill: ${skill.name}`, result);
+    if (result === 'created' || result === 'dry-run') count++;
+  }
+  const claudeRulesDst = join(claudeGlobal, '_team-rules');
+  if (!dryRun) ensureDir(claudeRulesDst);
+  for (const r of rules) {
+    const dest = join(claudeRulesDst, r.name);
+    const result = createSymlinkSafe(r.path, dest, { force: true, dryRun });
+    logGlobalResult(`${tag}Rule: ${r.name}`, result);
+    if (result === 'created' || result === 'dry-run') count++;
+  }
+
+  // Verify global symlinks
+  if (!dryRun) {
+    log.heading('验证全局安装');
+    let errors = 0;
+    const verify = (label, dest) => {
+      if (isSymlink(dest)) {
+        log.success(label);
+      } else {
+        log.error(`${label} 未正确安装`);
+        errors++;
+      }
+    };
+    for (const skill of allSkills) {
+      verify(`Cursor Skill: ${skill.name}`, join(cursorGlobal, skill.name));
+      verify(`Claude Skill: ${skill.name}`, join(claudeGlobal, skill.name));
+    }
+    for (const r of rules) {
+      verify(`Rule: ${r.name}`, join(cursorRulesDst, r.name));
+      verify(`Claude Rule: ${r.name}`, join(claudeRulesDst, r.name));
+    }
+    if (errors > 0) {
+      log.error(`有 ${errors} 个全局组件安装异常，请检查。`);
+    }
+  }
+
+  // ── Project: respect --with-score ──
   const ides = detectIDE(dir, ide);
 
   if (ides.length === 0) {
     log.info('当前项目未检测到 IDE 配置（.claude/ 或 .cursor/），跳过项目更新。');
+    log.done(`更新完成${dryRun ? ' (dry-run)' : ''}！共 ${count} 个组件。`);
     return;
   }
 
-  const tag = dryRun ? '[dry-run] ' : '';
-  let count = 0;
+  const projectSkills = discoverSkills(PACKAGE_ROOT, { exclude });
+  const projectSkillNames = new Set(projectSkills.map(s => s.name));
 
   log.heading('更新项目中的 team-skills');
   log.info(`项目目录: ${dir}`);
   log.info(`目标 IDE: ${ides.join(', ')}`);
 
-  const skills = discoverSkills(PACKAGE_ROOT, { exclude });
-  const rules = discoverSharedRules();
-  const currentSkillNames = new Set(skills.map(s => s.name));
-
-  // Cursor: skills → .cursor/skills/
   if (ides.includes('cursor')) {
     const skillsDst = join(dir, '.cursor', 'skills');
     log.heading(`更新 Skills → ${skillsDst}`);
 
-    cleanStaleSkills(skillsDst, currentSkillNames, dryRun);
+    cleanStaleSkills(skillsDst, projectSkillNames, dryRun, exclude);
 
     if (!dryRun) ensureDir(skillsDst);
-    for (const skill of skills) {
+    for (const skill of projectSkills) {
       const dest = join(skillsDst, skill.name);
       if (!dryRun) {
         if (existsSync(dest)) rmSync(dest, { recursive: true });
@@ -111,15 +185,14 @@ function runUpdate(dir, opts) {
     }
   }
 
-  // Claude Code: skills → .claude/skills/ (same structure as Cursor)
   if (ides.includes('claude')) {
     const skillsDst = join(dir, '.claude', 'skills');
     log.heading(`更新 Skills → ${skillsDst}`);
 
-    cleanStaleSkills(skillsDst, currentSkillNames, dryRun);
+    cleanStaleSkills(skillsDst, projectSkillNames, dryRun, exclude);
 
     if (!dryRun) ensureDir(skillsDst);
-    for (const skill of skills) {
+    for (const skill of projectSkills) {
       const dest = join(skillsDst, skill.name);
       if (!dryRun) {
         if (existsSync(dest)) rmSync(dest, { recursive: true });
@@ -140,3 +213,17 @@ function runUpdate(dir, opts) {
 
   log.done(`更新完成${dryRun ? ' (dry-run)' : ''}！共 ${count} 个组件。`);
 }
+
+function logGlobalResult(label, result) {
+  switch (result) {
+    case 'created':
+    case 'dry-run':
+      log.success(label);
+      break;
+    case 'exists':
+      log.skip(`${label}（已存在，跳过）`);
+      break;
+    default:
+      log.info(label);
+  }
+}