claude-mem-lite 2.53.1 → 2.54.0

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "2.53.1",
+      "version": "2.54.0",
       "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.53.1",
+  "version": "2.54.0",
   "description": "Lightweight persistent memory system for Claude Code — FTS5 search, episode batching, error-triggered recall",
   "author": {
     "name": "sdsrss"

package/hook-llm.mjs

@@ -677,9 +677,13 @@ search_aliases: 2-6 alternative search terms someone might use to find this memo
         files: episode.files,
         filesRead: episode.filesRead || [],
         // v2.33.1: when lesson is low-signal, don't trust Haiku's importance
-        // inflation for noise-prone types. rule-based floor still applies so
-        // error-in-test (→3) / config-change (→2) keep their floor.
-        importance: isLessonLowSignal && (parsed.type === 'change' || parsed.type === 'discovery')
+        // inflation. v2.54.0: extended from {change, discovery} to all types
+        // except `decision` after audit (2026-04-30) showed bugfix lesson
+        // coverage 11.2% / refactor hit-rate 18.1% — Haiku marks bugfix/refactor
+        // imp=2-3 even when lesson is null after retry. Keep `decision` exempt:
+        // it's rare (39 obs / 94.9% hit-rate) and the retry path already gave
+        // it a second chance; a no-lesson decision is still a worthwhile signal.
+        importance: isLessonLowSignal && parsed.type !== 'decision'
           ? Math.min(ruleImportance, 1)
           : Math.max(ruleImportance, clampImportance(parsed.importance)),
         lessonLearned,

package/hook-optimize.mjs

@@ -709,7 +709,14 @@ export async function handleLLMOptimize() {
   }
 
   try {
-    const results = await optimizeRun(db);
+    // v2.54.0: auto-maintain default scope is 'wide'. Narrow scope (the prior
+    // default) only matches fully-degraded rows (no concepts AND no facts AND
+    // no lesson AND no aliases) — production diagnostic 2026-04-30 found only
+    // 56 obs ever optimized after months of daily auto-maintain runs. Wide
+    // targets bugfix/refactor/feature/decision rows with substantive narrative
+    // but missing lesson_learned, which is exactly the audit's 11.2% coverage
+    // gap. CLI `mem optimize` keeps narrow as default for explicit invocations.
+    const results = await optimizeRun(db, { reenrichScope: 'wide' });
     const parts = [];
     if (results.reenrich?.processed) parts.push(`re-enriched: ${results.reenrich.processed}`);
     if (results.normalize?.processed) parts.push(`normalized: ${results.normalize.processed}`);

package/lib/low-signal-patterns.mjs

@@ -155,19 +155,26 @@ export function isNoiseObservation(obs, env = process.env) {
   const lesson = obs.lessonLearned ?? obs.lesson_learned;
   if (lesson && String(lesson).trim() && String(lesson).trim().toLowerCase() !== 'none') return false;
 
-  if ((obs.importance ?? 1) >= 2) return false;
-
-  if (Array.isArray(obs.facts) &&
-      obs.facts.filter(f => typeof f === 'string' && f.trim().length > 0).length >= 1) {
-    return false;
-  }
+  const hasFacts = Array.isArray(obs.facts) &&
+    obs.facts.filter(f => typeof f === 'string' && f.trim().length > 0).length >= 1;
+  if (hasFacts) return false;
 
   const narrative = (obs.narrative || '').trim();
-  if (narrative.length >= 40 &&
-      !/^Error[: ]/i.test(narrative) &&
-      !_isLikelyToolOutputPassthrough(narrative)) {
-    return false;
-  }
+  const isPassthrough = _isLikelyToolOutputPassthrough(narrative);
+  const isStderrShape = /^Error[: ]/i.test(narrative);
+
+  // v2.54.0: raw tool-output passthrough is always noise regardless of importance.
+  // Rule-based importance (computeRuleImportance) can hit 2-3 from filename
+  // heuristics (test/schema/migration) even when narrative is just
+  // "cmd → ERROR: stderr" — 30d audit found 64 'Error: X' titles surviving via
+  // imp=2 escape with raw stderr narratives. Per #8152 paired-gate model: this
+  // is the drop counterpart to capNoiseImportance's demote — both must check
+  // the same passthrough signal.
+  if (isPassthrough || isStderrShape) return true;
+
+  if ((obs.importance ?? 1) >= 2) return false;
+
+  if (narrative.length >= 40) return false;
 
   return true;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.53.1",
+  "version": "2.54.0",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "engines": {

package/scripts/launch-preflight.mjs

@@ -34,6 +34,12 @@ export function detectMissingImports(installRoot) {
     return ['server.mjs'];
   }
 
+  // Strip line + block comments so example strings in docblocks (e.g. the very
+  // patterns this regex enumerates) can't false-fire as "missing files".
+  src = src
+    .replace(/\/\*[\s\S]*?\*\//g, '')
+    .replace(/(^|\s)\/\/[^\n]*/g, '$1');
+
   const missing = new Set();
   for (const re of [FROM_RE, IMPORT_RE]) {
     re.lastIndex = 0;

package/scripts/launch.mjs

@@ -12,12 +12,22 @@ const ROOT = process.env.CLAUDE_PLUGIN_ROOT || join(__dirname, '..');
 
 if (!existsSync(join(ROOT, 'node_modules', 'better-sqlite3'))) {
   process.stderr.write('[claude-mem-lite] Installing dependencies...\n');
-  execSync('npm install --omit=dev', {
-    cwd: ROOT,
-    stdio: ['ignore', 'pipe', 'inherit'], // stdout piped (discard), stderr inherit
-    timeout: 120_000,
-  });
-  process.stderr.write('[claude-mem-lite] Dependencies installed\n');
+  try {
+    execSync('npm install --omit=dev', {
+      cwd: ROOT,
+      stdio: ['ignore', 'pipe', 'inherit'], // stdout piped (discard), stderr inherit
+      timeout: 120_000,
+    });
+    process.stderr.write('[claude-mem-lite] Dependencies installed\n');
+  } catch (e) {
+    // Plugin-cache / multi-user / disk-full installs can fail here. Without this
+    // catch the user sees a Node stack trace; with it they get an actionable line.
+    const detail = e.message?.split('\n')[0] || e.code || 'unknown error';
+    process.stderr.write(`[claude-mem-lite] npm install failed in ${ROOT} — ${detail}\n`);
+    process.stderr.write(`[claude-mem-lite] Likely cause: read-only directory, disk full, or network blocked.\n`);
+    process.stderr.write(`[claude-mem-lite] Repair: cd "${ROOT}" && npm install --omit=dev\n`);
+    process.exit(1);
+  }
 }
 
 // Verify MCP SDK is importable (exports mapping intact).