@lmzhen/dsh-evolution-review 0.3.56 → 0.3.58

package/lib/index.js

@@ -50,17 +50,6 @@ const REVIEW_OUTPUT_SCHEMA = {
 	}
 };
 let statelessReviewStateWarned = false;
-/**
-* G3.1 (0.3.23): clamp the numeric review config at assembly so a 0/negative/
-* NaN/±Infinity value falls back to the package default instead of folding as a
-* "disabled" special value (a 0 interval would fire a review every turn; NaN
-* folds as NaN into the cadence/timeout). The schema `.min(1)` guards the
-* loader path; this clamp also covers NaN/±Infinity (which schemastery lets a
-* bare number schema through) and direct construction. `reviewMaxDepth` clamps
-* to at least 1 because 0 is the historical 0.3.1 maximum-depth defect (a 0
-* rejects the spawn outright). Warn once when a user-supplied value had to be
-* corrected.
-*/
 function clampReviewConfig(rawConfig, ctx) {
 	const clamped = [];
 	const field = (name, value, fallback, min) => {
@@ -278,7 +267,7 @@ function apply(ctx, rawConfig) {
 				maxDepth: config.reviewMaxDepth,
 				agentOptions,
 				persona: reviewPrompt(kind, "plan"),
-				toolFilter: { allow: [...config.reviewToolAllow] },
+				toolFilter: { allow: [...config.reviewToolAllow ?? []] },
 				outputSchema: REVIEW_OUTPUT_SCHEMA
 			});
 			try {

package/lib/types/index.d.ts

@@ -28,8 +28,12 @@ export interface Config {
     reviewMaxDepth?: number;
     /** LLM provider for review subagents. Omit to inherit the deployment default route. */
     reviewProvider?: string;
-    /** Which channel runs the end-of-conversation summary: the cadence latch
-     * ('cadence'), the long-session completion gate ('completion'), or both. */
+    /** E3/P1-8 (v11) — TRUE semantics: this flag gates ONLY the completion
+     * channel. The cadence latch stays active regardless of the value (a
+     * completion-gated deployment still fires at the policy cadence), so
+     * 'completion' does NOT mean "cadence off", and 'both' is effectively
+     * "completion on top of the always-on cadence". The mutually-exclusive
+     * channel reading in earlier docs was wrong. */
     skillReviewTrigger?: 'cadence' | 'completion' | 'both';
     /** Cumulative session tool calls before a session counts as proven-long for the completion channel. */
     skillReviewCompletionMinToolCalls?: number;
@@ -81,7 +85,19 @@ export declare const REVIEW_OUTPUT_SCHEMA: {
  * rejects the spawn outright). Warn once when a user-supplied value had to be
  * corrected.
  */
-export declare function clampReviewConfig(rawConfig: Config, ctx: Context): Required<Config>;
+/** F5 (P2-16, v11): the clamp sets exactly these seven numeric fields — the
+ * old `as Required<Config>` lied about `reviewToolAllow` etc. being populated
+ * (`[...undefined]` TypeErrors under a direct `apply(ctx, {})`). */
+type ClampedReviewConfig = Config & {
+    memoryInterval: number;
+    skillInterval: number;
+    reviewTimeoutMs: number;
+    reviewContextMessages: number;
+    reviewMessageChars: number;
+    reviewMaxDepth: number;
+    skillReviewCompletionMinToolCalls: number;
+};
+export declare function clampReviewConfig(rawConfig: Config, ctx: Context): ClampedReviewConfig;
 export declare function apply(ctx: Context, rawConfig: Config): void;
 /** Completion-channel decision: task finished normally AND the session is proven long. */
 export declare function shouldCompletionReview(reason: {
@@ -121,4 +137,5 @@ export declare function filterUnreadSkillOps(ops: Array<{
  * buildReviewRequest and is unchanged).
  */
 export declare function renderToolResultLine(data: unknown): string;
+export {};
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-review",
   "description": "Background review orchestration (community build)",
-  "version": "0.3.56",
+  "version": "0.3.58",
   "publishConfig": {
     "access": "public"
   },
@@ -31,9 +31,9 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.3.56",
-    "@lmzhen/dsh-evolution-core": "^0.3.56",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.3.56"
+    "@lmzhen/dsh-evolution-approval": "^0.3.58",
+    "@lmzhen/dsh-evolution-core": "^0.3.58",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.3.58"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
@@ -42,8 +42,8 @@
     "@deepseek-ai/dsh-llm": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-state": "^0.3.56",
-    "@lmzhen/dsh-evolution-policy": "^0.3.56"
+    "@lmzhen/dsh-evolution-state": "^0.3.58",
+    "@lmzhen/dsh-evolution-policy": "^0.3.58"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-agent": "^0.1.1-rc.2",
@@ -54,9 +54,9 @@
     "@deepseek-ai/dsh-session-persistence": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-session-persistence-jsonl": "^0.1.1-rc.2",
     "@deepseek-ai/dsh-tools": "^0.1.1-rc.2",
-    "@lmzhen/dsh-evolution-approval": "^0.3.56",
-    "@lmzhen/dsh-evolution-core": "^0.3.56",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.3.56",
-    "@lmzhen/dsh-evolution-state": "^0.3.56"
+    "@lmzhen/dsh-evolution-approval": "^0.3.58",
+    "@lmzhen/dsh-evolution-core": "^0.3.58",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.3.58",
+    "@lmzhen/dsh-evolution-state": "^0.3.58"
   }
 }