@lmzhen/dsh-evolution-commands 0.3.35 → 0.3.37

package/lib/index.js

@@ -93,9 +93,10 @@ function apply(ctx, rawConfig = {}) {
 					const curator = ctx.get("evolutionCurator");
 					if (!curator) return err("Curator service not mounted.");
 					const records = await curator.skills.listMutations();
-					if (records.length === 0) return ok("No mutation records yet.");
-					const recent = records.slice(-5).reverse().map((record) => `${record.at.slice(0, 19)}  ${record.skillName}  ${record.action}  ${record.summary}`);
-					return ok(`Mutations: ${records.length} recorded (recent 5):\n${recent.join("\n")}`);
+					const usable = Array.isArray(records) ? records.filter((row) => typeof row === "object" && row !== null && typeof row.at === "string" && typeof row.skillName === "string" && typeof row.action === "string" && typeof row.summary === "string") : [];
+					if (usable.length === 0) return ok("No mutation records yet.");
+					const recent = usable.slice(-5).reverse().map((record) => `${record.at.slice(0, 19)}  ${record.skillName}  ${record.action}  ${record.summary}`);
+					return ok(`Mutations: ${usable.length} recorded (recent 5):\n${recent.join("\n")}`);
 				}
 				if (input === "curator scope") {
 					const curator = ctx.get("evolutionCurator");
@@ -108,7 +109,7 @@ function apply(ctx, rawConfig = {}) {
 						line("Watched (stale / quality-warned)", view.watched),
 						line("Quality-warned", view.qualityWarned),
 						line("Exempted (exclude / referenced)", view.exempted),
-						line("Protected (pinned / bundled / hub)", view.protected)
+						line("Protected (pinned / bundled / hub / builtin)", view.protected)
 					].join("\n"));
 				}
 				if (input === "curator report") {
@@ -116,11 +117,15 @@ function apply(ctx, rawConfig = {}) {
 					if (!curator) return err("Curator service not mounted.");
 					const report = await curator.latestReport();
 					if (!report) return ok("No curator report available.");
+					if (typeof report !== "object" || typeof report.runId !== "string" || typeof report.startedAt !== "string" || !Array.isArray(report.archived) || !Array.isArray(report.failed)) return err("Report file unreadable.");
+					const reportAny = report;
+					const nameOf = (item) => typeof item === "object" && item !== null && typeof item.name === "string" ? item.name : "?";
+					const reasonOf = (item) => typeof item === "object" && item !== null && typeof item.reason === "string" ? item.reason : "?";
 					return ok([
-						`runId=${report.runId}`,
-						`startedAt=${report.startedAt}`,
-						`archived=${report.archived.map((item) => item.name).join(", ") || "(none)"}`,
-						`failed=${report.failed.map((item) => `${item.name}: ${item.reason}`).join(", ") || "(none)"}`
+						`runId=${reportAny.runId}`,
+						`startedAt=${reportAny.startedAt}`,
+						`archived=${reportAny.archived.map(nameOf).join(", ") || "(none)"}`,
+						`failed=${reportAny.failed.map((item) => `${nameOf(item)}: ${reasonOf(item)}`).join(", ") || "(none)"}`
 					].join("\n"));
 				}
 				if (input.startsWith("restore ")) {
@@ -208,7 +213,7 @@ function apply(ctx, rawConfig = {}) {
 				const maintainArgs = /^maintain(?: --timeout (\d+))?\s*$/.exec(input);
 				if (maintainArgs) {
 					const runTimeoutMs = maintainArgs[1] ? Number(maintainArgs[1]) : config.maintainTimeoutMs ?? 6e5;
-					if (!Number.isSafeInteger(runTimeoutMs) || runTimeoutMs <= 0) return err("Invalid --timeout value: expected a positive integer number of milliseconds (e.g. /evolution maintain --timeout 600000).");
+					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.`);
 					const cooldownMs = config.maintainCooldownMs ?? 3e4;
 					const sinceLast = Date.now() - lastMaintainAt;
@@ -375,6 +380,8 @@ const defaultFs = {
 */
 function atomicWriteFiles(targetDir, writes, fs = defaultFs) {
 	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(", ")}`);
 	try {
 		for (const { name, content } of writes) fs.writeFileSync(stage(name), content);
 		for (const { name } of writes) {
@@ -391,7 +398,12 @@ function atomicWriteFiles(targetDir, writes, fs = defaultFs) {
 				fs.renameSync(tmp, finalPath);
 				committed.push(name);
 			} catch {
-				fs.rmSync(finalPath, { force: true });
+				try {
+					fs.rmSync(finalPath, { force: true });
+				} catch (removeError) {
+					const already = committed.length > 0 ? `; already committed: ${committed.join(", ")}` : "";
+					throw new Error(`atomicWriteFiles: could not remove "${name}" to replace it (${removeError instanceof Error ? removeError.message : String(removeError)})${already}`);
+				}
 				try {
 					fs.renameSync(tmp, finalPath);
 					committed.push(name);

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-commands",
   "description": "Human commands for the evolution family (community build)",
-  "version": "0.3.35",
+  "version": "0.3.37",
   "publishConfig": {
     "access": "public"
   },
@@ -30,9 +30,9 @@
   ],
   "license": "MIT",
   "dependencies": {
-    "@lmzhen/dsh-evolution-approval": "^0.3.35",
-    "@lmzhen/dsh-evolution-core": "^0.3.35",
-    "@lmzhen/dsh-evolution-maintenance": "^0.3.35"
+    "@lmzhen/dsh-evolution-approval": "^0.3.37",
+    "@lmzhen/dsh-evolution-core": "^0.3.37",
+    "@lmzhen/dsh-evolution-maintenance": "^0.3.37"
   },
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",