@lmzhen/dsh-evolution-commands 0.3.56 → 0.3.58

package/README.md

@@ -12,7 +12,9 @@ Human commands for the evolution family
 
 #### Token effect
 
-Zero direct token effect from this package; consumers add any model-visible tokens.
+P2-22 (v11) correction: this package's **only** direct model-visible token is the
+`/evolution learn` injection — the full learning guidance is injected as a user
+message in this session. Everything else adds no tokens; consumers add their own.
 
 #### KV Cache effect
 

package/lib/index.js

@@ -1,4 +1,5 @@
 import { createRequire } from "node:module";
+import z from "@deepseek-ai/schemastery";
 import { createUserMessage } from "@deepseek-ai/dsh-llm";
 import { SkillLibrary, appendEvolutionEvent, buildLearnPrompt, clampedNumber, composePresetComposition, eventsFile, evolutionRoot, resolveSkillsRoot } from "@lmzhen/dsh-evolution-core";
 import { buildEnrichment, buildMaintainFacts, runMaintain, snapshotFromLibrary } from "@lmzhen/dsh-evolution-maintenance";
@@ -60,7 +61,7 @@ async function diagnose(ctx, options = {}) {
 	if (host && preset) conflicts.push("evolution-host and evolution-preset are installed together — the infra rows double-mount. Keep ONE.");
 	if (full && layered) conflicts.push("evolution-all and the layered Evolution preset are both present — the model rows double-mount. Keep ONE (use layered without all, or drop the preset).");
 	const services = {
-		review: has("evolutionReview"),
+		review: full || host || preset,
 		curator: has("evolutionCurator"),
 		approval: has("evolutionApproval"),
 		skillUsage: has("skillUsage"),
@@ -189,6 +190,12 @@ function renderHelpText() {
 */
 /** Enrichment maps shared by the full scan and the `--facts` preview (v12). */
 const name = "evolution-commands";
+const Config = z.object({
+	skillsRoot: z.string().default(""),
+	maintainCooldownMs: z.number().min(0).default(3e4),
+	maintainTimeoutMs: z.number().min(1).default(6e5),
+	threatExemptLabels: z.array(z.string()).default([])
+});
 function apply(ctx, rawConfig = {}) {
 	const config = rawConfig;
 	let lastMaintainAt = 0;
@@ -306,7 +313,7 @@ function apply(ctx, rawConfig = {}) {
 						`failed=${reportAny.failed.map((item) => `${nameOf(item)}: ${reasonOf(item)}`).join(", ") || "(none)"}`
 					].join("\n"));
 				}
-				if (input.startsWith("restore ")) {
+				if (input === "restore" || input.startsWith("restore ")) {
 					const curator = ctx.get("evolutionCurator");
 					const result = curator ? await curator.restoreSnapshot() : {
 						ok: false,
@@ -391,7 +398,7 @@ function apply(ctx, rawConfig = {}) {
 				const maintainArgs = /^maintain(?:\s+--timeout[ =](\d+))?\s*$/.exec(input);
 				if (maintainArgs) {
 					const runTimeoutMs = maintainArgs[1] ? Number(maintainArgs[1]) : config.maintainTimeoutMs ?? 6e5;
-					if (!Number.isSafeInteger(runTimeoutMs) || runTimeoutMs <= 0 || runTimeoutMs > 4294967295) return err("Invalid --timeout value: expected a positive integer number of milliseconds up to 4294967295 (e.g. /evolution maintain --timeout 600000).");
+					if (!Number.isSafeInteger(runTimeoutMs) || runTimeoutMs <= 0 || runTimeoutMs > 4294967295) return err(maintainArgs[1] ? "Invalid --timeout value: expected a positive integer number of milliseconds up to 4294967295 (e.g. /evolution maintain --timeout 600000)." : "The maintainTimeoutMs config is invalid: expected a positive integer number of milliseconds up to 4294967295. Fix the evolution-commands row config (maintainTimeoutMs), then retry.");
 					if (maintainInFlightSince > 0) return err(`Maintenance scan is already running (since ~${Math.max(1, Math.round((Date.now() - maintainInFlightSince) / 1e3))}s ago) — re-submitting now would cancel it. Wait for it to settle; the result appears when it finishes.`);
 					const cooldownMs = clampedNumber(config.maintainCooldownMs, 3e4, { min: 0 });
 					const sinceLast = Date.now() - lastMaintainAt;
@@ -422,7 +429,7 @@ function apply(ctx, rawConfig = {}) {
 							catalogInvalid: () => enrichment.catalogInvalid,
 							usageObserved: () => enrichment.usageObservedValue
 						});
-						lastMaintainRunId = outcome.runId ?? "";
+						if (outcome.ok) lastMaintainRunId = outcome.runId ?? "";
 						if (!outcome.ok) return err(outcome.error ?? "Maintenance scan failed.");
 						appendEvolutionEvent(ioRegistry.provider(), eventsFile(evolutionRoot()), {
 							type: "maintain",
@@ -459,7 +466,9 @@ function apply(ctx, rawConfig = {}) {
 						}, {
 							name: "preset.yml",
 							content: readFileSync(presetPath)
-						}]);
+						}], void 0, (message) => {
+							ctx.logger.warn(message);
+						});
 						return ok(`Evolution agent preset installed to ${target} (runtime standard + delta). Restart the session switcher to select it.`);
 					} catch (error) {
 						return err(`Preset install failed: ${error instanceof Error ? error.message : String(error)}`);
@@ -565,7 +574,7 @@ const defaultFs = {
 * source (and mtime when available) so the caller knows the generation it
 * restored from.
 */
-function atomicWriteFiles(targetDir, writes, fs = defaultFs) {
+function atomicWriteFiles(targetDir, writes, fs = defaultFs, logger = console.warn) {
 	const stage = (name) => join(targetDir, `${name}.tmp`);
 	const dupes = [...new Set(writes.map((write) => write.name).filter((name, index, all) => all.indexOf(name) !== index))];
 	if (dupes.length > 0) throw new Error(`atomicWriteFiles: duplicate input names: ${dupes.join(", ")}`);
@@ -615,7 +624,7 @@ function atomicWriteFiles(targetDir, writes, fs = defaultFs) {
 			try {
 				if (fs.existsSync(finalPath)) fs.copyFileSync(finalPath, join(targetDir, `${name}.bak`));
 			} catch (refreshError) {
-				console.warn(`atomicWriteFiles: committed "${name}" but failed to refresh its .bak (${refreshError instanceof Error ? refreshError.message : String(refreshError)}); a later failed commit will recover to an OLDER generation`);
+				logger(`atomicWriteFiles: committed "${name}" but failed to refresh its .bak (${refreshError instanceof Error ? refreshError.message : String(refreshError)}); a later failed commit will recover to an OLDER generation`);
 			}
 		}
 	} catch (error) {
@@ -642,4 +651,4 @@ function safeStagedArgs(args) {
 	}
 }
 //#endregion
-export { apply, atomicWriteFiles, name };
+export { Config, apply, atomicWriteFiles, name };

package/lib/types/index.d.ts

@@ -3,6 +3,7 @@
  * @module @lmzhen/dsh-evolution-commands
  */
 import type { Context } from '@deepseek-ai/cordis';
+import z from '@deepseek-ai/schemastery';
 export declare const name = "evolution-commands";
 export interface Config {
     /** Skill-tree root for maintain/restructure; empty uses skillsRoot().
@@ -32,6 +33,17 @@ export interface Config {
      * widens them. */
     threatExemptLabels?: string[] | undefined;
 }
+export declare const Config: z<Schemastery.ObjectS<{
+    skillsRoot: z<string, string>;
+    maintainCooldownMs: z<number, number>;
+    maintainTimeoutMs: z<number, number>;
+    threatExemptLabels: z<string[], string[]>;
+}>, Schemastery.ObjectT<{
+    skillsRoot: z<string, string>;
+    maintainCooldownMs: z<number, number>;
+    maintainTimeoutMs: z<number, number>;
+    threatExemptLabels: z<string[], string[]>;
+}>>;
 export declare function apply(ctx: Context, rawConfig?: Config): void;
 /** F-211: the fs surface atomicWriteFiles uses — injectable so the commit-phase
  * rename-failure path is deterministically testable. Defaults to node:fs.
@@ -73,6 +85,6 @@ interface FsOps {
 export declare function atomicWriteFiles(targetDir: string, writes: Array<{
     name: string;
     content: string | Uint8Array;
-}>, fs?: FsOps): void;
+}>, fs?: FsOps, logger?: (message: string) => void): void;
 export {};
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-commands",
   "description": "Human commands for the evolution family (/evolution pending|curator|maintain|doctor) (community build)",
-  "version": "0.3.56",
+  "version": "0.3.58",
   "publishConfig": {
     "access": "public"
   },
@@ -30,12 +30,13 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-approval": "^0.3.56",
-    "@lmzhen/dsh-evolution-core": "^0.3.56",
-    "@lmzhen/dsh-evolution-maintenance": "^0.3.56"
+    "@deepseek-ai/schemastery": "^3.18.1",
+    "@lmzhen/dsh-evolution-approval": "^0.3.58",
+    "@lmzhen/dsh-evolution-core": "^0.3.58",
+    "@lmzhen/dsh-evolution-maintenance": "^0.3.58"
   },
   "optionalDependencies": {
-    "@lmzhen/dsh-evolution-agent-preset": "^0.3.56"
+    "@lmzhen/dsh-evolution-agent-preset": "^0.3.58"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",