sqlite: bind Boolean

PR-URL: #62001
Fixes: #57862
Reviewed-By: Zeyu "Alex" Yang <himself65@outlook.com>

Author: mike-git374

src/node_sqlite.cc

@@ -2736,10 +2736,11 @@ bool StatementSync::BindParams(const FunctionCallbackInfo<Value>& args) {
 
 bool StatementSync::BindValue(const Local<Value>& value, const int index) {
   // SQLite only supports a subset of JavaScript types. Some JS types such as
-  // functions don't make sense to support. Other JS types such as booleans and
+  // functions don't make sense to support. Other JS types such as
   // Dates could be supported by converting them to numbers. However, there
   // would not be a good way to read the values back from SQLite with the
-  // original type.
+  // original type. JS Boolean binds to 1 and 0 because SQLite maps true and
+  // false keywords to 1 and 0.
   Isolate* isolate = env()->isolate();
   int r;
   if (value->IsNumber()) {
@@ -2773,6 +2774,8 @@ bool StatementSync::BindValue(const Local<Value>& value, const int index) {
                             buf.data(),
                             static_cast<sqlite3_uint64>(buf.length()),
                             SQLITE_TRANSIENT);
+  } else if (value->IsBoolean()) {
+    r = sqlite3_bind_int(statement_, index, value->IsTrue() ? 1 : 0);
   } else if (value->IsBigInt()) {
     bool lossless;
     int64_t as_int = value.As<BigInt>()->Int64Value(&lossless);

test/parallel/test-sqlite-data-types.js

@@ -80,6 +80,15 @@ suite('data binding and mapping', () => {
       text: '',
       buf: new Uint8Array(),
     });
+
+    t.assert.deepStrictEqual(
+      stmt.run(5, true, false, true, null),
+      { changes: 1, lastInsertRowid: 5 }
+    );
+    t.assert.deepStrictEqual(
+      query.get(5),
+      { __proto__: null, key: 5, int: 1, double: 0, text: '1', buf: null }
+    );
   });
 
   test('large strings are bound correctly', (t) => {