@itwin/core-geometry 5.12.0-dev.9 → 5.13.0-dev.1

package/lib/cjs/curve/RegionOps.js

@@ -117,9 +117,10 @@ class RegionOps {
         return undefined;
     }
     /**
-     * Return an area tolerance for a given xy-range and optional distance tolerance.
+     * Return an area tolerance for a given xy-range and distance tolerance.
      * @param range range of planar region to tolerance.
-     * @param distanceTolerance optional absolute distance tolerance.
+     * @param distanceTolerance optional absolute distance tolerance. Default is [[Geometry.smallMetricDistance]].
+     * @see [[computeMinimumArea]] for an area tolerance unscaled by geometry size, which is more widely applicable for degeneracy testing.
      */
     static computeXYAreaTolerance(range, distanceTolerance = Geometry_1.Geometry.smallMetricDistance) {
         // ensure the result is nonzero: we never want to report a zero-area loop as a signed-area loop
@@ -130,6 +131,16 @@ class RegionOps {
         const halfDistTol = 0.5 * Math.abs(distanceTolerance);
         return halfDistTol * (range.xLength() + range.yLength() + halfDistTol);
     }
+    /**
+     * Return an area tolerance for a given distance tolerance.
+     * * The result can be used to determine whether a region's area is small enough to be considered degenerate.
+     * @param distanceTolerance optional minimum absolute distance between unique points. Default is [[Geometry.smallMetricDistance]].
+     */
+    static computeMinimumArea(distanceTolerance = Geometry_1.Geometry.smallMetricDistance) {
+        const areaTol = Math.PI * distanceTolerance * distanceTolerance;
+        const minAreaTol = Geometry_1.Geometry.smallFloatingPoint * 10; // observed area 2e-15 computed for a degenerate loop
+        return areaTol < minAreaTol ? minAreaTol : areaTol;
+    }
     /**
      * Return a (signed) xy area for a region.
      * * The input region should lie in a plane parallel to the xy-plane, as z-coords will be ignored.
@@ -378,13 +389,23 @@ class RegionOps {
         const localToWorld = FrameBuilder_1.FrameBuilder.createRightHandedFrame(undefined, xyRegion);
         return localToWorld ? localToWorld.origin.z : 0;
     }
+    /** Convert overlapping enums, from public to internal. */
+    static convertToRegionGroupOpType(op) {
+        if (RegionBinaryOpType.Parity === op)
+            return RegionOpsClassificationSweeps_1.RegionGroupOpType.Parity;
+        if (RegionBinaryOpType.Intersection === op)
+            return RegionOpsClassificationSweeps_1.RegionGroupOpType.Intersection;
+        return RegionOpsClassificationSweeps_1.RegionGroupOpType.Union;
+    }
     /**
      * Return areas defined by a boolean operation.
      * @note For best results, input regions should have correctly oriented loops. See [[sortOuterAndHoleLoopsXY]].
      * @note A common use case of this method is to split (a region with) overlapping loops into a `UnionRegion` with
      * adjacent `Loop`s: `regionOut = RegionOps.regionBooleanXY(regionIn, undefined, RegionBinaryOpType.Union)`.
-     * @note The Union operation does not currently attempt to return a region with minimal loops. This may change with
-     * future development.
+     * Such a union can be further simplified by passing the option `RegionBooleanXYOptions.simplifyUnion = true`.
+     * @note By default, this method does not discover holes implicitly formed by input loops. For example, the union of
+     * the four trapezoids that comprise a picture frame is not returned as a [[ParityRegion]] unless the option
+     * `simplifyUnion = true`.
      * @param loopsA first set of loops (treated as a union)
      * @param loopsB second set of loops (treated as a union)
      * @param operation indicates Union, Intersection, Parity, AMinusB, or BMinusA
@@ -392,48 +413,69 @@ class RegionOps {
      * @returns a region resulting from merging input loops and the boolean operation.
      */
     static regionBooleanXY(loopsA, loopsB, operation, mergeToleranceOrOptions = Geometry_1.Geometry.smallMetricDistance) {
-        let mergeTolerance;
-        let simplifyUnion;
-        if (typeof mergeToleranceOrOptions === "number") {
-            mergeTolerance = mergeToleranceOrOptions;
-            simplifyUnion = false;
-        }
-        else {
-            mergeTolerance = mergeToleranceOrOptions.mergeTolerance ?? Geometry_1.Geometry.smallMetricDistance;
-            simplifyUnion = mergeToleranceOrOptions.simplifyUnion ?? false;
-        }
-        let result = UnionRegion_1.UnionRegion.create();
-        const context = RegionOpsClassificationSweeps_1.RegionBooleanContext.create(RegionOpsClassificationSweeps_1.RegionGroupOpType.Union, RegionOpsClassificationSweeps_1.RegionGroupOpType.Union);
+        const optionIsNumber = typeof mergeToleranceOrOptions === "number";
+        const mergeTolerance = optionIsNumber ? mergeToleranceOrOptions : mergeToleranceOrOptions.mergeTolerance ?? Geometry_1.Geometry.smallMetricDistance;
+        const simplifyUnion = optionIsNumber ? false : mergeToleranceOrOptions.simplifyUnion ?? false;
+        const operationGroupA = optionIsNumber ? RegionOpsClassificationSweeps_1.RegionGroupOpType.Union : RegionOps.convertToRegionGroupOpType(mergeToleranceOrOptions.operationGroupA);
+        const operationGroupB = optionIsNumber ? RegionOpsClassificationSweeps_1.RegionGroupOpType.Union : RegionOps.convertToRegionGroupOpType(mergeToleranceOrOptions.operationGroupB);
+        let result;
+        const context = RegionOpsClassificationSweeps_1.RegionBooleanContext.create(operationGroupA, operationGroupB);
         context.addMembers(loopsA, loopsB);
         context.annotateAndMergeCurvesInGraph(mergeTolerance);
         const bridgeMask = Graph_1.HalfEdgeMask.BRIDGE_EDGE;
         const visitMask = context.graph.grabMask(false);
-        const range = context.groupA.range().union(context.groupB.range());
-        const areaTol = this.computeXYAreaTolerance(range, mergeTolerance);
+        // use our own outMask: we cannot use EXTERIOR as sweep clears it before processing each component
+        const outMask = simplifyUnion ? context.graph.grabMask(false) : Graph_1.HalfEdgeMask.NULL_MASK;
+        const outSeeds = simplifyUnion ? [] : undefined;
+        context.graph.clearMask(visitMask | outMask);
+        const areaTol = this.computeMinimumArea(mergeTolerance);
         const z = RegionOps.getZCoordinate(operation === RegionBinaryOpType.BMinusA ? loopsB : loopsA);
         const options = { compress: true, closureTol: mergeTolerance, bridgeMask, visitMask, z };
+        let numFacesIn = 0;
         context.runClassificationSweep(operation, (_graph, face, faceType, area) => {
             // ignore danglers and null faces, but not 2-edge "banana" faces with nonzero area
             if (face.countEdgesAroundFace() < 2)
                 return;
             if (Math.abs(area) < areaTol)
                 return;
-            if (faceType === 1) {
+            if (simplifyUnion) { // process the outer/negative faces later
+                if (faceType === 1) {
+                    numFacesIn++;
+                }
+                else {
+                    face.setMaskAroundFace(outMask);
+                    outSeeds?.push(face);
+                }
+            }
+            else if (faceType === 1) { // collect the inner faces now
                 const loopOrParityRegion = PlanarSubdivision_1.PlanarSubdivision.createLoopOrParityRegionInFace(face, options);
-                if (loopOrParityRegion)
-                    result.tryAddChild(loopOrParityRegion);
+                if (loopOrParityRegion) {
+                    if (!result)
+                        result = UnionRegion_1.UnionRegion.create();
+                    if (result.tryAddChild(loopOrParityRegion))
+                        numFacesIn++;
+                }
             }
         });
-        context.graph.dropMask(visitMask);
-        if (simplifyUnion && operation === RegionBinaryOpType.Union) {
-            const signedLoops = RegionOps.constructAllXYRegionLoops(result);
-            if (signedLoops) {
-                const outerLoops = signedLoops.map((component) => component.negativeAreaLoops).flat();
-                if (outerLoops.length > 0)
-                    result = UnionRegion_1.UnionRegion.create(...outerLoops);
+        // the outer faces are marked; now we collect their boundary super face loops and sort them
+        if (simplifyUnion && numFacesIn > 0 && outSeeds && outSeeds.length > 0) {
+            context.graph.clearMask(visitMask);
+            const outLoops = [];
+            for (const outSeed of outSeeds) {
+                const superFace = [];
+                const skipEdge = (he) => he.isMaskSet(visitMask) || !he.isMaskSet(outMask) || he.edgeMate.isMaskSet(outMask);
+                const announceEdge = (he) => superFace.push(he);
+                if (outSeed.announceEdgesInSuperFace(skipEdge, announceEdge)) {
+                    superFace.forEach((he) => he.setMask(visitMask));
+                    const outLoop = PlanarSubdivision_1.PlanarSubdivision.createLoopInFace(superFace, options);
+                    if (outLoop)
+                        outLoops.push(outLoop);
+                }
             }
+            result = this.sortOuterAndHoleLoopsXY(outLoops);
         }
-        return this.simplifyRegion(result);
+        context.graph.dropMask(visitMask | outMask);
+        return result ? this.simplifyRegion(result) : undefined;
     }
     /**
      * Return a polyface whose facets are a boolean operation between the input regions.
@@ -750,86 +792,216 @@ class RegionOps {
         return SortablePolygon_1.SortablePolygon.sortAsAnyRegion(loopAndArea);
     }
     /**
-     * Simplify the graph by removing bridge edges that do not serve to connect inner and outer loops.
-     * * If edgeTags are `CurveLocationDetail`s, e.g., as set by `PlanarSubdivision.assembleHalfEdgeGraph`, attempt to heal edges split by removed bridge edges.
+     * Compare two edges' geometry from CurveLocationDetail edge tags.
+     * @returns an object with following properties:
+     * * sameCurves: whether the edges' curve definitions are equivalent.
+     * * reversedCurves: whether the edges' curve definitions are equivalent after reversing one of them.
+     * * senseEdge0/1: whether the topological edge is oriented in the same direction as the curve geometry, or the opposite direction.
+     */
+    static getEdgeGeometry(edge0, edge1) {
+        let sameCurves = false;
+        let reversedCurves = false;
+        let senseEdge0 = false;
+        let senseEdge1 = false;
+        if (edge0.edgeTag instanceof CurveLocationDetail_1.CurveLocationDetail && edge0.edgeTag.curve !== undefined && edge0.edgeTag.isInterval() && edge0.sortData &&
+            edge1.edgeTag instanceof CurveLocationDetail_1.CurveLocationDetail && edge1.edgeTag.curve !== undefined && edge1.edgeTag.isInterval() && edge1.sortData) {
+            sameCurves = edge0.edgeTag.curve === edge1.edgeTag.curve || edge0.edgeTag.curve.isAlmostEqual(edge1.edgeTag.curve);
+            reversedCurves = !sameCurves && edge0.edgeTag.curve.isAlmostEqual(edge1.edgeTag.curve.clone().reverse());
+            senseEdge0 = edge0.sortData > 0;
+            senseEdge1 = edge1.sortData > 0;
+        }
+        return { sameCurves, reversedCurves, senseEdge0, senseEdge1 };
+    }
+    /**
+     * Compare two edges' orientations. They do not have to be adjacent.
+     * * Requires CurveLocationDetail edge tags.
+     * @returns an object with following properties:
+     * * diametric: whether edges have same/reversed curves and are oriented diametrically opposite each other.
+     * * reversedCurves: whether the underlying curve definitions are reversed.
+     * * senseEdge0/1: whether the topological edge is oriented in the same direction as the curve geometry, or the opposite direction.
+     */
+    static areEdgesDiametric(edge0, edge1) {
+        let diametric = false;
+        let reversedCurves = false;
+        let senseEdge0 = false;
+        let senseEdge1 = false;
+        if (edge0 !== edge1) {
+            const data = RegionOps.getEdgeGeometry(edge0, edge1);
+            reversedCurves = data.reversedCurves;
+            senseEdge0 = data.senseEdge0;
+            senseEdge1 = data.senseEdge1;
+            diametric = (data.sameCurves && (senseEdge0 !== senseEdge1)) || (reversedCurves && (senseEdge0 === senseEdge1));
+        }
+        return { diametric, reversedCurves, senseEdge0, senseEdge1 };
+    }
+    /**
+     * Determine if the edges at a vertex were created by splitting an edge or null face.
+     * * Requires CurveLocationDetail edge tags.
+     * @param he an edge created by the split operation.
+     * @param vs another edge at the same vertex. If undefined, use `vs = he.vertexSuccessor` and check for a
+     * degree-2 vertex (e.g., split edge); otherwise, ignore the degree check (e.g., inputs are a null face pair).
+     */
+    static isHealableEdgeOrNullFace(he, vs) {
+        if (vs) {
+            if (!he.findAroundVertex(vs))
+                return false;
+        }
+        else {
+            vs = he.vertexSuccessor;
+            if (vs.vertexSuccessor !== he)
+                return false; // not a split edge
+        }
+        const directionData = RegionOps.areEdgesDiametric(he, vs);
+        if (!directionData.diametric)
+            return false;
+        const heVertexFraction = directionData.senseEdge0 ? he.edgeTag.fraction : he.edgeTag.fraction1;
+        let vsVertexFraction = directionData.senseEdge1 ? vs.edgeTag.fraction : vs.edgeTag.fraction1;
+        if (directionData.reversedCurves)
+            vsVertexFraction = 1.0 - vsVertexFraction;
+        return Geometry_1.Geometry.isSmallRelative(heVertexFraction - vsVertexFraction); // unrelated vertex fractions are not healable
+    }
+    /**
+     * Determine if the vertex loop resulted from splitting a null face into two null faces (and thus can be healed).
+     * * Requires CurveLocationDetail edge tags.
+     * * If there are multiple null faces at the vertex (a null face bundle), they must be adjacent and positioned in
+     * diametrically opposite pairs, each of which resulted from splitting a null face. This implies there are exactly
+     * two non-null faces opposite each other in the vertex loop.
+     * @returns an object with properties:
+     * * isHealable: whether the input is a healable vertex loop in a null face bundle
+     * * nullFacePairs: if `isHealable` is true, an array of the matching null face pairs around the vertex.
+     */
+    static isHealableNullFaceBundle(vertex) {
+        const nonNullFaceIndex = [];
+        const vertexLoop = [];
+        // lambda: null face detector, with curve check to rule out banana faces
+        const isNullFace = (face) => {
+            const fSucc = face.faceSuccessor;
+            return (fSucc.faceSuccessor === face) && RegionOps.areEdgesDiametric(face, fSucc).diametric;
+        };
+        vertex.announceEdgesAroundVertex((e) => {
+            if (!isNullFace(e))
+                nonNullFaceIndex.push(vertexLoop.length);
+            vertexLoop.push(e);
+        });
+        const degree = vertexLoop.length;
+        if (degree < 4 || nonNullFaceIndex.length !== 2 || Geometry_1.Geometry.modulo(degree, 2) > 0)
+            return { isHealable: false };
+        const iNonNull0 = nonNullFaceIndex[0];
+        if (nonNullFaceIndex[1] - iNonNull0 !== degree / 2)
+            return { isHealable: false };
+        const nullFacePairs = [];
+        for (let i = 1; i < degree / 2; i++) {
+            const iNullFace = Geometry_1.Geometry.modulo(iNonNull0 + i, degree);
+            const iMatchingNullFace = Geometry_1.Geometry.modulo(2 * iNonNull0 - iNullFace, degree);
+            if (!RegionOps.isHealableEdgeOrNullFace(vertexLoop[iNullFace], vertexLoop[iMatchingNullFace]))
+                return { isHealable: false };
+            nullFacePairs.push([vertexLoop[iNullFace], vertexLoop[iMatchingNullFace]]);
+        }
+        return { isHealable: true, nullFacePairs };
+    }
+    /** Heal a split edge by removing the redundant vertex loop and extending the surviving curve details. */
+    static healSplitEdgeAndCurve(v) {
+        const vs = v.vertexSuccessor;
+        const fPred = Graph_1.HalfEdge.healEdge(v, false);
+        if (!fPred)
+            return;
+        // lambda to extend the surviving node's curve interval. Inputs are on the same side of the healed edge.
+        const extendDetailInterval = (doomed, survivor) => {
+            const doomedData = RegionOps.getEdgeGeometry(doomed, survivor);
+            (0, core_bentley_1.assert)(doomedData.sameCurves || doomedData.reversedCurves, "expect related curves");
+            let newEndFraction = doomedData.senseEdge0 ? doomed.edgeTag.fraction1 : doomed.edgeTag.fraction;
+            if (doomedData.reversedCurves)
+                newEndFraction = 1.0 - newEndFraction;
+            const newEndPoint = doomedData.senseEdge0 ? doomed.edgeTag.point1 : doomed.edgeTag.point;
+            if (doomedData.senseEdge1)
+                survivor.edgeTag.captureFraction1Point1(newEndFraction, newEndPoint);
+            else
+                survivor.edgeTag.captureFractionPoint(newEndFraction, newEndPoint);
+        };
+        extendDetailInterval(v, fPred);
+        extendDetailInterval(vs, fPred.edgeMate);
+    }
+    /**
+     * Disconnect extraneous bridge edges, which remain in the graph as isolated edges.
      * @param graph half edges to process.
-     * @param isBridgeEdge optional function to identify a bridge edge. Default looks for `HalfEdgeMask.BRIDGE_EDGE`.
-     * @param faceToArea optional function to compute face area. Default is `HalfEdgeGraphSearch.signedFaceArea`.
-     * @returns the number of extraneous bridge edges removed from the graph.
-     * @internal
+     * @param isBridgeEdge optional callback to identify a bridge edge.
+     * @param faceToArea optional callback to compute face area.
+     * @returns array of surviving vertices formerly adjacent to the isolated bridge edges (e.g., for the caller to heal).
      */
-    static removeExtraneousBridgeEdges(graph, isBridgeEdge, faceToArea) {
+    static isolateExtraneousBridgeEdges(graph, isBridgeEdge, faceToArea) {
         const toHeal = [];
         const interiorBridges = [];
-        if (!faceToArea)
-            faceToArea = (node) => HalfEdgeGraphSearch_1.HalfEdgeGraphSearch.signedFaceArea(node);
-        if (!isBridgeEdge)
-            isBridgeEdge = (node) => node.isMaskSet(Graph_1.HalfEdgeMask.BRIDGE_EDGE);
+        const isolateBridgeEdge = (edge) => {
+            if (!edge.isDangling)
+                toHeal.push(edge.vertexSuccessor);
+            if (!edge.edgeMate.isDangling)
+                toHeal.push(edge.edgeMate.vertexSuccessor);
+            edge.isolateEdge();
+        };
         // isolate dangling bridges, bridges separating different faces, and "exterior" bridges in the negative area face
         graph.announceEdges((_g, node) => {
             if (isBridgeEdge(node)) {
-                if (node.isDangling || node.edgeMate.isDangling || !node.findAroundFace(node.edgeMate) || faceToArea(node) < 0.0) {
-                    toHeal.push(node.vertexSuccessor);
-                    toHeal.push(node.edgeMate.vertexSuccessor);
-                    node.isolateEdge();
-                }
-                else {
+                if (node.isDangling || node.edgeMate.isDangling || !node.findAroundFace(node.edgeMate) || faceToArea(node) < 0.0)
+                    isolateBridgeEdge(node);
+                else
                     interiorBridges.push(node);
-                }
             }
             return true;
         });
-        // Relies only on face loop orientation. Doesn't use static HalfEdgeMasks!
-        const isBoundaryEdge = (node) => {
+        // lambda: check if a non-isolated interior bridge edge is extraneous, relying solely on face loop orientation (not HalfEdgeMasks!)
+        const isBridgeEdgeExtraneous = (node) => {
+            if (node.isIsolatedEdge)
+                return false; // already extraneous
             if (faceToArea(node) < 0.0)
                 return true; // exterior face
             if (node.findAroundFace(node.edgeMate))
                 return false; // interior washer face
             return faceToArea(node.edgeMate) < 0.0; // adjacent to exterior face
         };
-        // All bridges in the negative area face were isolated, but this may have promoted other bridges to the
-        // negative area face. Keep isolating these bridge edges until none remain.
+        // All bridges in the negative area face were isolated, but this may have caused surviving bridges to become extraneous
+        // (e.g., by swelling the negative area face around them). Keep isolating these bridge edges until none remain.
         let numIsolatedThisPass;
         do {
             numIsolatedThisPass = 0;
             for (const node of interiorBridges) {
-                if (!node.isIsolatedEdge && isBoundaryEdge(node)) {
-                    toHeal.push(node.vertexSuccessor);
-                    toHeal.push(node.edgeMate.vertexSuccessor);
-                    node.isolateEdge();
+                if (isBridgeEdgeExtraneous(node)) {
+                    isolateBridgeEdge(node);
                     numIsolatedThisPass++;
                 }
             }
         } while (numIsolatedThisPass > 0);
-        // lambda to extend the detail interval on a side of a healed edge
-        const mergeDetails = (he, newFraction, newPoint) => {
-            if (he && he.edgeTag instanceof CurveLocationDetail_1.CurveLocationDetail && he.sortData !== undefined && newFraction !== undefined && newPoint) {
-                if (he.sortData > 0)
-                    he.edgeTag.captureFraction1Point1(newFraction, newPoint);
-                else
-                    he.edgeTag.captureFractionPoint(newFraction, newPoint);
+        return toHeal.filter((edge) => !edge.isIsolatedEdge);
+    }
+    /**
+     * Simplify the graph by removing bridge edges that do not serve to connect inner and outer loops.
+     * * If edgeTags are `CurveLocationDetail`s, e.g., as set by `PlanarSubdivision.assembleHalfEdgeGraph`, attempt to heal edges split by removed bridge edges.
+     * @param graph half edges to process.
+     * @param isBridgeEdge optional callback to identify a bridge edge. Default looks for `HalfEdgeMask.BRIDGE_EDGE`.
+     * @param faceToArea optional callback to compute face area. Default is `HalfEdgeGraphSearch.signedFaceArea`.
+     * @returns the number of extraneous bridge edges removed from the graph.
+     * @internal
+     */
+    static removeExtraneousBridgeEdges(graph, isBridgeEdge, faceToArea) {
+        if (!faceToArea)
+            faceToArea = (node) => HalfEdgeGraphSearch_1.HalfEdgeGraphSearch.signedFaceArea(node);
+        if (!isBridgeEdge)
+            isBridgeEdge = (node) => node.isMaskSet(Graph_1.HalfEdgeMask.BRIDGE_EDGE);
+        const toHeal = RegionOps.isolateExtraneousBridgeEdges(graph, isBridgeEdge, faceToArea);
+        // clean up the isolated bridge edges' original vertex loops, if possible
+        for (const vertex of toHeal) {
+            if (vertex.isDangling || vertex.isIsolatedEdge)
+                continue;
+            if (RegionOps.isHealableEdgeOrNullFace(vertex)) {
+                RegionOps.healSplitEdgeAndCurve(vertex);
+                continue;
             }
-        };
-        // At this point all removable bridges are isolated. Clean up their original vertex loops, if possible.
-        for (const doomedA of toHeal) {
-            const doomedB = doomedA.vertexSuccessor;
-            if ( // are the geometries mergeable?
-            doomedA !== doomedB &&
-                doomedA.edgeTag instanceof CurveLocationDetail_1.CurveLocationDetail && doomedA.sortData !== undefined &&
-                doomedB.edgeTag instanceof CurveLocationDetail_1.CurveLocationDetail && doomedB.sortData !== undefined &&
-                doomedA.edgeTag.curve === doomedB.edgeTag.curve &&
-                doomedA.edgeTag.isInterval() && doomedB.edgeTag.isInterval() &&
-                doomedA.sortData * doomedB.sortData < 0 &&
-                ((doomedA.sortData > 0 && Geometry_1.Geometry.isSmallRelative(doomedA.edgeTag.fraction - doomedB.edgeTag.fraction1)) ||
-                    (doomedA.sortData < 0 && Geometry_1.Geometry.isSmallRelative(doomedA.edgeTag.fraction1 - doomedB.edgeTag.fraction)))) {
-                const survivorA = Graph_1.HalfEdge.healEdge(doomedA, false);
-                if (survivorA) {
-                    const endFractionA = (doomedA.sortData > 0) ? doomedA.edgeTag.fraction1 : doomedA.edgeTag.fraction;
-                    const endPointA = (doomedA.sortData > 0) ? doomedA.edgeTag.point1 : doomedA.edgeTag.point;
-                    mergeDetails(survivorA, endFractionA, endPointA);
-                    const endFractionB = (doomedB.sortData > 0) ? doomedB.edgeTag.fraction1 : doomedB.edgeTag.fraction;
-                    const endPointB = (doomedB.sortData > 0) ? doomedB.edgeTag.point1 : doomedB.edgeTag.point;
-                    mergeDetails(survivorA.edgeMate, endFractionB, endPointB);
+            const bundleData = RegionOps.isHealableNullFaceBundle(vertex);
+            if (bundleData.isHealable && bundleData.nullFacePairs) {
+                for (const [f0, f1] of bundleData.nullFacePairs)
+                    Graph_1.HalfEdge.pinch(f0, f1); // merge the null face pair into a single null face (with 2 split edges)
+                for (const [e0, e1] of bundleData.nullFacePairs) {
+                    RegionOps.healSplitEdgeAndCurve(e0);
+                    RegionOps.healSplitEdgeAndCurve(e1);
                 }
             }
         }
@@ -884,8 +1056,6 @@ class RegionOps {
     static constructAllXYRegionLoops(curvesAndRegions, tolerance = Geometry_1.Geometry.smallMetricDistance, addBridges = true) {
         let primitives = RegionOps.collectCurvePrimitives(curvesAndRegions, undefined, true, true);
         primitives = TransferWithSplitArcs_1.TransferWithSplitArcs.clone(CurveCollection_1.BagOfCurves.create(...primitives)).children;
-        const range = this.curveArrayRange(primitives);
-        const areaTol = this.computeXYAreaTolerance(range, tolerance);
         let hasOpenCurve = false;
         if (addBridges) { // generate a temp graph from ONLY the closed inputs to extract its bridge edges
             const context = RegionOpsClassificationSweeps_1.RegionBooleanContext.create(RegionOpsClassificationSweeps_1.RegionGroupOpType.Union, RegionOpsClassificationSweeps_1.RegionGroupOpType.Union);
@@ -910,7 +1080,8 @@ class RegionOps {
         const graph = PlanarSubdivision_1.PlanarSubdivision.assembleHalfEdgeGraph(primitives, intersections, tolerance);
         if (addBridges && hasOpenCurve)
             RegionOps.removeExtraneousBridgeEdges(graph);
-        return PlanarSubdivision_1.PlanarSubdivision.collectSignedLoopSetsInHalfEdgeGraph(graph, areaTol);
+        const areaTol = this.computeMinimumArea(tolerance);
+        return PlanarSubdivision_1.PlanarSubdivision.collectSignedLoopSetsInHalfEdgeGraph(graph, tolerance, areaTol);
     }
     /**
      * Collect all `CurvePrimitives` in loosely typed input.