@lmzhen/dsh-evolution-plan-validator 0.1.0-rc.5 → 0.1.0-rc.50

package/lib/index.js

@@ -1,3 +1,4 @@
+import { DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_SKILL_CONTENT_CHARS, DEFAULT_USER_CHAR_LIMIT } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic validator for model-produced evolution plans.
@@ -35,7 +36,7 @@ function hasValidEvidence(evidence, sessionSeq) {
 	});
 }
 function validateEvolutionPlan(plan, context) {
-	const maxOps = context.maxOpsPerPlan ?? 32;
+	const maxOps = context.maxOpsPerPlan ?? DEFAULT_MAX_OPS_PER_PLAN;
 	const rejected = [];
 	const memoryOps = [];
 	const skillOps = [];
@@ -102,7 +103,7 @@ function validateMemoryOp(op, context, index) {
 	const text = (op.facts ?? op.content ?? "").trim();
 	if (action !== "remove" && text.length === 0) return `memory op ${index}: ${action} requires facts/content`;
 	if (action !== "add" && !(op.old_text ?? "").trim()) return `memory op ${index}: ${action} requires old_text`;
-	const budget = op.target === "user" ? context.maxUserChars ?? 1375 : context.maxMemoryChars ?? 2200;
+	const budget = op.target === "user" ? context.maxUserChars ?? DEFAULT_USER_CHAR_LIMIT : context.maxMemoryChars ?? DEFAULT_MEMORY_CHAR_LIMIT;
 	if (text.length > budget) return `memory op ${index}: content exceeds ${op.target === "user" ? "user" : "memory"} budget`;
 	return null;
 }
@@ -116,7 +117,10 @@ function validateSkillOp(op, context, index) {
 	if (!SKILL_ACTIONS.has(action)) return `skill op ${index}: unknown action ${action}`;
 	if ((action === "create" || action === "edit" || action === "update") && !(op.content ?? "").trim()) return `skill op ${index}: ${action} requires content`;
 	if (action === "patch" && !(op.old_string ?? "")) return `skill op ${index}: patch requires old_string`;
-	if ((op.content ?? "").length > (context.maxSkillContentChars ?? 1e5)) return `skill op ${index}: content exceeds skill budget`;
+	if (action === "delete" && !(op.absorbed_into ?? "").trim()) return `skill op ${index}: delete requires absorbed_into`;
+	const writeContent = op.file_content ?? op.content ?? "";
+	if (action === "write_file" && !writeContent.trim()) return `skill op ${index}: write_file requires file_content`;
+	if (writeContent.length > (context.maxSkillContentChars ?? DEFAULT_SKILL_CONTENT_CHARS)) return `skill op ${index}: content exceeds skill budget`;
 	return null;
 }
 //#endregion

package/lib/types/index.d.ts

@@ -15,6 +15,8 @@ export interface SkillOp {
     action?: string;
     name?: string;
     content?: string;
+    /** write_file payload (the tool reads `file_content`, not `content`). */
+    file_content?: string;
     old_string?: string;
     new_string?: string;
     file_path?: string;

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.1.0-rc.5",
+  "version": "0.1.0-rc.50",
   "publishConfig": {
     "access": "public"
   },
@@ -32,7 +32,8 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@deepseek-ai/schemastery": "^3.18.1"
+    "@deepseek-ai/schemastery": "^3.18.1",
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.50"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",