Add local d1 migrations (#2260)

* fix: make it possible to use a local db for migrations

* Create violet-lamps-cry.md

* Update execute.tsx

* Update execute.tsx

* Update apply.tsx

* Update packages/wrangler/src/d1/migrations/apply.tsx

Co-authored-by: Somhairle MacLeòid <samuel@macleod.space>

* Update apply.tsx

* Update helpers.ts

* Update execute.tsx

* add isInteractiveAndNotCI

* Revert "add isInteractiveAndNotCI"

This reverts commit 02cb33f.

* use isCI

Co-authored-by: Somhairle MacLeòid <samuel@macleod.space>

.changeset/violet-lamps-cry.md

@@ -0,0 +1,7 @@
+---
+"wrangler": patch
+---
+
+fix: make it possible to use a local db for d1 migrations
+
+As of this change, wrangler's d1 migrations commands now accept `local` and `persist-to` as flags, so migrations can run against the local d1 db.

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

@@ -5,6 +5,8 @@ import Table from "ink-table";
 import React from "react";
 import { withConfig } from "../../config";
 import { confirm } from "../../dialogs";
+import { CI } from "../../is-ci";
+import isInteractive from "../../is-interactive";
 import { logger } from "../../logger";
 import { requireAuth } from "../../user";
 import { createBackup } from "../backups";
@@ -21,7 +23,17 @@ import type { BaseSqlExecuteArgs } from "../execute";
 import type { Argv } from "yargs";
 
 export function ApplyOptions(yargs: Argv): Argv<BaseSqlExecuteArgs> {
-	return Database(yargs);
+	return Database(yargs)
+		.option("local", {
+			describe:
+				"Execute commands/files against a local DB for use with wrangler dev --local",
+			type: "boolean",
+		})
+		.option("persist-to", {
+			describe: "Specify directory to use for local persistence (for --local)",
+			type: "string",
+			requiresArg: true,
+		});
 }
 
 export const ApplyHandler = withConfig<BaseSqlExecuteArgs>(
@@ -88,8 +100,7 @@ export const ApplyHandler = withConfig<BaseSqlExecuteArgs>(
 			return;
 		}
 
-		const isInteractive = process.stdout.isTTY;
-		if (isInteractive) {
+		if (isInteractive() && !CI.isCI()) {
 			const ok = await confirm(
 				`About to apply ${unappliedMigrations.length} migration(s)\n` +
 					"Your database may not be available to serve requests during the migration, continue?",
@@ -121,7 +132,7 @@ export const ApplyHandler = withConfig<BaseSqlExecuteArgs>(
 					local,
 					config,
 					database,
-					undefined,
+					isInteractive() && !CI.isCI(),
 					persistTo,
 					undefined,
 					query

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

@@ -1,6 +1,8 @@
 import fs from "node:fs";
 import path from "path";
 import { confirm } from "../../dialogs";
+import { CI } from "../../is-ci";
+import isInteractive from "../../is-interactive";
 import { logger } from "../../logger";
 import { DEFAULT_MIGRATION_PATH } from "../constants";
 import { executeSql } from "../execute";
@@ -79,7 +81,7 @@ const listAppliedMigrations = async (
 		local,
 		config,
 		name,
-		undefined,
+		isInteractive() && !CI.isCI(),
 		persistTo,
 		undefined,
 		Query
@@ -130,7 +132,7 @@ export const initMigrationsTable = async (
 		local,
 		config,
 		name,
-		undefined,
+		isInteractive() && !CI.isCI(),
 		persistTo,
 		undefined,
 		`

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

@@ -16,7 +16,17 @@ import type { BaseSqlExecuteArgs } from "../execute";
 import type { Argv } from "yargs";
 
 export function ListOptions(yargs: Argv): Argv<BaseSqlExecuteArgs> {
-	return Database(yargs);
+	return Database(yargs)
+		.option("local", {
+			describe:
+				"Execute commands/files against a local DB for use with wrangler dev",
+			type: "boolean",
+		})
+		.option("persist-to", {
+			describe: "Specify directory to use for local persistence (for --local)",
+			type: "string",
+			requiresArg: true,
+		});
 }
 
 export const ListHandler = withConfig<BaseSqlExecuteArgs>(