Skip to content

Update Deno x/sqlite version#24

Merged
lpil merged 4 commits intolpil:mainfrom
bchase:relax-deno-x-sqlite-version
Oct 21, 2025
Merged

Update Deno x/sqlite version#24
lpil merged 4 commits intolpil:mainfrom
bchase:relax-deno-x-sqlite-version

Conversation

@bchase
Copy link
Copy Markdown
Contributor

@bchase bchase commented Oct 19, 2025

Hello! 👋

I was running into the following error trying to get sqlight working with Deno:

error: Uncaught TypeError: Deno.seekSync is not a function
      Deno.seekSync(rid, offset, Deno.SeekMode.Start);
           ^
    at js_read (https://deno.land/x/sqlite@v3.7.0/build/vfs.js:48:12)
    at <anonymous> (wasm://wasm/00274a8a:1:4034)
    at <anonymous> (wasm://wasm/00274a8a:1:143632)
    at <anonymous> (wasm://wasm/00274a8a:1:138770)
    at <anonymous> (wasm://wasm/00274a8a:1:144940)
    at <anonymous> (wasm://wasm/00274a8a:1:4589)
    at <anonymous> (wasm://wasm/00274a8a:1:594290)
    at https://deno.land/x/sqlite@v3.7.0/src/db.ts:208:27
    at setStr (https://deno.land/x/sqlite@v3.7.0/src/wasm.ts:19:20)
    at new DB (https://deno.land/x/sqlite@v3.7.0/src/db.ts:205:20)

And it appears that Deno.seekSync has been removed, which breaks x/sqlite@v3.7.0 for modern Deno versions.

I bumped the version to latest (@3.9.1) and everything works (deno 2.5.4), but I'm not sure the consequences of setting this version, so please let me know if it should be something lower, or how I should go about determining what version to set it to.

Thanks in advance! 🙏

Cheers,
Brad

P.S. - I've pivoted off of my Electon + Lustre mission to Tauri + Deno + Lustre, and sqlight is going to be a huge help! 😁

Copy link
Copy Markdown
Owner

@lpil lpil left a comment

Choose a reason for hiding this comment

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

Seeing as this is a library should we use something like import { DB } from "https://deno.land/x/sqlite/mod.ts";?

@bchase
Copy link
Copy Markdown
Contributor Author

bchase commented Oct 20, 2025

That's actually what I did at first, but then got to wondering if there wasn't a good reason for pinning a specific version that I was missing... 😅

Sounds good to me! Upinned the version! 👍

Copy link
Copy Markdown
Owner

@lpil lpil left a comment

Choose a reason for hiding this comment

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

Fab! Could you update the changelog please 🙏

@bchase
Copy link
Copy Markdown
Contributor Author

bchase commented Oct 21, 2025

Updated! And bumped the patch version. 👍

Copy link
Copy Markdown
Owner

@lpil lpil left a comment

Choose a reason for hiding this comment

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

Thank you!

@lpil lpil merged commit 345a383 into lpil:main Oct 21, 2025
1 check passed
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.

2 participants