@lmzhen/dsh-evolution-curator 0.3.83 → 0.4.1

package/README.md

@@ -2,31 +2,18 @@
 
 Deterministic skill lifecycle and recovery
 
-## Model Experience
+## Model surface
 
-### Indirect model surface
-
-#### What the model sees
-
-`@lmzhen/dsh-evolution-curator` registers no direct prompt or tool schema itself. Model-visible effects are owned by the packages that consume this service.
-
-#### Token effect
-
-Zero direct token effect from this package; consumers add any model-visible tokens.
-
-#### KV Cache effect
-
-Independent of request-prefix construction. This package does not alter the assembled prompt or tool list.
-
-## Known Limitations and Deferred Work
+- **Model-visible:** nothing of its own: it registers no prompt section and no tool schema; the packages that consume this service own the model-visible effects.
+- **Prompt prefix / KV cache:** independent of request-prefix construction — it does not alter the assembled prompt or tool list; family-level rules: `packages/README.md` §"Model-visible prompt prefix and the KV cache".
+- **Mount it?** yes — the `evolution-curator` row, carried by the `evolution-host`, `evolution-all` and one-click `evolution-preset` bundles; it provides the `evolutionCurator` service the `/evolution curator …` commands and doctor read.
 
+## Known limitations
 
 - LLM nomination pass is advisory and disabled by default; deterministic lifecycle remains authoritative.
 - 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).
+ merge groups, the control plane gates them and applies the mutation — merged source bodies are appended
+ verbatim rather than rewritten into a synthesized skill.
 
 ## Recovery and consolidation
 
@@ -35,9 +22,12 @@ Independent of request-prefix construction. This package does not alter the asse
 - `consolidate(target, sources)` (service) / `/evolution consolidate` merges source bodies into the target, archives the sources with an absorbed-into marker, and folds their usage records into `archived` state. Both operations snapshot the full state first (`pre-consolidate` / `pre-restore`).
 - `restoreSnapshot` (service) / `/evolution restore` rolls the state back to the latest snapshot: active tree, usage/suppression sidecars, `.archive/` and the curator state carried in the snapshot (`curator-state.json`), so the interval gate does not immediately re-fire after a rollback. Skills that were skipped at snapshot time (a live writer held their lock; recorded in the manifest's `skipped` list) are NOT restored — the restore result names them, and `.backups` may hold a copy. The restore itself is undoable — the pre-rollback safety snapshot preserves the current tree plus its state.
 
-## Automatic scheduling
+## Configuration
 
-- `autoStart` (default true) arms an hourly interval check 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` (default true) arms an HOURLY tick that only asks whether the due-ness interval (`intervalHours`, 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.
 
 **Runtime invariant:** No companion is published. The platform auto-assembles nothing and the family mounts no `<pkg>/invariant` cordis row, so a companion here would never execute (v37 S2.1 / I-3).
+## Notes and history
+
+- 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).

package/lib/index.js

@@ -161,10 +161,16 @@ var EvolutionCurator = class extends Service {
 	}
 	lifecycle() {
 		const snapshot = this.ctx.get("evolutionPolicy")?.get();
+		const staleAfterDays = snapshot?.staleAfterDays ?? this.staleAfterDays;
+		let archiveAfterDays = snapshot?.archiveAfterDays ?? this.archiveAfterDays;
+		if (archiveAfterDays < staleAfterDays) {
+			this.ctx.logger.warn(`evolution-curator: policy archiveAfterDays (${archiveAfterDays}) < policy staleAfterDays (${staleAfterDays}); using the stale threshold as the archive threshold`);
+			archiveAfterDays = staleAfterDays;
+		}
 		return {
 			intervalHours: snapshot?.curatorIntervalHours ?? this.intervalHours,
-			staleAfterDays: snapshot?.staleAfterDays ?? this.staleAfterDays,
-			archiveAfterDays: snapshot?.archiveAfterDays ?? this.archiveAfterDays
+			staleAfterDays,
+			archiveAfterDays
 		};
 	}
 	start() {
@@ -691,8 +697,8 @@ var EvolutionCurator = class extends Service {
 				const pausedNow = current?.paused ?? false;
 				return {
 					schemaVersion: 1,
-					lastRunAt: dryRun || runAborted ? persisted?.lastRunAt ?? this.lastRun : Date.now(),
-					runCount: dryRun || runAborted ? persisted?.runCount ?? 0 : (current?.runCount ?? 0) + 1,
+					lastRunAt: dryRun || runAborted ? current?.lastRunAt ?? persisted?.lastRunAt ?? this.lastRun : Date.now(),
+					runCount: dryRun || runAborted ? current?.runCount ?? persisted?.runCount ?? 0 : (current?.runCount ?? 0) + 1,
 					lastSummary: summary,
 					paused: pausedNow
 				};

package/package.json

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