refactor!: remove collections/items state from context

BREAKING CHANGE: Remove collections, setCollections, getItem, addItem,
deleteItem from context. Context now only provides stacApi. Users should
use useCollections() hook directly instead of accessing collections from
context.

React Query cache is now the single source of truth for data.

Author: AliceR

src/context/StacApiProvider.test.tsx

@@ -165,25 +165,6 @@ describe('StacApiProvider', () => {
       });
     });
 
-    it('provides context methods', () => {
-      function ContextChecker() {
-        const context = useStacApiContext();
-        const hasMethods =
-          typeof context.getItem === 'function' &&
-          typeof context.addItem === 'function' &&
-          typeof context.deleteItem === 'function';
-        return <div data-testid="has-methods">{hasMethods ? 'true' : 'false'}</div>;
-      }
-
-      render(
-        <StacApiProvider apiUrl="https://test-stac-api.com">
-          <ContextChecker />
-        </StacApiProvider>
-      );
-
-      expect(screen.getByTestId('has-methods')).toHaveTextContent('true');
-    });
-
     it('passes options to useStacApi hook', () => {
       const customOptions = { headers: { 'X-Custom': 'value' } };
 

src/context/context.ts

@@ -1,14 +1,8 @@
 import { createContext } from 'react';
-import type { CollectionsResponse, Item } from '../types/stac';
 
 export type StacApiContextType = {
   // eslint-disable-next-line @typescript-eslint/no-explicit-any
   stacApi?: any;
-  collections?: CollectionsResponse;
-  setCollections: (collections?: CollectionsResponse) => void;
-  getItem: (id: string) => Item | undefined;
-  addItem: (id: string, item: Item) => void;
-  deleteItem: (id: string) => void;
 };
 
 export const StacApiContext = createContext<StacApiContextType>({} as StacApiContextType);

src/context/index.tsx

@@ -1,6 +1,5 @@
-import React, { useMemo, useState, useCallback, useContext } from 'react';
+import React, { useMemo, useContext } from 'react';
 import { StacApiContext } from './context';
-import type { CollectionsResponse, Item } from '../types/stac';
 import { GenericObject } from '../types';
 import { QueryClient, QueryClientProvider, QueryClientContext } from '@tanstack/react-query';
 
@@ -19,37 +18,12 @@ function StacApiProviderInner({
   options,
 }: Omit<StacApiProviderType, 'enableDevTools'>) {
   const { stacApi } = useStacApi(apiUrl, options);
-  const [collections, setCollections] = useState<CollectionsResponse>();
-  const [items, setItems] = useState(new Map<string, Item>());
-
-  const getItem = useCallback((id: string) => items.get(id), [items]);
-
-  const addItem = useCallback(
-    (itemPath: string, item: Item) => {
-      setItems(new Map(items.set(itemPath, item)));
-    },
-    [items]
-  );
-
-  const deleteItem = useCallback(
-    (itemPath: string) => {
-      const tempItems = new Map(items);
-      items.delete(itemPath);
-      setItems(tempItems);
-    },
-    [items]
-  );
 
   const contextValue = useMemo(
     () => ({
       stacApi,
-      collections,
-      setCollections,
-      getItem,
-      addItem,
-      deleteItem,
     }),
-    [addItem, collections, deleteItem, getItem, stacApi]
+    [stacApi]
   );
 
   return <StacApiContext.Provider value={contextValue}>{children}</StacApiContext.Provider>;

src/context/useStacApiContext.ts

@@ -2,15 +2,9 @@ import { useContext } from 'react';
 import { StacApiContext } from './context';
 
 export function useStacApiContext() {
-  const { stacApi, collections, setCollections, getItem, addItem, deleteItem } =
-    useContext(StacApiContext);
+  const { stacApi } = useContext(StacApiContext);
 
   return {
     stacApi,
-    collections,
-    setCollections,
-    getItem,
-    addItem,
-    deleteItem,
   };
 }

src/hooks/useCollections.ts

@@ -15,7 +15,7 @@ type StacCollectionsHook = {
 };
 
 function useCollections(): StacCollectionsHook {
-  const { stacApi, setCollections } = useStacApiContext();
+  const { stacApi } = useStacApiContext();
   const [state, setState] = useState<LoadingState>('IDLE');
 
   const fetchCollections = async (): Promise<CollectionsResponse> => {
@@ -47,15 +47,6 @@ function useCollections(): StacCollectionsHook {
     retry: false,
   });
 
-  // Sync collections with context
-  useEffect(() => {
-    if (collections) {
-      setCollections(collections);
-    } else if (error) {
-      setCollections(undefined);
-    }
-  }, [collections, error, setCollections]);
-
   const reload = useMemo(() => debounce(refetch), [refetch]);
 
   useEffect(() => {