@lmzhen/dsh-evolution-core 0.1.0-rc.64 → 0.1.0-rc.65

package/lib/index.js

@@ -55,9 +55,10 @@ function nodeEvolutionIo() {
 	/**
 	* 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.
+	* over. A held lock that outlasts the retry budget fails LOUD instead of
+	* proceeding unlocked — two unlocked writers overlap their RMW and lose
+	* updates, which is exactly how the rc.65 CI-only 7≠8 occurred (a slow
+	* writer held the lock past the budget and a peer entered unlocked).
 	*/
 	const withWriteLock = async (path, task) => {
 		const lock = `${path}.lock`;
@@ -83,7 +84,7 @@ function nodeEvolutionIo() {
 			}
 			await new Promise((resolve) => setTimeout(resolve, 50));
 		}
-		return await task();
+		throw new Error(`could not acquire write lock for ${path} after 10 attempts`);
 	};
 	return {
 		async readText(path) {
@@ -257,6 +258,11 @@ async function loadUsage(root, io = nodeEvolutionIo()) {
 */
 async function mutateUsage(root, io, task) {
 	await transactIo(io, usageFile(root), async (current) => {
+		if (current !== null) try {
+			JSON.parse(current);
+		} catch {
+			return current;
+		}
 		const map = parseUsage(current);
 		await task(map);
 		return JSON.stringify(Object.fromEntries(map.entries()), null, 2);
@@ -338,6 +344,11 @@ async function saveSuppressedNames(root, names, io = nodeEvolutionIo()) {
 */
 async function updateSuppressedNames(root, io, task) {
 	await transactIo(io, suppressedFile(root), async (current) => {
+		if (current !== null) try {
+			JSON.parse(current);
+		} catch {
+			return current;
+		}
 		const names = parseSuppressed(current);
 		await task(names);
 		return JSON.stringify({
@@ -1703,7 +1714,12 @@ async function loadMutations(root, io = nodeEvolutionIo()) {
 }
 /** Append one record, trim to `cap`, and write atomically (versioned shape). */
 async function recordMutation(root, io, record, cap = 500) {
-	await transactIo(io, mutationsFile(root), (current) => {
+	await transactIo(io, mutationsFile(root), async (current) => {
+		if (current !== null) try {
+			JSON.parse(current);
+		} catch {
+			return current;
+		}
 		const existing = parseMutationRecords(current);
 		existing.push(record);
 		const trimmed = existing.length > cap ? existing.slice(existing.length - cap) : existing;

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