@lmzhen/dsh-evolution-plan-validator 0.3.62 → 0.3.63

package/lib/index.js

@@ -144,6 +144,13 @@ function validateEvolutionPlan(plan, context) {
 	};
 }
 function validateMemoryOp(op, context, index) {
+	const badField = malformedStringField(op, [
+		"facts",
+		"content",
+		"old_text",
+		"target"
+	]);
+	if (badField) return `memory op ${index}: field ${badField} must be a string`;
 	if (!hasValidEvidence(op.evidence, context.sessionSeq)) return `memory op ${index}: evidence is required and must reference a valid session seq`;
 	for (const key of FORBIDDEN_KEYS) if (key in op) return `memory op ${index}: forbidden field ${key}`;
 	if (op.target !== "memory" && op.target !== "user") return `memory op ${index}: target must be memory or user`;
@@ -156,7 +163,28 @@ function validateMemoryOp(op, context, index) {
 	if (text.length > budget) return `memory op ${index}: content exceeds ${op.target === "user" ? "user" : "memory"} budget`;
 	return null;
 }
+/** P2-1 (v15): the string-typed fields the executors dereference with
+* `.trim()`/`.length`. `target` is included for memory ops because the
+* engine compares it against literals. */
+function malformedStringField(op, fields) {
+	const record = op;
+	for (const field of fields) {
+		const value = record[field];
+		if (value !== void 0 && value !== null && typeof value !== "string") return field;
+	}
+	return null;
+}
 function validateSkillOp(op, context, index) {
+	const badField = malformedStringField(op, [
+		"name",
+		"content",
+		"old_string",
+		"new_string",
+		"file_path",
+		"file_content",
+		"absorbed_into"
+	]);
+	if (badField) return `skill op ${index}: field ${badField} must be a string`;
 	for (const key of FORBIDDEN_KEYS) if (key in op) return `skill op ${index}: forbidden field ${key}`;
 	const name = (op.name ?? "").trim();
 	if (!name) return `skill op ${index}: name is required`;
@@ -172,7 +200,7 @@ function validateSkillOp(op, context, index) {
 		op.content ?? "",
 		op.new_string ?? ""
 	].reduce((max, value) => Math.max(max, value.length), 0);
-	if (action === "write_file" && !(op.file_content ?? op.content ?? "").trim()) return `skill op ${index}: write_file requires file_content`;
+	if (action === "write_file" && !(op.file_content ?? "").trim()) return `skill op ${index}: write_file requires file_content`;
 	if (writeBytes > (context.maxSkillContentChars ?? DEFAULT_SKILL_CONTENT_CHARS)) return `skill op ${index}: content exceeds skill budget`;
 	if (action === "restructure") {
 		if (!Array.isArray(op.restructure) || op.restructure.length === 0) return `skill op ${index}: restructure requires a non-empty restructure list`;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-plan-validator",
   "description": "Deterministic validator for model-produced evolution plans (community build)",
-  "version": "0.3.62",
+  "version": "0.3.63",
   "publishConfig": {
     "access": "public"
   },
@@ -30,7 +30,7 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.3.62"
+    "@lmzhen/dsh-evolution-core": "^0.3.63"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",