fix: deferred finally uncaught errors (#48)

packages/@react/src/composable/useOverlay.ts

@@ -1,4 +1,4 @@
-import { delay as _delay, noop } from '@overlays/core'
+import { delay, noop } from '@overlays/core'
 import type { Dispatch, SetStateAction } from 'react'
 import { useContext, useEffect } from 'react'
 
@@ -68,22 +68,24 @@ export function useOverlay(options: UseOverlayOptions = {}) {
   const { immediate = true, duration = 0, automatic = true } = options
   const context = useContext(Context)
   const dec = Reflect.get(context, 'in_dec')
-  const meta = dec ? useDeclarative(options) : context
+  const overlay = dec ? useDeclarative(options) : context
+  const { setVisible, vanish, deferred } = overlay
 
   // The component directly obtains the default value
   // vanish will have no effect, and no watch will be performed.
+  async function destroy() {
+    setVisible(false)
+    await delay(duration)
+    vanish?.()
+    return Promise.resolve()
+  }
+  useMount(() => immediate && setVisible(true))
   useMount(() => {
-    immediate && meta.setVisible?.(true)
-    if (!dec && automatic) {
-      meta.deferred?.finally(async () => {
-        meta.setVisible(false)
-        await _delay(duration)
-        meta.vanish?.()
-      })
-    }
+    if (!dec && automatic)
+      deferred?.then(destroy).catch(destroy)
   })
 
-  return meta as UseOverlayReturn
+  return overlay as UseOverlayReturn
 }
 
 export function useDeclarative(options: UseOverlayOptions = {}) {

packages/@svelte/src/components/index.ts

@@ -157,12 +157,18 @@ function instance($$self: any, $$props: any, $$invalidate: any) {
   let { visible = false } = $$props
   const { duration = 0, immediate = true } = getContext<UseOverlayOptions>(injectOptionsKey) || {}
   const { deferred, vanish } = getContext<UseOverlayReturn>(injectOverlayKey)
-  deferred?.finally(async () => {
+
+  // The component directly obtains the default value
+  // vanish will have no effect, and no watch will be performed.
+  async function destroy() {
     $$invalidate(0, visible = false)
     await delay(duration)
-    vanish()
-  })
+    vanish?.()
+    return Promise.resolve()
+  }
   onMount(() => immediate && $$invalidate(0, visible = true))
+  deferred?.then(destroy).catch(destroy)
+
   $$self.$$set = ($$props2: any) => {
     if ('visible' in $$props2)
       $$invalidate(0, visible = $$props2.visible)

packages/@vue/src/composable/useOverlay.ts

@@ -67,25 +67,26 @@ export interface UseOverlayReturn {
  */
 export function useOverlay(options: UseOverlayOptions = {}) {
   const { duration = 0, immediate = true, model = 'visible', automatic = true } = options
-  const meta = inject(OverlayMetaKey, useDeclarative(model, options))
-  const dec = Reflect.get(meta, 'in_dec')
+  const overlay = inject(OverlayMetaKey, useDeclarative(model, options))
+  const dec = Reflect.get(overlay, 'in_dec')
+  const { visible, deferred, vanish } = overlay
 
   // The component directly obtains the default value
   // vanish will have no effect, and no watch will be performed.
-  if (!dec && automatic) {
-    meta.deferred?.finally(async () => {
-      meta.visible.value = false
-      await delay(duration)
-      meta.vanish?.()
-    })
+  async function destroy() {
+    visible.value = false
+    await delay(duration)
+    vanish?.()
+    return Promise.resolve()
   }
 
+  if (!dec && automatic)
+    deferred?.then(destroy).catch(destroy)
   if (!dec && immediate)
-    onMounted(() => meta.visible.value = true)
+    onMounted(() => visible.value = true)
 
   provide(OverlayMetaKey, null)
-
-  return meta
+  return overlay
 }
 
 export function useDeclarative(model: string, options: UseOverlayOptions = {}) {