ts-data-forge 14.3.0 → 14.5.0

package/src/others/debounce-function.mts

@@ -11,7 +11,11 @@ export const debounce = <Args extends readonly unknown[]>(
   func: (...args: Args) => void,
   waitMilliseconds: number,
 ): ((...args: Args) => void) => {
-  let mut_timeoutId: ReturnType<typeof setTimeout> | undefined;
+  // Not `ReturnType<typeof setTimeout>`: with both the DOM and Node
+  // declarations in scope that resolves to `unknown`, which `clearTimeout`
+  // will not take. Naming what `clearTimeout` accepts says the same thing and
+  // stays true in either environment.
+  let mut_timeoutId: Parameters<typeof clearTimeout>[0];
 
   return (...args: Args): void => {
     clearTimeout(mut_timeoutId);

package/src/others/fast-deep-equal.mts

@@ -94,6 +94,13 @@ export const fastDeepEqual = <T,>(a: T, b: T): boolean => {
     );
   }
 
-  // true if both NaN, false otherwise
-  return Number.isNaN(a) && Number.isNaN(b);
+  // true if both NaN, false otherwise. `Number.isNaN` takes a number: called
+  // with anything else it is always false, so the check belongs behind a
+  // `typeof` guard rather than relying on that.
+  return (
+    typeof a === 'number' &&
+    typeof b === 'number' &&
+    Number.isNaN(a) &&
+    Number.isNaN(b)
+  );
 };