@lmzhen/dsh-evolution-commands 0.3.16 → 0.3.18

package/lib/index.js

@@ -2,13 +2,13 @@ import { createRequire } from "node:module";
 import { createUserMessage } from "@deepseek-ai/dsh-llm";
 import { SkillLibrary, appendEvolutionEvent, buildLearnPrompt, composePresetComposition, eventsFile } from "@lmzhen/dsh-evolution-core";
 import { buildEnrichment, buildMaintainFacts, runMaintain, snapshotFromLibrary } from "@lmzhen/dsh-evolution-maintenance";
-import { copyFileSync, existsSync, mkdirSync, readFileSync, writeFileSync } from "node:fs";
+import { copyFileSync, existsSync, mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";
 //#region lib/types/index.js
 /**
-* Human commands for the evolution family: /evolution learn|pending|curator|restore|consolidate.
+* Human commands for the evolution family: /evolution learn|pending|curator|restore|consolidate|skills refresh.
 * @module @lmzhen/dsh-evolution-commands
 */
 /** Enrichment maps shared by the full scan and the `--facts` preview (v12). */
@@ -19,11 +19,12 @@ function apply(ctx, rawConfig = {}) {
 	let lastMaintainRunId = "";
 	let maintainInFlightSince = 0;
 	ctx.inject(["commands"], (commandCtx) => {
-		commandCtx.commands.register({
+		const commands = commandCtx.commands;
+		commandCtx.effect(() => commands.register({
 			name: "evolution",
 			description: "Self-evolution status and approval controls",
 			recordInput: false,
-			input: { hint: "pending | approve <id> | reject <id> | curator run|pause|resume|status|report|scope | restore | consolidate <target> <sources...> | skill restore <name> | skills health | learn [request] | maintain [--timeout ms] | preset install | restructure <name> \"<heading>\" <to_file> | replay" },
+			input: { hint: "pending | approve <id> | reject <id> | curator run|pause|resume|status|report|scope | mutations | restore | consolidate <target> <sources...> | skill restore <name> | skills health | skills refresh | learn [request] | maintain [--timeout ms | --facts] | preset install | restructure <name> \"<heading>\" <to_file> | replay" },
 			async handler(invocation) {
 				const input = invocation.rawInput?.trim() ?? "";
 				const ok = (text) => ({
@@ -36,8 +37,9 @@ function apply(ctx, rawConfig = {}) {
 				});
 				const approval = ctx.get("evolutionApproval");
 				if (input === "pending") {
-					const pending = approval ? await approval.list("pending") : [];
-					return ok(pending.length === 0 ? "No pending evolution writes." : pending.map((p) => `${p.id}  ${p.kind}  ${p.summary}`).join("\n"));
+					const pending = approval ? [...await approval.list("pending"), ...await approval.list("executing")] : [];
+					if (pending.length === 0) return ok("No pending evolution writes.");
+					return ok(pending.map((p) => `${p.id}  ${p.kind}  ${p.status === "executing" ? "EXECUTING " : ""}${p.summary}`).join("\n") + (pending.some((p) => p.status === "executing") ? "\n(EXECUTING: a previous approve may have crashed after running; verify the write manually, then reject — approve will not re-run it)" : ""));
 				}
 				if (input.startsWith("approve ")) {
 					const id = input.slice(8).trim();
@@ -155,6 +157,10 @@ function apply(ctx, rawConfig = {}) {
 					if (rows.length === 0) return ok(banner ? `Structure health: all skills healthy. ${banner}` : "Structure health: all skills healthy.");
 					return ok([banner ? `Structure health (${rows.length} degraded): ${banner}` : `Structure health (${rows.length} degraded):`, ...rows.map(line)].join("\n"));
 				}
+				if (input === "skills refresh") {
+					ctx.emit("evolution/skills-refresh");
+					return ok("Skill catalog refresh requested: caches dropped, catalog re-read on next lookup.");
+				}
 				if (input === "learn" || input.startsWith("learn ")) {
 					const request = input === "learn" ? "" : input.slice(6).trim();
 					invocation.agent.inject(createUserMessage({
@@ -212,7 +218,8 @@ function apply(ctx, rawConfig = {}) {
 						const outcome = await runMaintain({
 							library,
 							subagents,
-							parent: invocation.agent
+							parent: invocation.agent,
+							evolutionPolicy: { get: () => ctx.get("evolutionPolicy")?.get() }
 						}, {
 							timeoutMs: runTimeoutMs,
 							descriptions: () => enrichment.descriptions,
@@ -229,7 +236,7 @@ function apply(ctx, rawConfig = {}) {
 							source: "manual",
 							runId: outcome.runId,
 							verdict: outcome.verdict,
-							recommendations: outcome.text?.split("\n").filter((line) => line.startsWith("- [")).length ?? 0
+							recommendations: countMaintainRecommendations(outcome.text)
 						}).catch((error) => {
 							ctx.logger.warn(`evolution-commands: failed to record maintain event: ${String(error)}`);
 						});
@@ -253,8 +260,13 @@ function apply(ctx, rawConfig = {}) {
 						if (!registry) return err("Agent preset registry not mounted — cannot compose the Evolution preset against the runtime standard.");
 						const composition = composePresetComposition(await registry.read("standard"), readFileSync(deltaPath, "utf8"));
 						mkdirSync(target, { recursive: true });
-						writeFileSync(join(target, "agent.cordis.yml"), composition);
-						copyFileSync(presetPath, join(target, "preset.yml"));
+						atomicWriteFiles(target, [{
+							name: "agent.cordis.yml",
+							content: composition
+						}, {
+							name: "preset.yml",
+							content: readFileSync(presetPath)
+						}]);
 						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)}`);
@@ -284,9 +296,9 @@ function apply(ctx, rawConfig = {}) {
 					if (!replay) return err("Replay service not mounted.");
 					return ok(replay.compare().report);
 				}
-				return ok("Evolution: memory, skills, review, curator. Use /evolution pending | approve <id> | reject <id> | curator run | curator status | curator pause | curator resume | curator report | curator scope | restore | consolidate <target> <source...> | skill restore <name> | skills health | learn [request] | maintain | preset install | restructure <name> \"<heading>\" <to_file> | replay.");
+				return ok("Evolution: memory, skills, review, curator. Use /evolution pending | approve <id> | reject <id> | curator run | curator status | curator pause | curator resume | curator report | curator scope | mutations | restore | consolidate <target> <source...> | skill restore <name> | skills health | skills refresh | learn [request] | maintain [--timeout ms | --facts] | preset install | restructure <name> \"<heading>\" <to_file> | replay.");
 			}
-		});
+		}));
 	});
 }
 /**
@@ -299,13 +311,62 @@ function apply(ctx, rawConfig = {}) {
 */
 function resolveAgentPresetDir(importMetaUrl) {
 	const dir = dirname(fileURLToPath(importMetaUrl));
-	const candidates = [join(dir, "..", "dsh-evolution-agent-preset"), join(dir, "..", "..", "evolution-agent")];
+	const siblingCandidate = join(dir, "..", "dsh-evolution-agent-preset");
+	const candidates = [siblingCandidate, join(dir, "..", "..", "evolution-agent")];
 	for (const candidate of candidates) if (existsSync(join(candidate, "agent.cordis.yml")) && existsSync(join(candidate, "preset.yml"))) return candidate;
 	try {
 		return dirname(createRequire(importMetaUrl).resolve("@lmzhen/dsh-evolution-agent-preset/package.json"));
 	} catch {
-		return candidates[0] ?? join(dir, "..", "dsh-evolution-agent-preset");
+		return siblingCandidate;
 	}
 }
+/**
+* S6.3 (E-40): atomically replace a set of files within one directory.
+* Each file is staged to a sibling `<name>.tmp`, then renamed into place
+* (atomic on the same filesystem); a pre-existing file keeps a single
+* `<name>.bak`. A failure at ANY staging step removes the staged temps and
+* leaves the previous files untouched — a half-written target is impossible.
+*/
+function atomicWriteFiles(targetDir, writes) {
+	const stage = (name) => join(targetDir, `${name}.tmp`);
+	try {
+		for (const { name, content } of writes) writeFileSync(stage(name), content);
+		for (const { name } of writes) {
+			const finalPath = join(targetDir, name);
+			const bakPath = join(targetDir, `${name}.bak`);
+			if (existsSync(finalPath) && !existsSync(bakPath)) copyFileSync(finalPath, bakPath);
+		}
+		for (const { name } of writes) {
+			const finalPath = join(targetDir, name);
+			try {
+				renameSync(stage(name), finalPath);
+			} catch {
+				rmSync(finalPath, { force: true });
+				renameSync(stage(name), finalPath);
+			}
+		}
+	} catch (error) {
+		for (const { name } of writes) rmSync(stage(name), {
+			force: true,
+			recursive: true
+		});
+		throw error;
+	}
+}
+/**
+* Count the recommendation bullets in a rendered maintain plan.
+* `formatPlan` renders each recommendation as a line starting with `- [<kind>]`;
+* this matches that bullet (anchored at line start) instead of any string
+* prefix, so a `- note` line can never inflate the count.
+*
+* @param text - the rendered maintain result text, or `undefined`.
+* @returns the number of recommendation bullets in `text`.
+* @remarks the count is still text-derived — MaintainOutcome does not expose the
+* structured plan array, so a structured count would require a
+* evolution-maintenance outcome change (out of this package's scope).
+*/
+function countMaintainRecommendations(text) {
+	return text?.match(/^- \[/gm)?.length ?? 0;
+}
 //#endregion
-export { apply, name };
+export { apply, countMaintainRecommendations, name };

package/lib/types/index.d.ts

@@ -1,5 +1,5 @@
 /**
- * Human commands for the evolution family: /evolution learn|pending|curator|restore|consolidate.
+ * Human commands for the evolution family: /evolution learn|pending|curator|restore|consolidate|skills refresh.
  * @module @deepseek-ai/dsh-evolution-commands
  */
 import type { Context } from '@deepseek-ai/cordis';
@@ -22,4 +22,17 @@ export interface Config {
     maintainTimeoutMs?: number | undefined;
 }
 export declare function apply(ctx: Context, rawConfig?: Config): void;
+/**
+ * Count the recommendation bullets in a rendered maintain plan.
+ * `formatPlan` renders each recommendation as a line starting with `- [<kind>]`;
+ * this matches that bullet (anchored at line start) instead of any string
+ * prefix, so a `- note` line can never inflate the count.
+ *
+ * @param text - the rendered maintain result text, or `undefined`.
+ * @returns the number of recommendation bullets in `text`.
+ * @remarks the count is still text-derived — MaintainOutcome does not expose the
+ * structured plan array, so a structured count would require a
+ * evolution-maintenance outcome change (out of this package's scope).
+ */
+export declare function countMaintainRecommendations(text: string | undefined): number;
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-commands",
   "description": "Human commands for the evolution family (community build)",
-  "version": "0.3.16",
+  "version": "0.3.18",
   "publishConfig": {
     "access": "public"
   },
@@ -33,8 +33,8 @@
   "license": "MIT",
   "dependencies": {
     "@deepseek-ai/schemastery": "^3.18.1",
-    "@lmzhen/dsh-evolution-core": "^0.3.16",
-    "@lmzhen/dsh-evolution-maintenance": "^0.3.16"
+    "@lmzhen/dsh-evolution-core": "^0.3.18",
+    "@lmzhen/dsh-evolution-maintenance": "^0.3.18"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",