memory-bank-skill 5.5.0 → 5.5.2

package/dist/cli.js

@@ -7,7 +7,7 @@ import { homedir } from "os";
 import { join, dirname } from "path";
 import { createHash } from "crypto";
 import { fileURLToPath } from "url";
-var VERSION = "5.4.0";
+var VERSION = "5.5.2";
 var colors = {
   reset: "\x1B[0m",
   bold: "\x1B[1m",
@@ -148,7 +148,8 @@ async function installSkillFiles(packageRoot, undoStack, manifestFiles) {
 }
 async function installPluginToConfig(undoStack) {
   const configPath = join(homedir(), ".config", "opencode", "opencode.json");
-  const pluginPackage = "memory-bank-skill";
+  const pluginPackageWithVersion = `memory-bank-skill@${VERSION}`;
+  const pluginPackagePrefix = "memory-bank-skill";
   let config = {};
   let existed = false;
   let modified = false;
@@ -165,7 +166,7 @@ async function installPluginToConfig(undoStack) {
 
 ` + JSON.stringify({
         permission: { skill: "allow" },
-        plugin: [pluginPackage]
+        plugin: [pluginPackageWithVersion]
       }, null, 2));
     }
   }
@@ -187,9 +188,30 @@ async function installPluginToConfig(undoStack) {
     changes.push("Removed old file:// plugin reference");
     modified = true;
   }
-  if (!config.plugin.includes(pluginPackage)) {
-    config.plugin.push(pluginPackage);
-    changes.push(`Added plugin: ${pluginPackage}`);
+  const matchingIndices = [];
+  config.plugin.forEach((p, i) => {
+    if (p === pluginPackagePrefix || p.startsWith(`${pluginPackagePrefix}@`)) {
+      matchingIndices.push(i);
+    }
+  });
+  if (matchingIndices.length > 1) {
+    for (let i = matchingIndices.length - 1;i > 0; i--) {
+      const removed = config.plugin.splice(matchingIndices[i], 1)[0];
+      changes.push(`Removed duplicate: ${removed}`);
+    }
+    modified = true;
+  }
+  const existingIndex = config.plugin.findIndex((p) => p === pluginPackagePrefix || p.startsWith(`${pluginPackagePrefix}@`));
+  if (existingIndex !== -1) {
+    const existing = config.plugin[existingIndex];
+    if (existing !== pluginPackageWithVersion) {
+      config.plugin[existingIndex] = pluginPackageWithVersion;
+      changes.push(`Updated plugin: ${existing} \u2192 ${pluginPackageWithVersion}`);
+      modified = true;
+    }
+  } else {
+    config.plugin.push(pluginPackageWithVersion);
+    changes.push(`Added plugin: ${pluginPackageWithVersion}`);
     modified = true;
   }
   if (modified) {
@@ -213,6 +235,60 @@ async function writeManifest(manifestFiles, undoStack) {
   await atomicWriteFile(manifestPath, JSON.stringify(manifest, null, 2) + `
 `, undoStack);
 }
+async function checkAndCleanOpenCodeCache() {
+  const cacheDir = join(homedir(), ".cache", "opencode");
+  const cachePackageJson = join(cacheDir, "package.json");
+  const pkgCacheDir = join(cacheDir, "node_modules", "memory-bank-skill");
+  const cacheNodeModulesPkg = join(pkgCacheDir, "package.json");
+  if (!await exists(cachePackageJson)) {
+    return { cleaned: false };
+  }
+  try {
+    const cacheDepContent = await fs.readFile(cachePackageJson, "utf-8");
+    const cacheDep = JSON.parse(cacheDepContent);
+    const recordedVersion = cacheDep.dependencies?.["memory-bank-skill"];
+    if (!recordedVersion) {
+      return { cleaned: false };
+    }
+    let actualVersion = null;
+    if (await exists(cacheNodeModulesPkg)) {
+      try {
+        const actualPkg = JSON.parse(await fs.readFile(cacheNodeModulesPkg, "utf-8"));
+        actualVersion = actualPkg.version;
+      } catch {}
+    }
+    const needsClean = recordedVersion === "latest" || recordedVersion !== VERSION || actualVersion && actualVersion !== VERSION;
+    if (needsClean) {
+      const realCacheDir = await fs.realpath(cacheDir).catch(() => null);
+      const expectedPrefix = join(homedir(), ".cache", "opencode");
+      if (!realCacheDir || !realCacheDir.startsWith(expectedPrefix)) {
+        return { cleaned: false };
+      }
+      if (await exists(pkgCacheDir)) {
+        await fs.rm(pkgCacheDir, { recursive: true, force: true });
+      }
+      const bunLockb = join(cacheDir, "bun.lockb");
+      if (await exists(bunLockb)) {
+        await fs.rm(bunLockb, { force: true });
+      }
+      const newDeps = { ...cacheDep.dependencies };
+      delete newDeps["memory-bank-skill"];
+      cacheDep.dependencies = newDeps;
+      const tmpPath = `${cachePackageJson}.tmp`;
+      await fs.writeFile(tmpPath, JSON.stringify(cacheDep, null, 2) + `
+`);
+      await fs.rename(tmpPath, cachePackageJson);
+      const reason = recordedVersion === "latest" ? `"latest" \u2192 ${VERSION}` : `${recordedVersion} \u2192 ${VERSION}`;
+      return {
+        cleaned: true,
+        message: `Cleaned stale OpenCode cache (${reason})`
+      };
+    }
+  } catch {
+    return { cleaned: false };
+  }
+  return { cleaned: false };
+}
 async function install() {
   log(`
 ${colors.bold}Memory Bank Skill Installer v${VERSION}${colors.reset}
@@ -222,6 +298,11 @@ ${colors.bold}Memory Bank Skill Installer v${VERSION}${colors.reset}
   const manifestFiles = [];
   const results = [];
   try {
+    const cacheResult = await checkAndCleanOpenCodeCache();
+    if (cacheResult.cleaned) {
+      logSuccess(cacheResult.message || "Cleaned OpenCode cache");
+      log("");
+    }
     logStep(1, 2, "Installing skill files...");
     const r1 = await installSkillFiles(packageRoot, undoStack, manifestFiles);
     logDetail(r1.details || "");
@@ -267,21 +348,51 @@ ${colors.bold}Memory Bank Skill Doctor v${VERSION}${colors.reset}
   }
   const configPath = join(homedir(), ".config", "opencode", "opencode.json");
   let pluginConfigured = false;
+  let pluginVersion = "";
   if (await exists(configPath)) {
     try {
       const content = await fs.readFile(configPath, "utf-8");
       const config = JSON.parse(content);
-      pluginConfigured = Array.isArray(config.plugin) && config.plugin.includes("memory-bank-skill");
+      if (Array.isArray(config.plugin)) {
+        const entry = config.plugin.find((p) => p === "memory-bank-skill" || p.startsWith("memory-bank-skill@"));
+        if (entry) {
+          pluginConfigured = true;
+          pluginVersion = entry;
+        }
+      }
     } catch {}
   }
   if (pluginConfigured) {
     log(`${colors.green}\u2713${colors.reset} Plugin configured in opencode.json`);
-    logDetail("memory-bank-skill");
+    logDetail(pluginVersion);
+    if (!pluginVersion.includes("@")) {
+      log(`${colors.yellow}\u26A0${colors.reset} Consider using pinned version: memory-bank-skill@${VERSION}`);
+    }
   } else {
     log(`${colors.red}\u2717${colors.reset} Plugin not configured`);
-    logDetail("Add 'memory-bank-skill' to plugin array in opencode.json");
+    logDetail(`Add 'memory-bank-skill@${VERSION}' to plugin array in opencode.json`);
     allOk = false;
   }
+  const cacheDir = join(homedir(), ".cache", "opencode");
+  const cachePackageJson = join(cacheDir, "package.json");
+  if (await exists(cachePackageJson)) {
+    try {
+      const content = await fs.readFile(cachePackageJson, "utf-8");
+      const cacheDep = JSON.parse(content);
+      const recordedVersion = cacheDep.dependencies?.["memory-bank-skill"];
+      if (recordedVersion) {
+        const cacheNodeModules = join(cacheDir, "node_modules", "memory-bank-skill", "package.json");
+        if (await exists(cacheNodeModules)) {
+          const actualPkg = JSON.parse(await fs.readFile(cacheNodeModules, "utf-8"));
+          if (recordedVersion !== actualPkg.version && recordedVersion !== "latest") {
+            log(`${colors.yellow}\u26A0${colors.reset} OpenCode cache version mismatch`);
+            logDetail(`Recorded: ${recordedVersion}, Actual: ${actualPkg.version}`);
+            logDetail(`Run 'bunx memory-bank-skill install' to fix`);
+          }
+        }
+      }
+    } catch {}
+  }
   const manifestPath = join(homedir(), ".config", "opencode", "skill", "memory-bank", ".manifest.json");
   if (await exists(manifestPath)) {
     try {

package/dist/plugin.js

@@ -121,11 +121,19 @@ function truncateToBudget(text, 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);
+    if (DEBUG)
+      console.error(`[MB-DEBUG] read ${rel} done, hasContent=${!!content}, elapsed=${Date.now() - fileStart}ms`);
     if (!content)
       continue;
     const trimmed = content.trim();
@@ -136,6 +144,8 @@ async function buildMemoryBankContextWithMeta(projectRoot) {
 ${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)
     return null;
   const fileList = files.map((f) => f.relPath.replace("memory-bank/", "")).join(", ");
@@ -562,14 +572,22 @@ ${triggers.join(`
   }
   return {
     "experimental.chat.system.transform": async (_input, output) => {
-      if (output.system.some((s) => s.includes(SENTINEL_OPEN)))
-        return;
-      const ctx = await buildMemoryBankContext(projectRoot);
-      if (ctx) {
-        output.system.push(ctx);
-        return;
-      }
-      const initInstruction = `${SENTINEL_OPEN}
+      const hookStart = Date.now();
+      log.info("[HOOK] system.transform START");
+      try {
+        if (output.system.some((s) => s.includes(SENTINEL_OPEN))) {
+          log.info("[HOOK] system.transform SKIP (sentinel exists)", { elapsed: Date.now() - hookStart });
+          return;
+        }
+        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
@@ -577,17 +595,29 @@ ${triggers.join(`
 ` + `**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(initInstruction);
+        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 });
+      }
     },
     "experimental.session.compacting": async (_input, output) => {
-      if (output.context.some((s) => s.includes(SENTINEL_OPEN)))
-        return;
-      const ctx = await buildMemoryBankContext(projectRoot);
-      if (ctx) {
-        output.context.push(ctx);
-        return;
-      }
-      const initInstruction = `${SENTINEL_OPEN}
+      const hookStart = Date.now();
+      log.info("[HOOK] session.compacting START");
+      try {
+        if (output.context.some((s) => s.includes(SENTINEL_OPEN))) {
+          log.info("[HOOK] session.compacting SKIP (sentinel exists)", { elapsed: Date.now() - hookStart });
+          return;
+        }
+        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
@@ -595,7 +625,11 @@ ${triggers.join(`
 ` + `**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(initInstruction);
+        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 });
+      }
     },
     event: async ({ event }) => {
       try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "memory-bank-skill",
-  "version": "5.5.0",
+  "version": "5.5.2",
   "description": "Memory Bank - 项目记忆系统，让 AI 助手在每次对话中都能快速理解项目上下文",
   "type": "module",
   "main": "dist/plugin.js",
@@ -40,5 +40,8 @@
   },
   "publishConfig": {
     "registry": "https://registry.npmjs.org/"
+  },
+  "devDependencies": {
+    "memory-bank-skill": "5.5.1"
   }
 }