From 3158537488c84fcbdfe070cae7cdd6b857a4ca75 Mon Sep 17 00:00:00 2001
From: Matt <matt@echo6.co>
Date: Sat, 2 May 2026 05:01:02 +0000
Subject: [PATCH] fix: Correct MultiPolygon coordinate flattening for fitBounds

MultiPolygon coordinates need .flat(2) not .flat(1) to get actual
coordinate pairs. With flat(1), we were iterating over rings instead
of coordinates, causing invalid lat values > 90.

Also added bounds validation before fitBounds to catch future issues.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/components/MapView.jsx | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/src/components/MapView.jsx b/src/components/MapView.jsx
index a279c9d..5d646f5 100644
--- a/src/components/MapView.jsx
+++ b/src/components/MapView.jsx
@@ -2401,7 +2401,7 @@ const MapView = forwardRef(function MapView(_, ref) {
           try {
             const coords = boundaryGeometry.type === 'Polygon'
               ? boundaryGeometry.coordinates[0]
-              : boundaryGeometry.coordinates.flat(1)
+              : boundaryGeometry.coordinates.flat(2)
 
             if (coords.length > 0) {
               let minLng = Infinity, maxLng = -Infinity, minLat = Infinity, maxLat = -Infinity
@@ -2411,11 +2411,17 @@ const MapView = forwardRef(function MapView(_, ref) {
                 if (lat < minLat) minLat = lat
                 if (lat > maxLat) maxLat = lat
               }
-              map.fitBounds([[minLng, minLat], [maxLng, maxLat]], {
-                padding: 50,
-                duration: 700,
-                maxZoom: 16,
-              })
+              // Validate bounds before fitting
+              if (minLng >= -180 && maxLng <= 180 && minLat >= -90 && maxLat <= 90 &&
+                  minLng < maxLng && minLat < maxLat) {
+                map.fitBounds([[minLng, minLat], [maxLng, maxLat]], {
+                  padding: 50,
+                  duration: 700,
+                  maxZoom: 16,
+                })
+              } else {
+                console.warn('Invalid bounds:', { minLng, maxLng, minLat, maxLat })
+              }
             }
           } catch (e) {
             console.warn('fitBounds error:', e)