@sanity/ui 3.1.13 → 3.1.14

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sanity/ui",
-  "version": "3.1.13",
+  "version": "3.1.14",
   "keywords": [
     "sanity",
     "ui",

package/src/core/hooks/useDelayed.test.ts

@@ -47,6 +47,26 @@ describe('useDelayedState', () => {
     expect(result.current[0]).toBe(true)
   })
 
+  it('should clear pending timeout on unmount', () => {
+    jest.useFakeTimers()
+    const clearTimeoutSpy = jest.spyOn(global, 'clearTimeout')
+    const {result, unmount} = renderHook(() => useDelayedState(false))
+    const [, setState] = result.current
+
+    act(() => {
+      setState(true, 1000)
+    })
+
+    clearTimeoutSpy.mockClear()
+
+    // Unmount while the delayed update is still pending
+    unmount()
+
+    // clearTimeout should have been called during cleanup
+    expect(clearTimeoutSpy).toHaveBeenCalledTimes(1)
+    clearTimeoutSpy.mockRestore()
+  })
+
   it('should cancel update if the set state was called with a new state', () => {
     const {result} = renderHook(() => useDelayedState(false))
     const [, setState] = result.current

package/src/core/hooks/useDelayedState.ts

@@ -1,4 +1,4 @@
-import {SetStateAction, useCallback, useRef, useState} from 'react'
+import {SetStateAction, useCallback, useEffect, useRef, useState} from 'react'
 
 /**
  * @beta
@@ -24,5 +24,13 @@ export function useDelayedState<S>(
     delayedAction.current = setTimeout(action, delay)
   }, [])
 
+  useEffect(() => {
+    return () => {
+      if (delayedAction.current) {
+        clearTimeout(delayedAction.current)
+      }
+    }
+  }, [])
+
   return [state, onStateChange]
 }