fix: statement cstring length, add test case for unicode strings

Author: DjDeveloperr

src/ffi.ts

@@ -574,6 +574,15 @@ const symbols = {
     parameters: [],
     result: "i32",
   },
+
+  sqlite3_update_hook: {
+    parameters: [
+      "pointer", // sqlite3 *db
+      "pointer", // void *pArg
+      "pointer", // void (*xUpdate)(sqlite3_context*,int,sqlite3_value**,sqlite3_value**)
+    ],
+    result: "pointer",
+  },
 } as const satisfies Deno.ForeignLibraryInterface;
 
 let lib: Deno.DynamicLibrary<typeof symbols>["symbols"];

src/statement.ts

@@ -231,11 +231,12 @@ export class Statement<TStatement extends object = Record<string, any>> {
 
   constructor(public db: Database, sql: string) {
     const pHandle = new BigUint64Array(1);
+    const cString = toCString(sql);
     unwrap(
       sqlite3_prepare_v2(
         db.unsafeHandle,
-        toCString(sql),
-        sql.length,
+        cString,
+        cString.byteLength,
         pHandle,
         null,
       ),

test/test.ts

@@ -611,6 +611,14 @@ Deno.test("sqlite", async (t) => {
     );
   });
 
+  await t.step("unicode support", () => {
+    const [value] = db.prepare("select ?").value<[string]>("💩")!;
+    assertEquals(value, "💩");
+
+    const [valueInStmt] = db.prepare("select '💩'").value<[string]>()!;
+    assertEquals(valueInStmt, "💩");
+  });
+
   await t.step({
     name: "close",
     sanitizeResources: false,