@lmzhen/dsh-evolution-review 0.3.79 → 0.3.81

package/lib/index.js

@@ -51,6 +51,16 @@ const REVIEW_OUTPUT_SCHEMA = {
 		summary: { type: "string" }
 	}
 };
+/**
+* Is this pending row our own notice of the SAME kind? `summary` is the kind
+* discriminator, so distinct notices (cadence review vs completion review vs
+* self-improvement) still queue side by side: a repeat of one kind replaces its
+* pending copy instead of adding a second.
+*/
+function isSameKindPending(message, summary) {
+	const source = message.source;
+	return source?.kind === "plugin" && source.plugin === "dsh-evolution-review" && source.form === "notice" && source.summary === summary;
+}
 function clampReviewConfig(rawConfig, ctx) {
 	const clamped = [];
 	const field = (name, value, fallback, min, max) => {
@@ -342,6 +352,16 @@ function apply(ctx, rawConfig = {}) {
 				summary
 			}
 		});
+		const inbox = agent.inbox;
+		if (inbox !== void 0 && typeof inbox.replace === "function") try {
+			const superseded = [...inbox.nextTurn ?? [], ...inbox.nextStep ?? []].find((row) => isSameKindPending(row, summary));
+			if (superseded !== void 0 && inbox.replace(superseded.id, message)) {
+				if (reviewPrompt) markReviewChannel(agent.session.id);
+				return true;
+			}
+		} catch (error) {
+			ctx.logger.warn(`dsh-evolution-review: inbox coalescing failed (${error instanceof Error ? error.message : String(error)}) — delivering a fresh message instead`);
+		}
 		const wake = agent;
 		try {
 			if (config.reviewWakeInject && typeof wake.followup === "function") {
@@ -612,6 +632,7 @@ function apply(ctx, rawConfig = {}) {
 				const opName = typeof args.name === "string" ? args.name : "";
 				const hashChecked = (args.action === "update" || args.action === "edit" || args.action === "patch") && opName !== "" && preRunHashes?.has(opName) === true;
 				const anchored = hashChecked && typeof stagedBytes === "string" ? stagedBytes : null;
+				let gateVerifiedBytes = null;
 				if (hashChecked) {
 					const live = anchored !== null ? anchored : hashLibrary ? await hashLibrary.read(opName).catch(() => null) : null;
 					const preRun = preRunHashes.get(opName);
@@ -620,8 +641,9 @@ function apply(ctx, rawConfig = {}) {
 						failedOps.push(`skill ${args.action} ${args.name}: the skill changed while this review ran — ${args.action === "patch" ? "patch" : "update"} refused as stale; produce a fresh plan`);
 						continue;
 					}
+					if (anchored === null) gateVerifiedBytes = live;
 				}
-				const anchorBytes = anchored ?? stagedBytes;
+				const anchorBytes = anchored ?? gateVerifiedBytes ?? stagedBytes;
 				if ((args.action === "update" || args.action === "edit") && anchorBytes !== void 0) args.staged_from_sha256 = anchorBytes === null ? "absent" : contentHash(anchorBytes);
 				const runnerArgs = {
 					operation: args,
@@ -739,7 +761,7 @@ function apply(ctx, rawConfig = {}) {
 				return updated;
 			}
 			if (op.action === "patch") {
-				const patched = await library.patch(name, op.old_string ?? "", op.new_string ?? "", op.file_path ?? "", false, origin);
+				const patched = await library.patch(name, op.old_string ?? "", op.new_string ?? "", op.file_path ?? "", op.replace_all === true, origin);
 				if (patched.ok) await ctx.get("skillUsage")?.record?.(name, "patch").catch(() => {});
 				return patched;
 			}

package/package.json

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