[wrangler] Recognise a lowercase end when splitting D1 SQL statements

.changeset/d1-splitter-lowercase-end.md

@@ -0,0 +1,9 @@
+---
+"wrangler": patch
+---
+
+Recognise a lowercase `end` when splitting D1 SQL into statements
+
+`wrangler d1 execute --file` and `wrangler d1 migrations apply` only treated an uppercase `END` as the terminator of a `BEGIN`/`CASE` compound statement, even though the opening `BEGIN`/`CASE` marker is matched case-insensitively. A trigger body closed with `end;` therefore never ended, and every following statement in the file was swallowed into it and executed as one statement.
+
+The closing marker is now matched case-insensitively too.

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

@@ -328,6 +328,25 @@ describe("splitSqlQuery()", () => {
 		`);
 	});
 
+	it("should handle a lowercase compound statement END", ({ expect }) => {
+		expect(
+			splitSqlQuery(`
+	CREATE TRIGGER IF NOT EXISTS update_trigger AFTER UPDATE ON items
+	begin
+		DELETE FROM updates WHERE item_id=old.id;
+	end;
+	CREATE TABLE tasks (id INTEGER PRIMARY KEY);`)
+		).toMatchInlineSnapshot(`
+			[
+			  "CREATE TRIGGER IF NOT EXISTS update_trigger AFTER UPDATE ON items
+				begin
+					DELETE FROM updates WHERE item_id=old.id;
+				end",
+			  "CREATE TABLE tasks (id INTEGER PRIMARY KEY)",
+			]
+		`);
+	});
+
 	it("should handle compound statements for CASEs", ({ expect }) => {
 		expect(
 			splitSqlQuery(`

packages/wrangler/src/d1/splitter.ts

@@ -166,5 +166,5 @@ function isCompoundStatementStart(str: string) {
  * Returns true if the `str` ends with a compound statement `END` marker.
  */
 function isCompoundStatementEnd(str: string) {
-	return /\sEND[;\s]$/.test(str);
+	return /\sEND[;\s]$/i.test(str);
 }