fix(cli): fail fast in non-TTY environments instead of hanging on config-creation prompts (#1199)

When `open-next.config.ts` or `wrangler.(toml|json|jsonc)` is missing,
the CLI prompts the user to auto-create it. In non-TTY environments
(Cloudflare Workers Builds, Docker, CI) the Enquirer prompt can't read
stdin, so the build hangs on a truncated prompt and exits with a
cryptic code. Both prompts now throw an actionable error in
non-interactive mode; interactive behavior is unchanged.

Author: SdSadat

.changeset/fix-non-tty-config-prompts.md

@@ -0,0 +1,18 @@
+---
+"@opennextjs/cloudflare": patch
+---
+
+fix(cli): fail fast in non-TTY environments instead of hanging on config-creation prompts
+
+When `open-next.config.ts` (or `wrangler.(toml|json|jsonc)`) is missing, the CLI
+prompts the user to auto-create it. In non-TTY environments (Cloudflare Workers
+Builds, Docker, CI) the Enquirer prompt can't read stdin, so the build hangs or
+fails with a truncated prompt and a cryptic exit code — the user sees
+`? Missing required open-next.config.ts file, do you want to create one? (Y/n)`
+and then ` ELIFECYCLE  Command failed with exit code 13`, with no hint at what
+to do next.
+
+Now, in non-interactive environments, both prompts throw an actionable error
+with the exact template to paste (for `open-next.config.ts`) or point at the
+existing `--skipWranglerConfigCheck` / `SKIP_WRANGLER_CONFIG_CHECK` escape
+hatch (for the wrangler config). Interactive behavior is unchanged.

packages/cloudflare/package.json

@@ -55,6 +55,7 @@
 		"@ast-grep/napi": "^0.40.5",
 		"@dotenvx/dotenvx": "catalog:",
 		"@opennextjs/aws": "3.10.2",
+		"ci-info": "^4.2.0",
 		"cloudflare": "^4.4.1",
 		"comment-json": "^4.5.1",
 		"enquirer": "^2.4.1",

packages/cloudflare/src/cli/commands/build.spec.ts

@@ -5,6 +5,10 @@ import { askConfirmation } from "../utils/ask-confirmation.js";
 import { createWranglerConfigFile } from "../utils/create-wrangler-config.js";
 import { buildCommand } from "./build.js";
 
+vi.mock("../utils/is-interactive.js", () => ({
+	isNonInteractiveOrCI: vi.fn(() => false),
+}));
+
 // Mock logger
 vi.mock("@opennextjs/aws/logger.js", () => ({
 	default: {

packages/cloudflare/src/cli/commands/build.ts

@@ -4,6 +4,7 @@ import type yargs from "yargs";
 import { build as buildImpl } from "../build/build.js";
 import { askConfirmation } from "../utils/ask-confirmation.js";
 import { createWranglerConfigFile, findWranglerConfig } from "../utils/create-wrangler-config.js";
+import { isNonInteractiveOrCI } from "../utils/is-interactive.js";
 import type { WithWranglerArgs } from "./utils/utils.js";
 import {
 	compileConfig,
@@ -41,6 +42,15 @@ export async function buildCommand(
 	//       nor when `--skipWranglerConfigCheck` is used.
 	if (!projectOpts.wranglerConfigPath && !args.skipWranglerConfigCheck) {
 		if (!findWranglerConfig(projectOpts.sourceDir)) {
+			// In non-interactive environments (CI, Cloudflare Workers Builds,
+			// Docker, etc.) the prompt would hang or crash. Fail fast with a
+			// clear message pointing at the existing escape hatch.
+			if (isNonInteractiveOrCI()) {
+				throw new Error(
+					"No `wrangler.(toml|json|jsonc)` config file found.\n\nCreate one at the project root before running the build, or skip this check with `--skipWranglerConfigCheck` or `SKIP_WRANGLER_CONFIG_CHECK=yes`."
+				);
+			}
+
 			const confirmCreate = "No `wrangler.(toml|json|jsonc)` config file found, do you want to create one?";
 			if (await askConfirmation(confirmCreate)) {
 				await createWranglerConfigFile(projectOpts.sourceDir);

packages/cloudflare/src/cli/commands/utils/utils.spec.ts

@@ -1,7 +1,8 @@
-import { describe, expect, it, vi } from "vitest";
+import { beforeEach, describe, expect, it, vi } from "vitest";
 
 import { askConfirmation } from "../../utils/ask-confirmation.js";
 import { createOpenNextConfigFile, findOpenNextConfig } from "../../utils/create-open-next-config.js";
+import { isNonInteractiveOrCI } from "../../utils/is-interactive.js";
 import { compileConfig } from "./utils.js";
 
 const { mockExistsSync } = vi.hoisted(() => ({
@@ -20,12 +21,16 @@ vi.mock("@opennextjs/aws/logger.js", () => ({
 }));
 
 // Mock compileOpenNextConfig
-const mockCompileOpenNextConfig = vi.fn(async () => ({
-	config: { default: {} },
-	buildDir: "/build",
-}));
+const mockCompileOpenNextConfig = vi.fn(async (...args: [unknown, unknown]) => {
+	void args;
+
+	return {
+		config: { default: {} },
+		buildDir: "/build",
+	};
+});
 vi.mock("@opennextjs/aws/build/compileConfig.js", () => ({
-	compileOpenNextConfig: (...args: unknown[]) => mockCompileOpenNextConfig(...args),
+	compileOpenNextConfig: (...args: [unknown, unknown]) => mockCompileOpenNextConfig(...args),
 }));
 
 // Mock ensureCloudflareConfig
@@ -38,6 +43,11 @@ vi.mock("../../utils/ask-confirmation.js", () => ({
 	askConfirmation: vi.fn(),
 }));
 
+// Mock CI/non-interactive detector — default to interactive local behavior.
+vi.mock("../../utils/is-interactive.js", () => ({
+	isNonInteractiveOrCI: vi.fn(() => false),
+}));
+
 // Mock create-config-files (unused import in utils.ts but required for module resolution)
 vi.mock("../../utils/create-config-files.js", () => ({
 	createOpenNextConfigIfNotExistent: vi.fn(),
@@ -47,6 +57,7 @@ vi.mock("../../utils/create-config-files.js", () => ({
 vi.mock("../../utils/create-open-next-config.js", () => ({
 	findOpenNextConfig: vi.fn(),
 	createOpenNextConfigFile: vi.fn(() => "/test/open-next.config.ts"),
+	OPEN_NEXT_CONFIG_FILE_NAME: "open-next.config.ts",
 }));
 
 // Mock wrangler
@@ -66,6 +77,10 @@ vi.mock("@opennextjs/aws/build/helper.js", () => ({
 }));
 
 describe("compileConfig", () => {
+	beforeEach(() => {
+		vi.mocked(isNonInteractiveOrCI).mockReturnValue(false);
+	});
+
 	it("should compile config when configPath is provided and file exists", async () => {
 		mockExistsSync.mockReturnValue(true);
 
@@ -123,4 +138,27 @@ describe("compileConfig", () => {
 		expect(askConfirmation).toHaveBeenCalledOnce();
 		expect(createOpenNextConfigFile).not.toHaveBeenCalled();
 	});
+
+	it("should throw a helpful error (without prompting) when no configPath found in a non-interactive environment", async () => {
+		vi.mocked(findOpenNextConfig).mockReturnValue(undefined);
+		vi.mocked(isNonInteractiveOrCI).mockReturnValue(true);
+
+		await expect(compileConfig(undefined)).rejects.toThrowError(
+			/No `open-next\.config\.ts` file was found.*opennextjs-cloudflare migrate/s
+		);
+
+		expect(askConfirmation).not.toHaveBeenCalled();
+		expect(createOpenNextConfigFile).not.toHaveBeenCalled();
+	});
+
+	it("should still prompt in interactive environments", async () => {
+		vi.mocked(findOpenNextConfig).mockReturnValue(undefined);
+		vi.mocked(isNonInteractiveOrCI).mockReturnValue(false);
+		vi.mocked(askConfirmation).mockResolvedValue(true);
+
+		await compileConfig(undefined);
+
+		expect(askConfirmation).toHaveBeenCalledOnce();
+		expect(createOpenNextConfigFile).toHaveBeenCalledOnce();
+	});
 });

packages/cloudflare/src/cli/commands/utils/utils.ts

@@ -13,7 +13,12 @@ import type yargs from "yargs";
 import type { OpenNextConfig } from "../../../api/config.js";
 import { ensureCloudflareConfig } from "../../build/utils/ensure-cf-config.js";
 import { askConfirmation } from "../../utils/ask-confirmation.js";
-import { createOpenNextConfigFile, findOpenNextConfig } from "../../utils/create-open-next-config.js";
+import {
+	createOpenNextConfigFile,
+	findOpenNextConfig,
+	OPEN_NEXT_CONFIG_FILE_NAME,
+} from "../../utils/create-open-next-config.js";
+import { isNonInteractiveOrCI } from "../../utils/is-interactive.js";
 
 export type WithWranglerArgs<T = unknown> = T & {
 	// Array of arguments that can be given to wrangler commands, including the `--config` and `--env` args.
@@ -58,12 +63,21 @@ export async function compileConfig(configPath: string | undefined) {
 	configPath ??= findOpenNextConfig(nextAppDir);
 
 	if (!configPath) {
+		// In non-interactive environments (CI, Cloudflare Workers Builds,
+		// Docker, etc.) there is no TTY to answer a prompt — the build would
+		// hang or crash. Fail fast with an actionable message instead.
+		if (isNonInteractiveOrCI()) {
+			throw new Error(
+				`No \`${OPEN_NEXT_CONFIG_FILE_NAME}\` file was found in the project root.\n\nThis file is required for OpenNext Cloudflare builds.\nRun \`opennextjs-cloudflare migrate\` to create it, or see https://opennext.js.org/cloudflare/get-started for setup guidance.\nCommit it and re-run the build.`
+			);
+		}
+
 		const answer = await askConfirmation(
-			"Missing required `open-next.config.ts` file, do you want to create one?"
+			`Missing required \`${OPEN_NEXT_CONFIG_FILE_NAME}\` file, do you want to create one?`
 		);
 
 		if (!answer) {
-			throw new Error("The `open-next.config.ts` file is required, aborting!");
+			throw new Error(`The \`${OPEN_NEXT_CONFIG_FILE_NAME}\` file is required, aborting!`);
 		}
 
 		configPath = createOpenNextConfigFile(nextAppDir, { cache: false });

packages/cloudflare/src/cli/utils/create-open-next-config.ts

@@ -5,14 +5,16 @@ import { patchCode } from "@opennextjs/aws/build/patch/astCodePatcher.js";
 
 import { getPackageTemplatesDirPath } from "../../utils/get-package-templates-dir-path.js";
 
+export const OPEN_NEXT_CONFIG_FILE_NAME = "open-next.config.ts";
+
 /**
  * Finds the path to the OpenNext configuration file if it exists.
  *
  * @param appDir The directory to check for the open-next.config.ts file
  * @returns The full path to open-next.config.ts if it exists, undefined otherwise
  */
 export function findOpenNextConfig(appDir: string): string | undefined {
-	const openNextConfigPath = join(appDir, "open-next.config.ts");
+	const openNextConfigPath = join(appDir, OPEN_NEXT_CONFIG_FILE_NAME);
 
 	if (existsSync(openNextConfigPath)) {
 		return openNextConfigPath;
@@ -29,9 +31,9 @@ export function findOpenNextConfig(appDir: string): string | undefined {
  * @returns The path to the created configuration file
  */
 export function createOpenNextConfigFile(appDir: string, options: { cache: boolean }): string {
-	const openNextConfigPath = join(appDir, "open-next.config.ts");
+	const openNextConfigPath = join(appDir, OPEN_NEXT_CONFIG_FILE_NAME);
 
-	let content = readFileSync(join(getPackageTemplatesDirPath(), "open-next.config.ts"), "utf8");
+	let content = readFileSync(join(getPackageTemplatesDirPath(), OPEN_NEXT_CONFIG_FILE_NAME), "utf8");
 
 	if (!options.cache) {
 		content = patchCode(content, commentOutR2ImportRule);

packages/cloudflare/src/cli/utils/is-interactive.ts

@@ -0,0 +1,15 @@
+import ci from "ci-info";
+
+/**
+ * Whether the current process is running in an interactive terminal.
+ */
+export function isInteractive(): boolean {
+	return Boolean(process.stdin.isTTY && process.stdout.isTTY);
+}
+
+/**
+ * Whether prompts should be suppressed.
+ */
+export function isNonInteractiveOrCI(): boolean {
+	return !isInteractive() || ci.isCI;
+}