feat: add sqlite-type symbol property for type identification in DatabaseSync -- see nodejs/node#59405

Author: mceachen

src/sqlite_impl.cpp

@@ -308,6 +308,17 @@ Napi::Object DatabaseSync::Init(Napi::Env env, Napi::Object exports) {
                  }));
   }
 
+  // Add Symbol.for('sqlite-type') property for type identification
+  // See: https://github.com/nodejs/node/pull/59405
+  Napi::Object symbolConstructor =
+      env.Global().Get("Symbol").As<Napi::Object>();
+  Napi::Function symbolFor = symbolConstructor.Get("for").As<Napi::Function>();
+  Napi::Value sqliteTypeSymbol = symbolFor.Call(
+      symbolConstructor, {Napi::String::New(env, "sqlite-type")});
+  func.Get("prototype")
+      .As<Napi::Object>()
+      .Set(sqliteTypeSymbol, Napi::String::New(env, "node:sqlite"));
+
   exports.Set("DatabaseSync", func);
   return exports;
 }

test/database.test.ts

@@ -154,6 +154,18 @@ describe("DatabaseSync Tests", () => {
     db.close();
     expect(db.isOpen).toBe(false);
   });
+
+  // See: https://github.com/nodejs/node/pull/59405
+  test("has sqlite-type symbol property", () => {
+    const db = new DatabaseSync(":memory:");
+
+    const sqliteTypeSymbol = Symbol.for("sqlite-type");
+    expect((db as unknown as Record<symbol, string>)[sqliteTypeSymbol]).toBe(
+      "node:sqlite",
+    );
+
+    db.close();
+  });
 });
 
 describe("File-based Database Tests", () => {