claude-mem-lite 3.33.0 → 3.34.0

package/.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "claude-mem-lite",
-      "version": "3.33.0",
+      "version": "3.34.0",
       "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.33.0",
+  "version": "3.34.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).",
   "author": {
     "name": "sdsrss"

package/cli/common.mjs

@@ -89,6 +89,72 @@ export function rejectBareStringFlags(flags, keys) {
   return false;
 }
 
+// ─── Unknown-flag typo guard ─────────────────────────────────────────────────
+
+/**
+ * Union of every flag name any CLI command reads (parseArgs silently drops the rest).
+ * Over-inclusive BY DESIGN: a flag listed here that a given command ignores just means
+ * "no typo warning for it" — harmless. The only real risk is OMITTING a valid flag, and
+ * the edit-distance gate in suggestUnknownFlags() makes even that non-fatal (a distinct
+ * real flag rarely lands within distance 2 of another). Add new flags here when adding
+ * them to a command — same maintenance contract as JSON_SUPPORTED_CMDS in mem-cli.
+ */
+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',
+  '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',
+  'to', 'trigger-patterns', 'type', 'use-cases', 'verbose',
+]);
+
+/** Levenshtein distance, early-exit past `max` (cheap enough for a handful of flags). */
+function editDistance(a, b, max = 2) {
+  const m = a.length, n = b.length;
+  if (Math.abs(m - n) > max) return max + 1;
+  let prev = Array.from({ length: n + 1 }, (_, j) => j);
+  for (let i = 1; i <= m; i++) {
+    const cur = [i];
+    let rowMin = i;
+    for (let j = 1; j <= n; j++) {
+      const d = Math.min(prev[j] + 1, cur[j - 1] + 1, prev[j - 1] + (a[i - 1] !== b[j - 1] ? 1 : 0));
+      cur[j] = d;
+      if (d < rowMin) rowMin = d;
+    }
+    if (rowMin > max) return max + 1; // whole row already past budget → give up
+    prev = cur;
+  }
+  return prev[n];
+}
+
+/**
+ * Detect likely-typo flags: names NOT in KNOWN_CLI_FLAGS but within edit distance 2 of
+ * a known flag. parseArgs silently drops unknown flags, so `save --improtance 3` used to
+ * persist the DEFAULT importance and `recent --projcte X` silently queried the inferred
+ * project — a typo produced a wrong result with zero signal. Returns [{flag, suggestion}].
+ * Unknown flags with NO close match are omitted: they may be a valid flag we didn't
+ * catalog, so silence beats a false alarm. Warning-only by contract — never fails.
+ * @param {object} flags Parsed flags from parseArgs.
+ * @returns {Array<{flag: string, suggestion: string}>}
+ */
+export function suggestUnknownFlags(flags) {
+  const result = [];
+  for (const key of Object.keys(flags)) {
+    if (!key || KNOWN_CLI_FLAGS.has(key)) continue;
+    let best = null, bestDist = 3;
+    for (const known of KNOWN_CLI_FLAGS) {
+      const d = editDistance(key, known);
+      if (d < bestDist) { bestDist = d; best = known; }
+    }
+    if (best && bestDist <= 2) result.push({ flag: key, suggestion: best });
+  }
+  return result;
+}
+
 // ─── Time Formatting ─────────────────────────────────────────────────────────
 
 /** "just now" / "5m ago" / "3h ago" / "2d ago" relative to now. */

package/mem-cli.mjs

@@ -37,7 +37,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, OBS_TIME_FIELDS, formatObsFieldValue } from './cli/common.mjs';
+import { parseArgs, out, fail, relativeTime, fmtDateShort, parseIdToken, formatProbeHints, rejectBareStringFlags, suggestUnknownFlags, OBS_TIME_FIELDS, formatObsFieldValue } from './cli/common.mjs';
 import { saveObservation } from './lib/save-observation.mjs';
 import { rebuildObservationDerived } from './lib/observation-write.mjs';
 import { recallByFile } from './lib/recall-core.mjs';
@@ -3007,6 +3007,15 @@ export async function run(argv) {
     return;
   }
 
+  // Typo guard: parseArgs silently DROPS unknown flags, so `save --improtance 3` used to
+  // persist the DEFAULT importance and `recent --projcte X` silently queried the inferred
+  // project — a misspelled flag changed results with zero signal. Warn (stderr, non-fatal)
+  // when a flag looks like a misspelling of a real one; stdout + exit code stay untouched,
+  // so JSON/text consumers are unaffected. Mirrors the unknown-COMMAND suggester in cli.mjs.
+  for (const { flag, suggestion } of suggestUnknownFlags(parseArgs(cmdArgs).flags)) {
+    process.stderr.write(`[mem] Unknown flag --${flag}; did you mean --${suggestion}?\n`);
+  }
+
   // adopt / unadopt do pure filesystem work on ~/.claude/projects/<encoded>/memory/ —
   // no DB needed. Route them before ensureDb() so an unbootable DB doesn't block.
   if (cmd === 'adopt') { cmdAdopt(cmdArgs); return; }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "3.33.0",
+  "version": "3.34.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",

package/project-utils.mjs

@@ -32,6 +32,17 @@ export function resolveProject(db, name) {
   ).get(`%--${name}`);
   if (suffixed) { _cache.set(name, suffixed.project); return suffixed.project; }
 
+  // 1.5) Exact-name match: a project literally named "p" (e.g. inferProject() at a
+  // filesystem-root cwd yields no "--", or a manually-saved bare name). MUST beat the
+  // fuzzy prefix/substring fallbacks below — otherwise `%p%` matches every "projects--*"
+  // row and ORDER BY COUNT(*) returns the biggest UNRELATED project, making the exact
+  // project permanently unreachable via --project. Ranks below step 1 only, preserving
+  // the documented "prefer canonical parent--name over a stray short name" intent.
+  const exact = db.prepare(
+    'SELECT project FROM observations WHERE project = ? LIMIT 1'
+  ).get(name);
+  if (exact) { _cache.set(name, exact.project); return exact.project; }
+
   // 2) Prefix-in-suffix match: "code-graph" → "projects--code-graph-mcp"
   const prefixed = db.prepare(
     'SELECT project FROM observations WHERE project LIKE ? GROUP BY project ORDER BY COUNT(*) DESC LIMIT 1'

package/scripts/pre-agent-inject.js

@@ -22,7 +22,10 @@ function readStdin() {
     process.stdin.setEncoding('utf8');
     process.stdin.on('data', (c) => {
       data += c;
-      if (data.length > 262144) { clearTimeout(timer); resolve(data.slice(0, 262144)); } // cap: agent prompts can be large
+      // cap: agent prompts can be large. destroy() so the loop can drain and exit on
+      // its own (see the no-forced-exit note at the bottom) rather than streaming to
+      // the 1.5s timeout.
+      if (data.length > 262144) { clearTimeout(timer); try { process.stdin.destroy(); } catch { /* */ } resolve(data.slice(0, 262144)); }
     });
     process.stdin.on('end', () => { clearTimeout(timer); resolve(data); });
     process.stdin.on('error', () => { clearTimeout(timer); resolve(data); });
@@ -60,4 +63,10 @@ async function main() {
   }
 }
 
-main().catch(() => {}).finally(() => process.exit(0));
+// No forced process.exit(0): every readStdin path ends/destroys stdin and db.close()
+// runs in main's finally, so the event loop drains and the process exits 0 on its own —
+// which FLUSHES stdout. The emitted updatedInput echoes the whole prompt back, so the
+// payload can exceed the ~64KB pipe buffer; a forced process.exit() would drop that
+// pending async write and truncate the JSON (the gotcha every sibling hook avoids).
+// Swallow any rejection so the exit code can never go non-zero.
+main().catch(() => {});