mirror of
https://github.com/supabase/supabase.git
synced 2026-06-28 03:19:09 -04:00
9eab4f8fbf
**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 <[email protected]> Co-authored-by: Ivan Vasilov <[email protected]>
53 lines
1.9 KiB
JavaScript
53 lines
1.9 KiB
JavaScript
// Shared .env parsing for the studio build/serve scripts (dispatch.js,
|
|
// serve.js). We parse the small subset of the dotenv format we actually rely
|
|
// on — `KEY=value`, an optional `export` prefix, and surrounding single/double
|
|
// quotes — rather than taking on the `dotenv` dependency for a couple of build
|
|
// scripts. Crucially, nothing here touches `process.env`: callers decide what
|
|
// to do with the parsed values, which is what lets dispatch.js read a single
|
|
// key without leaking the whole file into the child process.
|
|
//
|
|
// dispatch.js and serve.js must agree on this format, so it lives here once.
|
|
import { readFileSync } from 'node:fs'
|
|
import path from 'node:path'
|
|
|
|
const ENV_LINE = /^\s*(?:export\s+)?([\w.-]+)\s*=\s*(.*?)\s*$/
|
|
|
|
// Parse the contents of a single env file into a plain key/value object.
|
|
export function parseEnv(content) {
|
|
const parsed = {}
|
|
for (const raw of content.split('\n')) {
|
|
const m = ENV_LINE.exec(raw)
|
|
if (!m) continue
|
|
let value = m[2]
|
|
if (
|
|
(value.startsWith('"') && value.endsWith('"')) ||
|
|
(value.startsWith("'") && value.endsWith("'"))
|
|
) {
|
|
value = value.slice(1, -1)
|
|
}
|
|
parsed[m[1]] = value
|
|
}
|
|
return parsed
|
|
}
|
|
|
|
// Read a cascade of env files from `dir`, with later files overriding earlier
|
|
// ones — matching the precedence vite/Next use
|
|
// (`.env` < `.env.local` < `.env.<mode>` < `.env.<mode>.local`). A missing file
|
|
// is skipped, but any other read error (permissions, IO) is surfaced rather
|
|
// than silently dropping that file's values. Returns the merged key/value
|
|
// object; never mutates process.env.
|
|
export function readEnvFiles(dir, files) {
|
|
const parsed = {}
|
|
for (const file of files) {
|
|
let content
|
|
try {
|
|
content = readFileSync(path.join(dir, file), 'utf8')
|
|
} catch (err) {
|
|
if (err && typeof err === 'object' && err.code === 'ENOENT') continue
|
|
throw err
|
|
}
|
|
Object.assign(parsed, parseEnv(content))
|
|
}
|
|
return parsed
|
|
}
|