@lmzhen/dsh-evolution-curator 0.3.27 → 0.3.28

package/lib/index.js

@@ -185,6 +185,7 @@ var EvolutionCurator = class extends Service {
 			} catch (persistenceError) {
 				this.ctx.logger.warn(`evolution-curator: failed to persist auto-check error report: ${persistenceError instanceof Error ? persistenceError.message : String(persistenceError)}`);
 			}
+			await this.retainReports();
 		}
 	}
 	/**
@@ -319,6 +320,11 @@ var EvolutionCurator = class extends Service {
 			return await this.runCore(options);
 		} finally {
 			this.running = false;
+			try {
+				await this.retainReports();
+			} catch (error) {
+				this.ctx.logger.warn(`evolution-curator: failed to retain reports: ${error instanceof Error ? error.message : String(error)}`);
+			}
 		}
 	}
 	async runCore(options = {}) {
@@ -447,7 +453,6 @@ var EvolutionCurator = class extends Service {
 		try {
 			await this.io.writeText(join(reportsRoot, `curator-${runId}.json`), JSON.stringify(report, null, 2));
 			await this.io.writeText(join(reportsRoot, `curator-${runId}.md`), renderCuratorReportMarkdown(report));
-			await this.retainReports(20);
 		} catch (error) {
 			this.ctx.logger.warn(`evolution-curator: failed to persist report ${runId}`);
 			this.ctx.logger.warn(error);
@@ -668,7 +673,19 @@ var EvolutionCurator = class extends Service {
 	* like `retainSnapshots`: a failed removal must not fail the run that just
 	* persisted its report. The paired `.md` digest is pruned with its JSON.
 	*/
-	async retainReports(keep) {
+	/**
+	* Keep only the newest N curator run reports plus at most `errorKeep` error
+	* reports, ordered by the report's own `startedAt` (the runId is a UUID and
+	* cannot order history). Best-effort like `retainSnapshots`: a failed removal
+	* must not fail the run that just persisted its report. The paired `.md`
+	* digest is pruned with its JSON.
+	*
+	* V4-22: real reports (`curator-*.json`) and error reports
+	* (`curator-error-*.json`) are BUDGETED INDEPENDENTLY. Before this they shared
+	* one keep-20 window, so after 25 consecutive failures the next successful
+	* run's window held only a few real reports alongside the errors.
+	*/
+	async retainReports(keep = 20, errorKeep = 10) {
 		const reportsRoot = join(evolutionHome(), "reports");
 		let entries;
 		try {
@@ -676,7 +693,8 @@ var EvolutionCurator = class extends Service {
 		} catch {
 			return;
 		}
-		const dated = [];
+		const real = [];
+		const errors = [];
 		for (const name of entries.filter((entry) => entry.startsWith("curator-") && entry.endsWith(".json"))) try {
 			const raw = await this.io.readText(join(reportsRoot, name));
 			if (raw === null) continue;
@@ -686,12 +704,18 @@ var EvolutionCurator = class extends Service {
 				const mtime = await this.io.mtime?.(join(reportsRoot, name)) ?? null;
 				if (mtime !== null) startedAt = mtime;
 			}
-			if (Number.isFinite(startedAt)) dated.push({
+			if (Number.isFinite(startedAt)) (name.startsWith("curator-error-") ? errors : real).push({
 				name,
 				startedAt
 			});
 		} catch {}
-		dated.sort((a, b) => b.startedAt - a.startedAt);
+		real.sort((a, b) => b.startedAt - a.startedAt);
+		errors.sort((a, b) => b.startedAt - a.startedAt);
+		await this.pruneReportList(reportsRoot, real, keep);
+		await this.pruneReportList(reportsRoot, errors, errorKeep);
+	}
+	/** Remove the oldest reports beyond `keep` (each with its `.md` digest, best-effort). */
+	async pruneReportList(reportsRoot, dated, keep) {
 		for (const oldReport of dated.slice(keep)) {
 			const stem = oldReport.name.replace(/\.json$/, "");
 			try {

package/lib/types/index.d.ts

@@ -212,7 +212,21 @@ export declare class EvolutionCurator extends Service {
      * like `retainSnapshots`: a failed removal must not fail the run that just
      * persisted its report. The paired `.md` digest is pruned with its JSON.
      */
+    /**
+     * Keep only the newest N curator run reports plus at most `errorKeep` error
+     * reports, ordered by the report's own `startedAt` (the runId is a UUID and
+     * cannot order history). Best-effort like `retainSnapshots`: a failed removal
+     * must not fail the run that just persisted its report. The paired `.md`
+     * digest is pruned with its JSON.
+     *
+     * V4-22: real reports (`curator-*.json`) and error reports
+     * (`curator-error-*.json`) are BUDGETED INDEPENDENTLY. Before this they shared
+     * one keep-20 window, so after 25 consecutive failures the next successful
+     * run's window held only a few real reports alongside the errors.
+     */
     private retainReports;
+    /** Remove the oldest reports beyond `keep` (each with its `.md` digest, best-effort). */
+    private pruneReportList;
     latestReport(): Promise<CuratorRunReport | null>;
     /**
      * Read-only lifecycle scope classification: which skills are in scope,

package/package.json

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