create-zfb 0.1.0-next.76 → 0.1.0-next.78

package/bin/create-zfb.mjs

@@ -4,6 +4,7 @@ import { createRequire } from "node:module";
 import { constants as osConstants } from "node:os";
 import { dirname, join } from "node:path";
 import { checkForNewerVersion } from "./version-check.mjs";
+import { formatSpawnErrorMessage } from "./spawn-error-message.mjs";
 
 const require = createRequire(import.meta.url);
 
@@ -22,6 +23,18 @@ const zfbBin = join(dirname(zfbPkgJson), "bin", "zfb.mjs");
 
 const args = ["new", ...process.argv.slice(2)];
 const result = spawnSync(process.execPath, [zfbBin, ...args], { stdio: "inherit" });
+
+// Surface spawn failures clearly. Without this check, an ENOENT/EACCES
+// failure to even launch the resolved @takazudo/zfb bin/zfb.mjs (e.g. a
+// broken resolution, a wiped node_modules, an unreadable file) silently
+// exits 1 with no message — a broken @takazudo/zfb resolution looks like a
+// mystery exit. Mirrors packages/zfb/bin/zfb.mjs's child.on("error")
+// handling (issues #447/#441).
+if (result.error) {
+  process.stderr.write(formatSpawnErrorMessage(result.error, zfbBin));
+  process.exit(1);
+}
+
 if (result.signal) {
   // Re-raise the child's termination signal on ourselves so the caller sees
   // the real cause of death (e.g. WIFSIGNALED), not a plain exit code 1.

package/bin/spawn-error-message.mjs

@@ -0,0 +1,24 @@
+/**
+ * Build an actionable stderr message for a `spawnSync()` launch failure
+ * (`result.error` set — e.g. ENOENT/EACCES trying to invoke the resolved
+ * `@takazudo/zfb` launcher). Extracted as a pure function (no top-level
+ * side effects, unlike create-zfb.mjs) so it can be unit tested directly,
+ * mirroring the version-check.mjs pattern in this package.
+ *
+ * Mirrors packages/zfb/bin/zfb.mjs's `child.on("error", ...)` messaging
+ * (issues #447/#441) so both wrappers give the same reinstall guidance.
+ *
+ * @param {NodeJS.ErrnoException} error
+ * @param {string} zfbBin - absolute path to the resolved bin/zfb.mjs launcher
+ * @returns {string}
+ */
+export function formatSpawnErrorMessage(error, zfbBin) {
+  if (error.code === "EACCES") {
+    return (
+      "[create-zfb] not executable; was the install corrupt?\n" +
+      `      ${zfbBin}\n` +
+      "      Try reinstalling: npm install --include=optional\n"
+    );
+  }
+  return `[create-zfb] failed to spawn @takazudo/zfb: ${error.message}\n` + `      ${zfbBin}\n`;
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "create-zfb",
-  "version": "0.1.0-next.76",
+  "version": "0.1.0-next.78",
   "private": false,
   "type": "module",
   "description": "Scaffold a new zfb static-site project: `npm create zfb@latest my-site`",
@@ -36,7 +36,7 @@
     "node": ">=18"
   },
   "dependencies": {
-    "@takazudo/zfb": "0.1.0-next.76"
+    "@takazudo/zfb": "0.1.0-next.78"
   },
   "devDependencies": {
     "vitest": "^2.1.9"