sqlite: `db.prepare(sql).set*` chain

[mike-git374]

Problem

Currently you need multiple lines and multiple js statements to declare a single SQLite statement. If the statement.set* functions return the statement you could chain 1 or more set* functions to declare the sql statement with 1 line of code which is more readable and easier to work with:

Proposal

// before
const q = db.prepare(sql);
q.setReturnArrays(true); // returns void, requires 2 js statements

// after
const q = db.prepare(sql).setReturnArrays(true); // returns sql statement

[Renegade334]

I think your original proposal (an options argument to db.prepare()) is more consistent with existing node:sqlite patterns, and with Node.js conventions on the whole. That being said, is your usage case already covered by passing defaults to the DatabaseSync constructor?

[mike-git374]

is your usage case already covered by passing defaults to the DatabaseSync constructor?

I saw the options are available in new DatabaseSync(path[, options]) but I need settings for individual statements

I think your original proposal (an options argument to db.prepare()) is more consistent with existing node:sqlite patterns

The 4 statement.set* functions already exist so the easiest thing and what I prefer for readability is to have those return the statement for chaining. If you want to add an options argument to db.prepare() in addition I would not be opposed but it's not a requirement for me. Here is example of each way to set options:

const q = db.prepare('select * from a where b = ?').setReturnArrays(true); // chain
const q = db.prepare('select * from a where b = ?', { returnArrays: true }); // options
const q = db.prepare('select * from a where b = ?', arrays);
const q = db.prepare('select * from a where b = ?'); q.setReturnArrays(true); // current method

const arrays = { returnArrays: true };