Skip to content

fix: cloudflare build warning with node:sqlite - #4415

Merged
himself65 merged 2 commits into
better-auth:canaryfrom
himself65:himself65/2025/09/03/build
Sep 4, 2025
Merged

fix: cloudflare build warning with node:sqlite#4415
himself65 merged 2 commits into
better-auth:canaryfrom
himself65:himself65/2025/09/03/build

Conversation

@himself65

@himself65 himself65 commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Fixes: #4404


Summary by cubic

Prevents Cloudflare build warnings by stopping Vite/Webpack from pre-bundling node:sqlite in the Kysely adapter. Uses dynamic import hints so sqlite is loaded only at runtime. Fixes #4404.

  • Bug Fixes
    • Added /* @vite-ignore / and / webpackIgnore: true */ to the node:sqlite dynamic import inside the existing try/catch.

@himself65
himself65 requested a review from Bekacru as a code owner September 4, 2025 02:08
@vercel

vercel Bot commented Sep 4, 2025

Copy link
Copy Markdown

@himself65 is attempting to deploy a commit to the better-auth Team on Vercel.

A member of the Team first needs to authorize it.

@himself65
himself65 force-pushed the himself65/2025/09/03/build branch from 4dd8fa6 to 8bf3cbe Compare September 4, 2025 02:08
@pkg-pr-new

pkg-pr-new Bot commented Sep 4, 2025

Copy link
Copy Markdown

Open in StackBlitz

better-auth

npm i https://pkg.pr.new/better-auth/better-auth@4415

@better-auth/cli

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/cli@4415

@better-auth/expo

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/expo@4415

@better-auth/sso

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/sso@4415

@better-auth/stripe

npm i https://pkg.pr.new/better-auth/better-auth/@better-auth/stripe@4415

commit: 12a8132

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

@himself65
himself65 merged commit 14ad434 into better-auth:canary Sep 4, 2025
6 of 8 checks passed
@better-auth better-auth locked as resolved and limited conversation to collaborators Apr 1, 2026
@bytaesu bytaesu added the locked Locked conversations after being closed for 7 days label Apr 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

locked Locked conversations after being closed for 7 days

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Runtime warning on Cloudflare Edge with v1.3.8 beta

2 participants