@lmzhen/dsh-evolution-curator 0.1.0-rc.17 → 0.1.0-rc.19

package/lib/index.js

@@ -197,85 +197,23 @@ var EvolutionCurator = class extends Service {
 			suppressedNames,
 			referencedSkillNames: this.referencedSkillNames
 		});
-		const supportDirs = /* @__PURE__ */ new Map();
-		for (const name of treeNames) supportDirs.set(name, await this.skills.countSupportDirs(name));
-		const quality = computeQualityScores({
-			usage,
-			supportDirs
-		});
-		for (const [name, score] of quality) {
-			const record = usage.get(name);
-			if (record) {
-				record.quality_score = score.score;
-				record.quality_warn = score.warn;
-			}
-		}
-		const errors = [];
-		const archivedSkills = [];
+		await this.scoreTree(usage, treeNames);
 		const nominations = this.llmReview ? await this.recommend(result.markStale, { dryRun }) : {
 			prunings: [],
 			consolidations: []
 		};
 		const llmNominations = nominations.prunings;
 		const archiveCandidates = [...new Set([...result.archive, ...llmNominations])];
-		let suppressedChanged = false;
-		if (!dryRun) {
-			for (const name of archiveCandidates) {
-				const archived = await this.skills.archive(name, {
-					reason: "Lifecycle: reached archive threshold",
-					allowBundled: this.pruneBuiltins
-				});
-				if (!archived.ok) {
-					const record = usage.get(name);
-					if (record) record.state = "active";
-					errors.push(`${name}: ${archived.message}`);
-				} else {
-					archivedSkills.push({
-						name,
-						path: archived.path ?? "",
-						reason: "Lifecycle: reached archive threshold"
-					});
-					if (bundledNames.has(name)) {
-						suppressedNames.add(name);
-						suppressedChanged = true;
-					}
-				}
-			}
-			const alreadyArchived = new Set(archiveCandidates);
-			for (const nomination of nominations.consolidations) {
-				if (alreadyArchived.has(nomination.from)) continue;
-				if (!treeNames.has(nomination.from) || !treeNames.has(nomination.into)) {
-					errors.push(`${nomination.from}: consolidation target or source missing from the skill tree`);
-					continue;
-				}
-				const consolidated = await this.skills.consolidate(nomination.into, [nomination.from], "background_review");
-				if (!consolidated.ok) {
-					errors.push(`${nomination.from}: ${consolidated.message}`);
-					continue;
-				}
-				const record = usage.get(nomination.from);
-				if (record) {
-					record.state = "archived";
-					record.archived_at = (/* @__PURE__ */ new Date()).toISOString();
-				}
-				alreadyArchived.add(nomination.from);
-				archivedSkills.push({
-					name: nomination.from,
-					path: consolidated.path ?? "",
-					reason: `Consolidated into ${nomination.into}`
-				});
-			}
-			if (suppressedChanged) try {
-				await saveSuppressedNames(root, suppressedNames, this.io);
-			} catch {
-				this.ctx.logger.warn("evolution-curator: failed to persist suppressed names; archived built-ins may re-enter the lifecycle");
-			}
-			try {
-				await saveUsage(root, usage, this.io);
-			} catch {
-				this.ctx.logger.warn("evolution-curator: failed to persist usage sidecar");
-			}
-		}
+		const { archivedSkills, errors } = await this.applyMutations({
+			dryRun,
+			archiveCandidates,
+			nominations,
+			treeNames,
+			usage,
+			bundledNames,
+			suppressedNames,
+			root
+		});
 		if (!dryRun) this.lastRun = Date.now();
 		const report = buildCuratorRunReport({
 			runId,
@@ -302,6 +240,7 @@ var EvolutionCurator = class extends Service {
 		}
 		const summary = `${dryRun ? "dry-run" : "auto"}: stale:${result.markStale.length} archived:${archivedSkills.length} consolidated:${nominations.consolidations.length}`;
 		await stateService?.saveCuratorState({
+			schemaVersion: 1,
 			lastRunAt: dryRun ? persisted?.lastRunAt ?? this.lastRun : this.lastRun,
 			runCount: dryRun ? persisted?.runCount ?? 0 : (persisted?.runCount ?? 0) + 1,
 			lastSummary: summary,
@@ -334,6 +273,100 @@ var EvolutionCurator = class extends Service {
 			treeNames
 		};
 	}
+	/**
+	* F13 six-factor quality scoring, persisted onto the usage records.
+	*/
+	async scoreTree(usage, treeNames) {
+		const supportDirs = /* @__PURE__ */ new Map();
+		for (const name of treeNames) supportDirs.set(name, await this.skills.countSupportDirs(name));
+		const quality = computeQualityScores({
+			usage,
+			supportDirs
+		});
+		for (const [name, score] of quality) {
+			const record = usage.get(name);
+			if (record) {
+				record.quality_score = score.score;
+				record.quality_warn = score.warn;
+			}
+		}
+	}
+	/**
+	* Execute lifecycle archives and consolidation nominations, then persist the
+	* suppression and usage sidecars best-effort. A dry-run short-circuits: no
+	* file moves and no state persistence — the caller still writes the report.
+	*/
+	async applyMutations(input) {
+		if (input.dryRun) return {
+			archivedSkills: [],
+			errors: [],
+			suppressedChanged: false
+		};
+		const { archiveCandidates, nominations, treeNames, usage, bundledNames, suppressedNames, root } = input;
+		const errors = [];
+		const archivedSkills = [];
+		let suppressedChanged = false;
+		for (const name of archiveCandidates) {
+			const archived = await this.skills.archive(name, {
+				reason: "Lifecycle: reached archive threshold",
+				allowBundled: this.pruneBuiltins
+			});
+			if (!archived.ok) {
+				const record = usage.get(name);
+				if (record) record.state = "active";
+				errors.push(`${name}: ${archived.message}`);
+			} else {
+				archivedSkills.push({
+					name,
+					path: archived.path ?? "",
+					reason: "Lifecycle: reached archive threshold"
+				});
+				if (bundledNames.has(name)) {
+					suppressedNames.add(name);
+					suppressedChanged = true;
+				}
+			}
+		}
+		const alreadyArchived = new Set(archiveCandidates);
+		for (const nomination of nominations.consolidations) {
+			if (alreadyArchived.has(nomination.from)) continue;
+			if (!treeNames.has(nomination.from) || !treeNames.has(nomination.into)) {
+				errors.push(`${nomination.from}: consolidation target or source missing from the skill tree`);
+				continue;
+			}
+			const consolidated = await this.skills.consolidate(nomination.into, [nomination.from], "background_review");
+			if (!consolidated.ok) {
+				errors.push(`${nomination.from}: ${consolidated.message}`);
+				continue;
+			}
+			const record = usage.get(nomination.from);
+			if (record) {
+				record.state = "archived";
+				record.archived_at = (/* @__PURE__ */ new Date()).toISOString();
+			}
+			alreadyArchived.add(nomination.from);
+			archivedSkills.push({
+				name: nomination.from,
+				path: consolidated.path ?? "",
+				reason: `Consolidated into ${nomination.into}`
+			});
+		}
+		if (suppressedChanged) try {
+			await saveSuppressedNames(root, suppressedNames, this.io);
+		} catch {
+			this.ctx.logger.warn("evolution-curator: failed to persist suppressed names; archived built-ins may re-enter the lifecycle");
+		}
+		try {
+			await saveUsage(root, usage, this.io);
+		} catch {
+			this.ctx.logger.warn("evolution-curator: failed to persist usage sidecar");
+		}
+		return {
+			archivedSkills,
+			errors,
+			suppressedChanged
+		};
+	}
 	recentSessionActive() {
 		const agents = this.ctx.get("agents");
 		if (!agents) return false;

package/lib/types/index.d.ts

@@ -46,6 +46,14 @@ export interface CuratorRunOutcome {
     /** LLM nominations when the optional review pass is enabled (audit visibility). */
     nominations?: CuratorNominations;
 }
+/** Persisted curator-state record shape (schemaVersion optional for legacy reads). */
+export interface CuratorStateRecordShape {
+    schemaVersion?: number;
+    lastRunAt: number;
+    runCount: number;
+    lastSummary: string;
+    paused: boolean;
+}
 export declare class EvolutionCurator extends Service {
     static inject: string[];
     static Config: Schema<Config>;
@@ -97,6 +105,16 @@ export declare class EvolutionCurator extends Service {
      * returns the full active tree names for nomination validation.
      */
     private seedBaseline;
+    /**
+     * F13 six-factor quality scoring, persisted onto the usage records.
+     */
+    private scoreTree;
+    /**
+     * Execute lifecycle archives and consolidation nominations, then persist the
+     * suppression and usage sidecars best-effort. A dry-run short-circuits: no
+     * file moves and no state persistence — the caller still writes the report.
+     */
+    private applyMutations;
     private recentSessionActive;
     latestReport(): Promise<CuratorRunReport | null>;
     /**

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-curator",
   "description": "Deterministic skill lifecycle and recovery (community build)",
-  "version": "0.1.0-rc.17",
+  "version": "0.1.0-rc.19",
   "publishConfig": {
     "access": "public"
   },
@@ -33,20 +33,20 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.17"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.19"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.17",
-    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.17"
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.19",
+    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.19"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-invariants": "^0.1.0-rc.6",
     "@deepseek-ai/dsh-llm": "^0.1.0-rc.6",
-    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.17",
-    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.17",
-    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.17"
+    "@lmzhen/dsh-evolution-core": "^0.1.0-rc.19",
+    "@lmzhen/dsh-evolution-io": "^0.1.0-rc.19",
+    "@lmzhen/dsh-evolution-state": "^0.1.0-rc.19"
   }
 }