mr-memory 3.15.0 → 3.16.0

package/index.js

@@ -12,6 +12,7 @@ import { commitReflections, consolidateCheckout, dateSearch, deleteMemories, del
 import { isAutoConsolidationSession, maybeTriggerAutoConsolidate } from "./auto-consolidate.js";
 import { withMemoryInstructions } from "./memory-instructions.js";
 import { join, resolve, relative, isAbsolute, sep, dirname } from "node:path";
+import path from "node:path";
 import os from "node:os";
 const DEFAULT_ENDPOINT = "https://api.memoryrouter.ai";
 /** Keep-warm defaults.
@@ -291,6 +292,14 @@ const WORKSPACE_FILES = [
     "IDENTITY.md", "USER.md", "MEMORY.md", "HEARTBEAT.md",
     "TOOLS.md", "AGENTS.md", "SOUL.md", "BOOTSTRAP.md",
 ];
+/** Explicit host agent id wins; older hosts fall back to workspace basename. */
+export function isReadOnlyAgent(readOnlyAgents, ctx) {
+    const explicit = typeof ctx?.agentId === "string" ? ctx.agentId.trim() : "";
+    const dir = typeof ctx?.workspaceDir === "string"
+        ? path.basename(ctx.workspaceDir.replace(/[\\/]+$/, "")) : "";
+    const agentId = explicit || (dir === "workspace" ? "main" : dir.replace(/^workspace-/, ""));
+    return !!agentId && (readOnlyAgents ?? []).includes(agentId);
+}
 // ──────────────────────────────────────────────────────
 // Helpers
 // ──────────────────────────────────────────────────────
@@ -574,6 +583,7 @@ const memoryRouterPlugin = {
         const logging = cfg?.logging ?? false;
         const embeddings = cfg?.embeddings; // undefined = bge (default), "qwen" = qwen3-8b
         const agentKeys = cfg?.agentKeys;
+        const readOnlyAgents = cfg?.readOnlyAgents;
         const autoConsolidate = cfg?.autoConsolidate !== false; // default true
         const log = (msg) => { if (logging)
             api.logger.info?.(msg); };
@@ -582,6 +592,8 @@ const memoryRouterPlugin = {
         // Fire-and-forget; all rails (lock, cooldown, 3-batch bound) live in
         // auto-consolidate.ts. Failures never propagate into the hook path.
         function triggerAutoConsolidation(reflection, ctx, activeKey) {
+            if (isReadOnlyAgent(readOnlyAgents, ctx))
+                return;
             try {
                 maybeTriggerAutoConsolidate(reflection ?? null, {
                     autoConsolidate,
@@ -879,10 +891,10 @@ If setting up MemoryRouter for another agent:
                     if (data.context) {
                         const elapsed = Date.now() - startMs;
                         api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms) [llm_input]`);
-                        if (elapsed > 2000 && data.metrics)
+                        if (data.metrics)
                             log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
                         let contextText = data.context;
-                        const nudge = buildDebtNudgeLine(data.reflection);
+                        const nudge = isReadOnlyAgent(readOnlyAgents, ctx) ? null : buildDebtNudgeLine(data.reflection);
                         if (nudge)
                             contextText = `${contextText}\n\n${nudge}`;
                         const importNudge = buildImportNudgeLine(data.import);
@@ -993,11 +1005,11 @@ If setting up MemoryRouter for another agent:
                     triggerAutoConsolidation(data.reflection, ctx, activeKey);
                     if (data.context) {
                         const elapsed = Date.now() - startMs;
-                        api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms)`);
-                        if (elapsed > 2000 && data.metrics)
+                        api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms) agent=${ctx?.agentId || "unknown"} session=${ctx?.sessionKey || "unknown"}`);
+                        if (data.metrics)
                             log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
                         let contextText = data.context;
-                        const nudge = buildDebtNudgeLine(data.reflection);
+                        const nudge = isReadOnlyAgent(readOnlyAgents, ctx) ? null : buildDebtNudgeLine(data.reflection);
                         if (nudge)
                             contextText = `${contextText}\n\n${nudge}`;
                         const importNudge = buildImportNudgeLine(data.import);
@@ -1026,6 +1038,10 @@ If setting up MemoryRouter for another agent:
             // Core: agent_end — store this turn's conversation
             // ==================================================================
             api.on("agent_end", async (event, ctx) => {
+                if (isReadOnlyAgent(readOnlyAgents, ctx)) {
+                    api.logger.info?.(`memoryrouter: ingest skipped read-only agent=${ctx?.agentId || path.basename(ctx?.workspaceDir || "")} session=${ctx?.sessionKey || "unknown"}`);
+                    return;
+                }
                 // Never store the auto-consolidation session's own chatter as memory.
                 if (isAutoConsolidationSession(ctx?.sessionKey))
                     return;

package/index.ts

@@ -314,7 +314,17 @@ const WORKSPACE_FILES = [
   "TOOLS.md", "AGENTS.md", "SOUL.md", "BOOTSTRAP.md",
 ];
 
+/** Explicit host agent id wins; older hosts fall back to workspace basename. */
+export function isReadOnlyAgent(readOnlyAgents: string[] | undefined, ctx: any): boolean {
+  const explicit = typeof ctx?.agentId === "string" ? ctx.agentId.trim() : "";
+  const dir = typeof ctx?.workspaceDir === "string"
+    ? path.basename(ctx.workspaceDir.replace(/[\\/]+$/, "")) : "";
+  const agentId = explicit || (dir === "workspace" ? "main" : dir.replace(/^workspace-/, ""));
+  return !!agentId && (readOnlyAgents ?? []).includes(agentId);
+}
+
 type MemoryRouterConfig = {
+  readOnlyAgents?: string[];
   key: string;
   endpoint?: string;
   /** xhigh is accepted only for backward compatibility and normalizes to high. */
@@ -632,6 +642,7 @@ const memoryRouterPlugin = {
     const logging = cfg?.logging ?? false;
     const embeddings = cfg?.embeddings; // undefined = bge (default), "qwen" = qwen3-8b
     const agentKeys = cfg?.agentKeys;
+    const readOnlyAgents = cfg?.readOnlyAgents;
     const autoConsolidate = cfg?.autoConsolidate !== false; // default true
     const log = (msg: string) => { if (logging) api.logger.info?.(msg); };
 
@@ -640,6 +651,7 @@ const memoryRouterPlugin = {
     // Fire-and-forget; all rails (lock, cooldown, 3-batch bound) live in
     // auto-consolidate.ts. Failures never propagate into the hook path.
     function triggerAutoConsolidation(reflection: ReflectionDebt | undefined, ctx: any, activeKey: string): void {
+      if (isReadOnlyAgent(readOnlyAgents, ctx)) return;
       try {
         maybeTriggerAutoConsolidate(reflection ?? null, {
           autoConsolidate,
@@ -946,9 +958,9 @@ If setting up MemoryRouter for another agent:
           if (data.context) {
             const elapsed = Date.now() - startMs;
             api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms) [llm_input]`);
-            if (elapsed > 2000 && data.metrics) log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
+            if (data.metrics) log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
             let contextText = data.context;
-            const nudge = buildDebtNudgeLine(data.reflection);
+            const nudge = isReadOnlyAgent(readOnlyAgents, ctx) ? null : buildDebtNudgeLine(data.reflection);
             if (nudge) contextText = `${contextText}\n\n${nudge}`;
             const importNudge = buildImportNudgeLine(data.import);
             if (importNudge) contextText = `${contextText}\n\n${importNudge}`;
@@ -1072,10 +1084,10 @@ If setting up MemoryRouter for another agent:
 
           if (data.context) {
             const elapsed = Date.now() - startMs;
-            api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms)`);
-            if (elapsed > 2000 && data.metrics) log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
+            api.logger.info?.(`memoryrouter: injected ${data.memories_found || 0} memories (${data.memory_tokens || 0} tokens, ${elapsed}ms) agent=${ctx?.agentId || "unknown"} session=${ctx?.sessionKey || "unknown"}`);
+            if (data.metrics) log(`memoryrouter: prepare metrics ${JSON.stringify(data.metrics)}`);
             let contextText = data.context;
-            const nudge = buildDebtNudgeLine(data.reflection);
+            const nudge = isReadOnlyAgent(readOnlyAgents, ctx) ? null : buildDebtNudgeLine(data.reflection);
             if (nudge) contextText = `${contextText}\n\n${nudge}`;
             const importNudge = buildImportNudgeLine(data.import);
             if (importNudge) contextText = `${contextText}\n\n${importNudge}`;
@@ -1104,6 +1116,10 @@ If setting up MemoryRouter for another agent:
       // ==================================================================
 
       api.on("agent_end", async (event, ctx) => {
+        if (isReadOnlyAgent(readOnlyAgents, ctx)) {
+          api.logger.info?.(`memoryrouter: ingest skipped read-only agent=${ctx?.agentId || path.basename(ctx?.workspaceDir || "")} session=${ctx?.sessionKey || "unknown"}`);
+          return;
+        }
         // Never store the auto-consolidation session's own chatter as memory.
         if (isAutoConsolidationSession(ctx?.sessionKey)) return;
         const activeKey = resolveKey(ctx);

package/memory-instructions.js

@@ -13,7 +13,7 @@ export const MEMORY_INSTRUCTIONS = "MemoryRouter is the user’s external, persi
  */
 export const OPENCLAW_MAPPING = [
     "How this maps to OpenClaw: before each prompt the MemoryRouter plugin already runs the RETRIEVE search and adds the results as <memory_context>, so that search already ran; call memory_search only for more detail (tiers and importance are its filters), memory_inspect for the sources behind a reflection, and memory_date_search for time windows.",
-    "After each turn the plugin saves the user's message and your final reply automatically. There is no separate save tool, so useful details are saved when they appear in the conversation; to supersede outdated information, state the correction plainly so the saved turn records it.",
+    "After each turn the plugin saves the user's message and your final reply automatically. There is no separate save tool, so the STORE rule is met by stating useful details plainly in your reply: when the user makes a decision, states a preference, gives a fact about a person, project, or tool, sets a plan or commitment, makes a correction, or reports an outcome, restate it with the exact wording, numbers, and identifiers so the saved turn records it. To supersede outdated information, state the correction and what it replaces.",
     "If the user asks what was searched or saved, answer truthfully.",
 ].join(" ");
 export const OPENCLAW_MEMORY_INSTRUCTIONS = `MemoryRouter instructions\n${MEMORY_INSTRUCTIONS}\n\n${OPENCLAW_MAPPING}`;

package/memory-instructions.ts

@@ -14,7 +14,7 @@ export const MEMORY_INSTRUCTIONS = "MemoryRouter is the user’s external, persi
  */
 export const OPENCLAW_MAPPING = [
   "How this maps to OpenClaw: before each prompt the MemoryRouter plugin already runs the RETRIEVE search and adds the results as <memory_context>, so that search already ran; call memory_search only for more detail (tiers and importance are its filters), memory_inspect for the sources behind a reflection, and memory_date_search for time windows.",
-  "After each turn the plugin saves the user's message and your final reply automatically. There is no separate save tool, so useful details are saved when they appear in the conversation; to supersede outdated information, state the correction plainly so the saved turn records it.",
+  "After each turn the plugin saves the user's message and your final reply automatically. There is no separate save tool, so the STORE rule is met by stating useful details plainly in your reply: when the user makes a decision, states a preference, gives a fact about a person, project, or tool, sets a plan or commitment, makes a correction, or reports an outcome, restate it with the exact wording, numbers, and identifiers so the saved turn records it. To supersede outdated information, state the correction and what it replaces.",
   "If the user asks what was searched or saved, answer truthfully.",
 ].join(" ");
 

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "mr-memory",
   "name": "MemoryRouter Memory",
   "description": "Persistent memory for OpenClaw, powered by MemoryRouter. Memory that survives sessions, compaction, and model changes.",
-  "version": "3.15.0",
+  "version": "3.16.0",
   "activation": {
     "onStartup": true,
     "onCommands": [
@@ -92,6 +92,13 @@
           "pattern": "^mk[_-]"
         },
         "description": "Optional per-agent MemoryRouter keys, keyed by OpenClaw agent id"
+      },
+      "readOnlyAgents": {
+        "type": "array",
+        "items": {
+          "type": "string"
+        },
+        "description": "Agent IDs that receive memory retrieval but skip automatic ingest and automatic consolidation. Explicit tool writes must be separately restricted."
       }
     },
     "required": []

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mr-memory",
-  "version": "3.15.0",
+  "version": "3.16.0",
   "description": "Persistent memory for OpenClaw, powered by MemoryRouter. Relay-based AI agent memory across sessions, compaction, and model changes.",
   "type": "module",
   "files": [