claude-mem-lite 3.17.0 → 3.18.0

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.17.0",
+      "version": "3.18.0",
       "source": "./",
       "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark)."
     }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.17.0",
+  "version": "3.18.0",
   "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark).",
   "author": {
     "name": "sdsrss"

package/lib/lesson-bridge.mjs

@@ -0,0 +1,39 @@
+// lib/lesson-bridge.mjs — pure prompt builder + fail-open Haiku bridge for the
+// comprehension-bridge forcing-function (CLAUDE_MEM_SALIENCE=bridge). Loaded by
+// scripts/pre-tool-recall.js via dynamic import ONLY when the flag is on, so the
+// fast hook never pulls the LLM stack by default (lesson #8447).
+import { callLLM } from '../hook-shared.mjs';
+
+const LESSON_MAX = 600;   // chars — a lesson_learned fits well under this
+const HUNK_MAX = 1200;    // chars — the change region; bounds Haiku input cost
+const CHECK_MAX = 200;    // chars — bounded injected payload
+
+export function buildBridgePrompt(lesson, hunk) {
+  const l = String(lesson || '').slice(0, LESSON_MAX);
+  const h = String(hunk || '').slice(0, HUNK_MAX);
+  return [
+    'A past lesson and the code change about to be made are below.',
+    'In ONE line, state the single concrete check the lesson forces on THIS change,',
+    'naming the actual symbol involved. If the lesson cannot apply, output exactly: N/A',
+    'Be specific. No preamble, no markdown.',
+    '',
+    `LESSON: ${l}`,
+    '',
+    'CHANGE:',
+    h,
+  ].join('\n');
+}
+
+// { ok:true, check } when the bridge produced a usable, applicable check;
+// { ok:false } on N/A / empty / error / timeout. NEVER throws — the caller
+// falls back to the baseline ACK_DIRECTIVE on { ok:false }.
+export async function bridgeLesson({ lesson, hunk, timeoutMs = 2500, _callLLM = callLLM }) {
+  try {
+    const raw = await _callLLM(buildBridgePrompt(lesson, hunk), timeoutMs);
+    const line = String(raw || '').trim().split('\n')[0].trim();
+    if (!line || /^n\s*\/?\s*a$/i.test(line)) return { ok: false };
+    return { ok: true, check: line.slice(0, CHECK_MAX) };
+  } catch {
+    return { ok: false };
+  }
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.17.0",
+  "version": "3.18.0",
   "description": "Persistent long-term memory for Claude Code via MCP — captures coding decisions, bugfixes, and context across sessions. Hybrid FTS5 + TF-IDF search with episode batching. Single SQLite DB, no external services. A lighter, lower-cost alternative to claude-mem (episode batching + a smaller model; cost savings are an internal estimate, not a measured benchmark).",
   "type": "module",
   "packageManager": "npm@10.9.2",
@@ -71,6 +71,7 @@
     "lib/reread-guard.mjs",
     "lib/metrics.mjs",
     "lib/lesson-idents.mjs",
+    "lib/lesson-bridge.mjs",
     "lib/binding-probe.mjs",
     "lib/proc-lock.mjs",
     "lib/atomic-write.mjs",

package/scripts/pre-tool-recall.js

@@ -42,6 +42,7 @@ const COOLDOWN_MS = 5 * 60 * 1000; // 5 minutes (used only for legacy fallback)
 const SALIENCE_LEGACY = process.env.CLAUDE_MEM_SALIENCE === 'legacy'
   || process.env.CLAUDE_MEM_SALIENCE === '0';
 const SALIENCE_BIND = process.env.CLAUDE_MEM_SALIENCE === 'bind';
+const SALIENCE_BRIDGE = process.env.CLAUDE_MEM_SALIENCE === 'bridge';
 const ACK_DIRECTIVE = "apply each lesson to this edit or rule it out — state '#NN applied' or '#NN n/a — <reason>' in your next user-facing message.";
 // v-bind salience forcing-function (#8771 audit: ack ≠ act). Instead of a cheap
 // '#NN applied / n/a' verdict, demand the model bind the lesson to the concrete
@@ -77,6 +78,30 @@ function cooldownPathFor(sessionId) {
   return join(RUNTIME_DIR, `pre-recall-cooldown-${safe}.json`);
 }
 
+// Comprehension-bridge (CLAUDE_MEM_SALIENCE=bridge): rewrite the top bound lesson
+// into a check naming a symbol in THIS change. Dynamic import keeps the LLM stack
+// out of the default fast path (#8447). Fail-open: null → caller uses ACK line.
+async function bridgeTopLesson(rows, changeText) {
+  if (!SALIENCE_BRIDGE || !changeText) return null;
+  const fake = process.env.CLAUDE_MEM_BRIDGE_FAKE;
+  let extractIdents, bridgeLesson;
+  try {
+    ({ extractIdents } = await import('../lib/lesson-idents.mjs'));
+    if (!fake) ({ bridgeLesson } = await import('../lib/lesson-bridge.mjs'));
+  } catch { return null; }
+  for (const r of rows) {
+    const lesson = r.lesson_learned;
+    if (!lesson) continue;
+    if (!extractIdents(lesson).some((id) => changeText.includes(id))) continue;
+    let res;
+    if (fake) res = /^n\s*\/?\s*a$/i.test(fake.trim()) ? { ok: false } : { ok: true, check: fake.trim().slice(0, 200) };
+    else res = await bridgeLesson({ lesson, hunk: changeText });
+    if (res.ok) return { id: r.id, check: res.check };
+    return null; // top bound lesson abstained → fall back to ACK, don't scan further
+  }
+  return null;
+}
+
 // ─── Helpers ────────────────────────────────────────────────────────────────
 
 function inferProject() {
@@ -181,11 +206,13 @@ try {
   let filePath;
   let sessionId;
   let toolName;
+  let toolInput;
   // isFullRead: a Read with no offset/limit reads the whole file. The reread
   // guard only flags full-vs-full re-reads, so paging never trips it.
   let isFullRead = true;
   try {
     const event = JSON.parse(input);
+    toolInput = event.tool_input;
     filePath = event.tool_input?.file_path;
     sessionId = event.session_id || null;
     toolName = event.tool_name || null;
@@ -442,7 +469,11 @@ try {
       // Read keeps the quiet form; its forcing-function fires at the later Edit
       // via the Read→Edit ack nudge above.
       if (!isRead && !SALIENCE_LEGACY) {
-        lines.push(`[mem] ⚠ Before this edit: ${ACTIVE_DIRECTIVE}`);
+        const changeText = [toolInput?.old_string, toolInput?.new_string, toolInput?.content]
+          .filter(Boolean).join('\n');
+        const bridged = await bridgeTopLesson(allRows, changeText);
+        if (bridged) lines.push(`[mem] ⚠ #${bridged.id} → this edit must: ${bridged.check}. Confirm your new code satisfies it.`);
+        else lines.push(`[mem] ⚠ Before this edit: ${ACTIVE_DIRECTIVE}`);
       }
     } else if (!isRead && process.env.CLAUDE_MEM_PRETOOL_NUDGE === '1') {
       // R-4: Edit/Write empty → short backfill reminder. OPT-IN (default off) as

package/source-files.mjs

@@ -66,6 +66,11 @@ export const SOURCE_FILES = [
   // are present in the pre-edit file (component 2). Imported ONLY by
   // scripts/pre-tool-recall.js; kept here for the same reason as file-intel.mjs.
   'lib/lesson-idents.mjs',
+  // comprehension-bridge forcing-function (CLAUDE_MEM_SALIENCE=bridge): rewrites
+  // a recalled lesson into a check bound to the change hunk. Dynamic-imported by
+  // scripts/pre-tool-recall.js ONLY under the flag, but must still ship so the
+  // hook can resolve it at runtime when a user opts in.
+  'lib/lesson-bridge.mjs',
   // v2.71.x: better-sqlite3 ABI probe + auto-rebuild. Shared by install.mjs
   // (post-`npm install` verify) and scripts/launch.mjs (pre-server-launch
   // self-heal after Node ABI changes). Missing from manifest → auto-update