Fail fast when bootstrapping workspaces

The root `bootstrap` script ran `npm run bootstrap --workspaces
--if-present`, but npm does not fail fast across workspaces: when one
workspace's bootstrap fails, npm keeps running the bootstrap script for
the remaining workspaces and only reports a non-zero exit code at the
end.

Because later workspaces depend on artifacts produced by earlier ones
(e.g. the weak-node-api xcframework consumed by node-addon-examples,
node-tests and ferric-example), a failure while building weak-node-api
produced a cascade of confusing secondary "missing xcframework" errors
that buried the actual root cause.

Replace the `--workspaces` iteration with a small script that bootstraps
each workspace in declared order and stops at the first failing one, so
CI surfaces the real error instead of the downstream cascade.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01MFfJiaaqbgMqnrNUB6e76m

Author: claude

package.json

@@ -27,7 +27,7 @@
     "prettier:check": "prettier --experimental-cli --check .",
     "prettier:write": "prettier --experimental-cli --write .",
     "test": "npm test --workspace react-native-node-api --workspace cmake-rn --workspace gyp-to-cmake --workspace node-addon-examples",
-    "bootstrap": "node --run build && npm run bootstrap --workspaces --if-present",
+    "bootstrap": "node --run build && node scripts/bootstrap.ts",
     "changeset": "changeset",
     "release": "changeset publish",
     "prerelease": "node --run build && npm run prerelease --workspaces --if-present && node --run publint",

scripts/bootstrap.ts

@@ -0,0 +1,68 @@
+import path from "node:path";
+import assert from "node:assert/strict";
+import cp from "node:child_process";
+import fs from "node:fs";
+
+// Bootstrap every workspace sequentially, failing fast on the first error.
+//
+// We deliberately don't use "npm run bootstrap --workspaces --if-present" here:
+// npm keeps iterating the remaining workspaces even after one of them fails and
+// only surfaces a non-zero exit code once they've all run. Because later
+// workspaces depend on artifacts produced by earlier ones (e.g. the
+// weak-node-api xcframework consumed by node-addon-examples, node-tests and
+// ferric-example), a failure in an early workspace produces a cascade of
+// confusing secondary errors, hiding the actual root cause. Iterating ourselves
+// lets us stop at the first failing workspace.
+
+const rootDir = path.resolve(import.meta.dirname, "..");
+
+function readPackageJson(packageDir: string): Record<string, unknown> {
+  const contents = fs.readFileSync(
+    path.join(packageDir, "package.json"),
+    "utf8",
+  );
+  const parsed = JSON.parse(contents) as unknown;
+  assert(
+    typeof parsed === "object" && parsed !== null,
+    `Expected an object in ${packageDir}/package.json`,
+  );
+  return parsed as Record<string, unknown>;
+}
+
+const rootPackage = readPackageJson(rootDir);
+assert(
+  Array.isArray(rootPackage.workspaces),
+  "Expected a 'workspaces' array in the root package.json",
+);
+
+for (const workspace of rootPackage.workspaces) {
+  assert(typeof workspace === "string");
+  // The workspaces are declared as concrete directories – reject globs rather
+  // than silently skipping workspaces they were meant to expand to.
+  assert(
+    !workspace.includes("*"),
+    `Glob workspace patterns are not supported ('${workspace}')`,
+  );
+
+  const workspaceDir = path.resolve(rootDir, workspace);
+  const { name, scripts } = readPackageJson(workspaceDir);
+  assert(typeof name === "string");
+
+  const hasBootstrap =
+    typeof scripts === "object" && scripts !== null && "bootstrap" in scripts;
+  if (!hasBootstrap) {
+    continue;
+  }
+
+  console.log(`Bootstrapping '${name}'`);
+  const { status } = cp.spawnSync(
+    "npm",
+    ["run", "bootstrap", "--workspace", name],
+    { stdio: "inherit" },
+  );
+  assert.equal(
+    status,
+    0,
+    `Bootstrapping '${name}' failed (status = ${status})`,
+  );
+}