@lmzhen/dsh-evolution-core 0.3.56 → 0.3.57

package/lib/index.js

@@ -311,6 +311,10 @@ function nodeEvolutionIo(lockAttempts = 40) {
 					retryDelay: 100
 				}).catch(async () => {
 					const body = await readFile(lock, "utf8").catch(() => "");
+					if (pendingSelfCleanup.size >= 64) {
+						const oldest = pendingSelfCleanup.keys().next().value;
+						if (oldest !== void 0) pendingSelfCleanup.delete(oldest);
+					}
 					pendingSelfCleanup.set(lock, body);
 				});
 			}
@@ -428,7 +432,7 @@ function nodeEvolutionIo(lockAttempts = 40) {
 		},
 		async rename(path, destination) {
 			await mkdir(dirname(destination), { recursive: true });
-			await rename(path, destination);
+			await renameWithRetry(path, destination);
 		},
 		async copy(path, destination) {
 			await mkdir(dirname(destination), { recursive: true });
@@ -1207,8 +1211,8 @@ async function appendEvolutionEvent(io, path, event, rotateAt = EVENT_LOG_ROTATE
 				return current;
 			}
 		}
-		const nextEvents = await rotateIfDue(io, path, await rotateIfDue(io, path, v1EventRecords(parsedBody), rotateAt), rotateAt);
-		let maxSeq = nextEvents.reduce((max, entry) => Math.max(max, entry.seq), 0);
+		const events = await rotateIfDue(io, path, v1EventRecords(parsedBody), rotateAt);
+		let maxSeq = events.reduce((max, entry) => Math.max(max, entry.seq), 0);
 		if (maxSeq === 0) for (const name of await listEventArchives(io, path)) maxSeq = Math.max(maxSeq, Number.parseInt(name.slice(7, name.length - 5), 10));
 		const record = {
 			...event,
@@ -1218,7 +1222,7 @@ async function appendEvolutionEvent(io, path, event, rotateAt = EVENT_LOG_ROTATE
 		assigned = record.seq;
 		return JSON.stringify({
 			version: 1,
-			events: [...nextEvents, record]
+			events: [...events, record]
 		}, null, 2);
 	});
 	if (assigned === 0) throw new Error(`${refuseMessage || "evolution event log is malformed and was not touched"}: ${path}`);
@@ -3109,9 +3113,11 @@ function observeEvent(signal, event) {
 		return;
 	}
 	if (event.type === "tool/call") {
+		const data = event.data;
+		if (data === null || typeof data !== "object" || Array.isArray(data)) return;
 		signal.toolCalls += 1;
-		if (event.data.name === "skill") signal.skillSignal = true;
-		if (event.data.name === "skill_manage") signal.skillSignal = true;
+		const name = data.name;
+		if (name === "skill" || name === "skill_manage") signal.skillSignal = true;
 	}
 }
 /** Compute review cadence after `turn/end`. */
@@ -4145,6 +4151,9 @@ var SkillLibrary = class {
 	}
 	async create(name, content, origin = "foreground") {
 		const normalized = name.trim();
+		return await this.serial(() => this.createCore(normalized, content, origin));
+	}
+	async createCore(normalized, content, origin) {
 		const bad = this.badName(normalized);
 		if (bad) return {
 			ok: false,
@@ -4882,9 +4891,10 @@ var SkillLibrary = class {
 	*/
 	async restoreFromArchive(rawName) {
 		const name = rawName.trim();
-		if (!SKILL_NAME_RE.test(name)) return {
+		const bad = this.badName(name);
+		if (bad) return {
 			ok: false,
-			message: `Invalid skill name "${name}". Use lowercase letters, digits, and hyphens.`
+			message: bad
 		};
 		if (await this.io.exists(join(this.dirOf(name), "SKILL.md"))) return {
 			ok: false,
@@ -5013,6 +5023,9 @@ var SkillLibrary = class {
 	}
 	async removeSupportFile(rawName, filePath, origin = "foreground") {
 		const name = rawName.trim();
+		return await this.serial(() => this.removeSupportFileCore(name, filePath, origin));
+	}
+	async removeSupportFileCore(name, filePath, origin) {
 		const badName = this.badName(name);
 		if (badName) return {
 			ok: false,
@@ -5064,36 +5077,41 @@ var SkillLibrary = class {
 		const stamp = (/* @__PURE__ */ new Date()).toISOString().replace(/[:.]/g, "-");
 		let dest = join(backupRoot, `skills-${stamp}`);
 		while (await this.io.exists(dest)) dest = join(backupRoot, `skills-${stamp}-${Math.random().toString(36).slice(2, 8)}`);
-		const names = await listNames(this.root, this.io);
-		await Promise.all(names.map(async (name) => {
-			await this.io.copy(this.dirOf(name), join(dest, name));
-		}));
-		const sidecars = [];
-		for (const sidecar of [usageFile(this.root), suppressedFile(this.root)]) if (await this.io.exists(sidecar)) {
-			const name = basename(sidecar);
-			await this.io.copy(sidecar, join(dest, name));
-			sidecars.push(name);
-		}
-		const archiveRoot = join(this.root, ".archive");
-		let hasArchive = false;
-		if (await this.io.exists(archiveRoot)) {
-			await this.io.copy(archiveRoot, join(dest, ".archive"));
-			hasArchive = true;
+		try {
+			const names = await listNames(this.root, this.io);
+			await Promise.all(names.map(async (name) => {
+				await this.io.copy(this.dirOf(name), join(dest, name));
+			}));
+			const sidecars = [];
+			for (const sidecar of [usageFile(this.root), suppressedFile(this.root)]) if (await this.io.exists(sidecar)) {
+				const name = basename(sidecar);
+				await this.io.copy(sidecar, join(dest, name));
+				sidecars.push(name);
+			}
+			const archiveRoot = join(this.root, ".archive");
+			let hasArchive = false;
+			if (await this.io.exists(archiveRoot)) {
+				await this.io.copy(archiveRoot, join(dest, ".archive"));
+				hasArchive = true;
+			}
+			const validExtras = extras.filter((extra) => SNAPSHOT_EXTRA_NAME_RE.test(extra.name));
+			const extraNames = validExtras.map((extra) => extra.name);
+			await Promise.all(validExtras.map(async (extra) => {
+				await this.io.writeText(join(dest, "extras", extra.name), extra.content);
+			}));
+			await this.io.writeText(join(dest, "manifest.json"), JSON.stringify({
+				reason,
+				createdAt: (/* @__PURE__ */ new Date()).toISOString(),
+				skills: names,
+				sidecars,
+				hasArchive,
+				extras: extraNames
+			}, null, 2));
+			await this.retainSnapshots(5);
+		} catch (error) {
+			await this.io.remove(dest).catch(() => {});
+			throw error;
 		}
-		const validExtras = extras.filter((extra) => SNAPSHOT_EXTRA_NAME_RE.test(extra.name));
-		const extraNames = validExtras.map((extra) => extra.name);
-		await Promise.all(validExtras.map(async (extra) => {
-			await this.io.writeText(join(dest, "extras", extra.name), extra.content);
-		}));
-		await this.io.writeText(join(dest, "manifest.json"), JSON.stringify({
-			reason,
-			createdAt: (/* @__PURE__ */ new Date()).toISOString(),
-			skills: names,
-			sidecars,
-			hasArchive,
-			extras: extraNames
-		}, null, 2));
-		await this.retainSnapshots(5);
 		return dest;
 	}
 	/** Read and normalize a snapshot manifest; null when the file is missing or unparsable. */

package/lib/types/skill-store.d.ts

@@ -321,6 +321,7 @@ export declare class SkillLibrary {
      */
     setPinned(name: string, pinned: boolean, origin?: WriteOrigin): Promise<SkillActionResult>;
     create(name: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
+    private createCore;
     update(rawName: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     private updateCore;
     patch(rawName: string, oldString: string, newString: string, filePath?: string, replaceAll?: boolean, origin?: WriteOrigin): Promise<SkillActionResult>;
@@ -378,6 +379,7 @@ export declare class SkillLibrary {
     writeSupportFile(rawName: string, filePath: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     private writeSupportFileCore;
     removeSupportFile(rawName: string, filePath: string, origin?: WriteOrigin): Promise<SkillActionResult>;
+    private removeSupportFileCore;
     /**
      * Snapshot the recoverable skills state: active tree, usage/suppression
      * sidecars, `.archive/` and caller-supplied extras. `extras` are opaque

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@lmzhen/dsh-evolution-core",
   "description": "Shared stores, prompts, signals and lifecycle logic for the dsh-evolution plugin family (community build)",
-  "version": "0.3.56",
+  "version": "0.3.57",
   "publishConfig": {
     "access": "public"
   },