claude-mem-lite 3.47.0 → 3.49.0

package/server.mjs

@@ -12,19 +12,21 @@ import { reRankWithContext, autoBoostIfNeeded, runIdleCleanup, buildServerInstru
 import { searchObservationsHybrid } from './search-engine.mjs';
 import { deepSearch, resolveDeepMode, shouldEscalateToDeep, autoDeepLlmReady } from './deep-search.mjs';
 import { selectCompressionCandidates, groupByProjectWeek, compressGroup } from './lib/compress-core.mjs';
-import { buildNotLowSignalSql } from './lib/low-signal-patterns.mjs';
 import { resolveAnchorToken, formatAnchorError, resolveQueryAnchor, fetchRecentTimeline, fetchTimelineWindow } from './lib/timeline-core.mjs';
 import { buildSearchFtsQuery, parseDateBounds, parseDuration, coreRunSearchPipeline } from './lib/search-core.mjs';
 import {
   cleanupBroken, decayAndMarkIdle, boostAccessed, demotePinned, mergeDuplicates,
   recoverOrphanedChildren, recoverBuriedLessons, sweepDeferredWorkOrphans,
   purgeStale, purgeStalePreview, findDuplicates, maintenanceStats, rebuildVectors, vacuum,
-  recoverChildrenOf, hardDeleteCandidateCount,
+  hardDeleteCandidateCount,
   OP_CAP, STALE_AGE_MS,
 } from './lib/maintain-core.mjs';
 import { snapshotDb } from './lib/db-backup.mjs';
+import { deleteObservations } from './lib/delete-core.mjs';
 import { effectiveQuiet, RUNTIME_DIR } from './hook-shared.mjs';
 import { TIER_CASE_SQL, tierSqlParams } from './tier.mjs';
+import { computeStatsFeed } from './lib/stats-core.mjs';
+import { buildLessonNudge } from './lib/save-nudge.mjs';
 import { formatObsFieldValue } from './cli/common.mjs';
 import { memSearchSchema, memRecentSchema, memTimelineSchema, memGetSchema, memDeleteSchema, memSaveSchema, memStatsSchema, memCompressSchema, memMaintainSchema, memOptimizeSchema, memUpdateSchema, memExportSchema, memRecallSchema, memFtsCheckSchema, memRegistrySchema, memBrowseSchema, memUseSchema, memDeferSchema, memDeferListSchema, memDeferDropSchema, tools as TOOL_DEFS } from './tool-schemas.mjs';
 
@@ -45,7 +47,6 @@ import { probeOtherSources as probeIdSources, bucketIdTokens } from './lib/id-ro
 import { saveObservation } from './lib/save-observation.mjs';
 import { rebuildObservationDerived } from './lib/observation-write.mjs';
 import { EXPORT_COLUMNS_SQL } from './lib/export-columns.mjs';
-import { computeNoiseGauge } from './lib/stats-quality.mjs';
 import { recallByFile } from './lib/recall-core.mjs';
 import { AUTO_MERGE_THRESHOLD } from './lib/dedup-constants.mjs';
 import {
@@ -680,49 +681,14 @@ server.registerTool(
       return { content: [{ type: 'text', text: lines.join('\n') }] };
     }
 
-    // Snapshot before the irreversible hard-delete so a wrong-id delete has a pre-image,
-    // matching the CLI delete + maintain purge/cleanup paths (audit MED-2). Best-effort
-    // (never throws, skips :memory:). Must run OUTSIDE the transaction below (VACUUM).
-    snapshotDb(db, { tag: 'pre-delete' });
-
-    // Wrap cleanup + deletion in a transaction for consistency
-    const deletedIds = new Set(args.ids);
-    const deleteTx = db.transaction(() => {
-      // Clean up stale references in other observations' related_ids
-      // Use LIKE filter to avoid O(N) full-table scan — only fetch rows that may reference deleted IDs.
-      // NOTE: LIKE %id% has false positives (e.g. %1% matches [10], [21]). This is intentional —
-      // the LIKE is a coarse pre-filter; the JSON parse + Set.has below is the precise filter.
-      // Acceptable because observation count per user is typically <10K.
-      const likeConditions = args.ids.map(() => `related_ids LIKE ?`).join(' OR ');
-      const likeParams = args.ids.map(id => `%${id}%`);
-      const referencing = db.prepare(`
-        SELECT id, related_ids FROM observations
-        WHERE related_ids IS NOT NULL AND related_ids != '[]'
-          AND (${likeConditions})
-      `).all(...likeParams);
-      for (const r of referencing) {
-        let ids;
-        try { ids = JSON.parse(r.related_ids); } catch (e) { debugCatch(e, 'deleteRelatedIds'); continue; }
-        if (!Array.isArray(ids) || !ids.every(id => Number.isInteger(id))) continue;
-        const filtered = ids.filter(id => !deletedIds.has(id));
-        if (filtered.length !== ids.length) {
-          db.prepare('UPDATE observations SET related_ids = ? WHERE id = ?').run(JSON.stringify(filtered), r.id);
-        }
-      }
-      // Resurface rows merged/compressed INTO the doomed keepers before deleting, else
-      // they dangle behind a now-missing parent (compressed_into has no FK) — invisible
-      // to every COALESCE(compressed_into,0)=0 view and unrecoverable. Mirrors the CLI
-      // delete path + the maintain hard-delete guard (recoverChildrenOf).
-      const recovered = recoverChildrenOf(db, args.ids);
-      // Execute deletion (FTS5 cleanup handled by observations_ad trigger)
-      const deleted = db.prepare(`DELETE FROM observations WHERE id IN (${placeholders})`).run(...args.ids);
-      return { changes: deleted.changes, recovered };
-    });
-    const result = deleteTx();
+    // Full delete orchestration (snapshot + related_ids cleanup + child recovery +
+    // delete transaction) lives in lib/delete-core.mjs — single source of truth shared
+    // with the CLI `delete` path (was inlined + kept in sync by parity comments).
+    const result = deleteObservations(db, args.ids);
 
     const missing = args.ids.filter(id => !rows.some(r => r.id === id));
-    const msg = [`Deleted ${result.changes} observation(s).`];
-    if (result.recovered > 0) msg.push(`Recovered ${result.recovered} merged/compressed child observation(s) to live.`);
+    const msg = [`Deleted ${result.deleted} observation(s).`];
+    if (result.recoveredChildren > 0) msg.push(`Recovered ${result.recoveredChildren} merged/compressed child observation(s) to live.`);
     if (missing.length > 0) msg.push(`Note: ID(s) ${missing.join(', ')} not found.`);
     return { content: [{ type: 'text', text: msg.join(' ') }] };
   })
@@ -784,7 +750,8 @@ server.registerTool(
     const supersededNote = result.supersededIds && result.supersededIds.length > 0
       ? ` Superseded: ${result.supersededIds.map(i => `#${i}`).join(', ')}.`
       : '';
-    return { content: [{ type: 'text', text: `Saved as observation #${result.id} [${result.type}] in project "${project}".${lessonNote}${closedNote}${supersededNote}` }] };
+    const nudge = buildLessonNudge({ type: result.type, id: result.id, lessonCaptured: result.lessonCaptured, surface: 'mcp' });
+    return { content: [{ type: 'text', text: `Saved as observation #${result.id} [${result.type}] in project "${project}".${lessonNote}${closedNote}${supersededNote}${nudge}` }] };
   })
 );
 
@@ -882,97 +849,11 @@ server.registerTool(
       return { content: [{ type: 'text', text: formatQualityReport(data) }] };
     }
 
-    const cutoff = Date.now() - days * 86400000;
-    const projectFilter = args.project ? 'AND project = ?' : '';
-    const baseParams = args.project ? [args.project] : [];
-
-    // Total counts
-    const obsTotal = db.prepare(`SELECT COUNT(*) as c FROM observations WHERE 1=1 ${projectFilter}`).get(...baseParams);
-    const sessTotal = db.prepare(`SELECT COUNT(*) as c FROM session_summaries WHERE 1=1 ${projectFilter}`).get(...baseParams);
-    const promptTotal = args.project
-      ? db.prepare(`SELECT COUNT(*) as c FROM user_prompts p JOIN sdk_sessions s ON p.content_session_id = s.content_session_id WHERE s.project = ?`).get(args.project)
-      : db.prepare(`SELECT COUNT(*) as c FROM user_prompts`).get();
-
-    // Recent counts
-    const obsRecent = db.prepare(`SELECT COUNT(*) as c FROM observations WHERE created_at_epoch >= ? ${projectFilter}`).get(cutoff, ...baseParams);
-    const sessRecent = db.prepare(`SELECT COUNT(*) as c FROM session_summaries WHERE created_at_epoch >= ? ${projectFilter}`).get(cutoff, ...baseParams);
-
-    // Type distribution (recent)
-    const types = db.prepare(`
-      SELECT type, COUNT(*) as c FROM observations
-      WHERE created_at_epoch >= ? ${projectFilter}
-      GROUP BY type ORDER BY c DESC
-    `).all(cutoff, ...baseParams);
-
-    // Projects (global view — skipped when filtering by single project)
-    const projects = args.project ? [] : db.prepare(`
-      SELECT project, COUNT(*) as c FROM observations
-      GROUP BY project ORDER BY c DESC
-      LIMIT 20
-    `).all();
-
-    // Daily activity (last 7 days)
-    const daily = db.prepare(`
-      SELECT date(created_at) as day, COUNT(*) as c FROM observations
-      WHERE created_at_epoch >= ? ${projectFilter}
-      GROUP BY day ORDER BY day DESC
-      LIMIT 7
-    `).all(Date.now() - 7 * 86400000, ...baseParams);
-
-    // Health metrics
-    const tokenEst = db.prepare(`
-      SELECT SUM(LENGTH(COALESCE(title,'')) + LENGTH(COALESCE(narrative,'')) + LENGTH(COALESCE(text,''))) / 4 as t
-      FROM observations WHERE 1=1 ${projectFilter}
-    `).get(...baseParams);
-
-    const avgImp = db.prepare(`
-      SELECT AVG(COALESCE(importance,1)) as v FROM observations WHERE 1=1 ${projectFilter}
-    `).get(...baseParams);
-
-    const thirtyDaysAgo = Date.now() - 30 * 86400000;
-    // v3.23 noise-gauge de-blinding (mirrors mem-cli cmdStats): `<= 1` makes the
-    // imp=0 dormant population visible (decay floor + LLM low-signal filter push
-    // ~half the live corpus to 0); injection_count=0 mirrors decay's NEVER-INJECTED
-    // guard so injected-but-decayed pinned noise isn't miscounted as "never used".
-    const lowVal = db.prepare(`
-      SELECT COUNT(*) as c FROM observations
-      WHERE COALESCE(importance,1) <= 1 AND COALESCE(access_count,0) = 0
-        AND COALESCE(injection_count,0) = 0
-        AND COALESCE(compressed_into, 0) = 0
-        AND created_at_epoch < ? ${projectFilter}
-    `).get(thirtyDaysAgo, ...baseParams);
-
-    // Low-signal-title population (template / tool-log titles the read-side filter
-    // already excludes). The imp=1 "Low-value" metric can't see these, so the
-    // gauge under-reports real noise without it. See lib/low-signal-patterns.mjs.
-    const lowSignalTitle = db.prepare(`
-      SELECT COUNT(*) as c FROM observations
-      WHERE NOT ${buildNotLowSignalSql()}
-        AND COALESCE(compressed_into, 0) = 0 ${projectFilter}
-    `).get(...baseParams);
-    // F7: both noise numerators exclude compressed rows → divide by the LIVE count, not
-    // obsTotal (all rows), so a compress-heavy store isn't reported cleaner than it is.
-    const liveTotal = db.prepare(
-      `SELECT COUNT(*) as c FROM observations WHERE COALESCE(compressed_into, 0) = 0 ${projectFilter}`
-    ).get(...baseParams);
-    const { noiseRatio, lowSignalRatio } = computeNoiseGauge({ liveTotal: liveTotal.c, lowValCount: lowVal.c, lowSignalCount: lowSignalTitle.c });
-    const compressedCount = db.prepare(`
-      SELECT COUNT(*) as c FROM observations WHERE compressed_into IS NOT NULL ${projectFilter}
-    `).get(...baseParams);
-    const supersededOnlyCount = db.prepare(`
-      SELECT COUNT(*) as c FROM observations WHERE superseded_at IS NOT NULL AND compressed_into IS NULL ${projectFilter}
-    `).get(...baseParams);
-
-    // Tier distribution
-    const tierCtx = { now: Date.now(), currentProject: args.project || inferProject(), currentSessionId: '' };
-    const tdParams = tierSqlParams(tierCtx);
-    const tierDist = db.prepare(`
-      SELECT tier, COUNT(*) as c FROM (
-        SELECT ${TIER_CASE_SQL} as tier FROM observations
-        WHERE COALESCE(compressed_into, 0) = 0 AND superseded_at IS NULL ${projectFilter}
-      ) GROUP BY tier ORDER BY tier
-    `).all(...tdParams, ...baseParams);
-    const tierMap = Object.fromEntries(tierDist.map(r => [r.tier, r.c]));
+    const {
+      obsTotal, sessTotal, promptTotal, obsRecent, sessRecent,
+      types, projects, daily, tokenEst, avgImp, lowVal, lowSignalTitle,
+      noiseRatio, lowSignalRatio, compressedCount, supersededOnlyCount, tierMap,
+    } = computeStatsFeed(db, { project: args.project || null, days });
 
     const lines = [
       `Memory Statistics${args.project ? ` (project: ${args.project})` : ''}:`,

package/source-files.mjs

@@ -152,6 +152,29 @@ export const SOURCE_FILES = [
   // server.mjs, and hook.mjs before their destructive purge/cleanup — missing it
   // would crash maintain on auto-update with an unresolved import.
   'lib/db-backup.mjs',
+  // HIGH-1 events-injection: surfaces the `events` canonical store into the passive
+  // injection surfaces. Statically imported by hook.mjs (UserPromptSubmit) and
+  // hook-context.mjs (SessionStart) — missing it from the manifest would break both
+  // hooks on auto-update.
+  'lib/events-injection.mjs',
+  // Shared delete orchestration (snapshot + related_ids cleanup + child recovery
+  // + delete txn). Statically imported by server.mjs (mem_delete) and mem-cli.mjs
+  // (cmdDelete) — extracted to kill the byte-duplicated twin. Missing it from the
+  // manifest would leave both delete surfaces unsigned/broken on auto-update.
+  'lib/delete-core.mjs',
+  // Shared primary stats feed (audit 2026-07-17 MED-4) — statically imported by
+  // server.mjs (mem_stats) and mem-cli.mjs (cmdStats); killed the ~80-line
+  // byte-duplicated twin. Missing it breaks both stats surfaces on auto-update.
+  'lib/stats-core.mjs',
+  // Observation `type` vocabulary single source (audit 2026-07-17 MED-3) —
+  // statically imported by tool-schemas.mjs, mem-cli.mjs, hook-llm.mjs,
+  // hook-optimize.mjs, lib/activity.mjs. Missing it breaks every save/validate
+  // path on auto-update.
+  'lib/obs-types.mjs',
+  // Save-time lesson nudge (audit 2026-07-17 P4) — statically imported by server.mjs
+  // (mem_save) and mem-cli.mjs (cmdSave). Missing it breaks both save surfaces on
+  // auto-update.
+  'lib/save-nudge.mjs',
   // P10 dedup/merge threshold constants — single source of truth for the Jaccard
   // dedup/merge cutoffs. Statically imported by hook.mjs, hook-llm.mjs,
   // hook-optimize.mjs, mem-cli.mjs, server.mjs, and the save/maintain cores;

package/tool-schemas.mjs

@@ -3,8 +3,9 @@
 
 import { z } from 'zod';
 import { CLI_INVOKE } from './cli-path.mjs';
+import { OBS_TYPES } from './lib/obs-types.mjs';
 
-export const OBS_TYPE_ENUM = z.enum(['decision', 'bugfix', 'feature', 'refactor', 'discovery', 'change']);
+export const OBS_TYPE_ENUM = z.enum([...OBS_TYPES]);
 
 // LLM-friendly coercion: accept string numbers and normalize to proper types
 const coerceInt = z.preprocess(