@trackunit/react-map-adapter-google 0.0.137 → 0.0.140-alpha-b0d834f66ac.0

package/index.cjs.js

@@ -1849,6 +1849,44 @@ class GoogleMapsLayerPort {
                 const canvas = this.canvas;
                 if (canvas === null)
                     return;
+                // No markers to draw (everything clustered or DOM-promoted, or the data
+                // layer emptied the list in place): render an empty canvas and stop,
+                // regardless of how the viewport would size. Handling this up front
+                // keeps both sizing paths free of empty-list special cases, and stops
+                // the fallback below from arming a retry that can never succeed
+                // (observed as no-op retry churn at world view when everything is
+                // clustered).
+                if (canvasMarkers.length === 0) {
+                    const clearCtx = canvas.getContext("2d");
+                    if (clearCtx !== null) {
+                        // The canvas is not resized on this path, so a previous draw's
+                        // scale transform may still be active — reset it or the clear may
+                        // not cover the whole backing store.
+                        clearCtx.setTransform(1, 0, 0, 1, 0, 0);
+                        clearCtx.clearRect(0, 0, canvas.width, canvas.height);
+                    }
+                    drawnMarkers = [];
+                    // The dots are gone, so any active marker hover must end too — no
+                    // mousemove will fire over an empty canvas to end it later. Deferred
+                    // because draw() runs inside Google Maps' render pass: a consumer
+                    // reacting to hover-end may synchronously push new layer state,
+                    // re-entering draw().
+                    if (lastHoveredFeatureId !== null) {
+                        const hoverEndId = lastHoveredFeatureId;
+                        lastHoveredFeatureId = null;
+                        queueMicrotask(() => {
+                            layerPort.emitEntityInteraction("hover-end", {
+                                type: "marker",
+                                id: hoverEndId,
+                                position: [0, 0],
+                                clusterId: null,
+                            });
+                        });
+                    }
+                    canvas.style.cursor = reactMapAdapterShared.MAP_CURSORS.default;
+                    this.boundsDeferScheduled = false;
+                    return;
+                }
                 // Read viewport state from the map this overlay is CURRENTLY attached to.
                 // reconnectAllSources() re-parents the overlay onto recreated map instances
                 // (theme or renderingType changes); the closed-over `map` parameter keeps
@@ -1868,6 +1906,11 @@ class GoogleMapsLayerPort {
                 if (bounds) {
                     const ne = projection.fromLatLngToDivPixel(bounds.getNorthEast());
                     const sw = projection.fromLatLngToDivPixel(bounds.getSouthWest());
+                    // Runtime verification (SAGA-879, PR #25976): fromLatLngToDivPixel
+                    // projects to the world copy nearest the map center, so 180°-crossing
+                    // viewports keep a positive width and take this path. Only a
+                    // full-world viewport (bounds lng -180..180, raw width ~0) fails the
+                    // guard, deliberately falling through to the marker-bbox fallback.
                     if (ne !== null && sw !== null && ne.x - sw.x > 1 && sw.y - ne.y > 1) {
                         width = ne.x - sw.x + pad * 2;
                         height = sw.y - ne.y + pad * 2;
@@ -1899,6 +1942,9 @@ class GoogleMapsLayerPort {
                             maxY = pt.y;
                     }
                     if (!isFinite(minX)) {
+                        // Markers exist but are not projectable yet (the empty list is
+                        // handled at the top of draw(), so this is always the transient
+                        // case) — retry once the frame after the projection settles.
                         if (!this.boundsDeferScheduled) {
                             this.boundsDeferScheduled = true;
                             requestAnimationFrame(() => {

package/index.esm.js

@@ -1848,6 +1848,44 @@ class GoogleMapsLayerPort {
                 const canvas = this.canvas;
                 if (canvas === null)
                     return;
+                // No markers to draw (everything clustered or DOM-promoted, or the data
+                // layer emptied the list in place): render an empty canvas and stop,
+                // regardless of how the viewport would size. Handling this up front
+                // keeps both sizing paths free of empty-list special cases, and stops
+                // the fallback below from arming a retry that can never succeed
+                // (observed as no-op retry churn at world view when everything is
+                // clustered).
+                if (canvasMarkers.length === 0) {
+                    const clearCtx = canvas.getContext("2d");
+                    if (clearCtx !== null) {
+                        // The canvas is not resized on this path, so a previous draw's
+                        // scale transform may still be active — reset it or the clear may
+                        // not cover the whole backing store.
+                        clearCtx.setTransform(1, 0, 0, 1, 0, 0);
+                        clearCtx.clearRect(0, 0, canvas.width, canvas.height);
+                    }
+                    drawnMarkers = [];
+                    // The dots are gone, so any active marker hover must end too — no
+                    // mousemove will fire over an empty canvas to end it later. Deferred
+                    // because draw() runs inside Google Maps' render pass: a consumer
+                    // reacting to hover-end may synchronously push new layer state,
+                    // re-entering draw().
+                    if (lastHoveredFeatureId !== null) {
+                        const hoverEndId = lastHoveredFeatureId;
+                        lastHoveredFeatureId = null;
+                        queueMicrotask(() => {
+                            layerPort.emitEntityInteraction("hover-end", {
+                                type: "marker",
+                                id: hoverEndId,
+                                position: [0, 0],
+                                clusterId: null,
+                            });
+                        });
+                    }
+                    canvas.style.cursor = MAP_CURSORS.default;
+                    this.boundsDeferScheduled = false;
+                    return;
+                }
                 // Read viewport state from the map this overlay is CURRENTLY attached to.
                 // reconnectAllSources() re-parents the overlay onto recreated map instances
                 // (theme or renderingType changes); the closed-over `map` parameter keeps
@@ -1867,6 +1905,11 @@ class GoogleMapsLayerPort {
                 if (bounds) {
                     const ne = projection.fromLatLngToDivPixel(bounds.getNorthEast());
                     const sw = projection.fromLatLngToDivPixel(bounds.getSouthWest());
+                    // Runtime verification (SAGA-879, PR #25976): fromLatLngToDivPixel
+                    // projects to the world copy nearest the map center, so 180°-crossing
+                    // viewports keep a positive width and take this path. Only a
+                    // full-world viewport (bounds lng -180..180, raw width ~0) fails the
+                    // guard, deliberately falling through to the marker-bbox fallback.
                     if (ne !== null && sw !== null && ne.x - sw.x > 1 && sw.y - ne.y > 1) {
                         width = ne.x - sw.x + pad * 2;
                         height = sw.y - ne.y + pad * 2;
@@ -1898,6 +1941,9 @@ class GoogleMapsLayerPort {
                             maxY = pt.y;
                     }
                     if (!isFinite(minX)) {
+                        // Markers exist but are not projectable yet (the empty list is
+                        // handled at the top of draw(), so this is always the transient
+                        // case) — retry once the frame after the projection settles.
                         if (!this.boundsDeferScheduled) {
                             this.boundsDeferScheduled = true;
                             requestAnimationFrame(() => {

package/migrations/entry.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"entry.js","sourceRoot":"","sources":["../../../../../libs/react/map-adapter-google/migrations/entry.ts"],"names":[],"mappings":"","sourcesContent":["export {};\n"]}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/react-map-adapter-google",
-  "version": "0.0.137",
+  "version": "0.0.140-alpha-b0d834f66ac.0",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {
@@ -14,12 +14,12 @@
     "@loaders.gl/core": "~4.5.2",
     "@luma.gl/core": "^9.4.1",
     "@luma.gl/engine": "^9.4.1",
-    "@trackunit/react-map-adapter-shared": "0.0.148",
-    "@trackunit/react-map-color-utils": "0.0.123",
-    "@trackunit/geo-json-utils": "1.17.1",
+    "@trackunit/react-map-adapter-shared": "0.0.150-alpha-b0d834f66ac.0",
+    "@trackunit/react-map-color-utils": "0.0.125-alpha-b0d834f66ac.0",
+    "@trackunit/geo-json-utils": "1.17.3-alpha-b0d834f66ac.0",
     "@googlemaps/markerclusterer": "^2.6.2",
     "@vis.gl/react-google-maps": "^1.7.1",
-    "es-toolkit": "^1.39.10",
+    "es-toolkit": "^1.52.0",
     "react-device-detect": "^2.2.3",
     "zod": "^3.25.76"
   },