fix(dashboard): Use DB-less Better Auth initializer#456
Merged
Conversation
Use the minimal Better Auth entrypoint for dashboard auth so the Vercel production bundle does not pull in Kysely adapter and migration code that is unused by the cookie-backed dashboard flow. The dashboard already avoids passing a database option and stores OAuth state and cached sessions in cookies, so the minimal initializer matches the intended auth mode. Fixes GH-455 Co-Authored-By: GPT-5 Codex <codex@openai.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Switch the Junior dashboard to Better Auth's minimal initializer so DB-less dashboard auth no longer pulls Kysely adapter and migration code into Nitro production bundles. This matches the existing cookie-backed OAuth state and session cache configuration, and fixes the Rolldown missing-export failure from Better Auth's unused SQLite dialect chunks.
DB-less Auth Path
The dashboard still passes no
databaseoption and continues to store OAuth state and encrypted session cache in cookies. The auth config test now mocksbetter-auth/minimalso that contract stays explicit.Bundle Verification
Verified a packed dashboard tarball in a throwaway Nitro/Vercel app with
@sentry/junior@0.58.0:nitro buildsucceeds, the output containsbetter-auth/dist/context/init-minimal.mjsand@better-auth/memory-adapter, and an output scan found no@better-auth/kysely-adapter,DEFAULT_MIGRATION, orfrom "kysely"references.Fixes #455