claude-mem-lite 3.16.0 → 3.16.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.16.0",
+      "version": "3.16.1",
       "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.16.0",
+  "version": "3.16.1",
   "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/hook-optimize.mjs

@@ -203,10 +203,22 @@ search_aliases: 2-6 alternative search terms (include CJK if applicable).`;
 const NORMALIZE_GATE_FILE = join(RUNTIME_DIR, 'last-normalize.json');
 const NORMALIZE_INTERVAL_MS = 7 * 86400000; // 7 days
 
+// Pure gate decision (no IO) — exported for testing. Fail-OPEN on a
+// malformed-but-valid-JSON gate: a missing/non-numeric `epoch` makes
+// `now - epoch` NaN, and `NaN >= INTERVAL` is false — which would PERMANENTLY
+// block normalize with no recovery, contradicting the catch-branch's fail-open
+// intent (a corrupt file that fails JSON.parse already returns true). A future
+// epoch (clock skew / NTP correction) is equally suspect → run.
+export function _normalizeGateOpen(last, now) {
+  const epoch = last?.epoch;
+  if (typeof epoch !== 'number' || !Number.isFinite(epoch) || epoch > now) return true;
+  return now - epoch >= NORMALIZE_INTERVAL_MS;
+}
+
 export function shouldRunNormalize() {
   try {
     const last = JSON.parse(readFileSync(NORMALIZE_GATE_FILE, 'utf8'));
-    return Date.now() - last.epoch >= NORMALIZE_INTERVAL_MS;
+    return _normalizeGateOpen(last, Date.now());
   } catch {
     return true;
   }

package/mem-cli.mjs

@@ -1636,12 +1636,26 @@ function cmdRestore(db, argv) {
   const trimmed = raw.trim();
   if (!trimmed) { out('[mem] Empty file — nothing to restore.'); return; }
   let rows;
-  try {
-    rows = trimmed[0] === '['
-      ? JSON.parse(trimmed)
-      : trimmed.split('\n').filter(l => l.trim()).map(l => JSON.parse(l));
-  } catch (e) { fail(`[mem] "${file}" is not valid export JSON/JSONL: ${e.message}`); return; }
-  if (!Array.isArray(rows) || rows.length === 0) { out('[mem] No observations in file.'); return; }
+  let parseFailures = 0;
+  if (trimmed[0] === '[') {
+    // Whole-array JSON is a single document — partial parse is impossible, so
+    // a syntax error rejects the file (unchanged behavior).
+    try { rows = JSON.parse(trimmed); }
+    catch (e) { fail(`[mem] "${file}" is not valid export JSON/JSONL: ${e.message}`); return; }
+  } else {
+    // JSONL: tolerate per-line syntax errors so one corrupt line in a large
+    // backup doesn't discard every valid row. Parse failures fold into the
+    // malformed tally below (the loop already skips valid-JSON-but-wrong-shape
+    // rows the same way) — a backup tool must recover what it can.
+    rows = [];
+    for (const line of trimmed.split('\n')) {
+      if (!line.trim()) continue;
+      try { rows.push(JSON.parse(line)); }
+      catch { parseFailures++; }
+    }
+  }
+  if (!Array.isArray(rows) || (rows.length === 0 && parseFailures === 0)) { out('[mem] No observations in file.'); return; }
+  if (rows.length === 0) { fail(`[mem] "${file}": all ${parseFailures} line(s) failed to parse as JSONL.`); return; }
 
   const projOverride = flags.project ? resolveProject(db, flags.project) : null;
   const dryRun = flags['dry-run'] === true || flags['dry-run'] === 'true';
@@ -1693,7 +1707,12 @@ function cmdRestore(db, argv) {
       if (process.env.CLAUDE_MEM_DEBUG) process.stderr.write(`[mem] restore row failed: ${e.message}\n`);
     }
   }
-  out(`[mem] Restore${dryRun ? ' (dry-run)' : ''}: ${restored} restored, ${skipped} duplicate(s) skipped, ${malformed} malformed/failed from ${rows.length} row(s).`);
+  // Fold JSONL per-line syntax failures into the malformed tally and the
+  // denominator so the report reflects every non-blank input line, not just the
+  // ones that parsed.
+  const totalMalformed = malformed + parseFailures;
+  const totalLines = rows.length + parseFailures;
+  out(`[mem] Restore${dryRun ? ' (dry-run)' : ''}: ${restored} restored, ${skipped} duplicate(s) skipped, ${totalMalformed} malformed/failed from ${totalLines} row(s).`);
 }
 
 // ─── Compress ────────────────────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.16.0",
+  "version": "3.16.1",
   "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",