@lmzhen/dsh-evolution-replay 0.3.22 → 0.3.24

package/lib/index.js

@@ -1,4 +1,5 @@
 import z from "@deepseek-ai/schemastery";
+import { clampedNumber } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Replay/A-B evaluation for evolution plans.
@@ -19,14 +20,29 @@ const DEFAULT_WEIGHTS = {
 	cost: .001
 };
 const Config = z.object({
-	maxPlans: z.number().default(50),
+	maxPlans: z.number().min(1).default(50),
 	weights: z.object({
-		accepted: z.number().default(10),
-		rejectedPenalty: z.number().default(15),
-		evidence: z.number().default(2),
-		cost: z.number().default(.001)
+		accepted: z.number().min(1).default(10),
+		rejectedPenalty: z.number().min(1).default(15),
+		evidence: z.number().min(1).default(2),
+		cost: z.number().min(0).default(.001)
 	}).default(DEFAULT_WEIGHTS)
 });
+/**
+* Clamp a weights object to its per-field domain (G3.1). `accepted`,
+* `rejectedPenalty` and `evidence` are positive multipliers (min 1 — a zero or
+* negative weight would silently drop a scoring term or reverse its sign);
+* `cost` may be 0 (0 means "no cost penalty"), so its min is 0. NaN, ±Infinity
+* and out-of-range values fall back to the default for that field.
+*/
+function clampReplayWeights(weights, fallback = DEFAULT_WEIGHTS) {
+	return {
+		accepted: clampedNumber(weights?.accepted, fallback.accepted, { min: 1 }),
+		rejectedPenalty: clampedNumber(weights?.rejectedPenalty, fallback.rejectedPenalty, { min: 1 }),
+		evidence: clampedNumber(weights?.evidence, fallback.evidence, { min: 1 }),
+		cost: clampedNumber(weights?.cost, fallback.cost, { min: 0 })
+	};
+}
 function scorePlan(plan, weights = DEFAULT_WEIGHTS) {
 	return plan.acceptedOps * weights.accepted - plan.rejectedOps * weights.rejectedPenalty + plan.evidenceQuotes * weights.evidence - plan.estimatedInputChars * weights.cost;
 }
@@ -61,9 +77,11 @@ var EvolutionReplayDriver = class {
 	plans = [];
 	maxPlans;
 	weights;
-	constructor(config = {}) {
-		this.maxPlans = config.maxPlans ?? 50;
-		this.weights = config.weights ?? DEFAULT_WEIGHTS;
+	constructor(config = {}, warn = () => {}) {
+		const maxPlans = clampedNumber(config.maxPlans ?? 50, 50, { min: 1 });
+		this.maxPlans = maxPlans;
+		if (config.maxPlans !== void 0 && maxPlans !== config.maxPlans) warn(`evolution-replay: maxPlans=${String(config.maxPlans)} is invalid; falling back to the default 50`);
+		this.weights = clampReplayWeights(config.weights);
 	}
 	record(plan) {
 		this.plans.push({
@@ -86,11 +104,13 @@ var EvolutionReplayDriver = class {
 };
 const name = "evolution-replay";
 function apply(ctx, rawConfig = {}) {
-	const driver = new EvolutionReplayDriver(rawConfig);
+	const driver = new EvolutionReplayDriver(rawConfig, (message) => {
+		ctx.logger.warn(message);
+	});
 	ctx.provide("evolutionReplay", driver);
 	ctx.on("evolution/plan-applied", (event) => {
 		driver.record(event);
 	});
 }
 //#endregion
-export { Config, DEFAULT_WEIGHTS, EvolutionReplayDriver, apply, comparePlans, name };
+export { Config, DEFAULT_WEIGHTS, EvolutionReplayDriver, apply, clampReplayWeights, comparePlans, name };

package/lib/types/index.d.ts

@@ -40,6 +40,14 @@ export interface Config {
     weights?: ReplayWeights;
 }
 export declare const Config: z<Config>;
+/**
+ * Clamp a weights object to its per-field domain (G3.1). `accepted`,
+ * `rejectedPenalty` and `evidence` are positive multipliers (min 1 — a zero or
+ * negative weight would silently drop a scoring term or reverse its sign);
+ * `cost` may be 0 (0 means "no cost penalty"), so its min is 0. NaN, ±Infinity
+ * and out-of-range values fall back to the default for that field.
+ */
+export declare function clampReplayWeights(weights: ReplayWeights | undefined, fallback?: ReplayWeights): ReplayWeights;
 export declare function comparePlans(plans: ReplayPlan[], weights?: ReplayWeights): ReplayResult;
 declare module '@deepseek-ai/cordis' {
     interface Context {
@@ -50,7 +58,7 @@ export declare class EvolutionReplayDriver {
     private readonly plans;
     private readonly maxPlans;
     private readonly weights;
-    constructor(config?: Config);
+    constructor(config?: Config, warn?: (message: string) => void);
     record(plan: EvolutionPlanAppliedEvent): void;
     plansSnapshot(): ReplayPlan[];
     compare(weights?: ReplayWeights): ReplayResult;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-replay",
   "description": "Replay/A-B evaluation primitives for evolution plans (community build)",
-  "version": "0.3.22",
+  "version": "0.3.24",
   "publishConfig": {
     "access": "public"
   },
@@ -31,7 +31,7 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.22"
+    "@lmzhen/dsh-evolution-core": "^0.3.24"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
@@ -40,6 +40,6 @@
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.3.22"
+    "@lmzhen/dsh-evolution-core": "^0.3.24"
   }
 }