Update JSON.stringify typings to handle all functions

total-typescript · Mar 7, 2024 · a9b624d · a9b624d
1 parent 311c3f3
commit a9b624d
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 2 deletions.
diff --git a/src/entrypoints/json-stringify.d.ts b/src/entrypoints/json-stringify.d.ts
@@ -10,7 +10,7 @@ interface JSON {
     replacer?: (this: any, key: string, value: any) => any,
     space?: string | number,
   ): T extends {} | null
-    ? T extends () => void
+    ? T extends (..._: any) => void
       ? undefined
       : string
     : T extends undefined
@@ -27,7 +27,7 @@ interface JSON {
     replacer?: (number | string)[] | null,
     space?: string | number,
   ): T extends {} | null
-    ? T extends () => void
+    ? T extends (..._: any) => void
       ? undefined
       : string
     : T extends undefined

diff --git a/src/tests/json-stringify.ts b/src/tests/json-stringify.ts
@@ -41,6 +41,13 @@ doNotExecute(() => {
   type tests = [Expect<Equal<typeof result, undefined>>];
 });
 
+doNotExecute(() => {
+  // create a something that is a function
+  const result = JSON.stringify(function (hello: any, world: any) {});
+
+  type tests = [Expect<Equal<typeof result, undefined>>];
+});
+
 doNotExecute(() => {
   // create a something that is of type any
   let toBeStringified: any;