fix: properly type reload as async across all hooks

- Change reload return type from () => void to () => Promise<QueryObserverResult>
- Remove incorrect type casts from refetch in useCollection, useCollections, useItem
- Update tests to await reload() calls with proper act() wrapping
- Remove eslint-disable comment from useItem test
- Remove debounce wrapper from useCollections reload (was returning void)

The refetch function from React Query returns a Promise, not void. This fix
properly reflects the async nature of reload operations and resolves React
act() warnings in tests. Debouncing was removed from useCollections.reload
as it's uncommon to debounce explicit reload actions.

Author: AliceR

src/hooks/useCollection.test.ts

@@ -83,7 +83,9 @@ describe('useCollection', () => {
       expect(result.current.collection).toEqual({ id: 'abc', title: 'Collection A' })
     );
 
-    act(() => result.current.reload());
+    await act(async () => {
+      await result.current.reload();
+    });
 
     await waitFor(() =>
       expect(result.current.collection).toEqual({ id: 'abc', title: 'Collection A - Updated' })

src/hooks/useCollection.ts

@@ -1,4 +1,4 @@
-import { useQuery } from '@tanstack/react-query';
+import { useQuery, type QueryObserverResult } from '@tanstack/react-query';
 import type { ApiErrorType } from '../types';
 import type { Collection } from '../types/stac';
 import { ApiError } from '../utils/ApiError';
@@ -10,7 +10,7 @@ type StacCollectionHook = {
   isLoading: boolean;
   isFetching: boolean;
   error?: ApiErrorType;
-  reload: () => void;
+  reload: () => Promise<QueryObserverResult<Collection, ApiErrorType>>;
 };
 
 function useCollection(collectionId: string): StacCollectionHook {
@@ -50,7 +50,7 @@ function useCollection(collectionId: string): StacCollectionHook {
     isLoading,
     isFetching,
     error: error as ApiErrorType,
-    reload: refetch as () => void,
+    reload: refetch,
   };
 }
 

src/hooks/useCollections.test.ts

@@ -32,7 +32,9 @@ describe('useCollections', () => {
     await waitFor(() => expect(result.current.collections).toEqual({ data: 'original' }));
     await waitFor(() => expect(result.current.isLoading).toEqual(false));
 
-    act(() => result.current.reload());
+    await act(async () => {
+      await result.current.reload();
+    });
 
     await waitFor(() => expect(result.current.collections).toEqual({ data: 'reloaded' }));
   });

src/hooks/useCollections.ts

@@ -1,15 +1,13 @@
-import { useMemo } from 'react';
-import { useQuery } from '@tanstack/react-query';
+import { useQuery, type QueryObserverResult } from '@tanstack/react-query';
 import { type ApiErrorType } from '../types';
 import type { CollectionsResponse } from '../types/stac';
-import debounce from '../utils/debounce';
 import { ApiError } from '../utils/ApiError';
 import { generateCollectionsQueryKey } from '../utils/queryKeys';
 import { useStacApiContext } from '../context/useStacApiContext';
 
 type StacCollectionsHook = {
   collections?: CollectionsResponse;
-  reload: () => void;
+  reload: () => Promise<QueryObserverResult<CollectionsResponse, ApiErrorType>>;
   isLoading: boolean;
   isFetching: boolean;
   error?: ApiErrorType;
@@ -47,11 +45,9 @@ function useCollections(): StacCollectionsHook {
     retry: false,
   });
 
-  const reload = useMemo(() => debounce(refetch), [refetch]);
-
   return {
     collections,
-    reload,
+    reload: refetch,
     isLoading,
     isFetching,
     error: error as ApiErrorType,

src/hooks/useItem.test.ts

@@ -69,7 +69,6 @@ describe('useItem', () => {
     await waitFor(() => expect(result.current.item).toEqual({ id: 'abc' }));
 
     await act(async () => {
-      // eslint-disable-next-line @typescript-eslint/await-thenable
       await result.current.reload();
     });
 

src/hooks/useItem.ts

@@ -1,4 +1,4 @@
-import { useQuery } from '@tanstack/react-query';
+import { useQuery, type QueryObserverResult } from '@tanstack/react-query';
 import { Item } from '../types/stac';
 import { type ApiErrorType } from '../types';
 import { useStacApiContext } from '../context/useStacApiContext';
@@ -10,7 +10,7 @@ type ItemHook = {
   isLoading: boolean;
   isFetching: boolean;
   error?: ApiErrorType;
-  reload: () => void;
+  reload: () => Promise<QueryObserverResult<Item, ApiErrorType>>;
 };
 
 function useItem(url: string): ItemHook {
@@ -50,7 +50,7 @@ function useItem(url: string): ItemHook {
     isLoading,
     isFetching,
     error: error as ApiErrorType,
-    reload: refetch as () => void,
+    reload: refetch,
   };
 }