@lmzhen/dsh-evolution-plan-validator 0.3.79 → 0.3.80

package/lib/index.js

@@ -190,6 +190,7 @@ function validateSkillOp(op, context, index) {
 		"absorbed_into"
 	]);
 	if (badField) return `skill op ${index}: field ${badField} must be a string`;
+	if (op.replace_all !== void 0 && typeof op.replace_all !== "boolean") return `skill op ${index}: field replace_all must be a boolean`;
 	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`;

package/lib/types/index.d.ts

@@ -20,6 +20,12 @@ export interface SkillOp {
     old_string?: string;
     new_string?: string;
     file_path?: string;
+    /** S1-C1 (0.3.80): patch replaces EVERY occurrence when true — the same
+     * contract `skill_manage` documents to the model. Previously the validator
+     * neither knew nor rejected this field while the review executor hardcoded
+     * first-occurrence-only, so a plan carrying `replace_all` silently shrank
+     * to a single replacement and still reported success. */
+    replace_all?: boolean;
     absorbed_into?: string;
     /** restructure payload: body sections moved to references/ (008 batch B). */
     restructure?: Array<{

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.79",
+  "version": "0.3.80",
   "publishConfig": {
     "access": "public"
   },
@@ -26,6 +26,6 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-core": "^0.3.79"
+    "@lmzhen/dsh-evolution-core": "^0.3.80"
   }
 }