agenr 0.9.17 → 0.9.18

package/CHANGELOG.md

@@ -1,5 +1,36 @@
 # Changelog
 
+## 0.9.18 (2026-02-28)
+
+### Features
+- OpenClaw plugin: subsequent-turn auto-recall with heuristic message
+  classifier. Messages are classified as trivial (skip), normal (5 results),
+  or complex (8 results) based on entity detection, temporal references,
+  and explicit recall phrases. Queries built from a sliding window of
+  recent messages with Jaccard similarity dedup. Recalled entries are
+  deduplicated against session-start context. Configurable via
+  midSessionRecall plugin config.
+
+### Bug Fixes
+- MCP: fix stdout corruption during store and contradiction checks.
+  Diagnostic logs in db/store.ts and db/contradiction.ts were writing to
+  stdout via console.log, corrupting MCP JSON-RPC framing. Routed all
+  diagnostic logging to stderr via console.error.
+
+### Changed
+- Removed agenr_store tool from MCP server. Coding agents should rely on
+  Watcher for knowledge ingest from session transcripts. Reduces MCP
+  surface area and eliminates stdout corruption risk.
+- Removed store option from agenr_extract MCP tool. Extract now returns
+  extracted entries without storing them.
+
+### Tests
+- 16-case message classifier test suite
+- Query builder, similarity check, and state management tests
+- Integration tests for mid-session recall in before_prompt_build
+- MCP stdout corruption test
+- Updated MCP server tests for removed store tool and extract store option
+
 ## 0.9.17 - 2026-02-27
 
 ### Changed

package/dist/cli-main.js

@@ -5604,14 +5604,14 @@ async function classifyConflict(llmClient, newEntry, existing, model, config) {
       verbose: false
     });
     if (response.stopReason === "error" || response.errorMessage) {
-      console.warn(
+      console.error(
         `[contradiction] LLM judge error: ${response.errorMessage ?? "unknown stop reason"}`
       );
       return llmErrorResult();
     }
     const parsed = extractJudgeArgs(response);
     if (!parsed) {
-      console.warn("[contradiction] LLM judge returned unparseable response");
+      console.error("[contradiction] LLM judge returned unparseable response");
       return llmErrorResult();
     }
     return {
@@ -5620,7 +5620,7 @@ async function classifyConflict(llmClient, newEntry, existing, model, config) {
       explanation: parsed.explanation
     };
   } catch (err) {
-    console.warn(
+    console.error(
       `[contradiction] LLM judge call failed: ${err instanceof Error ? err.message : String(err)}`
     );
     return llmErrorResult();
@@ -5724,7 +5724,7 @@ async function resolveConflict(db, newEntryId, newEntry, conflict, subjectIndex,
       conflict.result.confidence,
       "coexist"
     );
-    console.log(
+    console.error(
       `[contradiction] resolution: coexist (events are immutable) entry=${conflict.existingEntryId.slice(0, 8)}`
     );
     return { action: "coexist", reason: "events are immutable" };
@@ -5753,7 +5753,7 @@ async function resolveConflict(db, newEntryId, newEntry, conflict, subjectIndex,
       conflict.result.confidence,
       "auto-superseded"
     );
-    console.log(
+    console.error(
       `[contradiction] resolution: auto-superseded entry=${conflict.existingEntryId.slice(0, 8)} (${conflict.existingType} confidence=${conflict.result.confidence.toFixed(2)})`
     );
     return {
@@ -5770,7 +5770,7 @@ async function resolveConflict(db, newEntryId, newEntry, conflict, subjectIndex,
       conflict.result.confidence,
       "pending"
     );
-    console.log(
+    console.error(
       `[contradiction] resolution: flagged for review entry=${conflict.existingEntryId.slice(0, 8)} (${conflict.result.relation} confidence=${conflict.result.confidence.toFixed(2)})`
     );
     return { action: "flagged", reason: "needs human review" };
@@ -5784,7 +5784,7 @@ async function resolveConflict(db, newEntryId, newEntry, conflict, subjectIndex,
       conflict.result.confidence,
       "pending"
     );
-    console.log(
+    console.error(
       `[contradiction] resolution: flagged for review entry=${conflict.existingEntryId.slice(0, 8)} (high-confidence supersedes blocked by importance)`
     );
     return { action: "flagged", reason: "high-confidence supersession blocked by importance" };
@@ -5797,7 +5797,7 @@ async function resolveConflict(db, newEntryId, newEntry, conflict, subjectIndex,
     conflict.result.confidence,
     "coexist"
   );
-  console.log(
+  console.error(
     `[contradiction] resolution: coexist entry=${conflict.existingEntryId.slice(0, 8)}`
   );
   return { action: "coexist", reason: "entries can coexist" };
@@ -7474,7 +7474,7 @@ async function storeEntries(db, entries, apiKey, options = {}) {
     }
     if (!entityHintsPromise) {
       entityHintsPromise = getDistinctEntities(db).catch((err) => {
-        console.warn(
+        console.error(
           `[store] entity hints lookup failed, proceeding without hints: ${err instanceof Error ? err.message : String(err)}`
         );
         return [];
@@ -7679,7 +7679,7 @@ async function storeEntries(db, entries, apiKey, options = {}) {
     }
     if ((processed.mutation.kind === "add" || processed.mutation.kind === "add_related") && options.llmClient && options.contradictionEnabled !== false && processed.mutation.embedding) {
       await contradictionSubjectIndex.ensureInitialized(db);
-      console.log(
+      console.error(
         `[contradiction] checking entry: type=${normalizedEntry.type} subject="${normalizedEntry.subject}" subjectKey=${normalizedEntry.subjectKey ?? "none"}`
       );
       const conflicts = await detectContradictions(
@@ -7700,11 +7700,11 @@ async function storeEntries(db, entries, apiKey, options = {}) {
         }
       );
       if (conflicts.length === 0) {
-        console.log("[contradiction] no conflicts detected");
+        console.error("[contradiction] no conflicts detected");
       } else {
-        console.log(`[contradiction] found ${conflicts.length} conflict(s):`);
+        console.error(`[contradiction] found ${conflicts.length} conflict(s):`);
         for (const c of conflicts) {
-          console.log(
+          console.error(
             `[contradiction]   vs entry=${c.existingEntryId.slice(0, 8)} relation=${c.result.relation} confidence=${c.result.confidence.toFixed(2)} explanation="${c.result.explanation.slice(0, 80)}"`
           );
         }
@@ -20756,15 +20756,13 @@ function normalizeSlugList(value) {
 function isPlatform(value) {
   return PLATFORM_VALUES.includes(value);
 }
-function buildSystemPromptBlock(projectSlug) {
+function buildSystemPromptBlock(_projectSlug) {
   return [
-    "You have access to agenr_recall and agenr_store for persistent memory across sessions.",
+    "You have access to agenr_recall for persistent memory across sessions.",
     "",
     'On session start, call agenr_recall with context="session-start" to load prior knowledge for this project. Mid-session, call agenr_recall with a query when you need context you do not have.',
     "",
-    `After any decision, user preference, lesson learned, or important event, immediately call agenr_store. Do not ask - just store it. Each entry needs: type (fact|decision|preference|todo|lesson|event), content (what and why), importance (1-10, default 7, use 9 for critical, 10 sparingly), project="${projectSlug}".`,
-    "",
-    "Do not store: secrets/credentials, temporary state, verbatim conversation, or information already in files."
+    "Do NOT call agenr_store. That tool has been removed. Your session transcript is automatically ingested by the Watcher, which extracts valuable knowledge after the session ends. Focus on your task - the memory system captures insights for you."
   ].join("\n");
 }
 async function pathExists(targetPath) {
@@ -22427,7 +22425,6 @@ var RpcError = class extends Error {
   }
 };
 var KNOWLEDGE_TYPE_SET2 = new Set(KNOWLEDGE_TYPES);
-var SCOPE_LEVEL_SET = new Set(SCOPE_LEVELS);
 var TOOL_DEFINITIONS = [
   {
     name: "agenr_recall",
@@ -22491,56 +22488,6 @@ var TOOL_DEFINITIONS = [
       }
     }
   },
-  {
-    name: "agenr_store",
-    description: "Store new knowledge entries in the database.",
-    inputSchema: {
-      type: "object",
-      additionalProperties: false,
-      required: ["entries"],
-      properties: {
-        platform: {
-          type: "string",
-          description: "Optional platform tag for all entries: openclaw, claude-code, codex."
-        },
-        project: {
-          type: "string",
-          description: "Optional project tag for all entries (lowercase)."
-        },
-        entries: {
-          type: "array",
-          items: {
-            type: "object",
-            additionalProperties: false,
-            required: ["content", "type"],
-            properties: {
-              content: { type: "string", minLength: 1 },
-              type: {
-                type: "string",
-                enum: [...KNOWLEDGE_TYPES]
-              },
-              importance: {
-                type: "integer",
-                minimum: 1,
-                maximum: 10,
-                default: 7
-              },
-              source: { type: "string" },
-              tags: {
-                type: "array",
-                items: { type: "string" }
-              },
-              scope: {
-                type: "string",
-                enum: [...SCOPE_LEVELS],
-                default: "personal"
-              }
-            }
-          }
-        }
-      }
-    }
-  },
   {
     name: "agenr_extract",
     description: "Extract knowledge entries from raw text.",
@@ -22554,11 +22501,6 @@ var TOOL_DEFINITIONS = [
           minLength: 1,
           description: "Raw text to extract knowledge from."
         },
-        store: {
-          type: "boolean",
-          default: false,
-          description: "Whether to store extracted entries."
-        },
         source: {
           type: "string",
           description: "Optional source label for extracted entries."
@@ -22717,95 +22659,6 @@ function parseCsvProjects(input) {
   }
   return parsed;
 }
-function normalizeTags3(value) {
-  if (!Array.isArray(value)) {
-    return [];
-  }
-  return Array.from(
-    new Set(
-      value.filter((item) => typeof item === "string").map((item) => item.trim().toLowerCase()).filter((item) => item.length > 0)
-    )
-  );
-}
-function parseScope(value) {
-  if (value === void 0 || value === null || String(value).trim().length === 0) {
-    return "personal";
-  }
-  if (typeof value !== "string") {
-    throw new RpcError(JSON_RPC_INVALID_PARAMS, "scope must be a string");
-  }
-  const normalized = value.trim().toLowerCase();
-  if (!SCOPE_LEVEL_SET.has(normalized)) {
-    throw new RpcError(JSON_RPC_INVALID_PARAMS, `Invalid scope: ${value}`);
-  }
-  return normalized;
-}
-function normalizeImportance2(value) {
-  const parsed = value === void 0 ? 7 : Number(value);
-  if (!Number.isInteger(parsed) || parsed < 1 || parsed > 10) {
-    throw new RpcError(JSON_RPC_INVALID_PARAMS, "importance must be an integer between 1 and 10");
-  }
-  return parsed;
-}
-function inferSubject(content) {
-  const trimmed = content.trim();
-  if (!trimmed) {
-    return "memory";
-  }
-  const firstClause = trimmed.split(/[.!?:;]\s/)[0].trim();
-  if (!firstClause) {
-    return "memory";
-  }
-  if (firstClause.length <= 80) {
-    return firstClause;
-  }
-  const cut = firstClause.lastIndexOf(" ", 80);
-  return cut > 0 ? firstClause.slice(0, cut) : firstClause.slice(0, 80);
-}
-function parseStoreEntries(rawEntries) {
-  if (!Array.isArray(rawEntries)) {
-    throw new RpcError(JSON_RPC_INVALID_PARAMS, "entries must be an array");
-  }
-  return rawEntries.map((raw, index) => {
-    if (!isRecord2(raw)) {
-      throw new RpcError(JSON_RPC_INVALID_PARAMS, `entries[${index}] must be an object`);
-    }
-    const type = typeof raw.type === "string" ? raw.type.trim().toLowerCase() : "";
-    if (!type || !KNOWLEDGE_TYPE_SET2.has(type)) {
-      throw new RpcError(JSON_RPC_INVALID_PARAMS, `entries[${index}].type is invalid`);
-    }
-    const content = typeof raw.content === "string" ? raw.content.trim() : "";
-    if (!content) {
-      throw new RpcError(JSON_RPC_INVALID_PARAMS, `entries[${index}].content is required`);
-    }
-    const importance = (() => {
-      try {
-        return normalizeImportance2(raw.importance);
-      } catch {
-        throw new RpcError(
-          JSON_RPC_INVALID_PARAMS,
-          `entries[${index}].importance must be an integer between 1 and 10`
-        );
-      }
-    })();
-    const source = typeof raw.source === "string" && raw.source.trim().length > 0 ? raw.source.trim() : "mcp:agenr_store";
-    const subject = typeof raw.subject === "string" && raw.subject.trim().length > 0 ? raw.subject.trim() : inferSubject(content);
-    const scope = parseScope(raw.scope);
-    return {
-      type,
-      subject,
-      content,
-      importance,
-      expiry: "temporary",
-      tags: normalizeTags3(raw.tags),
-      scope,
-      source: {
-        file: source,
-        context: "stored via MCP tool"
-      }
-    };
-  });
-}
 function formatDate(value) {
   if (!value) {
     return "unknown-date";
@@ -22857,15 +22710,7 @@ function formatBrowseText(results) {
   }
   return lines.join("\n");
 }
-function formatStoreSummary(result) {
-  const total = result.added + result.updated + result.skipped + result.superseded;
-  const parts = [`${result.added} new`, `${result.updated} updated`, `${result.skipped} duplicates skipped`];
-  if (result.superseded > 0) {
-    parts.push(`${result.superseded} superseded`);
-  }
-  return `Stored ${total} entries (${parts.join(", ")}).`;
-}
-function formatExtractedText(entries, stored) {
+function formatExtractedText(entries) {
   const lines = [`Extracted ${entries.length} entries from text:`, ""];
   for (let i = 0; i < entries.length; i += 1) {
     const entry = entries[i];
@@ -22874,12 +22719,6 @@ function formatExtractedText(entries, stored) {
     }
     lines.push(`[${i + 1}] (${entry.type}) ${entry.content}`);
   }
-  if (stored) {
-    lines.push("");
-    lines.push(
-      `Stored: ${stored.added} new, ${stored.updated} updated, ${stored.skipped} duplicates skipped, ${stored.superseded} superseded.`
-    );
-  }
   return lines.join("\n");
 }
 function extractIdForError(raw) {
@@ -22919,7 +22758,6 @@ function createMcpServer(options = {}, deps = {}) {
     closeDbFn: deps.closeDbFn ?? closeDb,
     recallFn: deps.recallFn ?? recall,
     updateRecallMetadataFn: deps.updateRecallMetadataFn ?? updateRecallMetadata,
-    storeEntriesFn: deps.storeEntriesFn ?? storeEntries,
     retireEntriesFn: deps.retireEntriesFn ?? retireEntries,
     parseTranscriptFileFn: deps.parseTranscriptFileFn ?? parseTranscriptFile,
     extractKnowledgeFromChunksFn: deps.extractKnowledgeFromChunksFn ?? extractKnowledgeFromChunks,
@@ -23153,56 +22991,11 @@ function createMcpServer(options = {}, deps = {}) {
     }
     return formatRecallText(query || context, filtered);
   }
-  async function callStoreTool(args) {
-    if (!hasOwn(args, "entries")) {
-      throw new RpcError(JSON_RPC_INVALID_PARAMS, "entries is required");
-    }
-    const platformRaw = typeof args.platform === "string" ? args.platform.trim() : "";
-    const platform = platformRaw ? normalizeKnowledgePlatform(platformRaw) : null;
-    if (platformRaw && !platform) {
-      throw new RpcError(
-        JSON_RPC_INVALID_PARAMS,
-        `platform must be one of: ${KNOWLEDGE_PLATFORMS.join(", ")}`
-      );
-    }
-    const projectRaw = typeof args.project === "string" ? args.project.trim() : "";
-    const explicitProject = projectRaw ? normalizeProject(projectRaw) : null;
-    if (projectRaw && !explicitProject) {
-      throw new RpcError(JSON_RPC_INVALID_PARAMS, "project must be a non-empty string");
-    }
-    const scoped = !explicitProject ? await loadScopedProjectConfig() : null;
-    const project = explicitProject ?? scoped?.project ?? null;
-    const parsed = parseStoreEntries(args.entries);
-    const entries = platform || project ? parsed.map((entry) => ({
-      ...entry,
-      ...platform ? { platform } : {},
-      ...project ? { project } : {}
-    })) : parsed;
-    const db = await ensureDb();
-    const config = resolvedDeps.readConfigFn(env);
-    const aggressiveDedup = config?.dedup?.aggressive === true;
-    const configDedupThreshold = typeof config?.dedup?.threshold === "number" ? config.dedup.threshold : void 0;
-    const contradictionEnabled = config?.contradiction?.enabled !== false;
-    const dedupClient = resolvedDeps.createLlmClientFn({ config, env });
-    const apiKey = resolvedDeps.resolveEmbeddingApiKeyFn(config, env);
-    const result = await resolvedDeps.storeEntriesFn(db, entries, apiKey, {
-      sourceFile: "mcp:agenr_store",
-      onlineDedup: true,
-      llmClient: dedupClient,
-      aggressiveDedup,
-      dedupThreshold: configDedupThreshold,
-      contradictionEnabled,
-      config: config ?? void 0,
-      dbPath: options.dbPath
-    });
-    return formatStoreSummary(result);
-  }
   async function callExtractTool(args) {
     const text4 = typeof args.text === "string" ? args.text : "";
     if (!text4.trim()) {
       throw new RpcError(JSON_RPC_INVALID_PARAMS, "text is required");
     }
-    const shouldStore = args.store === true;
     const sourceLabel = typeof args.source === "string" && args.source.trim().length > 0 ? args.source.trim() : void 0;
     const tempDir = await resolvedDeps.mkdtempFn(path31.join(resolvedDeps.tmpdirFn(), "agenr-mcp-"));
     const tempFile = path31.join(tempDir, "input.txt");
@@ -23226,32 +23019,7 @@ function createMcpServer(options = {}, deps = {}) {
           context: entry.source.context
         }
       }));
-      let stored;
-      if (shouldStore && extractedEntries.length > 0) {
-        const db = await ensureDb();
-        const apiKey = resolvedDeps.resolveEmbeddingApiKeyFn(config, env);
-        const contradictionEnabled = config?.contradiction?.enabled !== false;
-        stored = await resolvedDeps.storeEntriesFn(db, extractedEntries, apiKey, {
-          sourceFile: sourceLabel ?? "mcp:agenr_extract",
-          onlineDedup: true,
-          llmClient: client,
-          contradictionEnabled,
-          config: config ?? void 0,
-          dbPath: options.dbPath
-        });
-      } else if (shouldStore) {
-        stored = {
-          added: 0,
-          updated: 0,
-          skipped: 0,
-          superseded: 0,
-          llm_dedup_calls: 0,
-          relations_created: 0,
-          total_entries: 0,
-          duration_ms: 0
-        };
-      }
-      return formatExtractedText(extractedEntries, stored);
+      return formatExtractedText(extractedEntries);
     } finally {
       await resolvedDeps.rmFn(tempDir, { recursive: true, force: true });
     }
@@ -23310,22 +23078,6 @@ function createMcpServer(options = {}, deps = {}) {
           content: [{ type: "text", text: result }]
         };
       }
-      if (params.name === "agenr_store") {
-        const result = await callStoreTool(params.args);
-        const storeArgsEntries = Array.isArray(params.args.entries) ? params.args.entries : [];
-        const firstEntry = storeArgsEntries[0];
-        await appendMcpLog({
-          tool: "agenr_store",
-          count: storeArgsEntries.length,
-          firstType: typeof firstEntry?.type === "string" ? firstEntry.type : void 0,
-          firstSubject: typeof firstEntry?.subject === "string" ? firstEntry.subject.slice(0, 80) : void 0,
-          firstImportance: typeof firstEntry?.importance === "number" ? firstEntry.importance : void 0,
-          project: typeof params.args.project === "string" ? params.args.project : void 0
-        });
-        return {
-          content: [{ type: "text", text: result }]
-        };
-      }
       if (params.name === "agenr_extract") {
         return {
           content: [{ type: "text", text: await callExtractTool(params.args) }]
@@ -23783,8 +23535,8 @@ async function runRecallCommand(queryInput, options, deps) {
         budget,
         nonCoreLimit: limit
       });
-      finalResults = grouped.results;
-      budgetUsed = grouped.budgetUsed;
+      finalResults = grouped.results.slice(0, limit);
+      budgetUsed = finalResults.reduce((sum, item) => sum + estimateEntryTokens(item), 0);
     } else {
       const baseResults = await resolvedDeps.recallFn(db, queryForRecall, apiKey);
       if (budget === void 0) {
@@ -24187,7 +23939,7 @@ function readStdin() {
     process.stdin.on("error", reject);
   });
 }
-function normalizeTags4(value) {
+function normalizeTags3(value) {
   if (!Array.isArray(value)) {
     return [];
   }
@@ -24248,7 +24000,7 @@ function normalizeEntry(raw, fallbackFile) {
     content,
     importance,
     expiry,
-    tags: normalizeTags4(record.tags),
+    tags: normalizeTags3(record.tags),
     created_at: normalizeCreatedAt2(record.created_at),
     source: {
       file: sourceFile,

package/dist/openclaw-plugin/index.d.ts

@@ -92,7 +92,7 @@ declare function capTranscriptLength(params: {
     currentSurface: string;
     maxChars: number;
 }): string;
-declare function summarizeSessionForHandoff(currentRawMessages: BeforeResetEvent["messages"], sessionsDir: string, currentSessionFile: string, logger: PluginApi["logger"], includeBackground: boolean, streamSimpleImpl?: StreamSimpleFn, logEnabled?: boolean, logDir?: string): Promise<string | null>;
+declare function summarizeSessionForHandoff(currentRawMessages: BeforeResetEvent["messages"], sessionsDir: string, currentSessionFile: string, logger: PluginApi["logger"], includeBackground: boolean, streamSimpleImpl?: StreamSimpleFn, logEnabled?: boolean, logDir?: string, debugEnabled?: boolean): Promise<string | null>;
 declare function runHandoffForSession(opts: {
     messages: unknown[];
     sessionFile: string | null;
@@ -107,6 +107,7 @@ declare function runHandoffForSession(opts: {
     includeBackground?: boolean;
     logEnabled?: boolean;
     logDir?: string;
+    debugEnabled?: boolean;
     logger: PluginLogger | undefined;
     source: "before_reset" | "command" | "session_start";
     dbPath?: string;