claude-mem-lite 2.71.1 → 2.71.2

package/.claude-plugin/marketplace.json

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

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.1",
+  "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);