fix(delay): prevent infinite mode from throwing

Author: kettanaito

src/core/utils/internal/hasRefCounted.test.ts

@@ -14,3 +14,24 @@ it('returns false for a non-refcounted object', () => {
   expect(hasRefCounted({ ref() {} })).toBe(false)
   expect(hasRefCounted({ unref() {} })).toBe(false)
 })
+
+it('returns false for non-object values', () => {
+  expect(
+    hasRefCounted(
+      // @ts-expect-error Runtime value.
+      null,
+    ),
+  ).toBe(false)
+  expect(
+    hasRefCounted(
+      // @ts-expect-error Runtime value.
+      123,
+    ),
+  ).toBe(false)
+  expect(
+    hasRefCounted(
+      // @ts-expect-error Runtime value.
+      'invalid',
+    ),
+  ).toBe(false)
+})

src/core/utils/internal/hasRefCounted.ts

@@ -1,7 +1,14 @@
+import { isObject } from './isObject'
+
 export function hasRefCounted<T extends object>(
   value: T,
 ): value is T & NodeJS.RefCounted {
   return (
+    /**
+     * @note Guard against non-object values.
+     * E.g. `setTimeout` returns an object in Node.js but a number in the browser.
+     */
+    isObject(value) &&
     typeof Reflect.get(value, 'ref') === 'function' &&
     typeof Reflect.get(value, 'unref') === 'function'
   )