🤖 Merge PR DefinitelyTyped#72396 fix(better-sqlite3): use any instead of unknown for function() by @hkleungai

Author: hkleungai

types/better-sqlite3/better-sqlite3-tests.ts

@@ -31,12 +31,30 @@ db.table("vtable_parameters", {
         yield { name: name };
     },
 });
-db.function("noop", () => {});
-db.function("noop", {
-    deterministic: true,
-    varargs: true,
-    directOnly: true,
-}, () => {});
+db.function("fn", () => {});
+db.function("fn", (a: number) => a + 123);
+db.function("fn", {}, () => {});
+db.function("fn", {}, (a: number) => a + 123);
+db.function(
+    "fn",
+    {
+        varargs: true,
+        deterministic: true,
+        safeIntegers: true,
+        directOnly: true,
+    },
+    () => {},
+);
+db.function(
+    "fn",
+    {
+        varargs: true,
+        deterministic: true,
+        safeIntegers: true,
+        directOnly: true,
+    },
+    (a: number) => a + 123,
+);
 db.aggregate("add", {
     start: 0,
     step: (t, n) => t + n,

types/better-sqlite3/index.d.ts

@@ -62,8 +62,15 @@ declare namespace BetterSqlite3 {
         transaction<F extends VariableArgFunction>(fn: F): Transaction<F>;
         exec(source: string): this;
         pragma(source: string, options?: Database.PragmaOptions): unknown;
-        function(name: string, cb: (...params: unknown[]) => unknown): this;
-        function(name: string, options: Database.RegistrationOptions, cb: (...params: unknown[]) => unknown): this;
+        function(
+            name: string,
+            cb: (...params: any[]) => any,
+        ): this;
+        function(
+            name: string,
+            options: Database.RegistrationOptions,
+            cb: (...params: any[]) => any,
+        ): this;
         aggregate<T>(
             name: string,
             options: Database.RegistrationOptions & {