fix: add web globals preload for Bun tests to fix Request polyfill issue

Author: brandonkachen

bunfig.toml

@@ -7,4 +7,4 @@ linkWorkspacePackages = true
 [test]
 # Exclude test repositories, integration tests, and Playwright e2e tests from test execution by default
 exclude = ["evals/test-repos/**", "**/*.integration.test.*", "web/src/__tests__/e2e/**"]
-preload = ["./sdk/test/setup-env.ts", "./test/setup-bigquery-mocks.ts"]
+preload = ["./sdk/test/setup-env.ts", "./test/setup-bigquery-mocks.ts", "./web/test/setup-globals.ts"]

web/src/app/api/orgs/[orgId]/billing/portal/__tests__/org-billing-portal.test.ts

@@ -1,13 +1,9 @@
 import { describe, expect, mock, test } from 'bun:test'
-import { NextRequest } from 'next/server'
 
 import type { Logger } from '@codebuff/common/types/contracts/logger'
 
 import { postOrgBillingPortal } from '../_post'
 
-// NextRequest import above sets up the global Request polyfill needed by NextResponse
-void NextRequest
-
 import type {
   CreateBillingPortalSessionFn,
   GetMembershipFn,

web/src/app/api/user/billing-portal/__tests__/billing-portal.test.ts

@@ -1,13 +1,9 @@
 import { describe, expect, mock, test } from 'bun:test'
-import { NextRequest } from 'next/server'
 
 import type { Logger } from '@codebuff/common/types/contracts/logger'
 
 import { postBillingPortal } from '../_post'
 
-// NextRequest import above sets up the global Request polyfill needed by NextResponse
-void NextRequest
-
 import type { CreateBillingPortalSessionFn, GetSessionFn, Session } from '../_post'
 
 const createMockLogger = (errorFn = mock(() => {})): Logger => ({

web/test/setup-globals.ts

@@ -0,0 +1,33 @@
+/**
+ * Polyfill web globals for Bun tests that import Next.js server modules.
+ *
+ * Next.js's `next/server` module (NextRequest, NextResponse) expects the
+ * standard web globals (Request, Response, Headers, fetch) to exist.
+ * Bun provides these in its runtime, but they may not be available at
+ * module load time during tests.
+ *
+ * This preload script ensures these globals are set up before any test
+ * modules are imported.
+ */
+
+// Bun has built-in support for web APIs, but we need to ensure they're
+// available on globalThis for Next.js server modules
+if (typeof globalThis.Request === 'undefined') {
+  // @ts-expect-error - Bun provides these but TypeScript may not know
+  globalThis.Request = Request
+}
+
+if (typeof globalThis.Response === 'undefined') {
+  // @ts-expect-error - Bun provides these but TypeScript may not know
+  globalThis.Response = Response
+}
+
+if (typeof globalThis.Headers === 'undefined') {
+  // @ts-expect-error - Bun provides these but TypeScript may not know
+  globalThis.Headers = Headers
+}
+
+if (typeof globalThis.fetch === 'undefined') {
+  // @ts-expect-error - Bun provides these but TypeScript may not know
+  globalThis.fetch = fetch
+}