Skip to content

Add node:sqlite StatementSync compatibility methods#2

Merged
timsehn merged 2 commits into
mainfrom
statement-node-sqlite-compat
May 19, 2026
Merged

Add node:sqlite StatementSync compatibility methods#2
timsehn merged 2 commits into
mainfrom
statement-node-sqlite-compat

Conversation

@timsehn

@timsehn timsehn commented May 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add StatementSync compatibility methods used by node:sqlite consumers and Drizzle: setReturnArrays, setReadBigInts, setAllowBareNamedParameters, and setAllowUnknownNamedParameters
  • make get/all/iterate honor array-return mode and BigInt integer reads
  • match node:sqlite named-parameter defaults by throwing on unknown named params unless explicitly allowed

Testing

  • npm run build
  • npx -y bun test test/
  • opencode-shaped Drizzle repro using drizzle-orm/node-sqlite against the local binding

@timsehn timsehn force-pushed the statement-node-sqlite-compat branch 2 times, most recently from af54e95 to f1fffeb Compare May 18, 2026 23:54
@timsehn timsehn force-pushed the statement-node-sqlite-compat branch from f1fffeb to 5b22f74 Compare May 18, 2026 23:58
@timsehn timsehn merged commit bc57204 into main May 19, 2026
18 checks 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.

1 participant