fix(wrangler): show error when D1 migration commands run without config file (#11711)

* fix(wrangler): show error when D1 migration commands run without config file

Previously, running `wrangler d1 migrations apply`, `list`, or `create`
in a directory without a wrangler.toml or wrangler.json would silently
exit with no feedback. Now these commands display a clear error message
guiding users to create a configuration file.

Fixes #11709

* fix(wrangler): show error when D1 migration commands run without config file

Previously, running `wrangler d1 migrations apply`, `list`, or `create`
in a directory without a Wrangler configuration file would silently
exit with no feedback. Now these commands display a clear error message
guiding users to create a configuration file.

Fixes #11709

Author: southpolesteve

.changeset/itchy-squids-design.md

@@ -0,0 +1,9 @@
+---
+"wrangler": patch
+---
+
+Show an error when D1 migration commands are run without a configuration file
+
+Previously, running `wrangler d1 migrations apply`, `wrangler d1 migrations list`, or `wrangler d1 migrations create` in a directory without a Wrangler configuration file would silently exit with no feedback. Now these commands display a clear error message:
+
+"No configuration file found. Create a wrangler.jsonc file to define your D1 database."

packages/wrangler/src/__tests__/d1/migrate.test.ts

@@ -32,6 +32,15 @@ describe("migrate", () => {
 				runWrangler("d1 migrations create test some-message --local DATABASE")
 			).rejects.toThrowError(`Unknown argument: local`);
 		});
+
+		it("should error when no config file is present", async () => {
+			setIsTTY(false);
+			await expect(
+				runWrangler("d1 migrations create DATABASE test-migration")
+			).rejects.toThrowError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		});
 	});
 
 	describe("apply", () => {
@@ -69,6 +78,15 @@ describe("migrate", () => {
 			).rejects.toThrowError(`No migrations present at <cwd>/migrations.`);
 		});
 
+		it("should error when no config file is present", async () => {
+			setIsTTY(false);
+			await expect(
+				runWrangler("d1 migrations apply DATABASE")
+			).rejects.toThrowError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		});
+
 		it("should reject the use of --preview with --local", async () => {
 			setIsTTY(false);
 			writeWranglerConfig({
@@ -209,6 +227,15 @@ Your database may not be available to serve requests during the migration, conti
 			).rejects.toThrowError(`No migrations present at <cwd>/migrations.`);
 		});
 
+		it("should error when no config file is present", async () => {
+			setIsTTY(false);
+			await expect(
+				runWrangler("d1 migrations list DATABASE")
+			).rejects.toThrowError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		});
+
 		it("should use the custom migrations folder when provided", async () => {
 			setIsTTY(false);
 			writeWranglerConfig({

packages/wrangler/src/d1/migrations/apply.ts

@@ -68,6 +68,12 @@ export const d1MigrationsApplyCommand = createCommand({
 	},
 	positionalArgs: ["database"],
 	async handler({ database, local, remote, persistTo, preview }, { config }) {
+		if (!config.configPath) {
+			throw new UserError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		}
+
 		const databaseInfo = getDatabaseInfoFromConfig(config, database);
 
 		if (!databaseInfo && remote) {
@@ -76,10 +82,6 @@ export const d1MigrationsApplyCommand = createCommand({
 			);
 		}
 
-		if (!config.configPath) {
-			return;
-		}
-
 		const migrationsPath = await getMigrationsPath({
 			projectPath: path.dirname(config.configPath),
 			migrationsFolderPath:

packages/wrangler/src/d1/migrations/create.ts

@@ -40,17 +40,19 @@ export const d1MigrationsCreateCommand = createCommand({
 	},
 	positionalArgs: ["database", "message"],
 	async handler({ database, message }, { config }) {
+		if (!config.configPath) {
+			throw new UserError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		}
+
 		const databaseInfo = getDatabaseInfoFromConfig(config, database);
 		if (!databaseInfo) {
 			throw new UserError(
 				`Couldn't find a D1 DB with the name or binding '${database}' in your ${configFileName(config.configPath)} file.`
 			);
 		}
 
-		if (!config.configPath) {
-			return;
-		}
-
 		const migrationsPath = await getMigrationsPath({
 			projectPath: path.dirname(config.configPath),
 			migrationsFolderPath:

packages/wrangler/src/d1/migrations/list.ts

@@ -54,17 +54,19 @@ export const d1MigrationsListCommand = createCommand({
 			await requireAuth({});
 		}
 
+		if (!config.configPath) {
+			throw new UserError(
+				"No configuration file found. Create a wrangler.jsonc file to define your D1 database."
+			);
+		}
+
 		const databaseInfo = getDatabaseInfoFromConfig(config, database);
 		if (!databaseInfo && remote) {
 			throw new UserError(
 				`Couldn't find a D1 DB with the name or binding '${database}' in your ${configFileName(config.configPath)} file.`
 			);
 		}
 
-		if (!config.configPath) {
-			return;
-		}
-
 		const migrationsPath = await getMigrationsPath({
 			projectPath: path.dirname(config.configPath),
 			migrationsFolderPath: