feat: better dispatch

Author: gadomski

src/components/footer.tsx

@@ -1,7 +1,10 @@
 import { HStack, Text } from "@chakra-ui/react";
 import { useDuckDb } from "duckdb-wasm-kit";
+import { useStacGeoparquet } from "./stac-geoparquet/hooks";
 export default function Footer() {
   const { loading, error } = useDuckDb();
+  const state = useStacGeoparquet();
+
   return (
     <HStack
       style={{
@@ -14,6 +17,7 @@ export default function Footer() {
     >
       {loading && <Text>Loading DuckDB...</Text>}
       {error && <Text color={"red"}>DuckDB error: {error.toString()}</Text>}
+      {state.path && !state.metadata && <Text>Loading {state.path}</Text>}
     </HStack>
   );
 }

src/components/stac-geoparquet/context.ts

@@ -25,8 +25,8 @@ export type StacGeoparquetAction =
   | { type: "set-path"; path: string }
   | { type: "set-metadata"; metadata: StacGeoparquetMetadata }
   | { type: "set-table"; table: Table }
-  | { type: "set-id"; id?: string }
-  | { type: "set-item"; item?: StacItem };
+  | { type: "set-id"; id: string }
+  | { type: "set-item"; item: StacItem };
 
 export const StacGeoparquetContext =
   createContext<StacGeoparquetContextType | null>(null);

src/components/stac-geoparquet/provider.tsx

@@ -42,8 +42,6 @@ export default function StacGeoparquetProvider({
 
   useEffect(() => {
     if (state.path) {
-      dispatch({ type: "set-id" });
-      dispatch({ type: "set-item" });
       if (connection) {
         (async () => {
           let result;
@@ -129,9 +127,8 @@ export default function StacGeoparquetProvider({
     }
   }, [state.path, connection, dispatch]);
 
-  // TODO I don't like this depending on state.path.
   useEffect(() => {
-    if (state.id) {
+    if (state.id && state.path) {
       if (connection) {
         (async () => {
           // TODO refactor query so we don't repeat logic.
@@ -144,8 +141,6 @@ export default function StacGeoparquetProvider({
       } else {
         // TODO Handle missing connection
       }
-    } else {
-      dispatch({ type: "set-item" });
     }
   }, [state.id, state.path, connection, dispatch]);
 
@@ -159,13 +154,13 @@ export default function StacGeoparquetProvider({
 function reducer(state: StacGeoparquetState, action: StacGeoparquetAction) {
   switch (action.type) {
     case "set-path":
-      return { ...state, path: action.path };
+      return { ...state, path: action.path, id: undefined, item: undefined };
     case "set-metadata":
       return { ...state, metadata: action.metadata };
     case "set-table":
       return { ...state, table: action.table };
     case "set-id":
-      return { ...state, id: action.id };
+      return { ...state, id: action.id, item: undefined };
     case "set-item":
       return { ...state, item: action.item };
   }