@mapvx/web-js 1.3.0-dev.0 → 1.3.0-dev.1

package/dist/umd/index.js

@@ -21542,7 +21542,7 @@ var rollbar_umd_min_default = /*#__PURE__*/__webpack_require__.n(rollbar_umd_min
 // version placeholder is replaced with the package version by
 // scripts/inject-build-defines.js. The access token falls back to the MapVX
 // SDK-web Rollbar project token when none is injected at build time.
-var SDK_VERSION =  true ? "1.3.0-dev.0" : 0;
+var SDK_VERSION =  true ? "1.3.0-dev.1" : 0;
 var ACCESS_TOKEN =  true ? "1e7e62a3edc14868b4098d653d0f33b9" : 0;
 var _rollbarConfig = {
   accessToken: ACCESS_TOKEN,
@@ -21589,7 +21589,7 @@ function logger_toPrimitive(t, r) { if ("object" != logger_typeof(t) || !t) retu
 // by scripts/inject-build-defines.js. The Countly key falls back to the
 // MapVX key when none is injected at build time.
 var IS_DEBUG =  true ? false : 0;
-var logger_SDK_VERSION =  true ? "1.3.0-dev.0" : 0;
+var logger_SDK_VERSION =  true ? "1.3.0-dev.1" : 0;
 var COUNTLY_API_KEY =  true ? "f0c8d3b96d336e857a8628f49dd1baf7d7add0e9" : 0;
 var logger_countly = {
   apiKey: COUNTLY_API_KEY,
@@ -165674,6 +165674,9 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
           this.escalatorScene.remove(lastChild);
         }
       }
+
+      // Guard against an invalid initial center before it reaches fromLngLat.
+      if (!this.initialCenter || !Number.isFinite(this.initialCenter.lng) || !Number.isFinite(this.initialCenter.lat)) return;
       var refMercator = maplibre_gl.MercatorCoordinate.fromLngLat(this.initialCenter, 0);
       var meterScale = refMercator.meterInMercatorCoordinateUnits();
       var _iterator3 = map_createForOfIteratorHelper(positions),
@@ -165681,6 +165684,8 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
       try {
         for (_iterator3.s(); !(_step3 = _iterator3.n()).done;) {
           var pos = _step3.value;
+          // Defense in depth: never feed a non-finite coordinate to fromLngLat.
+          if (!Number.isFinite(pos.lng) || !Number.isFinite(pos.lat)) continue;
           var target = maplibre_gl_default().MercatorCoordinate.fromLngLat([pos.lng, pos.lat], 0);
           var sx = (target.x - refMercator.x) / meterScale;
           var sz = (target.y - refMercator.y) / meterScale;
@@ -165722,18 +165727,47 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
       try {
         for (_iterator4.s(); !(_step4 = _iterator4.n()).done;) {
           var f = _step4.value;
-          var coords = f.geometry.coordinates;
-          if (!coords || coords.length < 2) continue;
-          var start = coords[0];
-          var end = coords[coords.length - 1];
-          var midLng = (start[0] + end[0]) / 2;
-          var midLat = (start[1] + end[1]) / 2;
-          var bearing = Math.atan2(end[0] - start[0], end[1] - start[1]);
-          allMids.push({
-            lng: midLng,
-            lat: midLat,
-            bearing: bearing
-          });
+          var geom = f.geometry;
+
+          // queryRenderedFeatures may return a LineString or, when a line is split
+          // across tile boundaries, a MultiLineString. The previous code assumed
+          // LineString and read coords[0]/coords[n-1] as [lng,lat], producing
+          // NaN for MultiLineString (coordinates is number[][][]). Normalize both
+          // shapes to a list of line parts (Position[][]).
+          var lineParts = void 0;
+          if (geom.type === "LineString") {
+            lineParts = [geom.coordinates];
+          } else if (geom.type === "MultiLineString") {
+            lineParts = geom.coordinates;
+          } else {
+            continue;
+          }
+          var _iterator5 = map_createForOfIteratorHelper(lineParts),
+            _step5;
+          try {
+            for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
+              var coords = _step5.value;
+              if (!coords || coords.length < 2) continue;
+              var start = coords[0];
+              var end = coords[coords.length - 1];
+              var midLng = (start[0] + end[0]) / 2;
+              var midLat = (start[1] + end[1]) / 2;
+              var bearing = Math.atan2(end[0] - start[0], end[1] - start[1]);
+
+              // Skip any non-finite result so a bad feature can never poison the
+              // cache (and later crash MercatorCoordinate.fromLngLat).
+              if (!Number.isFinite(midLng) || !Number.isFinite(midLat) || !Number.isFinite(bearing)) continue;
+              allMids.push({
+                lng: midLng,
+                lat: midLat,
+                bearing: bearing
+              });
+            }
+          } catch (err) {
+            _iterator5.e(err);
+          } finally {
+            _iterator5.f();
+          }
         }
 
         // Merge points within ~15m of each other into one
@@ -165828,25 +165862,25 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
           });
           if (!roomFeatures.length) return null;
           var paths = [];
-          var _iterator5 = map_createForOfIteratorHelper(roomFeatures),
-            _step5;
+          var _iterator6 = map_createForOfIteratorHelper(roomFeatures),
+            _step6;
           try {
             var _loop = function _loop() {
               var _f$properties5;
-              var f = _step5.value;
+              var f = _step6.value;
               var z = (_f$properties5 = f.properties) !== null && _f$properties5 !== void 0 && _f$properties5.height ? Number(f.properties.height) : 5;
               var geom = f.geometry;
               var polygons = geom.type === "Polygon" ? [geom.coordinates] : geom.type === "MultiPolygon" ? geom.coordinates : [];
-              var _iterator6 = map_createForOfIteratorHelper(polygons),
-                _step6;
+              var _iterator7 = map_createForOfIteratorHelper(polygons),
+                _step7;
               try {
-                for (_iterator6.s(); !(_step6 = _iterator6.n()).done;) {
-                  var rings = _step6.value;
-                  var _iterator7 = map_createForOfIteratorHelper(rings),
-                    _step7;
+                for (_iterator7.s(); !(_step7 = _iterator7.n()).done;) {
+                  var rings = _step7.value;
+                  var _iterator8 = map_createForOfIteratorHelper(rings),
+                    _step8;
                   try {
-                    for (_iterator7.s(); !(_step7 = _iterator7.n()).done;) {
-                      var ring = _step7.value;
+                    for (_iterator8.s(); !(_step8 = _iterator8.n()).done;) {
+                      var ring = _step8.value;
                       paths.push({
                         path: ring.map(function (_ref) {
                           var _ref2 = map_slicedToArray(_ref, 2),
@@ -165857,24 +165891,24 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
                       });
                     }
                   } catch (err) {
-                    _iterator7.e(err);
+                    _iterator8.e(err);
                   } finally {
-                    _iterator7.f();
+                    _iterator8.f();
                   }
                 }
               } catch (err) {
-                _iterator6.e(err);
+                _iterator7.e(err);
               } finally {
-                _iterator6.f();
+                _iterator7.f();
               }
             };
-            for (_iterator5.s(); !(_step5 = _iterator5.n()).done;) {
+            for (_iterator6.s(); !(_step6 = _iterator6.n()).done;) {
               _loop();
             }
           } catch (err) {
-            _iterator5.e(err);
+            _iterator6.e(err);
           } finally {
-            _iterator5.f();
+            _iterator6.f();
           }
           return new path_layer(map_objectSpread(map_objectSpread({}, props), {}, {
             id: "".concat(props.id, "-paths"),
@@ -166673,11 +166707,11 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
         // lands immediately before it, so a bucket processed later paints above
         // the ones processed earlier. Canonical order keeps
         // belowLabels < aboveBasemap < top regardless of circle insertion order.
-        var _iterator8 = map_createForOfIteratorHelper(CIRCLE_RENDER_ORDERS),
-          _step8;
+        var _iterator9 = map_createForOfIteratorHelper(CIRCLE_RENDER_ORDERS),
+          _step9;
         try {
-          for (_iterator8.s(); !(_step8 = _iterator8.n()).done;) {
-            var order = _step8.value;
+          for (_iterator9.s(); !(_step9 = _iterator9.n()).done;) {
+            var order = _step9.value;
             if (!ordersInUse.has(order)) continue;
             var ids = circleLayerIdsFor(order);
             var beforeId = this.circleBeforeIdFor(order);
@@ -166715,9 +166749,9 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
             }
           }
         } catch (err) {
-          _iterator8.e(err);
+          _iterator9.e(err);
         } finally {
-          _iterator8.f();
+          _iterator9.f();
         }
       } catch (error) {
         // Style may not be loaded yet; the styledata listener re-adds the layers
@@ -166759,15 +166793,27 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
           floorKey: floorKeyString
         });
       }
-      if (this.mode === "3D") {
-        this.updateDeckOverlay();
-        this.placeEscalators();
-      }
+
+      // Critical base-map updates run first and must never be blocked by an
+      // optional 3D decoration failing. A throw in placeEscalators used to abort
+      // this method before updateFiltersTo, leaving the base polygons unfiltered
+      // (showing the wrong floor) while the caller's empty catch hid the error.
       this.updateFiltersTo(floorKeyString);
       this.updateMarkersTo(floorKeyString);
       this.refreshCircles();
       this.routeController.updateRouteLayers(floorKeyString);
       this.routeController.updateRouteMarkerVisibility(floorKeyString);
+
+      // 3D decorations are best-effort: isolate so any failure here can never
+      // break base-map rendering.
+      if (this.mode === "3D") {
+        try {
+          this.updateDeckOverlay();
+          this.placeEscalators();
+        } catch (error) {
+          this.logError(error, "filterByFloorKey: 3D layer update failed");
+        }
+      }
     }
   }, {
     key: "getContainer",
@@ -166973,11 +167019,11 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
         var minLng;
         var maxLat;
         var maxLng;
-        var _iterator9 = map_createForOfIteratorHelper(coordinates),
-          _step9;
+        var _iterator0 = map_createForOfIteratorHelper(coordinates),
+          _step0;
         try {
-          for (_iterator9.s(); !(_step9 = _iterator9.n()).done;) {
-            var _coord = _step9.value;
+          for (_iterator0.s(); !(_step0 = _iterator0.n()).done;) {
+            var _coord = _step0.value;
             var lat = _coord.lat,
               lng = _coord.lng;
             minLat = minLat === undefined ? lat : Math.min(minLat, lat);
@@ -166986,9 +167032,9 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
             maxLng = maxLng === undefined ? lng : Math.max(maxLng, lng);
           }
         } catch (err) {
-          _iterator9.e(err);
+          _iterator0.e(err);
         } finally {
-          _iterator9.f();
+          _iterator0.f();
         }
         var bounds = [minLng !== null && minLng !== void 0 ? minLng : 0, minLat !== null && minLat !== void 0 ? minLat : 0, maxLng !== null && maxLng !== void 0 ? maxLng : 0, maxLat !== null && maxLat !== void 0 ? maxLat : 0];
 
@@ -167193,7 +167239,7 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
       var _this18 = this;
       this.map.on("click", /*#__PURE__*/function () {
         var _ref6 = map_asyncToGenerator(/*#__PURE__*/map_regenerator().m(function _callee3(e) {
-          var features, roomFeatures, touchedFeature, ref, _clickedPlace, _ref7, _clickedPlace$mapvxId, _clickedPlace2, _clickedPlace3, clickedPlace, _iterator0, _step0, potentialParentPlace, cachedSubPlaces, cachedPlaces, clickedPlaceId, _t2;
+          var features, roomFeatures, touchedFeature, ref, _clickedPlace, _ref7, _clickedPlace$mapvxId, _clickedPlace2, _clickedPlace3, clickedPlace, _iterator1, _step1, potentialParentPlace, cachedSubPlaces, cachedPlaces, clickedPlaceId, _t2;
           return map_regenerator().w(function (_context3) {
             while (1) switch (_context3.p = _context3.n) {
               case 0:
@@ -167218,15 +167264,15 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
                   _context3.n = 7;
                   break;
                 }
-                _iterator0 = map_createForOfIteratorHelper(_this18.potentialParentPlaces);
+                _iterator1 = map_createForOfIteratorHelper(_this18.potentialParentPlaces);
                 _context3.p = 1;
-                _iterator0.s();
+                _iterator1.s();
               case 2:
-                if ((_step0 = _iterator0.n()).done) {
+                if ((_step1 = _iterator1.n()).done) {
                   _context3.n = 4;
                   break;
                 }
-                potentialParentPlace = _step0.value;
+                potentialParentPlace = _step1.value;
                 cachedSubPlaces = _this18.repository.getCachedSubPlaces(potentialParentPlace.mapvxId);
                 if (!cachedSubPlaces) {
                   _context3.n = 3;
@@ -167249,10 +167295,10 @@ var InternalMapVXMap = /*#__PURE__*/function (_Loggeable) {
               case 5:
                 _context3.p = 5;
                 _t2 = _context3.v;
-                _iterator0.e(_t2);
+                _iterator1.e(_t2);
               case 6:
                 _context3.p = 6;
-                _iterator0.f();
+                _iterator1.f();
                 return _context3.f(6);
               case 7:
                 if (!(clickedPlace === undefined)) {