claude-mem-lite 2.83.0 → 2.83.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "2.83.0",
+      "version": "2.83.1",
       "source": "./",
       "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall"
     }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.83.0",
+  "version": "2.83.1",
   "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall",
   "author": {
     "name": "sdsrss"

package/hook.mjs

@@ -61,7 +61,7 @@ import { checkForUpdate } from './hook-update.mjs';
 import { handleLLMOptimize } from './hook-optimize.mjs';
 import { silentAutoAdopt, hasAutoAdoptMarker } from './adopt-cli.mjs';
 import { emitV270UpgradeBanner } from './lib/upgrade-banner.mjs';
-import { loadCiteBackForEpisode, buildUnsavedBugfixHint } from './lib/cite-back-hint.mjs';
+import { loadCiteBackForEpisode, buildUnsavedBugfixHint, countUnsavedBugfixShape, buildCiteRecallNudge as libBuildCiteRecallNudge } from './lib/cite-back-hint.mjs';
 // plugin-cache-guard.mjs loaded dynamically — pre-2.31.2 installs that auto-upgraded
 // from an older hook-update.mjs SOURCE_FILES (which did not list this module) would
 // crash on static import. Degrade gracefully to no-op when the module is absent.
@@ -565,7 +565,12 @@ async function handleStop() {
           // unchanged.
           try {
             const stats = computeCiteRecall(transcriptPath);
-            const payload = { ...stats, project, savedAt: Date.now() };
+            // B2 (v2.83.1): also persist the bugfix-shape nudge/save delta so
+            // the next SessionStart can surface "N unsaved bugfix-shape edits"
+            // alongside cite-recall. Same scan target (transcript already in OS
+            // cache); same persistence file; one extra line in buildCiteRecallNudge.
+            const bugfixStats = countUnsavedBugfixShape(transcriptPath);
+            const payload = { ...stats, ...bugfixStats, project, savedAt: Date.now() };
             const dest = join(RUNTIME_DIR, `cite-recall-${project.replace(/[^a-zA-Z0-9_.-]/g, '-').slice(0, 64)}.json`);
             writeFileSync(dest, JSON.stringify(payload), { mode: 0o600 });
           } catch (e) { debugCatch(e, 'handleStop-cite-recall-persist'); }
@@ -589,21 +594,10 @@ async function handleStop() {
 // fell below threshold. Empty string = no surface (insufficient signal, recall
 // already healthy, or feature opted-out via env). Default threshold 0.6,
 // min injected 5 — both env-overridable for ops tuning + tests.
+// Thin wrapper: lib/cite-back-hint.mjs owns the logic so it stays unit-tested.
+// Passing module-level RUNTIME_DIR keeps the call site identical to pre-v2.83.1.
 function buildCiteRecallNudge(project) {
-  if (process.env.CLAUDE_MEM_NO_CITE_NUDGE === '1') return '';
-  try {
-    const safe = project.replace(/[^a-zA-Z0-9_.-]/g, '-').slice(0, 64);
-    const path = join(RUNTIME_DIR, `cite-recall-${safe}.json`);
-    const raw = readFileSync(path, 'utf8');
-    const data = JSON.parse(raw);
-    const threshold = Number(process.env.CLAUDE_MEM_CITE_NUDGE_THRESHOLD) || 0.6;
-    const minInjected = Number(process.env.CLAUDE_MEM_CITE_NUDGE_MIN_INJECTED) || 5;
-    if (typeof data.injected !== 'number' || typeof data.ratio !== 'number') return '';
-    if (data.injected < minInjected) return '';
-    if (data.ratio >= threshold) return '';
-    const pct = Math.round(data.ratio * 100);
-    return `[mem] Last session cite-recall ${pct}% (${data.recalled}/${data.injected}) — when injected lessons (#NN lines) inform your action, cite #NN explicitly so the contract loop stays observable.`;
-  } catch { return ''; /* no prior file, parse error, or FS error — silent */ }
+  return libBuildCiteRecallNudge(project, RUNTIME_DIR);
 }
 
 // GC pre-recall cooldown files older than 24h. Pulled out of pre-tool-recall.js

package/lib/cite-back-hint.mjs

@@ -12,7 +12,7 @@
 // Legacy schema   (pre-v2.81):  { "<path>": <number> } — tolerated, never emits.
 
 import { basename, join } from 'path';
-import { readFileSync } from 'fs';
+import { readFileSync, existsSync } from 'fs';
 import { EDIT_TOOLS } from '../utils.mjs';
 
 const MAX_FILES = 2;
@@ -103,6 +103,103 @@ function cooldownPathFor(sessionId, runtimeDir) {
   return join(runtimeDir, `pre-recall-cooldown-${safe}.json`);
 }
 
+// ─── countUnsavedBugfixShape (B2, v2.83.1) ──────────────────────────────────
+// At Stop time, count this session's transcript for:
+//   • bugfix-shape hint emissions (buildUnsavedBugfixHint fired)
+//   • lesson/bugfix save signals (mem_save tool_use with type ∈ {bugfix, lesson}
+//     OR Bash `activity save --type lesson|bugfix`)
+//
+// Returns {nudged, saved, unsaved} where unsaved = max(0, nudged - saved).
+// SessionStart surfaces `unsaved` as a follow-on to the cite-recall nudge,
+// turning per-episode hints into cross-session pressure.
+const UNSAVED_BUGFIX_LITERAL = 'Unsaved bugfix-shape';
+const ACTIVITY_SAVE_LESSON_RE = /activity\s+save\s+--type\s+(lesson|bugfix)\b/;
+const MEM_SAVE_TOOL_NAMES = new Set([
+  'mem_save',
+  'mcp__claude_mem_lite__mem_save',
+  'mcp__plugin_claude-mem-lite_mem-lite__mem_save',
+]);
+
+export function countUnsavedBugfixShape(transcriptPath) {
+  const empty = { nudged: 0, saved: 0, unsaved: 0 };
+  if (!transcriptPath || !existsSync(transcriptPath)) return empty;
+  let raw;
+  try { raw = readFileSync(transcriptPath, 'utf8'); } catch { return empty; }
+
+  let nudged = 0;
+  let saved = 0;
+
+  for (const line of raw.split('\n')) {
+    if (!line.trim()) continue;
+    let entry;
+    try { entry = JSON.parse(line); } catch { continue; }
+
+    // Bugfix-shape hint emissions live in PostToolUse attachment.stdout.
+    if (entry.type === 'attachment') {
+      const stdout = entry.attachment?.stdout || '';
+      if (stdout.includes(UNSAVED_BUGFIX_LITERAL)) nudged++;
+      continue;
+    }
+
+    // Save signals live in assistant tool_use blocks.
+    if (entry.type === 'assistant' || entry.message?.role === 'assistant') {
+      const content = entry.message?.content;
+      if (!Array.isArray(content)) continue;
+      for (const block of content) {
+        if (block.type !== 'tool_use') continue;
+        if (MEM_SAVE_TOOL_NAMES.has(block.name)) {
+          const t = block.input?.type;
+          if (t === 'bugfix' || t === 'lesson') saved++;
+          continue;
+        }
+        if (block.name === 'Bash') {
+          const cmd = block.input?.command || '';
+          if (ACTIVITY_SAVE_LESSON_RE.test(cmd)) saved++;
+        }
+      }
+    }
+  }
+
+  return { nudged, saved, unsaved: Math.max(0, nudged - saved) };
+}
+
+// ─── buildCiteRecallNudge (extracted from hook.mjs for unit-testability) ────
+// Reads `runtime/cite-recall-<project>.json` (written by handleStop) and
+// builds the SessionStart nudge surface. Two independent gates compose:
+//   • cite-recall ratio gate: prior session's ratio < threshold (default 0.6)
+//     AND injected count >= floor (default 5)
+//   • B2 (v2.83.1) unsaved-bugfix gate: `unsaved > 0` (no min-volume floor —
+//     the bugfix-shape heuristic already requires ≥3 entries)
+// Either gate can fire independently. Both off → empty string (no surface).
+//
+// Env opt-outs:
+//   • CLAUDE_MEM_NO_CITE_NUDGE=1 — disables BOTH gates (full silence)
+//   • CLAUDE_MEM_CITE_NUDGE_THRESHOLD — ratio gate threshold (default 0.6)
+//   • CLAUDE_MEM_CITE_NUDGE_MIN_INJECTED — ratio gate min-volume (default 5)
+export function buildCiteRecallNudge(project, runtimeDir, env = process.env) {
+  if (env.CLAUDE_MEM_NO_CITE_NUDGE === '1') return '';
+  try {
+    const safe = project.replace(/[^a-zA-Z0-9_.-]/g, '-').slice(0, 64);
+    const path = join(runtimeDir, `cite-recall-${safe}.json`);
+    const raw = readFileSync(path, 'utf8');
+    const data = JSON.parse(raw);
+    const threshold = Number(env.CLAUDE_MEM_CITE_NUDGE_THRESHOLD) || 0.6;
+    const minInjected = Number(env.CLAUDE_MEM_CITE_NUDGE_MIN_INJECTED) || 5;
+    const lines = [];
+    if (typeof data.injected === 'number'
+      && typeof data.ratio === 'number'
+      && data.injected >= minInjected
+      && data.ratio < threshold) {
+      const pct = Math.round(data.ratio * 100);
+      lines.push(`[mem] Last session cite-recall ${pct}% (${data.recalled}/${data.injected}) — when injected lessons (#NN lines) inform your action, cite #NN explicitly so the contract loop stays observable.`);
+    }
+    if (typeof data.unsaved === 'number' && data.unsaved > 0) {
+      lines.push(`[mem] Last session: ${data.unsaved} unsaved bugfix-shape edit(s) — if any was a real fix, save now: /lesson --file <path> "<root cause + fix>"`);
+    }
+    return lines.join('\n');
+  } catch { return ''; }
+}
+
 export function loadCiteBackForEpisode(episode, runtimeDir) {
   if (!episode || !episode.sessionId || !runtimeDir) return null;
   let cooldown;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.83.0",
+  "version": "2.83.1",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "packageManager": "npm@10.9.2",