claude-mem-lite 2.36.0 → 2.38.0

package/.claude-plugin/marketplace.json

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

package/hook-memory.mjs

@@ -1,7 +1,7 @@
 // claude-mem-lite — Semantic Memory Injection
 // Search past observations for relevant memories to inject as context at user-prompt time.
 
-import { sanitizeFtsQuery, relaxFtsQueryToOr, debugCatch, OBS_BM25, notLowSignalTitleClause } from './utils.mjs';
+import { sanitizeFtsQuery, relaxFtsQueryToOr, debugCatch, OBS_BM25, notLowSignalTitleClause, noisePenaltyClause } from './utils.mjs';
 
 const MAX_MEMORY_INJECTIONS = 3;
 const MEMORY_LOOKBACK_MS = 60 * 86400000; // 60 days
@@ -42,9 +42,14 @@ export function searchRelevantMemories(db, userPrompt, project, excludeIds = [])
     // R1: notLowSignalTitleClause() excludes hook-llm fallback titles
     // ("Modified X", "Worked on X", "Reviewed N files:", raw error logs, etc.)
     // that almost never get referenced (3.3% access rate) but compete for BM25 rank.
+    // v26 P0: noise_penalty is multiplied AFTER sort-BM25 so the column used
+    // for ORDER BY stays the penalty-adjusted `relevance` applied downstream
+    // in JS (scored.sort). SELECT exposes both raw BM25 (for sort) and the
+    // penalty factor (for the final JS score).
     const selectStmt = db.prepare(`
       SELECT o.id, o.type, o.title, o.importance, o.lesson_learned, o.project,
-             ${OBS_BM25} as relevance
+             ${OBS_BM25} as relevance,
+             ${noisePenaltyClause('o')} as noise_penalty
       FROM observations_fts
       JOIN observations o ON o.id = observations_fts.rowid
       WHERE observations_fts MATCH ?
@@ -80,7 +85,8 @@ export function searchRelevantMemories(db, userPrompt, project, excludeIds = [])
     try {
       const crossStmt = db.prepare(`
         SELECT o.id, o.type, o.title, o.importance, o.lesson_learned, o.project,
-               ${OBS_BM25} as relevance
+               ${OBS_BM25} as relevance,
+               ${noisePenaltyClause('o')} as noise_penalty
         FROM observations_fts
         JOIN observations o ON o.id = observations_fts.rowid
         WHERE observations_fts MATCH ?
@@ -105,12 +111,14 @@ export function searchRelevantMemories(db, userPrompt, project, excludeIds = [])
 
     // Merge and score: same-project full weight, cross-project 0.7x
     // OR-fallback results get 0.4x penalty — they matched individual words, not the full intent
+    // v26 P0: noise_penalty (from SQL) shrinks high-inject/low-cite rows.
     const allRows = [...rows.map(r => ({ ...r, _or: usedOrFallback })), ...crossRows.map(r => ({ ...r, _or: crossUsedOr }))];
     const scored = allRows
       .filter(r => !excludeSet.has(r.id))
       .map(r => {
         const crossProjectPenalty = r.project === project ? 1.0 : 0.7;
         const orFallbackPenalty = r._or ? 0.4 : 1.0;
+        const noisePenalty = typeof r.noise_penalty === 'number' ? r.noise_penalty : 1.0;
         return {
           ...r,
           score: Math.abs(r.relevance)
@@ -118,7 +126,8 @@ export function searchRelevantMemories(db, userPrompt, project, excludeIds = [])
             * (r.lesson_learned ? 1.5 : 1.0)
             * (r.importance >= 2 ? 1.0 : 0.6)
             * crossProjectPenalty
-            * orFallbackPenalty,
+            * orFallbackPenalty
+            * noisePenalty,
         };
       })
       .sort((a, b) => b.score - a.score);
@@ -133,12 +142,19 @@ export function searchRelevantMemories(db, userPrompt, project, excludeIds = [])
     const aboveThreshold = scored.filter(r => r.score >= threshold);
     if (aboveThreshold.length === 0) return [];
 
-    // Update access_count for injected memories
+    // v26 P0: bump injection_count (NOT access_count) for injected rows.
+    // Before v26 this was bumping access_count, which conflated auto-injection
+    // with real cites/recalls/opens — polluting the noise-ratio signal the
+    // penalty clause now depends on. access_count is reserved for explicit
+    // access (cmdRecall/cmdGet/cmdTimeline/pre-tool-recall/citation-tracker).
+    // Per-row try/catch for FTS trigger safety (project_non_obvious.md).
     const result = aboveThreshold.slice(0, MAX_MEMORY_INJECTIONS);
     const now = Date.now();
-    const updateStmt = db.prepare('UPDATE observations SET access_count = COALESCE(access_count, 0) + 1, last_accessed_at = ? WHERE id = ?');
+    const bumpStmt = db.prepare(
+      'UPDATE observations SET injection_count = COALESCE(injection_count, 0) + 1, last_injected_at = ? WHERE id = ?'
+    );
     for (const r of result) {
-      updateStmt.run(now, r.id);
+      try { bumpStmt.run(now, r.id); } catch {}
     }
 
     return result;

package/lib/id-routing.mjs

@@ -0,0 +1,37 @@
+// Shared probe for "ID-not-found-in-requested-source" hints.
+// Used by CLI cmdGet (mem-cli.mjs) and MCP mem_get (server.mjs) so both
+// produce consistent redirect hints — if the probe schema drifts, both
+// paths update together.
+//
+// The formatter stays per-call-site because CLI and MCP surface format
+// differently (stderr vs response text); only the SQL layer is shared.
+
+/**
+ * Probe the observations / session_summaries / user_prompts tables for any
+ * of the given numeric IDs, excluding the sources the caller already queried.
+ *
+ * @param {import('better-sqlite3').Database} db
+ * @param {number[]} ids Numeric IDs to probe (non-negative ints).
+ * @param {Set<'obs'|'session'|'prompt'>} excludeSrcs Sources to skip.
+ * @returns {{obs:number[], session:number[], prompt:number[]}}
+ */
+export function probeOtherSources(db, ids, excludeSrcs) {
+  const result = { obs: [], session: [], prompt: [] };
+  if (!ids || ids.length === 0) return result;
+  const placeholders = ids.map(() => '?').join(',');
+  try {
+    if (!excludeSrcs.has('obs')) {
+      const hits = db.prepare(`SELECT id FROM observations WHERE id IN (${placeholders})`).all(...ids);
+      result.obs = hits.map(r => r.id);
+    }
+    if (!excludeSrcs.has('session')) {
+      const hits = db.prepare(`SELECT id FROM session_summaries WHERE id IN (${placeholders})`).all(...ids);
+      result.session = hits.map(r => r.id);
+    }
+    if (!excludeSrcs.has('prompt')) {
+      const hits = db.prepare(`SELECT id FROM user_prompts WHERE id IN (${placeholders})`).all(...ids);
+      result.prompt = hits.map(r => r.id);
+    }
+  } catch { /* best-effort hint; never block the caller */ }
+  return result;
+}

package/mem-cli.mjs

@@ -15,6 +15,7 @@ import { searchResources } from './registry-retriever.mjs';
 import { optimizePreview, optimizeRun } from './hook-optimize.mjs';
 import { buildSessionContextLines } from './hook-context.mjs';
 import { cmdAdopt, cmdUnadopt } from './adopt-cli.mjs';
+import { probeOtherSources as probeIdSources } from './lib/id-routing.mjs';
 import { basename } from 'path';
 import { readFileSync } from 'fs';
 
@@ -77,6 +78,30 @@ function fmtDateShort(iso) {
   return iso.slice(0, 10); // YYYY-MM-DD
 }
 
+// Parse an ID token from a command positional argument.
+// Accepts: `123`, `#123`, `P#123` / `p123` (prompt), `S#123` / `s123` (session).
+// Returns { source: 'obs'|'session'|'prompt'|null, id: number } or null if unparseable.
+// source===null means no explicit prefix — caller picks default (typically 'obs').
+function parseIdToken(raw) {
+  const m = /^([PpSs]?)#?(\d+)$/.exec(String(raw).trim());
+  if (!m) return null;
+  const p = m[1].toUpperCase();
+  const id = parseInt(m[2], 10);
+  if (!Number.isFinite(id) || id <= 0) return null;
+  const source = p === 'P' ? 'prompt' : p === 'S' ? 'session' : null;
+  return { source, id };
+}
+
+// Format the shared `probeIdSources` output as CLI hint strings.
+// Example: ["#5419 (obs)", "P#5417 (prompt)"] — callers join with "; ".
+function formatProbeHints(probe) {
+  const hints = [];
+  if (probe.obs.length > 0)     hints.push(`#${probe.obs.join(', #')} (obs)`);
+  if (probe.session.length > 0) hints.push(`S#${probe.session.join(', S#')} (session)`);
+  if (probe.prompt.length > 0)  hints.push(`P#${probe.prompt.join(', P#')} (prompt)`);
+  return hints;
+}
+
 // ─── Commands ────────────────────────────────────────────────────────────────
 
 function cmdSearch(db, args) {
@@ -571,57 +596,107 @@ function cmdRecall(db, args) {
   }
 }
 
+const OBS_FIELDS = ['id', 'type', 'title', 'subtitle', 'narrative', 'text', 'facts', 'concepts', 'lesson_learned', 'search_aliases', 'files_read', 'files_modified', 'project', 'created_at', 'memory_session_id', 'prompt_number', 'importance', 'related_ids', 'access_count', 'branch', 'superseded_at', 'superseded_by', 'last_accessed_at'];
+
+function renderObsRows(db, ids, requestedFields) {
+  const placeholders = ids.map(() => '?').join(',');
+  try {
+    db.prepare(`UPDATE observations SET access_count = COALESCE(access_count, 0) + 1, last_accessed_at = ? WHERE id IN (${placeholders})`).run(Date.now(), ...ids);
+    autoBoostIfNeeded(db, ids);
+  } catch { /* non-critical: FTS5 trigger may fail on corrupted index */ }
+
+  const rows = db.prepare(`SELECT * FROM observations WHERE id IN (${placeholders}) ORDER BY created_at_epoch ASC`).all(...ids);
+  if (rows.length === 0) return null;
+  const fields = requestedFields || OBS_FIELDS;
+  const parts = [];
+  for (const r of rows) {
+    const lines = [`#${r.id} [${r.type}] ${fmtDateShort(r.created_at)}`];
+    for (const f of fields) {
+      if (f === 'id' || f === 'type' || f === 'created_at') continue;
+      const val = r[f];
+      if (val === null || val === undefined || val === '') continue;
+      if (f === 'text' && r.narrative && typeof val === 'string' && val.startsWith(r.narrative)) continue;
+      const maxLen = f === 'narrative' ? 1000 : f === 'lesson_learned' ? 500 : f === 'text' ? 500 : 200;
+      const display = typeof val === 'string' && val.length > maxLen ? val.slice(0, maxLen) + '…' : val;
+      lines.push(`${f}: ${display}`);
+    }
+    parts.push(lines.join('\n'));
+  }
+  return { text: parts.join('\n\n'), count: rows.length };
+}
+
+function renderSessionRows(db, ids) {
+  const placeholders = ids.map(() => '?').join(',');
+  const rows = db.prepare(`SELECT * FROM session_summaries WHERE id IN (${placeholders}) ORDER BY created_at_epoch ASC`).all(...ids);
+  if (rows.length === 0) return null;
+  const parts = [];
+  for (const r of rows) {
+    const lines = [`S#${r.id} ${fmtDateShort(r.created_at)}`];
+    if (r.request) lines.push(`Request: ${r.request}`);
+    if (r.completed) lines.push(`Completed: ${r.completed}`);
+    if (r.investigated) lines.push(`Investigated: ${r.investigated}`);
+    if (r.learned) lines.push(`Learned: ${r.learned}`);
+    if (r.next_steps) lines.push(`Next steps: ${r.next_steps}`);
+    if (r.project) lines.push(`Project: ${r.project}`);
+    parts.push(lines.join('\n'));
+  }
+  return { text: parts.join('\n\n'), count: rows.length };
+}
+
+function renderPromptRows(db, ids) {
+  const placeholders = ids.map(() => '?').join(',');
+  const rows = db.prepare(`SELECT * FROM user_prompts WHERE id IN (${placeholders}) ORDER BY created_at_epoch ASC`).all(...ids);
+  if (rows.length === 0) return null;
+  const parts = [];
+  for (const r of rows) {
+    const lines = [`P#${r.id} ${fmtDateShort(r.created_at)}`];
+    if (r.prompt_text) lines.push(`Text: ${r.prompt_text}`);
+    if (r.content_session_id) lines.push(`Session: ${r.content_session_id}`);
+    parts.push(lines.join('\n'));
+  }
+  return { text: parts.join('\n\n'), count: rows.length };
+}
+
 function cmdGet(db, args) {
   const { positional, flags } = parseArgs(args);
   const idStr = positional.join(',');
   if (!idStr) {
-    fail('[mem] Usage: mem get <id1,id2,...> [--source obs|session|prompt] [--fields f1,f2,...]');
+    fail('[mem] Usage: mem get <id1,id2,...> [--source obs|session|prompt] [--fields f1,f2,...]\n' +
+         '        IDs accept prefix from search output: #123 (obs), P#123 (prompt), S#123 (session).');
     return;
   }
 
-  const ids = idStr.split(',').map(s => parseInt(s.trim(), 10)).filter(n => !isNaN(n));
-  if (ids.length === 0) {
+  const tokens = idStr.split(',').map(s => s.trim()).filter(Boolean);
+  const unparseable = [];
+  const parsed = [];
+  for (const t of tokens) {
+    const p = parseIdToken(t);
+    if (p) parsed.push(p);
+    else unparseable.push(t);
+  }
+  if (unparseable.length > 0) {
+    process.stderr.write(`[mem] Ignoring unparseable ID token(s): ${unparseable.join(', ')}\n`);
+  }
+  if (parsed.length === 0) {
     fail('[mem] No valid IDs provided');
     return;
   }
 
-  const source = flags.source || 'obs';
-  const placeholders = ids.map(() => '?').join(',');
-
-  if (source === 'session') {
-    const rows = db.prepare(`SELECT * FROM session_summaries WHERE id IN (${placeholders}) ORDER BY created_at_epoch ASC`).all(...ids);
-    if (rows.length === 0) { fail('[mem] No sessions found for given IDs'); return; }
-    const parts = [];
-    for (const r of rows) {
-      const lines = [`S#${r.id} ${fmtDateShort(r.created_at)}`];
-      if (r.request) lines.push(`Request: ${r.request}`);
-      if (r.completed) lines.push(`Completed: ${r.completed}`);
-      if (r.investigated) lines.push(`Investigated: ${r.investigated}`);
-      if (r.learned) lines.push(`Learned: ${r.learned}`);
-      if (r.next_steps) lines.push(`Next steps: ${r.next_steps}`);
-      if (r.project) lines.push(`Project: ${r.project}`);
-      parts.push(lines.join('\n'));
-    }
-    out(parts.join('\n\n'));
+  // Explicit --source overrides any prefix; otherwise each token's prefix routes individually.
+  const explicit = flags.source;
+  const validSources = new Set(['obs', 'session', 'prompt']);
+  if (explicit && !validSources.has(explicit)) {
+    fail(`[mem] Invalid --source "${explicit}". Use: obs, session, prompt`);
     return;
   }
 
-  if (source === 'prompt') {
-    const rows = db.prepare(`SELECT * FROM user_prompts WHERE id IN (${placeholders}) ORDER BY created_at_epoch ASC`).all(...ids);
-    if (rows.length === 0) { fail('[mem] No prompts found for given IDs'); return; }
-    const parts = [];
-    for (const r of rows) {
-      const lines = [`P#${r.id} ${fmtDateShort(r.created_at)}`];
-      if (r.prompt_text) lines.push(`Text: ${r.prompt_text}`);
-      if (r.content_session_id) lines.push(`Session: ${r.content_session_id}`);
-      parts.push(lines.join('\n'));
-    }
-    out(parts.join('\n\n'));
-    return;
+  const bySrc = { obs: [], session: [], prompt: [] };
+  for (const p of parsed) {
+    const src = explicit || p.source || 'obs';
+    bySrc[src].push(p.id);
   }
 
-  // Default: observations (aligned with MCP mem_get)
-  const OBS_FIELDS = ['id', 'type', 'title', 'subtitle', 'narrative', 'text', 'facts', 'concepts', 'lesson_learned', 'search_aliases', 'files_read', 'files_modified', 'project', 'created_at', 'memory_session_id', 'prompt_number', 'importance', 'related_ids', 'access_count', 'branch', 'superseded_at', 'superseded_by', 'last_accessed_at'];
+  // Validate --fields against obs schema (only meaningful for obs rows).
   let requestedFields = null;
   if (flags.fields) {
     const allRequested = flags.fields.split(',').map(s => s.trim());
@@ -636,50 +711,82 @@ function cmdGet(db, args) {
     }
   }
 
-  // Update access_count + auto-boost (aligned with MCP mem_get)
-  try {
-    db.prepare(`UPDATE observations SET access_count = COALESCE(access_count, 0) + 1, last_accessed_at = ? WHERE id IN (${placeholders})`).run(Date.now(), ...ids);
-    autoBoostIfNeeded(db, ids);
-  } catch { /* non-critical: FTS5 trigger may fail on corrupted index */ }
-
-  const rows = db.prepare(`
-    SELECT * FROM observations
-    WHERE id IN (${placeholders})
-    ORDER BY created_at_epoch ASC
-  `).all(...ids);
-
-  if (rows.length === 0) {
-    fail('[mem] No observations found for given IDs');
-    return;
+  const sections = [];
+  let totalFound = 0;
+  if (bySrc.obs.length > 0) {
+    const s = renderObsRows(db, bySrc.obs, requestedFields);
+    if (s) { sections.push(s.text); totalFound += s.count; }
+  }
+  if (bySrc.session.length > 0) {
+    const s = renderSessionRows(db, bySrc.session);
+    if (s) { sections.push(s.text); totalFound += s.count; }
+  }
+  if (bySrc.prompt.length > 0) {
+    const s = renderPromptRows(db, bySrc.prompt);
+    if (s) { sections.push(s.text); totalFound += s.count; }
   }
 
-  const fields = requestedFields || OBS_FIELDS;
-  const parts = [];
-  for (const r of rows) {
-    const lines = [`#${r.id} [${r.type}] ${fmtDateShort(r.created_at)}`];
-    for (const f of fields) {
-      if (f === 'id' || f === 'type' || f === 'created_at') continue; // already in header
-      const val = r[f];
-      if (val === null || val === undefined || val === '') continue;
-      // Skip 'text' field when it duplicates narrative (aligned with MCP mem_get)
-      if (f === 'text' && r.narrative && typeof val === 'string' && val.startsWith(r.narrative)) continue;
-      const maxLen = f === 'narrative' ? 1000 : f === 'lesson_learned' ? 500 : f === 'text' ? 500 : 200;
-      const display = typeof val === 'string' && val.length > maxLen ? val.slice(0, maxLen) + '…' : val;
-      lines.push(`${f}: ${display}`);
-    }
-    parts.push(lines.join('\n'));
+  if (totalFound === 0) {
+    // Probe the OTHER sources so the caller can retry with the right prefix.
+    const queried = new Set(Object.entries(bySrc).filter(([, v]) => v.length > 0).map(([k]) => k));
+    const allIds = parsed.map(p => p.id);
+    const probe = probeIdSources(db, allIds, queried);
+    const hits = formatProbeHints(probe);
+    const hint = hits.length > 0 ? ` Try: ${hits.join('; ')}.` : '';
+    const queriedList = [...queried].join(', ');
+    fail(`[mem] No records found in source(s) [${queriedList}] for the given ID(s).${hint}`);
+    return;
   }
 
-  out(parts.join('\n\n'));
+  out(sections.join('\n\n'));
 }
 
 function cmdTimeline(db, args) {
   const { positional, flags } = parseArgs(args);
-  let anchorId = parseInt(flags.anchor, 10);
   const before = parseInt(flags.before, 10) || 5;
   const after = parseInt(flags.after, 10) || 5;
   const project = flags.project ? resolveProject(db, flags.project) : null;
 
+  // Parse --anchor, accepting P#/S#/# prefix so callers can paste search-result IDs verbatim.
+  // For prompt/session anchors, resolve to the nearest-in-time observation so timeline semantics
+  // (before/after observations) still apply.
+  let anchorId = null;
+  let anchorNote = null; // hint line for output when anchor was resolved via conversion
+  if (flags.anchor !== undefined && flags.anchor !== true) {
+    const parsed = parseIdToken(flags.anchor);
+    if (!parsed) {
+      fail(`[mem] Invalid --anchor "${flags.anchor}". Expected N, #N, P#N, or S#N.`);
+      return;
+    }
+    if (parsed.source === 'prompt') {
+      const row = db.prepare('SELECT created_at_epoch FROM user_prompts WHERE id = ?').get(parsed.id);
+      if (!row) { fail(`[mem] Prompt P#${parsed.id} not found`); return; }
+      const proj = project;
+      const nearest = db.prepare(`
+        SELECT id FROM observations
+        WHERE COALESCE(compressed_into, 0) = 0 ${proj ? 'AND project = ?' : ''}
+        ORDER BY ABS(created_at_epoch - ?) ASC LIMIT 1
+      `).get(...(proj ? [proj, row.created_at_epoch] : [row.created_at_epoch]));
+      if (!nearest) { fail(`[mem] No observations near P#${parsed.id}`); return; }
+      anchorId = nearest.id;
+      anchorNote = `(anchored to #${nearest.id}, closest obs to P#${parsed.id})`;
+    } else if (parsed.source === 'session') {
+      const row = db.prepare('SELECT created_at_epoch FROM session_summaries WHERE id = ?').get(parsed.id);
+      if (!row) { fail(`[mem] Session S#${parsed.id} not found`); return; }
+      const proj = project;
+      const nearest = db.prepare(`
+        SELECT id FROM observations
+        WHERE COALESCE(compressed_into, 0) = 0 ${proj ? 'AND project = ?' : ''}
+        ORDER BY ABS(created_at_epoch - ?) ASC LIMIT 1
+      `).get(...(proj ? [proj, row.created_at_epoch] : [row.created_at_epoch]));
+      if (!nearest) { fail(`[mem] No observations near S#${parsed.id}`); return; }
+      anchorId = nearest.id;
+      anchorNote = `(anchored to #${nearest.id}, closest obs to S#${parsed.id})`;
+    } else {
+      anchorId = parsed.id;
+    }
+  }
+
   // Support query-based anchor: `timeline --query "search terms"` or positional
   // Uses recency-weighted BM25 + project filter (aligned with MCP mem_timeline)
   const queryStr = flags.query || positional.join(' ');
@@ -773,7 +880,7 @@ function cmdTimeline(db, args) {
 
   const all = [...beforeRows.reverse(), anchor, ...afterRows];
 
-  out(`[mem] Timeline around #${anchorId}:`);
+  out(`[mem] Timeline around #${anchorId}${anchorNote ? ' ' + anchorNote : ''}:`);
   for (const r of all) {
     const marker = r.id === anchorId ? ' <--' : '';
     const time = relativeTime(r.created_at_epoch);
@@ -1162,7 +1269,19 @@ function cmdDelete(db, args) {
     return;
   }
 
-  const ids = idStr.split(',').map(s => parseInt(s.trim(), 10)).filter(n => !isNaN(n));
+  // delete operates on observations only. Reject P#/S# explicitly so callers aren't
+  // surprised by silent NaN filtering when they paste search-output IDs.
+  const tokens = idStr.split(',').map(s => s.trim()).filter(Boolean);
+  const nonObs = tokens.filter(t => /^[PpSs]#?\d+$/.test(t));
+  if (nonObs.length > 0) {
+    fail(`[mem] delete only works on observations. Rejected: ${nonObs.join(', ')}. ` +
+         `Prompts and sessions are append-only — inspect with \`mem get P#N --source prompt\` / \`--source session\`.`);
+    return;
+  }
+  const ids = tokens.map(t => {
+    const p = parseIdToken(t);
+    return p && p.source === null ? p.id : NaN;
+  }).filter(n => !isNaN(n));
   if (ids.length === 0) {
     fail('[mem] No valid IDs provided');
     return;
@@ -1215,7 +1334,14 @@ function cmdDelete(db, args) {
 
 function cmdUpdate(db, args) {
   const { positional, flags } = parseArgs(args);
-  const id = parseInt(positional[0], 10);
+  const raw = positional[0];
+  if (raw && /^[PpSs]#?\d+$/.test(String(raw).trim())) {
+    fail(`[mem] update only works on observations. Rejected: ${raw}. ` +
+         `Prompts and sessions are append-only.`);
+    return;
+  }
+  const parsed = raw ? parseIdToken(raw) : null;
+  const id = parsed && parsed.source === null ? parsed.id : parseInt(raw, 10);
   if (!id || isNaN(id)) {
     fail('[mem] Usage: mem update <id> [--title T] [--type T] [--importance N] [--lesson T] [--narrative T] [--concepts T]');
     return;
@@ -1919,11 +2045,14 @@ Commands:
     --limit N           Max results (default 10)
 
   get <id1,id2,...>     Get full details by ID
-    --source S          Record type: obs (default), session, prompt
-    --fields f1,f2,...  Select specific fields to return
+    IDs accept search-output prefixes: #123 (obs), P#123 (prompt), S#123 (session).
+    Bare N defaults to obs. Mixed prefixes in one call route each token correctly.
+    --source S          Force record type (obs|session|prompt); overrides prefixes.
+    --fields f1,f2,...  Select specific fields to return (observations only).
 
   timeline              Show observations around an anchor (shows recent if no anchor)
-    --anchor ID         Center on this observation ID
+    --anchor ID         Center on this ID. Accepts N, #N, P#N, or S#N — P#/S# anchors
+                        resolve to the nearest-in-time observation in the same project.
     --query "text"      Find anchor by FTS5 search
     --before N          Show N before anchor (default 5)
     --after N           Show N after anchor (default 5)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.36.0",
+  "version": "2.38.0",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "engines": {
@@ -51,6 +51,7 @@
     "lib/stats-quality.mjs",
     "lib/low-signal-patterns.mjs",
     "lib/citation-tracker.mjs",
+    "lib/id-routing.mjs",
     "registry.mjs",
     "registry-retriever.mjs",
     "registry-indexer.mjs",

package/schema.mjs

@@ -13,7 +13,7 @@ export const DB_PATH = join(DB_DIR, 'claude-mem-lite.db');
 export const REGISTRY_DB_PATH = join(DB_DIR, 'resource-registry.db');
 
 // Increment when schema changes (tables, columns, indexes, FTS, migrations)
-export const CURRENT_SCHEMA_VERSION = 25;
+export const CURRENT_SCHEMA_VERSION = 26;
 
 const CORE_SCHEMA = `
   CREATE TABLE IF NOT EXISTS sdk_sessions (
@@ -112,6 +112,13 @@ const MIGRATIONS = [
   'ALTER TABLE observations ADD COLUMN superseded_by INTEGER DEFAULT NULL',
   'ALTER TABLE observations ADD COLUMN last_accessed_at INTEGER DEFAULT NULL',
   'ALTER TABLE observations ADD COLUMN optimized_at INTEGER DEFAULT NULL',
+  // v26 (P0 injection-noise): per-obs injection tracking for noise-ratio
+  // penalty. injection_count bumps only on UserPromptSubmit / hook-memory
+  // auto-injection (not on explicit recall/get/timeline — those keep bumping
+  // access_count). Pair with access_count to compute noise ratio: high
+  // injection_count + low access_count = low-signal, deprioritize.
+  'ALTER TABLE observations ADD COLUMN injection_count INTEGER NOT NULL DEFAULT 0',
+  'ALTER TABLE observations ADD COLUMN last_injected_at INTEGER DEFAULT NULL',
 ];
 
 /**

package/scoring-sql.mjs

@@ -61,6 +61,44 @@ export const TYPE_QUALITY_CASE = `(
   END
 )`;
 
+/**
+ * Noise-ratio penalty: deprioritizes observations that get auto-injected often
+ * but rarely "used" (cited via Stop-hook citation tracker, or explicitly
+ * recalled/opened via pre-tool-recall / cmdRecall / cmdGet / cmdTimeline).
+ *
+ * Signal sources:
+ *   - injection_count: bumped ONLY on UserPromptSubmit / hook-memory auto-inject
+ *   - access_count: bumped on citation (c039352 P4), explicit recall, get, timeline
+ *
+ * Empirical thresholds (see docs/p0-injection-noise-baseline.txt, 53 transcripts):
+ *   • High-noise legitimate use (#5597 29/10=2.9x): kept at 1.0× (below tier-1)
+ *   • Moderate noise (#4352 44/9=4.89x): drops to 0.5× (tier-1 hit)
+ *   • Pure noise (#4046 14/0=inf): drops to 0.5× (tier-1; count≥10 gate protects
+ *     cold-start obs with legitimately no cites yet)
+ *   • Entrenched noise (≥20 inject, ≥5× ratio): drops to 0.2× (tier-2)
+ *
+ * Applied as: BM25 × time_decay × TYPE_QUALITY × (0.5 + 0.5·importance) × NOISE_PENALTY
+ * Note: multiplicative so ORDER BY relevance ASC (negative scores) still works —
+ * penalty shrinks magnitude, making the row less preferable.
+ *
+ * @param {string} [alias='o'] Table alias for the observations row.
+ * @returns {string} SQL CASE expression (already parenthesized).
+ */
+export function noisePenaltyClause(alias = 'o') {
+  const a = alias ? `${alias}.` : '';
+  return `(
+    CASE
+      WHEN COALESCE(${a}injection_count, 0) >= 20
+        AND COALESCE(${a}injection_count, 0) > COALESCE(${a}access_count, 0) * 5
+        THEN 0.2
+      WHEN COALESCE(${a}injection_count, 0) >= 10
+        AND COALESCE(${a}injection_count, 0) > COALESCE(${a}access_count, 0) * 3
+        THEN 0.5
+      ELSE 1.0
+    END
+  )`;
+}
+
 /**
  * SQL WHERE clause fragment excluding LOW_SIGNAL degraded titles — the fallback
  * titles hook-llm.mjs writes when Haiku summarization is unavailable or skipped

package/scripts/user-prompt-search.js

@@ -4,7 +4,7 @@
 // Lightweight: only imports schema.mjs and utils.mjs, no MCP SDK
 
 import { ensureDb, DB_DIR, REGISTRY_DB_PATH } from '../schema.mjs';
-import { sanitizeFtsQuery, relaxFtsQueryToOr, truncate, typeIcon, inferProject, OBS_BM25, TYPE_DECAY_CASE, TYPE_QUALITY_CASE, notLowSignalTitleClause } from '../utils.mjs';
+import { sanitizeFtsQuery, relaxFtsQueryToOr, truncate, typeIcon, inferProject, OBS_BM25, TYPE_DECAY_CASE, TYPE_QUALITY_CASE, notLowSignalTitleClause, noisePenaltyClause } from '../utils.mjs';
 import { writeFileSync, readFileSync, existsSync, renameSync } from 'fs';
 import { join } from 'path';
 import Database from 'better-sqlite3';
@@ -87,12 +87,16 @@ function searchByFts(db, queryText, project, limit, typeFilter) {
   const now = Date.now();
   // R1: notLowSignalTitleClause() excludes hook-llm degraded titles
   // ("Modified X", "Worked on X", "Reviewed N files:", raw error logs).
+  // v26 P0: noise penalty shrinks relevance magnitude for obs with high
+  // inject:access ratio (auto-injected often, never cited/opened). See
+  // docs/p0-injection-noise-baseline.txt.
   const sql = `
     SELECT o.id, o.type, o.title, o.lesson_learned,
            ${OBS_BM25}
              * (1.0 + EXP(-0.693 * (? - o.created_at_epoch) / ${TYPE_DECAY_CASE}))
              * ${TYPE_QUALITY_CASE}
-             * (0.5 + 0.5 * COALESCE(o.importance, 1)) as relevance
+             * (0.5 + 0.5 * COALESCE(o.importance, 1))
+             * ${noisePenaltyClause('o')} as relevance
     FROM observations_fts
     JOIN observations o ON o.id = observations_fts.rowid
     WHERE observations_fts MATCH ?
@@ -460,6 +464,22 @@ async function main() {
           count: prevCount + 1,
         }));
       } catch {}
+      // v26 P0: bump injection_count for obs-based emits only (prompt-corpus
+      // rows have "P<id>" string IDs; skip those — they live in user_prompts).
+      // Per-row try/catch: observations_au trigger reinserts FTS on any UPDATE
+      // (project_non_obvious.md); an FTS corruption on one row must not abort
+      // counter bumps for other rows.
+      if (rows.length > 0) {
+        try {
+          const now = Date.now();
+          const bumpStmt = db.prepare(
+            'UPDATE observations SET injection_count = COALESCE(injection_count, 0) + 1, last_injected_at = ? WHERE id = ?'
+          );
+          for (const r of rows) {
+            try { bumpStmt.run(now, r.id); } catch {}
+          }
+        } catch {}
+      }
     }
 
     // ─── L1: Registry skill pointer (T4 v2.31) ──────────────────────────

package/server.mjs

@@ -27,6 +27,7 @@ import { basename, join } from 'path';
 import { homedir } from 'os';
 import { ensureRegistryDb, upsertResource } from './registry.mjs';
 import { searchResources } from './registry-retriever.mjs';
+import { probeOtherSources as probeIdSources } from './lib/id-routing.mjs';
 import { getVocabulary, rebuildVocabulary, _resetVocabCache, computeVector, vectorSearch, rrfMerge } from './tfidf.mjs';
 import { createRequire } from 'module';
 
@@ -898,17 +899,14 @@ server.registerTool(
     }
 
     if (rows.length === 0) {
-      // P2-7: for source=session/prompt, check whether the IDs exist as observations so the
-      // caller can switch source instead of chasing a phantom miss.
-      let hint = '';
-      if (source === 'session' || source === 'prompt') {
-        try {
-          const obsHits = db.prepare(`SELECT id FROM observations WHERE id IN (${placeholders})`).all(...args.ids);
-          if (obsHits.length > 0) {
-            hint = ` These ID(s) exist as observations: ${obsHits.map(r => r.id).join(', ')}. Try source='obs'.`;
-          }
-        } catch { /* best-effort hint */ }
-      }
+      // Symmetric probe via shared lib/id-routing.mjs so CLI cmdGet and MCP mem_get
+      // stay aligned if a table's ID semantics change.
+      const probe = probeIdSources(db, args.ids, new Set([source]));
+      const hints = [];
+      if (probe.obs.length > 0)     hints.push(`#${probe.obs.join(', #')} (obs — use source='obs')`);
+      if (probe.session.length > 0) hints.push(`S#${probe.session.join(', S#')} (session — use source='session')`);
+      if (probe.prompt.length > 0)  hints.push(`P#${probe.prompt.join(', P#')} (prompt — use source='prompt')`);
+      const hint = hints.length > 0 ? ` Try: ${hints.join('; ')}.` : '';
       const msg = `No ${sourceLabel} found for given IDs.${hint}`;
       return { content: [{ type: 'text', text: fieldsNote ? `${msg}\n\n${fieldsNote}` : msg }] };
     }

package/source-files.mjs

@@ -38,6 +38,7 @@ export const SOURCE_FILES = [
   'lib/stats-quality.mjs',
   'lib/low-signal-patterns.mjs',
   'lib/citation-tracker.mjs',
+  'lib/id-routing.mjs',
   // v2.32 invited-memory: memdir primitives + adopt/unadopt CLI
   'memdir.mjs',
   'adopt-content.mjs',

package/tool-schemas.mjs

@@ -275,7 +275,9 @@ export const tools = [
       '  - A mem_search hit looks relevant and you need the supporting detail\n' +
       '  - For session (S#) or prompt (P#) hits, pass source="session" or "prompt"\n' +
       '\n' +
-      'Equivalent CLI: claude-mem-lite get <id>[,<id>,...]',
+      'On miss, response includes "Try: …" hint listing other sources the ID lives in.\n' +
+      '\n' +
+      'Equivalent CLI: claude-mem-lite get <id>[,<id>,...] — accepts P#/S#/# prefix.',
     inputSchema: memGetSchema,
   },
   {

package/utils.mjs

@@ -9,7 +9,7 @@ import { buildLowSignalRegex } from './lib/low-signal-patterns.mjs';
 // ─── Re-exports from extracted modules ──────────────────────────────────────
 // Backward compatibility: all consumers import from utils.mjs
 
-export { DECAY_HALF_LIFE_BY_TYPE, DEFAULT_DECAY_HALF_LIFE_MS, OBS_BM25, SESS_BM25, TYPE_DECAY_CASE, TYPE_QUALITY_CASE, OBS_FTS_COLUMNS, notLowSignalTitleClause } from './scoring-sql.mjs';
+export { DECAY_HALF_LIFE_BY_TYPE, DEFAULT_DECAY_HALF_LIFE_MS, OBS_BM25, SESS_BM25, TYPE_DECAY_CASE, TYPE_QUALITY_CASE, OBS_FTS_COLUMNS, notLowSignalTitleClause, noisePenaltyClause } from './scoring-sql.mjs';
 export { cjkBigrams, extractCjkSynonymTokens, extractCjkKeywords, extractCjkLikePatterns, SYNONYM_MAP, expandToken, sanitizeFtsQuery, relaxFtsQueryToOr, FTS_STOP_WORDS, CJK_COMPOUNDS } from './nlp.mjs';
 export { resolveProject, _resetProjectCache } from './project-utils.mjs';
 export { scrubSecrets, SECRET_PATTERNS } from './secret-scrub.mjs';