@lmzhen/dsh-evolution-curator 0.3.56 → 0.3.58

package/README.md

@@ -22,7 +22,11 @@ Independent of request-prefix construction. This package does not alter the asse
 
 
 - LLM nomination pass is advisory and disabled by default; deterministic lifecycle remains authoritative.
-- Consolidation is control-plane only (`/evolution consolidate <target> <source...>`): no LLM pass proposes merge groups yet, and merged source bodies are appended verbatim rather than rewritten into a synthesized skill.
+- Consolidation is a REAL merger path: the LLM nomination (when enabled / via `recommend()`) proposes
+  merge groups, the control plane gates them and applies the mutation — merged source bodies are appended
+  verbatim rather than rewritten into a synthesized skill. The one-click `/evolution consolidate
+  <target> <source...>` remains the direct manual path (P2-8, v11: the earlier "no LLM pass proposes
+  merge groups" wording contradicted the tested nomination chain).
 
 ## Recovery and consolidation
 

package/lib/index.js

@@ -3,7 +3,7 @@ import { randomUUID } from "node:crypto";
 import { join } from "node:path";
 import { BlockAssembler, createUserMessage } from "@deepseek-ai/dsh-llm";
 import z from "@deepseek-ai/schemastery";
-import { CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, SkillLibrary, buildCuratorRunReport, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, emptyRecord, evolutionHome, evolutionIoAdapter, foldCuratorFields, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, parseCuratorNominations, parseFrontmatter, relatedSkillNames, renderCuratorReportMarkdown, updateSuppressedNames, usageObserved } from "@lmzhen/dsh-evolution-core";
+import { CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, SKILL_NAME_RE, SkillLibrary, buildCuratorRunReport, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, emptyRecord, evolutionHome, evolutionIoAdapter, foldCuratorFields, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, parseCuratorNominations, parseFrontmatter, relatedSkillNames, renderCuratorReportMarkdown, resolveSkillsRoot, updateSuppressedNames, usageObserved } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic skill lifecycle curator with interval gate and archive.
@@ -75,7 +75,7 @@ var EvolutionCurator = class extends Service {
 	constructor(ctx, config = {}) {
 		super(ctx, "evolutionCurator");
 		this.io = evolutionIoAdapter(() => ctx.evolutionIo.provider());
-		this.skills = new SkillLibrary(void 0, this.io, void 0, (event) => {
+		this.skills = new SkillLibrary(resolveSkillsRoot({ root: config.root }), this.io, void 0, (event) => {
 			this.ctx.emit("evolution/skill-mutated", event);
 		});
 		this.enabled = config.enabled ?? true;
@@ -166,11 +166,10 @@ var EvolutionCurator = class extends Service {
 		return await this.curatorStateService()?.loadCuratorState() ?? null;
 	}
 	/**
-	* One automatic schedule check: run a pass when the persisted curator state
-	* (falling back to the in-memory clock for state-less compositions) is at
-	* least one interval old. All gates — interval, idle, first-run defer,
-	* reentrancy — stay inside `run()`, so this method only decides whether to
-	* wake it, and never duplicates gate logic.
+	* One automatic schedule check. P2-11 (v11): the interval gate is exercised
+	* HERE as a cheap pre-check (persisted or in-memory clock) AND again inside
+	* run() as the authoritative gate — the docstring no longer claims the two
+	* never duplicate; a future interval change must update both.
 	*/
 	async autoCheck() {
 		try {
@@ -578,6 +577,7 @@ var EvolutionCurator = class extends Service {
 					if (!wasBundled) {
 						const archiveEntries = await this.io.list(join(this.skills.root, ".archive"));
 						for (const entry of archiveEntries) {
+							if (!SKILL_NAME_RE.test(name)) continue;
 							if (!new RegExp(`^${name}-\\d{14}(-[0-9a-z]{1,6})?$`).test(entry)) continue;
 							wasBundled = await this.io.exists(join(this.skills.root, ".archive", entry, markerEntryName("bundled")));
 							if (wasBundled) {
@@ -679,7 +679,9 @@ var EvolutionCurator = class extends Service {
 		const usageRegistry = this.ctx.get("skillUsage");
 		try {
 			await usageRegistry?.invalidate?.();
-		} catch {}
+		} catch (error) {
+			this.ctx.logger.warn(`evolution-curator: skillUsage cache invalidate failed after curation: ${error instanceof Error ? error.message : String(error)}`);
+		}
 		return {
 			archivedSkills,
 			errors,

package/lib/types/index.d.ts

@@ -17,6 +17,11 @@ export interface Config {
     intervalHours?: number;
     staleAfterDays?: number;
     archiveAfterDays?: number;
+    /** Skill-tree root for curator scope/snapshot/archive (D2, v11) — a custom
+     * root deployment (review/tool-skill-manage write another tree) must point
+     * the curator at the SAME tree or lifecycle decisions land on the default
+     * tree and never see the deployed skills. Empty uses skillsRoot(). */
+    root?: string;
     /** Spend one LLM review pass on stale candidates before the deterministic archive step. */
     llmReview?: boolean;
     curatorProvider?: string;
@@ -130,11 +135,10 @@ export declare class EvolutionCurator extends Service {
     /** Current persisted curator state (read-only view for /evolution curator status). */
     status(): Promise<CuratorStateRecordShape | null>;
     /**
-     * One automatic schedule check: run a pass when the persisted curator state
-     * (falling back to the in-memory clock for state-less compositions) is at
-     * least one interval old. All gates — interval, idle, first-run defer,
-     * reentrancy — stay inside `run()`, so this method only decides whether to
-     * wake it, and never duplicates gate logic.
+     * One automatic schedule check. P2-11 (v11): the interval gate is exercised
+     * HERE as a cheap pre-check (persisted or in-memory clock) AND again inside
+     * run() as the authoritative gate — the docstring no longer claims the two
+     * never duplicate; a future interval change must update both.
      */
     private autoCheck;
     /**

package/package.json

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