@lmzhen/dsh-evolution-curator 0.3.77 → 0.3.78

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_BOOT_GRACE_SECONDS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_CURATOR_MODEL, DEFAULT_CURATOR_REVIEW_MAX_TOKENS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, MAX_TIMER_DELAY_MS, SKILL_NAME_RE, buildCuratorRunReport, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, emptyRecord, evolutionHome, evolutionIoAdapter, foldCuratorFields, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, parseCuratorNominations, parseFrontmatter, relatedSkillNames, renderCuratorReportMarkdown, updateSuppressedNames, usageObserved } from "@lmzhen/dsh-evolution-core";
+import { CURATOR_DRY_RUN_BANNER, CURATOR_PROMPT, DEFAULT_ARCHIVE_AFTER_DAYS, DEFAULT_CURATOR_BOOT_GRACE_SECONDS, DEFAULT_CURATOR_INTERVAL_HOURS, DEFAULT_CURATOR_MODEL, DEFAULT_CURATOR_REVIEW_MAX_TOKENS, DEFAULT_HEALTH_THRESHOLDS, DEFAULT_MIN_IDLE_HOURS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, INSTANCE_KEYS, MAX_TIMER_DELAY_MS, SKILL_NAME_RE, buildCuratorRunReport, claimInstance, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, emptyRecord, evolutionHome, evolutionIoAdapter, foldCuratorFields, isPresent, isUnknown, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, parseCuratorNominations, parseFrontmatter, probeList, probeMtime, relatedSkillNames, releaseInstance, renderCuratorReportMarkdown, updateSuppressedNames, usageObserved } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic skill lifecycle curator with interval gate and archive.
@@ -92,6 +92,13 @@ var EvolutionCurator = class extends Service {
 	statelessFirstRunDeferred = false;
 	/** P2-5 (v14): one-shot warning that the interval baseline is process-only. */
 	statelessStateWarned = false;
+	/** B3 / G4: the home this instance claimed, plus whether the claim was
+	* granted. A non-owning instance schedules nothing and runs nothing. */
+	instanceHome = "";
+	holdsInstance = false;
+	/** B3 / G4: this instance's identity in the claim — pid + short token, so the
+	* yielding instance's log line names a concrete holder. */
+	instanceOwner = `evolution-curator[${process.pid}:${randomUUID().slice(0, 8)}]`;
 	constructor(ctx, config = {}) {
 		super(ctx, "evolutionCurator");
 		this.io = evolutionIoAdapter(() => ctx.evolutionIo.provider());
@@ -101,6 +108,10 @@ var EvolutionCurator = class extends Service {
 			ctx: this.ctx
 		});
 		this.enabled = config.enabled ?? true;
+		this.instanceHome = evolutionHome();
+		const claim = claimInstance(this.instanceHome, INSTANCE_KEYS.curator, this.instanceOwner);
+		this.holdsInstance = claim.granted;
+		if (!claim.granted) this.ctx.logger.warn(`evolution-curator: this instance YIELDS — ${claim.key} is already held by ${claim.holder}; a second curator over one home would race the report retention sweep. It schedules nothing and every run() returns skipped "instance-held".`);
 		const clamped = [];
 		const field = (name, value, fallback, min, max) => {
 			const result = clampedNumber(value, fallback, max === void 0 ? { min } : {
@@ -138,6 +149,7 @@ var EvolutionCurator = class extends Service {
 			return () => {
 				this.disposed = true;
 				this.stop();
+				releaseInstance(this.instanceHome, INSTANCE_KEYS.curator, this.instanceOwner);
 			};
 		}, "evolution-curator.stop");
 		if (config.autoStart ?? true) this.start();
@@ -151,6 +163,7 @@ var EvolutionCurator = class extends Service {
 		};
 	}
 	start() {
+		if (!this.holdsInstance) return;
 		if (this.isDisposed() || !this.enabled || this.timer) return;
 		this.bootCheck = setTimeout(() => {
 			this.bootCheck = void 0;
@@ -372,6 +385,13 @@ var EvolutionCurator = class extends Service {
 	* skipped with an explicit `already-running` outcome.
 	*/
 	async run(options = {}) {
+		if (!this.holdsInstance) return {
+			stale: [],
+			archived: [],
+			errors: [],
+			report: this.skippedReport("instance-held", (/* @__PURE__ */ new Date()).toISOString()),
+			skipped: "instance-held"
+		};
 		if (this.mutexDepth > 0) return {
 			stale: [],
 			archived: [],
@@ -905,12 +925,12 @@ var EvolutionCurator = class extends Service {
 	*/
 	async retainReports(keep = 20, errorKeep = 10) {
 		const reportsRoot = join(evolutionHome(), "reports");
-		let entries;
-		try {
-			entries = await this.io.list(reportsRoot);
-		} catch {
+		const listed = await probeList(this.io, reportsRoot);
+		if (isUnknown(listed)) {
+			this.ctx.logger.warn(`evolution-curator: report retention skipped — the reports directory could not be listed (${listed.reason})`);
 			return;
 		}
+		const entries = isPresent(listed) ? listed.value : [];
 		const real = [];
 		const errors = [];
 		let unorderableWarned = false;
@@ -920,8 +940,8 @@ var EvolutionCurator = class extends Service {
 			const parsed = JSON.parse(raw);
 			let startedAt = typeof parsed.startedAt === "string" ? Date.parse(parsed.startedAt) : NaN;
 			if (!Number.isFinite(startedAt)) {
-				const mtime = await this.io.mtime?.(join(reportsRoot, name)) ?? null;
-				if (mtime !== null) startedAt = mtime;
+				const probe = await probeMtime(this.io, join(reportsRoot, name));
+				if (isPresent(probe)) startedAt = probe.value;
 			}
 			if (!Number.isFinite(startedAt) && typeof parsed.at === "string") startedAt = Date.parse(parsed.at);
 			if (Number.isFinite(startedAt)) (name.startsWith("curator-error-") ? errors : real).push({
@@ -952,27 +972,25 @@ var EvolutionCurator = class extends Service {
 	}
 	async latestReport() {
 		const reportsRoot = join(evolutionHome(), "reports");
-		let names;
-		try {
-			names = (await this.io.list(reportsRoot)).filter((name) => name.startsWith("curator-") && name.endsWith(".json") && !name.startsWith("curator-error-"));
-		} catch (error) {
-			this.ctx.logger.warn(`evolution-curator: latestReport could not list the reports directory (${error instanceof Error ? error.message : String(error)}) - answering "no report"`);
+		const listed = await probeList(this.io, reportsRoot);
+		if (isUnknown(listed)) {
+			this.ctx.logger.warn(`evolution-curator: latestReport could not list the reports directory (${listed.reason}) - answering "no report"`);
 			return null;
 		}
+		const names = (isPresent(listed) ? listed.value : []).filter((name) => name.startsWith("curator-") && name.endsWith(".json") && !name.startsWith("curator-error-"));
 		let latest = null;
 		let probeWarned = false;
 		for (const name of names) {
-			let mtime;
-			try {
-				mtime = await this.io.mtime?.(join(reportsRoot, name)) ?? null;
-			} catch (error) {
+			const probe = await probeMtime(this.io, join(reportsRoot, name));
+			if (isUnknown(probe)) {
 				if (!probeWarned) {
 					probeWarned = true;
-					this.ctx.logger.warn(`evolution-curator: latestReport could not read a report mtime (${error instanceof Error ? error.message : String(error)}) - the affected report(s) are not eligible as the latest`);
+					this.ctx.logger.warn(`evolution-curator: latestReport could not read a report mtime (${probe.reason}) - the affected report(s) are not eligible as the latest`);
 				}
 				continue;
 			}
-			if (mtime === null) continue;
+			if (!isPresent(probe)) continue;
+			const mtime = probe.value;
 			if (latest === null || mtime > latest.mtime) latest = {
 				name,
 				mtime

package/lib/types/index.d.ts

@@ -127,6 +127,13 @@ export declare class EvolutionCurator extends Service {
     private statelessFirstRunDeferred;
     /** P2-5 (v14): one-shot warning that the interval baseline is process-only. */
     private statelessStateWarned;
+    /** B3 / G4: the home this instance claimed, plus whether the claim was
+     * granted. A non-owning instance schedules nothing and runs nothing. */
+    private instanceHome;
+    private holdsInstance;
+    /** B3 / G4: this instance's identity in the claim — pid + short token, so the
+     * yielding instance's log line names a concrete holder. */
+    private readonly instanceOwner;
     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.77",
+  "version": "0.3.78",
   "publishConfig": {
     "access": "public"
   },
@@ -27,20 +27,20 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.77"
+    "@lmzhen/dsh-evolution-core": "^0.3.78"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-llm": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-io": "^0.3.77",
-    "@lmzhen/dsh-evolution-state": "^0.3.77"
+    "@lmzhen/dsh-evolution-io": "^0.3.78",
+    "@lmzhen/dsh-evolution-state": "^0.3.78"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-llm": "^0.1.5-rc.2",
     "@deepseek-ai/dsh-session": "^0.1.5-rc.2",
-    "@lmzhen/dsh-evolution-core": "^0.3.77",
-    "@lmzhen/dsh-evolution-io": "^0.3.77",
-    "@lmzhen/dsh-evolution-state": "^0.3.77"
+    "@lmzhen/dsh-evolution-core": "^0.3.78",
+    "@lmzhen/dsh-evolution-io": "^0.3.78",
+    "@lmzhen/dsh-evolution-state": "^0.3.78"
   }
 }