claude-mem-lite 3.58.2 → 3.59.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.58.2",
+      "version": "3.59.1",
       "source": "./",
       "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)."
     }

package/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.58.2",
+  "version": "3.59.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).",
   "author": {
     "name": "sdsrss"

package/cli/common.mjs

@@ -89,6 +89,37 @@ export function rejectBareStringFlags(flags, keys) {
   return false;
 }
 
+/**
+ * Resolve a required value that may arrive positionally or via an MCP-field flag alias.
+ * LLM callers map the MCP tool schema onto flags (#233): mem_save.content → `--content`,
+ * mem_defer.title → `defer add --title`, mem_search.query → `--query`, mem_get.ids →
+ * `--ids` — each previously fell to a stderr-only usage line that a `2>/dev/null`
+ * caller reads as "CLI doesn't support this".
+ *
+ * Returns the resolved string ('' when neither shape is present — caller emits its own
+ * usage). On ambiguity (positional AND an alias, or two aliases at once) emits fail()
+ * and returns null — caller must `return` on null. Bare alias flags (boolean true) are
+ * ignored here; guard them with rejectBareStringFlags BEFORE calling.
+ *
+ * @param {string} positionalStr Joined positional tokens (caller picks the separator).
+ * @param {object} flags Parsed flags from parseArgs.
+ * @param {string[]} aliasKeys Alias flag names, first match wins (without dashes).
+ * @returns {string|null} Resolved value, or null after a conflict fail().
+ */
+export function resolvePositionalAlias(positionalStr, flags, aliasKeys) {
+  const given = aliasKeys.filter(k => typeof flags[k] === 'string' && flags[k].trim() !== '');
+  if (given.length > 1) {
+    fail(`[mem] Both --${given[0]} and --${given[1]} provided — pass the value once.`);
+    return null;
+  }
+  const flagVal = given.length === 1 ? flags[given[0]] : '';
+  if (positionalStr.trim() !== '' && flagVal.trim() !== '') {
+    fail(`[mem] Value given both positionally and via --${given[0]} — pass it once.`);
+    return null;
+  }
+  return positionalStr.trim() !== '' ? positionalStr : flagVal;
+}
+
 // ─── Unknown-flag typo guard ─────────────────────────────────────────────────
 
 /**
@@ -103,12 +134,12 @@ export const KNOWN_CLI_FLAGS = new Set([
   'after', 'age-days', 'all', 'anchor', 'batch', 'before', 'benchmark', 'body', 'branch',
   'capability-summary', 'category', 'closes-deferred', 'concepts', 'confirm', 'days', 'deep',
   'detail', 'domain-tags', 'dry-run', 'enrich', 'execute', 'fields', 'file', 'files', 'floors',
-  'force', 'format', 'from', 'has', 'help', 'importance', 'include-compressed', 'include-noise',
+  'force', 'format', 'from', 'has', 'help', 'id', 'ids', 'content', 'importance', 'include-compressed', 'include-noise',
   'intent-tags', 'invocation-name', 'json', 'key', 'keywords', 'lesson', 'lesson-learned', 'limit',
   'local-path', 'margins', 'max', 'memdir', 'merge-ids', 'metrics', 'name', 'narrative', 'no-deep',
   'offset', 'ops', 'or', 'out', 'priority', 'project', 'quality', 'query', 'reason', 'repo-url',
   'rerank', 'resource-type', 'retain-days', 'retry', 'run', 'run-all', 'scope', 'session-audit',
-  'sidechain', 'since', 'sort', 'source', 'status', 'sweep', 'task', 'tech-stack', 'tier', 'title',
+  'sidechain', 'since', 'sort', 'source', 'status', 'sweep', 'task', 'tech-stack', 'text', 'tier', 'title',
   'to', 'trigger-patterns', 'type', 'use-cases', 'verbose',
 ]);
 

package/lib/activity.mjs

@@ -7,7 +7,9 @@
 import { sanitizeFtsQuery } from '../utils.mjs';
 import { scrubRecord } from './scrub-record.mjs';
 import { saveObservation } from './save-observation.mjs';
-import { notLowSignalTitleClause } from '../scoring-sql.mjs';
+// Pure title-only builder: this query runs on the EVENTS table, which has no
+// lesson_learned column — the lesson-escape variant would be a SQL error here.
+import { buildNotLowSignalSql } from './low-signal-patterns.mjs';
 import { OBS_TYPE_SET } from './obs-types.mjs';
 
 // Observation types (mirrors the observations.type enum) — events carry a wider
@@ -150,7 +152,7 @@ export function promoteInsightEvents(db, { project = null, minImportance = 2, ex
     FROM events
     WHERE body IS NOT NULL AND TRIM(body) != '' AND importance >= ?
       AND superseded_at_epoch IS NULL
-      AND ${notLowSignalTitleClause('')}
+      AND ${buildNotLowSignalSql('')}
       ${projClause}
     ORDER BY created_at_epoch DESC LIMIT ?
   `;

package/lib/low-signal-patterns.mjs

@@ -50,13 +50,29 @@ export function buildLowSignalRegex() {
  * Build the SQL NOT LIKE clause chain, optionally prefixed with a table alias.
  * Output is a single parenthesized AND-chain — safe to combine with other AND/OR.
  *
+ * `lessonEscape` (2026-07-24 audit P1, D#11): retrieval consumers on the
+ * observations table pass `{ lessonEscape: true }` to admit rows whose title
+ * matches a LOW_SIGNAL pattern but which carry a real lesson_learned — the
+ * read-side counterpart of the isNoiseObservation/capNoiseImportance write-side
+ * signal escapes. Without it, a substantive obs titled "npm pack drops …" is
+ * unsearchable forever. Escape is lesson-only by design: an importance>=2
+ * escape would resurrect pre-v2.47 Haiku-inflated noise on legacy DBs.
+ * Default stays title-only because two consumers query the events table
+ * (no lesson_learned column) and two are noise-title METRICS, not filters.
+ *
  * @param {string} [alias=''] Table alias (e.g. 'o') — empty for unqualified.
+ * @param {object} [opts]
+ * @param {boolean} [opts.lessonEscape=false] Admit rows with non-empty, non-'none' lesson_learned.
  * @returns {string} SQL boolean expression
  */
-export function buildNotLowSignalSql(alias = '') {
+export function buildNotLowSignalSql(alias = '', { lessonEscape = false } = {}) {
   const p = alias ? `${alias}.` : '';
   const clauses = LOW_SIGNAL_PATTERNS.map(({ like }) => `${p}title NOT LIKE '${like}'`);
-  return '(\n    ' + clauses.join('\n    AND ') + '\n  )';
+  const chain = '(\n    ' + clauses.join('\n    AND ') + '\n  )';
+  if (!lessonEscape) return chain;
+  // Mirrors the write-side lesson test: String(lesson).trim().toLowerCase() not in ('', 'none').
+  const escape = `(${p}lesson_learned IS NOT NULL AND LOWER(TRIM(${p}lesson_learned)) NOT IN ('', 'none'))`;
+  return `(${chain} OR ${escape})`;
 }
 
 // Cached singleton — isNoiseObservation is called once per observation insert.

package/lib/stats-quality.mjs

@@ -3,7 +3,7 @@
 // Splits pure data aggregation from text rendering so MCP handlers don't
 // collide with CLI's `out()` stdout-write pattern.
 
-import { notLowSignalTitleClause } from '../scoring-sql.mjs';
+import { buildNotLowSignalSql } from './low-signal-patterns.mjs';
 import { truncate } from '../format-utils.mjs';
 import { COMPRESSED_PENDING_PURGE } from '../utils.mjs';
 
@@ -27,9 +27,11 @@ export function computeQualityStats(db, { project, days }) {
   const baseParams = project ? [project] : [];
   const cutoff = Date.now() - days * 86400000;
 
-  // LOW_SIGNAL match = NOT notLowSignal. Shared helper keeps SQL in sync
-  // with scoring-sql.mjs and pre-tool-recall.js Edit-fallback filter.
-  const lowSignalIsMatchExpr = `NOT ${notLowSignalTitleClause('')}`;
+  // LOW_SIGNAL match = NOT notLowSignal. Pure title-only builder: this is a
+  // METRIC counting pattern-titled rows ("Low-signal titles" in stats output),
+  // not a retrieval filter — the lesson-escape variant would silently exclude
+  // lesson-bearing rows from the count and understate title degradation.
+  const lowSignalIsMatchExpr = `NOT ${buildNotLowSignalSql('')}`;
 
   // Narrative-text proxy for bugfix investigations that never landed a fix.
   const unresolvedNarrativeExpr = `(

package/mem-cli.mjs

@@ -40,7 +40,7 @@ import { readFileSync, existsSync, readdirSync } from 'fs';
 // v2.41: shared CLI helpers extracted to cli/common.mjs. Keep this file as the
 // router + remaining-command bodies during the incremental split. Future work:
 // move each cmdXxx into its own cli/<cmd>.mjs; mem-cli.mjs becomes pure dispatch.
-import { parseArgs, out, fail, relativeTime, fmtDateShort, parseIdToken, formatProbeHints, rejectBareStringFlags, suggestUnknownFlags, OBS_TIME_FIELDS, formatObsFieldValue } from './cli/common.mjs';
+import { parseArgs, out, fail, relativeTime, fmtDateShort, parseIdToken, formatProbeHints, rejectBareStringFlags, resolvePositionalAlias, suggestUnknownFlags, OBS_TIME_FIELDS, formatObsFieldValue } from './cli/common.mjs';
 import { saveObservation } from './lib/save-observation.mjs';
 import { rebuildObservationDerived, normalizeScope, insertObservationVector } from './lib/observation-write.mjs';
 import { EXPORT_COLUMNS_SQL } from './lib/export-columns.mjs';
@@ -65,16 +65,18 @@ import { shouldQueueSaveEnrich, queueSaveEnrich } from './lib/save-enrich.mjs';
 
 async function cmdSearch(db, args, { llm } = {}) {
   const { positional, flags } = parseArgs(args);
-  const query = positional.join(' ');
-  if (!query) {
-    fail('[mem] Usage: claude-mem-lite search <query> [--type TYPE] [--source SOURCE] [--limit N] [--project P] [--from DATE] [--to DATE] [--since DUR] [--importance N] [--branch B] [--offset N] [--sort relevance|time|importance] [--include-noise] [--deep] [--no-deep] [--rerank]');
-    return;
-  }
 
   // Bare string flags parse to boolean `true`; without this guard `--branch` reaches
   // the SQLite bind and crashes, while `--to`/`--project` silently change results
   // (epoch-1 upper bound → zero rows; unscoped search). (audit P1 #3)
-  if (rejectBareStringFlags(flags, ['source', 'project', 'from', 'to', 'branch'])) return;
+  if (rejectBareStringFlags(flags, ['query', 'source', 'project', 'from', 'to', 'branch'])) return;
+
+  const query = resolvePositionalAlias(positional.join(' '), flags, ['query']);
+  if (query === null) return;
+  if (!query) {
+    fail('[mem] Usage: claude-mem-lite search <query> [--type TYPE] [--source SOURCE] [--limit N] [--project P] [--from DATE] [--to DATE] [--since DUR] [--importance N] [--branch B] [--offset N] [--sort relevance|time|importance] [--include-noise] [--deep] [--no-deep] [--rerank] — query may also be passed via --query "<query>"');
+    return;
+  }
 
   const limit = parseIntFlag(flags.limit, { name: '--limit', defaultValue: 20, max: 1000 });
   const type = flags.type || null;
@@ -446,9 +448,11 @@ function cmdRecent(db, args) {
 
 function cmdRecall(db, args) {
   const { positional, flags } = parseArgs(args);
-  const file = positional.join(' ');
+  if (rejectBareStringFlags(flags, ['file'])) return;
+  const file = resolvePositionalAlias(positional.join(' '), flags, ['file']);
+  if (file === null) return;
   if (!file) {
-    fail('[mem] Usage: claude-mem-lite recall <file> [--limit N] [--include-noise] [--json]');
+    fail('[mem] Usage: claude-mem-lite recall <file> [--limit N] [--include-noise] [--json] — file may also be passed via --file <file>');
     return;
   }
 
@@ -586,9 +590,11 @@ function renderEventRows(db, ids) {
 
 function cmdGet(db, args) {
   const { positional, flags } = parseArgs(args);
-  const idStr = positional.join(',');
+  if (rejectBareStringFlags(flags, ['ids'])) return;
+  const idStr = resolvePositionalAlias(positional.join(','), flags, ['ids']);
+  if (idStr === null) return;
   if (!idStr) {
-    fail('[mem] Usage: claude-mem-lite get <id1,id2,...> [--source obs|session|prompt|event] [--fields f1,f2,...]\n' +
+    fail('[mem] Usage: claude-mem-lite get <id1,id2,...> [--source obs|session|prompt|event] [--fields f1,f2,...] — ids may also be passed via --ids 1,2\n' +
          '        IDs accept prefix from search output: #123 (obs), P#123 (prompt), S#123 (session), E#123 (event), D#123 (deferred item, full detail).');
     return;
   }
@@ -834,15 +840,22 @@ function cmdTimeline(db, args) {
 
 function cmdSave(db, args) {
   const { positional, flags } = parseArgs(args);
-  const text = positional.join(' ');
-  if (!text.trim()) {
-    fail('[mem] Usage: claude-mem-lite save "<text>" [--type T] [--title T] [--importance N] [--project P] [--files f1,f2] [--lesson T] [--closes-deferred 1,D#42] [--supersedes 8754,8771]');
-    return;
-  }
 
   // Reject value-less string flags before they reach .split()/saveObservation as a
   // boolean `true` (#8470): bare --files/--title/--lesson crashed with a raw stacktrace.
-  if (rejectBareStringFlags(flags, ['title', 'files', 'lesson', 'lesson-learned', 'project', 'type'])) return;
+  // Runs before content resolution so a bare --text gets this clean error, not the usage line.
+  if (rejectBareStringFlags(flags, ['text', 'content', 'title', 'files', 'lesson', 'lesson-learned', 'project', 'type'])) return;
+
+  // Content: positional, or --text/--content as flags-only aliases (--content is the
+  // literal MCP mem_save field name, #233). Callers coming from the MCP schema map
+  // every field to a named flag and omit the positional — the usage error then lands
+  // on stderr and reads as "CLI doesn't support save".
+  const text = resolvePositionalAlias(positional.join(' '), flags, ['text', 'content']);
+  if (text === null) return;
+  if (!text.trim()) {
+    fail('[mem] Usage: claude-mem-lite save "<text>" [--type T] [--title T] [--importance N] [--project P] [--files f1,f2] [--lesson T] [--closes-deferred 1,D#42] [--supersedes 8754,8771] — content may also be passed via --text/--content "<text>"');
+    return;
+  }
 
   const type = flags.type || 'discovery';
   const validTypes = OBS_TYPE_SET;
@@ -980,9 +993,16 @@ function cmdDefer(db, args) {
 
 function cmdDeferAdd(db, args) {
   const { positional, flags } = parseArgs(args);
-  const title = positional.join(' ').trim();
+  // Reject bare --files/--detail/--project before .split()/bind sees a boolean true (#8470).
+  // Runs before title resolution so a bare --title gets this clean error, not the usage line.
+  if (rejectBareStringFlags(flags, ['title', 'files', 'detail', 'project'])) return;
+  // --title alias: the MCP mem_defer schema's required field IS `title` (#233), so
+  // flags-only callers emit `defer add --title "..." --detail "..."` with no positional.
+  const resolvedTitle = resolvePositionalAlias(positional.join(' '), flags, ['title']);
+  if (resolvedTitle === null) return;
+  const title = resolvedTitle.trim();
   if (!title) {
-    fail('[mem] Usage: claude-mem-lite defer add "<title>" [--priority 1|2|3] [--detail T] [--files f1,f2] [--project P]');
+    fail('[mem] Usage: claude-mem-lite defer add "<title>" [--priority 1|2|3] [--detail T] [--files f1,f2] [--project P] — title may also be passed via --title "<title>"');
     return;
   }
   // Mirror MCP memDeferSchema.title (z.string().min(1).max(200)). CLI used to
@@ -992,8 +1012,6 @@ function cmdDeferAdd(db, args) {
     fail(`[mem] defer add: title too long (${title.length} chars, max 200). Move detail to --detail "<text>".`);
     return;
   }
-  // Reject bare --files/--detail/--project before .split()/bind sees a boolean true (#8470).
-  if (rejectBareStringFlags(flags, ['files', 'detail', 'project'])) return;
   const priority = flags.priority !== undefined ? parseInt(flags.priority, 10) : 2;
   // isNumericToken first: bare parseInt would coerce "3xyz"→3 and silently escalate a
   // deferred item's urgency. Float literals still truncate (#8277).
@@ -1047,8 +1065,12 @@ function cmdDeferList(db, args) {
 
 function cmdDeferDrop(db, args) {
   const { positional, flags } = parseArgs(args);
-  if (positional.length === 0) {
-    fail('[mem] Usage: claude-mem-lite defer drop <id-or-D#N>[,id2,...] --reason "<reason>" [--project P]');
+  // --id alias (MCP mem_defer_drop.id field shape, #233).
+  if (rejectBareStringFlags(flags, ['id'])) return;
+  const idStr = resolvePositionalAlias(positional.join(' '), flags, ['id']);
+  if (idStr === null) return;
+  if (!idStr.trim()) {
+    fail('[mem] Usage: claude-mem-lite defer drop <id-or-D#N>[,id2,...] --reason "<reason>" [--project P] — id may also be passed via --id D#N');
     return;
   }
   const reason = flags.reason;
@@ -1060,7 +1082,7 @@ function cmdDeferDrop(db, args) {
   // already accepts the batch form (cmdSave uses resolveDeferredIds on a split list);
   // drop now mirrors that ergonomic so users can prune multiple items in one call
   // without N shell invocations.
-  const rawTokens = positional.join(' ').split(',').map(s => s.trim()).filter(Boolean);
+  const rawTokens = idStr.split(',').map(s => s.trim()).filter(Boolean);
   const tokens = rawTokens.map(t => /^\d+$/.test(t) ? parseInt(t, 10) : t);
   const project = flags.project ? resolveProject(db, flags.project) : inferProject();
 
@@ -1435,9 +1457,11 @@ function getActiveSessionId(db, project) {
 
 function cmdDelete(db, args) {
   const { positional, flags } = parseArgs(args);
-  const idStr = positional.join(',');
+  if (rejectBareStringFlags(flags, ['ids'])) return;
+  const idStr = resolvePositionalAlias(positional.join(','), flags, ['ids']);
+  if (idStr === null) return;
   if (!idStr) {
-    fail('[mem] Usage: claude-mem-lite delete <id1,id2,...> [--confirm]');
+    fail('[mem] Usage: claude-mem-lite delete <id1,id2,...> [--confirm] — ids may also be passed via --ids 1,2');
     return;
   }
 
@@ -1490,7 +1514,11 @@ function cmdDelete(db, args) {
 
 function cmdUpdate(db, args) {
   const { positional, flags } = parseArgs(args);
-  const raw = positional[0];
+  // --id alias (MCP mem_update.id field shape, #233). Bare --id → resolved as absent
+  // here (boolean true is not a string), falling through to the usage line below.
+  const resolvedId = resolvePositionalAlias(positional[0] ?? '', flags, ['id']);
+  if (resolvedId === null) return;
+  const raw = resolvedId || undefined;
   if (raw && /^[EePpSs]#?\d+$/.test(String(raw).trim())) {
     fail(`[mem] update only works on observations. Rejected: ${raw}. ` +
          `Prompts, sessions, and events are not editable here.`);
@@ -1502,7 +1530,7 @@ function cmdUpdate(db, args) {
   const parsed = raw ? parseIdToken(raw) : null;
   const id = parsed && parsed.source === null ? parsed.id : NaN;
   if (!id || isNaN(id)) {
-    fail('[mem] Usage: claude-mem-lite update <id> [--title T] [--type T] [--importance N] [--lesson T] [--narrative T] [--concepts T]');
+    fail('[mem] Usage: claude-mem-lite update <id> [--title T] [--type T] [--importance N] [--lesson T] [--narrative T] [--concepts T] — id may also be passed via --id N');
     return;
   }
 
@@ -2606,6 +2634,7 @@ function cmdHelp() {
 
 Commands:
   search <query>        FTS5 search across observations, sessions, and prompts
+    --query Q           Query as a flag (alias for the positional; use one, not both)
     --source S          Table: observations|sessions|prompts (default: all)
     --type T            Filter obs type (bugfix|decision|discovery|feature|refactor|change)
     --limit N           Max results (default 20)
@@ -2630,6 +2659,7 @@ Commands:
     --json              Output as JSON: {project,limit,type,total,results:[…]}
 
   recall <file>         Show observations related to a file
+    --file F            File as a flag (alias for the positional)
     --limit N           Max results (default 10)
     --include-noise     Include hook-llm fallback titles ("Modified X", raw error logs)
     --json              Output as JSON: {file,limit,include_noise,total,results:[…]}
@@ -2638,6 +2668,7 @@ Commands:
     IDs accept search-output prefixes: #123 (obs), P#123 (prompt), S#123 (session),
     D#123 (deferred item — FULL detail; defer list is title-only).
     Bare N defaults to obs. Mixed prefixes in one call route each token correctly.
+    --ids 1,2           IDs as a flag (alias for the positional list)
     --source S          Force record type (obs|session|prompt); overrides prefixes
                         (D# tokens exempt — they always read deferred_work).
     --fields f1,f2,...  Select specific fields to return (observations only).
@@ -2657,6 +2688,8 @@ Commands:
                         (or {anchor:null,fallback:"recent",results:[…]} when no anchor)
 
   save "<text>"         Save a new observation
+    --text T            Content as a flag (alias for the positional; use one, not both)
+    --content T         Same alias under the MCP mem_save field name
     --type T            Observation type (default: discovery)
     --title T           Title (auto-generated if omitted)
     --importance N      1=routine, 2=notable, 3=critical (default: 2)
@@ -2667,6 +2700,7 @@ Commands:
 
   defer <action>        First-class deferred work (v2.70+)
     add "<title>"       Mark deferred work for next session (≤200 chars)
+      --title T         Title as a flag (alias for the positional)
       --priority N      1=low, 2=normal, 3=urgent (default: 2)
       --detail T        Constraint + why deferred
       --files f1,f2     Comma-separated file paths
@@ -2675,14 +2709,17 @@ Commands:
       --limit N         Max results (default 10)
       --project P       Filter by project
     drop <D#N|ordinal>[,...]  Drop one or more deferred items (no fix needed)
+      --id D#N          ID as a flag (alias for the positional)
       --reason "..."    Required audit trail
       --project P       Project for ordinal resolution (default: current; must
                         match the "defer list --project P" you read ordinals from)
 
   delete <id1,id2,...>  Delete observations by ID
+    --ids 1,2           IDs as a flag (alias for the positional list)
     --confirm           Execute deletion (preview by default)
 
   update <id>           Update an existing observation
+    --id N              ID as a flag (alias for the positional)
     --title T           New title
     --type T            New type
     --importance N      New importance (1=routine, 2=notable, 3=critical)

package/memdir.mjs

@@ -304,9 +304,30 @@ export function removePluginDoc(memdir, slug) {
 // noise.
 
 const AUDIT_FILE_RE = /^(feedback|project)_[A-Za-z0-9_-]+\.md$/;
+// Legacy skip prefixes: user_*/reference_* have no Why/How requirement. A known
+// filename prefix wins over frontmatter type (a user_*.md stays excluded even if
+// its frontmatter says feedback).
+const SKIP_FILE_RE = /^(user|reference)_[A-Za-z0-9_-]+\.md$/;
 const WHY_RE = /^\s*\*\*Why:\*\*/m;
 const HOW_RE = /^\s*\*\*How to apply:\*\*/m;
 
+/**
+ * Extract the memory type from a file's YAML frontmatter (2026-07-24 audit P2).
+ * The current CC harness writes kebab-case filenames (ship-runbook.md) and puts
+ * the type in frontmatter — either top-level `type: X` or nested under
+ * `metadata:` as `  type: X`. Anchored `^\s*type:` cannot match `node_type:`.
+ *
+ * @param {string} raw Full file content
+ * @returns {string|null} Lowercased type, or null when absent/no frontmatter
+ */
+function frontmatterType(raw) {
+  if (!raw.startsWith('---\n') && !raw.startsWith('---\r\n')) return null;
+  const m = raw.match(/^---\r?\n([\s\S]*?)\r?\n---(\r?\n|$)/);
+  if (!m) return null;
+  const t = m[1].match(/^\s*type:\s*([A-Za-z]+)\s*$/m);
+  return t ? t[1].toLowerCase() : null;
+}
+
 /**
  * Strip the leading YAML frontmatter block (between `---` fences) so audit
  * checks run only against body content. Returns input unchanged if no
@@ -339,13 +360,29 @@ export function auditMemdir(memdir) {
   let entries;
   try { entries = readdirSync(memdir); } catch { return result; }
 
-  const targets = entries.filter(n => AUDIT_FILE_RE.test(n)).sort();
-  for (const name of targets) {
-    let body = '';
-    try {
-      const raw = readFileSync(join(memdir, name), 'utf8');
-      body = stripFrontmatter(raw);
-    } catch { /* unreadable — count as missingBoth */ }
+  // Two selection paths (2026-07-24 audit P2):
+  //   1. legacy filename prefix — feedback_*/project_* audited, user_*/reference_* skipped
+  //   2. kebab-case (current harness) — frontmatter type ∈ {feedback, project}
+  // Path 2 needs the file content; read once and reuse for the body check.
+  const candidates = entries
+    .filter(n => n.endsWith('.md') && n !== 'MEMORY.md' && !n.startsWith('.'))
+    .sort();
+  let total = 0;
+  for (const name of candidates) {
+    const legacyAudit = AUDIT_FILE_RE.test(name);
+    if (!legacyAudit && SKIP_FILE_RE.test(name)) continue;
+
+    let raw = null;
+    try { raw = readFileSync(join(memdir, name), 'utf8'); } catch { /* unreadable */ }
+
+    if (!legacyAudit) {
+      // Frontmatter decides; unreadable or untyped files carry no Why/How contract.
+      const type = raw === null ? null : frontmatterType(raw);
+      if (type !== 'feedback' && type !== 'project') continue;
+    }
+    // Legacy-prefixed unreadable file keeps the old behavior: counted as missingBoth.
+    const body = raw === null ? '' : stripFrontmatter(raw);
+    total += 1;
 
     const hasWhy = WHY_RE.test(body);
     const hasHow = HOW_RE.test(body);
@@ -354,6 +391,6 @@ export function auditMemdir(memdir) {
     else if (!hasWhy) result.missingWhy.push(name);
     else result.missingHowToApply.push(name);
   }
-  result.total = targets.length;
+  result.total = total;
   return result;
 }

package/npm-shrinkwrap.json

@@ -1,12 +1,12 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.58.2",
+  "version": "3.59.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "claude-mem-lite",
-      "version": "3.58.2",
+      "version": "3.59.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.58.2",
+  "version": "3.59.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",

package/scoring-sql.mjs

@@ -152,8 +152,17 @@ export function noisePenaltyClause(alias = 'o') {
 // The SQL path (this), the regex path (utils.mjs::LOW_SIGNAL_TITLE), and the
 // pre-tool-recall.js inline SQL now all derive from one authoritative
 // pattern list. Previously hand-mirrored with "keep in sync" comments.
+//
+// lessonEscape (2026-07-24 audit P1, D#11): every consumer of THIS clause is an
+// observations-table retrieval surface (search, recall, error-recall, context/
+// handoff/UPS injection, optimize candidates), so all get the read-side lesson
+// escape — a low-signal TITLE no longer hides a row with a real lesson_learned.
+// Consumers that must stay title-only import buildNotLowSignalSql directly:
+// events-table queries (lib/activity.mjs, pre-tool-recall.js events fallback —
+// no lesson_learned column) and noise-title metrics (lib/stats-core.mjs,
+// lib/stats-quality.mjs — they COUNT pattern-titled rows, not filter them).
 export function notLowSignalTitleClause(alias = 'o') {
-  return buildNotLowSignalSql(alias);
+  return buildNotLowSignalSql(alias, { lessonEscape: true });
 }
 
 // ─── Cite-history factor (A1, v2.83) ────────────────────────────────────────