@lmzhen/dsh-evolution-review 0.3.66 → 0.3.68

package/README.md

@@ -25,7 +25,7 @@ Independent of request-prefix construction. This package does not alter the asse
 - Review subagents run as `spawn` children on the deployment default preset rather than inheriting the parent agent's composition (`fork`): a fork child is always promoted by the Anchored Standard bootstrap and its narrowed resident catalog would drop the plain `skill` tool from the review allow-list.
 - The review request text is redacted for credential-shaped patterns before it reaches the subagent, but redaction is pattern-based and best-effort, not a security boundary.
 - Read-before-write tracks only reads through the `skill` tool. `skill_manage` has no per-skill read action (`list`/`review` are whole-library, not targeted at one name), so a skill that was only listed via `skill_manage` is not marked as read — a background review may still reject a patch to it until it is actually loaded.
-- The completion-channel counters (`cumulativeToolCalls` / `completionInjected`) are in-memory only. A process restart resets them, which is accepted behavior: the completion review is a one-per-session post-task adaptation and a restart is treated as a fresh conversation boundary. The cadence state (`turnsSinceMemory` / `turnsSinceSkill`) is persisted via `ReviewState` and survives restart.
+- The completion-channel counters (`cumulativeToolCalls` / `completionInjected`) are in-memory only. A process restart resets them, which is accepted behavior: the completion review is a one-per-session post-task adaptation and a restart is treated as a fresh conversation boundary. The cadence state (`turnsSinceMemory` / `turnsSinceSkill`) is persisted via `ReviewState` and survives restart — bounded by `REVIEW_STATE_SESSION_CAP` (500, seam constant): the least-recently-active sessions are pruned on save, so a very old session restarting resumes from a fresh cadence baseline rather than an unbounded store.
 - `evolution/review-scheduled` and `evolution/review-error` are emitted for platform/user wiring only — this family has no in-repo production `ctx.on` consumer for them. They are declared externally owned (the platform side wires consumption), which matches the `EXEMPT_ORPHANS` set in `scripts/verify-event-pairing.mjs`.
 - When the `evolution-state` service is not mounted, the memory/skill cadence state is not persisted and every turn restarts from a clean `{ turnsSinceMemory: 0, turnsSinceSkill: 0 }` baseline — the review schedule is stateless and re-decided each turn rather than accumulating across the conversation. The loss is surfaced once per process as a logger warning at the first turn/end.
 - Read-before-write can see the review subagent's own `skill` reads only when the subagent backend exposes `localAgent` (the in-process driver does; out-of-process backends such as ACP and the CLI providers set `localAgent: undefined`). With a remote backend the subagent's reads are invisible, so a plan item patching a skill the subagent itself loaded is dropped as "unread" — the review then falls back to the parent session's reads only. Documented rather than worked around: recovering the child read set needs a `SubagentLike` contract change (v14 P2-6).

package/lib/index.js

@@ -1,7 +1,7 @@
 import { createHash, randomUUID } from "node:crypto";
 import z from "@deepseek-ai/schemastery";
 import { createUserMessage } from "@deepseek-ai/dsh-llm";
-import { COMPLETION_SKILL_REVIEW_PROMPT, DEFAULT_MAX_OPS_PER_PLAN, DEFAULT_MEMORY_CHAR_LIMIT, 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_REVIEW_COMPLETION_MIN_TOOL_CALLS, DEFAULT_SKILL_REVIEW_TRIGGER, DEFAULT_USER_CHAR_LIMIT, PROMPT_BUNDLE, SkillLibrary, advanceReview, clampedNumber, contentHash, evolutionIoAdapter, foldTurn, redactSecrets, resolveOrigins, resolveRootConfig, resolveSkillsRoot, reviewPrompt, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
+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_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, SkillLibrary, advanceReview, assertSkillsRootAliasRetired, clampedNumber, contentHash, evolutionIoAdapter, foldTurn, redactSecrets, resolveOrigins, resolveRootConfig, resolveSkillsRoot, reviewPrompt, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 import { validateEvolutionPlan } from "@lmzhen/dsh-evolution-plan-validator";
 //#region lib/types/index.js
 /**
@@ -10,13 +10,6 @@ import { validateEvolutionPlan } from "@lmzhen/dsh-evolution-plan-validator";
 */
 const name = "evolution-review";
 const inject = ["agents"];
-/** Node's 32-bit timer-delay ceiling (`AbortSignal.timeout`/`setTimeout`):
-* a larger value throws RangeError. B-2 (v18): without a max, a misconfigured
-* `reviewTimeoutMs` made `AbortSignal.timeout` throw inside the subagent
-* start call; the outer catch logged it and silently degraded the review to
-* the inject path. The schema and the assembly clamp both reject it (same
-* bound as commands/maintenance). */
-const MAX_TIMER_DELAY_MS = 2147483647;
 const Config = z.object({
 	reviewEnabled: z.boolean().default(true),
 	reviewMode: z.union([z.const("subagent"), z.const("inject")]).default("subagent"),
@@ -81,8 +74,8 @@ function clampReviewConfig(rawConfig, ctx) {
 function apply(ctx, rawConfig = {}) {
 	if (!verifyPromptBundle(PROMPT_BUNDLE)) throw new Error("dsh-evolution prompt bundle integrity check failed; refusing to schedule review work");
 	const config = clampReviewConfig(rawConfig, ctx);
+	assertSkillsRootAliasRetired(rawConfig);
 	const rootConfig = resolveRootConfig(rawConfig);
-	if (rootConfig.usedDeprecatedAlias) ctx.logger.warn("evolution-review: config \"skillsRoot\" is deprecated (E-7); use \"root\" — the alias is honoured until 0.3.65");
 	const turnStarts = /* @__PURE__ */ new Map();
 	let statelessReviewStateWarned = false;
 	const cumulativeToolCalls = /* @__PURE__ */ new Map();
@@ -93,6 +86,16 @@ function apply(ctx, rawConfig = {}) {
 	const cadenceResetWarned = /* @__PURE__ */ new Set();
 	let reviewInFlight = false;
 	const policy = () => ctx.get("evolutionPolicy")?.get();
+	const reviewStateLocks = /* @__PURE__ */ new Map();
+	async function withReviewStateLock(id, task) {
+		const next = (reviewStateLocks.get(id) ?? Promise.resolve()).catch(() => {}).then(task);
+		reviewStateLocks.set(id, next);
+		try {
+			return await next;
+		} finally {
+			if (reviewStateLocks.get(id) === next) reviewStateLocks.delete(id);
+		}
+	}
 	ctx.on("session/event", (session, event) => {
 		if (event.type === "turn/start" && session.header.origin !== "subagent") turnStarts.set(session.id, session.seq - 1);
 		if (event.type !== "turn/end") return;
@@ -132,24 +135,32 @@ function apply(ctx, rawConfig = {}) {
 			statelessReviewStateWarned = true;
 			ctx.logger.warn("dsh-evolution-review: evolution-state service not mounted — memory/skill review cadence is not persisted and resets every turn (see README Known Limitations).");
 		}
-		const state = await stateService?.loadReviewState(session.id) ?? {
+		const snapshot = policy();
+		const skipFire = skipNextCadenceFire.get(session.id) ?? false;
+		if (skipFire) skipNextCadenceFire.delete(session.id);
+		let state = {
 			turnsSinceMemory: 0,
 			turnsSinceSkill: 0,
 			lastTurn: -1
 		};
-		const snapshot = policy();
-		const skipFire = skipNextCadenceFire.get(session.id) ?? false;
-		if (skipFire) skipNextCadenceFire.delete(session.id);
-		const rawKind = advanceReview(state, event.data.turn, signal, {
-			memoryInterval: snapshot?.reviewMemoryInterval ?? config.memoryInterval,
-			skillInterval: snapshot?.reviewSkillInterval ?? config.skillInterval,
-			substantiveMinToolCalls: snapshot?.substantiveMinToolCalls ?? 3,
-			substantiveMinUserChars: snapshot?.substantiveMinUserChars ?? 200,
-			substantiveMinAgentChars: snapshot?.substantiveMinAgentChars ?? 500,
-			resetOnFire: false
+		const advanced = { kind: null };
+		await withReviewStateLock(session.id, async () => {
+			state = await stateService?.loadReviewState(session.id) ?? {
+				turnsSinceMemory: 0,
+				turnsSinceSkill: 0,
+				lastTurn: -1
+			};
+			advanced.kind = advanceReview(state, event.data.turn, signal, {
+				memoryInterval: snapshot?.reviewMemoryInterval ?? config.memoryInterval,
+				skillInterval: snapshot?.reviewSkillInterval ?? config.skillInterval,
+				substantiveMinToolCalls: snapshot?.substantiveMinToolCalls ?? DEFAULT_SUBSTANTIVE_MIN_TOOL_CALLS,
+				substantiveMinUserChars: snapshot?.substantiveMinUserChars ?? DEFAULT_SUBSTANTIVE_MIN_USER_CHARS,
+				substantiveMinAgentChars: snapshot?.substantiveMinAgentChars ?? DEFAULT_SUBSTANTIVE_MIN_AGENT_CHARS,
+				resetOnFire: false
+			});
+			await stateService?.saveReviewState(session.id, state);
 		});
-		const kind = skipFire ? null : rawKind;
-		await stateService?.saveReviewState(session.id, state);
+		const kind = skipFire ? null : advanced.kind;
 		const cumulative = (cumulativeToolCalls.get(session.id) ?? 0) + signal.toolCalls;
 		cumulativeToolCalls.set(session.id, cumulative);
 		if (event.data.reason.kind === "completed") {
@@ -158,6 +169,18 @@ function apply(ctx, rawConfig = {}) {
 				pendingCadenceReviews.delete(session.id);
 				if ((policy()?.reviewMode ?? config.reviewMode) === "inject") try {
 					deliverMessage(agent, reviewPrompt(pendingKind), "auto-review");
+					try {
+						ctx.emit("evolution/review-scheduled", {
+							sessionId: session.id,
+							kind: pendingKind,
+							toolCalls: signal.toolCalls,
+							userChars: signal.userChars,
+							assistantChars: signal.assistantChars,
+							channel: "inject"
+						});
+					} catch (emitError) {
+						ctx.logger.warn(`dsh-evolution-review: review-scheduled emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
+					}
 				} catch (injectError) {
 					ctx.logger.warn(`dsh-evolution-review: deferred review inject failed: ${injectError instanceof Error ? injectError.message : String(injectError)}`);
 				}
@@ -169,21 +192,40 @@ function apply(ctx, rawConfig = {}) {
 							kind: pendingKind,
 							toolCalls: signal.toolCalls,
 							userChars: signal.userChars,
-							assistantChars: signal.assistantChars
+							assistantChars: signal.assistantChars,
+							channel: "subagent"
 						});
 					} catch (emitError) {
 						ctx.logger.warn(`dsh-evolution-review: review-scheduled emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
 					}
-					else if (reviewOutcome !== "deferred") try {
+					else if (reviewOutcome === "dropped") {
+						pendingCadenceReviews.set(session.id, pendingKind);
+						return;
+					} else if (reviewOutcome !== "deferred") try {
 						deliverMessage(agent, reviewPrompt(pendingKind), "auto-review");
+						try {
+							ctx.emit("evolution/review-scheduled", {
+								sessionId: session.id,
+								kind: pendingKind,
+								toolCalls: signal.toolCalls,
+								userChars: signal.userChars,
+								assistantChars: signal.assistantChars,
+								channel: "inject"
+							});
+						} catch (emitError) {
+							ctx.logger.warn(`dsh-evolution-review: review-scheduled emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
+						}
 					} catch (injectError) {
 						ctx.logger.warn(`dsh-evolution-review: deferred review inject failed: ${injectError instanceof Error ? injectError.message : String(injectError)}`);
 					}
 				}
-				state.turnsSinceMemory = 0;
-				state.turnsSinceSkill = 0;
 				try {
-					await stateService?.saveReviewState(session.id, state);
+					await withReviewStateLock(session.id, async () => {
+						const fresh = await stateService?.loadReviewState(session.id) ?? state;
+						fresh.turnsSinceMemory = 0;
+						fresh.turnsSinceSkill = 0;
+						await stateService?.saveReviewState(session.id, fresh);
+					});
 				} catch (resetError) {
 					if (!cadenceResetWarned.has(session.id)) {
 						cadenceResetWarned.add(session.id);
@@ -207,6 +249,22 @@ function apply(ctx, rawConfig = {}) {
 		if (completionInjected.has(session.id)) return;
 		if (!shouldCompletionReview(event.data.reason, cumulative, config.skillReviewCompletionMinToolCalls)) return;
 		completionInjected.add(session.id);
+		if (reviewInFlight) {
+			if (deferredFallbackReviews.length < DEFERRED_REVIEW_CAP) deferredFallbackReviews.push({
+				agent,
+				sessionId: session.id,
+				kind: "skill",
+				prompt: COMPLETION_SKILL_REVIEW_PROMPT,
+				label: "completion review",
+				channel: "completion",
+				counts: signal
+			});
+			else {
+				completionInjected.delete(session.id);
+				ctx.logger.warn(`dsh-evolution-review: deferred-review queue at cap (${DEFERRED_REVIEW_CAP}) — dropped one deferred completion review prompt; the completion gate re-arms for the next completed turn`);
+			}
+			return;
+		}
 		try {
 			deliverMessage(agent, COMPLETION_SKILL_REVIEW_PROMPT, "completion review");
 		} catch (injectError) {
@@ -220,7 +278,8 @@ function apply(ctx, rawConfig = {}) {
 				kind: "skill",
 				toolCalls: signal.toolCalls,
 				userChars: signal.userChars,
-				assistantChars: signal.assistantChars
+				assistantChars: signal.assistantChars,
+				channel: "completion"
 			});
 		} catch (emitError) {
 			ctx.logger.warn(`dsh-evolution-review: review-scheduled emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
@@ -271,17 +330,25 @@ function apply(ctx, rawConfig = {}) {
 		const subagents = ctx.get("subagents");
 		if (!subagents) return false;
 		if (reviewInFlight) {
-			if (deferredFallbackReviews.length < DEFERRED_REVIEW_CAP) deferredFallbackReviews.push({
-				agent,
-				kind
-			});
-			else ctx.logger.warn(`dsh-evolution-review: deferred-review queue at cap (${DEFERRED_REVIEW_CAP}) — dropping one fallback review prompt`);
-			return "deferred";
+			if (deferredFallbackReviews.length < DEFERRED_REVIEW_CAP) {
+				deferredFallbackReviews.push({
+					agent,
+					sessionId: session.id,
+					kind,
+					prompt: reviewPrompt(kind),
+					label: "auto-review",
+					channel: "inject",
+					counts: signal
+				});
+				return "deferred";
+			}
+			ctx.logger.warn(`dsh-evolution-review: deferred-review queue at cap (${DEFERRED_REVIEW_CAP}) — dropping one fallback review prompt`);
+			return "dropped";
 		}
 		reviewInFlight = true;
 		try {
 			const snapshot = policy();
-			const model = kind === "memory" ? snapshot?.memoryReviewModel ?? "deepseek-v4-flash" : snapshot?.skillReviewModel ?? "deepseek-v4-pro";
+			const model = kind === "memory" ? snapshot?.memoryReviewModel ?? DEFAULT_MEMORY_REVIEW_MODEL : snapshot?.skillReviewModel ?? DEFAULT_SKILL_REVIEW_MODEL;
 			const reviewText = redactSecrets(buildReviewRequest(session, kind, signal, config.reviewContextMessages, config.reviewMessageChars));
 			const agentOptions = { model };
 			if (config.reviewProvider) agentOptions.provider = config.reviewProvider;
@@ -323,9 +390,38 @@ function apply(ctx, rawConfig = {}) {
 				});
 				const acceptedSkillOps = validation.accepted.skillOps ?? [];
 				const skippedUnread = filterUnreadSkillOps(acceptedSkillOps, new Set([...collectReadSkillNames(session), ...childReads]));
-				const executed = await withTimeout(executePlan(validation.accepted, session), config.reviewTimeoutMs, "review plan execution");
-				const actions = executed.actions;
 				const evidenceQuotes = [...validation.accepted.memoryOps ?? [], ...acceptedSkillOps].reduce((total, op) => total + (Array.isArray(op.evidence) ? op.evidence.length : 0), 0);
+				const emitApplied = (report) => {
+					try {
+						ctx.emit("evolution/plan-applied", {
+							sessionId: session.id,
+							planId: randomUUID(),
+							policyFingerprint,
+							memoryApplied: report.actions.filter((action) => action.startsWith("Memory")).length,
+							skillApplied: report.actions.filter((action) => action.startsWith("Skill ")).length,
+							rejectedOps: validation.rejected.length,
+							...skippedUnread > 0 ? { skippedUnread } : {},
+							executionFailures: report.failedOps?.length ?? 0,
+							...report.executionError !== void 0 ? { executionError: report.executionError } : {},
+							evidenceQuotes,
+							estimatedInputChars: reviewText.length
+						});
+					} catch (emitError) {
+						ctx.logger.warn(`dsh-evolution-review: plan-applied emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
+					}
+				};
+				const landed = [];
+				let executed;
+				try {
+					executed = await withTimeout(executePlan(validation.accepted, session, (action) => landed.push(action)), config.reviewTimeoutMs, "review plan execution");
+				} catch (error) {
+					emitApplied({
+						actions: landed,
+						executionError: `execution timed out after ${config.reviewTimeoutMs}ms`
+					});
+					throw error;
+				}
+				const actions = executed.actions;
 				if (actions.length > 0) {
 					const applied = actions.join(" · ");
 					const note = executed.ok ? "" : "\n部分操作失败。以下操作已应用，请勿重复执行。";
@@ -347,23 +443,11 @@ function apply(ctx, rawConfig = {}) {
 				} catch (injectError) {
 					ctx.logger.warn(`dsh-evolution-review: zero-landing notice inject failed: ${injectError instanceof Error ? injectError.message : String(injectError)}`);
 				}
-				try {
-					ctx.emit("evolution/plan-applied", {
-						sessionId: session.id,
-						planId: randomUUID(),
-						policyFingerprint,
-						memoryApplied: actions.filter((action) => action.startsWith("Memory")).length,
-						skillApplied: actions.filter((action) => action.startsWith("Skill ")).length,
-						rejectedOps: validation.rejected.length + skippedUnread,
-						executionFailures: executed.failedOps.length,
-						...executed.aborted !== void 0 ? { executionError: executed.aborted } : {},
-						...executed.failedOps[0] !== void 0 && executed.aborted === void 0 ? { executionError: executed.failedOps[0] } : {},
-						evidenceQuotes,
-						estimatedInputChars: reviewText.length
-					});
-				} catch (emitError) {
-					ctx.logger.warn(`dsh-evolution-review: plan-applied emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
-				}
+				emitApplied({
+					actions,
+					failedOps: executed.failedOps,
+					...executed.aborted !== void 0 ? { executionError: executed.aborted } : executed.failedOps[0] !== void 0 ? { executionError: executed.failedOps[0] } : {}
+				});
 				return true;
 			} finally {
 				try {
@@ -379,14 +463,30 @@ function apply(ctx, rawConfig = {}) {
 		} finally {
 			reviewInFlight = false;
 			const deferred = deferredFallbackReviews.splice(0);
-			for (const { agent: waitingAgent, kind: waitingKind } of deferred) try {
-				deliverMessage(waitingAgent, reviewPrompt(waitingKind), "auto-review");
-			} catch (injectError) {
-				ctx.logger.warn(`dsh-evolution-review: deferred review inject failed: ${injectError instanceof Error ? injectError.message : String(injectError)}`);
+			for (const { agent: waitingAgent, sessionId: entrySession, kind: waitingKind, prompt, label, channel, counts: entryCounts } of deferred) {
+				try {
+					deliverMessage(waitingAgent, prompt, label);
+				} catch (injectError) {
+					if (channel === "completion") completionInjected.delete(entrySession);
+					ctx.logger.warn(`dsh-evolution-review: deferred review inject failed: ${injectError instanceof Error ? injectError.message : String(injectError)}`);
+					continue;
+				}
+				try {
+					ctx.emit("evolution/review-scheduled", {
+						sessionId: entrySession,
+						kind: waitingKind,
+						toolCalls: entryCounts.toolCalls,
+						userChars: entryCounts.userChars,
+						assistantChars: entryCounts.assistantChars,
+						channel
+					});
+				} catch (emitError) {
+					ctx.logger.warn(`dsh-evolution-review: review-scheduled emit failed: ${emitError instanceof Error ? emitError.message : String(emitError)}`);
+				}
 			}
 		}
 	}
-	async function executePlan(plan, session) {
+	async function executePlan(plan, session, onLanded) {
 		const sessionId = session?.id;
 		const memory = ctx.get("memory");
 		const approval = ctx.get("evolutionApproval");
@@ -413,8 +513,10 @@ function apply(ctx, rawConfig = {}) {
 					old_text: op.old_text
 				};
 				const result = approval ? await runApproved("memory", `memory ${normalized.target} ${normalized.action}`, normalized, normalized, session) : await memory?.applyBatch(normalized.target, [normalized]);
-				if (result?.ok) actions.push("Memory updated");
-				else {
+				if (result?.ok) {
+					actions.push("Memory updated");
+					onLanded?.("Memory updated");
+				} else {
 					ok = false;
 					failedOps.push(`memory ${normalized.action} ${normalized.target}: ${result?.message ?? "service unavailable"}`);
 				}
@@ -438,8 +540,10 @@ function apply(ctx, rawConfig = {}) {
 					origin: origins.library
 				};
 				const result = approval ? await runApproved("skill", `skill ${op.action ?? "patch"} ${op.name}`, runnerArgs, runnerArgs, session) : await executeSkillDirect(args);
-				if (result?.ok) actions.push(`Skill ${op.name} ${op.action ?? "patch"}`);
-				else {
+				if (result?.ok) {
+					actions.push(`Skill ${op.name} ${op.action ?? "patch"}`);
+					onLanded?.(`Skill ${op.name} ${op.action ?? "patch"}`);
+				} else {
 					ok = false;
 					failedOps.push(`skill ${op.action ?? "patch"} ${op.name}: ${result?.message ?? "service unavailable"}`);
 				}
@@ -559,6 +663,7 @@ function apply(ctx, rawConfig = {}) {
 		pendingCadenceWarned.clear();
 		skipNextCadenceFire.clear();
 		cadenceResetWarned.clear();
+		reviewStateLocks.clear();
 	}, "dsh-evolution-review.cleanup");
 }
 /** Completion-channel decision: task finished normally AND the session is proven long. */

package/lib/types/index.d.ts

@@ -47,8 +47,9 @@ export interface Config {
      * skills in the SAME tree the catalog/tools read instead of writing a
      * parallel tree the rest of the family cannot see. E-7 (v18): canonical key. */
     root?: string;
-    /** Deprecated alias of `root` (E-7, v18); honoured only while `root` is
-     * empty, with a warning; removed after 0.3.65. */
+    /** V27 G2.4 (M-08): the retired `skillsRoot` alias, kept in the schema so a
+     * config that still sets it reaches {@link assertSkillsRootAliasRetired} and
+     * fails the load instead of being silently dropped. Never read. */
     skillsRoot?: string;
 }
 export declare const Config: z<Config>;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-review",
   "description": "Background review orchestration (community build)",
-  "version": "0.3.66",
+  "version": "0.3.68",
   "publishConfig": {
     "access": "public"
   },
@@ -31,9 +31,9 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-approval": "^0.3.66",
-    "@lmzhen/dsh-evolution-core": "^0.3.66",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.3.66"
+    "@lmzhen/dsh-evolution-approval": "^0.3.68",
+    "@lmzhen/dsh-evolution-core": "^0.3.68",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.3.68"
   },
   "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.66",
-    "@lmzhen/dsh-evolution-policy": "^0.3.66"
+    "@lmzhen/dsh-evolution-state": "^0.3.68",
+    "@lmzhen/dsh-evolution-policy": "^0.3.68"
   },
   "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.66",
-    "@lmzhen/dsh-evolution-core": "^0.3.66",
-    "@lmzhen/dsh-evolution-plan-validator": "^0.3.66",
-    "@lmzhen/dsh-evolution-state": "^0.3.66"
+    "@lmzhen/dsh-evolution-approval": "^0.3.68",
+    "@lmzhen/dsh-evolution-core": "^0.3.68",
+    "@lmzhen/dsh-evolution-plan-validator": "^0.3.68",
+    "@lmzhen/dsh-evolution-state": "^0.3.68"
   }
 }