fix: prevent duplicate API instances in MapContainer

- fixed state management bug where API instances were appended to existing array instead of replacing it
- this caused duplicate API instances in React Strict Mode and potential memory leaks
- replaced setApis((current) => [...current, newApi]) with proper array replacement

- compute full API array from layers, then replace state completely
- use .map() for transformation instead of side effects
- the apis array has 1:1 relationship with layers (matched by ID), so accumulation was never intended

app/src/pages/MapContainer.tsx

@@ -36,12 +36,12 @@ function MapContainer({ layers, map }: { layers: LayerProps[]; map: any }) {
   const [apis, setApis] = useState<layerApi[]>([])
 
   useEffect(() => {
-    // get timestamp for the end of the current day
+    // get timestamp for the start of the current day
     const now = new Date()
     const startOfDay = new Date(now.getFullYear(), now.getMonth(), now.getDate())
     const startOfDayISO = startOfDay.toISOString()
 
-    layers.map((layer: LayerProps) => {
+    const newApis = layers.map((layer: LayerProps) => {
       // Only apply date filter if showPastItems is not explicitly set to true
       const dateFilter = layer.showPastItems
         ? undefined
@@ -60,15 +60,13 @@ function MapContainer({ layers, map }: { layers: LayerProps[]; map: any }) {
             ],
           }
 
-      apis &&
+      return {
-        setApis((current) => [
+        id: layer.id,
-          ...current,
+        api: new itemsApi<Place>('items', layer.id, undefined, dateFilter),
-          {
+      }
-            id: layer.id,
-            api: new itemsApi<Place>('items', layer.id, undefined, dateFilter),
-          },
-        ])
     })
+
+    setApis(newApis)
   }, [layers])
 
   useEffect(() => {}, [apis])