@house365/comate 0.7.0 → 0.8.0

package/bin/init-run.mjs

@@ -11,10 +11,12 @@ import {
   SKILLS_TIMEOUT_MS,
   npmInvocation,
   openSpecState,
+  pruneDeprecatedSkills,
   run,
   scaffoldDocs,
   skillsInvocation,
   superpowersInstalled,
+  userSkillsRoot,
 } from './init.mjs';
 
 // `comate init` — no flags. Skills go user-level (~/.agents/skills), matching how
@@ -133,6 +135,13 @@ export async function init(pkgRoot) {
     results.push(['comate', r.ok ? 'installed' : 'failed', r.ok ? '' : r.reason, r.output]);
   }
 
+  // Prune comate skill dirs renamed/dropped in past releases. `skills add` above
+  // only adds/overwrites, so /comate-hotfix etc. would otherwise linger as dead
+  // slash commands. The postinstall hook does this too, but --ignore-scripts
+  // users never run it — init is their only path, so it must clean up here.
+  const pruned = pruneDeprecatedSkills(userSkillsRoot(home));
+  if (pruned.length) results.push(['清理废弃 skill', 'installed', pruned.join('、')]);
+
   // 4. openspec/ project structure. Not optional: comate's open and archive
   // phases read it, so an install without it cannot actually run. An existing
   // one is left alone, which covers projects that already keep it elsewhere.

package/bin/init.mjs

@@ -9,7 +9,7 @@
 // ponytail: no SKILLS_AGENT_MAP here. The skills CLI detects the running agent
 // on its own, so translating platform ids to agent names would be dead code.
 import { spawnSync } from 'node:child_process';
-import { existsSync, mkdirSync, readdirSync, copyFileSync, statSync } from 'node:fs';
+import { existsSync, mkdirSync, readdirSync, copyFileSync, statSync, lstatSync, rmSync } from 'node:fs';
 import { createRequire } from 'node:module';
 import { dirname, join } from 'node:path';
 
@@ -211,6 +211,83 @@ function scaffoldDocs(pkgRoot, cwd = process.cwd()) {
   return { ok: true, created, skipped };
 }
 
+// comate skill dirs that later releases renamed or dropped. `skills add` only
+// adds/overwrites, never removes, so without an explicit prune these linger in
+// ~/.agents/skills as dead slash commands (e.g. /comate-hotfix after it became
+// /comate-fix). ONLY these exact names are ever deleted: the dir also holds
+// unrelated third-party skills (lark-*, openspec-*), so a prefix sweep or a
+// diff-against-current strategy could wipe a user's own work. Add a line here
+// whenever a comate skill is renamed or removed.
+const DEPRECATED_SKILLS = ['comate-hotfix', 'comate-tweak', 'comate-intake'];
+
+function userSkillsRoot(home = process.env.HOME ?? process.env.USERPROFILE ?? '') {
+  return join(home, '.agents', 'skills');
+}
+
+// lstat, not stat: only a real directory is a comate-installed skill. A symlink
+// is a developer's deliberate dev setup pointing elsewhere — never remove it.
+function isRealDir(p) {
+  try {
+    return lstatSync(p).isDirectory();
+  } catch {
+    return false;
+  }
+}
+
+// Delete only the exact DEPRECATED_SKILLS dirs present under skillsRoot. Every
+// dependency is injectable so tests never touch a real ~/.agents/skills. Never
+// throws — a permission error or race on one dir must not abort an install.
+// Returns the names actually removed (for the caller to report).
+function pruneDeprecatedSkills(
+  skillsRoot,
+  {
+    names = DEPRECATED_SKILLS,
+    exists = existsSync,
+    isDir = isRealDir,
+    rm = (p) => rmSync(p, { recursive: true, force: true }),
+  } = {},
+) {
+  const removed = [];
+  for (const name of names) {
+    const dir = join(skillsRoot, name);
+    if (!exists(dir) || !isDir(dir)) continue;
+    try {
+      rm(dir);
+      removed.push(name);
+    } catch {
+      // A stuck dir (locked, permission) is not worth failing the install over.
+    }
+  }
+  return removed;
+}
+
+// True only when the pinned `skills` dependency resolves locally. A postinstall
+// must never reach the network, so this gates syncComateSkills: if resolution
+// fails we skip rather than let skillsInvocation fall back to `npx skills@latest`.
+function skillsResolvesLocally(pkgRoot) {
+  try {
+    createRequire(join(pkgRoot, 'package.json')).resolve('skills/package.json');
+    return true;
+  } catch {
+    return false;
+  }
+}
+
+// Copy comate's own skills into ~/.agents/skills via the bundled skills CLI —
+// the exact command `comate init` runs, so the layout is identical. Local-only:
+// skips (never goes to network) when the skills dep can't be resolved, leaving
+// `comate init` as the fallback. Returns { ok, skipped }.
+function syncComateSkills(pkgRoot, { home = process.env.HOME ?? process.env.USERPROFILE ?? '', runner = run } = {}) {
+  if (!skillsResolvesLocally(pkgRoot)) return { ok: false, skipped: 'no-local-cli' };
+  const r = runner(
+    skillsInvocation(pkgRoot, ['add', pkgRoot, '--skill', '*', '-g', '-y']),
+    SKILLS_TIMEOUT_MS,
+    { quiet: true },
+  );
+  const landed = existsSync(join(home, '.agents', 'skills', 'comate', 'SKILL.md'));
+  return { ok: r.ok && landed, skipped: null };
+}
+
 export {
   MIN_OPENSPEC,
   NPM_TIMEOUT_MS,
@@ -228,4 +305,8 @@ export {
   scaffoldSourceDir,
   copyTreeSkipExisting,
   scaffoldDocs,
+  DEPRECATED_SKILLS,
+  userSkillsRoot,
+  pruneDeprecatedSkills,
+  syncComateSkills,
 };

package/bin/postinstall.mjs

@@ -14,13 +14,33 @@ if (isCI() || isDevInstall()) {
 
 const { banner, pkgVersion } = await import('./init-run.mjs');
 
+// Best-effort: refresh the user-level skills so `npm i -g @latest` alone updates
+// them, and prune dirs renamed/dropped in past releases. All guarded — a failure
+// here must never break the npm install, and `comate init` remains the fallback
+// for --ignore-scripts users (who never reach this hook) and per-project setup.
+const sync = await syncSkillsQuietly();
+
 console.log(banner(pkgVersion(pkgRoot)));
 console.log(`  🎉 Congratulations! comate 已成功安装。\n`);
+if (sync.synced) console.log('  ✓ 已把 skills 同步到 ~/.agents/skills');
+if (sync.removed.length) console.log(`  ✓ 已清理废弃 skill：${sync.removed.join('、')}`);
 console.log('  下一步：');
-console.log('    在你的项目目录下运行  comate init  开始使用');
+console.log('    在你的项目目录下运行  comate init  完成项目初始化（openspec/、docs/）');
 console.log('    或运行                comate doctor  健康检查');
 console.log('    更多帮助              comate --help\n');
 
+async function syncSkillsQuietly() {
+  try {
+    const { syncComateSkills, pruneDeprecatedSkills, userSkillsRoot } = await import('./init.mjs');
+    const { ok } = syncComateSkills(pkgRoot);
+    const removed = pruneDeprecatedSkills(userSkillsRoot());
+    return { synced: ok, removed };
+  } catch {
+    // Any failure (no skills CLI resolvable, fs error) is silent: init covers it.
+    return { synced: false, removed: [] };
+  }
+}
+
 function isCI() {
   return Boolean(process.env.CI || process.env.GITHUB_ACTIONS);
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@house365/comate",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "comate —— OpenSpec + Superpowers 规约驱动的 AI 开发编排器（Agent Skills）",
   "type": "module",
   "license": "MIT",