@lmzhen/dsh-evolution-maintenance 0.17.1 → 0.18.0

package/lib/index.js

@@ -1,4 +1,4 @@
-import { i as snapshotFromLibrary, n as PROBE_SIGNALS, r as computeProbe, t as buildEnrichment } from "./enrichment-CjZmhRI9.js";
+import { a as snapshotFromLibrary, i as enrichmentSnapshotOptions, n as computeProbe, r as buildEnrichment, t as PROBE_SIGNALS } from "./probe-Bojb0piw.js";
 import { AUTHORING_DESCRIPTION_BAR, DEFAULT_CURATOR_MODEL, DEFAULT_HEALTH_THRESHOLDS, DRIFT_MAX_LINE_CHARS, DRIFT_SIGNALS_VERSION, DRIFT_SIGNAL_NOUNS, LOW_QUALITY_THRESHOLD, MAINTAIN_OUTPUT_INSTRUCTION, MAINTAIN_PROMPT, MAX_TIMER_DELAY_MS, MIN_STAMP_BODY_CHARS, PROMPT_BUNDLE, PROMPT_BUNDLE_ID, computeDriftSignals, findDriftSignal, redactSecrets, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 import { createHash, randomUUID } from "node:crypto";
 //#region lib/types/render-facts.js
@@ -266,9 +266,12 @@ function validateAndNormalizeMaintainPlan(raw, report, validSignals) {
 		forcedHuman.push(...item.names.filter((name) => unknownQualitySkills.has(name)));
 	}
 	for (const item of plan) {
-		const referencesUnknown = item.evidence.some((ev) => {
-			return findSignalInReport(report, ev.signal)?.verdict === "unknown";
+		const unknownIn = (signals) => item.evidence.some((ev) => findDriftSignal(signals, ev.signal)?.verdict === "unknown");
+		const namedSkills = item.names.flatMap((name) => {
+			const skill = report.skills.find((candidate) => candidate.name.trim() === name);
+			return skill === void 0 ? [] : [skill];
 		});
+		const referencesUnknown = namedSkills.length > 0 ? namedSkills.some((skill) => unknownIn(skill.signals)) : unknownIn(report.library);
 		const lowConfidence = item.confidence < .6;
 		const irreversible = item.reversibility === "rename" || item.reversibility === "none";
 		if (!item.needs_human && (lowConfidence || irreversible || item.is_override || referencesUnknown)) item.needs_human = true;
@@ -284,9 +287,6 @@ function validateAndNormalizeMaintainPlan(raw, report, validSignals) {
 		forcedHuman
 	};
 }
-function findSignalInReport(report, id) {
-	return report.library.find((signal) => signal.id === id) ?? report.skills.flatMap((skill) => skill.signals).find((signal) => signal.id === id);
-}
 //#endregion
 //#region lib/types/orchestrate.js
 /**
@@ -405,18 +405,10 @@ async function runMaintain(runtime, options = {}) {
 			recommendationCount: 0,
 			error: "dsh-evolution prompt bundle integrity check failed; refusing to run maintain"
 		};
-		const usageObserved = options.usageObserved ? options.usageObserved() : void 0;
+		const enrichment = options.enrichment?.();
 		const readFailures = [];
 		const snapshots = await snapshotFromLibrary(runtime.library, {
-			supportFiles: options.supportFiles ? options.supportFiles() : void 0,
-			descriptions: options.descriptions ? options.descriptions() : void 0,
-			quality: options.quality ? options.quality() : void 0,
-			protected: options.protected ? options.protected() : void 0,
-			catalogInvalid: options.catalogInvalid ? options.catalogInvalid() : void 0,
-			usageObserved,
-			...options.demand === void 0 ? {} : { demand: options.demand() },
-			...options.liveness === void 0 ? {} : { liveness: options.liveness() },
-			...options.supportChars === void 0 ? {} : { supportChars: options.supportChars() },
+			...enrichment === void 0 ? {} : enrichmentSnapshotOptions(enrichment),
 			onReadError: (name, error) => {
 				readFailures.push(name);
 				runtime.logger?.warn(`evolution-maintenance: skipping unreadable skill "${name}": ${error instanceof Error ? error.message : String(error)}`);
@@ -441,7 +433,7 @@ async function runMaintain(runtime, options = {}) {
 				text: "Maintenance scan: empty skill library. Nothing to do."
 			};
 		}
-		const { facts, report, signalsVersion, signature } = buildMaintainFacts(snapshots, usageObserved, options.redact);
+		const { facts, report, signalsVersion, signature } = buildMaintainFacts(snapshots, enrichment?.usageObservedValue, options.redact);
 		const template = renderMaintainTemplate(MAINTAIN_PROMPT, PROMPT_BUNDLE_ID, signalsVersion, signature);
 		const probeRegistered = runtime.tools?.get("maintenance_probe") != null;
 		const probeDegraded = options.toolAllow === void 0 && !probeRegistered;
@@ -589,4 +581,4 @@ ${MAINTAIN_OUTPUT_INSTRUCTION}`;
 	}
 }
 //#endregion
-export { MAINTAIN_RULE_IDS, PROBE_SIGNALS, buildEnrichment, buildMaintainFacts, computeProbe, renderFacts, renderMaintainTemplate, runMaintain, snapshotFromLibrary, validateAndNormalizeMaintainPlan };
+export { MAINTAIN_RULE_IDS, PROBE_SIGNALS, buildEnrichment, buildMaintainFacts, computeProbe, enrichmentSnapshotOptions, renderFacts, renderMaintainTemplate, runMaintain, snapshotFromLibrary, validateAndNormalizeMaintainPlan };

package/lib/{enrichment-CjZmhRI9.js → probe-Bojb0piw.js}

@@ -52,6 +52,91 @@ async function snapshotFromLibrary(library, options = {}) {
 	return snapshots;
 }
 //#endregion
+//#region lib/types/enrichment.js
+/**
+* Enrichment maps shared by the full maintain scan and the `--facts` preview
+* (v12) AND the maintenance_probe tool (0.3.9): one construction = the probe
+* and the facts block can never disagree about descriptions, support files,
+* or quality (the 0.3.8 review found the probe answering "description=missing"
+* while the facts block measured real lengths — probe snapshots carried no
+* enrichment at all).
+*/
+/**
+* The ONE enrichment → snapshot-options mapping (T3-02/A30).
+*
+* Three callers assemble drift snapshots from the same enrichment — the full scan, the
+* `--facts` preview, and the `maintenance_probe` tool. Kept as one function because a hand-written
+* literal per caller is a copy that drifts: the probe's literal omitted `liveness`, so the deep-dive
+* channel answered `retire: no-age` ("age unknown ⇒ not enough evidence") for every skill, for ever,
+* while the facts block listed retirement candidates for the same tree. Adding a measured field to
+* {@link Enrichment} now reaches every surface through this mapping.
+* @param enrichment - the built enrichment maps.
+* @returns the snapshot options carrying every field the enrichment measured.
+*/
+function enrichmentSnapshotOptions(enrichment) {
+	return {
+		descriptions: enrichment.descriptions,
+		supportFiles: enrichment.supportFiles,
+		quality: enrichment.quality,
+		protected: enrichment.protected,
+		catalogInvalid: enrichment.catalogInvalid,
+		usageObserved: enrichment.usageObservedValue,
+		demand: enrichment.demand,
+		liveness: enrichment.liveness,
+		supportChars: enrichment.supportChars
+	};
+}
+async function buildEnrichment(ctx, library) {
+	const skillUsage = ctx.get("skillUsage");
+	const usageMap = skillUsage?.report ? await skillUsage.report() : void 0;
+	const usageObservedValue = usageMap ? usageObserved(usageMap) : void 0;
+	const descriptions = /* @__PURE__ */ new Map();
+	const supportFiles = /* @__PURE__ */ new Map();
+	const quality = /* @__PURE__ */ new Map();
+	const protectedMap = /* @__PURE__ */ new Map();
+	const catalogInvalid = /* @__PURE__ */ new Map();
+	const demand = /* @__PURE__ */ new Map();
+	const liveness = /* @__PURE__ */ new Map();
+	const supportChars = /* @__PURE__ */ new Map();
+	const now = /* @__PURE__ */ new Date();
+	for (const entry of await library.list()) {
+		if (entry.protectionUnknown) protectedMap.set(entry.name, "unknown");
+		else if (entry.protectedBy) protectedMap.set(entry.name, entry.protectedBy);
+		let body;
+		try {
+			body = await library.read(entry.name);
+		} catch (error) {
+			ctx.logger.warn(`evolution-maintenance: enrichment skipped unreadable skill "${entry.name}": ${error instanceof Error ? error.message : String(error)}`);
+			continue;
+		}
+		if (typeof body !== "string") continue;
+		const parsed = parseFrontmatter(body);
+		if (frontmatterCatalogInvalid(body)) catalogInvalid.set(entry.name, true);
+		const description = parsed?.frontmatter.description;
+		if (typeof description === "string" && description.trim().length > 0) descriptions.set(entry.name, description);
+		const files = await library.listSupportFiles(entry.name);
+		if (isPresent(files) && files.value.length > 0) supportFiles.set(entry.name, files.value);
+		const record = usageMap?.get(entry.name);
+		if (typeof record?.quality_score === "number") quality.set(entry.name, record.quality_score);
+		const reads = record?.support_reads;
+		if (reads !== void 0 && Object.keys(reads).length > 0) demand.set(entry.name, reads);
+		if (record !== void 0) liveness.set(entry.name, { idleDays: idleDays(record, now) });
+		const big = await library.supportFileChars(entry.name);
+		if (big !== null && Object.keys(big).length > 0) supportChars.set(entry.name, big);
+	}
+	return {
+		descriptions,
+		supportFiles,
+		quality,
+		usageObservedValue,
+		protected: protectedMap,
+		catalogInvalid,
+		demand,
+		liveness,
+		supportChars
+	};
+}
+//#endregion
 //#region lib/types/probe.js
 /**
 * Probe detail computation (011 Phase 3).
@@ -156,64 +241,4 @@ function computeProbe(signal, target, snapshots) {
 	}
 }
 //#endregion
-//#region lib/types/enrichment.js
-/**
-* Enrichment maps shared by the full maintain scan and the `--facts` preview
-* (v12) AND the maintenance_probe tool (0.3.9): one construction = the probe
-* and the facts block can never disagree about descriptions, support files,
-* or quality (the 0.3.8 review found the probe answering "description=missing"
-* while the facts block measured real lengths — probe snapshots carried no
-* enrichment at all).
-*/
-async function buildEnrichment(ctx, library) {
-	const skillUsage = ctx.get("skillUsage");
-	const usageMap = skillUsage?.report ? await skillUsage.report() : void 0;
-	const usageObservedValue = usageMap ? usageObserved(usageMap) : void 0;
-	const descriptions = /* @__PURE__ */ new Map();
-	const supportFiles = /* @__PURE__ */ new Map();
-	const quality = /* @__PURE__ */ new Map();
-	const protectedMap = /* @__PURE__ */ new Map();
-	const catalogInvalid = /* @__PURE__ */ new Map();
-	const demand = /* @__PURE__ */ new Map();
-	const liveness = /* @__PURE__ */ new Map();
-	const supportChars = /* @__PURE__ */ new Map();
-	const now = /* @__PURE__ */ new Date();
-	for (const entry of await library.list()) {
-		if (entry.protectionUnknown) protectedMap.set(entry.name, "unknown");
-		else if (entry.protectedBy) protectedMap.set(entry.name, entry.protectedBy);
-		let body;
-		try {
-			body = await library.read(entry.name);
-		} catch (error) {
-			ctx.logger.warn(`evolution-maintenance: enrichment skipped unreadable skill "${entry.name}": ${error instanceof Error ? error.message : String(error)}`);
-			continue;
-		}
-		if (typeof body !== "string") continue;
-		const parsed = parseFrontmatter(body);
-		if (frontmatterCatalogInvalid(body)) catalogInvalid.set(entry.name, true);
-		const description = parsed?.frontmatter.description;
-		if (typeof description === "string" && description.trim().length > 0) descriptions.set(entry.name, description);
-		const files = await library.listSupportFiles(entry.name);
-		if (isPresent(files) && files.value.length > 0) supportFiles.set(entry.name, files.value);
-		const record = usageMap?.get(entry.name);
-		if (typeof record?.quality_score === "number") quality.set(entry.name, record.quality_score);
-		const reads = record?.support_reads;
-		if (reads !== void 0 && Object.keys(reads).length > 0) demand.set(entry.name, reads);
-		if (record !== void 0) liveness.set(entry.name, { idleDays: idleDays(record, now) });
-		const big = await library.supportFileChars(entry.name);
-		if (big !== null && Object.keys(big).length > 0) supportChars.set(entry.name, big);
-	}
-	return {
-		descriptions,
-		supportFiles,
-		quality,
-		usageObservedValue,
-		protected: protectedMap,
-		catalogInvalid,
-		demand,
-		liveness,
-		supportChars
-	};
-}
-//#endregion
-export { snapshotFromLibrary as i, PROBE_SIGNALS as n, computeProbe as r, buildEnrichment as t };
+export { snapshotFromLibrary as a, enrichmentSnapshotOptions as i, computeProbe as n, buildEnrichment as r, PROBE_SIGNALS as t };

package/lib/tools.js

@@ -1,4 +1,4 @@
-import { i as snapshotFromLibrary, n as PROBE_SIGNALS, r as computeProbe, t as buildEnrichment } from "./enrichment-CjZmhRI9.js";
+import { a as snapshotFromLibrary, i as enrichmentSnapshotOptions, n as computeProbe, r as buildEnrichment, t as PROBE_SIGNALS } from "./probe-Bojb0piw.js";
 import { assertSkillsRootAliasRetired, newSkillLibrary, redactSecrets, resolveRootConfig } from "@lmzhen/dsh-evolution-core";
 import z from "@deepseek-ai/schemastery";
 import { defineTool } from "@deepseek-ai/dsh-tools";
@@ -69,16 +69,7 @@ function apply(ctx, rawConfig = {}) {
 					config: rootConfig,
 					io: ioRegistry.provider()
 				});
-				const enrichment = await buildEnrichment(ctx, library);
-				const probe = computeProbe(signal, target, await snapshotFromLibrary(library, {
-					descriptions: enrichment.descriptions,
-					supportFiles: enrichment.supportFiles,
-					quality: enrichment.quality,
-					protected: enrichment.protected,
-					catalogInvalid: enrichment.catalogInvalid,
-					usageObserved: enrichment.usageObservedValue,
-					demand: enrichment.demand
-				}));
+				const probe = computeProbe(signal, target, await snapshotFromLibrary(library, enrichmentSnapshotOptions(await buildEnrichment(ctx, library))));
 				const redacted = redactSecrets(probe.detail.join("\n"));
 				return {
 					...probe,

package/lib/types/enrichment.d.ts

@@ -7,6 +7,7 @@
  * enrichment at all).
  */
 import type { Context } from '@deepseek-ai/cordis';
+import type { SnapshotOptions } from './drift-scan.ts';
 import { type SkillLibrary, type SkillLiveness } from '@lmzhen/dsh-evolution-core';
 export interface Enrichment {
     descriptions: ReadonlyMap<string, string>;
@@ -28,5 +29,18 @@ export interface Enrichment {
      * common measured-and-clean case adds no field. */
     supportChars: ReadonlyMap<string, Readonly<Record<string, number>>>;
 }
+/**
+ * The ONE enrichment → snapshot-options mapping (T3-02/A30).
+ *
+ * Three callers assemble drift snapshots from the same enrichment — the full scan, the
+ * `--facts` preview, and the `maintenance_probe` tool. Kept as one function because a hand-written
+ * literal per caller is a copy that drifts: the probe's literal omitted `liveness`, so the deep-dive
+ * channel answered `retire: no-age` ("age unknown ⇒ not enough evidence") for every skill, for ever,
+ * while the facts block listed retirement candidates for the same tree. Adding a measured field to
+ * {@link Enrichment} now reaches every surface through this mapping.
+ * @param enrichment - the built enrichment maps.
+ * @returns the snapshot options carrying every field the enrichment measured.
+ */
+export declare function enrichmentSnapshotOptions(enrichment: Enrichment): SnapshotOptions;
 export declare function buildEnrichment(ctx: Context, library: SkillLibrary): Promise<Enrichment>;
 //# sourceMappingURL=enrichment.d.ts.map

package/lib/types/orchestrate.d.ts

@@ -4,9 +4,10 @@
  * text. Deterministic parts stay pure; the subagent call is the only
  * external dependency (injected, fake-able in tests).
  */
-import { type DriftReport, type DriftSkillSnapshot, type SkillLiveness } from '@lmzhen/dsh-evolution-core';
+import { type DriftReport, type DriftSkillSnapshot } from '@lmzhen/dsh-evolution-core';
 import type { Agent } from '@deepseek-ai/dsh-agent';
 import { type SkillLibraryLike } from './drift-scan.ts';
+import { type Enrichment } from './enrichment.ts';
 /** I-5 (v37): the platform's `SubagentStartRequest` fields this orchestrator sends.
  * Typed locally (not imported) because the platform type lives behind the subagent row this
  * package does not depend on; the required `parent` and `signal` mirror the platform. */
@@ -92,18 +93,10 @@ export interface MaintainOptions {
     provider?: string;
     toolAllow?: readonly string[];
     redact?: ((text: string) => string) | undefined;
-    supportFiles?: () => ReadonlyMap<string, readonly string[]>;
-    descriptions?: () => ReadonlyMap<string, string>;
-    quality?: () => ReadonlyMap<string, number>;
-    protected?: () => ReadonlyMap<string, string>;
-    catalogInvalid?: () => ReadonlyMap<string, boolean>;
-    usageObserved?: () => boolean | undefined;
-    /** Per-support-file read counts per skill (design §5.5). */
-    demand?: () => ReadonlyMap<string, Readonly<Record<string, number>>>;
-    /** Idle age per skill (design §5.6). */
-    liveness?: () => ReadonlyMap<string, SkillLiveness>;
-    /** Support-file char counts for files that can exceed the cap (design §16.6). */
-    supportChars?: () => ReadonlyMap<string, Readonly<Record<string, number>>>;
+    /** T3-02/A30: the measured enrichment, read through {@link enrichmentSnapshotOptions} — one
+     * mapping shared with the `--facts` preview and the probe tool. A closure per field was a second
+     * account of the same facts and drifted (the probe's copy lost `liveness`). */
+    enrichment?: (() => Enrichment) | undefined;
 }
 export interface MaintainOutcome {
     ok: boolean;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-maintenance",
   "description": "Deterministic maintenance-scan surface: skill-library snapshot assembly, drift signals and mechanical-facts rendering (design 011) (community build)",
-  "version": "0.17.1",
+  "version": "0.18.0",
   "publishConfig": {
     "access": "public"
   },
@@ -30,8 +30,8 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.17.1"
+    "@deepseek-ai/schemastery": "^3.18.4",
+    "@lmzhen/dsh-evolution-core": "^0.18.0"
   },
   "peerDependencies": {
     "@deepseek-ai/dsh-agent": "^0.2.0-rc.1",