@openviking/dsh-memory-plugin 0.5.14 → 0.5.16

package/README.md

@@ -117,6 +117,15 @@ context under such presets with no error. Pre-step injection also makes each
 injection a session event that replays, is visible to compaction, and never
 reaches `request/header`.
 
+### What triggers recall
+
+Recall runs on a pre-step whose entering batch contains user-authored input.
+The query is built only from those messages: user-role messages with
+`source.kind` absent or `user`. Messages that DSH or other plugins inject
+(`time-context`, `plugin:*` notices, this plugin's own blocks) and tool results
+are excluded, using the same rule capture uses. A batch made only of injected
+context sends no search request.
+
 ### How the tool surface is mounted
 
 `mcp.mjs` mounts `@deepseek-ai/dsh-mcp-client` on `servers/mcp-proxy.mjs`, the

package/capture.mjs

@@ -96,9 +96,18 @@ function captureMessage(event, message, config, toolNames) {
   return payload;
 }
 
+// Recall queries use only what the user typed. Injected context (time-context,
+// job notices, other plugins' blocks, our own recall) and tool results would
+// otherwise trigger recall on their own and pollute the query text.
+function isUserInput(message) {
+  if (!message || message.role !== "user") return false;
+  if (message.source?.kind === "tool") return false;
+  return !isSyntheticUserMessage(message);
+}
+
 export function promptText(messages) {
   return (messages || [])
-    .filter(message => !isOpenVikingPluginMessage(message))
+    .filter(isUserInput)
     .map(message => extractTextFromPayload(message))
     .filter(Boolean)
     .join("\n\n")

package/config.mjs

@@ -1,7 +1,7 @@
 import { buildPluginConfig } from "./shared/plugin-config.mjs";
 import { loadCredentialFiles } from "./shared/credentials.mjs";
 
-export const PLUGIN_VERSION = "0.5.14";
+export const PLUGIN_VERSION = "0.5.16";
 
 /**
  * Namespace for the bridged OpenViking MCP tools. DSH publishes every MCP tool

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@openviking/dsh-memory-plugin",
-  "version": "0.5.14",
+  "version": "0.5.16",
   "description": "OpenViking memory and context bundle for DeepSeek Harness",
   "icon": "./icon.svg",
   "type": "module",

package/runtime.mjs

@@ -131,7 +131,7 @@ export class OpenVikingRuntime {
     const state = await this.initialize(agent);
     if (!state.ready || !isRecallEnabled(state.config)) return null;
     const query = promptText(messages);
-    if (query.length < state.config.minQueryLength) return null;
+    if (!query || query.length < state.config.minQueryLength) return null;
     const block = await buildRecallBlock(
       (path, init, options) => this.client.fetchJSON(path, init, options),
       state.config,

package/shared/config-schema.mjs

@@ -160,7 +160,7 @@ export const KNOBS = [
   { name: "noAutoInject", type: "bool", default: false, env: "OPENVIKING_NO_AUTO_INJECT", capability: "session" },
   { name: "profileTokenBudget", type: "int", default: 10000, min: 500, max: 50000, env: "OPENVIKING_PROFILE_TOKEN_BUDGET", aliases: ["profileBudget"], capability: "session" },
   { name: "resumeContextBudget", type: "int", default: 32000, min: 1024, max: 128000, env: "OPENVIKING_RESUME_CONTEXT_BUDGET", capability: "session" },
-  { name: "resumeArchiveInject", type: "bool", default: true, env: "OPENVIKING_RESUME_ARCHIVE_INJECT", capability: "session" },
+  { name: "resumeArchiveInject", type: "bool", default: false, env: "OPENVIKING_RESUME_ARCHIVE_INJECT", capability: "session" },
   { name: "resumeArchiveTokenBudget", type: "int", default: 32000, min: 0, max: 128000, env: "OPENVIKING_RESUME_ARCHIVE_TOKEN_BUDGET", capability: "session" },
   { name: "resumeArchiveMaxChars", type: "int", default: 6000, min: 1000, max: 200000, env: "OPENVIKING_RESUME_ARCHIVE_MAX_CHARS", capability: "session" },
   { name: "skillExperience", type: "bool", default: false, env: "OPENVIKING_SKILL_EXPERIENCE", capability: "session" },
@@ -184,7 +184,7 @@ export const KNOBS = [
   { name: "repoContext", type: "bool", default: true, capability: "session" },
   { name: "repoContextCacheTtlMs", type: "int", default: 60000, min: 1000, max: 3600000, capability: "session" },
 
-  { name: "takeoverEnabled", type: "bool", default: true, env: "OPENVIKING_TAKEOVER", capability: "session" },
+  { name: "takeoverEnabled", type: "bool", default: false, env: "OPENVIKING_TAKEOVER", capability: "session" },
   { name: "takeoverTokenThreshold", type: "int", default: 30000, min: 1, max: 1000000, capability: "session" },
   { name: "takeoverKeepRecentTurns", type: "int", default: 3, min: 0, max: 100, capability: "session" },
   { name: "takeoverOverviewBudget", type: "int", default: 3000, min: 100, max: 50000, capability: "session" },