@lmzhen/dsh-tool-memory 0.3.82 → 0.4.0

package/README.md

@@ -1,26 +1,18 @@
-# @deepseek-ai/dsh-tool-memory
+# @lmzhen/dsh-tool-memory
 
-Model-facing memory tool and prompt context
+Model-facing memory tool and prompt context: it registers the `memory` tool, one fixed
+guidance section and the runtime snapshot of the current memory entries. It owns no store —
+the memory files and their provider do.
 
-## Model Experience
+## Model surface
 
-### Memory tool and runtime snapshot
-
-#### What the model sees
-
-The model sees the `memory` tool schema, one fixed guidance section, and a runtime snapshot containing the current memory entries.
-
-#### Token effect
-
-Fixed guidance has a constant token cost. The runtime snapshot scales with stored memory entries and is absent when memory is empty.
-
-#### KV Cache effect
-
-Guidance text is prefix-stable. The runtime snapshot is replaced after successful memory writes and is otherwise unchanged between requests.
-
-## Known Limitations and Deferred Work
+- **Model-visible:** the `memory` tool schema, one fixed guidance section, and a runtime snapshot containing the current memory entries; the snapshot scales with the stored entries and is absent when memory is empty.
+- **Prompt prefix / KV cache:** guidance text is prefix-stable; the runtime snapshot is replaced after successful memory writes and is otherwise unchanged between requests; family rules: `packages/README.md` §"Model-visible prompt prefix and the KV cache".
+- **Mount it?** yes — the `tool-memory` row, carried by the `evolution-all` and one-click `evolution-preset` bundles and by the Evolution agent preset delta.
 
+## Known limitations
 
 - No known durable consumer gaps at this time. Runtime contracts are covered by package and boundary tests.
+- The pre-approval required-field pre-check and the store's own rejection text are hand-copied twins: the check runs on BOTH paths (every `operations[]` element and the normalized single operation) BEFORE the approval gate, so an approval-enabled deployment never stages a write the approved replay must then fail. Keeping the two texts in step is still manual; unifying them is deferred.
 
-**Runtime invariant:** No companion is published. The platform auto-assembles nothing and the family mounts no `<pkg>/invariant` cordis row, so a companion here would never execute (v37 S2.1 / I-3).
+**Runtime invariant:** No companion is published. The platform auto-assembles nothing and the family mounts no `<pkg>/invariant` cordis row, so a companion here would never execute (v37 S2.1 / I-3).

package/lib/index.js

@@ -104,6 +104,14 @@ async function apply(ctx, rawConfig = {}) {
 			limit: result.limit
 		};
 	}
+	const requiredFieldRejection = (op, position) => {
+		const body = (op.facts ?? op.content ?? "").trim();
+		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.`;
+		if (op.action === "replace" && !body) return `Operation ${position} (replace): facts is required.`;
+		return null;
+	};
 	if (!memoryDisabled) ctx.tools.register(defineTool({
 		name: "memory",
 		description: MEMORY_TOOL_DESCRIPTION,
@@ -234,6 +242,26 @@ async function apply(ctx, rawConfig = {}) {
 				facts: args.facts ?? args.content,
 				old_text: args.old_text
 			};
+			if (normalized.operations) for (const [index, op] of normalized.operations.entries()) {
+				const rejection = requiredFieldRejection(op, index + 1);
+				if (rejection) return {
+					ok: false,
+					message: rejection,
+					entries: [],
+					chars: 0,
+					limit: 0
+				};
+			}
+			else {
+				const rejection = requiredFieldRejection(normalized, 1);
+				if (rejection) return {
+					ok: false,
+					message: rejection,
+					entries: [],
+					chars: 0,
+					limit: 0
+				};
+			}
 			const targetLabel = target === "memory" ? "" : `${target} `;
 			const origin = resolveExecOrigins(exec).approval;
 			const sessionPolicy = effectiveSessionPolicy(ctx, exec.agent?.session);

package/package.json

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