claude-mem-lite 3.57.0 → 3.58.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.57.0",
+      "version": "3.58.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.57.0",
+  "version": "3.58.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/adopt-cli.mjs

@@ -22,6 +22,7 @@ import {
 } from './memdir.mjs';
 import {
   writeManaged, removeManaged, isAdopted as claudeMdIsAdopted,
+  hasResidue as claudeMdHasResidue,
   needsRefresh, migrateLegacyMemoryDir, hasLegacyMemdirSentinel,
   claudeMdPath, detailDocPath,
 } from './claudemd.mjs';
@@ -311,8 +312,8 @@ function statusAll() {
 
   const known = listKnownProjectDirs();
   let adoptedCount = 0;
-  for (const dir of known) if (claudeMdIsAdopted(dir, PLUGIN_SLUG)) adoptedCount++;
-  log(`[adopt --status] known projects (~/.claude.json): ${known.length} scanned, ${adoptedCount} with a CLAUDE.md managed block.`);
+  for (const dir of known) if (claudeMdHasResidue(dir, PLUGIN_SLUG)) adoptedCount++;
+  log(`[adopt --status] known projects (~/.claude.json): ${known.length} scanned, ${adoptedCount} with a CLAUDE.md managed block or partial residue (detail doc/state).`);
   if (adoptedCount > 0) log('[adopt --status] run `claude-mem-lite unadopt --all` to remove every CLAUDE.md block.');
 
   const pluginRoot = process.env.CLAUDE_PLUGIN_ROOT ? 'set' : 'unset';
@@ -347,16 +348,20 @@ function unadoptAll(args) {
 
   // 1. New scheme: scrub CLAUDE.md managed blocks across known project paths.
   const projectDirs = listKnownProjectDirs();
-  let blocks = 0;
+  let blocks = 0, partial = 0;
   for (const dir of projectDirs) {
-    if (!claudeMdIsAdopted(dir, PLUGIN_SLUG)) continue;
+    // hasResidue, not isAdopted: the sweep must also catch PARTIAL residue
+    // (block without detail doc, or an orphaned doc/state sidecar) —
+    // isAdopted's block-AND-doc gate skipped those projects forever.
+    if (!claudeMdHasResidue(dir, PLUGIN_SLUG)) continue;
     if (dryRun) {
-      log(`[unadopt --all --dry-run] ${dir} → would-remove CLAUDE.md block + detail doc`);
+      log(`[unadopt --all --dry-run] ${dir} → would-remove plugin residue (CLAUDE.md block and/or detail doc/state)`);
       blocks++;
       continue;
     }
     const r = removeManaged(dir, PLUGIN_SLUG);
     if (r.action === 'removed') { log(`[unadopt --all] ${dir} → removed`); blocks++; }
+    else { log(`[unadopt --all] ${dir} → cleaned partial residue (detail doc/state, no block)`); partial++; }
   }
 
   // 2. Legacy memory-dir cleanup across every memdir (foreign-content guarded).
@@ -372,7 +377,8 @@ function unadoptAll(args) {
   }
 
   log('');
-  log(`[unadopt --all] ${dryRun ? 'would remove' : 'removed'} ${blocks} CLAUDE.md block(s) across ${projectDirs.length} known project(s); ${legacy} legacy memory-dir sentinel(s) ${dryRun ? 'pending' : 'cleaned'}.`);
+  const partialNote = partial > 0 ? ` (+${partial} partial-residue cleanup(s))` : '';
+  log(`[unadopt --all] ${dryRun ? 'would remove' : 'removed'} ${blocks} CLAUDE.md block(s)${partialNote} across ${projectDirs.length} known project(s); ${legacy} legacy memory-dir sentinel(s) ${dryRun ? 'pending' : 'cleaned'}.`);
   if (projectDirs.length === 0) {
     log('[unadopt --all] no known projects found in ~/.claude.json — if a project was adopted but never opened in Claude Code, run `claude-mem-lite unadopt` from inside it.');
   }
@@ -389,7 +395,7 @@ export function cmdUnadopt(args = []) {
 
   const cwd = detectCwd();
   if (dryRun) {
-    const blockState = claudeMdIsAdopted(cwd, PLUGIN_SLUG) ? 'would-remove CLAUDE.md block + detail doc' : 'no CLAUDE.md block';
+    const blockState = claudeMdHasResidue(cwd, PLUGIN_SLUG) ? 'would-remove CLAUDE.md block + detail doc' : 'no CLAUDE.md block';
     const legacy = hasLegacyMemdirSentinel(cwd, PLUGIN_SLUG) ? 'would-clean legacy memory-dir sentinel' : 'no legacy residue';
     log(`[unadopt --dry-run] ${cwd}`);
     log(`  ${blockState}`);

package/claudemd.mjs

@@ -98,6 +98,20 @@ export function isAdopted(cwd, slug) {
   return blk.body !== null && existsSync(detailDocPath(cwd, slug));
 }
 
+/**
+ * Any trace of adoption that unadopt should clean: managed block OR detail doc
+ * OR state sidecar. Deliberately weaker than isAdopted (whose AND lets a
+ * half-written adopt self-heal on the next SessionStart): the unadopt sweep
+ * gated on isAdopted skipped partial residue forever — e.g. a user deleted the
+ * detail doc but the CLAUDE.md block remained, and `unadopt --all` never
+ * removed it. removeManaged cleans all three pieces, so sweep on any of them.
+ */
+export function hasResidue(cwd, slug) {
+  return readBlock(cwd, slug).body !== null
+    || existsSync(detailDocPath(cwd, slug))
+    || existsSync(stateFilePath(cwd, slug));
+}
+
 /**
  * Whether the installed block/doc has drifted from the shipped content — i.e.
  * a version bump or a template edit means we should refresh. Returns true when

package/hook-shared.mjs

@@ -6,7 +6,7 @@ import { randomUUID } from 'crypto';
 import { join } from 'path';
 import { existsSync, readFileSync, writeFileSync, mkdirSync, renameSync, readdirSync, statSync, unlinkSync, chmodSync } from 'fs';
 import { inferProject, debugCatch } from './utils.mjs';
-import { ensureDb, DB_DIR } from './schema.mjs';
+import { ensureDbWithWalRecovery, DB_DIR } from './schema.mjs';
 import { getClaudePath as getClaudePathShared, resolveModel as resolveModelShared, flattenForCLI as _flattenForCLI, detectMode as detectLLMMode, callHaiku } from './haiku-client.mjs';
 // Phase D: invited-memory sentinel detection. memdir.mjs/claudemd.mjs only pull in
 // fs/path/os/crypto; adopt-content.mjs is pure strings. No circular deps —
@@ -154,7 +154,9 @@ export function createSessionId() {
 
 export function openDb() {
   try {
-    return ensureDb();
+    // WAL-corruption self-heal (was server.mjs-only): without it, hooks stayed
+    // silently dead (null DB) on a corrupt WAL until the next MCP server start.
+    return ensureDbWithWalRecovery();
   } catch {
     return null;
   }

package/hook-update.mjs

@@ -635,6 +635,32 @@ function smokeInstalledRelease(targetDir) {
       const p = join(targetDir, entry);
       if (existsSync(p)) execSync(`${q(process.execPath)} --check ${q(p)}`, { timeout: 10000, stdio: 'ignore' });
     }
+    // `cli.mjs help` exits without opening the DB, so it cannot see a
+    // present-but-unusable better-sqlite3 binding: npm >= 12 blocks
+    // install/lifecycle scripts by default, so the staging `npm install`
+    // above exits 0 with the native .node never compiled (a Node major bump
+    // strands a stale ABI the same way). Direct installs register server.mjs
+    // without the launch.mjs probe, so this gate is their only check. Probe
+    // in a child process (execSync so the unit-test mock intercepts, and so
+    // the running old-version process's require cache can't mask it); on
+    // failure rebuild with scripts enabled for just this dep — plain-rebuild
+    // fallback for older npm — then re-probe. A still-broken binding throws
+    // out of the try, smoke fails, and the caller rolls back to the old
+    // (working) install.
+    if (existsSync(join(targetDir, 'node_modules', 'better-sqlite3'))) {
+      const probeSrc = 'const{createRequire}=require("node:module");const D=createRequire(process.argv[1])("better-sqlite3");new D(":memory:").close();';
+      const probeCmd = `${q(process.execPath)} -e ${q(probeSrc)} ${q(join(targetDir, 'package.json'))}`;
+      try {
+        execSync(probeCmd, { timeout: 20000, stdio: 'ignore' });
+      } catch {
+        try {
+          execSync('npm rebuild better-sqlite3 --dangerously-allow-all-scripts', { cwd: targetDir, timeout: 120000, stdio: 'ignore' });
+        } catch {
+          execSync('npm rebuild better-sqlite3', { cwd: targetDir, timeout: 120000, stdio: 'ignore' });
+        }
+        execSync(probeCmd, { timeout: 20000, stdio: 'ignore' });
+      }
+    }
     return true;
   } catch (e) {
     debugLog('WARN', 'hook-update', `post-install smoke failed (rolling back): ${e.message}`);

package/install.mjs

@@ -466,7 +466,7 @@ if (IS_DEV) {
     ok(`better-sqlite3: ${verify.action}`);
   } else {
     fail(`better-sqlite3 binding unusable after rebuild: ${verify.error}`);
-    log('Try manually: cd ' + INSTALL_DIR + ' && npm rebuild better-sqlite3 --build-from-source');
+    log('Try manually: cd ' + INSTALL_DIR + ' && npm rebuild better-sqlite3 --dangerously-allow-all-scripts');
     process.exit(1);
   }
 }

package/mem-cli.mjs

@@ -3,7 +3,7 @@
 // No MCP SDK or heavy deps — only imports schema.mjs and utils.mjs
 
 import { homedir } from 'os';
-import { ensureDb, DB_PATH, DB_DIR, REGISTRY_DB_PATH } from './schema.mjs';
+import { ensureDbWithWalRecovery, DB_PATH, DB_DIR, REGISTRY_DB_PATH } from './schema.mjs';
 import { truncate, typeIcon, inferProject, scrubSecrets } from './utils.mjs';
 import { resolveProject } from './project-utils.mjs';
 import { TIER_CASE_SQL, tierSqlParams } from './tier.mjs';
@@ -3156,14 +3156,16 @@ export async function run(argv) {
   }
 
   // adopt / unadopt do pure filesystem work on ~/.claude/projects/<encoded>/memory/ —
-  // no DB needed. Route them before ensureDb() so an unbootable DB doesn't block.
+  // no DB needed. Route them before the DB open so an unbootable DB doesn't block.
   if (cmd === 'adopt') { cmdAdopt(cmdArgs); return; }
   if (cmd === 'unadopt') { cmdUnadopt(cmdArgs); return; }
   if (cmd === 'memdir-audit') { cmdMemdirAudit(cmdArgs); return; }
 
   let db;
   try {
-    db = ensureDb();
+    // Corruption-gated WAL recovery (shared with server/hooks) — a corrupt WAL
+    // previously threw here with no auto-repair until the next MCP start.
+    db = ensureDbWithWalRecovery({ warn: (m) => process.stderr.write(`[mem] ${m}\n`) });
   } catch (e) {
     out(`[mem] Error: Cannot open database: ${e.message}`);
     out(`[mem] DB path: ${DB_PATH}`);