memory-bank-skill 7.4.0 → 7.4.2

package/README.md

@@ -180,5 +180,5 @@ service=memory-bank Plugin initialized (unified) {"projectRoot":"..."}
 
 ## 版本
 
-- **版本**: 7.4.0
-- **主要更新**: Writer 轻量化 — 去掉 writer subagent，主 agent 直接写入 + Plugin 注入 writing guideline
+- **版本**: 7.4.1
+- **主要更新**: Writing guideline 改为 post-write advisory — 写入后建议参考规范检查，per-turn 去重

package/dist/cli.js

@@ -1,21 +1,5 @@
 #!/usr/bin/env bun
 // @bun
-var __create = Object.create;
-var __getProtoOf = Object.getPrototypeOf;
-var __defProp = Object.defineProperty;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __toESM = (mod, isNodeMode, target) => {
-  target = mod != null ? __create(__getProtoOf(mod)) : {};
-  const to = isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target;
-  for (let key of __getOwnPropNames(mod))
-    if (!__hasOwnProp.call(to, key))
-      __defProp(to, key, {
-        get: () => mod[key],
-        enumerable: true
-      });
-  return to;
-};
 var __require = import.meta.require;
 
 // src/cli.ts
@@ -27,7 +11,7 @@ import { fileURLToPath } from "url";
 // package.json
 var package_default = {
   name: "memory-bank-skill",
-  version: "7.4.0",
+  version: "7.4.2",
   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

@@ -1,24 +1,9 @@
 // @bun
-var __create = Object.create;
-var __getProtoOf = Object.getPrototypeOf;
-var __defProp = Object.defineProperty;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __toESM = (mod, isNodeMode, target) => {
-  target = mod != null ? __create(__getProtoOf(mod)) : {};
-  const to = isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target;
-  for (let key of __getOwnPropNames(mod))
-    if (!__hasOwnProp.call(to, key))
-      __defProp(to, key, {
-        get: () => mod[key],
-        enumerable: true
-      });
-  return to;
-};
 var __require = import.meta.require;
 
 // plugin/memory-bank.ts
 import { stat, readFile, access, realpath } from "fs/promises";
+import { realpathSync } from "fs";
 import { execSync } from "child_process";
 import path from "path";
 var DEBUG = process.env.MEMORY_BANK_DEBUG === "1";
@@ -374,7 +359,7 @@ function getMessageGatingState(gatingKey, sessionId, projectRoot) {
         meta.pendingDocFirstSatisfied = false;
       }
     }
-    state = { readFiles: new Set, contextSatisfied: false, warnedThisMessage: false, docFirstSatisfied: inheritDocFirst, docFirstWarned: false };
+    state = { readFiles: new Set, contextSatisfied: false, warnedThisMessage: false, docFirstSatisfied: inheritDocFirst, docFirstWarned: false, writingGuideInjected: false };
     messageGatingStates.set(gatingKey, state);
     if (messageGatingStates.size > 100) {
       const first = messageGatingStates.keys().next().value;
@@ -398,12 +383,63 @@ var FALLBACK_ANCHORS = [
 ];
 function canonicalizeRelPath(rawPath, projectRoot) {
   const abs = path.isAbsolute(rawPath) ? rawPath : path.resolve(projectRoot, rawPath);
-  const rel = path.relative(projectRoot, abs);
-  if (rel.startsWith(".."))
-    return "";
+  let rel = path.relative(projectRoot, abs);
+  if (rel.startsWith("..")) {
+    try {
+      const realAbs = realpathSync(abs);
+      const realRoot = realpathSync(projectRoot);
+      rel = path.relative(realRoot, realAbs);
+      if (rel.startsWith(".."))
+        return "";
+    } catch {
+      return "";
+    }
+  }
   const posix = rel.replace(/\\/g, "/");
   return process.platform === "darwin" || process.platform === "win32" ? posix.toLowerCase() : posix;
 }
+function stripQuotedContent(cmd) {
+  let result = "";
+  let inSingle = false;
+  let inDouble = false;
+  for (let i = 0;i < cmd.length; i++) {
+    const ch = cmd[i];
+    if (inSingle) {
+      if (ch === "'")
+        inSingle = false;
+      continue;
+    }
+    if (inDouble) {
+      if (ch === "\\" && i + 1 < cmd.length) {
+        i++;
+        continue;
+      }
+      if (ch === '"')
+        inDouble = false;
+      continue;
+    }
+    if (ch === "'") {
+      inSingle = true;
+      continue;
+    }
+    if (ch === '"') {
+      inDouble = true;
+      continue;
+    }
+    result += ch;
+  }
+  return result;
+}
+var BASH_REDIRECT_PATTERNS = [
+  /(?<![0-9&])>(?![&>])\s*\S/,
+  /(?<![0-9&])>>\s*\S/,
+  /\|\s*tee\s/,
+  /\bsed\s+(-[^-]*)?-i/,
+  /\bperl\s+(-[^-]*)?-[pi]/
+];
+function hasBashRedirectOutsideQuotes(cmd) {
+  return BASH_REDIRECT_PATTERNS.some((p) => p.test(stripQuotedContent(cmd)));
+}
 function isAnchorPath(canonicalPath) {
   return ANCHOR_PATH_PATTERNS.some((p) => p.test(canonicalPath));
 }
@@ -1061,17 +1097,7 @@ ${triggers.join(`
         if (anchorState.recovery?.required) {
           const writeToolsForRecovery = ["write", "edit", "multiedit", "apply_patch", "patch"];
           const isWriteTool = writeToolsForRecovery.includes(toolLowerForRecovery);
-          const isBashWrite = toolLowerForRecovery === "bash" && (() => {
-            const cmd = output.args?.command || "";
-            const bashWritePatterns = [
-              /(?<![0-9&])>(?![&>])\s*\S/,
-              /(?<![0-9&])>>\s*\S/,
-              /\|\s*tee\s/,
-              /\bsed\s+(-[^-]*)?-i/,
-              /\bperl\s+(-[^-]*)?-[pi]/
-            ];
-            return bashWritePatterns.some((p) => p.test(cmd));
-          })();
+          const isBashWrite = toolLowerForRecovery === "bash" && hasBashRedirectOutsideQuotes(output.args?.command || "");
           if (isWriteTool || isBashWrite) {
             const riskLevel = isWriteTool ? assessWriteRisk(toolLowerForRecovery, output.args || {}, projectRoot) : "medium";
             if (riskLevel !== "low") {
@@ -1229,13 +1255,6 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
         }
         if (toolLower2 === "bash" && !gatingState.contextSatisfied) {
           const command = output.args?.command || "";
-          const bashWritePatterns = [
-            /(?<![0-9&])>(?![&>])\s*\S/,
-            /(?<![0-9&])>>\s*\S/,
-            /\|\s*tee\s/,
-            /\bsed\s+(-[^-]*)?-i/,
-            /\bperl\s+(-[^-]*)?-[pi]/
-          ];
           const bashSensitivePatterns = [
             /package\.json/i,
             /\.env/i,
@@ -1247,7 +1266,7 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
             /\.github\/workflows/i,
             /infra\//i
           ];
-          const isLikelyWrite = bashWritePatterns.some((p) => p.test(command));
+          const isLikelyWrite = hasBashRedirectOutsideQuotes(command);
           const isSensitiveTarget = bashSensitivePatterns.some((p) => p.test(command));
           if (isLikelyWrite) {
             const riskLevel = isSensitiveTarget ? "high" : "medium";
@@ -1375,7 +1394,16 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
           }
         }
       }
-      async function injectWritingGuideline(sid) {
+      async function injectWritingGuideline(sid, writtenPaths) {
+        const meta = getSessionMeta(sid, projectRoot);
+        const messageKey = meta.lastUserMessageKey || "default";
+        const gatingKey = `${sid}::${messageKey}`;
+        const gatingState = getMessageGatingState(gatingKey, sid, projectRoot);
+        if (gatingState.writingGuideInjected)
+          return;
+        gatingState.writingGuideInjected = true;
+        const pathList = writtenPaths.map((p) => `- ${p}`).join(`
+`);
         client.session.prompt({
           path: { id: sid },
           body: {
@@ -1385,8 +1413,13 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
               type: "text",
               text: `## [Memory Bank Writing Guide]
 
-` + `\u6B63\u5728\u5199\u5165 memory-bank/\uFF0C\u8BF7\u5148\u52A0\u8F7D\u5199\u5165\u89C4\u8303\uFF1A
-` + `read({ filePath: "~/.config/opencode/skills/memory-bank/references/writer.md" })`
+` + `\u5DF2\u68C0\u6D4B\u5230 memory-bank/ \u5199\u5165\uFF1A
+${pathList}
+
+` + `\u5EFA\u8BAE\u53C2\u8003\u5199\u5165\u89C4\u8303\u786E\u8BA4\u5185\u5BB9\u662F\u5426\u9700\u8981\u8C03\u6574\uFF1A
+` + `read({ filePath: "~/.config/opencode/skills/memory-bank/references/writer.md" })
+
+` + `\u5982\u9700\u4FEE\u6B63\u8BF7\u76F4\u63A5\u4FEE\u6539\uFF0C\u7136\u540E\u7EE7\u7EED\u539F\u672C\u7684\u4EFB\u52A1\u3002`
             }]
           }
         }).catch((err) => log.error("Failed to send writing guideline:", String(err)));
@@ -1439,6 +1472,7 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
         const targetPaths = extractPaths(toolLower, output.args || {});
         if (targetPaths.length === 0)
           return;
+        const mbWrittenPaths = [];
         for (const targetPath of targetPaths) {
           if (!await isMemoryBankPath(targetPath))
             continue;
@@ -1447,10 +1481,13 @@ Or call: proxy_task({ subagent_type: "memory-reader", ... })`);
             throw new Error(`[Memory Bank Guard] memory-bank/ \u4E0B\u53EA\u5141\u8BB8\u5199\u5165 .md \u6587\u4EF6\u3002
 ` + `\u76EE\u6807\u6587\u4EF6: ${targetPath}`);
           }
-          await injectWritingGuideline(sessionID);
+          mbWrittenPaths.push(targetPath);
           markDocFirstSatisfied(sessionID);
           log.debug("Memory Bank write allowed", { sessionID, tool, targetPath });
         }
+        if (mbWrittenPaths.length > 0) {
+          await injectWritingGuideline(sessionID, mbWrittenPaths);
+        }
       }
       if (tool.toLowerCase() === "bash") {
         const command = output.args?.command;

package/package.json

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

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

@@ -284,6 +284,33 @@ USER_BLOCK 将保持不变。
 | Decision Highlights | 新增决策时追加（保持 10-20 条以内） |
 | Routing Rules | 仅在 details/ 结构变化时更新 |
 
+### MEMORY.md 膨胀控制
+
+MEMORY.md 是单入口索引，必须保持精简。按行数控制膨胀：
+
+| 行数 | 动作 |
+|------|------|
+| ≤ 120 行 | 正常维护 |
+| 120 - 200 行 | 审查：Decision Highlights 超 15 条归档旧条目到 details/patterns.md，Quick Answers 超 8 条删过期项 |
+| > 200 行 | 强制分区：膨胀区块提取到 details/，MEMORY.md 只留摘要 + 指针 |
+
+**分区操作**（> 200 行时）：
+
+```
+1. 识别膨胀区块（通常是 Decision Highlights 或 Routing Rules）
+2. 提取到 details/：
+   - Decision Highlights 旧条目 → details/patterns.md
+   - Routing Rules 过多 → details/routing-extended.md
+   - Quick Answers 过多 → details/faq.md
+3. MEMORY.md 保留：
+   - 最近 5-8 条 Decision Highlights
+   - 核心 Routing Rules（通用 + 高频项目特定）
+   - 3-5 条最高频 Quick Answers
+4. 添加指针：「完整历史见 details/patterns.md」
+```
+
+**原则**：MEMORY.md 是路由器不是数据库，超过一屏能扫完的量就该分区。
+
 ### 详情文件写入
 
 ```