@trackunit/geo-json-utils 1.15.20-alpha-aedae0c85f6.0 → 1.15.20

package/index.cjs.js

@@ -900,12 +900,20 @@ const getPolygonFromPointAndRadius = (point, radius) => {
     const deltaLat = (radius / EARTH_RADIUS) * (180 / Math.PI);
     const deltaLon = deltaLat / Math.cos((lat * Math.PI) / 180);
     const coordinates = [];
-    for (let i = 0; i <= pointsCount; i++) {
+    for (let i = 0; i < pointsCount; i++) {
         const angle = i * angleStep;
         const newLat = Math.max(-90, Math.min(90, lat + deltaLat * Math.sin(angle)));
         const newLon = lonCenter + deltaLon * Math.cos(angle);
         coordinates.push([newLon, newLat]);
     }
+    // Close the ring by copying vertex 0 rather than recomputing it at angle 2π: Math.sin(2π) is
+    // -2.449e-16, and near the equator that offset exceeds half an ulp of the latitude, leaving the
+    // ring open and failing RFC 7946 validation.
+    const firstVertex = coordinates[0];
+    if (firstVertex === undefined) {
+        return null;
+    }
+    coordinates.push([...firstVertex]);
     const rawLngs = coordinates.map(c => c[0]);
     const minRawLon = Math.min(...rawLngs);
     const maxRawLon = Math.max(...rawLngs);
@@ -1277,6 +1285,15 @@ const isFullyContainedInGeoJsonPolygon = (polygon1, polygon2) => {
 /**
  * @description Gets the intersection between two GeoJSON polygons/multi-polygons.
  * Returns a MultiPolygon representing the intersection, or null if there is no intersection.
+ *
+ * Delegates to `polygon-clipping`, which can **throw** `"Unable to complete output ring"`
+ * for degenerate or near-self-intersecting geometry that is still schema-valid. Callers must
+ * decide how to handle that: `null` here is reserved for a *provably empty* intersection, so
+ * a thrown failure must not be silently folded into `null` (see SAGA-743). Callers for which
+ * "no overlap" is a safe degradation (e.g. overlap grouping in `@trackunit/react-map`) should
+ * catch and treat the throw as no intersection; callers that turn `null` into "no spatial
+ * filter" (e.g. `mapUtils` viewport/area merges) must instead fall back to a conservative
+ * geometry rather than assume disjointness.
  */
 const getGeoJsonPolygonIntersection = (polygon1, polygon2) => {
     const geom1 = polygon1.type === "MultiPolygon" ? toClipMultiPolygon(polygon1.coordinates) : [toClipPolygon(polygon1.coordinates)];

package/index.esm.js

@@ -898,12 +898,20 @@ const getPolygonFromPointAndRadius = (point, radius) => {
     const deltaLat = (radius / EARTH_RADIUS) * (180 / Math.PI);
     const deltaLon = deltaLat / Math.cos((lat * Math.PI) / 180);
     const coordinates = [];
-    for (let i = 0; i <= pointsCount; i++) {
+    for (let i = 0; i < pointsCount; i++) {
         const angle = i * angleStep;
         const newLat = Math.max(-90, Math.min(90, lat + deltaLat * Math.sin(angle)));
         const newLon = lonCenter + deltaLon * Math.cos(angle);
         coordinates.push([newLon, newLat]);
     }
+    // Close the ring by copying vertex 0 rather than recomputing it at angle 2π: Math.sin(2π) is
+    // -2.449e-16, and near the equator that offset exceeds half an ulp of the latitude, leaving the
+    // ring open and failing RFC 7946 validation.
+    const firstVertex = coordinates[0];
+    if (firstVertex === undefined) {
+        return null;
+    }
+    coordinates.push([...firstVertex]);
     const rawLngs = coordinates.map(c => c[0]);
     const minRawLon = Math.min(...rawLngs);
     const maxRawLon = Math.max(...rawLngs);
@@ -1275,6 +1283,15 @@ const isFullyContainedInGeoJsonPolygon = (polygon1, polygon2) => {
 /**
  * @description Gets the intersection between two GeoJSON polygons/multi-polygons.
  * Returns a MultiPolygon representing the intersection, or null if there is no intersection.
+ *
+ * Delegates to `polygon-clipping`, which can **throw** `"Unable to complete output ring"`
+ * for degenerate or near-self-intersecting geometry that is still schema-valid. Callers must
+ * decide how to handle that: `null` here is reserved for a *provably empty* intersection, so
+ * a thrown failure must not be silently folded into `null` (see SAGA-743). Callers for which
+ * "no overlap" is a safe degradation (e.g. overlap grouping in `@trackunit/react-map`) should
+ * catch and treat the throw as no intersection; callers that turn `null` into "no spatial
+ * filter" (e.g. `mapUtils` viewport/area merges) must instead fall back to a conservative
+ * geometry rather than assume disjointness.
  */
 const getGeoJsonPolygonIntersection = (polygon1, polygon2) => {
     const geom1 = polygon1.type === "MultiPolygon" ? toClipMultiPolygon(polygon1.coordinates) : [toClipPolygon(polygon1.coordinates)];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@trackunit/geo-json-utils",
-  "version": "1.15.20-alpha-aedae0c85f6.0",
+  "version": "1.15.20",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {

package/src/GeoJsonUtils.d.ts

@@ -100,6 +100,15 @@ export declare const isFullyContainedInGeoJsonPolygon: (polygon1: GeoJsonPolygon
 /**
  * @description Gets the intersection between two GeoJSON polygons/multi-polygons.
  * Returns a MultiPolygon representing the intersection, or null if there is no intersection.
+ *
+ * Delegates to `polygon-clipping`, which can **throw** `"Unable to complete output ring"`
+ * for degenerate or near-self-intersecting geometry that is still schema-valid. Callers must
+ * decide how to handle that: `null` here is reserved for a *provably empty* intersection, so
+ * a thrown failure must not be silently folded into `null` (see SAGA-743). Callers for which
+ * "no overlap" is a safe degradation (e.g. overlap grouping in `@trackunit/react-map`) should
+ * catch and treat the throw as no intersection; callers that turn `null` into "no spatial
+ * filter" (e.g. `mapUtils` viewport/area merges) must instead fall back to a conservative
+ * geometry rather than assume disjointness.
  */
 export declare const getGeoJsonPolygonIntersection: (polygon1: GeoJsonPolygon | GeoJsonMultiPolygon, polygon2: GeoJsonPolygon | GeoJsonMultiPolygon) => GeoJsonMultiPolygon | GeoJsonPolygon | null;
 /**