@pylonsync/create-pylon 0.3.277 → 0.3.279

package/bin/create-pylon.js

@@ -42,7 +42,12 @@ import { stdin, stdout, exit, argv, cwd } from "node:process";
 // ---------------------------------------------------------------------------
 
 const HERE = dirname(fileURLToPath(import.meta.url));
-const TEMPLATES = resolve(HERE, "..", "templates");
+// PYLON_CREATE_TEMPLATES_DIR overrides the template source dir. Only used by
+// the scaffolder's own tests (to point at a fixture / empty dir and exercise
+// the missing-template guard); undocumented + irrelevant in normal use.
+const TEMPLATES = process.env.PYLON_CREATE_TEMPLATES_DIR
+	? resolve(process.env.PYLON_CREATE_TEMPLATES_DIR)
+	: resolve(HERE, "..", "templates");
 
 // ---------------------------------------------------------------------------
 // Version pin — every generated dep references this version of @pylonsync/*.
@@ -257,22 +262,35 @@ Examples:
 }
 
 const rl = createInterface({ input: stdin, output: stdout });
+// Non-interactive stdin (CI, piped, `| npm create …`): never block on a prompt.
+// Fall back to the same defaults the interactive path uses. The skill-install
+// below already gates on `stdin.isTTY`; the INPUT prompts must too — without
+// this, `npm create @pylonsync/pylon my-app` (or any partial-flag invocation)
+// HANGS forever in CI waiting on input that never comes.
+const isInteractive = !!stdin.isTTY;
 if (!projectName) {
-	projectName = (await rl.question("Project name: ")).trim() || "my-pylon-app";
+	projectName =
+		(isInteractive ? (await rl.question("Project name: ")).trim() : "") ||
+		"my-pylon-app";
 }
 if (!flags.template) {
-	const lines = Object.entries(TEMPLATE_REGISTRY)
-		.map(([k, v]) => `  ${k.padEnd(10)} ${v.blurb}`)
-		.join("\n");
-	process.stdout.write(`\n${lines}\n`);
-	const ans = (
-		await rl.question(
-			`Template (${TEMPLATES_AVAILABLE.join(", ")}) [default]: `,
+	if (isInteractive) {
+		const lines = Object.entries(TEMPLATE_REGISTRY)
+			.map(([k, v]) => `  ${k.padEnd(10)} ${v.blurb}`)
+			.join("\n");
+		process.stdout.write(`\n${lines}\n`);
+		const ans = (
+			await rl.question(
+				`Template (${TEMPLATES_AVAILABLE.join(", ")}) [default]: `,
+			)
 		)
-	)
-		.trim()
-		.toLowerCase();
-	flags.template = TEMPLATES_AVAILABLE.includes(ans) ? ans : "default";
+			.trim()
+			.toLowerCase();
+		flags.template = TEMPLATES_AVAILABLE.includes(ans) ? ans : "default";
+	} else {
+		console.log("Non-interactive stdin — using --template default.");
+		flags.template = "default";
+	}
 }
 // `ssr` was the original name of the default template; keep it working as a
 // quiet alias so older `--template ssr` invocations don't break.
@@ -286,32 +304,44 @@ if (flags.template === "saas") flags.template = "default";
 const isUnified = TEMPLATE_REGISTRY[flags.template]?.unified === true;
 if (!isUnified && !flags.platforms) {
 	const supported = TEMPLATE_REGISTRY[flags.template].platforms.join(", ");
-	const ans = (
-		await rl.question(
-			`Platforms for ${flags.template} (${supported}, comma-separated) [web]: `,
-		)
-	).trim();
+	const ans = isInteractive
+		? (
+				await rl.question(
+					`Platforms for ${flags.template} (${supported}, comma-separated) [web]: `,
+				)
+			).trim()
+		: "";
 	flags.platforms = ans || "web";
 }
 if (!flags.pm) {
 	const detected = detectPackageManager();
 	const def = detected ?? "bun";
-	const choice = (
-		await rl.question(`Package manager (bun, pnpm, yarn, npm) [${def}]: `)
-	)
-		.trim()
-		.toLowerCase();
-	flags.pm = ["bun", "pnpm", "yarn", "npm"].includes(choice) ? choice : def;
+	if (isInteractive) {
+		const choice = (
+			await rl.question(`Package manager (bun, pnpm, yarn, npm) [${def}]: `)
+		)
+			.trim()
+			.toLowerCase();
+		flags.pm = ["bun", "pnpm", "yarn", "npm"].includes(choice) ? choice : def;
+	} else {
+		flags.pm = def;
+	}
 }
 if (flags.skill === undefined) {
-	const ans = (
-		await rl.question(
-			"Add the Pylon skill to your coding agent (Claude Code / Codex / Cursor)? [Y/n]: ",
+	if (isInteractive) {
+		const ans = (
+			await rl.question(
+				"Add the Pylon skill to your coding agent (Claude Code / Codex / Cursor)? [Y/n]: ",
+			)
 		)
-	)
-		.trim()
-		.toLowerCase();
-	flags.skill = ans !== "n" && ans !== "no";
+			.trim()
+			.toLowerCase();
+		flags.skill = ans !== "n" && ans !== "no";
+	} else {
+		// Non-interactive: can't prompt. The actual install is TTY-gated below,
+		// so this only controls whether the footer prints the one-liner hint.
+		flags.skill = true;
+	}
 }
 rl.close();
 
@@ -462,6 +492,26 @@ function copyTemplate(srcSubpath, destSubpath = "") {
 	return true;
 }
 
+// Like copyTemplate but FATAL when the source is missing. A required
+// template dir that isn't on disk means a corrupt/partial create-pylon
+// install (or a published tarball that dropped files) — historically this
+// scaffolded an EMPTY project and STILL printed "✓ Created", sending the
+// user to a dead `pylon dev` with no clue why. Fail loud + actionable
+// instead. (The @pylonsync/client publish-drop that broke the default
+// scaffold is exactly this failure mode.)
+function mustCopy(srcSubpath, destSubpath = "", label = srcSubpath) {
+	if (!copyTemplate(srcSubpath, destSubpath)) {
+		console.error(
+			`\nError: template files for "${label}" are missing from this install\n` +
+				`       (expected ${join(TEMPLATES, srcSubpath)}).\n` +
+				`       This usually means a corrupt or partial create-pylon install.\n` +
+				`       Re-run with @latest:  npm create @pylonsync/pylon@latest\n` +
+				`       or report it:         https://github.com/pylonsync/pylon/issues\n`,
+		);
+		exit(1);
+	}
+}
+
 // ---------------------------------------------------------------------------
 // Apply templates in order:
 //   1. _root            — gitignore, env.example, README
@@ -475,24 +525,24 @@ if (isUnified) {
 	// Single unified app: app.ts + app/ routes + functions/, served by
 	// `pylon dev` (frontend + API, one port). The template ships its own
 	// package.json — no monorepo root, no turbo, no workspaces.
-	copyTemplate(flags.template);
+	mustCopy(flags.template);
 } else {
-	copyTemplate("_root");
-	copyTemplate(`backend/${flags.template}`);
+	mustCopy("_root");
+	mustCopy(`backend/${flags.template}`);
 
 	// `web` (Next.js) and `vite` are alternative web-frontend toolchains;
 	// the mutex check above guarantees at most one of them is set. Either
 	// way we also pull in packages/ui so the shared primitives are present.
 	if (platforms.includes("web")) {
-		copyTemplate("ui");
-		copyTemplate(`web/${flags.template}`);
+		mustCopy("ui");
+		mustCopy(`web/${flags.template}`);
 	}
 	if (platforms.includes("vite")) {
-		copyTemplate("ui");
-		copyTemplate(`vite/${flags.template}`);
+		mustCopy("ui");
+		mustCopy(`vite/${flags.template}`);
 	}
 	for (const p of ["ios", "mac", "expo"]) {
-		if (platforms.includes(p)) copyTemplate(`${p}/${flags.template}`);
+		if (platforms.includes(p)) mustCopy(`${p}/${flags.template}`);
 	}
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pylonsync/create-pylon",
-  "version": "0.3.277",
+  "version": "0.3.279",
   "description": "Scaffold a new Pylon app — realtime backend + web/mobile/expo frontends in one command. Run via `npm create @pylonsync/pylon@latest`.",
   "publishConfig": {
     "access": "public"
@@ -9,6 +9,9 @@
   "bin": {
     "create-pylon": "./bin/create-pylon.js"
   },
+  "scripts": {
+    "test": "node --test"
+  },
   "files": [
     "bin",
     "templates"