Do not report "d1 execute" command file missing error to Sentry (#9162)

Author: petebacondarwin

.changeset/social-cups-roll.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+Do not report "d1 execute" command file missing error to Sentry

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

@@ -142,6 +142,24 @@ describe("execute", () => {
 		);
 	});
 
+	it("should throw a UserError if file does not exist", async () => {
+		setIsTTY(false);
+		writeWranglerConfig({
+			d1_databases: [
+				{ binding: "DATABASE", database_name: "db", database_id: "xxxx" },
+			],
+		});
+
+		await expect(runWrangler(`d1 execute db --file missing.sql --local --json`))
+			.rejects.toThrowErrorMatchingInlineSnapshot(`
+			[Error: {
+			  "error": {
+			    "text": "Unable to read SQL text file /"missing.sql/". Please check the file path and try again."
+			  }
+			}]
+		`);
+	});
+
 	it("should show banner by default", async () => {
 		setIsTTY(false);
 		writeWranglerConfig({

packages/wrangler/src/d1/execute.ts

@@ -598,9 +598,17 @@ function shorten(query: string | undefined, length: number) {
 }
 
 async function checkForSQLiteBinary(filename: string) {
-	const fd = await fs.open(filename, "r");
 	const buffer = Buffer.alloc(15);
-	await fd.read(buffer, 0, 15);
+
+	try {
+		const fd = await fs.open(filename, "r");
+		await fd.read(buffer, 0, 15);
+	} catch (e) {
+		throw new UserError(
+			`Unable to read SQL text file "${filename}". Please check the file path and try again.`
+		);
+	}
+
 	if (buffer.toString("utf8") === "SQLite format 3") {
 		throw new UserError(
 			"Provided file is a binary SQLite database file instead of an SQL text file. The execute command can only process SQL text files. Please export an SQL file from your SQLite database and try again."