chore(vitest): upgrade to Vitest 5 and share one preset across package configs

.gitignore

@@ -26,3 +26,6 @@ packages/cli-*/bin/
 
 # Transient render dir created by packages/api/scripts/generated-output-sync.unit.test.ts
 packages/api/.generated-output-sync-*/
+
+# Vitest 5 artifact directory (blob reports, attachments, html/json output)
+.vitest/

AGENTS.md

@@ -47,6 +47,17 @@ Expected exceptions:
 }
 ```
 
+**vitest.config.ts:**
+
+Package configs build on the repo-root `vitest.shared.ts` preset. `definePackageConfig` merges the
+shared defaults (bun export-condition resolution for workspace packages, v8 coverage,
+`passWithNoTests`, console output only from failing tests), and `testProject("unit" | "integration" |
+"e2e" | "live", overrides)` declares one inline project per test kind with the repo's file-suffix
+convention baked in. Package-specific settings such as timeouts, setup files, serial execution, or
+Vite plugins go in the overrides. The root `vitest.config.mts` loads every package config as a nested
+project group, so `bun --bun vitest run --project '*(unit)'` from the repo root runs one kind across
+all workspaces while `pnpm test:unit` inside a package still works standalone.
+
 ## Config Naming Vocabulary
 
 `@supabase/config` and its CLI consumer use three settled names:

apps/cli-e2e/vitest.config.ts

@@ -1,22 +1,29 @@
-import { defineConfig } from "vitest/config";
 import { BaseSequencer, type TestSpecification } from "vitest/node";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
-export default defineConfig({
+export default definePackageConfig({
   test: {
-    passWithNoTests: true,
-    include: ["**/*.e2e.test.ts"],
-    exclude: ["**/node_modules/**"],
-    fileParallelism: false,
-    maxWorkers: 1,
-    globalSetup: ["tests/setup.ts"],
-    testTimeout: 60_000,
-    hookTimeout: 30_000,
+    // Replay fixtures are shared across files, so run them in a deterministic
+    // lexicographic order. `sequence.sequencer` is a run-level option: it
+    // applies to standalone runs of this package, not when the repo root loads
+    // this config as a project.
     sequence: {
       sequencer: class extends BaseSequencer {
         override async sort(files: TestSpecification[]) {
           return [...files].sort((a, b) => a.moduleId.localeCompare(b.moduleId));
         }
       },
     },
+    projects: [
+      testProject("e2e", {
+        test: {
+          fileParallelism: false,
+          maxWorkers: 1,
+          globalSetup: ["tests/setup.ts"],
+          testTimeout: 60_000,
+          hookTimeout: 30_000,
+        },
+      }),
+    ],
   },
 });

apps/cli/src/shared/issue/issue-template-contract.unit.test.ts

@@ -35,7 +35,7 @@ function isBodyItem(value: unknown): value is IssueFormBodyItem {
 }
 
 function issueTemplateDir() {
-  return resolve(process.cwd(), "../../.github/ISSUE_TEMPLATE");
+  return resolve(import.meta.dirname, "../../../../../.github/ISSUE_TEMPLATE");
 }
 
 function readTemplate(template: string): ReadonlyArray<IssueFormBodyItem> {

apps/cli/vitest.config.ts

@@ -1,7 +1,8 @@
 import { readFileSync } from "node:fs";
-import { defaultClientConditions, defaultServerConditions } from "vite";
-import { defineConfig } from "vitest/config";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
+// `src/shared/services/dockerfile-images.ts` imports the Go CLI's Dockerfile
+// with Bun's `{ type: "text" }` import attribute; Vite needs a loader for it.
 function dockerfileTextPlugin() {
   return {
     name: "dockerfile-text-loader",
@@ -16,31 +17,17 @@ function dockerfileTextPlugin() {
   };
 }
 
-// Workspace packages such as @supabase/config publish a `bun` export
-// condition pointing at their TypeScript source (see
-// packages/config/package.json's `exports` map); without it, Vite's resolver
-// falls through to the `default` condition and loads the built `dist/*.js`
-// output instead — which is stale, or missing entirely on a fresh clone
-// before the package has been built. Extending (not replacing) Vite's
-// default condition lists keeps every other package's exports resolution
-// unchanged. Required on every inline `test.projects` entry below too:
-// Vitest builds a separate Vite config per project and does not inherit
-// these from the root config (see PR #6366 finding 0).
-const workspacePackageResolve = { conditions: [...defaultClientConditions, "bun"] };
-const workspacePackageSsrResolve = { conditions: [...defaultServerConditions, "bun"] };
+const e2eRuntime = {
+  globalSetup: ["tests/e2e-global-setup.ts"],
+  setupFiles: ["tests/e2e-setup.ts"],
+  testTimeout: 120_000,
+  hookTimeout: 120_000,
+};
 
-export default defineConfig({
-  resolve: workspacePackageResolve,
-  ssr: { resolve: workspacePackageSsrResolve },
+export default definePackageConfig({
   plugins: [dockerfileTextPlugin()],
   test: {
-    passWithNoTests: true,
     coverage: {
-      enabled: false,
-      provider: "v8",
-      include: ["src/**/*.ts"],
-      reporter: ["text", "lcov"],
-      reportsDirectory: "coverage",
       exclude: [
         "tests/**",
         "scripts/**",
@@ -56,79 +43,26 @@ export default defineConfig({
       ],
     },
     projects: [
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        plugins: [dockerfileTextPlugin()],
+      testProject("unit", { test: { env: { FORCE_COLOR: "1" } } }),
+      testProject("integration"),
+      // Stackless e2e: black-box CLI subprocesses against an isolated temporary
+      // home, safe under file-level parallelism.
+      testProject("e2e", { test: e2eRuntime }),
+      // Stack-backed e2e: `*.stack.e2e.test.ts` files start a local stack or
+      // run Docker containers and run one at a time (ADR 0024).
+      testProject("e2e-stack", { test: e2eRuntime }),
+      // Live tests run against one provisioned project on the configured
+      // platform. They are never part of the default unit/integration/e2e
+      // loop; an explicit run fails fast when required configuration is absent.
+      testProject("live", {
         test: {
-          name: "unit",
-          include: ["**/*.unit.test.ts"],
-          env: { FORCE_COLOR: "1" },
-        },
-      },
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        plugins: [dockerfileTextPlugin()],
-        test: {
-          name: "integration",
-          include: ["**/*.integration.test.ts"],
-        },
-      },
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        plugins: [dockerfileTextPlugin()],
-        test: {
-          // Stackless e2e: black-box CLI subprocesses against an isolated
-          // temporary home. Safe under file-level parallelism (see the flake
-          // policy in AGENTS.md), so Vitest's default parallelism applies.
-          name: "e2e",
-          include: ["**/*.e2e.test.ts"],
-          exclude: ["**/*.stack.e2e.test.ts", "**/node_modules/**"],
-          globalSetup: ["tests/e2e-global-setup.ts"],
-          setupFiles: ["tests/e2e-setup.ts"],
-          testTimeout: 120_000,
-          hookTimeout: 120_000,
-        },
-      },
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        plugins: [dockerfileTextPlugin()],
-        test: {
-          // Stack-backed e2e: files that start a local Supabase stack or run
-          // Docker containers claim machine-level resources, so they run one
-          // file at a time. Vitest schedules this serial group after the
-          // parallel projects finish. Opt in with the `*.stack.e2e.test.ts`
-          // suffix (ADR 0024).
-          name: "e2e-stack",
-          include: ["**/*.stack.e2e.test.ts"],
-          fileParallelism: false,
-          maxWorkers: 1,
-          globalSetup: ["tests/e2e-global-setup.ts"],
-          setupFiles: ["tests/e2e-setup.ts"],
-          testTimeout: 120_000,
-          hookTimeout: 120_000,
-        },
-      },
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        plugins: [dockerfileTextPlugin()],
-        test: {
-          // Live tests run against one provisioned project on the configured
-          // platform. They are never part of the default unit/integration/e2e
-          // loop; an explicit run fails fast when required configuration is absent.
-          name: "live",
-          include: ["**/*.live.test.ts"],
           fileParallelism: false,
           maxWorkers: 1,
           globalSetup: ["tests/live-global-setup.ts"],
           testTimeout: 300_000,
           hookTimeout: 300_000,
         },
-      },
+      }),
     ],
   },
 });

knip.json

@@ -3,10 +3,11 @@
   "exclude": ["catalogReferences"],
   "workspaces": {
     ".": {
-      "entry": [".github/scripts/**/*.ts", "tools/*.ts", "tools/release/*.ts"],
+      "entry": [".github/scripts/**/*.ts", "tools/*.ts", "tools/release/*.ts", "vitest.config.mts"],
       "ignore": [".repos/**", "apps/cli-go/**"],
       "ignoreBinaries": ["go"],
-      "ignoreDependencies": ["verdaccio"]
+      "ignoreDependencies": ["verdaccio"],
+      "vitest": false
     },
     "apps/cli": {
       "entry": [

package.json

@@ -12,6 +12,7 @@
     "test:unit": "pnpm exec turbo run test:unit:run --filter=!@supabase/cli-go --",
     "test:integration": "pnpm exec turbo run test:integration:run --",
     "test:e2e": "pnpm exec turbo run supabase#build && pnpm exec turbo run test:e2e:run --only --concurrency=1 --",
+    "test:vitest": "bun --bun vitest run --project '!supabase (live)'",
     "check:all": "pnpm exec turbo run types:check lint:check fmt:check knip:check lint:effect:check",
     "fix:all": "pnpm exec turbo run lint:fix fmt:fix knip:fix && pnpm run lint:effect:fix",
     "lint:check": "oxlint --config .oxlintrc.json",
@@ -40,7 +41,8 @@
     "turbo": "catalog:",
     "typescript": "catalog:",
     "verdaccio": "^6.10.0",
-    "vite": "^8.2.2"
+    "vite": "^8.2.2",
+    "vitest": "catalog:"
   },
   "devEngines": {
     "packageManager": {

packages/api/vitest.config.ts

@@ -1,23 +1,5 @@
-import { defineConfig } from "vitest/config";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
-export default defineConfig({
-  test: {
-    passWithNoTests: true,
-    coverage: {
-      enabled: false,
-      provider: "v8",
-      clean: false,
-      include: ["src/**/*.ts"],
-      reporter: ["text", "lcov"],
-      reportsDirectory: "coverage",
-    },
-    projects: [
-      {
-        test: {
-          name: "unit",
-          include: ["**/*.unit.test.ts"],
-        },
-      },
-    ],
-  },
+export default definePackageConfig({
+  test: { projects: [testProject("unit")] },
 });

packages/cli-test-helpers/vitest.config.ts

@@ -1,23 +1,5 @@
-import { defineConfig } from "vitest/config";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
-export default defineConfig({
-  test: {
-    passWithNoTests: true,
-    coverage: {
-      enabled: false,
-      provider: "v8",
-      clean: false,
-      include: ["src/**/*.ts"],
-      reporter: ["text", "lcov"],
-      reportsDirectory: "coverage",
-    },
-    projects: [
-      {
-        test: {
-          name: "unit",
-          include: ["**/*.unit.test.ts"],
-        },
-      },
-    ],
-  },
+export default definePackageConfig({
+  test: { projects: [testProject("unit")] },
 });

packages/config/vitest.config.ts

@@ -1,40 +1,5 @@
-import { defaultClientConditions, defaultServerConditions } from "vite";
-import { defineConfig } from "vitest/config";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
-// This package publishes a `bun` export condition pointing at its
-// TypeScript source (see package.json's `exports` map); without it, Vite's
-// resolver falls through to the `default` condition and loads the built
-// `dist/*.js` output instead — stale, or missing entirely on a fresh clone
-// before the package has been built. Extending (not replacing) Vite's
-// default condition lists keeps every other package's exports resolution
-// unchanged. Required on every inline `test.projects` entry too: Vitest
-// builds a separate Vite config per project and does not inherit these from
-// the root config (see PR #6366 finding 0).
-const workspacePackageResolve = { conditions: [...defaultClientConditions, "bun"] };
-const workspacePackageSsrResolve = { conditions: [...defaultServerConditions, "bun"] };
-
-export default defineConfig({
-  resolve: workspacePackageResolve,
-  ssr: { resolve: workspacePackageSsrResolve },
-  test: {
-    passWithNoTests: true,
-    coverage: {
-      enabled: false,
-      provider: "v8",
-      clean: false,
-      include: ["src/**/*.ts"],
-      reporter: ["text", "lcov"],
-      reportsDirectory: "coverage",
-    },
-    projects: [
-      {
-        resolve: workspacePackageResolve,
-        ssr: { resolve: workspacePackageSsrResolve },
-        test: {
-          name: "unit",
-          include: ["**/*.unit.test.ts"],
-        },
-      },
-    ],
-  },
+export default definePackageConfig({
+  test: { projects: [testProject("unit")] },
 });

packages/process-compose/vitest.config.ts

@@ -1,29 +1,5 @@
-import { defineConfig } from "vitest/config";
+import { definePackageConfig, testProject } from "../../vitest.shared.ts";
 
-export default defineConfig({
-  test: {
-    passWithNoTests: true,
-    coverage: {
-      enabled: false,
-      provider: "v8",
-      clean: false,
-      include: ["src/**/*.ts"],
-      reporter: ["text", "lcov"],
-      reportsDirectory: "coverage",
-    },
-    projects: [
-      {
-        test: {
-          name: "unit",
-          include: ["**/*.unit.test.ts"],
-        },
-      },
-      {
-        test: {
-          name: "integration",
-          include: ["**/*.integration.test.ts"],
-        },
-      },
-    ],
-  },
+export default definePackageConfig({
+  test: { projects: [testProject("unit"), testProject("integration")] },
 });

packages/stack/src/ApiProxy.ts

@@ -172,8 +172,11 @@ function makeProxyHandler(
       const noBodyMethods = new Set(["GET", "HEAD", "OPTIONS", "TRACE"]);
       const contentType = Option.getOrUndefined(Headers.get(req.headers, "content-type"));
 
+      const hasBody =
+        Option.isSome(Headers.get(req.headers, "transfer-encoding")) ||
+        Option.getOrElse(Headers.get(req.headers, "content-length"), () => "0") !== "0";
       let body: HttpBody.HttpBody;
-      if (noBodyMethods.has(req.method)) {
+      if (noBodyMethods.has(req.method) || !hasBody) {
         body = HttpBody.empty;
       } else if (opts.retryColdStart === true) {
         // Buffer the body so the request can be safely re-sent if we retry.