claude-mem-lite 3.65.0 → 3.66.0

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;
@@ -342,16 +344,55 @@ export function extractInjectedFromFyi(transcriptPath, opts = {}) {
   return ids;
 }
 
+/**
+ * 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 enter this session's denominator. No
+ * coordinates → empty set, so transcript-only callers keep their old behaviour.
+ *
+ * @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 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.
+ * user-prompt-search FYI block + the SessionStart Key Context block. Single
+ * integration point the Stop handler calls.
  *
  * @param {string|null|undefined} transcriptPath
  * @param {object} [opts]
  * @param {boolean} [opts.mainOnly=false] Skip sidechain-injected IDs. The
  *   citation-decay caller passes true so the injected denominator matches the
  *   mainOnly cited numerator; the P4 access-bump caller omits it (broader).
+ * @param {string} [opts.runtimeDir] With `project`, enables the Key Context
+ *   face. Omitted by callers that only have a transcript path (computeCiteRecall
+ *   over an arbitrary file), which then see the four transcript-derived faces.
+ * @param {string} [opts.project]
+ * @param {string|null} [opts.sessionId]
  * @returns {Set<number>}
  */
 export function extractAllInjected(transcriptPath, opts = {}) {
@@ -360,6 +401,9 @@ export function extractAllInjected(transcriptPath, opts = {}) {
     ...extractInjectedFromUserPromptSubmit(transcriptPath, opts),
     ...extractInjectedFromErrorRecall(transcriptPath, opts),
     ...extractInjectedFromFyi(transcriptPath, opts),
+    // Marker-derived, not transcript-derived: no-op unless the caller passes
+    // runtimeDir + project (see extractInjectedFromKeyContext).
+    ...extractInjectedFromKeyContext(opts),
   ]);
 }
 
@@ -808,8 +852,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,71 @@
+// 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);
+  }
+
+  let bumped = 0;
+  if (db && clean.length > 0) {
+    try {
+      const now = Date.now();
+      // Mirrors hook-memory.mjs's UPS bump verbatim so the two surfaces feed the
+      // same counter with the same semantics. Per-row try/catch for FTS trigger
+      // safety (project_non_obvious.md).
+      const stmt = db.prepare(
+        'UPDATE observations SET injection_count = COALESCE(injection_count, 0) + 1, last_injected_at = ? WHERE id = ?'
+      );
+      for (const id of clean) {
+        try { stmt.run(now, id); bumped++; } catch { /* single-row failure must not drop the rest */ }
+      }
+    } catch (e) { debugCatch(e, 'keyctx-bump'); }
+  }
+
+  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.0",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "claude-mem-lite",
-      "version": "3.65.0",
+      "version": "3.66.0",
       "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.0",
   "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 */ } }

package/scoring-sql.mjs

@@ -3,6 +3,7 @@
 
 import { buildNotLowSignalSql } from './lib/low-signal-patterns.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 // ─── Why these multipliers exist (read before "simplifying" them) ────────────
 //
 // The recency-decay, type-quality, project-boost, importance, cite, and noise
@@ -33,14 +34,14 @@ import { buildNotLowSignalSql } from './lib/low-signal-patterns.mjs';
 
 /** Recency half-life per observation type (in milliseconds) */
 export const DECAY_HALF_LIFE_BY_TYPE = {
-  decision:  90 * 86400000,  // 90 days — architectural decisions persist
-  discovery: 60 * 86400000,  // 60 days — learned patterns last
-  feature:   30 * 86400000,  // 30 days — feature work is mid-range
-  bugfix:    14 * 86400000,  // 14 days — bugs are usually one-off
-  refactor:  14 * 86400000,  // 14 days — code cleanup
-  change:     7 * 86400000,  //  7 days — routine changes decay fast
+  decision:  90 * DAY_MS,  // 90 days — architectural decisions persist
+  discovery: 60 * DAY_MS,  // 60 days — learned patterns last
+  feature:   30 * DAY_MS,  // 30 days — feature work is mid-range
+  bugfix:    14 * DAY_MS,  // 14 days — bugs are usually one-off
+  refactor:  14 * DAY_MS,  // 14 days — code cleanup
+  change:     7 * DAY_MS,  //  7 days — routine changes decay fast
 };
-export const DEFAULT_DECAY_HALF_LIFE_MS = 14 * 86400000;
+export const DEFAULT_DECAY_HALF_LIFE_MS = 14 * DAY_MS;
 
 // ─── BM25 Weight Constants ──────────────────────────────────────────────────
 // Single source of truth for FTS5 BM25 weight expressions.

package/scripts/pre-tool-recall.js

@@ -20,6 +20,7 @@ import { recordMetric } from '../lib/metrics.mjs';
 import { presentIdents } from '../lib/lesson-idents.mjs';
 import { neutralizeContextDelimiters } from '../format-utils.mjs';
 
+import { DAY_MS } from '../lib/time-constants.mjs';
 // 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.
@@ -372,7 +373,7 @@ try {
     const fileMatch = fileMatchClause('of2');
     const fileParams = fileMatchParams(filePath);
     // 60-day lookback to avoid surfacing ancient observations
-    const cutoff = Date.now() - 60 * 86400000;
+    const cutoff = Date.now() - 60 * DAY_MS;
 
     // Surface actionable lessons first, then high-importance bugfix/decision observations.
     // Priority: 1) observations with lesson_learned (most actionable for preventing repeat bugs)

package/scripts/user-prompt-search.js

@@ -18,6 +18,7 @@ import { recommendSkill } from '../registry-recommend.mjs';
 import { recordHookError } from '../lib/hook-telemetry.mjs';
 import { atomicWriteFileSync } from '../lib/atomic-write.mjs';
 
+import { DAY_MS } from '../lib/time-constants.mjs';
 // ─── Constants ──────────────────────────────────────────────────────────────
 
 // Telemetry sink (lib/hook-telemetry.mjs contract): env override for tests, else
@@ -37,7 +38,7 @@ const injectedIdsFileFor = (sessionId) =>
 // gated by explicit "before/previously/记得" prompts where breadth is the
 // point). Env override for projects that want broader recall or to A/B.
 const MAX_RESULTS = Number(process.env.CLAUDE_MEM_UPS_MAX_RESULTS || 3);
-const LOOKBACK_MS = 60 * 86400000; // 60 days
+const LOOKBACK_MS = 60 * DAY_MS; // 60 days
 
 // v2.56.x: Past-similar-questions fallback row cap. Cut from 3 → 1 after
 // 30d transcript scan (#8062 follow-up, 2026-05-09) showed UPS prompt-fallback

package/search-scoring.mjs

@@ -11,6 +11,7 @@ import { porterStem } from './tfidf.mjs';
 import { CLI_INVOKE } from './cli-path.mjs';
 import { liveObsFilterSql } from './lib/inject-search-core.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 // ─── MCP Server Instructions Builder ───────────────────────────────────────
 // Phase A (v2.31.3+): when quiet=true, drops WHEN-TO-USE proactive-trigger and
 // Decision-rules sections; keeps the irreducible CLI/MCP tool list. Intended
@@ -302,7 +303,7 @@ export function runIdleCleanup(db) {
 
   db.transaction(() => {
     for (const { types, days } of staleThresholds) {
-      const cutoff = Date.now() - days * 86400000;
+      const cutoff = Date.now() - days * DAY_MS;
 
       const marked = db.prepare(`
         UPDATE observations SET compressed_into = ${COMPRESSED_PENDING_PURGE}

package/server.mjs

@@ -52,6 +52,7 @@ import { probeOtherSources as probeIdSources, bucketIdTokens, splitDeferredToken
 import { saveObservation } from './lib/save-observation.mjs';
 import { applyObsUpdate } from './lib/observation-write.mjs';
 import { EXPORT_COLUMNS_SQL } from './lib/export-columns.mjs';
+import { liveObsFilterSql } from './lib/inject-search-core.mjs';
 import { recallByFile } from './lib/recall-core.mjs';
 import { fetchRecent } from './lib/recent-core.mjs';
 import { AUTO_MERGE_THRESHOLD } from './lib/dedup-constants.mjs';
@@ -1032,7 +1033,7 @@ server.registerTool(
     if (args.project) args = { ...args, project: resolveProject(args.project) };
     const preview = args.preview !== false;
     const ageDays = args.age_days ?? 30;
-    const cutoff = Date.now() - ageDays * 86400000;
+    const cutoff = Date.now() - ageDays * DAY_MS;
     const candidates = selectCompressionCandidates(db, { cutoff, project: args.project || null });
 
     if (candidates.length === 0) {
@@ -1181,7 +1182,7 @@ server.registerTool(
         // only rows a PRIOR run marked (backed up); rows decay marks below wait one cycle.
         if (ops.includes('purge_stale')) {
           const retainDays = args.retain_days ?? 30;
-          const retainCutoff = Date.now() - retainDays * 86400000;
+          const retainCutoff = Date.now() - retainDays * DAY_MS;
           if (!purgeConfirmed) {
             // Dry-run preview (parity with CLI `maintain` without --confirm): the other
             // requested non-destructive ops still run below.
@@ -1687,8 +1688,12 @@ export async function handleExportForTest(db, args) {
 async function runExport(db, args) {
   const wheres = [];
   const params = [];
-  if (!args.include_compressed) wheres.push('COALESCE(compressed_into, 0) = 0');
-  wheres.push('superseded_at IS NULL');
+  // Composed, not hand-written: the CLI twin (mem-cli.mjs cmdExport) already
+  // routes through liveObsFilterSql, and a hand-rolled copy here is how the
+  // live-row predicate drifted apart on other surfaces (P2-11/D#123). With
+  // include_compressed the compressed half is dropped but retractions still
+  // are not — tombstone export is opt-in, supersession is never exported.
+  wheres.push(args.include_compressed ? 'superseded_at IS NULL' : liveObsFilterSql(''));
   if (args.project) { wheres.push('project = ?'); params.push(_resolveProjectShared(db, args.project)); }
   if (args.type) { wheres.push('type = ?'); params.push(args.type); }
   // T3-P1-A: surface invalid dates instead of silently dropping the filter — mirrors
@@ -1801,6 +1806,7 @@ server.registerTool(
 // Handler extracted to server/fts-check.mjs (v2.41 split).
 import { handleMemFtsCheck } from './server/fts-check.mjs';
 
+import { DAY_MS } from './lib/time-constants.mjs';
 server.registerTool(
   'mem_fts_check',
   {

package/source-files.mjs

@@ -123,6 +123,12 @@ export const SOURCE_FILES = [
   // scripts/user-prompt-search.js + scripts/pre-tool-recall.js. Under lib/ for
   // the same scripts-dir-rename reason as mem-override.mjs above.
   'lib/injected-ids.mjs',
+  // P2-13 (narrowed): millisecond time units, single-sourced from the four
+  // modules that each declared their own DAY_MS. Leaf module, zero imports.
+  'lib/time-constants.mjs',
+  // D#124: Key Context marker write + injection_count bump, shared by
+  // handleSessionStart and handlePreCompact. Statically imported by hook.mjs.
+  'lib/keyctx-marker.mjs',
   // P2-11: injection-side shared SQL core (live-row filter / clamped decay /
   // injection relevance chain). Statically imported by hook.mjs, hook-memory.mjs,
   // search-engine.mjs AND the standalone hook scripts — missing it from the