fix: debounce cancel should clear pending timeout

docs/curry/debounce.mdx

@@ -34,7 +34,7 @@ Source Invocations: - - - - - - - - - - x - - - - - - - - - - - - - - - - - x -
 
 ### Cancel
 
-The function returned by `debounce` has a `cancel` method that when called will permanently stop the source function from being debounced.
+The function returned by `debounce` has a `cancel` method that when called will cancel any pending invocation.
 
 ```ts
 const debounced = _.debounce({ delay: 100 }, api.feed.refresh)

src/curry/debounce.ts

@@ -41,24 +41,20 @@ export function debounce<TArgs extends any[]>(
   func: (...args: TArgs) => any,
 ): DebounceFunction<TArgs> {
   let timer: unknown = undefined
-  let active = true
 
   const debounced: DebounceFunction<TArgs> = (...args: TArgs) => {
-    if (active) {
-      clearTimeout(timer)
-      timer = setTimeout(() => {
-        active && func(...args)
-        timer = undefined
-      }, delay)
-    } else {
+    clearTimeout(timer)
+    timer = setTimeout(() => {
       func(...args)
-    }
+      timer = undefined
+    }, delay)
   }
   debounced.isPending = () => {
     return timer !== undefined
   }
   debounced.cancel = () => {
-    active = false
+    clearTimeout(timer)
+    timer = undefined
   }
   debounced.flush = (...args: TArgs) => func(...args)
 

tests/curry/debounce.test.ts

@@ -27,14 +27,23 @@ describe('debounce', () => {
     expect(mockFunc).toHaveBeenCalledTimes(1)
   })
 
-  test('does not debounce after cancel is called', () => {
-    runFunc3Times()
+  test('does not execute if cancel called before timeout', () => {
+    func()
+    expect(func.isPending()).toBe(true)
+    func.cancel()
+    expect(func.isPending()).toBe(false)
+    vi.advanceTimersByTime(delay + 10)
     expect(mockFunc).toHaveBeenCalledTimes(0)
+  })
+
+  test('continues to debounce after cancel is called', () => {
+    func()
     func.cancel()
-    runFunc3Times()
-    expect(mockFunc).toHaveBeenCalledTimes(3)
-    runFunc3Times()
-    expect(mockFunc).toHaveBeenCalledTimes(6)
+    vi.advanceTimersByTime(delay + 10)
+    expect(mockFunc).toHaveBeenCalledTimes(0)
+    func()
+    vi.advanceTimersByTime(delay + 10)
+    expect(mockFunc).toHaveBeenCalledTimes(1)
   })
 
   test('executes the function immediately when the flush method is called', () => {