From 030eb5b89ace3c68029e3f06ccc30b9cbab236f3 Mon Sep 17 00:00:00 2001
From: Adam Daley <admdly@users.noreply.github.com>
Date: Tue, 17 Mar 2026 12:50:11 +0000
Subject: [PATCH 1/2] Apply suggested fix to test/utils/d1-mock.ts from Copilot
 Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
---
 test/utils/d1-mock.ts | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/test/utils/d1-mock.ts b/test/utils/d1-mock.ts
index 9195267..6a2a901 100644
--- a/test/utils/d1-mock.ts
+++ b/test/utils/d1-mock.ts
@@ -50,6 +50,10 @@ function isVersionInRange(
   );
 }
 
+function notImplementedInMock(methodName: string): Error {
+  return new Error(`MockD1Database.${methodName}() is not implemented for tests`);
+}
+
 export class MockD1Database implements D1Database {
   private alerts: DatabaseAlert[] = [];
 
@@ -288,28 +292,28 @@ export class MockD1Database implements D1Database {
   }
 
   dump(): Promise<ArrayBuffer> {
-    throw new Error("dump() not implemented in mock");
+    throw notImplementedInMock("dump");
   }
 
   batch<T = unknown>(
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     _statements: D1PreparedStatement[]
   ): Promise<D1Result<T>[]> {
-    throw new Error("batch() not implemented in mock");
+    throw notImplementedInMock("batch");
   }
 
   exec(
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     _query: string
   ): Promise<D1ExecResult> {
-    throw new Error("exec() not implemented in mock");
+    throw notImplementedInMock("exec");
   }
 
   withSession(
     // eslint-disable-next-line @typescript-eslint/no-unused-vars
     _constraintOrBookmark?: string
   ): D1DatabaseSession {
-    throw new Error("withSession() not implemented in mock");
+    throw notImplementedInMock("withSession");
   }
 }
 

From a8f2b879eb837b7ddc7a2619607caf8ca2e5d91e Mon Sep 17 00:00:00 2001
From: Adam Daley <admdly@users.noreply.github.com>
Date: Tue, 17 Mar 2026 12:50:11 +0000
Subject: [PATCH 2/2] Apply suggested fix to test/utils/d1-mock.ts from Copilot
 Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
---
 test/utils/d1-mock.ts | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/test/utils/d1-mock.ts b/test/utils/d1-mock.ts
index 6a2a901..04b16f1 100644
--- a/test/utils/d1-mock.ts
+++ b/test/utils/d1-mock.ts
@@ -20,8 +20,8 @@ function compareVersions(a: string, b: string): number {
   const length = Math.max(aParts.length, bParts.length);
 
   for (let i = 0; i < length; i++) {
-    const aNum = Number.isNaN(aParts[i]) ? 0 : aParts[i] ?? 0;
-    const bNum = Number.isNaN(bParts[i]) ? 0 : bParts[i] ?? 0;
+    const aNum = aParts[i] ?? 0;
+    const bNum = bParts[i] ?? 0;
 
     if (aNum < bNum) {
       return -1;