@lad-sapienza/scms-core 0.1.0-alpha.8 → 0.1.0-alpha.9

package/components/Map/Map.tsx

@@ -27,6 +27,18 @@ import type { SearchQuery } from './types';
 // Default base layer (OSM)
 const DEFAULT_BASE_LAYERS: BaseLayerConfig[] = [defaultBasemaps.OSM];
 
+// Stable identity for the "no explicit overlayLayers prop" case. A literal
+// `overlayLayers = []` default in the destructure below allocates a fresh
+// array on every render whenever the caller uses only a shorthand prop
+// (`geojson` / `csv` / `json` / `directus`) and omits `overlayLayers`. That
+// fresh array is a dependency of the `allOverlayLayers` useMemo, so the memo
+// recomputes every render → the data-loading effect keyed on it re-runs →
+// `setLayersData` → re-render → loop ("Maximum update depth exceeded", plus a
+// `fitToContent` fitBounds animation that restarts forever). Hoisting the
+// empty array to a module constant (same pattern as DEFAULT_BASE_LAYERS)
+// keeps its reference stable across renders.
+const EMPTY_OVERLAY_LAYERS: OverlayLayerConfig[] = [];
+
 // Vector-tile, xyz and wms sources are rendered natively/via RasterLayerLibre
 // and never go through fetchData — this narrows the union so TS knows
 // `layer.source` is a plain SourceConfig wherever this guard is used.
@@ -40,7 +52,7 @@ export function Map({
   mapStyle,
   styleOverrides,
   baseLayers = DEFAULT_BASE_LAYERS,
-  overlayLayers = [],
+  overlayLayers = EMPTY_OVERLAY_LAYERS,
   geolocateControl,
   fullscreenControl,
   navigationControl = 'top-left',

package/components/Map/MapMdx.tsx

@@ -22,16 +22,22 @@ export function Map(props: MapProps) {
   const stableGeojson = useMemo(() => props?.geojson, [JSON.stringify(props?.geojson || null)]);
   const stableCsv = useMemo(() => props?.csv, [JSON.stringify(props?.csv || null)]);
   const stableJson = useMemo(() => props?.json, [JSON.stringify(props?.json || null)]);
+  // `directus` feeds the same `implicitSource` → `allOverlayLayers` memo chain
+  // as `geojson`/`csv`/`json`; without stabilising it here an inline
+  // `directus={{ table: '…' }}` object from MDX gets a new identity on every
+  // render and drives the same render loop the others are memoised against.
+  const stableDirectus = useMemo(() => props?.directus, [JSON.stringify(props?.directus || null)]);
 
   return (
-    <MapCore 
-      {...props} 
+    <MapCore
+      {...props}
       center={stableCenter}
       baseLayers={stableBaseLayers}
       overlayLayers={stableOverlayLayers}
       geojson={stableGeojson}
       csv={stableCsv}
       json={stableJson}
+      directus={stableDirectus}
     />
   );
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lad-sapienza/scms-core",
-  "version": "0.1.0-alpha.8",
+  "version": "0.1.0-alpha.9",
   "description": "Framework components and Astro integrations for s:CMS — DataTb, Gallery, Map, ZoteroGeoViewer, Record.",
   "type": "module",
   "license": "BSD-0-Clause",