memory-bank-skill 5.10.0 → 5.12.0

package/README.md

@@ -192,5 +192,5 @@ service=memory-bank Plugin initialized (unified) {"projectRoot":"..."}
 
 ## 版本
 
-- **版本**: 5.8.0
-- **主要更新**: 修复 skill 路径（`skill/` → `skills/`）+ 写入守卫 + Writer Agent 自动注册
+- **版本**: 5.11.0
+- **主要更新**: Writer 自动清理（目录文件数检查 + active.md 归档）+ 确认职责分离

package/dist/cli.js

@@ -10,7 +10,7 @@ import { fileURLToPath } from "url";
 // package.json
 var package_default = {
   name: "memory-bank-skill",
-  version: "5.10.0",
+  version: "5.12.0",
   description: "Memory Bank - \u9879\u76EE\u8BB0\u5FC6\u7CFB\u7EDF\uFF0C\u8BA9 AI \u52A9\u624B\u5728\u6BCF\u6B21\u5BF9\u8BDD\u4E2D\u90FD\u80FD\u5FEB\u901F\u7406\u89E3\u9879\u76EE\u4E0A\u4E0B\u6587",
   type: "module",
   main: "dist/plugin.js",

package/dist/plugin.js

@@ -4,17 +4,6 @@ import { stat, readFile, realpath } from "fs/promises";
 import { execSync } from "child_process";
 import path from "path";
 var DEBUG = process.env.MEMORY_BANK_DEBUG === "1";
-var DEFAULT_MAX_CHARS = 12000;
-var TRUNCATION_NOTICE = `
-
----
-
-[TRUNCATED] Memory Bank context exceeded size limit. Read files directly for complete content.`;
-var MEMORY_BANK_FILES = [
-  "memory-bank/brief.md",
-  "memory-bank/active.md",
-  "memory-bank/_index.md"
-];
 var SENTINEL_OPEN = "<memory-bank-bootstrap>";
 var SENTINEL_CLOSE = "</memory-bank-bootstrap>";
 var SERVICE_NAME = "memory-bank";
@@ -30,11 +19,6 @@ var agentBySessionID = new Map;
 function makeStateKey(sessionId, root) {
   return `${sessionId}::${root}`;
 }
-function maxChars() {
-  const raw = process.env.MEMORY_BANK_MAX_CHARS;
-  const n = raw ? Number(raw) : NaN;
-  return Number.isFinite(n) && n > 0 ? Math.floor(n) : DEFAULT_MAX_CHARS;
-}
 function isPluginGeneratedPrompt(message, content) {
   if (message?.variant === PLUGIN_PROMPT_VARIANT)
     return true;
@@ -103,98 +87,25 @@ function createLogger(client) {
     flush: () => pending
   };
 }
-async function readTextCached(absPath) {
-  try {
-    const st = await stat(absPath);
-    const cached = fileCache.get(absPath);
-    if (cached && cached.mtimeMs === st.mtimeMs)
-      return cached.text;
-    const text = await readFile(absPath, "utf8");
-    fileCache.set(absPath, { mtimeMs: st.mtimeMs, text });
-    return text;
-  } catch {
-    return null;
-  }
-}
-function truncateToBudget(text, budget) {
-  if (text.length <= budget)
-    return text;
-  const reserve = TRUNCATION_NOTICE.length;
-  if (budget <= reserve)
-    return TRUNCATION_NOTICE.slice(0, budget);
-  return text.slice(0, budget - reserve) + TRUNCATION_NOTICE;
-}
 async function buildMemoryBankContextWithMeta(projectRoot) {
-  const fnStart = Date.now();
   if (DEBUG)
     console.error(`[MB-DEBUG] buildMemoryBankContextWithMeta START projectRoot=${projectRoot}`);
-  const parts = [];
-  const files = [];
-  for (const rel of MEMORY_BANK_FILES) {
-    const fileStart = Date.now();
-    const abs = path.join(projectRoot, rel);
-    if (DEBUG)
-      console.error(`[MB-DEBUG] reading ${rel}...`);
-    const content = await readTextCached(abs);
+  const mbPath = path.join(projectRoot, "memory-bank");
+  try {
+    await stat(mbPath);
+  } catch {
     if (DEBUG)
-      console.error(`[MB-DEBUG] read ${rel} done, hasContent=${!!content}, elapsed=${Date.now() - fileStart}ms`);
-    if (!content)
-      continue;
-    const trimmed = content.trim();
-    if (!trimmed)
-      continue;
-    parts.push(`## ${rel}
-
-${trimmed}`);
-    files.push({ relPath: rel, chars: trimmed.length });
-  }
-  if (DEBUG)
-    console.error(`[MB-DEBUG] all files read, parts=${parts.length}, totalElapsed=${Date.now() - fnStart}ms`);
-  if (parts.length === 0)
+      console.error(`[MB-DEBUG] memory-bank not found`);
     return null;
-  const fileList = files.map((f) => f.relPath.replace("memory-bank/", "")).join(", ");
-  const totalChars = files.reduce((sum, f) => sum + f.chars, 0);
-  const header = `# Memory Bank Bootstrap (Auto-injected by OpenCode plugin)
-
-` + `Use \`memory-bank/_index.md\` to locate additional context files.
-` + `Read more files from \`memory-bank/\` as needed based on the task.
-
-` + `**AI \u884C\u4E3A\u6307\u4EE4**\uFF1A
-` + `- \u6BCF\u6B21\u56DE\u590D\u672B\u5C3E\u52A0\u4E00\u884C\u786E\u8BA4\uFF1A\`| \uD83D\uDCDA Memory Bank | ${fileList} (${totalChars.toLocaleString()} chars) |\`
-` + `- **Memory-first\uFF08\u6838\u5FC3\u539F\u5219\uFF09**\uFF1A
-` + `  - \u4EFB\u4F55\u95EE\u9898\u5148\u67E5 _index.md \u5E76\u6253\u5F00\u76F8\u5173\u6587\u6863\uFF08active.md/brief.md/tech.md/docs/requirements/learnings/patterns.md\uFF09
-` + `  - \u627E\u5230\u7B54\u6848 \u2192 \u5F15\u7528\u6587\u4EF6\u8DEF\u5F84\u76F4\u63A5\u56DE\u7B54\uFF1B_index.md \u4E0E\u5BF9\u5E94\u76EE\u5F55\u68C0\u7D22\u65E0\u679C\u6216\u786E\u8BA4\u5DF2\u8FC7\u65F6 \u2192 \u624D\u8BFB\u4EE3\u7801
-` + `  - \u6587\u6863\u4E0E\u4EE3\u7801\u4E0D\u4E00\u81F4\u65F6\u4EE5\u4EE3\u7801\u4E3A\u51C6\uFF0C\u4F46\u5FC5\u987B\u63D0\u8BAE\u66F4\u65B0\u6587\u6863
-` + `  - \u82E5\u56E0\u6587\u6863\u7F3A\u5931\u800C\u8BFB\u4E86\u4EE3\u7801 \u2192 \u8FD9\u672C\u8EAB\u5C31\u662F\u5199\u5165\u89E6\u53D1\u70B9\uFF0C\u9700\u70B9\u540D\u8981\u8865\u7684\u6587\u4EF6\u8DEF\u5F84
-` + `- **\u6587\u6863\u9A71\u52A8\u5F00\u53D1**\uFF1A
-` + `  - \u65B9\u6848\u8BA8\u8BBA\u5B8C\u6210\u540E\uFF0C**\u5148\u5199\u8BBE\u8BA1\u6587\u6863\u5230 memory-bank/docs/\uFF0C\u518D\u5199\u4EE3\u7801**
-` + `  - \u8BBE\u8BA1\u6587\u6863\u662F\u5951\u7EA6\uFF0C\u4EE3\u7801\u5B9E\u73B0\u8981\u7B26\u5408\u6587\u6863
-` + `  - \u5B9E\u73B0\u5B8C\u6210\u540E\u56DE\u987E\uFF1A\u5982\u6709\u504F\u5DEE\uFF0C\u51B3\u5B9A\u662F\u6539\u6587\u6863\u8FD8\u662F\u6539\u5B9E\u73B0
-` + `- **\u5199\u5165\u89E6\u53D1\u573A\u666F**\uFF08\u8BED\u4E49\u5224\u65AD\uFF0C\u975E\u5173\u952E\u8BCD\u5339\u914D\uFF09\uFF1A
-` + `  - \u65B9\u6848\u8BA8\u8BBA\u786E\u5B9A / \u8BBE\u8BA1\u53D8\u66F4\uFF08"\u91CD\u65B0\u8BBE\u8BA1"\u3001"\u6539\u4E00\u4E0B\u8BBE\u8BA1"\u3001"\u90A3\u5C31\u8FD9\u6837\u5427"\uFF09\u2192 \u68C0\u67E5 docs/design-*.md \u662F\u5426\u5DF2\u5B58\u5728\uFF0C\u5B58\u5728\u5219\u66F4\u65B0\uFF0C\u5426\u5219\u521B\u5EFA
-` + `  - \u7528\u6237\u63CF\u8FF0\u65B0\u529F\u80FD/\u9700\u6C42\uFF08"\u6211\u9700\u8981..."\u3001"\u80FD\u4E0D\u80FD\u52A0..."\u3001"\u5E2E\u6211\u505A..."\u3001"\u8981\u5B9E\u73B0..."\uFF09\u2192 requirements/
-` + `  - \u7528\u6237\u505A\u51FA\u6280\u672F\u9009\u578B\uFF08"\u6211\u4EEC\u7528 X \u5427"\u3001"\u51B3\u5B9A\u91C7\u7528..."\u3001"\u9009\u62E9..."\uFF09\u2192 patterns.md
-` + `  - \u4FEE\u590D\u4E86 bug \u6216\u8E29\u5751\u7ECF\u9A8C\uFF08"\u539F\u6765\u95EE\u9898\u662F..."\u3001"\u8FD9\u4E2A\u5751\u662F..."\u3001"\u53D1\u73B0..."\uFF09\u2192 learnings/
-` + `  - AI \u4FEE\u6539\u4E86\u4EE3\u7801/\u914D\u7F6E\u6587\u4EF6 \u2192 active.md\uFF08\u5982\u6D89\u53CA bug \u4FEE\u590D\u5219\u540C\u65F6 learnings/\uFF09
-` + `  - \u5F53\u524D\u4EFB\u52A1\u5B8C\u6210\uFF0C\u7126\u70B9\u5207\u6362 \u2192 active.md
-` + `- **Todo \u521B\u5EFA\u89C4\u5219\uFF08\u5FC5\u987B\uFF09**\uFF1A
-` + `  - \u65B9\u6848\u8BA8\u8BBA\u5B8C\u6210\u540E\u5F00\u59CB\u843D\u5730\uFF1A\u7B2C\u4E00\u9879\u5FC5\u987B\u662F"\u5199\u5165\u8BBE\u8BA1\u6587\u6863\u5230 memory-bank/docs/"
-` + `  - \u6700\u540E\u4E00\u9879\u5FC5\u987B\u662F"\u66F4\u65B0 Memory Bank"\uFF08\u68C0\u67E5\u89E6\u53D1\u573A\u666F\u5E76\u66F4\u65B0\u76F8\u5E94\u6587\u4EF6\uFF09
-
-` + `---
-
-`;
-  const wrapped = `${SENTINEL_OPEN}
-` + header + parts.join(`
-
----
-
-`) + `
-${SENTINEL_CLOSE}`;
-  const budget = maxChars();
-  const truncated = wrapped.length > budget;
-  const text = truncateToBudget(wrapped, budget);
-  return { text, files, totalChars, truncated };
+  }
+  const text = `${SENTINEL_OPEN}
+` + `**\u6267\u884C\u4EFB\u4F55\u5DE5\u4F5C\u6216\u56DE\u7B54\u95EE\u9898\u4E4B\u524D\uFF0C\u5FC5\u987B\u5148\u52A0\u8F7D \`/memory-bank\` skill\u3002**
+` + `${SENTINEL_CLOSE}`;
+  return { text, files: [], totalChars: 0, truncated: false };
+}
+async function buildMemoryBankContext(projectRoot) {
+  const result = await buildMemoryBankContextWithMeta(projectRoot);
+  return result?.text ?? null;
 }
 async function checkMemoryBankExists(root, log) {
   if (memoryBankExistsCache.has(root)) {
@@ -584,15 +495,24 @@ ${triggers.join(`
           log.info("[HOOK] system.transform SKIP (sentinel exists)", { elapsed: Date.now() - hookStart });
           return;
         }
-        const hasMemoryBank = await checkMemoryBankExists(projectRoot, log);
-        const instruction = hasMemoryBank ? `${SENTINEL_OPEN}
-` + `\u542F\u7528 Memory Bank\u3002\u4EFB\u4F55\u5DE5\u4F5C\u524D\uFF1A\u5148\u52A0\u8F7D memory-bank skill\uFF1B\u4F60\u7684\u7B2C\u4E00\u4E2A Read \u5FC5\u987B\u662F memory-bank/_index.md\uFF1B\u5728\u6B64\u4E4B\u524D\u7981\u6B62 Glob/Grep/\u4EE3\u7801\u641C\u7D22\u3002
-` + `${SENTINEL_CLOSE}` : `${SENTINEL_OPEN}
+        log.info("[HOOK] system.transform building context...");
+        const ctx = await buildMemoryBankContext(projectRoot);
+        log.info("[HOOK] system.transform context built", { hasCtx: !!ctx, elapsed: Date.now() - hookStart });
+        if (ctx) {
+          output.system.push(ctx);
+          log.info("[HOOK] system.transform DONE (ctx pushed)", { elapsed: Date.now() - hookStart });
+          return;
+        }
+        const initInstruction = `${SENTINEL_OPEN}
+` + `# Memory Bank \u672A\u521D\u59CB\u5316
+
 ` + `\u9879\u76EE \`${path.basename(projectRoot)}\` \u5C1A\u672A\u521D\u59CB\u5316 Memory Bank\u3002
-` + `\u5F00\u59CB\u5DE5\u4F5C\u524D\uFF1A\u5148\u52A0\u8F7D memory-bank skill \u67E5\u770B\u521D\u59CB\u5316\u6307\u5F15\u3002
+
+` + `**AI \u884C\u4E3A\u6307\u4EE4**\uFF1A
+` + `- **Todo \u521B\u5EFA\u89C4\u5219\uFF08\u5FC5\u987B\uFF09**\uFF1A\u521B\u5EFA todo \u65F6\uFF0C\u7B2C\u4E00\u9879\u5FC5\u987B\u662F"\u521D\u59CB\u5316 Memory Bank"\uFF08\u626B\u63CF\u9879\u76EE\u7ED3\u6784\uFF0C\u521B\u5EFA brief.md + tech.md + _index.md\uFF09\uFF0C\u6700\u540E\u4E00\u9879\u5FC5\u987B\u662F"\u66F4\u65B0 Memory Bank"\uFF08\u66F4\u65B0 active.md\uFF09
 ` + `${SENTINEL_CLOSE}`;
-        output.system.push(instruction);
-        log.info("[HOOK] system.transform DONE", { hasMemoryBank, elapsed: Date.now() - hookStart });
+        output.system.push(initInstruction);
+        log.info("[HOOK] system.transform DONE (init pushed)", { elapsed: Date.now() - hookStart });
       } catch (err) {
         log.error("[HOOK] system.transform ERROR", String(err), { elapsed: Date.now() - hookStart });
       }
@@ -605,15 +525,24 @@ ${triggers.join(`
           log.info("[HOOK] session.compacting SKIP (sentinel exists)", { elapsed: Date.now() - hookStart });
           return;
         }
-        const hasMemoryBank = await checkMemoryBankExists(projectRoot, log);
-        const instruction = hasMemoryBank ? `${SENTINEL_OPEN}
-` + `\u542F\u7528 Memory Bank\u3002\u4EFB\u4F55\u5DE5\u4F5C\u524D\uFF1A\u5148\u52A0\u8F7D memory-bank skill\uFF1B\u4F60\u7684\u7B2C\u4E00\u4E2A Read \u5FC5\u987B\u662F memory-bank/_index.md\uFF1B\u5728\u6B64\u4E4B\u524D\u7981\u6B62 Glob/Grep/\u4EE3\u7801\u641C\u7D22\u3002
-` + `${SENTINEL_CLOSE}` : `${SENTINEL_OPEN}
+        log.info("[HOOK] session.compacting building context...");
+        const ctx = await buildMemoryBankContext(projectRoot);
+        log.info("[HOOK] session.compacting context built", { hasCtx: !!ctx, elapsed: Date.now() - hookStart });
+        if (ctx) {
+          output.context.push(ctx);
+          log.info("[HOOK] session.compacting DONE (ctx pushed)", { elapsed: Date.now() - hookStart });
+          return;
+        }
+        const initInstruction = `${SENTINEL_OPEN}
+` + `# Memory Bank \u672A\u521D\u59CB\u5316
+
 ` + `\u9879\u76EE \`${path.basename(projectRoot)}\` \u5C1A\u672A\u521D\u59CB\u5316 Memory Bank\u3002
-` + `\u5F00\u59CB\u5DE5\u4F5C\u524D\uFF1A\u5148\u52A0\u8F7D memory-bank skill \u67E5\u770B\u521D\u59CB\u5316\u6307\u5F15\u3002
+
+` + `**AI \u884C\u4E3A\u6307\u4EE4**\uFF1A
+` + `- **Todo \u521B\u5EFA\u89C4\u5219\uFF08\u5FC5\u987B\uFF09**\uFF1A\u521B\u5EFA todo \u65F6\uFF0C\u7B2C\u4E00\u9879\u5FC5\u987B\u662F"\u521D\u59CB\u5316 Memory Bank"\uFF08\u626B\u63CF\u9879\u76EE\u7ED3\u6784\uFF0C\u521B\u5EFA brief.md + tech.md + _index.md\uFF09\uFF0C\u6700\u540E\u4E00\u9879\u5FC5\u987B\u662F"\u66F4\u65B0 Memory Bank"\uFF08\u66F4\u65B0 active.md\uFF09
 ` + `${SENTINEL_CLOSE}`;
-        output.context.push(instruction);
-        log.info("[HOOK] session.compacting DONE", { hasMemoryBank, elapsed: Date.now() - hookStart });
+        output.context.push(initInstruction);
+        log.info("[HOOK] session.compacting DONE (init pushed)", { elapsed: Date.now() - hookStart });
       } catch (err) {
         log.error("[HOOK] session.compacting ERROR", String(err), { elapsed: Date.now() - hookStart });
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "memory-bank-skill",
-  "version": "5.10.0",
+  "version": "5.12.0",
   "description": "Memory Bank - 项目记忆系统，让 AI 助手在每次对话中都能快速理解项目上下文",
   "type": "module",
   "main": "dist/plugin.js",

package/skills/memory-bank-writer/SKILL.md

@@ -74,24 +74,52 @@ description: Memory Bank 专用写入 Agent - 负责所有 memory-bank/ 目录
 | `patterns.md` | 追加内容（不覆盖） |
 | `_index.md` | 每次写入后自动更新 |
 
-## 输出格式
+## 确认职责分离
 
-每次写入前，输出计划：
+**重要**：用户确认由主 Agent 前置完成，Writer 只负责执行。
+
+| 步骤 | 负责方 | 动作 |
+|------|--------|------|
+| 1 | 主 Agent | 决定写入内容，输出计划 |
+| 2 | 主 Agent | 跟用户确认 |
+| 3 | 用户 | 确认或拒绝 |
+| 4 | 主 Agent | delegate 给 Writer（附带完整内容） |
+| 5 | **Writer** | **直接执行**（不再确认） |
+
+Writer 收到的 prompt 应包含：
+- 明确的写入目标（文件路径）
+- 完整的写入内容
+- 是创建还是更新
+
+## 执行输出格式
+
+执行完成后，输出报告：
+
+```
+[Memory Bank Writer 执行完成]
+
+已执行：
+- 创建: memory-bank/docs/design-auth.md (45 行)
+- 更新: memory-bank/_index.md
+
+状态：成功
+```
+
+如果执行前需要检查（如判断更新还是新建），输出决策理由后直接执行：
 
 ```
-[Memory Bank Writer 写入计划]
+[Memory Bank Writer]
 
 检查结果：
 - Glob docs/design-*.md → 找到 3 个文件
 - 相关文件：design-auth.md（标题含 "认证"）
+- 决策：更新（而非新建）
 
-决策：更新 memory-bank/docs/design-auth.md（而非新建）
+执行写入...
 
-将要写入：
+已完成：
 - 更新: memory-bank/docs/design-auth.md
 - 更新: memory-bank/_index.md
-
-执行写入...
 ```
 
 ## 索引更新
@@ -102,12 +130,64 @@ description: Memory Bank 专用写入 Agent - 负责所有 memory-bank/ 目录
 2. 更新文件 → 更新 `updated` 和 `size` 字段
 3. 删除文件 → 移除索引条目
 
+---
+
+## 自动清理（写入时执行）
+
+### 目录文件数检查
+
+写入 `learnings/`、`requirements/`、`docs/` 及其子目录时：
+
+```
+1. Glob 统计目标目录的 .md 文件数
+2. 如果 > 20 个：
+   - 分析文件主题（按文件名/标题聚类）
+   - 同主题 >= 4 个文件 → 创建子目录迁移
+   - 无明显主题 → 旧文件（> 90 天）移入 archive/
+3. 更新 _index.md
+```
+
+### active.md 归档检查
+
+写入 `active.md` 后：
+
+```
+1. 统计行数和已完成条目数（`- [x]`）
+2. 如果行数 > 120 或已完成 > 20：
+   - 创建/更新 archive/active_YYYY-MM.md
+   - 移出：已完成条目 + 超过 30 天的变更记录
+   - 保留：当前焦点、下一步、阻塞项、近 30 天变更
+3. 更新 _index.md
+```
+
+归档文件格式：
+
+```markdown
+# Active Archive - YYYY-MM
+
+> 归档于: YYYY-MM-DD
+
+## 已完成条目
+- {条目}
+
+## 历史变更
+| 日期 | 变更 |
+|------|------|
+```
+
+### 保护规则
+
+不参与清理：
+- 根目录文件：`_index.md`、`brief.md`、`tech.md`、`active.md`、`patterns.md`、`progress.md`
+- `archive/` 目录
+
 ## 禁止行为
 
-- 不要跳过 Glob 检查
-- 不要在不确定时创建新文件（宁可询问）
+- 不要跳过 Glob 检查（判断更新 vs 新建）
+- 不要等待用户确认（确认已由主 Agent 前置完成）
 - 不要修改 `memory-bank/` 以外的文件
 - 不要删除文件（除非明确要求）
+- 不要自行决定写入内容（内容由主 Agent 提供）
 
 ## 错误处理