fix: Address Copilot review comments and fix build errors

- Fix null searchParams handling in useRouterQuery hook
- Add useMemo to prevent query object recreation
- Fix stale closure issue in updateQuery callback
- Fix router.push missing pathname in claim-nodes page
- Remove unused router import in admin page
- Fix grammar in providers.tsx comment

Fixes TypeScript error: 'searchParams' is possibly 'null'
Resolves Vercel build failure

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: snomiao

app/admin/claim-nodes/page.tsx

@@ -29,7 +29,7 @@ function ClaimNodesPage() {
     // Update URL with new page parameter
     const params = new URLSearchParams(searchParams?.toString())
     params.set('page', page.toString())
-    router.push(`?${params.toString()}`)
+    router.push(`/admin/claim-nodes?${params.toString()}`)
   }
 
   // Use the page from searchParams for the API call

app/admin/page.tsx

@@ -2,7 +2,6 @@
 
 import { Breadcrumb } from 'flowbite-react'
 import Link from 'next/link'
-import { useRouter } from 'next/navigation'
 import {
   HiHome,
   HiOutlineAdjustments,
@@ -14,7 +13,6 @@ import withAdmin from '@/components/common/HOC/authAdmin'
 import { useNextTranslation } from '@/src/hooks/i18n'
 
 function AdminDashboard() {
-  const router = useRouter()
   const { t } = useNextTranslation()
 
   return (

app/providers.tsx

@@ -52,7 +52,7 @@ export function Providers({ children }: { children: React.ReactNode }) {
 
   useEffect(() => {
     // General localStorage cache invalidation for all endpoints
-    // this interceptors will user always have latest data after edit.
+    // This interceptor ensures users always have the latest data after edit.
     const responseInterceptor = AXIOS_INSTANCE.interceptors.response.use(
       function onSuccess(response: AxiosResponse) {
         const req = response.config

src/hooks/useRouterQuery.app.ts

@@ -1,6 +1,6 @@
 import { useRouter, useSearchParams, usePathname } from 'next/navigation'
 import { filter, omit } from 'rambda'
-import { useCallback } from 'react'
+import { useCallback, useMemo } from 'react'
 
 /**
  * A hook to easily access and update URL query parameters (App Router version)
@@ -29,8 +29,11 @@ export function useRouterQuery<
   const pathname = usePathname()
   const searchParams = useSearchParams()
 
-  // Convert URLSearchParams to object
-  const query = Object.fromEntries(searchParams.entries()) as T
+  // Convert URLSearchParams to object (memoized to avoid recreation)
+  const query = useMemo(
+    () => Object.fromEntries(searchParams?.entries() ?? []) as T,
+    [searchParams]
+  )
 
   /**
    * Update query parameters
@@ -44,14 +47,19 @@ export function useRouterQuery<
    */
   const updateQuery = useCallback(
     (newParams: Partial<T>, replace = false) => {
+      // Get current query from searchParams to avoid stale closures
+      const currentQuery = Object.fromEntries(
+        searchParams?.entries() ?? []
+      ) as T
+
       // Filter out null and undefined values
       const filteredParams = filter((e) => e != null, newParams)
 
       // Prepare the final query object
       const finalQuery = replace
         ? filteredParams
         : {
-            ...omit(Object.keys(newParams), query),
+            ...omit(Object.keys(newParams), currentQuery),
             ...filteredParams,
           }
 
@@ -68,9 +76,9 @@ export function useRouterQuery<
         ? `${pathname}?${params.toString()}`
         : pathname
 
-      router.push(newUrl)
+      router.push(newUrl ?? '/')
     },
-    [router, pathname, query]
+    [router, pathname, searchParams]
   )
 
   return [query, updateQuery] as const