@hasna/mementos 0.14.69 → 0.14.71

package/dist/mcp/index.js

@@ -145,6 +145,27 @@ var init_types = __esm(() => {
   };
 });
 
+// src/generated/storage-kit/mode.ts
+function normalizeStorageMode(value) {
+  const normalized = value.trim().toLowerCase().replace(/-/g, "_");
+  if (normalized === "local")
+    return { mode: "local", deprecatedAlias: null };
+  if (normalized === "cloud")
+    return { mode: "cloud", deprecatedAlias: null };
+  if (DEPRECATED_STORAGE_MODE_ALIASES.includes(normalized)) {
+    return { mode: "cloud", deprecatedAlias: normalized };
+  }
+  throw new Error(`Unknown storage mode: ${value}. Use local or cloud.`);
+}
+var DEPRECATED_STORAGE_MODE_ALIASES;
+var init_mode = __esm(() => {
+  DEPRECATED_STORAGE_MODE_ALIASES = [
+    "remote",
+    "hybrid",
+    "self_hosted"
+  ];
+});
+
 // src/storage.ts
 import { Database } from "bun:sqlite";
 import { existsSync, mkdirSync, readFileSync, writeFileSync } from "fs";
@@ -403,18 +424,20 @@ function warnDeprecatedStorageMode(alias) {
   warnedDeprecatedModes.add(alias);
   process.emitWarning(`${MEMENTOS_STORAGE_ENV.mode}="${alias}" is deprecated; use "cloud". ` + `"${alias}" now maps to pure-remote cloud storage. The local<->remote ` + `sync path is deprecated and is not the fleet cutover path.`, { type: "DeprecationWarning", code: "MEMENTOS_STORAGE_MODE_ALIAS" });
 }
-function normalizeStorageMode(value) {
-  if (!value)
+function normalizeStorageMode2(value, source) {
+  if (!value || !value.trim())
     return null;
-  const normalized = value.trim().toLowerCase();
-  if (normalized === "local" || normalized === "cloud") {
-    return normalized;
+  let normalized;
+  try {
+    normalized = normalizeStorageMode(value);
+  } catch (error) {
+    const detail = error instanceof Error ? error.message : String(error);
+    throw new Error(`mementos: ${source}=${value} is not a valid mode. ${detail}`);
   }
-  if (normalized === "remote" || normalized === "hybrid") {
-    warnDeprecatedStorageMode(normalized);
-    return "cloud";
+  if (normalized.deprecatedAlias) {
+    warnDeprecatedStorageMode(normalized.deprecatedAlias);
   }
-  return null;
+  return normalized.mode;
 }
 function readConfigFile() {
   if (!existsSync(STORAGE_CONFIG_PATH)) {
@@ -444,7 +467,7 @@ function getStorageDatabaseUrl() {
 }
 function getStorageModeOverride() {
   for (const env of MODE_ENV_NAMES) {
-    const value = normalizeStorageMode(readEnv(env.name) ?? undefined);
+    const value = normalizeStorageMode2(readEnv(env.name) ?? undefined, env.name);
     if (value)
       return value;
   }
@@ -454,7 +477,7 @@ function getStorageConfig() {
   const fileConfig = readConfigFile();
   const modeOverride = getStorageModeOverride();
   const envConnectionString = getConfiguredConnectionString();
-  const fileMode = normalizeStorageMode(fileConfig.mode);
+  const fileMode = normalizeStorageMode2(fileConfig.mode, `${STORAGE_CONFIG_PATH} "mode"`);
   const merged = {
     ...DEFAULT_STORAGE_CONFIG,
     ...fileConfig,
@@ -865,6 +888,7 @@ CREATE TABLE IF NOT EXISTS _sync_meta (
   direction TEXT DEFAULT 'push'
 )`;
 var init_storage = __esm(() => {
+  init_mode();
   PgSyncPool = class PgSyncPool {
     worker;
     status;
@@ -1216,7 +1240,24 @@ var init_api_mode = __esm(() => {
 });
 
 // src/db/migrations.ts
-var MIGRATIONS;
+var MEMORY_VERSION_SNAPSHOT_TRIGGER = `
+CREATE TRIGGER IF NOT EXISTS memories_version_snapshot
+BEFORE UPDATE ON memories
+WHEN NEW.version > OLD.version
+BEGIN
+  INSERT OR IGNORE INTO memory_versions (
+    id, memory_id, version, value, importance, scope, category, tags,
+    summary, pinned, status, when_to_use, created_at
+  ) VALUES (
+    lower(hex(randomblob(4))) || '-' || lower(hex(randomblob(2))) || '-' ||
+      lower(hex(randomblob(2))) || '-' || lower(hex(randomblob(2))) || '-' ||
+      lower(hex(randomblob(6))),
+    OLD.id, OLD.version, OLD.value, OLD.importance, OLD.scope, OLD.category,
+    OLD.tags, OLD.summary, OLD.pinned, OLD.status, OLD.when_to_use,
+    OLD.updated_at
+  );
+END;
+`, MIGRATIONS;
 var init_migrations = __esm(() => {
   MIGRATIONS = [
     `
@@ -2114,6 +2155,10 @@ CREATE INDEX IF NOT EXISTS idx_memory_reflection_lessons_memory ON memory_reflec
 CREATE INDEX IF NOT EXISTS idx_memory_reflection_lessons_kind ON memory_reflection_lessons(kind);
 
 INSERT OR IGNORE INTO _migrations (id) VALUES (35);
+`,
+    `
+${MEMORY_VERSION_SNAPSHOT_TRIGGER}
+INSERT OR IGNORE INTO _migrations (id) VALUES (36);
 `
   ];
 });
@@ -2128,6 +2173,7 @@ __export(exports_database, {
   now: () => now,
   getDbPath: () => getDbPath,
   getDatabase: () => getDatabase,
+  escapeLikePrefix: () => escapeLikePrefix,
   closeDatabase: () => closeDatabase
 });
 import { existsSync as existsSync2, mkdirSync as mkdirSync2, cpSync } from "fs";
@@ -2302,15 +2348,20 @@ function uuid() {
 function shortUuid() {
   return crypto.randomUUID().slice(0, 8);
 }
+function escapeLikePrefix(s) {
+  return s.replace(/\\/g, "\\\\").replace(/%/g, "\\%").replace(/_/g, "\\_");
+}
 function resolvePartialId(db, table, partialId) {
   if (!ALLOWED_TABLES.has(table)) {
     throw new Error(`Invalid table name: ${table}`);
   }
+  if (partialId === "")
+    return null;
   if (partialId.length >= 36) {
     const row = db.query(`SELECT id FROM ${table} WHERE id = ?`).get(partialId);
     return row?.id ?? null;
   }
-  const rows = db.query(`SELECT id FROM ${table} WHERE id LIKE ?`).all(`${partialId}%`);
+  const rows = db.query(`SELECT id FROM ${table} WHERE id LIKE ? ESCAPE '\\'`).all(`${escapeLikePrefix(partialId)}%`);
   if (rows.length === 1) {
     return rows[0].id;
   }
@@ -3372,33 +3423,19 @@ function updateMemory(id, input, db) {
     const { status, data } = apiJson("PATCH", `/memories/${encodeURIComponent(id)}`, input, { allow404: true });
     if (status === 404)
       throw new MemoryNotFoundError(id);
+    if (data && typeof input.version === "number" && typeof data.version === "number" && data.version <= input.version) {
+      throw new Error(`Update did not persist for memory ${id}: the server returned success but the record is unchanged ` + `(version still ${data.version}). Your data was NOT written. ` + `The server is likely running a build predating the partial-id fix \u2014 pass the full 36-character id as a workaround.`);
+    }
     return data;
   }
   const d = db || getDatabase();
   const existing = getMemory(id, d);
   if (!existing)
     throw new MemoryNotFoundError(id);
+  const memoryId = existing.id;
   if (existing.version !== input.version) {
     throw new VersionConflictError(id, input.version, existing.version);
   }
-  try {
-    d.run(`INSERT OR IGNORE INTO memory_versions (id, memory_id, version, value, importance, scope, category, tags, summary, pinned, status, when_to_use, created_at)
-       VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`, [
-      uuid(),
-      existing.id,
-      existing.version,
-      existing.value,
-      existing.importance,
-      existing.scope,
-      existing.category,
-      JSON.stringify(existing.tags),
-      existing.summary,
-      existing.pinned ? 1 : 0,
-      existing.status,
-      existing.when_to_use || null,
-      existing.updated_at
-    ]);
-  } catch {}
   const sets = ["version = version + 1", "updated_at = ?"];
   const params = [now()];
   if (input.value !== undefined) {
@@ -3448,15 +3485,18 @@ function updateMemory(id, input, db) {
   if (input.tags !== undefined) {
     sets.push("tags = ?");
     params.push(JSON.stringify(input.tags));
-    d.run("DELETE FROM memory_tags WHERE memory_id = ?", [id]);
+    d.run("DELETE FROM memory_tags WHERE memory_id = ?", [memoryId]);
     const insertTag = d.prepare("INSERT OR IGNORE INTO memory_tags (memory_id, tag) VALUES (?, ?)");
     for (const tag of input.tags) {
-      insertTag.run(id, tag);
+      insertTag.run(memoryId, tag);
     }
   }
-  params.push(id);
-  d.run(`UPDATE memories SET ${sets.join(", ")} WHERE id = ?`, params);
-  const updated = getMemory(id, d);
+  params.push(memoryId);
+  const result = d.run(`UPDATE memories SET ${sets.join(", ")} WHERE id = ?`, params);
+  if (result.changes === 0) {
+    throw new Error(`Update affected no rows for memory ${memoryId}: the record was read but not written. ` + `This is a bug in @hasna/mementos, not a bad argument \u2014 please report it.`);
+  }
+  const updated = getMemory(memoryId, d);
   if (input.value !== undefined) {
     try {
       const oldLinks = getEntityMemoryLinks(undefined, updated.id, d);
@@ -3481,10 +3521,11 @@ function deleteMemory(id, db) {
     return status !== 404;
   }
   const d = db || getDatabase();
-  const result = d.run("DELETE FROM memories WHERE id = ?", [id]);
+  const memoryId = resolvePartialId(d, "memories", id) ?? id;
+  const result = d.run("DELETE FROM memories WHERE id = ?", [memoryId]);
   if (result.changes > 0) {
     hookRegistry.runHooks("PostMemoryDelete", {
-      memoryId: id,
+      memoryId,
       timestamp: Date.now()
     });
   }
@@ -3498,11 +3539,12 @@ function bulkDeleteMemories(ids, db) {
     return data?.deleted ?? 0;
   }
   const d = db || getDatabase();
-  const placeholders = ids.map(() => "?").join(",");
-  const countRow = d.query(`SELECT COUNT(*) as c FROM memories WHERE id IN (${placeholders})`).get(...ids);
+  const resolvedIds = ids.map((id) => resolvePartialId(d, "memories", id) ?? id);
+  const placeholders = resolvedIds.map(() => "?").join(",");
+  const countRow = d.query(`SELECT COUNT(*) as c FROM memories WHERE id IN (${placeholders})`).get(...resolvedIds);
   const count = countRow.c;
   if (count > 0) {
-    d.run(`DELETE FROM memories WHERE id IN (${placeholders})`, ids);
+    d.run(`DELETE FROM memories WHERE id IN (${placeholders})`, resolvedIds);
   }
   return count;
 }
@@ -5704,6 +5746,32 @@ class AutoMemoryQueue {
   getStats() {
     return { ...this.stats, pending: this.queue.length };
   }
+  async waitForIdleForTests(timeoutMs = 3000) {
+    const start = Date.now();
+    while (Date.now() - start < timeoutMs) {
+      if (this.queue.length === 0 && this.activeCount === 0)
+        return;
+      await new Promise((r) => setTimeout(r, 20));
+    }
+    throw new Error("autoMemoryQueue did not become idle before test reset");
+  }
+  resetForTests(handler) {
+    if (this.activeCount !== 0) {
+      throw new Error("Cannot reset autoMemoryQueue while jobs are processing");
+    }
+    this.queue = [];
+    this.running = false;
+    this.stats = {
+      pending: 0,
+      processing: 0,
+      processed: 0,
+      failed: 0,
+      dropped: 0
+    };
+    if (handler !== undefined) {
+      this.handler = handler;
+    }
+  }
   startLoop() {
     this.running = true;
     this.loop();
@@ -5749,6 +5817,7 @@ var init_auto_memory_queue = __esm(() => {
 // src/lib/auto-memory.ts
 var exports_auto_memory = {};
 __export(exports_auto_memory, {
+  resetAutoMemoryForTests: () => resetAutoMemoryForTests,
   processConversationTurn: () => processConversationTurn,
   getAutoMemoryStats: () => getAutoMemoryStats,
   configureAutoMemory: () => configureAutoMemory
@@ -5909,6 +5978,12 @@ function getAutoMemoryStats() {
 function configureAutoMemory(config) {
   providerRegistry.configure(config);
 }
+async function resetAutoMemoryForTests() {
+  if (autoMemoryQueue.getStats().processing > 0) {
+    await autoMemoryQueue.waitForIdleForTests();
+  }
+  autoMemoryQueue.resetForTests(processJob);
+}
 var DEDUP_SIMILARITY_THRESHOLD = 0.85;
 var init_auto_memory = __esm(() => {
   init_memories();
@@ -6597,6 +6672,8 @@ var init_built_in_hooks = __esm(() => {
     priority: 100,
     description: "Trigger async LLM entity extraction when a memory is saved",
     handler: async (ctx) => {
+      if (process.env["NODE_ENV"] === "test")
+        return;
       if (ctx.wasUpdated)
         return;
       const processConversationTurn2 = await getAutoMemory();
@@ -11971,11 +12048,14 @@ __export(exports_gdpr, {
 });
 function gdprErase(identifier, options = {}, db) {
   const d = db || getDatabase();
+  if (identifier.trim() === "") {
+    throw new Error("GDPR erase requires a non-empty identifier: an empty or whitespace-only " + "identifier matches every memory and would redact the entire store");
+  }
   const timestamp = now();
   const conditions = [
-    "(key LIKE ? OR value LIKE ? OR summary LIKE ? OR tags LIKE ? OR metadata LIKE ?)"
+    "(key LIKE ? ESCAPE '\\' OR value LIKE ? ESCAPE '\\' OR summary LIKE ? ESCAPE '\\' OR tags LIKE ? ESCAPE '\\' OR metadata LIKE ? ESCAPE '\\')"
   ];
-  const searchParam = `%${identifier}%`;
+  const searchParam = `%${escapeLikePrefix(identifier)}%`;
   const params = [searchParam, searchParam, searchParam, searchParam, searchParam];
   if (options.project_id) {
     conditions.push("project_id = ?");
@@ -11994,18 +12074,23 @@ function gdprErase(identifier, options = {}, db) {
       timestamp
     };
   }
-  const memoryIds = [];
-  for (const row of rows) {
-    d.run(`UPDATE memories SET
+  const memoryIds = d.transaction(() => {
+    const ids = [];
+    for (const row of rows) {
+      const redactedKey = `[REDACTED]:${crypto.randomUUID()}`;
+      d.run(`UPDATE memories SET
+        key = ?,
         value = '[REDACTED]',
         summary = NULL,
         tags = '[]',
         metadata = '{}',
         updated_at = ?
-       WHERE id = ?`, [timestamp, row.id]);
-    d.run("DELETE FROM memory_tags WHERE memory_id = ?", [row.id]);
-    memoryIds.push(row.id);
-  }
+       WHERE id = ?`, [redactedKey, timestamp, row.id]);
+      d.run("DELETE FROM memory_tags WHERE memory_id = ?", [row.id]);
+      ids.push(row.id);
+    }
+    return ids;
+  });
   return {
     erased_count: memoryIds.length,
     memory_ids: memoryIds,
@@ -12897,6 +12982,31 @@ var init_pg_migrations = __esm(() => {
   );
   CREATE INDEX IF NOT EXISTS idx_task_comments_task ON task_comments(task_id);
   CREATE INDEX IF NOT EXISTS idx_task_comments_agent ON task_comments(agent_id);
+  `,
+    `
+  CREATE OR REPLACE FUNCTION snapshot_memory_version() RETURNS trigger AS $$
+  BEGIN
+    INSERT INTO memory_versions (
+      id, memory_id, version, value, importance, scope, category, tags,
+      summary, pinned, status, when_to_use, created_at
+    ) VALUES (
+      gen_random_uuid()::text,
+      OLD.id, OLD.version, OLD.value, OLD.importance, OLD.scope, OLD.category,
+      OLD.tags, OLD.summary, OLD.pinned, OLD.status, OLD.when_to_use,
+      OLD.updated_at
+    ) ON CONFLICT DO NOTHING;
+    RETURN NEW;
+  END;
+  $$ LANGUAGE plpgsql;
+
+  DROP TRIGGER IF EXISTS memories_version_snapshot ON memories;
+  CREATE TRIGGER memories_version_snapshot
+    BEFORE UPDATE ON memories
+    FOR EACH ROW
+    WHEN (NEW.version > OLD.version)
+    EXECUTE FUNCTION snapshot_memory_version();
+
+  INSERT INTO _migrations (id) VALUES (36) ON CONFLICT DO NOTHING;
   `
   ];
 });
@@ -55710,7 +55820,7 @@ function getAgent(idOrName, db) {
   row = d.query("SELECT * FROM agents WHERE LOWER(name) = ?").get(idOrName.trim().toLowerCase());
   if (row)
     return parseAgentRow(row);
-  const rows = d.query("SELECT * FROM agents WHERE id LIKE ?").all(`${idOrName}%`);
+  const rows = d.query("SELECT * FROM agents WHERE id LIKE ? ESCAPE '\\'").all(`${escapeLikePrefix(idOrName)}%`);
   if (rows.length === 1)
     return parseAgentRow(rows[0]);
   return null;
@@ -63762,7 +63872,7 @@ ${lines.join(`
     }
   });
   server.tool("memory_gdpr_erase", "GDPR right to be forgotten: erase all memories containing a PII identifier. Replaces content with [REDACTED], preserves anonymized audit trail. IRREVERSIBLE.", {
-    identifier: z.string().describe("PII to search for and erase (name, email, etc.)"),
+    identifier: z.string().min(1).describe("PII to search for and erase (name, email, etc.)"),
     project_id: z.string().optional(),
     dry_run: z.boolean().optional().describe("Preview what would be erased without actually erasing (default: false)")
   }, async (args) => {
@@ -65427,11 +65537,12 @@ function hasFlag(...flags) {
 function printHelp() {
   process.stdout.write(`Usage: mementos-mcp [options]
 
-Mementos MCP server (stdio transport by default)
+Mementos MCP server (Streamable HTTP transport by default)
 
 Options:
-  --http           Serve MCP over Streamable HTTP (127.0.0.1)
-  --port <number>  HTTP port (default: 8824, env: MCP_HTTP_PORT)
+  --http           Serve MCP over Streamable HTTP (default, 127.0.0.1)
+  --stdio          Serve MCP over stdio (env: MCP_STDIO=1)
+  --port <number>  HTTP port (default: 8867, env: MCP_HTTP_PORT)
   -h, --help       Show help
   -V, --version    Show version
 `);

package/dist/mcp/tools/system-tools-memory-admin.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"system-tools-memory-admin.d.ts","sourceRoot":"","sources":["../../../src/mcp/tools/system-tools-memory-admin.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAA6B,MAAM,0BAA0B,CAAC;AAI1F,wBAAgB,8BAA8B,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,YAAY,EAAE,SAAS,EAAE,WAAW,EAAE,SAAS,EAAE,iBAAiB,EAAE,EAAE,cAAc,GAAG,IAAI,CAmTtJ"}
+{"version":3,"file":"system-tools-memory-admin.d.ts","sourceRoot":"","sources":["../../../src/mcp/tools/system-tools-memory-admin.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,cAAc,EAA6B,MAAM,0BAA0B,CAAC;AAI1F,wBAAgB,8BAA8B,CAAC,EAAE,MAAM,EAAE,CAAC,EAAE,YAAY,EAAE,SAAS,EAAE,WAAW,EAAE,SAAS,EAAE,iBAAiB,EAAE,EAAE,cAAc,GAAG,IAAI,CAwTtJ"}