feat(react): use useEffectEvent for effect event handlers

Extract event handlers in useQueryBasic from inside useEffect
to useEffectEvent, reducing the dependency array from 12 entries
to 2 (key, subscribe). Bumps react peer dep to ^19.2.0.

Author: ConsoleTVs

package.json

@@ -51,7 +51,7 @@
     "build:only": "vite build",
     "build": "npm run build:only",
     "prebuild": "npm run format:check && npm run lint",
-    "lint": "oxlint --react-plugin --vitest-plugin",
+    "lint": "oxlint --deny-warnings",
     "format": "oxfmt --write .",
     "format:check": "oxfmt --check .",
     "dev": "vitest",
@@ -82,8 +82,8 @@
     "vitest": "^4.1.0"
   },
   "peerDependencies": {
-    "react": "^19.1.0",
-    "react-dom": "^19.1.0",
+    "react": "^19.2.0",
+    "react-dom": "^19.2.0",
     "solid-js": "^1.9.0"
   },
   "peerDependenciesMeta": {

src/react/hooks/useQueryBasic.ts

@@ -1,4 +1,4 @@
-import { useEffect, use, useState, useTransition } from 'react'
+import { useEffect, useEffectEvent, use, useState, useTransition } from 'react'
 import { type ContextValue } from 'query/react:context'
 import { type Options } from 'query:index'
 import { useQueryContext } from './useQueryContext'
@@ -74,7 +74,7 @@ export function useQueryBasic<T = unknown>(
     useQueryTransitionContext()
 
   const [isLocalPending, startLocalTransition] = useTransition()
-  const { query, expiration, subscribe } = useQueryInstance(options)
+  const { query, subscribe } = useQueryInstance(options)
   const ignoreTransitionContext = oIgnoreTransitionContext ?? cIgnoreTransitionContext ?? false
   const isPending = ignoreTransitionContext ? isLocalPending : (isContextPending ?? isLocalPending)
 
@@ -94,64 +94,64 @@ export function useQueryBasic<T = unknown>(
 
   const [data, setData] = useState<T>(use(promise))
 
-  useEffect(
-    function () {
-      function onResolved(event: CustomEventInit<T>) {
-        startTransition(function () {
-          setData(event.detail as T)
-        })
-      }
+  const onResolved = useEffectEvent(function (event: CustomEventInit<T>) {
+    startTransition(function () {
+      setData(event.detail as T)
+    })
+  })
 
-      function onMutating(event: CustomEventInit<Promise<T>>) {
-        startTransition(async function () {
-          const value = await (event.detail as Promise<T>)
+  const onMutating = useEffectEvent(function (event: CustomEventInit<Promise<T>>) {
+    startTransition(async function () {
+      const value = await (event.detail as Promise<T>)
 
-          startTransition(function () {
-            setData(value)
-          })
-        })
-      }
+      startTransition(function () {
+        setData(value)
+      })
+    })
+  })
 
-      function onMutated(event: CustomEventInit<T>) {
-        startTransition(function () {
-          setData(event.detail as T)
-        })
-      }
+  const onMutated = useEffectEvent(function (event: CustomEventInit<T>) {
+    startTransition(function () {
+      setData(event.detail as T)
+    })
+  })
 
-      function onHydrated(event: CustomEventInit<T>) {
-        startTransition(function () {
-          setData(event.detail as T)
-        })
-      }
+  const onHydrated = useEffectEvent(function (event: CustomEventInit<T>) {
+    startTransition(function () {
+      setData(event.detail as T)
+    })
+  })
 
-      function onRefetching(event: CustomEventInit<Promise<T>>) {
-        startTransition(async function () {
-          const value = await (event.detail as Promise<T>)
+  const onRefetching = useEffectEvent(function (event: CustomEventInit<Promise<T>>) {
+    startTransition(async function () {
+      const value = await (event.detail as Promise<T>)
 
-          startTransition(function () {
-            setData(value)
-          })
+      startTransition(function () {
+        setData(value)
+      })
+    })
+  })
+
+  const onForgotten = useEffectEvent(function () {
+    if (clearOnForget) {
+      startTransition(async function () {
+        const data = await query<T>(key, {
+          expiration: oExpiration,
+          fetcher: oFetcher,
+          stale: oStale,
+          removeOnError: oRemoveOnError,
+          fresh: oFresh,
         })
-      }
 
-      function onForgotten() {
-        if (clearOnForget) {
-          startTransition(async function () {
-            const data = await query<T>(key, {
-              expiration: oExpiration,
-              fetcher: oFetcher,
-              stale: oStale,
-              removeOnError: oRemoveOnError,
-              fresh: oFresh,
-            })
-
-            startTransition(function () {
-              setData(data)
-            })
-          })
-        }
-      }
+        startTransition(function () {
+          setData(data)
+        })
+      })
+    }
+  })
 
+  useEffect(
+    function () {
       const unsubscribeResolved = subscribe(key, 'resolved', onResolved)
       const unsubscribeMutating = subscribe(key, 'mutating', onMutating)
       const unsubscribeMutated = subscribe(key, 'mutated', onMutated)
@@ -168,19 +168,7 @@ export function useQueryBasic<T = unknown>(
         unsubscribeForgotten()
       }
     },
-    [
-      query,
-      expiration,
-      subscribe,
-      key,
-      clearOnForget,
-      oExpiration,
-      oFetcher,
-      oStale,
-      oRemoveOnError,
-      oFresh,
-      startTransition,
-    ]
+    [key, subscribe]
   )
 
   return { data, isPending }