fix(dev-scripts): make gen:docs writes to docs/package.json atomic; run unit tests once in CI

Two concurrent gen:docs processes were racing on the non-atomic
read-modify-write of the shared, tracked docs/package.json in
addDependenciesToExample, so one process truncated the file to 0 bytes
while another read it -> JSON.parse('') 'Unexpected end of JSON input'
(crashed on the backend/s3 example).

Two changes:
- writeFileAtomic(): write to a unique temp file then rename() (atomic on
  POSIX), and skip the write entirely when content is unchanged. Readers
  now always observe complete content, never an empty file, so a plain
  read+JSON.parse is safe. Used for the docs/package.json write in
  addDependenciesToExample.
- CI 'Run unit tests' now calls 'vp run test' (the root test script, run
  once) instead of 'vp run -r test'. The recursive form ran every
  package's test AND re-spawned the root test script's filtered graph
  concurrently, launching gen:docs twice. 'vp run test' runs it once and
  also preserves the intended !@blocknote/xl-ai exclusion.

Author: nperez0111

.github/workflows/build.yml

@@ -38,7 +38,7 @@ jobs:
         run: vp lint
 
       - name: Run unit tests
-        run: vp run -r test
+        run: vp run test
 
       - name: Run Next.js integration test (production build)
         run: NEXTJS_TEST_MODE=build vp test run src/unit/nextjs/serverUtil.test.ts

.github/workflows/fresh-install-tests.yml

@@ -84,7 +84,7 @@ jobs:
 
       - id: run_unit_tests
         name: Run unit tests
-        run: vp run -r test
+        run: vp run test
 
       - name: Notify Slack on workflow failure
         if: ${{ failure() }}

packages/dev-scripts/examples/genDocs.ts

@@ -8,6 +8,7 @@ import {
   getProjectFiles,
   groupProjects,
   Project,
+  writeFileAtomic,
   writeGeneratedFile,
 } from "./util.js";
 
@@ -165,8 +166,7 @@ async function addDependenciesToExample(project: Project, written: string[]) {
     Object.keys(devDependencies).length > 0
   ) {
     const packageJson = path.join(DOCS_DIR, "package.json");
-    const packageJsonContent = fs.readFileSync(packageJson, "utf-8");
-    const packageJsonObject = JSON.parse(packageJsonContent);
+    const packageJsonObject = JSON.parse(fs.readFileSync(packageJson, "utf-8"));
     packageJsonObject.dependencies = {
       ...packageJsonObject.dependencies,
       ...dependencies,
@@ -186,11 +186,13 @@ async function addDependenciesToExample(project: Project, written: string[]) {
         packageJsonObject.devDependencies[key] = "workspace:*";
       }
     });
-    writeGeneratedFile(
-      packageJson,
-      JSON.stringify(packageJsonObject, null, 2),
-      written,
-    );
+    // Atomic write so a concurrent `gen:docs` process can never read a
+    // truncated/empty docs/package.json (which previously caused
+    // `JSON.parse('')` to throw "Unexpected end of JSON input"). The merged
+    // content is deterministic, so skip-if-unchanged keeps this idempotent.
+    const target = path.resolve(packageJson);
+    writeFileAtomic(target, JSON.stringify(packageJsonObject, null, 2));
+    written.push(target);
   }
 }
 

packages/dev-scripts/examples/util.ts

@@ -22,6 +22,30 @@ export function writeGeneratedFile(
   written.push(absolute);
 }
 
+/**
+ * Atomically writes a file by writing to a unique temp file in the same
+ * directory and renaming it into place (rename is atomic on POSIX). This
+ * prevents readers from ever observing a truncated/empty file, which matters
+ * for shared files (e.g. docs/package.json) that may be read and written by
+ * concurrent `gen:docs` processes. Skips the write entirely when the on-disk
+ * content already matches, avoiding an unnecessary truncation window.
+ */
+export function writeFileAtomic(absolute: string, content: string) {
+  try {
+    if (fs.readFileSync(absolute, "utf-8") === content) {
+      return;
+    }
+  } catch {
+    // File does not exist yet (or could not be read) — fall through to write.
+  }
+  fs.mkdirSync(path.dirname(absolute), { recursive: true });
+  const tmp = `${absolute}.${process.pid}.${Date.now()}.${Math.random()
+    .toString(36)
+    .slice(2)}.tmp`;
+  fs.writeFileSync(tmp, content);
+  fs.renameSync(tmp, absolute);
+}
+
 /**
  * Formats the given files in-place using `vp fmt`. Files that are excluded by
  * oxfmt ignore rules (e.g. *.mdx) are tolerated. Runs in chunks to avoid argv