memory-bank-skill 5.6.2 → 5.7.1

package/dist/cli.js

@@ -10,7 +10,7 @@ import { fileURLToPath } from "url";
 // package.json
 var package_default = {
   name: "memory-bank-skill",
-  version: "5.6.2",
+  version: "5.7.1",
   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,6 +1,6 @@
 // @bun
 // plugin/memory-bank.ts
-import { stat, readFile } from "fs/promises";
+import { stat, readFile, realpath } from "fs/promises";
 import { execSync } from "child_process";
 import path from "path";
 var DEBUG = process.env.MEMORY_BANK_DEBUG === "1";
@@ -753,12 +753,44 @@ ${triggers.join(`
     },
     "tool.execute.before": async (input, output) => {
       const { tool, sessionID } = input;
-      const isMemoryBankPath = (targetPath) => {
+      const isCaseInsensitiveFS = process.platform === "darwin" || process.platform === "win32";
+      const normalize = (p) => isCaseInsensitiveFS ? p.toLowerCase() : p;
+      let realProjectRoot = null;
+      let realMemoryBankDir = null;
+      const getRealPaths = async () => {
+        if (realProjectRoot === null) {
+          try {
+            realProjectRoot = normalize(await realpath(projectRoot));
+          } catch {
+            realProjectRoot = normalize(projectRoot);
+          }
+          try {
+            realMemoryBankDir = normalize(await realpath(path.join(projectRoot, "memory-bank")));
+          } catch {
+            realMemoryBankDir = normalize(path.join(projectRoot, "memory-bank"));
+          }
+        }
+        return { realProjectRoot, realMemoryBankDir };
+      };
+      const isMemoryBankPath = async (targetPath) => {
+        const { realProjectRoot: rootReal, realMemoryBankDir: mbReal } = await getRealPaths();
         const absPath = path.isAbsolute(targetPath) ? targetPath : path.resolve(projectRoot, targetPath);
-        const relativePath = path.relative(projectRoot, absPath);
-        if (relativePath === ".." || relativePath.startsWith(".." + path.sep))
-          return false;
-        return relativePath === "memory-bank" || relativePath.startsWith("memory-bank" + path.sep) || relativePath.startsWith("memory-bank/");
+        const lexicalNorm = normalize(absPath);
+        const lexicalMBDir = normalize(path.join(projectRoot, "memory-bank"));
+        const lexicalMatch = lexicalNorm === lexicalMBDir || lexicalNorm.startsWith(lexicalMBDir + path.sep) || lexicalNorm.startsWith(lexicalMBDir + "/");
+        let physicalMatch = false;
+        try {
+          const resolved = normalize(await realpath(absPath));
+          physicalMatch = resolved === mbReal || resolved.startsWith(mbReal + path.sep) || resolved.startsWith(mbReal + "/");
+        } catch {
+          try {
+            const parentResolved = normalize(await realpath(path.dirname(absPath)));
+            const targetName = path.basename(absPath);
+            const fullResolved = path.join(parentResolved, targetName);
+            physicalMatch = fullResolved === mbReal || fullResolved.startsWith(mbReal + path.sep) || fullResolved.startsWith(mbReal + "/");
+          } catch {}
+        }
+        return lexicalMatch || physicalMatch;
       };
       const isWriterAllowed = (sid) => {
         if (writerSessionIDs.has(sid))
@@ -778,36 +810,79 @@ ${triggers.join(`
 ` + `\u8BF7\u4F7F\u7528 delegate_task \u8C03\u7528 memory-bank-writer agent \u6765\u66F4\u65B0 Memory Bank\u3002
 ` + `\u793A\u4F8B: delegate_task(subagent_type="memory-bank-writer", load_skills=["memory-bank-writer"], prompt="\u66F4\u65B0...")`);
       };
-      if (tool === "Write" || tool === "Edit") {
-        const targetPath = output.args?.filePath ?? output.args?.path ?? output.args?.filename;
-        if (!targetPath)
-          return;
-        if (!isMemoryBankPath(targetPath))
-          return;
-        if (!targetPath.endsWith(".md")) {
-          blockWrite("only .md files allowed", { targetPath });
+      const extractPaths = (toolName, args) => {
+        const paths = [];
+        const pathArgs = ["filePath", "path", "filename", "file", "dest", "destination", "target"];
+        for (const arg of pathArgs) {
+          const val = args[arg];
+          if (typeof val === "string" && val.trim())
+            paths.push(val);
         }
-        if (isWriterAllowed(sessionID)) {
-          log.debug("Writer agent write allowed", { sessionID, tool, targetPath });
+        if (toolName === "multiedit" && Array.isArray(args.edits)) {
+          for (const edit of args.edits) {
+            if (typeof edit === "object" && edit !== null) {
+              const e = edit;
+              if (typeof e.path === "string")
+                paths.push(e.path);
+              if (typeof e.filePath === "string")
+                paths.push(e.filePath);
+            }
+          }
+        }
+        if (toolName === "apply_patch" || toolName === "patch") {
+          const patchText = args.patchText ?? args.patch ?? args.diff;
+          if (typeof patchText === "string") {
+            const gitDiffPattern = /^(?:\+\+\+|---)\s+[ab]\/(.+)$/gm;
+            let match;
+            while ((match = gitDiffPattern.exec(patchText)) !== null) {
+              if (match[1])
+                paths.push(match[1]);
+            }
+            const openCodePattern = /^\*\*\*\s+(?:Add|Update|Delete|Move to)\s+(?:File:\s*)?(.+)$/gm;
+            while ((match = openCodePattern.exec(patchText)) !== null) {
+              if (match[1])
+                paths.push(match[1].trim());
+            }
+          }
+        }
+        return [...new Set(paths)];
+      };
+      const fileWriteTools = ["write", "edit", "multiedit", "apply_patch", "patch"];
+      const toolLower = tool.toLowerCase();
+      if (fileWriteTools.includes(toolLower)) {
+        const targetPaths = extractPaths(toolLower, output.args || {});
+        if (targetPaths.length === 0)
           return;
+        for (const targetPath of targetPaths) {
+          if (!await isMemoryBankPath(targetPath))
+            continue;
+          if (!targetPath.toLowerCase().endsWith(".md")) {
+            blockWrite("only .md files allowed", { targetPath });
+          }
+          if (isWriterAllowed(sessionID)) {
+            log.debug("Writer agent write allowed", { sessionID, tool, targetPath });
+            return;
+          }
+          blockWrite("not writer agent", {
+            targetPath,
+            isSubAgent: !!sessionsById.get(sessionID)?.parentID,
+            agentName: agentBySessionID.get(sessionID)
+          });
         }
-        blockWrite("not writer agent", {
-          targetPath,
-          isSubAgent: !!sessionsById.get(sessionID)?.parentID,
-          agentName: agentBySessionID.get(sessionID)
-        });
       }
       if (tool.toLowerCase() === "bash") {
         const command = output.args?.command;
         if (!command || typeof command !== "string")
           return;
-        if (!/(?:^|[^A-Za-z0-9_-])memory-bank(?:$|[^A-Za-z0-9_-])/.test(command) && !command.startsWith("memory-bank"))
+        const cmdToCheck = isCaseInsensitiveFS ? command.toLowerCase() : command;
+        const mbPattern = isCaseInsensitiveFS ? /(?:^|[^a-z0-9_.-])memory-bank(?:[\/\\]|$|\s|['"])/ : /(?:^|[^A-Za-z0-9_.-])memory-bank(?:[\/\\]|$|\s|['"])/;
+        if (!mbPattern.test(cmdToCheck))
           return;
         const hasShellOperators = /[;&|]|\$\(|`/.test(command);
         if (!hasShellOperators) {
           const readOnlyPatterns = [
-            /^\s*(ls|cat|head|tail|less|more|grep|rg|ag|find|tree|wc|file|stat)\b/,
-            /^\s*git\s+(status|log|diff|show|blame)\b/
+            /^\s*(ls|cat|head|tail|less|more|grep|rg|ag|find|tree|wc|file|stat)\b/i,
+            /^\s*git\s+(status|log|diff|show|blame)\b/i
           ];
           if (readOnlyPatterns.some((p) => p.test(command)))
             return;
@@ -817,16 +892,16 @@ ${triggers.join(`
           />>/,
           /<</,
           /\|/,
-          /\btee\b/,
-          /\bsed\s+-i/,
-          /\bperl\s+-[ip]/,
-          /\bcp\b/,
-          /\bmv\b/,
-          /\brm\b/,
-          /\bmkdir\b/,
-          /\btouch\b/,
-          /\bgit\s+(add|rm|mv|apply|checkout|restore|reset|clean|stash|commit)\b/,
-          /\bpython\b.*\bopen\b/
+          /\btee\b/i,
+          /\bsed\s+-i/i,
+          /\bperl\s+-[ip]/i,
+          /\bcp\b/i,
+          /\bmv\b/i,
+          /\brm\b/i,
+          /\bmkdir\b/i,
+          /\btouch\b/i,
+          /\bgit\s+(add|rm|mv|apply|checkout|restore|reset|clean|stash|commit)\b/i,
+          /\bpython\b.*\bopen\b/i
         ];
         const isWriteOperation = writePatterns.some((p) => p.test(command));
         if (!isWriteOperation)

package/package.json

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