fix(wrangler): handle d1 export with multiple tables

this was clearly intended to work, and did work in earlier version, but has been broken.
added a test and fixed the command args.

Author: flostellbrink

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

@@ -191,6 +191,46 @@ describe("export", () => {
 			`[Error: Found a database with name or binding D1 but it is missing a database_id, which is needed for operations on remote resources. Please create the remote D1 database by deploying your project or running 'wrangler d1 create D1'.]`
 		);
 	});
+
+	it("should handle multiple tables", async ({ expect }) => {
+		setIsTTY(false);
+		writeWranglerConfig({
+			d1_databases: [
+				{ binding: "DATABASE", database_name: "db", database_id: "xxxx" },
+			],
+		});
+
+		// Fill with data
+		fs.writeFileSync(
+			"data.sql",
+			`
+				CREATE TABLE foo(id INTEGER PRIMARY KEY, value TEXT);
+				CREATE TABLE bar(id INTEGER PRIMARY KEY, value TEXT);
+				CREATE TABLE baz(id INTEGER PRIMARY KEY, value TEXT);
+				INSERT INTO foo (value) VALUES ('xxx'),('yyy'),('zzz');
+				INSERT INTO bar (value) VALUES ('aaa'),('bbb'),('ccc');
+				INSERT INTO baz (value) VALUES ('111'),('222'),('333');
+			`
+		);
+		await runWrangler("d1 execute db --file data.sql");
+
+		await runWrangler(
+			"d1 export db --output test-multiple.sql --table foo --table baz"
+		);
+		expect(fs.readFileSync("test-multiple.sql", "utf8")).toBe(
+			[
+				"PRAGMA defer_foreign_keys=TRUE;",
+				"CREATE TABLE foo(id INTEGER PRIMARY KEY, value TEXT);",
+				"INSERT INTO \"foo\" VALUES(1,'xxx');",
+				"INSERT INTO \"foo\" VALUES(2,'yyy');",
+				"INSERT INTO \"foo\" VALUES(3,'zzz');",
+				"CREATE TABLE baz(id INTEGER PRIMARY KEY, value TEXT);",
+				"INSERT INTO \"baz\" VALUES(1,'111');",
+				"INSERT INTO \"baz\" VALUES(2,'222');",
+				"INSERT INTO \"baz\" VALUES(3,'333');",
+			].join("\n")
+		);
+	});
 });
 
 function mockResponses() {

packages/wrangler/src/d1/export.ts

@@ -50,6 +50,7 @@ export const d1ExportCommand = createCommand({
 		table: {
 			type: "string",
 			description: "Specify which tables to include in export",
+			array: true,
 		},
 		"no-schema": {
 			type: "boolean",
@@ -91,11 +92,7 @@ export const d1ExportCommand = createCommand({
 		}
 
 		// Allow multiple --table x --table y flags or none
-		const tables: string[] = table
-			? Array.isArray(table)
-				? table
-				: [table]
-			: [];
+		const tables = table ?? [];
 
 		if (remote) {
 			return await exportRemotely(config, name, output, tables, !schema, !data);