claude-mem-lite 2.71.0 → 2.71.2

package/.claude-plugin/marketplace.json

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

package/cli.mjs

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-const CLI_COMMANDS = new Set(['search', 'recent', 'recall', 'get', 'timeline', 'save', 'stats', 'context', 'browse', 'delete', 'update', 'export', 'compress', 'maintain', 'optimize', 'fts-check', 'registry', 'import', 'enrich', 'activity', 'adopt', 'unadopt', 'memdir-audit', 'defer', 'help']);
+const CLI_COMMANDS = new Set(['search', 'recent', 'recall', 'get', 'timeline', 'save', 'stats', 'context', 'browse', 'delete', 'update', 'export', 'compress', 'maintain', 'optimize', 'fts-check', 'registry', 'import', 'import-jsonl', 'enrich', 'activity', 'adopt', 'unadopt', 'memdir-audit', 'defer', 'help']);
 const INSTALL_COMMANDS = new Set(['install', 'uninstall', 'status', 'doctor', 'cleanup', 'cleanup-hooks', 'self-update', 'release']);
 
 const cmd = process.argv[2];

package/lib/import-jsonl.mjs

@@ -34,12 +34,21 @@ function parseLine(line) {
   try { return JSON.parse(line); } catch { return null; }
 }
 
+// Distinct mem-internal id derived from the CC session UUID. The schema
+// trigger `sdk_sessions_id_mix_check_*` aborts when memory_session_id ==
+// content_session_id and both look like a CC UUID (length 36 + hyphen
+// pattern); writing the raw UUID into both columns would reproduce the
+// v2.33.1 mix bug. The `import-` prefix makes the origin obvious in audits.
+function memId(sessionId) {
+  return `import-${sessionId}`;
+}
+
 function ensureSession(db, sessionId, project, ts) {
   db.prepare(`
     INSERT OR IGNORE INTO sdk_sessions
       (content_session_id, memory_session_id, project, started_at, started_at_epoch, status)
     VALUES (?, ?, ?, ?, ?, 'completed')
-  `).run(sessionId, sessionId, project, ts, Date.parse(ts) || Date.now());
+  `).run(sessionId, memId(sessionId), project, ts, Date.parse(ts) || Date.now());
 }
 
 function importPrompt(db, ev, project, seenPrompts) {
@@ -109,7 +118,7 @@ function importToolPair(db, toolUse, toolResult, project) {
       (memory_session_id, project, text, type, title, subtitle, narrative, concepts, facts, files_read, files_modified, importance, created_at, created_at_epoch)
     VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)
   `).run(
-    sessionId, project, safe.text, type, safe.title, safe.subtitle,
+    memId(sessionId), project, safe.text, type, safe.title, safe.subtitle,
     safe.narrative, safe.concepts, safe.facts,
     JSON.stringify(filesRead), JSON.stringify(filesModified),
     1, ts, Date.parse(ts) || Date.now(),
@@ -171,7 +180,9 @@ export async function importJsonl(db, path, { project }) {
     const toolName = useEv.name || 'unknown';
     const titlePreview = `${toolName}: ${(useEv.input?.command || useEv.input?.file_path || '').slice(0, 80)}`;
     const ts = useEv.timestamp || new Date().toISOString();
-    const crossKey = dedupKey([sessionId, `existing:${titlePreview}:${ts}`]);
+    // Match the storage convention from importToolPair (memId-prefixed) so
+    // the seenObs entries seeded from the DB can be matched on a re-run.
+    const crossKey = dedupKey([memId(sessionId), `existing:${titlePreview}:${ts}`]);
     if (seenObs.has(crossKey)) return false;
 
     return importToolPair(db, useEv, resultEv, project);
@@ -183,7 +194,31 @@ export async function importJsonl(db, path, { project }) {
       const ev = parseLine(line);
       if (!ev) { skipped++; continue; }
       if (ev.type === 'user') {
-        if (importPrompt(db, ev, project, seenPrompts)) prompts++; else skipped++;
+        // Real Claude Code transcripts wrap tool_result inside a user-typed
+        // event's message.content array (alongside the rare text part). The
+        // top-level {"type":"tool_result"} shape only appears in our test
+        // fixtures. Consume any embedded tool_result parts here; only fall
+        // through to importPrompt when the event is an actual user prompt.
+        const content = ev?.message?.content;
+        let consumedAsToolResult = false;
+        if (Array.isArray(content)) {
+          for (const part of content) {
+            if (part?.type === 'tool_result' && part.tool_use_id) {
+              consumedAsToolResult = true;
+              const useEv = pendingToolUse.get(part.tool_use_id);
+              if (useEv) {
+                const synth = { content: part.content, tool_use_id: part.tool_use_id, timestamp: ev.timestamp };
+                if (tryImportToolPair(useEv, synth)) observations++;
+                pendingToolUse.delete(part.tool_use_id);
+              } else {
+                skipped++;
+              }
+            }
+          }
+        }
+        if (!consumedAsToolResult) {
+          if (importPrompt(db, ev, project, seenPrompts)) prompts++; else skipped++;
+        }
       } else if (ev.type === 'assistant' && Array.isArray(ev.message?.content)) {
         for (const part of ev.message.content) {
           if (part.type === 'tool_use') {

package/lib/stats-quality.mjs

@@ -30,10 +30,10 @@ export function computeQualityStats(db, { project, days }) {
   const windowRow = db.prepare(`
     SELECT
       COUNT(*) as total,
-      SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END) as with_lesson,
-      SUM(CASE WHEN ${lowSignalIsMatchExpr} THEN 1 ELSE 0 END) as low_signal,
-      SUM(CASE WHEN type = 'bugfix' THEN 1 ELSE 0 END) as bugfix_total,
-      SUM(CASE WHEN type = 'bugfix' AND ${unresolvedNarrativeExpr} THEN 1 ELSE 0 END) as bugfix_unresolved
+      COALESCE(SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END), 0) as with_lesson,
+      COALESCE(SUM(CASE WHEN ${lowSignalIsMatchExpr} THEN 1 ELSE 0 END), 0) as low_signal,
+      COALESCE(SUM(CASE WHEN type = 'bugfix' THEN 1 ELSE 0 END), 0) as bugfix_total,
+      COALESCE(SUM(CASE WHEN type = 'bugfix' AND ${unresolvedNarrativeExpr} THEN 1 ELSE 0 END), 0) as bugfix_unresolved
     FROM observations
     WHERE created_at_epoch >= ? ${projectFilter}
   `).get(cutoff, ...baseParams);
@@ -41,8 +41,8 @@ export function computeQualityStats(db, { project, days }) {
   const allTimeRow = db.prepare(`
     SELECT
       COUNT(*) as total,
-      SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END) as with_lesson,
-      SUM(CASE WHEN ${lowSignalIsMatchExpr} THEN 1 ELSE 0 END) as low_signal
+      COALESCE(SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END), 0) as with_lesson,
+      COALESCE(SUM(CASE WHEN ${lowSignalIsMatchExpr} THEN 1 ELSE 0 END), 0) as low_signal
     FROM observations
     WHERE 1=1 ${projectFilter}
   `).get(...baseParams);
@@ -51,8 +51,8 @@ export function computeQualityStats(db, { project, days }) {
     SELECT
       type,
       COUNT(*) as total,
-      SUM(CASE WHEN COALESCE(access_count, 0) > 0 THEN 1 ELSE 0 END) as accessed,
-      SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END) as with_lesson
+      COALESCE(SUM(CASE WHEN COALESCE(access_count, 0) > 0 THEN 1 ELSE 0 END), 0) as accessed,
+      COALESCE(SUM(CASE WHEN lesson_learned IS NOT NULL AND lesson_learned != '' THEN 1 ELSE 0 END), 0) as with_lesson
     FROM observations
     WHERE created_at_epoch >= ? ${projectFilter}
     GROUP BY type
@@ -75,8 +75,8 @@ export function computeQualityStats(db, { project, days }) {
   // age > 37d, so a sudden write surge inflates this until the cohort ages out.
   const purgeRow = db.prepare(`
     SELECT
-      SUM(CASE WHEN compressed_into IS NOT NULL AND compressed_into != 0 THEN 1 ELSE 0 END) as compressed,
-      SUM(CASE WHEN compressed_into = ${COMPRESSED_PENDING_PURGE} THEN 1 ELSE 0 END) as pending_purge
+      COALESCE(SUM(CASE WHEN compressed_into IS NOT NULL AND compressed_into != 0 THEN 1 ELSE 0 END), 0) as compressed,
+      COALESCE(SUM(CASE WHEN compressed_into = ${COMPRESSED_PENDING_PURGE} THEN 1 ELSE 0 END), 0) as pending_purge
     FROM observations
     WHERE 1=1 ${projectFilter}
   `).get(...baseParams);

package/mem-cli.mjs

@@ -1832,12 +1832,12 @@ function cmdMaintain(db, args) {
     const stats = db.prepare(`
       SELECT
         COUNT(*) as total,
-        SUM(CASE WHEN COALESCE(importance, 1) = 1 AND COALESCE(access_count, 0) = 0
-                      AND created_at_epoch < ? THEN 1 ELSE 0 END) as stale,
-        SUM(CASE WHEN (title IS NULL OR title = '') AND (narrative IS NULL OR narrative = '')
-                 THEN 1 ELSE 0 END) as broken,
-        SUM(CASE WHEN COALESCE(access_count, 0) > 3 AND COALESCE(importance, 1) < 3
-                 THEN 1 ELSE 0 END) as boostable
+        COALESCE(SUM(CASE WHEN COALESCE(importance, 1) = 1 AND COALESCE(access_count, 0) = 0
+                      AND created_at_epoch < ? THEN 1 ELSE 0 END), 0) as stale,
+        COALESCE(SUM(CASE WHEN (title IS NULL OR title = '') AND (narrative IS NULL OR narrative = '')
+                 THEN 1 ELSE 0 END), 0) as broken,
+        COALESCE(SUM(CASE WHEN COALESCE(access_count, 0) > 3 AND COALESCE(importance, 1) < 3
+                 THEN 1 ELSE 0 END), 0) as boostable
       FROM observations
       WHERE COALESCE(compressed_into, 0) = 0 ${projectFilter}
     `).get(staleAge, ...baseParams);

package/package.json

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

package/schema.mjs

@@ -181,6 +181,21 @@ export function initSchema(db) {
     if (e.message?.startsWith('DB schema is v')) throw e;
   }
 
+  // Concurrent-init guard: serialize schema setup against peer processes via
+  // BEGIN IMMEDIATE (busy_timeout=3000 from ensureDb makes peers wait). Required
+  // because the sdk_sessions_id_mix_check_{ai,au} migration uses DROP+CREATE
+  // without IF NOT EXISTS to update the trigger body, which races at cold-start.
+  // Re-check schema_version under the lock — a peer may have completed init
+  // while we were blocked. Connection close auto-rollbacks if body throws.
+  db.exec('BEGIN IMMEDIATE');
+  try {
+    const underlock = db.prepare('SELECT version FROM schema_version LIMIT 1').get();
+    if (underlock && underlock.version === CURRENT_SCHEMA_VERSION) {
+      db.exec('COMMIT');
+      return db;
+    }
+  } catch { /* table absent — proceed */ }
+
   // Create core tables
   db.exec(CORE_SCHEMA);
 
@@ -264,7 +279,6 @@ export function initSchema(db) {
     });
     dedupAndIndex();
   }
-  db.pragma('foreign_keys = ON');
 
   // Performance indexes
   db.exec(`CREATE INDEX IF NOT EXISTS idx_obs_epoch_project ON observations(created_at_epoch DESC, project)`);
@@ -580,6 +594,10 @@ export function initSchema(db) {
     db.prepare('INSERT INTO schema_version (version) VALUES (?)').run(CURRENT_SCHEMA_VERSION);
   })();
 
+  db.exec('COMMIT');
+  // PRAGMA foreign_keys must run OUTSIDE the transaction (no-op inside).
+  db.pragma('foreign_keys = ON');
+
   return db;
 }
 

package/scripts/pre-tool-recall.js

@@ -9,10 +9,12 @@ import { basename, join } from 'path';
 import { homedir } from 'os';
 import { buildNotLowSignalSql } from '../lib/low-signal-patterns.mjs';
 
-// CLAUDE_MEM_DB_PATH / CLAUDE_MEM_RUNTIME_DIR env overrides allow tests and debug tools to
-// point the hook at an isolated DB + cooldown dir without touching the user's real state.
-const DB_PATH = process.env.CLAUDE_MEM_DB_PATH || join(homedir(), '.claude-mem-lite', 'claude-mem-lite.db');
-const RUNTIME_DIR = process.env.CLAUDE_MEM_RUNTIME_DIR || join(homedir(), '.claude-mem-lite', 'runtime');
+// CLAUDE_MEM_DIR matches schema.mjs / main CLI — one env var sandboxes the
+// whole system. CLAUDE_MEM_DB_PATH / CLAUDE_MEM_RUNTIME_DIR remain as
+// per-component overrides for tests that mix isolated + real paths.
+const DATA_DIR = process.env.CLAUDE_MEM_DIR || join(homedir(), '.claude-mem-lite');
+const DB_PATH = process.env.CLAUDE_MEM_DB_PATH || join(DATA_DIR, 'claude-mem-lite.db');
+const RUNTIME_DIR = process.env.CLAUDE_MEM_RUNTIME_DIR || join(DATA_DIR, 'runtime');
 // v2.33.1: cooldown path is session-scoped so same-file-twice within one
 // session never re-injects (was: global file, 5-min window). Cross-session:
 // fresh file, fresh nudges — this is intended. No session_id → fall back to

package/server.mjs

@@ -1329,12 +1329,12 @@ server.registerTool(
       const stats = db.prepare(`
         SELECT
           COUNT(*) as total,
-          SUM(CASE WHEN COALESCE(importance, 1) = 1 AND COALESCE(access_count, 0) = 0
-                        AND created_at_epoch < ? THEN 1 ELSE 0 END) as stale,
-          SUM(CASE WHEN (title IS NULL OR title = '') AND (narrative IS NULL OR narrative = '')
-                   THEN 1 ELSE 0 END) as broken,
-          SUM(CASE WHEN COALESCE(access_count, 0) > 3 AND COALESCE(importance, 1) < 3
-                   THEN 1 ELSE 0 END) as boostable
+          COALESCE(SUM(CASE WHEN COALESCE(importance, 1) = 1 AND COALESCE(access_count, 0) = 0
+                        AND created_at_epoch < ? THEN 1 ELSE 0 END), 0) as stale,
+          COALESCE(SUM(CASE WHEN (title IS NULL OR title = '') AND (narrative IS NULL OR narrative = '')
+                   THEN 1 ELSE 0 END), 0) as broken,
+          COALESCE(SUM(CASE WHEN COALESCE(access_count, 0) > 3 AND COALESCE(importance, 1) < 3
+                   THEN 1 ELSE 0 END), 0) as boostable
         FROM observations
         WHERE COALESCE(compressed_into, 0) = 0 ${projectFilter}
       `).get(staleAge, ...baseParams);