@lmzhen/dsh-evolution-commands 0.3.51 → 0.3.52

package/lib/index.js

@@ -23,7 +23,7 @@ function apply(ctx, rawConfig = {}) {
 			name: "evolution",
 			description: "Self-evolution status and approval controls",
 			recordInput: false,
-			input: { hint: "pending [--detail] | 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" },
+			input: { hint: "pending [--detail] | 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) => ({
@@ -74,7 +74,8 @@ function apply(ctx, rawConfig = {}) {
 					if (!curator) return err("Curator service not mounted.");
 					const paused = input === "curator pause";
 					await curator.setPaused(paused);
-					return ok(paused ? "Curator automatic curation paused. Manual /evolution curator run is unaffected; resume with /evolution curator resume." : "Curator automatic curation resumed. The next scheduled pass waits one interval (first-run defer semantics).");
+					const notPersisted = ctx.get("evolutionState") !== void 0 ? "" : "\nNOTE: no evolution state service is mounted — this setting is NOT persisted and will not survive this process.";
+					return ok((paused ? "Curator automatic curation paused. Manual /evolution curator run is unaffected; resume with /evolution curator resume." : "Curator automatic curation resumed. The next scheduled pass waits one interval (first-run defer semantics).") + notPersisted);
 				}
 				if (input === "curator status") {
 					const curator = ctx.get("evolutionCurator");
@@ -210,14 +211,14 @@ function apply(ctx, rawConfig = {}) {
 					}), enrichment.usageObservedValue, void 0);
 					return ok(`Maintenance facts (0-token preview):\n${facts}`);
 				}
-				const maintainArgs = /^maintain(?: --timeout (\d+))?\s*$/.exec(input);
+				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 (maintainInFlightSince > 0) return ok(`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.`);
+					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;
-					if (cooldownMs > 0 && sinceLast < cooldownMs) return ok(`Maintenance cooldown active (${Math.ceil((cooldownMs - sinceLast) / 1e3)}s) — latest scan ${lastMaintainRunId}; re-running now would spend another model call.`);
+					if (cooldownMs > 0 && sinceLast < cooldownMs) return err(`Maintenance cooldown active (${Math.ceil((cooldownMs - sinceLast) / 1e3)}s) — latest scan ${lastMaintainRunId}; re-running now would spend another model call.`);
 					const ioRegistry = ctx.get("evolutionIo");
 					const subagents = ctx.get("subagents");
 					if (!ioRegistry) return err("Evolution IO registry not mounted — maintenance scan unavailable.");
@@ -231,6 +232,7 @@ function apply(ctx, rawConfig = {}) {
 							subagents,
 							parent: invocation.agent,
 							evolutionPolicy: { get: () => ctx.get("evolutionPolicy")?.get() },
+							tools: { get: (name) => ctx.get("tools")?.get(name) },
 							logger: { warn: (message) => {
 								ctx.logger.warn(message);
 							} }
@@ -250,7 +252,7 @@ function apply(ctx, rawConfig = {}) {
 							source: "manual",
 							runId: outcome.runId,
 							verdict: outcome.verdict,
-							recommendations: countMaintainRecommendations(outcome.text)
+							recommendations: outcome.recommendationCount
 						}).catch((error) => {
 							ctx.logger.warn(`evolution-commands: failed to record maintain event: ${String(error)}`);
 						});
@@ -262,7 +264,7 @@ function apply(ctx, rawConfig = {}) {
 						lastMaintainAt = Date.now();
 					}
 				}
-				if (/^maintain\b/.test(input)) return err("Unknown maintain arguments: expected bare `maintain` or `maintain --timeout <ms>` (a positive integer). Got: " + input);
+				if (/^maintain\b/.test(input)) return err("Unknown maintain arguments: expected bare `maintain`, `maintain --timeout <ms>` or `maintain --timeout=<ms>` (a positive integer). Got: " + input);
 				if (input === "preset install") {
 					const target = join(evolutionRoot(), ".agent-presets", "evolution");
 					try {
@@ -300,7 +302,7 @@ function apply(ctx, rawConfig = {}) {
 					if (!ioRegistry) return err("Evolution IO registry not mounted — restructure unavailable.");
 					const result = await new SkillLibrary(resolveSkillsRoot({ root: config.skillsRoot }), ioRegistry.provider(), void 0, (event) => {
 						ctx.emit("evolution/skill-mutated", event);
-					}).restructure(name, [{
+					}, void 0, config.threatExemptLabels ?? []).restructure(name, [{
 						heading,
 						toFile
 					}], "foreground");
@@ -313,7 +315,7 @@ 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 | 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.");
+				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.");
 			}
 		}));
 	});
@@ -442,32 +444,6 @@ function atomicWriteFiles(targetDir, writes, fs = defaultFs) {
 		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) {
-	if (!text) return 0;
-	return beforeNotesHeader(text).match(/^- \[/gm)?.length ?? 0;
-}
-/** Locate the maintain-plan `Notes:` section header as a standalone line.
-* Returns everything before it (the plan section), or the whole text when the
-* header is absent. formatPlan emits the header via `lines.push('Notes:')`, so
-* it is always a line that is exactly `Notes:` (never an indented/embedded
-* continuation), letting us ignore a `Notes:` that appears inside a field. */
-function beforeNotesHeader(text) {
-	const header = /(^|\n)Notes:(?=\n)/.exec(text);
-	if (!header) return text;
-	return text.slice(0, header.index);
-}
 /** F-328: render staged args for `pending --detail`, truncated to 500 chars.
 * args may be non-JSON (a tool produced garbage), so the render is fail-safe.
 * V4-19: a truncation is marked with `…(truncated N chars)` so the operator is
@@ -476,6 +452,7 @@ function beforeNotesHeader(text) {
 function safeStagedArgs(args) {
 	try {
 		const json = JSON.stringify(args);
+		if (typeof json !== "string") return "(unserializable)";
 		if (json.length > 500) return `${json.slice(0, 500)}…(truncated ${json.length - 500} chars)`;
 		return json;
 	} catch {
@@ -483,4 +460,4 @@ function safeStagedArgs(args) {
 	}
 }
 //#endregion
-export { apply, atomicWriteFiles, countMaintainRecommendations, name };
+export { apply, atomicWriteFiles, name };

package/lib/types/index.d.ts

@@ -22,6 +22,15 @@ export interface Config {
      * 4-8 min — the 13:38 successful run used --timeout 600000; the old 120s
      * default deadlined bare runs mid-analysis (14:37 run aborted at 119.94s). */
     maintainTimeoutMs?: number | undefined;
+    /** Threat-scan exemption labels for WRITE-path skill mutations (P2-18).
+     * Threaded into the SkillLibrary this package constructs for `restructure`
+     * (its only write path); the core-side constructor option carries the same
+     * field name `threatExemptLabels` (P2-18 core batch — the field name is the
+     * linkage contract, keep in sync). Default empty: strict-scan behavior is
+     * unchanged; a deployment opts specific labels out explicitly. Read-only
+     * paths (maintain / maintain --facts) never threat-scan, so this never
+     * widens them. */
+    threatExemptLabels?: string[] | undefined;
 }
 export declare function apply(ctx: Context, rawConfig?: Config): void;
 /** F-211: the fs surface atomicWriteFiles uses — injectable so the commit-phase
@@ -65,18 +74,5 @@ export declare function atomicWriteFiles(targetDir: string, writes: Array<{
     name: string;
     content: string | Uint8Array;
 }>, fs?: FsOps): 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;
 export {};
 //# sourceMappingURL=index.d.ts.map

package/package.json

@@ -1,14 +1,14 @@
 {
   "name": "@lmzhen/dsh-evolution-commands",
   "description": "Human commands for the evolution family (community build)",
-  "version": "0.3.51",
+  "version": "0.3.52",
   "publishConfig": {
     "access": "public"
   },
   "repository": {
     "type": "git",
     "url": "git+https://github.com/lmzhen/dsh-evolution.git",
-    "directory": "packages/dsh-evolution-commands"
+    "directory": "packages/evolution-commands"
   },
   "type": "module",
   "main": "lib/index.js",
@@ -30,12 +30,12 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-approval": "^0.3.51",
-    "@lmzhen/dsh-evolution-core": "^0.3.51",
-    "@lmzhen/dsh-evolution-maintenance": "^0.3.51"
+    "@lmzhen/dsh-evolution-approval": "^0.3.52",
+    "@lmzhen/dsh-evolution-core": "^0.3.52",
+    "@lmzhen/dsh-evolution-maintenance": "^0.3.52"
   },
   "optionalDependencies": {
-    "@lmzhen/dsh-evolution-agent-preset": "^0.3.51"
+    "@lmzhen/dsh-evolution-agent-preset": "^0.3.52"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",