Skip to content

feat(drizzle): add node-sqlite - #1206

Open
sacrosanctic wants to merge 3 commits into
sveltejs:mainfrom
sacrosanctic:double-rabbit
Open

feat(drizzle): add node-sqlite#1206
sacrosanctic wants to merge 3 commits into
sveltejs:mainfrom
sacrosanctic:double-rabbit

Conversation

@sacrosanctic

@sacrosanctic sacrosanctic commented Jul 31, 2026

Copy link
Copy Markdown
Contributor
image

Closes #

Description

Checklist

  • Update snapshots (if applicable)
  • Add a changeset (if applicable)
  • Allow maintainers to edit this PR
  • I care about what I'm doing, no matter the tool I use (Notepad, Sublime, VSCode, AI...)

@pkg-svelte-dev

pkg-svelte-dev Bot commented Jul 31, 2026

Copy link
Copy Markdown

Install the latest version of sv from c83fc7b:

pnpm add https://pkg.svelte.dev/sv/c/c83fc7b3d89a9d9a6f803ba4ba86a1d5f50bf9bb

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/cli/pr/1206

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Jul 31, 2026

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: c83fc7b

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
sv Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@AdrianGonz97

Copy link
Copy Markdown
Member

huh, well TIL about node:sqlite!

however it appears that the api is not stable yet and is currently an RC as of node@24.15.0. adding support for an 'unstable' package hasn't really stopped us in the past (e.g. drizzle), so I'm not opposed to this addition.

the only question is: what is the current minimum node version that we support?

outside of seeing what we test against in CI:

node-version: 22

i don't see it documented anywhere. perhaps we should bump this up as part of the v1 release?

@sacrosanctic

Copy link
Copy Markdown
Contributor Author

Node version is dictated by the svelte and kit repo.

I didn't realize it is still experimental in node 22. This is based around a convo I had in discord and was curious what it would look like in code.

@jycouet

jycouet commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

Would be cool to add node:sqlite 👍
I didn't realise that node 22 was already in maintenance mode https://nodejs.org/en/about/previous-releases !

I would not mind adding node-sqlite and bump node version later (in version-1 ?)

@sacrosanctic

Copy link
Copy Markdown
Contributor Author

node - available since 22, --experimental-sqlite flag removed in 22.13.0
deno - available since 2.2
bun - not available

@sacrosanctic
sacrosanctic marked this pull request as ready for review August 1, 2026 02:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants