claude-mem-lite 2.41.0 → 2.42.0

package/.claude-plugin/marketplace.json

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

package/cli/common.mjs

@@ -1,10 +1,11 @@
 // cli/common.mjs — shared helpers used by every per-command file under cli/.
 // Extracted from mem-cli.mjs (v2.41) as first step in the god-module split.
 //
-// Scope: pure utilities only. No DB, no imports from other cli/ files. This
-// module is the single source of truth for stdout/stderr framing, arg parsing,
-// ID-token parsing, and relative-time formatting — every command imports from
-// here so the CLI stays consistent.
+// Scope: pure utilities only. No DB, no imports from other cli/ files; only
+// `lib/` leaf utilities may be re-exported through here (currently:
+// parseIdToken). This module is the single source of truth for stdout/stderr
+// framing, arg parsing, ID-token parsing, and relative-time formatting —
+// every command imports from here so the CLI stays consistent.
 
 // ─── Argument Parsing ────────────────────────────────────────────────────────
 
@@ -75,22 +76,10 @@ export function fmtDateShort(iso) {
 }
 
 // ─── ID Token Parsing ────────────────────────────────────────────────────────
-
-/**
- * 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 } | null}
- *   source===null means no explicit prefix — caller picks default (typically 'obs').
- */
-export 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 };
-}
+// Re-exported from lib/id-routing.mjs so CLI and MCP (server.mjs) share a single
+// parser — parity per #8050. Keep this re-export for back-compat with the
+// 5 CLI call sites that already import parseIdToken from cli/common.mjs.
+export { parseIdToken } from '../lib/id-routing.mjs';
 
 /**
  * Format the shared `probeIdSources` output as CLI hint strings.

package/lib/id-routing.mjs

@@ -1,10 +1,29 @@
-// 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.
+// Shared probe for "ID-not-found-in-requested-source" hints + shared token
+// parser. Used by CLI (mem-cli.mjs, cli/common.mjs re-export) and MCP
+// (server.mjs) so both paths stay aligned — parity per #8050.
 //
 // The formatter stays per-call-site because CLI and MCP surface format
-// differently (stderr vs response text); only the SQL layer is shared.
+// differently (stderr vs response text); only the SQL + token-parse layers
+// are shared.
+
+// ─── ID Token Parsing ────────────────────────────────────────────────────────
+
+/**
+ * Parse an ID token as it appears in search output or CLI positional args.
+ * Accepts: `123`, `#123`, `P#123` / `p123` (prompt), `S#123` / `s123` (session).
+ * @param {unknown} raw
+ * @returns {{ source: 'obs'|'session'|'prompt'|null, id: number } | null}
+ *   source===null means no explicit prefix — caller picks default (typically 'obs').
+ */
+export 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 };
+}
 
 /**
  * Probe the observations / session_summaries / user_prompts tables for any

package/mem-cli.mjs

@@ -476,7 +476,7 @@ function cmdRecent(db, args) {
   for (const r of rows) {
     const time = relativeTime(r.created_at_epoch);
     const title = truncate(r.title || r.subtitle || '(untitled)', 80);
-    out(`#${String(r.id).padStart(5)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}`);
+    out(`${('#' + r.id).padEnd(6)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}`);
   }
 }
 
@@ -787,7 +787,7 @@ function cmdTimeline(db, args) {
     for (const r of rows.reverse()) {
       const time = relativeTime(r.created_at_epoch);
       const title = truncate(r.title || r.subtitle || '(untitled)', 80);
-      out(`#${String(r.id).padStart(5)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}`);
+      out(`${('#' + r.id).padEnd(6)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}`);
     }
     return;
   }
@@ -840,7 +840,7 @@ function cmdTimeline(db, args) {
     const marker = r.id === anchorId ? ' <--' : '';
     const time = relativeTime(r.created_at_epoch);
     const title = truncate(r.title || r.subtitle || '(untitled)', 80);
-    out(`#${String(r.id).padStart(5)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}${marker}`);
+    out(`${('#' + r.id).padEnd(6)} ${typeIcon(r.type)} ${time.padEnd(8)} ${title}${marker}`);
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-mem-lite",
-  "version": "2.41.0",
+  "version": "2.42.0",
   "description": "Lightweight persistent memory system for Claude Code",
   "type": "module",
   "engines": {

package/server.mjs

@@ -27,7 +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 { probeOtherSources as probeIdSources, parseIdToken } from './lib/id-routing.mjs';
 import { getVocabulary, rebuildVocabulary, _resetVocabCache, computeVector, vectorSearch, rrfMerge } from './tfidf.mjs';
 import { createRequire } from 'module';
 
@@ -762,6 +762,57 @@ server.registerTool(
     const before = args.before ?? 5;
     const after = args.after ?? 5;
     let anchorId = args.anchor;
+    let anchorNote = null;
+
+    // Resolve prefixed-token anchor (e.g. "P#3462" / "S#53" / "#8121") — users pasting
+    // from mem_search results expect the same routing as CLI `timeline --anchor`.
+    // Prompt/session anchors resolve to the nearest-in-time observation so
+    // before/after semantics still apply to the observations timeline.
+    if (typeof anchorId === 'string') {
+      const parsed = parseIdToken(anchorId);
+      if (!parsed) {
+        return { content: [{ type: 'text', text: `Invalid anchor "${args.anchor}". Expected N, #N, P#N, or S#N.` }] };
+      }
+      if (parsed.source === 'prompt' || parsed.source === 'session') {
+        const srcTable = parsed.source === 'prompt' ? 'user_prompts' : 'session_summaries';
+        const srcPrefix = parsed.source === 'prompt' ? 'P#' : 'S#';
+        const row = db.prepare(`SELECT created_at_epoch FROM ${srcTable} WHERE id = ?`).get(parsed.id);
+        if (!row) return { content: [{ type: 'text', text: `${parsed.source === 'prompt' ? 'Prompt' : 'Session'} ${srcPrefix}${parsed.id} not found.` }] };
+        const projArg = args.project;
+        const nearest = db.prepare(`
+          SELECT id FROM observations
+          WHERE COALESCE(compressed_into, 0) = 0 ${projArg ? 'AND project = ?' : ''}
+          ORDER BY ABS(created_at_epoch - ?) ASC LIMIT 1
+        `).get(...(projArg ? [projArg, row.created_at_epoch] : [row.created_at_epoch]));
+        if (!nearest) return { content: [{ type: 'text', text: `No observations near ${srcPrefix}${parsed.id}.` }] };
+        anchorId = nearest.id;
+        anchorNote = `(anchored to #${nearest.id}, closest obs to ${srcPrefix}${parsed.id})`;
+      } else {
+        // Bare "#N" or "N" — resolve to obs, falling back to prompt/session like CLI bare-int path.
+        const obsExists = db.prepare('SELECT 1 FROM observations WHERE id = ?').get(parsed.id);
+        if (obsExists) {
+          anchorId = parsed.id;
+        } else {
+          const promptRow = db.prepare('SELECT created_at_epoch FROM user_prompts WHERE id = ?').get(parsed.id);
+          const sessionRow = promptRow ? null : db.prepare('SELECT created_at_epoch FROM session_summaries WHERE id = ?').get(parsed.id);
+          const hit = promptRow ? { row: promptRow, prefix: 'P#', name: 'prompt' }
+                    : sessionRow ? { row: sessionRow, prefix: 'S#', name: 'session' }
+                    : null;
+          if (!hit) {
+            return { content: [{ type: 'text', text: `Observation, prompt, or session with id ${parsed.id} not found.` }] };
+          }
+          const projArg = args.project;
+          const nearest = db.prepare(`
+            SELECT id FROM observations
+            WHERE COALESCE(compressed_into, 0) = 0 ${projArg ? 'AND project = ?' : ''}
+            ORDER BY ABS(created_at_epoch - ?) ASC LIMIT 1
+          `).get(...(projArg ? [projArg, hit.row.created_at_epoch] : [hit.row.created_at_epoch]));
+          if (!nearest) return { content: [{ type: 'text', text: `No observations near ${hit.prefix}${parsed.id} (${hit.name}).` }] };
+          anchorId = nearest.id;
+          anchorNote = `(anchored to #${nearest.id}, closest obs to ${hit.prefix}${parsed.id})`;
+        }
+      }
+    }
 
     // Auto-find anchor via FTS (with recency decay)
     if (!anchorId && args.query) {
@@ -845,7 +896,7 @@ server.registerTool(
     const anchor = db.prepare('SELECT id, type, title, subtitle, project, created_at FROM observations WHERE id = ?').get(anchorId);
 
     const all = [...beforeRows.reverse(), anchor, ...afterRows];
-    const lines = [`Timeline around #${anchorId}:\n`];
+    const lines = [`Timeline around #${anchorId}${anchorNote ? ' ' + anchorNote : ''}:\n`];
     for (const r of all) {
       const marker = r.id === anchorId ? ' ◀' : '';
       lines.push(`#${r.id} ${typeIcon(r.type)} [${r.type}] ${truncate(r.title || r.subtitle || '(untitled)')} | ${r.project} | ${fmtDate(r.created_at)}${marker}`);

package/tool-schemas.mjs

@@ -50,8 +50,27 @@ export const memRecentSchema = {
   project: z.string().optional().describe('Filter by project (default: inferred from CWD)'),
 };
 
+// Anchor accepts plain int, "123" string-int, or prefixed token from search output:
+// "#123" / "P#123" (prompt) / "S#123" (session). Prompt/session anchors resolve to
+// the nearest-in-time observation so timeline semantics still apply. CLI --anchor
+// parity per #8050 (CLI has supported prefixed tokens since v2.39.0).
+const coerceAnchor = z.preprocess(
+  (v) => {
+    if (typeof v === 'string') {
+      const s = v.trim();
+      if (/^-?\d+$/.test(s)) return parseInt(s, 10);
+      return s;
+    }
+    return v;
+  },
+  z.union([
+    z.number().int(),
+    z.string().regex(/^[PpSs]?#?\d+$/, 'Expected N, #N, P#N, or S#N'),
+  ])
+);
+
 export const memTimelineSchema = {
-  anchor: coerceInt.pipe(z.number().int()).optional().describe('Observation ID as center point. Takes precedence over query when both are provided.'),
+  anchor: coerceAnchor.optional().describe('Anchor as observation ID (int) or prefixed token string: "#123", "P#123" (prompt → nearest obs), "S#123" (session → nearest obs). Takes precedence over query.'),
   query: z.string().optional().describe('FTS5 query to auto-find anchor. Ignored when anchor is also given; use one or the other.'),
   before: coerceInt.pipe(z.number().int().min(0).max(50)).optional().describe('Items before anchor (default 5)'),
   after: coerceInt.pipe(z.number().int().min(0).max(50)).optional().describe('Items after anchor (default 5)'),
@@ -59,6 +78,9 @@ export const memTimelineSchema = {
 };
 
 export const memGetSchema = {
+  // TODO(#8126): accept P#/S#/# prefix strings for paste-from-search parity with
+  // CLI cmdGet bucketed routing (~40 LOC handler refactor). mem_timeline already
+  // accepts prefixes via coerceAnchor; this is the matched-pair gap.
   ids: coerceIntArray.pipe(z.array(z.number().int()).min(1).max(20)).describe('Observation IDs to retrieve'),
   source: z.enum(['obs', 'session', 'prompt']).optional().describe('Record type: obs (default), session (S# from search), prompt (P# from search)'),
   fields: z.array(z.string()).optional().describe('Specific fields to return (default: all)'),