@lmzhen/dsh-tool-memory 0.17.1 → 0.18.0

package/lib/index.js

@@ -1,7 +1,7 @@
 import { effectiveSessionPolicy } from "@lmzhen/dsh-evolution-approval";
 import z from "@deepseek-ai/schemastery";
 import { defineTool } from "@deepseek-ai/dsh-tools";
-import { MEMORY_GUIDANCE_SECTION_ORDER, clampedNumber, resolveExecOrigins } from "@lmzhen/dsh-evolution-core";
+import { MEMORY_GUIDANCE_SECTION_ORDER, clampedNumber, payloadText, resolveExecOrigins } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Model-facing memory tool and runtime-context memory snapshot.
@@ -35,7 +35,7 @@ async function apply(ctx, rawConfig = {}) {
 	ctx.inject(["settings"], (injected) => {
 		const settings = injected.settings;
 		if (typeof settings?.configure !== "function") return;
-		const disposer = settings.configure({ auto: false });
+		const disposer = settings.configure({ auto: false }, ctx.fiber);
 		if (typeof disposer === "function") ctx.effect(() => disposer, "tool-memory: settings presentation");
 	});
 	const memoryDisabled = rawConfig.memoryEnabled === false;
@@ -84,6 +84,7 @@ async function apply(ctx, rawConfig = {}) {
 					} catch {}
 				})();
 			}, 500);
+			retryTimer.unref();
 			ctx.effect(() => () => {
 				clearInterval(retryTimer);
 			}, "tool-memory.snapshot-retry");
@@ -111,7 +112,11 @@ async function apply(ctx, rawConfig = {}) {
 			chars: 0,
 			limit: 0
 		};
-		for (const field of ["facts", "old_text"]) {
+		for (const field of [
+			"facts",
+			"old_text",
+			"content"
+		]) {
 			const value = normalized[field];
 			if (value !== void 0 && value !== null && typeof value !== "string") return {
 				ok: false,
@@ -121,6 +126,21 @@ async function apply(ctx, rawConfig = {}) {
 				limit: 0
 			};
 		}
+		for (const [index, operation] of (normalized.operations ?? []).entries()) for (const field of [
+			"facts",
+			"content",
+			"old_text"
+		]) {
+			const value = operation[field];
+			if (value === void 0 || value === null || typeof value === "string") continue;
+			return {
+				ok: false,
+				message: `memory: operations[${index}].${field} must be a string (got ${typeof value}); refusing the write — re-approving cannot fix a payload defect, so reject the record.`,
+				entries: [],
+				chars: 0,
+				limit: 0
+			};
+		}
 		const result = normalized.operations ? await ctx.memory.applyBatch(normalized.target, normalized.operations) : await ctx.memory.applyBatch(normalized.target, [{
 			action: normalized.action ?? "add",
 			facts: normalized.facts,
@@ -135,7 +155,7 @@ async function apply(ctx, rawConfig = {}) {
 		};
 	}
 	const requiredFieldRejection = (op, position) => {
-		const body = (op.facts ?? op.content ?? "").trim();
+		const body = payloadText(op);
 		if (op.action === "add") return body ? null : `Operation ${position} (add): facts is required. No operations were applied.`;
 		if (op.action !== "remove" && op.action !== "replace") return `Operation ${position}: unknown action "${String(op.action)}" (expected add/remove/replace). No operations were applied.`;
 		if (!(op.old_text ?? "").trim()) return `Operation ${position} (${op.action}): old_text is required. No operations were applied.`;
@@ -221,8 +241,9 @@ async function apply(ctx, rawConfig = {}) {
 		isConcurrencySafe: () => false,
 		async execute(args, exec) {
 			const conflict = (a) => {
-				if (a.facts === void 0 || a.content === void 0) return false;
-				return a.facts !== a.content;
+				const facts = payloadText({ facts: a.facts });
+				const content = payloadText({ content: a.content });
+				return facts !== "" && content !== "" && facts !== content;
 			};
 			if (Array.isArray(args.operations)) for (const op of args.operations) {
 				const raw = op;
@@ -269,7 +290,7 @@ async function apply(ctx, rawConfig = {}) {
 			} : {
 				target,
 				action: args.action ?? "add",
-				facts: args.facts ?? args.content,
+				facts: payloadText(args),
 				old_text: args.old_text
 			};
 			if (normalized.operations) for (const [index, op] of normalized.operations.entries()) {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-tool-memory",
   "description": "Model-facing memory tool and prompt context (community build)",
-  "version": "0.17.1",
+  "version": "0.18.0",
   "publishConfig": {
     "access": "public"
   },
@@ -26,22 +26,22 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.17.1",
-    "@lmzhen/dsh-evolution-core": "^0.17.1"
+    "@deepseek-ai/schemastery": "^3.18.4",
+    "@lmzhen/dsh-evolution-approval": "^0.18.0",
+    "@lmzhen/dsh-evolution-core": "^0.18.0"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-tools": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-system-prompt": "^0.2.0-rc.1",
-    "@lmzhen/dsh-memory": "^0.17.1"
+    "@lmzhen/dsh-memory": "^0.18.0"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-tools": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-system-prompt": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-agent-loop-testkit": "^0.2.0-rc.1",
-    "@lmzhen/dsh-evolution-approval": "^0.17.1",
-    "@lmzhen/dsh-memory": "^0.17.1",
-    "@lmzhen/dsh-memory-files": "^0.17.1"
+    "@lmzhen/dsh-evolution-approval": "^0.18.0",
+    "@lmzhen/dsh-memory": "^0.18.0",
+    "@lmzhen/dsh-memory-files": "^0.18.0"
   }
 }