@trackunit/geo-json-utils 1.15.21 → 1.15.22-alpha-3e7014314a8.0

package/index.cjs.js

@@ -900,20 +900,12 @@ 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);
@@ -1285,15 +1277,6 @@ 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,20 +898,12 @@ 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);
@@ -1283,15 +1275,6 @@ 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.21",
+  "version": "1.15.22-alpha-3e7014314a8.0",
   "repository": "https://github.com/Trackunit/manager",
   "license": "SEE LICENSE IN LICENSE.txt",
   "engines": {

package/src/GeoJsonUtils.d.ts

@@ -100,15 +100,6 @@ 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;
 /**

package/migrations/entry.js.map

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