chore: Upgrade vitest from v2 to v4 (#1192)

* chore: upgrade vitest to v4

* docs: fix changeset description for vitest v4 upgrade

* test: configure global mock cleanup in vitest and fix spy isolation

* test: replace deprecated Vitest call matchers

* test: limit vitest discovery to src directory

* remove changeset

---------

Co-authored-by: James <james@eli.cx>

Author: 314systems

packages/cloudflare/src/api/overrides/queue/memory-queue.spec.ts

@@ -28,7 +28,6 @@ describe("MemoryQueue", () => {
 		return () => {
 			vi.useRealTimers();
 			vi.unstubAllEnvs();
-			vi.clearAllMocks();
 		};
 	});
 

packages/cloudflare/src/api/overrides/tag-cache/d1-next-tag-cache.spec.ts

@@ -33,8 +33,6 @@ describe("D1NextModeTagCache", () => {
 	let mockBatch: ReturnType<typeof vi.fn>;
 
 	beforeEach(() => {
-		vi.clearAllMocks();
-
 		// Setup mock database.
 		// All read methods now use .raw() (via #resolveTagValues) to fetch full rows.
 		mockRaw = vi.fn().mockResolvedValue([]);

packages/cloudflare/src/api/overrides/tag-cache/do-sharded-tag-cache.spec.ts

@@ -30,8 +30,6 @@ vi.mock("../../cloudflare-context", () => ({
 }));
 
 describe("DOShardedTagCache", () => {
-	afterEach(() => vi.clearAllMocks());
-
 	describe("generateShardId", () => {
 		it("should generate a shardId", () => {
 			const cache = shardedDOTagCache();

packages/cloudflare/src/api/overrides/tag-cache/kv-next-tag-cache.spec.ts

@@ -31,8 +31,6 @@ describe("KVNextModeTagCache", () => {
 	let mockPut: ReturnType<typeof vi.fn>;
 
 	beforeEach(() => {
-		vi.clearAllMocks();
-
 		// Setup mock database
 		mockGet = vi.fn();
 		mockPut = vi.fn();

packages/cloudflare/src/api/overrides/tag-cache/tag-cache-filter.spec.ts

@@ -1,5 +1,5 @@
 import { NextModeTagCache } from "@opennextjs/aws/types/overrides.js";
-import { beforeEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 import { softTagFilter, withFilter } from "./tag-cache-filter.js";
 
@@ -16,10 +16,6 @@ const mockedTagCache = {
 const filterFn = (tag: string) => tag.startsWith("valid_");
 
 describe("withFilter", () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-	});
-
 	it("should filter out tags based on writeTags", async () => {
 		const tagCache = withFilter({
 			tagCache: mockedTagCache,

packages/cloudflare/src/cli/build/patches/plugins/use-cache.spec.ts

@@ -76,8 +76,6 @@ function createSnapshot() {
         return fn(...args);
     };
 }
-
-//# sourceMappingURL=async-local-storage.js.map
 `;
 
 test("patch the createSnapshot function", () => {

packages/cloudflare/src/cli/commands/build.spec.ts

@@ -1,5 +1,5 @@
 import logger from "@opennextjs/aws/logger.js";
-import { afterEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 import { askConfirmation } from "../utils/ask-confirmation.js";
 import { createWranglerConfigFile } from "../utils/create-wrangler-config.js";
@@ -55,10 +55,6 @@ const defaultArgs = {
 };
 
 describe("buildCommand", () => {
-	afterEach(() => {
-		vi.restoreAllMocks();
-	});
-
 	it("should create wrangler config when user confirms", async () => {
 		vi.mocked(askConfirmation).mockResolvedValue(true);
 

packages/cloudflare/src/cli/commands/populate-cache.spec.ts

@@ -188,7 +188,7 @@ describe("populateCache", () => {
 					expect(ensureR2Bucket).not.toHaveBeenCalled();
 				}
 
-				expect(fetchMock).toBeCalled();
+				expect(fetchMock).toHaveBeenCalled();
 
 				for (const [input, init] of fetchMock.mock.calls) {
 					expect(input).toBe("http://localhost:12345/populate");
@@ -285,6 +285,7 @@ describe("populateCache", () => {
 		test("retries 5xx responses from the R2 worker", async () => {
 			setupMockFileSystem();
 			vi.useFakeTimers();
+			vi.spyOn(AbortSignal, "timeout");
 
 			const mockWorkerDispose = vi.fn();
 			// @ts-expect-error - Mock unstable_startWorker to return a mock worker instance

packages/cloudflare/src/cli/commands/populate-cache.ts

@@ -412,7 +412,7 @@ async function sendEntryToR2Worker(options: {
 						"x-opennext-cache-key": key,
 						"content-length": fs.statSync(filename).size.toString(),
 					},
-					body: Readable.toWeb(fs.createReadStream(filename)) as ReadableStream,
+					body: Readable.toWeb(fs.createReadStream(filename)) as unknown as ReadableStream,
 					signal: AbortSignal.timeout(60_000),
 					// @ts-expect-error - `duplex` is required for streaming request bodies in Node.js
 					duplex: "half",

packages/cloudflare/src/cli/commands/utils/utils.spec.ts

@@ -1,4 +1,4 @@
-import { afterEach, describe, expect, it, vi } from "vitest";
+import { describe, expect, it, vi } from "vitest";
 
 import { askConfirmation } from "../../utils/ask-confirmation.js";
 import { createOpenNextConfigFile, findOpenNextConfig } from "../../utils/create-open-next-config.js";
@@ -66,10 +66,6 @@ vi.mock("@opennextjs/aws/build/helper.js", () => ({
 }));
 
 describe("compileConfig", () => {
-	afterEach(() => {
-		vi.restoreAllMocks();
-	});
-
 	it("should compile config when configPath is provided and file exists", async () => {
 		mockExistsSync.mockReturnValue(true);