feat(sqlite): option to avoid pushing parameters to the stack

[aidant]

What is the problem this feature will solve?

At the moment the only way to pass sqlite anonymous parameters is via the stack. This presents a problem as you can encounter a RangeError: Maximum call stack size exceeded when passing parameters.

Example

node --stack-size=50 # lowered to simulate the exception 

import { DatabaseSync } from 'node:sqlite'

const numbers = Array.from({ length: 1000 }, (_, index) => index)

console.log(
  JSON.parse( 
    new DatabaseSync(':memory:')
      .prepare(`select json_array(${numbers.map(() => '?')}) as numbers`)
      .get(...numbers).numbers
  )
)

What is the feature you are proposing to solve the problem?

Consider providing functionality for binding anonymous parameters to SQLite queries without having to place them on the stack.

What alternatives have you considered?

No response

[cjihrig]

Two questions:

1. Is this causing a real problem, or is this hypothetical? The current approach seems fine for reasonable queries.
2. If you have run into a real problem, how has it been solved by other Node.js SQLite libraries?

[aidant]

1. This is not causing a real problem, it's just hypothetical at the moment.
2. I've run into a similar problem before in a query builder where I was pushing fragments to an array using sql.push(...fragments) and the alternative in that scenario is for (const fragment of fragments) sql.push(fragment).

[jlarmstrongiv]

That’s great to know! I regularly have queries where I insert thousands of variables. I usually just hit the SQLite limit of 2000 columns per table.