@opencraw/studio 0.0.13 → 0.0.15

package/dist/index.esm.js

@@ -8,7 +8,7 @@ import { randomBytes, createHash } from 'node:crypto';
 import { execFile } from 'node:child_process';
 import { createRequire } from 'node:module';
 import { load } from 'cheerio';
-import { readFile, writeFile, mkdir } from 'node:fs/promises';
+import { readFile, writeFile, rm, rename, mkdir } from 'node:fs/promises';
 import { describeJson } from '@opencraw/probe';
 import { tmpdir } from 'node:os';
 import { existsSync, createReadStream } from 'node:fs';
@@ -2877,11 +2877,40 @@ function parseOne(file, content, kind, parse) {
  * round-trip for a hand-formatted file is future work; see this file's test
  * for what is guaranteed today.
  *
+ * The text goes to a sibling temporary file that is then renamed over the
+ * recipe, so a reader (or a crash) never sees a half-written file. Windows
+ * refuses a rename over a file another process has open (`EPERM`, `EBUSY`),
+ * so the rename is retried briefly and then gives way to a plain write.
+ *
  * @param path - The recipe file to write.
  * @param recipe - The whole recipe object.
  */
 async function saveRecipe(path, recipe) {
-  await writeFile(path, `${JSON.stringify(recipe, null, 2)}\n`, 'utf8');
+  const temporary = `${path}.${String(process.pid)}.tmp`;
+  const text = `${JSON.stringify(recipe, null, 2)}\n`;
+  await writeFile(temporary, text, 'utf8');
+  try {
+    await renameWithRetry(temporary, path);
+  } catch {
+    await writeFile(path, text, 'utf8');
+  } finally {
+    await rm(temporary, {
+      force: true
+    });
+  }
+}
+const RENAME_ATTEMPTS = 5;
+const RENAME_PAUSE_MS = 20;
+async function renameWithRetry(from, to) {
+  for (let attempt = 1;; attempt += 1) {
+    try {
+      await rename(from, to);
+      return;
+    } catch (error) {
+      if (attempt >= RENAME_ATTEMPTS) throw error;
+      await new Promise(resolve => setTimeout(resolve, RENAME_PAUSE_MS));
+    }
+  }
 }
 
 /** How many `name-2.ext`, `name-3.ext`… are tried before giving up on a folder full of same-named copies. */

package/dist/src/recipe-workspace/save-recipe.use-case.d.ts

@@ -9,6 +9,11 @@
  * round-trip for a hand-formatted file is future work; see this file's test
  * for what is guaranteed today.
  *
+ * The text goes to a sibling temporary file that is then renamed over the
+ * recipe, so a reader (or a crash) never sees a half-written file. Windows
+ * refuses a rename over a file another process has open (`EPERM`, `EBUSY`),
+ * so the rename is retried briefly and then gives way to a plain write.
+ *
  * @param path - The recipe file to write.
  * @param recipe - The whole recipe object.
  */