@lmzhen/dsh-evolution-curator 0.20.0 → 0.22.0

package/README.md

@@ -28,6 +28,7 @@ Deterministic skill lifecycle and recovery
 
 - `autoStart` (default true) arms an HOURLY tick that only asks whether the due-ness interval (`curatorIntervalHours`, default 168 h) has elapsed — the tick is not the interval — plus a deferred catch-up check `bootGraceSeconds` (default 10) after host boot. Both decide due-ness from the **persisted** `lastRunAt`, so a restart with an overdue schedule runs the first pass within the boot grace instead of waiting a full interval. `bootGraceSeconds: 0` disables the deferral (not recommended: the check may run against a half-mounted host). All scheduling gates — interval, idle, first-run deferral, and the reentrancy guard — remain inside `run()`.
 - `autoStart: false` disables both automatic checks; `/evolution curator run` (manual, gate-skipping) still works.
+- `autoStart` is the STARTUP default. The runtime switch is `/evolution curator pause` / `resume`: the pause is persisted in curation state, survives restarts, and wins over `autoStart` until it is cleared — turning `autoStart` off keeps curation manual, it does not freeze a schedule that is already paused or running.
 - Deprecated name (G0/S0.3): `intervalHours` is the legacy spelling of the policy
   row's `curatorIntervalHours`, whose value the snapshot shadows. Reading it still
   works; writing it is refused. Removal was planned for 0.7.0 and is deferred: the alias still ships, so no later version is claimed here.

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_SKILL_LIMITS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, INSTANCE_KEYS, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, SKILL_NAME_RE, buildCuratorRunReport, claimInstance, clampOnce, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, emptyRecord, entryTarget, evolutionHome, evolutionIoAdapter, foldCuratorFields, isPresent, isUnknown, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, paramRowId, parseCuratorNominations, parseFrontmatter, pickWithPolicy, policyStageLimits, probeList, probeText, readNumberParam, relatedSkillNames, releaseInstance, renderCuratorReportMarkdown, reportTime, reportsSweepLockTarget, sessionLastEventTime, sweepReports, transactIo, updateSuppressedNames, usageObserved, userSetKeys, verifyPromptBundle } 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_SKILL_LIMITS, DEFAULT_STALE_AFTER_DAYS, EvolutionGateSet, INSTANCE_KEYS, MAX_TIMER_DELAY_MS, PROMPT_BUNDLE, SKILL_NAME_RE, buildCuratorRunReport, claimInstance, clampOnce, clampedNumber, computeDedupGroups, computeLifecycleTransitions, computePrefixClusters, computeQualityScores, computeScopeView, contentHash, emptyRecord, entryTarget, evolutionHome, evolutionIoAdapter, foldCuratorFields, isPresent, isUnknown, liveBool, loadSuppressedNames, loadUsage, markerEntryName, mutateUsage, newSkillLibrary, paramRowId, parseCuratorNominations, parseFrontmatter, pickWithPolicy, policyStageLimits, probeList, probeText, readNumberParam, relatedSkillNames, releaseInstance, renderCuratorReportMarkdown, reportTime, reportsSweepLockTarget, sessionLastEventTime, sweepReports, transactIo, unwrapLive, updateSuppressedNames, usageObserved, userSetKeys, verifyPromptBundle } from "@lmzhen/dsh-evolution-core";
 //#region lib/types/index.js
 /**
 * Deterministic skill lifecycle curator with interval gate and archive.
@@ -49,7 +49,7 @@ var EvolutionCurator = class extends Service {
 		archiveAfterDays: z.number().min(1).default(DEFAULT_ARCHIVE_AFTER_DAYS).volatile(),
 		llmReview: z.boolean().default(false).volatile(),
 		curatorProvider: z.string().default("deepseek-official"),
-		qualityWarnStaleAfterDays: z.number().min(1).default(DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS).volatile(),
+		qualityWarnStaleAfterDays: z.number().min(1).default(DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS),
 		minIdleHours: z.number().min(0).default(DEFAULT_MIN_IDLE_HOURS).volatile(),
 		minIdleFailOpen: z.boolean().default(true).volatile(),
 		excludeSkillNames: z.array(z.string()).default([]),
@@ -174,9 +174,9 @@ var EvolutionCurator = class extends Service {
 	* @param key - the canonical registry id of the field.
 	* @returns the current plain value, or undefined when nothing supplied one.
 	*/
-	liveNumber(key) {
-		if (key === "curatorIntervalHours") return this.config.curatorIntervalHours?.get() ?? this.config.intervalHours;
-		return this.config[key]?.get();
+	rowValue(key) {
+		if (key === "curatorIntervalHours") return unwrapLive(this.config.curatorIntervalHours) ?? this.config.intervalHours;
+		return unwrapLive(this.config[key]);
 	}
 	/**
 	* One live numeric row field, clamped and warned once (G3.1).
@@ -187,7 +187,23 @@ var EvolutionCurator = class extends Service {
 	* @returns the value the engine may use.
 	*/
 	rowNumber(key, fallback, min, max) {
-		return clampOnce(this.clampedKeys, this.clampWarn, key, this.liveNumber(key), fallback, min, max);
+		return clampOnce(this.clampedKeys, this.clampWarn, key, this.rowValue(key), fallback, min, max);
+	}
+	/**
+	* One DEPLOYMENT numeric row field, clamped and warned once.
+	*
+	* The sibling of {@link rowNumber} for a field with no user layer (no `.volatile()`): it reads
+	* the config directly, because unwrapping a reference that never arrives would answer undefined
+	* and quietly fall back to the default. The clamp rule is the same one — a malformed value must
+	* not reach a comparison or a timer.
+	* @param key - the registry id of the field.
+	* @param fallback - the package default a corrected value falls back to.
+	* @param min - the smallest value the engine can act on.
+	* @param max - the largest value the carrier can hold, when bounded.
+	* @returns the value the engine may use.
+	*/
+	plainNumber(key, fallback, min, max) {
+		return clampOnce(this.clampedKeys, this.clampWarn, key, this.config[key], fallback, min, max);
 	}
 	/**
 	* One live boolean row field at USE time.
@@ -196,7 +212,7 @@ var EvolutionCurator = class extends Service {
 	* @returns the current value.
 	*/
 	rowBoolean(key, fallback) {
-		return this.config[key]?.get() ?? fallback;
+		return liveBool(this.config[key], fallback);
 	}
 	/**
 	* This row's resolved values — the base layer of the precedence chain (G1 §8.3).
@@ -209,7 +225,7 @@ var EvolutionCurator = class extends Service {
 			curatorIntervalHours: number("curatorIntervalHours", DEFAULT_CURATOR_INTERVAL_HOURS, 1),
 			staleAfterDays: number("staleAfterDays", DEFAULT_STALE_AFTER_DAYS, 1),
 			archiveAfterDays: number("archiveAfterDays", DEFAULT_ARCHIVE_AFTER_DAYS, 1),
-			qualityWarnStaleAfterDays: number("qualityWarnStaleAfterDays", DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS, 1),
+			qualityWarnStaleAfterDays: this.plainNumber("qualityWarnStaleAfterDays", DEFAULT_QUALITY_WARN_STALE_AFTER_DAYS, 1),
 			minIdleHours: number("minIdleHours", DEFAULT_MIN_IDLE_HOURS, 0),
 			minIdleFailOpen: this.rowBoolean("minIdleFailOpen", true),
 			llmReview: this.rowBoolean("llmReview", false),

package/lib/types/index.d.ts

@@ -42,7 +42,10 @@ export interface Config {
     llmReview?: Volatile<boolean>;
     curatorProvider?: string;
     /** Quality-warned skills may turn stale after this many idle days. */
-    qualityWarnStaleAfterDays?: Volatile<number>;
+    /** Deployment face since 0.21.0/G5: the quality-warning pair (this window and feedback's
+     * threshold) moved to the deployment rows together, so the field is a plain number — no
+     * `.volatile()`, no user layer, and the reader must not unwrap a reference that never arrives. */
+    qualityWarnStaleAfterDays?: number | undefined;
     /** Skip automatic runs while any session was active within this many hours (0 disables). */
     minIdleHours?: Volatile<number>;
     /** When true (default), a missing `agents` service is treated as "no active
@@ -151,7 +154,7 @@ export declare class EvolutionCurator extends Service {
         archiveAfterDays: z<number, number, "volatile-defined">;
         llmReview: z<boolean, boolean, "volatile-defined">;
         curatorProvider: z<string, string, "defined">;
-        qualityWarnStaleAfterDays: z<number, number, "volatile-defined">;
+        qualityWarnStaleAfterDays: z<number, number, "defined">;
         minIdleHours: z<number, number, "volatile-defined">;
         minIdleFailOpen: z<boolean, boolean, "volatile-defined">;
         excludeSkillNames: z<string[], string[], "defined">;
@@ -174,7 +177,7 @@ export declare class EvolutionCurator extends Service {
         archiveAfterDays: z<number, number, "volatile-defined">;
         llmReview: z<boolean, boolean, "volatile-defined">;
         curatorProvider: z<string, string, "defined">;
-        qualityWarnStaleAfterDays: z<number, number, "volatile-defined">;
+        qualityWarnStaleAfterDays: z<number, number, "defined">;
         minIdleHours: z<number, number, "volatile-defined">;
         minIdleFailOpen: z<boolean, boolean, "volatile-defined">;
         excludeSkillNames: z<string[], string[], "defined">;
@@ -241,7 +244,7 @@ export declare class EvolutionCurator extends Service {
      * @param key - the canonical registry id of the field.
      * @returns the current plain value, or undefined when nothing supplied one.
      */
-    private liveNumber;
+    private rowValue;
     /**
      * One live numeric row field, clamped and warned once (G3.1).
      * @param key - the canonical registry id of the field.
@@ -251,6 +254,20 @@ export declare class EvolutionCurator extends Service {
      * @returns the value the engine may use.
      */
     private rowNumber;
+    /**
+     * One DEPLOYMENT numeric row field, clamped and warned once.
+     *
+     * The sibling of {@link rowNumber} for a field with no user layer (no `.volatile()`): it reads
+     * the config directly, because unwrapping a reference that never arrives would answer undefined
+     * and quietly fall back to the default. The clamp rule is the same one — a malformed value must
+     * not reach a comparison or a timer.
+     * @param key - the registry id of the field.
+     * @param fallback - the package default a corrected value falls back to.
+     * @param min - the smallest value the engine can act on.
+     * @param max - the largest value the carrier can hold, when bounded.
+     * @returns the value the engine may use.
+     */
+    private plainNumber;
     /**
      * One live boolean row field at USE time.
      * @param key - the canonical registry id of the field.

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-curator",
   "description": "Deterministic skill lifecycle and recovery (community build)",
-  "version": "0.20.0",
+  "version": "0.22.0",
   "publishConfig": {
     "access": "public"
   },
@@ -27,20 +27,20 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.4",
-    "@lmzhen/dsh-evolution-core": "^0.20.0"
+    "@lmzhen/dsh-evolution-core": "^0.22.0"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-llm": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-session": "^0.2.0-rc.1",
-    "@lmzhen/dsh-evolution-io": "^0.20.0",
-    "@lmzhen/dsh-evolution-state": "^0.20.0"
+    "@lmzhen/dsh-evolution-io": "^0.22.0",
+    "@lmzhen/dsh-evolution-state": "^0.22.0"
   },
   "devDependencies": {
     "@deepseek-ai/dsh-llm": "^0.2.0-rc.1",
     "@deepseek-ai/dsh-session": "^0.2.0-rc.1",
-    "@lmzhen/dsh-evolution-core": "^0.20.0",
-    "@lmzhen/dsh-evolution-io": "^0.20.0",
-    "@lmzhen/dsh-evolution-state": "^0.20.0"
+    "@lmzhen/dsh-evolution-core": "^0.22.0",
+    "@lmzhen/dsh-evolution-io": "^0.22.0",
+    "@lmzhen/dsh-evolution-state": "^0.22.0"
   }
 }