fix memoize and useCallbackFactory

Author: garronej

src/lib/tools/memoize.ts

@@ -1,14 +1,14 @@
-export function memoize<F extends (...args: (string | number | boolean)[]) => any>(
-    fn: F,
-    options: {
-        argsLength: number;
+export function memoize<Args extends (number | boolean | string)[], R>(
+    fn: (...args: Args) => R,
+    options?: {
+        argsLength?: number;
     }
-): F {
-    const cache: Record<string, ReturnType<F>> = {};
+): (...args: Args) => R {
+    const cache: Record<string, R> = {};
 
-    const { argsLength } = options;
+    const { argsLength = fn.length } = options ?? {};
 
-    return ((...args: Parameters<F>) => {
+    return ((...args: Args) => {
         const key = JSON.stringify(args.slice(0, argsLength).join("-sIs9sAslOdeWlEdIos3-"));
 
         console.log(key, JSON.stringify({ argsLength, args }));

src/lib/tools/powerhooks/useCallbackFactory.ts

@@ -36,14 +36,14 @@ export function useCallbackFactory<
         id<Out>((...factoryArgs) => {
             if (memoizedRef.current === undefined) {
                 memoizedRef.current = memoize(
-                    ((...factoryArgs: FactoryArgs) =>
+                    (...factoryArgs: FactoryArgs) =>
                         (...args: Args) =>
-                            callbackRef.current(factoryArgs, args)) as any,
+                            callbackRef.current(factoryArgs, args),
                     { "argsLength": factoryArgs.length }
                 );
             }
 
-            return (memoizedRef.current as any)(...factoryArgs);
+            return memoizedRef.current(...factoryArgs);
         })
     )[0];
 }