Files
supabase/apps/studio/scripts/smoke-server.mjs
Alaister Young 9eab4f8fbf build(studio): Vite/TanStack-Start build pipeline behind flag (stack 1/6, from #46424) (#47107)
**Stack 1/6** of the TanStack Start migration (#46424), split into
reviewable, independently-mergeable PRs.

> [!IMPORTANT]
> **Next stays the default and only active framework after this PR.**
This wires up the Vite/TanStack-Start build pipeline behind the
`STUDIO_FRAMEWORK` flag, but there are no TanStack routes yet — so the
TanStack build isn't functional or tested until later PRs in the stack.
Nothing about the Next build, dev, or deploy changes behaviourally here.

## What's in this PR
- **Dispatch:** `dev`/`build`/`start` now go through
`scripts/dispatch.js`, which runs the Next variant unless
`STUDIO_FRAMEWORK=tanstack`. The original commands are preserved as
`dev:next`/`build:next`/`start:next`.
- **Build pipeline:** `vite.config.ts`, `serve.js`, `smoke-server.mjs`,
vite/tanstack deps, `turbo.jsonc`.
- **`tsconfig.json`:** `jsx: react-jsx`, `moduleResolution: Bundler`,
`target: ES2022`. Because `include` is `**/*.ts(x)`, this re-typechecks
the whole app, so the companion adaptations below land with it.
- **Shared adaptations (companions to the tsconfig change):**
`BufferSource` casts, `packages/ui` unused-`React` import removals, etc.
- **Routing/middleware plumbing:** `next.config.ts` +
`redirects.shared.ts` (redirect rules now shared with `vercel.ts`),
`proxy.ts`/`start.ts` middleware + `hosted-api-allowlist.ts`.

## Verification
Run locally off `master`: frozen install ✓, `studio` typecheck ✓, **Next
build ✓** (compiles + generates all routes), lint ratchet ✓ ("some rules
improved"), prettier ✓.


<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **New Features**
* Added a hosted API endpoint allowlist to return 404 for non-supported
`/api/*` routes.
* Introduced a TanStack route-migration checklist and expanded TanStack
Start routing support.
* **Improvements**
* Enhanced deployment refresh/detection by tightening cookie handling
for “latest deployment” updates.
* Centralized redirect/maintenance-mode rules for consistent platform vs
self-hosted behavior.
* Improved production serving with a dedicated static + proxy server and
a post-build smoke test.
* **Dependencies**
* Updated TanStack-related packages and React Table/query tooling
versions.
* **Documentation / Chores**
* Updated formatting and tooling config; added shared build environment
parsing utilities.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

---------

Co-authored-by: Alaister Young <10985857+alaister@users.noreply.github.com>
Co-authored-by: Ivan Vasilov <vasilov.ivan@gmail.com>
2026-06-24 17:55:22 +08:00

74 lines
3.1 KiB
JavaScript

#!/usr/bin/env node
// Post-build smoke test: boot the built TanStack server handler and make a
// real request with a platform-like environment, so module-scope boot
// crashes fail the BUILD instead of the deployed function at runtime.
//
// Why this exists: the TanStack server entry eagerly imports the entire route
// tree (`loadEntries`), so every route module is evaluated the first time the
// single function handler serves a request. A bad top-level side effect in any
// one route — e.g. `createClient(process.env.SUPABASE_URL!, ...)` at module
// scope, where SUPABASE_URL is unset on platform — throws during that import
// and 500s every route, including trivial ones like /api/get-utc-time. Those
// failures only showed up at runtime on Vercel; this catches them at build.
//
// What this catches: anything that throws while the route tree is imported
// (the most common class — missing env vars read at module scope).
// What this does NOT catch: assets missing from the *Vercel function bundle*
// (e.g. libpg-query.wasm). Local node_modules still has those, so booting the
// plain `dist/server` build won't surface them — that needs booting the
// `vercel build` output. Pass that function's entry as argv[1] to reuse this
// script in a deploy-gating CI step.
import path from 'node:path'
import { fileURLToPath } from 'node:url'
const studioRoot = path.resolve(path.dirname(fileURLToPath(import.meta.url)), '..')
const serverEntry = process.argv[2]
? path.resolve(process.argv[2])
: path.join(studioRoot, 'dist/server/server.js')
// Simulate the platform function runtime: these are only set on self-hosted,
// so removing them surfaces any route that needs them at module-load time.
for (const key of ['SUPABASE_URL', 'SUPABASE_SERVICE_KEY', 'SUPABASE_SERVICE_ROLE_KEY']) {
delete process.env[key]
}
const basePath = process.env.NEXT_PUBLIC_BASE_PATH ?? ''
// A request to any route forces `loadEntries` to import the full route tree,
// so a single cheap, dependency-free endpoint is enough to exercise the boot.
const ROUTES = ['/api/get-utc-time']
console.log(`[smoke] booting ${path.relative(studioRoot, serverEntry)}`)
const { default: handler } = await import(serverEntry)
let failed = false
for (const route of ROUTES) {
const url = `http://localhost${basePath}${route}`
try {
const res = await handler.fetch(new Request(url))
if (res.status >= 500) {
failed = true
const body = await res.text().catch(() => '')
console.error(`[smoke] ✗ ${route}${res.status}\n${body.slice(0, 800)}`)
} else {
console.log(`[smoke] ✓ ${route}${res.status}`)
}
} catch (err) {
failed = true
console.error(`[smoke] ✗ ${route} threw while booting the server:\n`, err)
}
}
if (failed) {
console.error(
'\n[smoke] FAILED — the server bundle does not boot cleanly. This usually means a\n' +
'route module has a top-level side effect (e.g. createClient at module scope with a\n' +
'missing env var). Make it lazy so it only runs inside the handler.'
)
process.exit(1)
}
console.log('\n[smoke] passed — server boots and routes respond without a 5xx.')
process.exit(0)