claude-mem-lite 3.65.0 → 3.66.1

package/hook.mjs

@@ -42,7 +42,7 @@ import {
   SESSION_EXPIRY_MS, STALE_SESSION_MS, STALE_LOCK_MS,
   HANDOFF_EXPIRY_CLEAR, HANDOFF_EXPIRY_EXIT,
   sessionFile, getSessionId, createSessionId, openDb,
-  spawnBackground, sweepOrphanEpisodeFiles,
+  spawnBackground, sweepOrphanEpisodeFiles, sweepStaleProjectMarkers,
 } from './hook-shared.mjs';
 import { handleLLMEpisode, handleLLMSummary, saveObservation, buildImmediateObservation, saveEpisodeImmediate } from './hook-llm.mjs';
 import { scrubRecord } from './lib/scrub-record.mjs';
@@ -54,6 +54,7 @@ import { snapshotDb } from './lib/db-backup.mjs';
 import {
   extractCitationsFromTranscript,
   extractAllInjected,
+  extractInjectedFromKeyContext,
   bumpCitationAccess,
   computeCiteRecall,
   applyCitationDecay,
@@ -69,6 +70,7 @@ import { recordSkillAdoption, gcOldShadowShards } from './registry-recommend.mjs
 import { gcOldMetricShards, recordMetric } from './lib/metrics.mjs';
 import { detectMemOverride } from './lib/mem-override.mjs';
 import { injectedIdsFileName, keyContextIdsFileName } from './lib/injected-ids.mjs';
+import { recordKeyContextInjection } from './lib/keyctx-marker.mjs';
 import { liveObsFilterSql, recencyDecaySql } from './lib/inject-search-core.mjs';
 import { buildAndSaveHandoff, detectContinuationIntent, renderHandoffInjection, pickHandoffToInject, extractUnfinishedSummary } from './hook-handoff.mjs';
 import { checkForUpdate, getCachedUpdateBanner, isUpdateCheckDue } from './hook-update.mjs';
@@ -90,6 +92,7 @@ async function loadCacheGuard() {
 import { SKIP_TOOLS, SKIP_PREFIXES } from './skip-tools.mjs';
 import { getVocabulary } from './tfidf.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 // Prevent recursive hooks from background claude -p calls
 // Background workers (llm-episode, llm-summary) are exempt — they're ours
 const event = process.argv[2];
@@ -745,7 +748,19 @@ async function handleStop() {
             // edit promotes them even without a literal #NN in text.
             const citeBackIds = extractCiteBackSignals(transcriptPath);
             for (const id of citeBackIds) injected.add(id);
-            if (injected.size > 0) {
+            // D#124, promotion-only (v3.66.1): the SessionStart Key Context block
+            // leaves no hook attachment, so its ids come from the per-session
+            // marker. They are added to the decay set ONLY where they were
+            // actually cited (below), never as bare denominator: the block
+            // re-renders the same fixed top-10 unconditionally, so an uncited
+            // render says nothing about relevance — and since keyObs gates on
+            // `importance >= 2`, one demotion evicts the common importance-2 row
+            // from Key Context for good. v3.66.0 fed them in as denominator and
+            // that made the block eat its own contents.
+            const keyCtxIds = extractInjectedFromKeyContext({
+              runtimeDir: RUNTIME_DIR, project, sessionId: ccSessionId,
+            });
+            if (injected.size > 0 || keyCtxIds.size > 0) {
               // Text-floor gate: skip decay on tool-only Stops. Without this,
               // a turn that ends on tool_use locks every injected obs as
               // uncited (last_decided_session_id set), so a later turn that
@@ -758,6 +773,10 @@ async function handleStop() {
               } else {
                 const citedMain = extractCitationsFromTranscript(transcriptPath, { mainOnly: true });
                 for (const id of citeBackIds) citedMain.add(id);
+                // The promotion-only half: a Key Context row the agent actually
+                // cited joins the decay set (and takes the promote branch); one
+                // it ignored is never entered, so it cannot streak or demote.
+                for (const id of keyCtxIds) if (citedMain.has(id)) injected.add(id);
                 // D#60: the idempotency key must be the CC session UUID, NOT the
                 // project-scoped memory sessionId — concurrent same-project CC
                 // sessions share the latter, so the second session's decay pass
@@ -890,7 +909,7 @@ function gcStalePreRecallCooldowns() {
 function runSessionStartDbMutations(db, { sessionId, project, prevSessionId, now }) {
   // ── DB mutations in a transaction (crash-safe consistency) ──
   const staleSessionCutoff = Date.now() - STALE_SESSION_MS;
-  const autoCompressAge = Date.now() - 30 * 86400000; // 30 days (accelerated from 90)
+  const autoCompressAge = Date.now() - 30 * DAY_MS; // 30 days (accelerated from 90)
 
   db.transaction(() => {
     // Ensure session exists in DB (INSERT OR IGNORE avoids race condition)
@@ -945,7 +964,7 @@ function runSessionStartDbMutations(db, { sessionId, project, prevSessionId, now
     // imp=1 on these already; this just shrinks the GC latency so the
     // projected 32.5% corpus reduction materializes within a week on live
     // DBs instead of bleeding into the 30-day tier.
-    const noiseCompressAge = Date.now() - 7 * 86400000;
+    const noiseCompressAge = Date.now() - 7 * DAY_MS;
     const noiseCompressed = db.prepare(`
       UPDATE observations SET compressed_into = ${COMPRESSED_AUTO}
       WHERE COALESCE(compressed_into, 0) = 0
@@ -975,7 +994,7 @@ function runSessionStartAutoMaintain(db) {
   } catch {}
   if (shouldMaintain) {
     try {
-      const STALE_AGE = Date.now() - 30 * 86400000;
+      const STALE_AGE = Date.now() - 30 * DAY_MS;
       const OP_CAP = 500;
       // Shared maintenance context (whole-DB, cap 500) — used by every maintain-core
       // op below AND the MED-2 snapshot guard. injection_count>0 protection lives in
@@ -995,7 +1014,7 @@ function runSessionStartAutoMaintain(db) {
       // children (compressed_into dangling at a deleted id). purgeStale recovers them
       // first and caps at opCap. Schema has no marked_at_epoch, so retention anchors on
       // created_at_epoch: 30d marking gate + 7d grace = 37d.
-      const purged = purgeStale(db, mctx, Date.now() - 37 * 86400000);
+      const purged = purgeStale(db, mctx, Date.now() - 37 * DAY_MS);
       if (purged > 0) debugLog('DEBUG', 'auto-maintain', `purged ${purged} stale observations`);
 
       const cleaned = cleanupBroken(db, mctx);
@@ -1109,7 +1128,7 @@ function runSessionStartAutoMaintain(db) {
           DELETE FROM session_handoffs
           WHERE (type = 'clear' AND created_at_epoch < ?)
              OR (type != 'clear' AND created_at_epoch < ?)
-        `).run(Date.now() - HANDOFF_EXPIRY_CLEAR - 86400000, Date.now() - HANDOFF_EXPIRY_EXIT - 86400000);
+        `).run(Date.now() - HANDOFF_EXPIRY_CLEAR - DAY_MS, Date.now() - HANDOFF_EXPIRY_EXIT - DAY_MS);
         if (gc.changes > 0) debugLog('DEBUG', 'auto-maintain', `gc'd ${gc.changes} expired session_handoffs`);
       } catch (e) { debugCatch(e, 'auto-maintain-handoff-gc'); }
 
@@ -1349,6 +1368,10 @@ async function handleSessionStart() {
   // GC stale per-session cooldown files. Cheap (<5ms typical) and idempotent;
   // moved here from pre-tool-recall.js's hot path.
   gcStalePreRecallCooldowns();
+  // P2-15: the per-PROJECT half of the same problem — markers written once per
+  // project and never revisited (session-/cite-recall-/skill cooldowns). Same
+  // SessionStart cadence, 30d gate, named family list (hook-shared.mjs).
+  try { sweepStaleProjectMarkers(RUNTIME_DIR); } catch { /* best-effort */ }
   // Bound the shadow-recommendation log (daily JSONL shards, no GC at write time).
   try { gcOldShadowShards(); } catch { /* best-effort, never blocks SessionStart */ }
   // Same for the opt-in metrics sink (RUNTIME_DIR's parent is DB_DIR). Runs even when
@@ -1487,12 +1510,15 @@ async function handleSessionStart() {
     // exclude-set blanked the same-project leg on adopted projects). Written
     // unconditionally (even when empty) so a resumed session can't act on a
     // previous session's stale marker semantics; 24h GC below.
-    try {
-      writeFileSync(
-        join(RUNTIME_DIR, keyContextIdsFileName(project, ccSessionId)),
-        JSON.stringify({ ids: contextCollector.keyContextIds || [], ts: Date.now(), session: ccSessionId || null }),
-      );
-    } catch (e) { debugCatch(e, 'session-start-keyctx-marker'); }
+    // D#124: the same call also bumps injection_count on the rendered rows —
+    // Key Context was a shown-but-uncounted surface, so its rows could never
+    // reach applyCitationDecay's denominator. One recorder, both writers.
+    recordKeyContextInjection(db, {
+      runtimeDir: RUNTIME_DIR,
+      project,
+      sessionId: ccSessionId,
+      ids: contextCollector.keyContextIds || [],
+    });
 
     // One-time migration: remove any stale <claude-mem-context> block left in
     // CLAUDE.md by pre-v2.30 installs. Idempotent no-op afterwards.
@@ -1797,7 +1823,7 @@ function handleAutoCompress() {
   if (!db) return;
 
   try {
-    const compressCutoff = Date.now() - 60 * 86400000; // 60 days
+    const compressCutoff = Date.now() - 60 * DAY_MS; // 60 days
     const compressCandidates = selectCompressionCandidates(db, { cutoff: compressCutoff, includeAutoMarked: true });
     if (compressCandidates.length < 3) return;
 

package/lib/citation-tracker.mjs

@@ -13,7 +13,9 @@
 import { readFileSync, existsSync, readdirSync, statSync } from 'fs';
 import { join } from 'path';
 import { debugCatch } from '../utils.mjs';
+import { keyContextIdsFileName } from './injected-ids.mjs';
 
+import { DAY_MS } from './time-constants.mjs';
 // `#123` / `#45678` at a word boundary — matches the CLAUDE.md cite pattern.
 // Bounded to 1-7 digits to skip URL fragments, markdown anchors, etc.
 const CITATION_RE = /#(\d{1,7})\b/g;
@@ -343,9 +345,61 @@ export function extractInjectedFromFyi(transcriptPath, opts = {}) {
 }
 
 /**
- * Union of every injection surface's IDs for a transcript: pre-tool-recall +
- * UserPromptSubmit `<memory-context>` + PostToolUse error-recall + the
- * user-prompt-search FYI block. Single integration point the Stop handler calls.
+ * Extract observation IDs rendered into the SessionStart / PreCompact
+ * `<claude-mem-context>` File Lessons + Key Context sections (D#124).
+ *
+ * The odd one out among the extractors: this surface leaves no hook attachment
+ * in the transcript — the block is written straight to stdout at SessionStart —
+ * so there is nothing to parse. It reads the per-session marker instead, the
+ * same file handleUserPrompt uses as its exclude-set, which by construction
+ * lists what was ACTUALLY rendered (empty on quiet/adopted projects where the
+ * sections never appear).
+ *
+ * Session-gated: a marker whose recorded session differs from the caller's is
+ * another window's render and must not be attributed to this session.
+ *
+ * PROMOTION-ONLY (v3.66.1). Deliberately NOT part of extractAllInjected: the
+ * other four faces are query-conditioned — a row appears there because it
+ * MATCHED something, so its absence from the cited set is evidence it was not
+ * useful. A Key Context render is unconditional and re-renders the same fixed
+ * top-10 every session, so an uncited render is evidence of nothing but elapsed
+ * time. Feeding these ids into the decay DENOMINATOR made the block consume
+ * itself: keyObs gates on `importance >= 2` (hook-context.mjs), and a demotion
+ * takes the common importance-2 row to 1, dropping it out of Key Context
+ * permanently after 3 uncited sessions — each departure promoting the next row
+ * into the same grinder. Callers must therefore intersect with the cited set
+ * (see handleStop) so a CITED Key Context row is credited while an uncited one
+ * is left alone.
+ *
+ * @param {object} [ctx]
+ * @param {string} [ctx.runtimeDir]
+ * @param {string} [ctx.project]
+ * @param {string|null} [ctx.sessionId]
+ * @returns {Set<number>}
+ */
+export function extractInjectedFromKeyContext({ runtimeDir, project, sessionId = null } = {}) {
+  const ids = new Set();
+  if (!runtimeDir || !project) return ids;
+  try {
+    const raw = readFileSync(join(runtimeDir, keyContextIdsFileName(project, sessionId)), 'utf8');
+    const parsed = JSON.parse(raw);
+    if (sessionId && parsed?.session && parsed.session !== sessionId) return ids;
+    for (const id of Array.isArray(parsed?.ids) ? parsed.ids : []) addObsId(ids, id);
+  } catch { /* no marker / torn write → nothing was rendered, fail open */ }
+  return ids;
+}
+
+/**
+ * Union of the QUERY-CONDITIONED injection surfaces for a transcript:
+ * pre-tool-recall + UserPromptSubmit `<memory-context>` + PostToolUse
+ * error-recall + the user-prompt-search FYI block. Single integration point the
+ * Stop handler calls for the decay DENOMINATOR.
+ *
+ * Key Context is intentionally absent (v3.66.1 — it was unioned here for one
+ * release): every face above appears because a row matched something, so an
+ * uncited appearance carries relevance information. An unconditional
+ * SessionStart render does not. Callers wanting the Key Context ids ask
+ * `extractInjectedFromKeyContext` directly and use them promotion-only.
  *
  * @param {string|null|undefined} transcriptPath
  * @param {object} [opts]
@@ -808,8 +862,8 @@ export function computeCitationFunnelTrend(db, { days = 7, limit = 10, project =
   if (!db) return empty;
   try {
     const now = Date.now();
-    const windowStart = now - days * 86400000;
-    const priorStart = now - 2 * days * 86400000;
+    const windowStart = now - days * DAY_MS;
+    const priorStart = now - 2 * days * DAY_MS;
     const projClause = project ? 'AND project = ?' : '';
 
     const sessions = db.prepare(`

package/lib/db-backup.mjs

@@ -11,6 +11,7 @@ import { readdirSync, unlinkSync, statSync } from 'fs';
 import { dirname, basename, join } from 'path';
 import { debugLog } from '../utils.mjs';
 
+import { DAY_MS } from './time-constants.mjs';
 // M-9 (audit 2026-08-14): per-tag retention alone let one-shot tags live forever —
 // each tag kept its newest 3, but a tag written once (pre-backfill-v3390 and four
 // siblings) never got a second snapshot to age it out, so 9 orphaned .bak files held
@@ -35,7 +36,7 @@ export function backupBudgetBytes() {
 // a per-tag exemption would make every one-shot tag's only snapshot immortal —
 // exactly the 360MB orphan shape M-9 exists to evict. Mirrors the purge grace
 // convention (7d) used by stale-observation retention.
-export const BACKUP_EVICTION_GRACE_MS = 7 * 86400000;
+export const BACKUP_EVICTION_GRACE_MS = 7 * DAY_MS;
 
 // Canonical snapshot shape `<base>.<tag>-<ISO stamp>-<pid>-<seq>.bak` (what
 // snapshotDb writes). Budget eviction deletes ONLY names matching this — a

package/lib/deferred-work.mjs

@@ -1,3 +1,4 @@
+import { DAY_MS } from './time-constants.mjs';
 // claude-mem-lite — deferred_work data layer
 // Pure-data CRUD + ordinal resolver + transactional closure helper.
 // Decoupled from observations table: different lifecycle, different scoring.
@@ -60,7 +61,6 @@ export function listOpenWithOrdinal(db, project, limit = 10) {
 
 // ─── G11: list age + stale refresh hint (roadmap 2026-07-18) ─────────────────
 
-const DAY_MS = 86_400_000;
 // Internal const (was exported at v3.51.0 birth with zero external importers —
 // knip-baseline discipline: un-export rather than grow the unused-exports list).
 const DEFER_STALE_DAYS = 30;

package/lib/err-sampler.mjs

@@ -19,8 +19,8 @@ import { appendFileSync, mkdirSync, existsSync, readdirSync, statSync, unlinkSyn
 import { join } from 'path';
 import { scrubSecrets } from '../secret-scrub.mjs';
 
-const DAY_MS = 86400000;
 
+import { DAY_MS } from './time-constants.mjs';
 function today() {
   // UTC date string; sharding daily keeps files bounded even at high sample rates.
   return new Date(Date.now()).toISOString().slice(0, 10);

package/lib/hook-telemetry.mjs

@@ -26,7 +26,7 @@ import { join } from 'path';
 import { isNativeBindingError } from './binding-probe.mjs';
 import { recordNativeBindingBreakage } from './native-binding-hint.mjs';
 
-const DAY_MS = 86400000;
+import { DAY_MS } from './time-constants.mjs';
 const RETENTION_MS = 14 * DAY_MS;
 const HOOK_ERRORS_SUBDIR = 'hook-errors';
 

package/lib/keyctx-marker.mjs

@@ -0,0 +1,74 @@
+// lib/keyctx-marker.mjs — the one place the SessionStart Key Context render is
+// recorded. Two things happen together because they must describe the SAME set:
+//
+//   ① the per-session marker file (D#123 review C-1): the obs ids ACTUALLY
+//      rendered into <claude-mem-context>, which handleUserPrompt reads as its
+//      exclude-set. Written even when empty, so a resumed session can never act
+//      on a previous session's stale semantics.
+//   ② injection_count / last_injected_at on those same rows (D#124): before
+//      this, Key Context was a shown-but-uncounted surface — up to 10 rows per
+//      session that could accrue no denominator and therefore never promote or
+//      demote through applyCitationDecay.
+//
+// handleSessionStart (hook.mjs) and handlePreCompact (hook-precompact.mjs) both
+// render the block, so both call this. Keeping the pair in one function is the
+// point: the CLI/MCP and SessionStart/PreCompact twin pairs in this repo have
+// drifted often enough that "two callers, one body" is the standing rule.
+//
+// Never throws: a marker-write failure must not break context delivery, and it
+// must not cost the metering either — the bump runs first.
+
+import { writeFileSync } from 'fs';
+import { join } from 'path';
+import { debugCatch } from '../utils.mjs';
+import { keyContextIdsFileName } from './injected-ids.mjs';
+
+/**
+ * Record one Key Context render: bump the rendered rows, then persist the id
+ * list for the prompt-time exclude-set and the citation extractor.
+ *
+ * @param {import('better-sqlite3').Database} db
+ * @param {object} ctx
+ * @param {string} ctx.runtimeDir  RUNTIME_DIR of the caller
+ * @param {string} ctx.project     inferProject() value (filename-safe)
+ * @param {string|null} [ctx.sessionId] CC session id
+ * @param {number[]} [ctx.ids]     obs ids ACTUALLY rendered (empty on quiet projects)
+ * @returns {{bumped: number, written: boolean}}
+ */
+export function recordKeyContextInjection(db, { runtimeDir, project, sessionId = null, ids = [] } = {}) {
+  const clean = [];
+  for (const raw of Array.isArray(ids) ? ids : []) {
+    const id = Number(raw);
+    if (Number.isInteger(id) && id > 0 && id < 1e7) clean.push(id);
+  }
+
+  // NO injection_count bump. v3.66.0 added one here and it had to be reverted in
+  // v3.66.1: injection_count is not a neutral counter. scoring-sql.mjs states the
+  // invariant — "bumped ONLY on UserPromptSubmit / hook-memory auto-inject" —
+  // because noisePenaltyClause reads it as a NOISE signal: a row scores x0.5 once
+  // injection_count >= 4 (and > access_count * 3), x0.2 at >= 8. Nothing bumps
+  // access_count for a rendered row (bumpCitationAccess fires on CITED ids only),
+  // so the counter crosses those thresholds purely as a function of elapsed
+  // sessions, deprioritising the highest-importance rows — the exact rows Key
+  // Context renders — in mem_search, UPS ranking and injectionRelevanceSql.
+  //
+  // The UPS bump the reverted code claimed to "mirror verbatim" is
+  // query-conditioned: a row is counted only when it MATCHED a query, so the
+  // counter means "auto-injected and never useful". A Key Context render is
+  // unconditional and hits the same fixed row set every session, so it would have
+  // measured nothing but time. D#124's requirement is decay reachability, which
+  // the extractor face delivers on its own — decay reads decay_seen_count /
+  // uncited_streak / cited_count, never injection_count.
+  const bumped = 0;
+
+  let written = false;
+  try {
+    writeFileSync(
+      join(runtimeDir, keyContextIdsFileName(project, sessionId)),
+      JSON.stringify({ ids: clean, ts: Date.now(), session: sessionId || null }),
+    );
+    written = true;
+  } catch (e) { debugCatch(e, 'keyctx-marker-write'); }
+
+  return { bumped, written };
+}

package/lib/maintain-core.mjs

@@ -17,7 +17,8 @@ import { rebuildVocabulary, computeVector, _resetVocabCache, vectorsEnabled, vec
 import { DEDUP_JACCARD_THRESHOLD, MINHASH_PRE_THRESHOLD as MINHASH_PRE_THRESHOLD_SRC, FUZZY_DEDUP_THRESHOLD, FUZZY_BODY_THRESHOLD, MINHASH_PREFILTER } from './dedup-constants.mjs';
 import { liveObsFilterSql } from './inject-search-core.mjs';
 
-export const STALE_AGE_MS = 30 * 86400000;
+import { DAY_MS } from './time-constants.mjs';
+export const STALE_AGE_MS = 30 * DAY_MS;
 export const OP_CAP = 1000;
 export const SCAN_LIMIT = 500;
 export const DUPLICATE_LIMIT = 50;

package/lib/metrics.mjs

@@ -19,8 +19,8 @@
 import { appendFileSync, existsSync, mkdirSync, readdirSync, readFileSync, unlinkSync } from 'fs';
 import { join } from 'path';
 
-const DAY_MS = 86400000;
 
+import { DAY_MS } from './time-constants.mjs';
 function today() { return new Date(Date.now()).toISOString().slice(0, 10); }
 
 function metricsEnabled() {

package/lib/save-enrich.mjs

@@ -84,7 +84,8 @@ export async function executeSaveEnrich(db, id, { callJson } = {}) {
   const wantAliases = !(row.search_aliases && row.search_aliases.trim());
   if (!wantLesson && !wantAliases) return { enriched: false, reason: 'complete' };
 
-  const call = callJson || (await import('../haiku-client.mjs')).callModelJSON;
+  const { callModelJSON, BG_LLM_TIMEOUT_MS } = await import('../haiku-client.mjs');
+  const call = callJson || callModelJSON;
   const prompt = `A coding memory was just saved. Enrich its retrievability. Return ONLY valid JSON, no markdown fences.
 
 Title: ${truncate(row.title || '(untitled)', 200)}
@@ -96,7 +97,7 @@ search_aliases: 3-6 terms a user might search for the SAME memory that are NOT a
 
   let parsed = null;
   try {
-    parsed = await call(prompt, 'haiku', { timeout: 15000, maxTokens: 400 });
+    parsed = await call(prompt, 'haiku', { timeout: BG_LLM_TIMEOUT_MS, maxTokens: 400 });
   } catch (e) { debugCatch(e, 'save-enrich-llm'); }
   if (!parsed || typeof parsed !== 'object') return { enriched: false, reason: 'llm-null' };
 

package/lib/search-core.mjs

@@ -31,6 +31,7 @@ import { countSearchTotal, attachBodyTokens } from '../search-engine.mjs';
 import { notLowSignalTitleClause } from '../scoring-sql.mjs';
 import { liveObsFilterSql, recencyDecaySql } from './inject-search-core.mjs';
 
+import { DAY_MS } from './time-constants.mjs';
 /** Sanitize a user query to FTS5 syntax; optionally force OR semantics. */
 export function buildSearchFtsQuery(query, { or = false } = {}) {
   let ftsQuery = sanitizeFtsQuery(query);
@@ -40,7 +41,7 @@ export function buildSearchFtsQuery(query, { or = false } = {}) {
 
 // Relative-duration units for `--since`. Months/years intentionally omitted —
 // they're calendar-ambiguous; absolute --from/--to cover longer windows.
-const DURATION_UNIT_MS = { s: 1000, m: 60000, h: 3600000, d: 86400000, w: 604800000 };
+const DURATION_UNIT_MS = { s: 1000, m: 60000, h: 3600000, d: DAY_MS, w: 604800000 };
 
 /**
  * Parse a relative duration like "7d", "24h", "90m", "2w", "30s" into ms.

package/lib/stats-core.mjs

@@ -12,13 +12,14 @@ import { TIER_CASE_SQL, tierSqlParams } from '../tier.mjs';
 import { computeNoiseGauge } from './stats-quality.mjs';
 import { liveObsFilterSql } from './inject-search-core.mjs';
 
+import { DAY_MS } from './time-constants.mjs';
 /**
  * Compute the primary stats feed. Row shapes are returned exactly as the twin
  * blocks produced them ({c}/{t}/{v} single-row objects, arrays for distributions)
  * so both call sites render with minimal diff.
  */
 export function computeStatsFeed(db, { project = null, days = 30, now = Date.now() } = {}) {
-  const cutoff = now - days * 86400000;
+  const cutoff = now - days * DAY_MS;
   const projectFilter = project ? 'AND project = ?' : '';
   const baseParams = project ? [project] : [];
 
@@ -51,7 +52,7 @@ export function computeStatsFeed(db, { project = null, days = 30, now = Date.now
     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(now - 7 * 86400000, ...baseParams);
+  `).all(now - 7 * DAY_MS, ...baseParams);
 
   // Data health
   const tokenEst = db.prepare(`
@@ -60,7 +61,7 @@ export function computeStatsFeed(db, { project = null, days = 30, now = Date.now
   `).get(...baseParams);
   const avgImp = db.prepare(`SELECT AVG(COALESCE(importance,1)) as v FROM observations WHERE 1=1 ${projectFilter}`).get(...baseParams);
 
-  const thirtyDaysAgo = now - 30 * 86400000;
+  const thirtyDaysAgo = now - 30 * DAY_MS;
   // v3.23 noise-gauge de-blinding: `<= 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

package/lib/stats-quality.mjs

@@ -7,6 +7,7 @@ import { buildNotLowSignalSql } from './low-signal-patterns.mjs';
 import { truncate } from '../format-utils.mjs';
 import { COMPRESSED_PENDING_PURGE } from '../utils.mjs';
 
+import { DAY_MS } from './time-constants.mjs';
 // v3.42 F7: noise/low-signal ratios must divide by the LIVE (non-compressed) observation
 // count, NOT the all-rows total. Both numerators (lowVal, lowSignalTitle) already exclude
 // compressed rows (`compressed_into = 0`); dividing by an all-inclusive denominator
@@ -25,7 +26,7 @@ export function computeNoiseGauge({ liveTotal, lowValCount, lowSignalCount }) {
 export function computeQualityStats(db, { project, days }) {
   const projectFilter = project ? 'AND project = ?' : '';
   const baseParams = project ? [project] : [];
-  const cutoff = Date.now() - days * 86400000;
+  const cutoff = Date.now() - days * DAY_MS;
 
   // LOW_SIGNAL match = NOT notLowSignal. Pure title-only builder: this is a
   // METRIC counting pattern-titled rows ("Low-signal titles" in stats output),

package/lib/time-constants.mjs

@@ -0,0 +1,21 @@
+// lib/time-constants.mjs — millisecond time units, defined once.
+//
+// Four modules each carried their own `const DAY_MS` (deferred-work, metrics,
+// err-sampler, hook-telemetry) with ~60 more bare 86400000 literals scattered
+// across the runtime. The value cannot drift — that is not the problem. The
+// problem is that a reader auditing a window ("is this 30 days or 30 hours?")
+// had to re-derive the magic number at every site, and a grep for the policy
+// returned a number rather than a name.
+//
+// Zero imports on purpose: lib/metrics.mjs and lib/err-sampler.mjs are sinks
+// that must never crash or slow their callers, so anything they import has to
+// stay a leaf.
+
+// Kept internal rather than exported: only DAY_MS has runtime consumers today,
+// and an export born with zero importers is how the knip baseline drifted
+// 46→47 during v3.50–3.53 (DEFER_STALE_DAYS). Promote one when a caller needs it.
+const SECOND_MS = 1000;
+const MINUTE_MS = 60 * SECOND_MS;
+const HOUR_MS = 60 * MINUTE_MS;
+
+export const DAY_MS = 24 * HOUR_MS;

package/mem-cli.mjs

@@ -1197,7 +1197,7 @@ async function cmdStats(db, args) {
   // recorded in the last 24h. Surfaces silent breakage (DB corruption,
   // CC upstream field rename) that would otherwise stay invisible — the
   // failure mode that left code-graph's matcher bug undetected for 10 sessions.
-  const hookErrors24h = countRecentHookErrors(join(DB_DIR, 'runtime'), now - 86400000);
+  const hookErrors24h = countRecentHookErrors(join(DB_DIR, 'runtime'), now - DAY_MS);
 
   // M-9 (audit 2026-08-14): disk footprint — a "lite" store had accumulated 360MB of
   // pre-maintain snapshots against a 59MB DB with nothing reporting it. Cheap probes
@@ -1636,7 +1636,7 @@ function cmdExport(db, args) {
   if (flags.to) {
     exportToEpoch = new Date(flags.to).getTime();
     if (isNaN(exportToEpoch)) { fail(`[mem] Invalid --to date: "${flags.to}". Use YYYY-MM-DD or ISO 8601.`); return; }
-    if (/^\d{4}-\d{2}-\d{2}$/.test(flags.to)) exportToEpoch += 86400000 - 1;
+    if (/^\d{4}-\d{2}-\d{2}$/.test(flags.to)) exportToEpoch += DAY_MS - 1;
     wheres.push('created_at_epoch <= ?'); params.push(exportToEpoch);
   }
   if (exportFromEpoch !== null && exportToEpoch !== null && exportFromEpoch > exportToEpoch) {
@@ -1883,7 +1883,7 @@ function cmdCompress(db, args) {
     }
     ageDays = parsed;
   }
-  const cutoff = Date.now() - ageDays * 86400000;
+  const cutoff = Date.now() - ageDays * DAY_MS;
   const project = flags.project ? resolveProject(db, flags.project) : null;
 
   const candidates = selectCompressionCandidates(db, { cutoff, project });
@@ -2021,7 +2021,7 @@ function cmdMaintain(db, args) {
     }
     retainDays = parsed;
   }
-  const retainCutoff = Date.now() - retainDays * 86400000;
+  const retainCutoff = Date.now() - retainDays * DAY_MS;
   // purge_stale is the only DELETE here — require --confirm so a mis-typed run can't wipe rows.
   const confirmed = flags.confirm === true || flags.confirm === 'true';
 
@@ -2592,7 +2592,7 @@ function cmdCitationStats(db, args) {
   out(`Recently demoted (last ${days}d, importance ↓):`);
   if (demoted.length === 0) out('  (none)');
   for (const r of demoted) {
-    const ago = Math.round((Date.now() - r.demoted_at) / 86400000);
+    const ago = Math.round((Date.now() - r.demoted_at) / DAY_MS);
     out(`  #${r.id} [${r.type}] ${(r.title || '').slice(0, 60)}   imp=${r.importance}   ${ago}d ago`);
   }
 }
@@ -3140,6 +3140,7 @@ import { cmdDoctor } from './cli/doctor.mjs';
 // cmdActivity (T7 v2.31) extracted to cli/activity.mjs (v2.41 split).
 import { cmdActivity } from './cli/activity.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 // ─── Main Entry Point ────────────────────────────────────────────────────────
 
 export async function run(argv) {

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.65.0",
+  "version": "3.66.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "claude-mem-lite",
-      "version": "3.65.0",
+      "version": "3.66.1",
       "dependencies": {
         "@modelcontextprotocol/sdk": "^1.26.0",
         "better-sqlite3": "^12.6.2",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.65.0",
+  "version": "3.66.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).",
   "type": "module",
   "packageManager": "npm@10.9.2",
@@ -83,6 +83,8 @@
     "lib/release-digest.mjs",
     "lib/mem-override.mjs",
     "lib/injected-ids.mjs",
+    "lib/time-constants.mjs",
+    "lib/keyctx-marker.mjs",
     "lib/inject-search-core.mjs",
     "lib/get-core.mjs",
     "lib/browse-core.mjs",

package/registry-enricher.mjs

@@ -2,7 +2,7 @@
 // Sends resource content to Haiku for semantic metadata generation
 // Graceful degradation: failure preserves existing data
 
-import { callHaikuJSON } from './haiku-client.mjs';
+import { callHaikuJSON, BG_LLM_TIMEOUT_MS } from './haiku-client.mjs';
 import { truncate, debugCatch } from './utils.mjs';
 
 /**
@@ -84,7 +84,7 @@ export async function enrichResource(db, name, type, content) {
 
   try {
     const prompt = buildEnrichPrompt(name, content, existing);
-    const result = await callHaikuJSON(prompt, { timeout: 15000, maxTokens: 500 });
+    const result = await callHaikuJSON(prompt, { timeout: BG_LLM_TIMEOUT_MS, maxTokens: 500 });
 
     if (!result || !result.capability_summary) {
       db.prepare("UPDATE resources SET enrichment_status = 'failed' WHERE name = ? AND type = ?").run(name, type);

package/registry-recommend.mjs

@@ -8,6 +8,7 @@ import { join } from 'path';
 import { resolveDataDir } from './lib/resolve-data-dir.mjs';
 import { searchResources, cjkIntentTokens } from './registry-retriever.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 const VALID_MODES = new Set(['shadow', 'live', 'off']);
 
 /** Recommendation mode from env; default 'shadow', unknown → 'shadow'. */
@@ -137,7 +138,7 @@ export function gcOldShadowShards(retainDays = 90) {
   try {
     const dir = shadowDir();
     if (!existsSync(dir)) return 0;
-    const cutoff = new Date(Date.now() - retainDays * 86_400_000).toISOString().slice(0, 10);
+    const cutoff = new Date(Date.now() - retainDays * DAY_MS).toISOString().slice(0, 10);
     let removed = 0;
     for (const name of readdirSync(dir)) {
       const m = /^(\d{4}-\d{2}-\d{2})\.jsonl$/.exec(name);
@@ -155,7 +156,7 @@ export function logShadowAdoption(project, rec) { appendShadow({ ts: new Date().
 /** Yield parsed shadow rows from the last `days` daily shards. */
 export function* readShadowLog(days = 7) {
   for (let i = 0; i < days; i++) {
-    const d = new Date(Date.now() - i * 86_400_000).toISOString().slice(0, 10);
+    const d = new Date(Date.now() - i * DAY_MS).toISOString().slice(0, 10);
     let raw;
     try { raw = readFileSync(join(shadowDir(), `${d}.jsonl`), 'utf8'); } catch { continue; }
     for (const line of raw.split('\n')) { if (!line) continue; try { yield JSON.parse(line); } catch { /* skip malformed */ } }