@lmzhen/dsh-evolution-plan-validator 0.1.0-rc.9 → 0.2.0-rc.1

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, MAX_RESTRUCTURE_MOVES, RESTRUCTURE_TARGET_RE } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic validator for model-produced evolution plans.
@@ -16,7 +17,8 @@ const SKILL_ACTIONS = new Set([
 	"patch",
 	"delete",
 	"write_file",
-	"remove_file"
+	"remove_file",
+	"restructure"
 ]);
 const FORBIDDEN_KEYS = [
 	"policy",
@@ -30,12 +32,12 @@ function hasValidEvidence(evidence, sessionSeq) {
 	return evidence.every((item) => {
 		if (!item || typeof item !== "object") return false;
 		const record = item;
-		const seq = typeof record.event_seq === "number" ? record.event_seq : typeof record.seq === "number" ? record.seq : Number(record.event_seq);
+		const seq = typeof record.event_seq === "number" ? record.event_seq : typeof record.seq === "number" ? record.seq : typeof record.event_seq === "string" && /^\d+$/.test(record.event_seq) ? Number(record.event_seq) : -1;
 		return Number.isInteger(seq) && seq >= 0 && seq <= 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 +104,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 +118,18 @@ 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`;
+	if (action === "restructure") {
+		if (!Array.isArray(op.restructure) || op.restructure.length === 0) return `skill op ${index}: restructure requires a non-empty restructure list`;
+		if (op.restructure.length > MAX_RESTRUCTURE_MOVES) return `skill op ${index}: restructure exceeds ${MAX_RESTRUCTURE_MOVES} moves`;
+		for (const [moveIndex, move] of op.restructure.entries()) {
+			if (!move || typeof move.heading !== "string" || !move.heading.trim()) return `skill op ${index}: restructure[${moveIndex}] requires a non-empty heading`;
+			if (typeof move.to_file !== "string" || !RESTRUCTURE_TARGET_RE.test(move.to_file)) return `skill op ${index}: restructure[${moveIndex}] to_file must be references/<topic>.md`;
+		}
+	}
 	return null;
 }
 //#endregion

package/lib/types/index.d.ts

@@ -15,10 +15,17 @@ 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;
     absorbed_into?: string;
+    /** restructure payload: body sections moved to references/ (008 batch B). */
+    restructure?: Array<{
+        heading?: string;
+        to_file?: string;
+    } | null>;
     evidence?: unknown[];
 }
 export interface EvolutionPlan {

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.9",
+  "version": "0.2.0-rc.1",
   "publishConfig": {
     "access": "public"
   },
@@ -32,13 +32,14 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@deepseek-ai/schemastery": "^3.18.1"
+    "@deepseek-ai/schemastery": "^3.18.1",
+    "@lmzhen/dsh-evolution-core": "^0.2.0-rc.1"
   },
   "peerDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
+    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/cordis": "^4.0.1"
   },
   "devDependencies": {
-    "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6"
+    "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2"
   }
 }