@lmzhen/dsh-evolution-curator 0.3.17 → 0.3.19

package/lib/index.js

@@ -32,6 +32,7 @@ var EvolutionCurator = class extends Service {
 		curatorProvider: z.string().default("deepseek-official"),
 		qualityWarnStaleAfterDays: z.number().default(DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS),
 		minIdleHours: z.number().default(DEFAULT_MIN_IDLE_HOURS),
+		minIdleFailOpen: z.boolean().default(true),
 		excludeSkillNames: z.array(z.string()).default([]),
 		manageUnmanaged: z.boolean().default(false),
 		pruneBuiltins: z.boolean().default(false),
@@ -53,6 +54,7 @@ var EvolutionCurator = class extends Service {
 	curatorProvider;
 	qualityWarnStaleAfterDays;
 	minIdleHours;
+	minIdleFailOpen;
 	excludeSkillNames;
 	manageUnmanaged;
 	pruneBuiltins;
@@ -66,6 +68,10 @@ var EvolutionCurator = class extends Service {
 	timer;
 	bootCheck;
 	running = false;
+	/** 0.3.18 (E-18): stateless first-run defer fires ONCE per process — the
+	* in-memory clock is seeded and later due runs must proceed, or the
+	* persisted===null defer repeats forever (no state service to persist). */
+	statelessFirstRunDeferred = false;
 	constructor(ctx, config = {}) {
 		super(ctx, "evolutionCurator");
 		this.io = evolutionIoAdapter(() => ctx.evolutionIo.provider());
@@ -80,6 +86,7 @@ var EvolutionCurator = class extends Service {
 		this.curatorProvider = config.curatorProvider ?? "deepseek-official";
 		this.qualityWarnStaleAfterDays = config.qualityWarnStaleAfterDays ?? DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS;
 		this.minIdleHours = config.minIdleHours ?? DEFAULT_MIN_IDLE_HOURS;
+		this.minIdleFailOpen = config.minIdleFailOpen ?? true;
 		this.excludeSkillNames = new Set(config.excludeSkillNames ?? []);
 		this.manageUnmanaged = config.manageUnmanaged ?? false;
 		this.pruneBuiltins = config.pruneBuiltins ?? false;
@@ -134,15 +141,13 @@ var EvolutionCurator = class extends Service {
 	* kept deliberately: an unattended resume must not auto-run mid-boot).
 	*/
 	async setPaused(paused) {
-		const stateService = this.curatorStateService();
-		const persisted = await stateService?.loadCuratorState() ?? null;
-		await stateService?.saveCuratorState({
+		await this.curatorStateService()?.transactCuratorState((current) => ({
 			schemaVersion: 1,
-			lastRunAt: persisted?.lastRunAt ?? Date.now(),
-			runCount: persisted?.runCount ?? 0,
-			lastSummary: persisted?.lastSummary ?? (paused ? "paused" : "resumed"),
+			lastRunAt: current?.lastRunAt ?? Date.now(),
+			runCount: current?.runCount ?? 0,
+			lastSummary: current?.lastSummary ?? (paused ? "paused" : "resumed"),
 			paused
-		});
+		}));
 	}
 	/** Current persisted curator state (read-only view for /evolution curator status). */
 	async status() {
@@ -156,8 +161,24 @@ var EvolutionCurator = class extends Service {
 	* wake it, and never duplicates gate logic.
 	*/
 	async autoCheck() {
-		const last = (await this.curatorStateService()?.loadCuratorState())?.lastRunAt ?? this.lastRun;
-		if (Date.now() - last >= this.lifecycle().intervalHours * 36e5) await this.run();
+		try {
+			const last = (await this.curatorStateService()?.loadCuratorState())?.lastRunAt ?? this.lastRun;
+			if (Date.now() - last >= this.lifecycle().intervalHours * 36e5) await this.run();
+		} catch (error) {
+			const reason = error instanceof Error ? error.message : String(error);
+			this.ctx.logger.warn(`evolution-curator: automatic check failed: ${reason}`);
+			try {
+				const runId = randomUUID();
+				await this.io.writeText(join(evolutionHome(), "reports", `curator-error-${runId}.json`), JSON.stringify({
+					runId,
+					failed: true,
+					error: reason,
+					at: (/* @__PURE__ */ new Date()).toISOString()
+				}, null, 2));
+			} catch (persistenceError) {
+				this.ctx.logger.warn(`evolution-curator: failed to persist auto-check error report: ${persistenceError instanceof Error ? persistenceError.message : String(persistenceError)}`);
+			}
+		}
 	}
 	/**
 	* Optional Hermes-curator LLM pass. The model may only NOMINATE pruning and
@@ -173,14 +194,14 @@ var EvolutionCurator = class extends Service {
 		if (candidates.length === 0) return empty;
 		const llm = this.ctx.get("llm");
 		if (!llm) return empty;
-		const model = this.ctx.get("evolutionPolicy")?.get().curatorModel ?? "deepseek-v4-pro";
+		const model = this.ctx.get("evolutionPolicy")?.get()?.curatorModel ?? "deepseek-v4-pro";
 		const clusters = computePrefixClusters(candidates);
 		const clusterLines = clusters.length === 0 ? ["Prefix clusters observed in the candidate list: (none)"] : ["Prefix clusters observed in the candidate list (orientation only — verify against the names above; you may also flag additional clusters):", ...clusters.map((cluster) => `- '${cluster.key}': ${cluster.members.join(", ")}`)];
 		const prompt = [
 			options.dryRun ? CURATOR_DRY_RUN_BANNER : "",
 			CURATOR_PROMPT,
 			"",
-			`Stale candidates observed by the deterministic lifecycle scanner:${candidates.length === 0 ? " (none)" : ""}`,
+			"Stale candidates observed by the deterministic lifecycle scanner:",
 			...candidates.map((name) => `- ${name}`),
 			"",
 			...clusterLines,
@@ -211,7 +232,8 @@ var EvolutionCurator = class extends Service {
 				prunings: parsed.prunings.filter((name) => candidates.includes(name)),
 				consolidations: parsed.consolidations.filter((item) => candidates.includes(item.from))
 			};
-		} catch {
+		} catch (error) {
+			this.ctx.logger.warn(`evolution-curator: LLM nomination pass failed: ${error instanceof Error ? error.message : String(error)}`);
 			return empty;
 		}
 	}
@@ -320,20 +342,24 @@ var EvolutionCurator = class extends Service {
 			report: this.skippedReport(runId, startedAt),
 			skipped: "active-session"
 		};
-		if (!ignoreGates && persisted === null) {
-			await stateService?.saveCuratorState({
+		if (!ignoreGates && persisted === null && (stateService !== void 0 || !this.statelessFirstRunDeferred)) {
+			if (stateService) await stateService.saveCuratorState({
 				schemaVersion: 1,
 				lastRunAt: Date.now(),
 				runCount: 0,
 				lastSummary: "first-run-deferred",
 				paused: false
 			});
+			else {
+				this.lastRun = Date.now();
+				this.statelessFirstRunDeferred = true;
+			}
 			return {
 				stale: [],
 				archived: [],
 				errors: [],
 				report: this.skippedReport(runId, startedAt),
-				skipped: "first-run-deferred"
+				skipped: stateService ? "first-run-deferred" : "first-run-deferred(stateless)"
 			};
 		}
 		const root = this.skills.root;
@@ -420,13 +446,15 @@ var EvolutionCurator = class extends Service {
 		}
 		const llmHint = !this.llmReview && result.markStale.length > 0 ? " (llmReview: off - deterministic archive only; set llmReview: true for the LLM merge channel)" : "";
 		const summary = `${dryRun ? "dry-run" : "auto"}: stale:${result.markStale.length} archived:${archivedSkills.length} consolidated:${gatedNominations.consolidations.length}${llmHint}`;
-		const pausedNow = (await stateService?.loadCuratorState())?.paused ?? false;
-		await stateService?.saveCuratorState({
-			schemaVersion: 1,
-			lastRunAt: dryRun ? persisted?.lastRunAt ?? this.lastRun : this.lastRun,
-			runCount: dryRun ? persisted?.runCount ?? 0 : (persisted?.runCount ?? 0) + 1,
-			lastSummary: summary,
-			paused: pausedNow
+		await stateService?.transactCuratorState((current) => {
+			const pausedNow = current?.paused ?? false;
+			return {
+				schemaVersion: 1,
+				lastRunAt: dryRun ? persisted?.lastRunAt ?? this.lastRun : Date.now(),
+				runCount: dryRun ? persisted?.runCount ?? 0 : (current?.runCount ?? 0) + 1,
+				lastSummary: summary,
+				paused: pausedNow
+			};
 		});
 		return {
 			stale: result.markStale,
@@ -511,6 +539,15 @@ var EvolutionCurator = class extends Service {
 		const suppressedAdded = /* @__PURE__ */ new Set();
 		const stateOwned = new Set(input.stateOwned ?? []);
 		for (const name of archiveCandidates) {
+			if (!treeNames.has(name)) {
+				const record = usage.get(name);
+				if (record) {
+					record.state = "archived";
+					record.archived_at = record.archived_at ?? (/* @__PURE__ */ new Date()).toISOString();
+					stateOwned.add(name);
+				}
+				continue;
+			}
 			const archived = await this.skills.archive(name, {
 				reason: "Lifecycle: reached archive threshold",
 				allowBundled: this.pruneBuiltins
@@ -603,7 +640,7 @@ var EvolutionCurator = class extends Service {
 	}
 	recentSessionActive() {
 		const agents = this.ctx.get("agents");
-		if (!agents) return false;
+		if (!agents) return !this.minIdleFailOpen;
 		let latest = 0;
 		for (const agent of agents.list()) {
 			const events = agent.session.events;
@@ -650,20 +687,15 @@ var EvolutionCurator = class extends Service {
 	}
 	async latestReport() {
 		const reportsRoot = join(evolutionHome(), "reports");
-		const names = (await this.io.list(reportsRoot)).filter((name) => name.startsWith("curator-") && name.endsWith(".json")).sort();
+		const names = (await this.io.list(reportsRoot)).filter((name) => name.startsWith("curator-") && name.endsWith(".json"));
 		let latest = null;
 		for (const name of names) {
-			const raw = await this.io.readText(join(reportsRoot, name));
-			if (raw === null) continue;
-			try {
-				const parsed = JSON.parse(raw);
-				const startedAt = typeof parsed.startedAt === "string" ? Date.parse(parsed.startedAt) : 0;
-				if (!Number.isFinite(startedAt)) continue;
-				if (latest === null || startedAt > latest.startedAt) latest = {
-					name,
-					startedAt
-				};
-			} catch {}
+			const mtime = await this.io.mtime?.(join(reportsRoot, name)) ?? null;
+			if (mtime === null) continue;
+			if (latest === null || mtime > latest.mtime) latest = {
+				name,
+				mtime
+			};
 		}
 		if (latest === null) return null;
 		const raw = await this.io.readText(join(reportsRoot, latest.name));

package/lib/types/index.d.ts

@@ -6,6 +6,7 @@ import { Context, Service } from '@deepseek-ai/cordis';
 import type Schema from '@deepseek-ai/schemastery';
 import { EvolutionGateSet, SkillLibrary } from '@deepseek-ai/dsh-evolution-core';
 import { type CuratorConsolidation, type CuratorNominations, type CuratorRunReport, type ScopeView, type SkillActionResult, type SkillHealthVerdict } from '@deepseek-ai/dsh-evolution-core';
+import type { CuratorStateRecord } from '@deepseek-ai/dsh-evolution-state';
 declare module '@deepseek-ai/cordis' {
     interface Context {
         evolutionCurator: EvolutionCurator;
@@ -23,6 +24,10 @@ export interface Config {
     qualityWarnStaleAfterDays?: number;
     /** Skip automatic runs while any session was active within this many hours (0 disables). */
     minIdleHours?: number;
+    /** When true (default), a missing `agents` service is treated as "no active
+     * session" (fail-open — the idle gate lets the run proceed); when false the
+     * gate fails closed and defers the run until activity can be measured. */
+    minIdleFailOpen?: boolean;
     /** Skill names excluded from the automated lifecycle. */
     excludeSkillNames?: string[];
     /** Include usage records whose created_by is not 'agent' in lifecycle decisions. */
@@ -54,14 +59,15 @@ 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 {
+/**
+ * Persisted curator-state record. The base fields are the authoritative
+ * `CuratorStateRecord` from evolution-state-storage; `schemaVersion` is an
+ * optional on-disk shape marker the storage seam leaves untyped (kept for
+ * legacy reads — 0.3.18, E-53).
+ */
+export type CuratorStateRecordShape = CuratorStateRecord & {
     schemaVersion?: number;
-    lastRunAt: number;
-    runCount: number;
-    lastSummary: string;
-    paused: boolean;
-}
+};
 /**
  * Block LLM-nominated consolidations that would touch a gate-protected name:
  * exclude / referenced / suppressed skills must never merge (neither as the
@@ -86,6 +92,7 @@ export declare class EvolutionCurator extends Service {
     private readonly curatorProvider;
     private readonly qualityWarnStaleAfterDays;
     private readonly minIdleHours;
+    private readonly minIdleFailOpen;
     private readonly excludeSkillNames;
     private readonly manageUnmanaged;
     private readonly pruneBuiltins;
@@ -99,6 +106,10 @@ export declare class EvolutionCurator extends Service {
     private timer;
     private bootCheck;
     private running;
+    /** 0.3.18 (E-18): stateless first-run defer fires ONCE per process — the
+     * in-memory clock is seeded and later due runs must proceed, or the
+     * persisted===null defer repeats forever (no state service to persist). */
+    private statelessFirstRunDeferred;
     constructor(ctx: Context, config?: Config);
     private lifecycle;
     start(): void;

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-curator",
   "description": "Deterministic skill lifecycle and recovery (community build)",
-  "version": "0.3.17",
+  "version": "0.3.19",
   "publishConfig": {
     "access": "public"
   },
@@ -33,20 +33,20 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.17"
+    "@lmzhen/dsh-evolution-core": "^0.3.19"
   },
   "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.17",
-    "@lmzhen/dsh-evolution-state": "^0.3.17"
+    "@lmzhen/dsh-evolution-io": "^0.3.19",
+    "@lmzhen/dsh-evolution-state": "^0.3.19"
   },
   "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.17",
-    "@lmzhen/dsh-evolution-io": "^0.3.17",
-    "@lmzhen/dsh-evolution-state": "^0.3.17"
+    "@lmzhen/dsh-evolution-core": "^0.3.19",
+    "@lmzhen/dsh-evolution-io": "^0.3.19",
+    "@lmzhen/dsh-evolution-state": "^0.3.19"
   }
 }