memory-eternal 0.20.0 → 0.22.0

package/lib/db.js

@@ -213,6 +213,18 @@ function initTables(db) {
 
     CREATE INDEX IF NOT EXISTS idx_card_updates_card ON card_updates(card_id);
 
+    -- 审核制修改：待审副本 → 原卡。agent 只能提交副本，通过后由 setCardStatus 覆盖原卡。
+    -- 单独建表（而不是给两张卡表加列）：旧库打开时 CREATE TABLE IF NOT EXISTS 即可补上，无需迁移。
+    CREATE TABLE IF NOT EXISTS card_revisions (
+      revision_path TEXT PRIMARY KEY,
+      target_path TEXT NOT NULL,
+      target_id INTEGER,
+      created_at TEXT DEFAULT (datetime('now')),
+      updated_at TEXT DEFAULT (datetime('now'))
+    );
+
+    CREATE INDEX IF NOT EXISTS idx_card_revisions_target ON card_revisions(target_path);
+
     CREATE TABLE IF NOT EXISTS feedback (
       id INTEGER PRIMARY KEY AUTOINCREMENT,
       query TEXT NOT NULL,

package/lib/mcp.js

@@ -5,7 +5,8 @@
 // （stdout 是协议通道，被 console.log 污染会破坏握手）。
 //
 // 支持的协议版本：2025-06-18 / 2025-03-26 / 2024-11-05（协商取请求版本）。
-// 工具：memory_recall / memory_capture / memory_stats / memory_audit_list。
+// 工具：memory_recall / memory_capture / memory_stats / memory_audit_list / memory_revise。
+// memory_revise 只**提交**修改副本进审核（不会直接改库、不能批准），与 issue #17 的审核边界一致。
 //
 // 审核边界（issue #17）：MCP 侧**只读**。新增的 memory_audit_list 只列待审/驳回，
 // 不提供任何 approve/reject —— 否则任何接了 MCP 的智能体都能批准自己刚写进去的卡，
@@ -14,7 +15,7 @@
 // 启动：node bin/dsh-memory.mjs mcp   （或 node lib/mcp.js）
 
 import readline from 'node:readline'
-import { search, overview, stats, ensureVault, listCards, countCards } from './vault.js'
+import { search, overview, stats, ensureVault, listCards, countCards, reviseCard } from './vault.js'
 import { runStandaloneCapture, defaultVaultDir } from './capture-run.js'
 import { recordLedger } from './ledger.js'
 
@@ -72,6 +73,25 @@ export const MCP_TOOLS = [
       },
     },
   },
+  {
+    name: 'memory_revise',
+    description:
+      '提交「审核制修改」：把一张已批准的知识卡复制成待审副本（可改标题/标签/正文），进入审核中心。' +
+      '人工批准后才会覆盖原卡；agent 不能直接改已入库内容，也不能自己批准。' +
+      '用于修正过期/错误的知识 —— 比再写一张新卡更干净（审核界面会标成「修改」，通过即覆盖）。',
+    inputSchema: {
+      type: 'object',
+      properties: {
+        path: { type: 'string', description: '要修改的原卡 path（memory_recall 结果里的「路径：…」）' },
+        title: { type: 'string', description: '新标题（不改就不传）' },
+        tags: { type: 'array', items: { type: 'string' }, description: '新标签（不改就不传）' },
+        body: { type: 'string', description: '新正文（不改就不传）' },
+        reason: { type: 'string', description: '修改原因，会显示在审核中心' },
+        source: { type: 'string', description: '来源标记，如 claude-code / codex' },
+      },
+      required: ['path'],
+    },
+  },
 ]
 
 function toolResult(text, isError = false) {
@@ -108,6 +128,25 @@ export async function callTool(name, args, vaultRoot, meta = {}) {
       if (!out.ok && out.action === 'failed') return toolResult(`沉淀失败：${out.reason}`, true)
       return toolResult(JSON.stringify(out))
     }
+    case 'memory_revise': {
+      const rel = String(args?.path || '').trim()
+      if (!rel) return toolResult('（未提供要修改的卡片 path）')
+      try {
+        const src = String(args?.source || meta.submittedBy || 'mcp')
+        const out = await reviseCard(vaultRoot, {
+          target: rel,
+          title: args?.title === undefined ? undefined : String(args.title),
+          tags: Array.isArray(args?.tags) ? args.tags : undefined,
+          body: args?.body === undefined ? undefined : String(args.body),
+          reason: String(args?.reason || ''),
+          source: src,
+          submittedBy: src,
+        })
+        return toolResult('已提交修改，待人工审核通过后覆盖原卡。\n修改副本：' + out.path + '\n原卡：' + out.target + (out.reused ? '\n（已存在待审修改，本次就地更新，未新增条目）' : ''))
+      } catch (e) {
+        return toolResult('提交修改失败：' + ((e && e.message) || e), true)
+      }
+    }
     case 'memory_audit_list': {
       await ensureVault(vaultRoot)
       const wanted = String(args?.status || 'pending').toLowerCase()

package/lib/node-bin.js

@@ -11,8 +11,9 @@
 //
 // 本模块把「找出一个真正能跑 node 脚本的子进程入口」收敛到一处：
 //   1. MEMORY_ETERNAL_NODE 显式指定（逃生口，便于测试与非常规宿主）；
-//   2. 非 Electron 宿主：process.execPath 本身就是 node，直接用；
-//   3. Electron 宿主：优先从 PATH 找真正的 node；
+//   2. 非 Electron 宿主：**优先从 PATH 找 node**（issue #56：brew/nvm/volta 在 PATH 上的入口
+//      是稳定 shim，而 process.execPath 是带版本号的真实路径），execPath 降为兜底；
+//   3. Electron 宿主：同样优先从 PATH 找真正的 node；
 //   4. 都找不到时退回 process.execPath，并由 childEnv() 打开
 //      ELECTRON_RUN_AS_NODE，让 Electron 二进制以纯 node 模式运行。
 
@@ -22,8 +23,8 @@ import path from 'node:path'
 let cachedBinary = null
 let cachedSource = null
 
-/** 在 PATH 中查找真正的 node 可执行文件；找不到返回 null。 */
-function findNodeOnPath() {
+/** 在 PATH 中查找真正的 node 可执行文件；找不到返回 null。（导出便于测试断言 PATH 优先） */
+export function findNodeOnPath() {
   const names = process.platform === 'win32' ? ['node.exe', 'node.cmd', 'node'] : ['node', 'node.exe']
   const dirs = String(process.env.PATH || '').split(path.delimiter)
   for (const dir of dirs) {
@@ -72,17 +73,21 @@ export function nodeBinary() {
   }
 
   if (!process.versions.electron) {
-    // 非 Electron：正常情况下 process.execPath 就是 node 本身；但**长命进程**（watchdog）的
-    // execPath 指向的是它启动时那份 node —— brew 升级后可能已被删除（#44）。此时退回 PATH。
-    if (exists(process.execPath)) {
-      cachedBinary = process.execPath
-      cachedSource = 'execPath'
-      return cachedBinary
-    }
+    // issue #56：非 Electron 下**必须 PATH 优先**。process.execPath 是「这一次装的那份 node」的
+    // 真实路径（macOS Homebrew：/opt/homebrew/Cellar/node/26.13.0_2/bin/node），它会被写进
+    // ~/.claude.json 等外部客户端的 MCP 条目；brew upgrade node 一删旧目录，这些条目就 ENOENT。
+    // PATH 上的入口（/opt/homebrew/bin/node、~/.volta/bin/node、nvm 的 shim）才是稳定的。
+    // 同时它决定了 align 的 currentNodePath —— 写入侧与校验侧一起收敛到 shim，才不会互相打脸。
+    // execPath 仍保留为兜底（PATH 上没有 node 时行为与 #44 之前一致）。
     const viaPath = findNodeOnPath()
     if (viaPath) {
       cachedBinary = viaPath
-      cachedSource = 'path-fallback'
+      cachedSource = 'path'
+      return cachedBinary
+    }
+    if (exists(process.execPath)) {
+      cachedBinary = process.execPath
+      cachedSource = 'execPath'
       return cachedBinary
     }
     cachedBinary = process.execPath