claude-mem-lite 3.59.0 → 3.59.1

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.59.0",
+      "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.59.0",
+  "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/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/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.59.0",
+  "version": "3.59.1",
   "lockfileVersion": 3,
   "requires": true,
   "packages": {
     "": {
       "name": "claude-mem-lite",
-      "version": "3.59.0",
+      "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.59.0",
+  "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) ────────────────────────────────────────