claude-mem-lite 2.47.0 → 2.48.0

package/.claude-plugin/marketplace.json

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

package/hook-context.mjs

@@ -2,13 +2,13 @@
 // Handles adaptive time windows, token-budgeted selection, and legacy CLAUDE.md cleanup.
 
 import { basename, join } from 'path';
-import { readFileSync, writeFileSync, renameSync, unlinkSync } from 'fs';
+import { existsSync, readFileSync, writeFileSync, renameSync, unlinkSync } from 'fs';
 import {
-  estimateTokens, truncate, typeIcon, fmtTime,
+  estimateTokens, truncate, typeIcon, fmtTime, inferProject,
   debugLog, debugCatch,
   DECAY_HALF_LIFE_BY_TYPE, DEFAULT_DECAY_HALF_LIFE_MS, notLowSignalTitleClause,
 } from './utils.mjs';
-import { STALE_SESSION_MS, FALLBACK_OBS_WINDOW_MS, effectiveQuiet } from './hook-shared.mjs';
+import { STALE_SESSION_MS, FALLBACK_OBS_WINDOW_MS, RUNTIME_DIR, effectiveQuiet } from './hook-shared.mjs';
 import { extractUnfinishedSummary } from './hook-handoff.mjs';
 
 /**
@@ -176,9 +176,26 @@ export function selectWithTokenBudget(db, project, budget = 2000) {
  * at the seam. Uses atomic tmp+rename write.
  */
 export function cleanupClaudeMdLegacyBlock() {
+  // v2.48 P2-4: idempotent marker. First run (whether it finds a block or not,
+  // whether CLAUDE.md exists or not) drops a project-scoped marker in RUNTIME_DIR.
+  // Subsequent SessionStarts short-circuit here — no CLAUDE.md stat, no regex scan.
+  // Recovery path if a user manually re-adds a legacy block: delete the marker
+  // file (`~/.claude-mem-lite/runtime/.legacy-claude-md-cleaned-<project>`) and
+  // the next SessionStart will sweep again.
+  const markerPath = join(RUNTIME_DIR, `.legacy-claude-md-cleaned-${inferProject()}`);
+  if (existsSync(markerPath)) return;
+
   const claudeMdPath = join(inferProjectDir(), 'CLAUDE.md');
   let content;
-  try { content = readFileSync(claudeMdPath, 'utf8'); } catch { return; }
+  try { content = readFileSync(claudeMdPath, 'utf8'); } catch {
+    // CLAUDE.md missing — still drop the marker so we don't re-stat every session
+    try { writeFileSync(markerPath, String(Date.now())); } catch {}
+    return;
+  }
+
+  // Helper: drop the marker regardless of exit path (found / not found / write failed).
+  // Kept inline so the early-return sites below stay readable.
+  const dropMarker = () => { try { writeFileSync(markerPath, String(Date.now())); } catch {} };
 
   const startTag = '<claude-mem-context>';
   const endTag = '</claude-mem-context>';
@@ -187,7 +204,10 @@ export function cleanupClaudeMdLegacyBlock() {
   // (e.g. inside a code block in architecture notes) are not accidentally swept.
   const startIdx = content.lastIndexOf(startTag);
   const endIdx = content.lastIndexOf(endTag);
-  if (startIdx === -1 || endIdx === -1 || startIdx >= endIdx) return;
+  if (startIdx === -1 || endIdx === -1 || startIdx >= endIdx) {
+    dropMarker();
+    return;
+  }
 
   // Extend forward to swallow a trailing newline so we don't leave a stranded blank line.
   let removeEnd = endIdx + endTag.length;
@@ -213,15 +233,17 @@ export function cleanupClaudeMdLegacyBlock() {
   // Collapse any ≥3 consecutive newlines to two, then ensure exactly one trailing newline.
   const normalized = cleaned.replace(/\n{3,}/g, '\n\n').replace(/\s*$/, '\n');
 
-  if (normalized === content) return;
+  if (normalized === content) { dropMarker(); return; }
 
   const tmp = claudeMdPath + '.mem-tmp';
   try {
     writeFileSync(tmp, normalized);
     renameSync(tmp, claudeMdPath);
+    dropMarker();
   } catch (e) {
     try { unlinkSync(tmp); } catch {}
     debugLog('ERROR', 'cleanupClaudeMdLegacyBlock', `CLAUDE.md write failed: ${e.message}`);
+    // Intentionally do NOT drop the marker on write failure — retry next session.
   }
 }
 

package/install.mjs

@@ -289,6 +289,17 @@ async function install() {
     const manifestSrc = join(PROJECT_DIR, 'registry', 'preinstalled.json');
     if (existsSync(manifestSrc)) copyFileSync(manifestSrc, join(registryDir, 'preinstalled.json'));
     ok('Source files copied to ~/.claude-mem-lite/');
+
+    // v2.48 P1-4: prune stale top-level .mjs + 0-byte .db files left behind by
+    // prior upgrades (e.g. dispatch.mjs removed in v2.20.0, zero-byte mem.db /
+    // memory.db / registry.db from pre-consolidation installs). Subdirs +
+    // symlinks + non-empty DBs are always preserved.
+    try {
+      const pruned = pruneStaleInstallFiles(DATA_DIR, SOURCE_FILES);
+      if (pruned.length > 0) {
+        ok(`Pruned ${pruned.length} stale file(s): ${pruned.map(p => p.split('/').pop()).join(', ')}`);
+      }
+    } catch (e) { /* prune is best-effort — never block install */ void e; }
   }
 
   // 2. npm install (skip for --dev: node_modules is symlinked)
@@ -1295,6 +1306,57 @@ function hasMemHooksConfigured(settings) {
   );
 }
 
+/**
+ * v2.48 P1-4: prune top-level stale files left behind by removed-module upgrades.
+ *
+ * Strict whitelist: only removes files under `dataDir` (no recursion) that match
+ *   - `*.mjs` whose basename is NOT in SOURCE_FILES (comparing against both the
+ *     bare entry and any `subdir/basename` entry flattened to its basename — the
+ *     prune intentionally skips subdir files; see below)
+ *   - 0-byte `.db` files that are NOT in the protected-db allow-list
+ *
+ * Protections (never touched):
+ *   - subdirectories (managed/, runtime/, scripts/, lib/, cli/, commands/, server/, node_modules/, .claude-plugin/, registry/, etc.)
+ *   - non-empty `.db` files — real data risk, always preserved
+ *   - WAL/SHM (`*-wal`, `*-shm`) transients
+ *   - files not ending in `.mjs` or `.db`
+ *   - the two canonical DBs (`claude-mem-lite.db`, `resource-registry.db`) even when 0-byte (fresh-install transient state)
+ *
+ * @param {string} dataDir Absolute path, typically `~/.claude-mem-lite`
+ * @param {string[]} sourceFiles SOURCE_FILES manifest
+ * @returns {string[]} Absolute paths of files that were deleted (ordered by readdir)
+ */
+export function pruneStaleInstallFiles(dataDir, sourceFiles) {
+  if (!existsSync(dataDir)) return [];
+  // Flatten manifest to just top-level basenames. SOURCE_FILES contains entries
+  // like 'lib/activity.mjs' — those belong to a subdir and prune never touches
+  // subdirs anyway. For top-level entries ('server.mjs'), basename === entry.
+  const topLevelAllowed = new Set(
+    sourceFiles
+      .filter(f => !f.includes('/'))
+      .map(f => f)
+  );
+  const PROTECTED_DBS = new Set(['claude-mem-lite.db', 'resource-registry.db']);
+  const removed = [];
+  let entries;
+  try { entries = readdirSync(dataDir); } catch { return removed; }
+  for (const name of entries) {
+    const full = join(dataDir, name);
+    let st;
+    try { st = lstatSync(full); } catch { continue; }
+    // Skip directories and symlinks (dev mode uses symlinks; treat as intentional).
+    if (!st.isFile()) continue;
+    if (name.endsWith('.mjs') && !topLevelAllowed.has(name)) {
+      try { unlinkSync(full); removed.push(full); } catch { /* best-effort */ }
+      continue;
+    }
+    if (name.endsWith('.db') && !PROTECTED_DBS.has(name) && st.size === 0) {
+      try { unlinkSync(full); removed.push(full); } catch { /* best-effort */ }
+    }
+  }
+  return removed;
+}
+
 export function clearPluginDisabledMarkerForDirectInstall(settings) {
   if (settings?.enabledPlugins?.[PLUGIN_KEY] !== false) return false;
   delete settings.enabledPlugins[PLUGIN_KEY];

package/package.json

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