@fanchao8609/agent_brain_sync 1.8.2 → 1.8.3

package/README.md

@@ -32,7 +32,11 @@ abs install --agent pi --no-mcp   # 只装 hook + skill，不要 MCP
 | claude-code | `~/.claude/settings.json` hooks | `mcpServers.abs` (stdio) | `~/.claude/skills/abs-agent-brain-sync/` |
 | codex | `~/.codex/hooks.json` | config.toml `[mcp_servers.abs]` | `~/.codex/skills/` |
 | opencode | `~/.config/opencode/plugins/abs.ts` | opencode.json mcp.abs | skills/ |
-| pi | `~/.pi/agent/extensions/abs.ts` | extension 内桥接 | skills/ |
+| pi | `~/.pi/agent/extensions/abs.ts` | `~/.pi/agent/mcp.json` `mcpServers.abs` | `~/.pi/agent/skills/` |
+
+> **skill 规则**：`skill/` 下每个含 `SKILL.md` 的子目录 = 一个 skill，
+> **目录名即安装名**（须与 frontmatter `name` 一致，否则 pi 会告警）。
+> 新增 skill 只需建目录，无需改代码。
 
 - **幂等**：重复安装 = 更新，写入前自动备份
 - **共存**：追加式合并，不会顶掉你这个事件上的其它 hook
@@ -110,7 +114,7 @@ npm i -g @fanchao8609/agent_brain_sync@latest
 abs install    # 重新刷 hook/skill
 ```
 
-> ⚠️ 改完源码（尤其 `skill/SKILL.md`、`src/`）后**必须 `abs install --yes` 重扇出**，否则宿主还在跑旧副本 —— 版本号与代码会脱节。发布用 `npm version patch`（自动打 tag），别手改 `package.json` 的 version；发布后 `npm view` 有缓存延迟，必要时 `npm cache clean --force` 再验。
+> ⚠️ 改完源码（尤其 `skill/*/SKILL.md`、`src/`）后**必须 `abs install --yes` 重扇出**，否则宿主还在跑旧副本 —— 版本号与代码会脱节。发布用 `npm version patch`（自动打 tag），别手改 `package.json` 的 version；发布后 `npm view` 有缓存延迟，必要时 `npm cache clean --force` 再验。
 
 ---
 
@@ -150,7 +154,7 @@ agent_brain_sync/
 ├── src/userconfig.js  使用者姓名配置（作者标记）
 ├── src/wrapup.js   Stop 收尾快照/归档
 ├── hooks/event.sh  hook 模板
-├── skill/SKILL.md  技能（装到各智能体）
+├── skill/<名称>/SKILL.md  技能（每个子目录 = 一个 skill，装到各智能体）
 └── test/           单测
 ```
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fanchao8609/agent_brain_sync",
-  "version": "1.8.2",
+  "version": "1.8.3",
   "description": "agent-brain-sync: 跨会话 AI 编码记忆 — hook 纯触发 + CLI/MCP 读写 .brain markdown 图谱, 防并发写保护。",
   "type": "module",
   "license": "MIT",

package/skill/{bug-hunter → abs-bug-hunter}/SKILL.md

@@ -1,5 +1,5 @@
 ---
-name: bug-hunter
+name: abs-bug-hunter
 description: 按流程论排查 bug，先定位改动 → 日志打印 → 顺藤摸瓜逐层排查
 ---
 

package/src/install.js

@@ -14,30 +14,64 @@ import { HOSTS, hostByKey } from './hosts.js';
 
 const ABS_DIR = resolve(dirname(fileURLToPath(import.meta.url)), '..');
 const HOOK_TEMPLATE = join(ABS_DIR, 'hooks', 'event.sh');
-const SKILL_SOURCE = join(ABS_DIR, 'skill', 'SKILL.md');
 
-// 随 abs 一并安装的附带 skill（agent 排查 bug 时挂载）。
-// 必须放在本包内随包发布 —— 不能假设宿主已装：实测 co​dex 只有 abs 自己装的 skill，
-// 若引用外部 skill 会成悬空。ponytail: 只加这一个，需要更多时改成扫 skill/ 子目录。
-const BUNDLED_SKILLS = [
-  { name: 'bug-hunter', src: join(ABS_DIR, 'skill', 'bug-hunter', 'SKILL.md') },
-];
-
-/** 安装主 skill + 全部附带 skill 到该宿主。收口在此：四处安装点共用，
- *  新增附带 skill 时只改本函数（避免「改一份不算改」）。返回步骤行。 */
+// ---- skill 包规则（单一规则，无特例）----
+// skill/ 下**每个含 SKILL.md 的子目录**就是一个 skill，随包发布。
+// 目录名 = 宿主 skills/ 下的安装目录名 = frontmatter 的 name（三者必须一致：
+// pi 在目录名与 name 不一致时会告警）。
+//
+// 为什么全带 abs- 前缀:
+//   1. 归属可辨 —— 带前缀才敢判「这是 abs 装的、可随 install 更新/卸载」；
+//      不带前缀的副本可能是用户自己放的，卸载时不敢动。
+//   2. 避开同名冲突 —— pi 同时扫 ~/.pi/agent/skills 与 ~/.agents/skills
+//      （pi 文档 Locations 一节），后者常有同名副本。两份同名实文件 → pi 报
+//      skill 冲突（实测: 曾装成 bug-hunter，与 ~/.agents/skills/bug-hunter 相撞）。
+//
+// 新增 skill: 建 skill/<名称>/SKILL.md 即可，无需改代码（原先写死一条，见旧注释 ponytail）。
+const SKILL_ROOT = join(ABS_DIR, 'skill');
+const ALL_SKILLS = (() => {
+  const out = [];
+  let entries = [];
+  try { entries = fsSync.readdirSync(SKILL_ROOT, { withFileTypes: true }); } catch { /* 无 skill/ 目录: 无 skill 可装 */ }
+  for (const e of entries) {
+    if (!e.isDirectory()) continue;
+    const src = join(SKILL_ROOT, e.name, 'SKILL.md');
+    if (!fsSync.existsSync(src)) continue; // 只认含 SKILL.md 的子目录
+    out.push({ name: e.name, src });
+  }
+  return out;
+})();
+// 主 skill（会话开场加载的那个）。它也只是 ALL_SKILLS 的一员，此处仅用于告警比对。
+const SKILL_SOURCE = join(SKILL_ROOT, 'abs-agent-brain-sync', 'SKILL.md');
+
+/** 安装全部 skill 到该宿主。收口在此：四处安装点共用 —— 新增 skill 只改
+ *  skill/ 目录，本函数与调用方都不动（避免「改一份不算改」）。返回步骤行。 */
 async function installSkills(agentKey) {
   const steps = [];
-  const base = hostSkillDir(agentKey);
-  await atomicWrite(join(base, 'SKILL.md'), await fs.readFile(SKILL_SOURCE, 'utf8'));
-  steps.push(`✓ skill  → ${join(base, 'SKILL.md')}`);
-  for (const b of BUNDLED_SKILLS) {
-    const t = join(base, '..', b.name, 'SKILL.md');
-    await atomicWrite(t, await fs.readFile(b.src, 'utf8'));
+  const root = join(hostSkillDir(agentKey), '..');
+  for (const s of ALL_SKILLS) {
+    const t = join(root, s.name, 'SKILL.md');
+    await atomicWrite(t, await fs.readFile(s.src, 'utf8'));
     steps.push(`✓ skill  → ${t}`);
   }
   return steps;
 }
 
+/** 卸载该宿主的全部 skill（主 + 附带）。与 installSkills 同源同规则。
+ *  曾经的坑: 只有 claude-code 删了附带 skill，其余三宿主留下 abs-bug-hunter/ 残留。
+ */
+async function uninstallSkills(agentKey) {
+  const steps = [];
+  const root = join(hostSkillDir(agentKey), '..');
+  for (const s of ALL_SKILLS) {
+    const d = join(root, s.name);
+    await fs.rm(d, { recursive: true, force: true });
+    steps.push(`✓ skill 已删除 → ${d}`);
+  }
+  return steps;
+}
+
+
 /**
  * 本包的稳定入口路径解析（mcp.js / abs.js 通用）。
  *
@@ -512,10 +546,7 @@ async function uninstallClaudeCode() {
   await fs.rm(join(homedir(), '.abs', 'hooks', 'claude-code'), { recursive: true, force: true });
   steps.push(`✓ ~/.abs/hooks/claude-code/ (本 agent hook 脚本) 已删除`);
   // skill
-  const skillDir = hostSkillDir('claude-code');
-  await fs.rm(skillDir, { recursive: true, force: true });
-  for (const b of BUNDLED_SKILLS) await fs.rm(join(skillDir, '..', b.name), { recursive: true, force: true });
-  steps.push(`✓ skill 已删除（含附带: ${BUNDLED_SKILLS.map((x) => x.name).join(", ")}）`);
+  steps.push(...await uninstallSkills('claude-code'));
   return steps;
 }
 
@@ -640,8 +671,7 @@ async function uninstallCodex() {
       steps.push(`✓ MCP 已从 ${mcpP} 移除`);
     }
   } catch {}
-  await fs.rm(hostSkillDir('codex'), { recursive: true, force: true });
-  steps.push(`✓ skill 已删除`);
+  steps.push(...await uninstallSkills('codex'));
   return steps;
 }
 
@@ -768,8 +798,7 @@ async function uninstallOpenCode() {
     await atomicWrite(mcpP, JSON.stringify(cfg, null, 2));
     steps.push(`✓ MCP 已从 ${mcpP} 移除`);
   }
-  await fs.rm(hostSkillDir('opencode'), { recursive: true, force: true });
-  steps.push(`✓ skill 已删除`);
+  steps.push(...await uninstallSkills('opencode'));
   return steps;
 }
 
@@ -801,8 +830,7 @@ async function uninstallPi() {
   const steps = [];
   await fs.rm(join(hostConfigRoot('pi'), 'agent', 'extensions', 'abs.ts'), { force: true });
   steps.push(`✓ extension 已删除`);
-  await fs.rm(hostSkillDir('pi'), { recursive: true, force: true });
-  steps.push(`✓ skill 已删除`);
+  steps.push(...await uninstallSkills('pi'));
   const mcpP = join(hostConfigRoot('pi'), 'agent', 'mcp.json');
   const cfg = await readJson(mcpP, { strict: false });
   if (cfg === null) {