@lmzhen/dsh-evolution-review 0.26.15 → 0.26.26

package/lib/index.js

@@ -1,8 +1,8 @@
+import { COMPLETION_SKILL_REVIEW_PROMPT, DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_MEMORY_REVIEW_MODEL, DEFAULT_REVIEW_CONTEXT_MESSAGES, DEFAULT_REVIEW_MEMORY_INTERVAL, DEFAULT_REVIEW_MESSAGE_CHARS, DEFAULT_REVIEW_SKILL_INTERVAL, DEFAULT_REVIEW_TIMEOUT_MS, DEFAULT_SKILL_CONTENT_CHARS, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_REVIEW_COMPLETION_MIN_TOOL_CALLS, DEFAULT_SKILL_REVIEW_MODEL, DEFAULT_SKILL_REVIEW_TRIGGER, DEFAULT_SUBSTANTIVE_MIN_AGENT_CHARS, DEFAULT_SUBSTANTIVE_MIN_TOOL_CALLS, DEFAULT_SUBSTANTIVE_MIN_USER_CHARS, DEFAULT_USER_CHAR_LIMIT, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, advanceReview, assertSkillsRootAliasRetired, authorityFrom, clampOnce, clearReviewChannel, contentHash, evolutionIoAdapter, filterUnreadSkillOps, filterUnreadSkillOps as filterUnreadSkillOps$1, foldUntrustedLines, isPresent, isUnknown, liveBool, markReviewChannel, newSkillLibrary, paramRowId, payloadText, pickWithPolicy, policyStageLimits, readDispatchSignal, redactSecrets, resolveOrigins, resolveRootConfig, reviewPrompt, sessionAudited, sessionEvidenceIndex, sessionReadNames, sessionTurnSignals, sweepReviewChannelSessions, unwrapLive, userSetKeys, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 import { createHash, randomUUID } from "node:crypto";
 import z from "@deepseek-ai/schemastery";
 import { createUserMessage } from "@deepseek-ai/dsh-llm";
 import { SessionId } from "@deepseek-ai/dsh-session";
-import { COMPLETION_SKILL_REVIEW_PROMPT, DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, DEFAULT_MEMORY_REVIEW_MODEL, DEFAULT_REVIEW_CONTEXT_MESSAGES, DEFAULT_REVIEW_MEMORY_INTERVAL, DEFAULT_REVIEW_MESSAGE_CHARS, DEFAULT_REVIEW_SKILL_INTERVAL, DEFAULT_REVIEW_TIMEOUT_MS, DEFAULT_SKILL_CONTENT_CHARS, DEFAULT_SKILL_LIMITS, DEFAULT_SKILL_REVIEW_COMPLETION_MIN_TOOL_CALLS, DEFAULT_SKILL_REVIEW_MODEL, DEFAULT_SKILL_REVIEW_TRIGGER, DEFAULT_SUBSTANTIVE_MIN_AGENT_CHARS, DEFAULT_SUBSTANTIVE_MIN_TOOL_CALLS, DEFAULT_SUBSTANTIVE_MIN_USER_CHARS, DEFAULT_USER_CHAR_LIMIT, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, advanceReview, assertSkillsRootAliasRetired, clampOnce, clearReviewChannel, contentHash, evolutionIoAdapter, filterUnreadSkillOps, filterUnreadSkillOps as filterUnreadSkillOps$1, foldUntrustedLines, liveBool, markReviewChannel, newSkillLibrary, paramRowId, payloadText, pickWithPolicy, policyStageLimits, readDispatchSignal, redactSecrets, resolveOrigins, resolveRootConfig, reviewPrompt, sessionAudited, sessionEvidenceIndex, sessionReadNames, sessionTurnSignals, sweepReviewChannelSessions, unwrapLive, userSetKeys, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 import { validateEvolutionPlan } from "@lmzhen/dsh-evolution-plan-validator";
 /**
 * Whether a message is a notice this plugin delivered, judged only by durable source fields.
@@ -731,6 +731,7 @@ function apply(ctx, rawConfig = {}) {
 		const io = ctx.get("evolutionIo");
 		if (!io) return hashes;
 		const library = newSkillLibrary({
+			authority: authorityFrom({ origin: "foreground" }),
 			config: rootConfig,
 			io: evolutionIoAdapter(() => io.provider())
 		});
@@ -931,6 +932,7 @@ function apply(ctx, rawConfig = {}) {
 			const io = ctx.get("evolutionIo");
 			if (!io) return null;
 			return newSkillLibrary({
+				authority: authorityFrom({ origin: "foreground" }),
 				config: rootConfig,
 				io: evolutionIoAdapter(() => io.provider())
 			});
@@ -941,7 +943,7 @@ function apply(ctx, rawConfig = {}) {
 		let ok = true;
 		try {
 			for (const op of plan.memoryOps ?? []) {
-				if (signal?.aborted === true) {
+				if (ctx.fiber.uid === null || signal?.aborted === true) {
 					const reason = "the review was unloaded mid-plan";
 					ctx.logger.warn(`dsh-evolution-review: executePlan stopped at an op boundary after ${actions.length} op(s): ${reason}`);
 					return {
@@ -972,7 +974,7 @@ function apply(ctx, rawConfig = {}) {
 				}
 			}
 			for (const op of plan.skillOps ?? []) {
-				if (signal?.aborted === true) {
+				if (ctx.fiber.uid === null || signal?.aborted === true) {
 					const reason = "the review was unloaded mid-plan";
 					ctx.logger.warn(`dsh-evolution-review: executePlan stopped at an op boundary after ${actions.length} op(s): ${reason}`);
 					return {
@@ -992,17 +994,35 @@ function apply(ctx, rawConfig = {}) {
 					evidence: op.evidence
 				};
 				let stagedBytes;
-				if ((args.action === "update" || args.action === "edit") && hashLibrary && typeof args.name === "string" && args.name !== "") stagedBytes = await hashLibrary.read(args.name).catch(() => void 0);
+				if ((args.action === "update" || args.action === "edit") && hashLibrary && typeof args.name === "string" && args.name !== "") {
+					const probe = await hashLibrary.readSkillProbe(args.name);
+					if (isUnknown(probe)) {
+						ok = false;
+						failedOps.push(`skill ${args.action} ${args.name}: target is unreadable (${probe.reason}); refusing to stage an unverifiable anchor`);
+						continue;
+					}
+					stagedBytes = isPresent(probe) ? probe.value : null;
+				}
 				if ((args.action === "write_file" || args.action === "remove_file") && hashLibrary && typeof args.name === "string" && args.name !== "" && typeof args.file_path === "string" && args.file_path !== "") {
-					const stageFile = await hashLibrary.readSupportFile(args.name, args.file_path).catch(() => void 0);
-					if (stageFile !== void 0) args.staged_from_sha256 = stageFile === null ? "absent" : contentHash(stageFile);
+					const supportProbe = await hashLibrary.readSupportFileProbe(args.name, args.file_path);
+					if (isUnknown(supportProbe)) {
+						ok = false;
+						failedOps.push(`file ${args.action} ${args.name}/${args.file_path}: target is unreadable (${supportProbe.reason}); refusing to stage an unverifiable anchor`);
+						continue;
+					}
+					const stageFile = isPresent(supportProbe) ? supportProbe.value : null;
+					args.staged_from_sha256 = stageFile === null ? "absent" : contentHash(stageFile);
 				}
 				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 live = anchored !== null ? anchored : await (async () => {
+						if (hashLibrary === null) return null;
+						const probed = await hashLibrary.readSkillProbe(opName);
+						return isPresent(probed) ? probed.value : null;
+					})();
 					const preRun = preRunHashes.get(opName);
 					if (live === null || preRun !== void 0 && contentHash(live) !== preRun) {
 						ok = false;
@@ -1022,7 +1042,8 @@ function apply(ctx, rawConfig = {}) {
 					actions.push(`Skill ${op.name} ${op.action ?? "patch"}`);
 					onLanded?.(`Skill ${op.name} ${op.action ?? "patch"}`);
 					if (hashChecked) {
-						const landed = await hashLibrary?.read(opName).catch(() => null) ?? null;
+						const landedProbe = hashLibrary === null ? void 0 : await hashLibrary.readSkillProbe(opName);
+						const landed = landedProbe !== void 0 && isPresent(landedProbe) ? landedProbe.value : null;
 						if (landed !== null) preRunHashes.set(opName, contentHash(landed));
 					}
 				} else {
@@ -1101,6 +1122,7 @@ function apply(ctx, rawConfig = {}) {
 			};
 			const policySnapshot = policySnapshotOf(ctx.get("evolutionPolicy"));
 			const library = newSkillLibrary({
+				authority: authorityFrom({ origin: resolveOrigins(void 0, true).library }),
 				config: rootConfig,
 				io: evolutionIoAdapter(() => io.provider()),
 				limits: {

package/package.json

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