@lmzhen/dsh-evolution-core 0.1.0-rc.34 → 0.1.0-rc.36

package/lib/index.js

@@ -32,6 +32,39 @@ function nodeEvolutionIo() {
 		const code = error?.code;
 		return code === "ENOENT" || code === "ENOTDIR";
 	};
+	/**
+	* Cross-process write lock (claw `withFileLock` parity): an O_EXCL lock file
+	* guards the atomic write; a >5s-old lock is treated as stale and taken
+	* over. After the retry budget the write proceeds unlocked — the lock is a
+	* best-effort accommodation for multi-process deployments, never a read of
+	* availability.
+	*/
+	const withWriteLock = async (path, task) => {
+		const lock = `${path}.lock`;
+		for (let attempt = 0; attempt < 10; attempt += 1) try {
+			await writeFile(lock, String(process.pid), { flag: "wx" });
+			try {
+				return await task();
+			} finally {
+				await rm(lock, { force: true }).catch(() => {});
+			}
+		} catch (error) {
+			if (error?.code !== "EEXIST") throw error;
+			try {
+				const st = await stat(lock);
+				if (Date.now() - st.mtimeMs > 5e3) {
+					try {
+						await rm(lock, { force: true });
+					} catch {}
+					continue;
+				}
+			} catch {
+				continue;
+			}
+			await new Promise((resolve) => setTimeout(resolve, 50));
+		}
+		return await task();
+	};
 	return {
 		async readText(path) {
 			try {
@@ -43,9 +76,11 @@ function nodeEvolutionIo() {
 		},
 		async writeText(path, content) {
 			await mkdir(dirname(path), { recursive: true });
-			const tmp = `${path}.${process.pid}.${randomBytes(6).toString("hex")}.tmp`;
-			await writeFile(tmp, content, "utf8");
-			await rename(tmp, path);
+			await withWriteLock(path, async () => {
+				const tmp = `${path}.${process.pid}.${randomBytes(6).toString("hex")}.tmp`;
+				await writeFile(tmp, content, "utf8");
+				await rename(tmp, path);
+			});
 		},
 		async remove(path) {
 			await rm(path, {
@@ -1837,7 +1872,7 @@ var SkillLibrary = class {
 			path: dir
 		};
 	}
-	async create(name, content, origin) {
+	async create(name, content, origin = "foreground") {
 		const normalized = name.trim();
 		if (!SKILL_NAME_RE.test(normalized) || normalized.length > this.limits.maxNameLength) return {
 			ok: false,
@@ -1859,7 +1894,7 @@ var SkillLibrary = class {
 			message: `Skill "${normalized}" already exists.`
 		};
 		await this.io.writeText(join(dir, "SKILL.md"), content.trimEnd() + "\n");
-		if (origin === "background_review") await this.io.writeText(markerPath(dir, "hermes-managed"), "");
+		if (origin !== "foreground") await this.io.writeText(markerPath(dir, "hermes-managed"), "");
 		await this.audit(normalized, "create", null, content, "created");
 		return {
 			ok: true,
@@ -2212,8 +2247,9 @@ var SkillLibrary = class {
 		for (const name of names) await this.io.copy(skillDir(this.root, name), join(dest, name));
 		const sidecars = [];
 		for (const sidecar of [usageFile(this.root), suppressedFile(this.root)]) if (await this.io.exists(sidecar)) {
-			await this.io.copy(sidecar, join(dest, sidecar.split("/").pop() ?? sidecar));
-			sidecars.push(sidecar.split("/").pop() ?? sidecar);
+			const name = basename(sidecar);
+			await this.io.copy(sidecar, join(dest, name));
+			sidecars.push(name);
 		}
 		await this.io.writeText(join(dest, "manifest.json"), JSON.stringify({
 			reason,
@@ -2221,8 +2257,16 @@ var SkillLibrary = class {
 			skills: names,
 			sidecars
 		}, null, 2));
+		await this.retainSnapshots(5);
 		return dest;
 	}
+	/** Keep only the newest N snapshots (Hermes keep=5 parity); oldest folded into .backups history. */
+	async retainSnapshots(keep) {
+		const snapshots = await this.listSnapshots();
+		for (const snapshot of snapshots.slice(keep)) try {
+			await this.io.remove(snapshot.path);
+		} catch {}
+	}
 	async listSnapshots() {
 		const backupRoot = join(this.root, ".backups");
 		let entries;

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

@@ -29,7 +29,7 @@ export interface SkillActionResult {
     path?: string;
 }
 /** Who is writing: a foreground user-directed tool call, or the autonomous review/curator pipeline. */
-export type WriteOrigin = 'foreground' | 'background_review';
+export type WriteOrigin = 'foreground' | 'subagent' | 'background_review';
 /** Options for `SkillLibrary.archive`. The absorbed-into name and the archival reason are distinct fields. */
 export interface ArchiveOptions {
     /** Umbrella skill this one was consolidated into; when set it must exist (consolidate semantics). */
@@ -81,7 +81,7 @@ export declare class SkillLibrary {
      * marker write is the only state change; content is untouched.
      */
     setPinned(name: string, pinned: boolean, origin?: WriteOrigin): Promise<SkillActionResult>;
-    create(name: string, content: string, origin: 'foreground' | 'background_review'): Promise<SkillActionResult>;
+    create(name: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     update(name: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     patch(name: string, oldString: string, newString: string, filePath?: string, replaceAll?: boolean, origin?: WriteOrigin): Promise<SkillActionResult>;
     archive(name: string, options?: ArchiveOptions): Promise<SkillActionResult>;
@@ -100,6 +100,8 @@ export declare class SkillLibrary {
     writeSupportFile(name: string, filePath: string, content: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     removeSupportFile(name: string, filePath: string, origin?: WriteOrigin): Promise<SkillActionResult>;
     snapshotAll(reason?: string): Promise<string>;
+    /** Keep only the newest N snapshots (Hermes keep=5 parity); oldest folded into .backups history. */
+    private retainSnapshots;
     listSnapshots(): Promise<Array<{
         path: string;
         createdAt: string;

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.1.0-rc.34",
+  "version": "0.1.0-rc.36",
   "publishConfig": {
     "access": "public"
   },