@itwin/ecschema-rpcinterface-tests 5.14.0-dev.7 → 5.14.0-dev.9

package/lib/dist/bundled-tests.js

@@ -35922,6 +35922,7 @@ function isBinaryImageSource(source) {
 __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   getPullChangesIpcChannel: () => (/* binding */ getPullChangesIpcChannel),
+/* harmony export */   getPushChangesIpcChannel: () => (/* binding */ getPushChangesIpcChannel),
 /* harmony export */   ipcAppChannels: () => (/* binding */ ipcAppChannels)
 /* harmony export */ });
 /*---------------------------------------------------------------------------------------------
@@ -35932,9 +35933,16 @@ __webpack_require__.r(__webpack_exports__);
  * @module NativeApp
  */
 /** Get IPC channel name used for reporting progress of pulling changes into iModel.
+ * @param key the key of the briefcase being pulled into.
  * @internal
  */
-const getPullChangesIpcChannel = (iModelId) => `${ipcAppChannels.functions}/pullChanges/${iModelId}`;
+const getPullChangesIpcChannel = (key) => `${ipcAppChannels.functions}/pullChanges/${key}`;
+/** Get IPC channel name used for reporting the progress of the changeset download that [[IpcAppFunctions.pushChanges]] performs before
+ * uploading. Kept distinct from [[getPullChangesIpcChannel]] so that a listener attached for a pull never observes a push's download.
+ * @param key the key of the briefcase being pushed from.
+ * @internal
+ */
+const getPushChangesIpcChannel = (key) => `${ipcAppChannels.functions}/pushChanges/pullProgress/${key}`;
 /** @internal */
 const ipcAppChannels = {
     functions: "itwinjs-core/ipc-app",
@@ -45770,6 +45778,7 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */   getMarkerText: () => (/* reexport safe */ _annotation_TextBlock__WEBPACK_IMPORTED_MODULE_3__.getMarkerText),
 /* harmony export */   getMaximumMajorTileFormatVersion: () => (/* reexport safe */ _tile_TileMetadata__WEBPACK_IMPORTED_MODULE_163__.getMaximumMajorTileFormatVersion),
 /* harmony export */   getPullChangesIpcChannel: () => (/* reexport safe */ _IpcAppProps__WEBPACK_IMPORTED_MODULE_80__.getPullChangesIpcChannel),
+/* harmony export */   getPushChangesIpcChannel: () => (/* reexport safe */ _IpcAppProps__WEBPACK_IMPORTED_MODULE_80__.getPushChangesIpcChannel),
 /* harmony export */   getTileObjectReference: () => (/* reexport safe */ _TileProps__WEBPACK_IMPORTED_MODULE_122__.getTileObjectReference),
 /* harmony export */   iModelTileTreeIdToString: () => (/* reexport safe */ _tile_TileMetadata__WEBPACK_IMPORTED_MODULE_163__.iModelTileTreeIdToString),
 /* harmony export */   iTwinChannel: () => (/* reexport safe */ _ipc_IpcSocket__WEBPACK_IMPORTED_MODULE_74__.iTwinChannel),
@@ -62394,6 +62403,17 @@ class SchemaCache {
             return entry.schemaInfo;
         return undefined;
     }
+    /**
+      * Gets the schema info which matches the provided SchemaKey.  The schema info may be returned before the schema is fully loaded.
+      * Does not await partially loaded schemas.
+      * @param schemaKey The SchemaKey describing the schema to get from the cache.
+      * @param matchType The match type to use when locating the schema
+      */
+    getSchemaInfoSync(schemaKey, matchType = _ECObjects__WEBPACK_IMPORTED_MODULE_0__.SchemaMatchType.Latest) {
+        if (this.count === 0)
+            return undefined;
+        return this.findEntry(schemaKey, matchType)?.schemaInfo;
+    }
     /**
      * Gets the schema which matches the provided SchemaKey.  If the schema is partially loaded an exception will be thrown.
      * @param schemaKey The SchemaKey describing the schema to get from the cache.
@@ -62591,6 +62611,16 @@ class SchemaContext {
     getCachedSchemaSync(schemaKey, matchType = _ECObjects__WEBPACK_IMPORTED_MODULE_0__.SchemaMatchType.Latest) {
         return this._knownSchemas.getSchemaSync(schemaKey, matchType);
     }
+    /**
+     * Attempts to get a SchemaInfo from the context's cache.
+     * Returns the info even if the schema is only partially loaded.
+     * @param schemaKey The SchemaKey to identify the Schema.
+     * @param matchType The SchemaMatch type to use. Default is SchemaMatchType.Latest.
+     * @internal
+     */
+    getCachedSchemaInfoSync(schemaKey, matchType = _ECObjects__WEBPACK_IMPORTED_MODULE_0__.SchemaMatchType.Latest) {
+        return this._knownSchemas.getSchemaInfoSync(schemaKey, matchType);
+    }
     async getSchemaItem(schemaNameOrKey, itemNameOrCtor, itemConstructor) {
         let schemaKey;
         if (typeof schemaNameOrKey === "string") {
@@ -62866,11 +62896,25 @@ class SchemaReadHelper {
         }
         this._schemaInfo = schemaInfo;
         // Need to add this schema to the context to be able to locate schemaItems within the context.
-        if (addSchemaToCache && !this._context.schemaExists(schema.schemaKey)) {
-            await this._context.addSchemaPromise(schemaInfo, schema, this.loadSchema(schemaInfo, schema));
+        if (addSchemaToCache) {
+            this.checkForReadVersionConflict(schema.schemaKey);
+            if (!this._context.schemaExists(schema.schemaKey))
+                await this._context.addSchemaPromise(schemaInfo, schema, this.loadSchema(schemaInfo, schema));
         }
         return schemaInfo;
     }
+    /**
+     * Two read-incompatible versions of the same schema can never substitute for one another.
+     * Detect it here and fail with a clear error.
+     * @param schemaKey The exact key of the schema about to be loaded into the context.
+     */
+    checkForReadVersionConflict(schemaKey) {
+        const existingInfo = this._context.getCachedSchemaInfoSync(new _SchemaKey__WEBPACK_IMPORTED_MODULE_5__.SchemaKey(schemaKey.name), _ECObjects__WEBPACK_IMPORTED_MODULE_1__.SchemaMatchType.Latest);
+        if (existingInfo && existingInfo.schemaKey.readVersion !== schemaKey.readVersion) {
+            throw new _Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaError(_Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaStatus.DuplicateSchema, `The schema '${schemaKey.toString(true)}' cannot be loaded: the read-incompatible version '${existingInfo.schemaKey.toString(true)}' is already loaded in this context. ` +
+                `A schema graph cannot require two read-incompatible versions of the same schema.`);
+        }
+    }
     /**
      * Populates the given Schema from a serialized representation.
      * @param schema The Schema to populate
@@ -62892,7 +62936,7 @@ class SchemaReadHelper {
                 throw new _Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaError(_Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaStatus.UnableToLoadSchema, `Could not load schema ${schema.schemaKey.toString()}`);
             return loadedSchema;
         }
-        const cachedSchema = await this._context.getCachedSchema(schemaInfo.schemaKey, _ECObjects__WEBPACK_IMPORTED_MODULE_1__.SchemaMatchType.Latest);
+        const cachedSchema = await this._context.getCachedSchema(schemaInfo.schemaKey, _ECObjects__WEBPACK_IMPORTED_MODULE_1__.SchemaMatchType.LatestReadCompatible);
         if (undefined === cachedSchema)
             throw new _Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaError(_Exception__WEBPACK_IMPORTED_MODULE_2__.ECSchemaStatus.UnableToLoadSchema, `Could not load schema ${schema.schemaKey.toString()}`);
         return cachedSchema;
@@ -62944,6 +62988,7 @@ class SchemaReadHelper {
         schema.fromJSONSync(this._parser.parseSchema());
         this._schema = schema;
         // Need to add this schema to the context to be able to locate schemaItems within the context.
+        this.checkForReadVersionConflict(schema.schemaKey);
         if (!this._context.schemaExists(schema.schemaKey))
             this._context.addSchemaSync(schema);
         // Load schema references first
@@ -85504,29 +85549,38 @@ class BriefcaseConnection extends _IModelConnection__WEBPACK_IMPORTED_MODULE_5__
     async abandonChanges() {
         await _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.abandonChanges(this.key); // eslint-disable-line @typescript-eslint/no-deprecated
     }
+    /** Subscribes to changeset download progress events on `channel` and wires `abortSignal` to `cancel`.
+     * @returns a function that removes every listener that was added.
+     */
+    listenForChangesetDownloadProgress(args) {
+        const { channel, cancel, downloadProgressCallback, abortSignal } = args;
+        const removeListeners = [];
+        if (downloadProgressCallback) {
+            const handleProgress = (_evt, data) => downloadProgressCallback(data);
+            removeListeners.push(_IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.addListener(channel, handleProgress));
+        }
+        if (abortSignal) {
+            const abort = () => void cancel();
+            abortSignal.addEventListener("abort", abort);
+            removeListeners.push(() => abortSignal.removeEventListener("abort", abort));
+        }
+        return () => removeListeners.forEach((remove) => remove());
+    }
     /** Pull (and potentially merge if there are local changes) up to a specified changeset from iModelHub into this briefcase
      * @param toIndex The changeset index to pull changes to. If `undefined`, pull all changes.
      * @param options Options for pulling changes.
      * @see [[BriefcaseTxns.onChangesPulled]] for the event dispatched after changes are pulled.
      */
     async pullChanges(toIndex, options) {
-        const removeListeners = [];
-        const shouldReportProgress = !!options?.downloadProgressCallback;
-        if (shouldReportProgress) {
-            const handleProgress = (_evt, data) => {
-                options?.downloadProgressCallback?.(data);
-            };
-            const removeProgressListener = _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.addListener((0,_itwin_core_common__WEBPACK_IMPORTED_MODULE_1__.getPullChangesIpcChannel)(this.iModelId), handleProgress);
-            removeListeners.push(removeProgressListener);
-        }
-        if (options?.abortSignal) {
-            const abort = () => void _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.cancelPullChangesRequest(this.key);
-            options?.abortSignal.addEventListener("abort", abort);
-            removeListeners.push(() => options?.abortSignal?.removeEventListener("abort", abort));
-        }
         this.requireTimeline();
+        const removeListeners = this.listenForChangesetDownloadProgress({
+            channel: (0,_itwin_core_common__WEBPACK_IMPORTED_MODULE_1__.getPullChangesIpcChannel)(this.key),
+            cancel: async () => _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.cancelPullChangesRequest(this.key),
+            downloadProgressCallback: options?.downloadProgressCallback,
+            abortSignal: options?.abortSignal,
+        });
         const ipcAppOptions = {
-            reportProgress: shouldReportProgress,
+            reportProgress: !!options?.downloadProgressCallback,
             progressInterval: options?.progressInterval,
             enableCancellation: !!options?.abortSignal,
         };
@@ -85534,18 +85588,29 @@ class BriefcaseConnection extends _IModelConnection__WEBPACK_IMPORTED_MODULE_5__
             this.changeset = await _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.pullChanges(this.key, toIndex, ipcAppOptions);
         }
         finally {
-            removeListeners.forEach((remove) => remove());
+            removeListeners();
         }
         await this.invalidateSchemaViewIfChanged();
     }
-    /** Create a changeset from local Txns and push to iModelHub. On success, clear Txn table.
-     * @param description The description for the changeset
-     * @returns the changesetId of the pushed changes
-     * @see [[BriefcaseTxns.onChangesPushed]] for the event dispatched after changes are pushed.
-     */
-    async pushChanges(description) {
+    async pushChanges(description, options) {
         this.requireTimeline();
-        return this.changeset = await _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.pushChanges(this.key, description);
+        const removeListeners = this.listenForChangesetDownloadProgress({
+            channel: (0,_itwin_core_common__WEBPACK_IMPORTED_MODULE_1__.getPushChangesIpcChannel)(this.key),
+            cancel: async () => _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.cancelPushChangesRequest(this.key),
+            downloadProgressCallback: options?.downloadProgressCallback,
+            abortSignal: options?.abortSignal,
+        });
+        const ipcAppOptions = {
+            reportDownloadProgress: !!options?.downloadProgressCallback,
+            downloadProgressInterval: options?.downloadProgressInterval,
+            enableCancellation: !!options?.abortSignal,
+        };
+        try {
+            return this.changeset = await _IpcApp__WEBPACK_IMPORTED_MODULE_6__.IpcApp.appFunctionIpc.pushChanges(this.key, description, ipcAppOptions);
+        }
+        finally {
+            removeListeners();
+        }
     }
     /** The current graphical editing scope, if one is in progress.
      * @see [[enterEditingScope]] to begin graphical editing.
@@ -195147,15 +195212,18 @@ __webpack_require__.r(__webpack_exports__);
 
 
 /**
- * fitPoints and end condition data for [[AkimaCurve3d]]
+ * Data for an [[AkimaCurve3d]]
  * * This is a "typed object" version of the serializer-friendly [[AkimaCurve3dProps]]
- * * Typical use cases rarely require all parameters, so the constructor does not itemize them as parameters.
  * @public
  */
 class AkimaCurve3dOptions {
+    /**
+     * Points that the curve must pass through.
+     * * Traditional DGN-style Akima curves interpret the first two and last two fit points as end tangent conditions,
+     * however as the current implementation uses another interpolation algorithm, there is no such interpretation here.
+     */
     fitPoints;
     /**
-     *
      * @param fitPoints points to CAPTURE
      * @param knots array to CAPTURE
      */
@@ -195166,7 +195234,7 @@ class AkimaCurve3dOptions {
      * First and last 2 points are "beyond the end" for control of end slope.
     fitPoints: Point3d[];
    
-   /** Clone with strongly typed members reduced to simple json. */
+     /** Clone with strongly typed members reduced to simple json. */
     cloneAsAkimaCurve3dProps() {
         const props = {
             fitPoints: _geometry3d_PointHelpers__WEBPACK_IMPORTED_MODULE_2__.Point3dArray.cloneDeepJSONNumberArrays(this.fitPoints),
@@ -195183,24 +195251,25 @@ class AkimaCurve3dOptions {
         const result = new AkimaCurve3dOptions(_geometry3d_PointHelpers__WEBPACK_IMPORTED_MODULE_2__.Point3dArray.clonePoint3dArray(source.fitPoints));
         return result;
     }
+    /** Whether the two options are equivalent or both undefined. */
     static areAlmostEqual(dataA, dataB) {
         if (dataA === undefined && dataB === undefined)
             return true;
-        if (dataA !== undefined && dataB !== undefined) {
+        if (dataA !== undefined && dataB !== undefined)
             return _Geometry__WEBPACK_IMPORTED_MODULE_1__.Geometry.almostEqualArrays(dataA.fitPoints, dataB.fitPoints, (a, b) => a.isAlmostEqual(b));
-        }
         return false;
     }
 }
 /**
- * Interpolating curve.
+ * Interpolating curve using the Akima formulation.
  * * Derive from [[ProxyCurve]]
  * * Use a [[BSplineCurve3d]] as the proxy
- * *
+ * * Currently the Akima formulation is replaced with a Greville interpolation.
  * @public
  */
 class AkimaCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_0__.ProxyCurve {
-    curvePrimitiveType = "interpolationCurve";
+    /** String name for schema properties. */
+    curvePrimitiveType = "akimaCurve";
     _options;
     /** CAPTURE properties and proxy curve. */
     constructor(properties, proxyCurve) {
@@ -195215,9 +195284,9 @@ class AkimaCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_0__.ProxyC
         return result;
     }
     /**
-     * Create an [[AkimaCurve3d]] based on points, knots, and other properties in the [[AkimaCurve3dProps]] or [[AkimaCurve3dOptions]].
+     * Create an [[AkimaCurve3d]] based on an [[AkimaCurve3dProps]] or [[AkimaCurve3dOptions]].
      * * This saves a COPY OF the options or props.
-     * * Use createCapture () if the options or props can be used without copy
+     * * Use createCapture() if the options or props can be used without copy
      */
     static create(options) {
         let optionsCopy;
@@ -195270,11 +195339,9 @@ class AkimaCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_0__.ProxyC
      * Transform this [[AkimaCurve3d]] and its defining data in place
      */
     tryTransformInPlace(transform) {
-        const proxyOk = this._proxyCurve.tryTransformInPlace(transform);
-        if (proxyOk) {
-            transform.multiplyPoint3dArray(this._options.fitPoints);
-        }
-        return proxyOk;
+        this._proxyCurve.tryTransformInPlace(transform);
+        transform.multiplyPoint3dArray(this._options.fitPoints);
+        return true; // we know this succeeds
     }
     /**
      * Find intervals of this CurvePrimitive that are interior to a clipper.
@@ -195296,6 +195363,7 @@ class AkimaCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_0__.ProxyC
     }
     /** Test if `other` is also an [[AkimaCurve3d]] */
     isSameGeometryClass(other) { return other instanceof AkimaCurve3d; }
+    /** Test if this [[AkimaCurve3d]] is almost equal to another GeometryQuery object. */
     isAlmostEqual(other) {
         if (other instanceof AkimaCurve3d) {
             return AkimaCurve3dOptions.areAlmostEqual(this._options, other._options);
@@ -196200,10 +196268,11 @@ class BSplineCurve3d extends BSplineCurve3dBase {
     }
     /**
      * Create a B-spline curve from an Akima curve.
-     * @param options collection of points and end conditions.
+     * * The Akima formulation of the curve is currently replaced by a Greville interpolation.
+     * @param options data for construction
      */
     static createFromAkimaCurve3dOptions(options) {
-        return _BSplineCurveOps__WEBPACK_IMPORTED_MODULE_19__.BSplineCurveOps.createThroughPoints(options.fitPoints, 4); // temporary
+        return _BSplineCurveOps__WEBPACK_IMPORTED_MODULE_19__.BSplineCurveOps.createThroughPoints(options.fitPoints, 4);
     }
     /**
      * Create a B-spline curve with given knots.
@@ -200240,8 +200309,7 @@ class InterpolationCurve3dOptions {
         result._endTangent = source.endTangent ? _geometry3d_Point3dVector3d__WEBPACK_IMPORTED_MODULE_3__.Vector3d.fromJSON(source.endTangent) : undefined;
         return result;
     }
-    // ugh.
-    // vector equality test with awkward rule that 000 matches undefined.
+    /** Vector equality test, with the additional rule that the zero vector matches undefined. */
     static areAlmostEqualAllow000AsUndefined(a, b) {
         if (a !== undefined && a.maxAbs() === 0)
             a = undefined;
@@ -200251,6 +200319,7 @@ class InterpolationCurve3dOptions {
             return a.isAlmostEqual(b);
         return a === undefined && b === undefined;
     }
+    /** Whether the two options are equivalent or both undefined. */
     static areAlmostEqual(dataA, dataB) {
         if (dataA === undefined && dataB === undefined)
             return true;
@@ -200299,6 +200368,7 @@ class InterpolationCurve3dOptions {
  * @public
  */
 class InterpolationCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_1__.ProxyCurve {
+    /** String name for schema properties. */
     curvePrimitiveType = "interpolationCurve";
     _options;
     /** CAPTURE properties and proxy curve. */
@@ -200371,15 +200441,13 @@ class InterpolationCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_1_
      * Transform this [[InterpolationCurve3d]] and its defining data in place
      */
     tryTransformInPlace(transform) {
-        const proxyOk = this._proxyCurve.tryTransformInPlace(transform);
-        if (proxyOk) {
-            transform.multiplyPoint3dArrayInPlace(this._options.fitPoints);
-            if (this._options.startTangent)
-                transform.multiplyVectorInPlace(this._options.startTangent);
-            if (this._options.endTangent)
-                transform.multiplyVectorInPlace(this._options.endTangent);
-        }
-        return proxyOk;
+        this._proxyCurve.tryTransformInPlace(transform);
+        transform.multiplyPoint3dArrayInPlace(this._options.fitPoints);
+        if (this._options.startTangent)
+            transform.multiplyVectorInPlace(this._options.startTangent);
+        if (this._options.endTangent)
+            transform.multiplyVectorInPlace(this._options.endTangent);
+        return true; // we know this succeeds
     }
     /**
      * Find intervals of this CurvePrimitive that are interior to a clipper.
@@ -200399,6 +200467,7 @@ class InterpolationCurve3d extends _curve_ProxyCurve__WEBPACK_IMPORTED_MODULE_1_
     cloneTransformed(transform) {
         return super.cloneTransformed(transform);
     }
+    /** Test if this [[InterpolationCurve3d]] is almost equal to another GeometryQuery object. */
     isAlmostEqual(other) {
         if (other instanceof InterpolationCurve3d) {
             return InterpolationCurve3dOptions.areAlmostEqual(this._options, other._options);
@@ -203847,8 +203916,6 @@ class ClipUtilities {
         if (!worldToLocal)
             return result;
         const localRegion = region.cloneTransformed(worldToLocal); // parallel to xy-plane so we can ignore z
-        if (!localRegion)
-            return result;
         // We can only clip convex polygons with our clipper machinery, but the input region doesn't have to be
         // convex or even a polygon. We get around this limitation by using a Boolean operation, which admits
         // *any* planar regions, albeit in local coordinates. First, we clip a rectangle that covers the input region
@@ -209484,10 +209551,7 @@ class CurveChainWithDistanceIndex extends _curve_CurvePrimitive__WEBPACK_IMPORTE
      * @param options how finely to stroke the path to create the distance index
      */
     cloneTransformed(transform, options) {
-        const c = this._path.clone();
-        if (c.tryTransformInPlace(transform))
-            return CurveChainWithDistanceIndex.createCapture(c, options);
-        return undefined;
+        return CurveChainWithDistanceIndex.createCapture(this._path.cloneTransformed(transform), options);
     }
     /**
      * Reference to the contained path.
@@ -209508,8 +209572,7 @@ class CurveChainWithDistanceIndex extends _curve_CurvePrimitive__WEBPACK_IMPORTE
      * @param options how finely to stroke the path to create the distance index
      */
     clone(options) {
-        const c = this._path.clone();
-        return CurveChainWithDistanceIndex.createCapture(c, options);
+        return CurveChainWithDistanceIndex.createCapture(this._path.clone(), options);
     }
     /**
      * Return a portion of this curve with its own distance index.
@@ -209687,15 +209750,10 @@ class CurveChainWithDistanceIndex extends _curve_CurvePrimitive__WEBPACK_IMPORTE
      * @return cloned flattened CurveChain, or reference to the input chain if no nesting
     */
     static flattenNestedChains(chain) {
-        if (-1 === chain.children.findIndex((child) => { return child instanceof CurveChainWithDistanceIndex; }))
+        if (-1 === chain.children.findIndex((c) => c instanceof CurveChainWithDistanceIndex))
             return chain;
         const flatChain = chain.clone();
-        const flatChildren = flatChain.children.flatMap((child) => {
-            if (child instanceof CurveChainWithDistanceIndex)
-                return child.path.children;
-            else
-                return [child];
-        });
+        const flatChildren = flatChain.children.flatMap((c) => c instanceof CurveChainWithDistanceIndex ? c.path.children : c);
         flatChain.children.splice(0, Infinity, ...flatChildren);
         return flatChain;
     }
@@ -209876,18 +209934,13 @@ class CurveChainWithDistanceIndex extends _curve_CurvePrimitive__WEBPACK_IMPORTE
         return result;
     }
     /**
-     * Attempt to transform in place.
-     * * Warning: If any child transform fails, `this` object becomes invalid but that should never happen.
-     * @param transform the transform to be applied.
-     * @returns true if all of child transforms succeed and false otherwise.
+     * Transform the chain in place.
+     * * Does NOT recompute the distance index.
+     * * For best results, use a rigid transform. Otherwise, it is better to call [[cloneTransformed]] so that the
+     * distance index is recomputed.
      */
     tryTransformInPlace(transform) {
-        let numFail = 0;
-        for (const c of this._path.children) {
-            if (!c.tryTransformInPlace(transform))
-                numFail++;
-        }
-        return numFail === 0;
+        return this._path.tryTransformInPlace(transform);
     }
     /** Reverse the curve's data so that its fractional stroking moves in the opposite direction. */
     reverseInPlace() {
@@ -210190,8 +210243,6 @@ __webpack_require__.r(__webpack_exports__);
 class CurveCollection extends _GeometryQuery__WEBPACK_IMPORTED_MODULE_6__.GeometryQuery {
     /** String name for schema properties */
     geometryCategory = "curveCollection";
-    /** Flag for inner loop status. Only used by `Loop`. */
-    isInner = false;
     /** Return the sum of the lengths of all contained curves. */
     sumLengths() {
         return _internalContexts_SumLengthsContext__WEBPACK_IMPORTED_MODULE_13__.SumLengthsContext.sumLengths(this);
@@ -210599,6 +210650,18 @@ class CurveChain extends CurveCollection {
         }
         return undefined;
     }
+    /** Return a deep copy. */
+    clone() {
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
+    }
     /**
      * Add a child curve.
      * @param child curve to add to the chain. The curve is captured by this instance.
@@ -210729,6 +210792,18 @@ class BagOfCurves extends CurveCollection {
     cloneEmptyPeer() {
         return new BagOfCurves();
     }
+    /** Return a deep copy. */
+    clone() {
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
+    }
     /** Add a child  */
     tryAddChild(child) {
         if (child)
@@ -211720,11 +211795,10 @@ class CurveFactory {
             // The alignment condition is equivalent to positive projected curve area computed wrt to the plane normal.
             const toLocal = _geometry3d_Matrix3d__WEBPACK_IMPORTED_MODULE_4__.Matrix3d.createRigidHeadsUp(planeNormal).transpose();
             const projection = closedCurve.cloneTransformed(_geometry3d_Transform__WEBPACK_IMPORTED_MODULE_11__.Transform.createOriginAndMatrix(undefined, toLocal));
-            if (projection) { // now we can ignore z-coords
-                const areaXY = _RegionOps__WEBPACK_IMPORTED_MODULE_25__.RegionOps.computeXYArea(projection);
-                if (areaXY && areaXY < 0)
-                    curve.reverseInPlace();
-            }
+            // now we can ignore z-coords
+            const areaXY = _RegionOps__WEBPACK_IMPORTED_MODULE_25__.RegionOps.computeXYArea(projection);
+            if (areaXY && areaXY < 0)
+                curve.reverseInPlace();
         }
     }
     /**
@@ -212883,9 +212957,9 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony import */ var _geometry3d_Transform__WEBPACK_IMPORTED_MODULE_7__ = __webpack_require__(/*! ../geometry3d/Transform */ "../../core/geometry/lib/esm/geometry3d/Transform.js");
 /* harmony import */ var _CurveLocationDetail__WEBPACK_IMPORTED_MODULE_8__ = __webpack_require__(/*! ./CurveLocationDetail */ "../../core/geometry/lib/esm/curve/CurveLocationDetail.js");
 /* harmony import */ var _GeometryQuery__WEBPACK_IMPORTED_MODULE_9__ = __webpack_require__(/*! ./GeometryQuery */ "../../core/geometry/lib/esm/curve/GeometryQuery.js");
-/* harmony import */ var _internalContexts_AppendPlaneIntersectionStrokeHandler__WEBPACK_IMPORTED_MODULE_10__ = __webpack_require__(/*! ./internalContexts/AppendPlaneIntersectionStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/AppendPlaneIntersectionStrokeHandler.js");
-/* harmony import */ var _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_11__ = __webpack_require__(/*! ./internalContexts/ClosestPointStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/ClosestPointStrokeHandler.js");
-/* harmony import */ var _internalContexts_AnnounceTangentStrokeHandler__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! ./internalContexts/AnnounceTangentStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/AnnounceTangentStrokeHandler.js");
+/* harmony import */ var _internalContexts_AnnounceTangentStrokeHandler__WEBPACK_IMPORTED_MODULE_10__ = __webpack_require__(/*! ./internalContexts/AnnounceTangentStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/AnnounceTangentStrokeHandler.js");
+/* harmony import */ var _internalContexts_AppendPlaneIntersectionStrokeHandler__WEBPACK_IMPORTED_MODULE_11__ = __webpack_require__(/*! ./internalContexts/AppendPlaneIntersectionStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/AppendPlaneIntersectionStrokeHandler.js");
+/* harmony import */ var _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_12__ = __webpack_require__(/*! ./internalContexts/ClosestPointStrokeHandler */ "../../core/geometry/lib/esm/curve/internalContexts/ClosestPointStrokeHandler.js");
 /* harmony import */ var _internalContexts_CurveLengthContext__WEBPACK_IMPORTED_MODULE_13__ = __webpack_require__(/*! ./internalContexts/CurveLengthContext */ "../../core/geometry/lib/esm/curve/internalContexts/CurveLengthContext.js");
 /*---------------------------------------------------------------------------------------------
 * Copyright (c) Bentley Systems, Incorporated. All rights reserved.
@@ -213304,7 +213378,7 @@ class CurvePrimitive extends _GeometryQuery__WEBPACK_IMPORTED_MODULE_9__.Geometr
      * @returns details `d` of the closest point. The distance from spacePoint to the closest point is stored in `d.a`.
      */
     closestPoint(spacePoint, extend = false, result) {
-        const strokeHandler = new _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_11__.ClosestPointStrokeHandler(spacePoint, extend, result);
+        const strokeHandler = new _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_12__.ClosestPointStrokeHandler(spacePoint, extend, result);
         this.emitStrokableParts(strokeHandler);
         return strokeHandler.claimResult();
     }
@@ -213320,7 +213394,7 @@ class CurvePrimitive extends _GeometryQuery__WEBPACK_IMPORTED_MODULE_9__.Geometr
      * @returns details `d` of the closest point. The distance from spacePoint to the closest point is stored in `d.a`.
      */
     closestPointXY(spacePoint, extend = false, result) {
-        const strokeHandler = new _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_11__.ClosestPointStrokeHandler(spacePoint, extend, result, true);
+        const strokeHandler = new _internalContexts_ClosestPointStrokeHandler__WEBPACK_IMPORTED_MODULE_12__.ClosestPointStrokeHandler(spacePoint, extend, result, true);
         this.emitStrokableParts(strokeHandler);
         return strokeHandler.claimResult();
     }
@@ -213336,7 +213410,7 @@ class CurvePrimitive extends _GeometryQuery__WEBPACK_IMPORTED_MODULE_9__.Geometr
      * @param options (optional) options for computing tangents. See [[TangentOptions]] for defaults.
      */
     emitTangents(spacePoint, announceTangent, options) {
-        const strokeHandler = new _internalContexts_AnnounceTangentStrokeHandler__WEBPACK_IMPORTED_MODULE_12__.AnnounceTangentStrokeHandler(spacePoint, announceTangent, options);
+        const strokeHandler = new _internalContexts_AnnounceTangentStrokeHandler__WEBPACK_IMPORTED_MODULE_10__.AnnounceTangentStrokeHandler(spacePoint, announceTangent, options);
         this.emitStrokableParts(strokeHandler, options?.strokeOptions);
     }
     /**
@@ -213431,7 +213505,7 @@ class CurvePrimitive extends _GeometryQuery__WEBPACK_IMPORTED_MODULE_9__.Geometr
      * @returns Return the number of CurveLocationDetail's added to the result array.
      */
     appendPlaneIntersectionPoints(plane, result) {
-        const strokeHandler = new _internalContexts_AppendPlaneIntersectionStrokeHandler__WEBPACK_IMPORTED_MODULE_10__.AppendPlaneIntersectionStrokeHandler(plane, result);
+        const strokeHandler = new _internalContexts_AppendPlaneIntersectionStrokeHandler__WEBPACK_IMPORTED_MODULE_11__.AppendPlaneIntersectionStrokeHandler(plane, result);
         const n0 = result.length;
         this.emitStrokableParts(strokeHandler);
         return result.length - n0;
@@ -213629,6 +213703,13 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */   RecursiveCurveProcessorWithStack: () => (/* binding */ RecursiveCurveProcessorWithStack)
 /* harmony export */ });
 /* harmony import */ var _CurvePrimitive__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ./CurvePrimitive */ "../../core/geometry/lib/esm/curve/CurvePrimitive.js");
+/*---------------------------------------------------------------------------------------------
+* Copyright (c) Bentley Systems, Incorporated. All rights reserved.
+* See LICENSE.md in the project root for license terms and full copyright notice.
+*--------------------------------------------------------------------------------------------*/
+/** @packageDocumentation
+ * @module Curve
+ */
 
 /** base class for detailed traversal of curve artifacts.
  * * This recurses to children in the quickest way (no records of path)
@@ -213664,7 +213745,10 @@ class RecursiveCurveProcessor {
     announceUnionRegion(data, _indexInParent = -1) {
         let i = 0;
         for (const child of data.children) {
-            child.announceToCurveProcessor(this, i++);
+            if (child.curveCollectionType === "loop")
+                this.announceLoop(child, i++);
+            else
+                this.announceParityRegion(child, i++);
         }
     }
     /** announce a bag of curves.
@@ -213726,9 +213810,15 @@ class RecursiveCurveProcessorWithStack extends RecursiveCurveProcessor {
         this.leave();
     }
     /** announce beginning or end of a parity region */
-    announceUnionRegion(data, indexInParent = -1) {
+    announceUnionRegion(data, _indexInParent = -1) {
         this.enter(data);
-        super.announceUnionRegion(data, indexInParent);
+        let i = 0;
+        for (const child of data.children) {
+            if (child.curveCollectionType === "loop")
+                this.announceLoop(child, i++);
+            else
+                this.announceParityRegion(child, i++);
+        }
         this.leave();
     }
     /**
@@ -214334,7 +214424,7 @@ class LineSegment3d extends _CurvePrimitive__WEBPACK_IMPORTED_MODULE_11__.CurveP
         this._point0 = this._point1;
         this._point1 = a;
     }
-    /** Transform the two endpoints of this LinSegment. */
+    /** Transform the two endpoints of this line segment. */
     tryTransformInPlace(transform) {
         this._point0 = transform.multiplyPoint3d(this._point0, this._point0);
         this._point1 = transform.multiplyPoint3d(this._point1, this._point1);
@@ -216149,12 +216239,19 @@ __webpack_require__.r(__webpack_exports__);
 class Loop extends _CurveCollection__WEBPACK_IMPORTED_MODULE_1__.CurveChain {
     /** String name for schema properties */
     curveCollectionType = "loop";
-    /** Tag value that can be set to true for user code to mark inner and outer loops. */
+    /**
+     * Flag for inner loop status (default value is `false`).
+     * * Typical usage is to set to `true` on hole `Loop`s in a `ParityRegion` to distinguish them from the outer `Loop`.
+     * * This property is only set by the user, and does not affect region processing.
+     * * This property is propagated through [[clone]] and JSON/FlatBuffer de/serialization.
+     * * For best de/serialization results, avoid setting to `false` on multiple `Loop`s of a `ParityRegion`.
+     */
     isInner = false;
     /** Test if `other` is a `Loop` */
     isSameGeometryClass(other) {
         return other instanceof Loop;
     }
+    /** Construct an empty loop. */
     constructor() {
         super();
     }
@@ -216224,10 +216321,28 @@ class Loop extends _CurveCollection__WEBPACK_IMPORTED_MODULE_1__.CurveChain {
         emptyClone.isInner = this.isInner;
         return emptyClone;
     }
+    /** Return a deep copy. */
+    clone() {
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
+    }
     /** Second step of double dispatch:  call `handler.handleLoop(this)` */
     dispatchToGeometryHandler(handler) {
         return handler.handleLoop(this);
     }
+    /** Test for near equality */
+    isAlmostEqual(other) {
+        if (!super.isAlmostEqual(other))
+            return false;
+        return this.isInner === other.isInner;
+    }
 }
 /**
  * Structure carrying a pair of loops with curve geometry.
@@ -216535,14 +216650,15 @@ class ParityRegion extends _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.CurveCo
     }
     /** Return a deep copy. */
     clone() {
-        const clone = new ParityRegion();
-        let child;
-        for (child of this.children) {
-            const childClone = child.clone();
-            if (childClone instanceof _Loop__WEBPACK_IMPORTED_MODULE_1__.Loop)
-                clone.children.push(childClone);
-        }
-        return clone;
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
     }
     /** Stroke these curves into a new ParityRegion. */
     cloneStroked(options) {
@@ -216671,6 +216787,18 @@ class Path extends _CurveCollection__WEBPACK_IMPORTED_MODULE_2__.CurveChain {
     cloneEmptyPeer() {
         return new Path();
     }
+    /** Return a deep copy. */
+    clone() {
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
+    }
     /** Second step of double dispatch: call `handler.handlePath(this)` */
     dispatchToGeometryHandler(handler) {
         return handler.handlePath(this);
@@ -216946,9 +217074,8 @@ class ProxyCurve extends _curve_CurvePrimitive__WEBPACK_IMPORTED_MODULE_0__.Curv
     /** Return a transformed clone. */
     cloneTransformed(transform) {
         const myClone = this.clone();
-        if (myClone.tryTransformInPlace(transform))
-            return myClone;
-        return undefined;
+        myClone.tryTransformInPlace(transform);
+        return myClone;
     }
     /** Implement by proxyCurve. Subclasses may eventually override this default implementation. */
     clonePartialCurve(fractionA, fractionB) {
@@ -219017,8 +219144,6 @@ class RegionOps {
             const worldToLocal = localToWorld.inverse();
             (0,_itwin_core_bentley__WEBPACK_IMPORTED_MODULE_0__.assert)(worldToLocal !== undefined, "FrameBuilder's transform is invertible");
             regionXY = region.cloneTransformed(worldToLocal);
-            if (!regionXY)
-                return undefined;
         }
         const momentData = RegionOps.computeXYAreaMoments(regionXY);
         if (!momentData)
@@ -221247,6 +221372,18 @@ class UnionRegion extends _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.CurveCol
     cloneEmptyPeer() {
         return new UnionRegion();
     }
+    /** Return a deep copy. */
+    clone() {
+        return super.clone();
+    }
+    /** Create a deep copy of transformed curves. */
+    cloneTransformed(transform) {
+        return super.cloneTransformed(transform);
+    }
+    /** Create a deep copy with all linestrings broken down into multiple LineSegment3d. */
+    cloneWithExpandedLineStrings() {
+        return super.cloneWithExpandedLineStrings();
+    }
     /**
      * Try to add a child (by capturing it).
      * * Returns false if the `AnyCurve` child is not a region type.
@@ -221859,6 +221996,7 @@ __webpack_require__.r(__webpack_exports__);
  * Algorithmic class for cloning curve collections.
  * * recurse through collection nodes, building image nodes as needed and inserting clones of children.
  * * for individual primitive, invoke doClone (protected) for direct clone; insert into parent
+ * @internal
  */
 class CloneCurvesContext extends _CurveProcessor__WEBPACK_IMPORTED_MODULE_1__.RecursiveCurveProcessorWithStack {
     _result;
@@ -221897,7 +222035,7 @@ class CloneCurvesContext extends _CurveProcessor__WEBPACK_IMPORTED_MODULE_1__.Re
         const c = this.doClone(primitive);
         if (c !== undefined && this._stack.length > 0) {
             const parent = this._stack[this._stack.length - 1];
-            if (parent instanceof _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.CurveChain || parent instanceof _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.BagOfCurves)
+            if (parent instanceof _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.CurveChain || parent instanceof _CurveCollection__WEBPACK_IMPORTED_MODULE_0__.BagOfCurves) {
                 if (Array.isArray(c)) {
                     for (const c1 of c) {
                         parent.tryAddChild(c1);
@@ -221906,6 +222044,7 @@ class CloneCurvesContext extends _CurveProcessor__WEBPACK_IMPORTED_MODULE_1__.Re
                 else {
                     parent.tryAddChild(c);
                 }
+            }
         }
     }
 }
@@ -227733,26 +227872,34 @@ __webpack_require__.r(__webpack_exports__);
 /* harmony export */ __webpack_require__.d(__webpack_exports__, {
 /* harmony export */   TransformInPlaceContext: () => (/* binding */ TransformInPlaceContext)
 /* harmony export */ });
-/* harmony import */ var _CurveProcessor__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! ../CurveProcessor */ "../../core/geometry/lib/esm/curve/CurveProcessor.js");
+/* harmony import */ var _itwin_core_bentley__WEBPACK_IMPORTED_MODULE_0__ = __webpack_require__(/*! @itwin/core-bentley */ "../../core/bentley/lib/esm/core-bentley.js");
+/* harmony import */ var _CurveProcessor__WEBPACK_IMPORTED_MODULE_1__ = __webpack_require__(/*! ../CurveProcessor */ "../../core/geometry/lib/esm/curve/CurveProcessor.js");
+/*---------------------------------------------------------------------------------------------
+* Copyright (c) Bentley Systems, Incorporated. All rights reserved.
+* See LICENSE.md in the project root for license terms and full copyright notice.
+*--------------------------------------------------------------------------------------------*/
+/** @packageDocumentation
+ * @module Curve
+ */
 
-/** Algorithmic class: Transform curves in place.
+
+/** Algorithmic class: Transform curves in place. Always expected to succeed.
  * @internal
  */
-class TransformInPlaceContext extends _CurveProcessor__WEBPACK_IMPORTED_MODULE_0__.RecursiveCurveProcessor {
-    numFail;
-    numOK;
+class TransformInPlaceContext extends _CurveProcessor__WEBPACK_IMPORTED_MODULE_1__.RecursiveCurveProcessor {
     transform;
-    constructor(transform) { super(); this.numFail = 0; this.numOK = 0; this.transform = transform; }
+    constructor(transform) {
+        super();
+        this.transform = transform;
+    }
     static tryTransformInPlace(target, transform) {
         const context = new TransformInPlaceContext(transform);
         target.announceToCurveProcessor(context);
-        return context.numFail === 0;
+        return true;
     }
     announceCurvePrimitive(curvePrimitive, _indexInParent) {
         if (!curvePrimitive.tryTransformInPlace(this.transform))
-            this.numFail++;
-        else
-            this.numOK++;
+            (0,_itwin_core_bentley__WEBPACK_IMPORTED_MODULE_0__.assert)(false, "TransformInPlaceContext: unexpected failure of tryTransformInPlace");
     }
 }
 
@@ -231188,9 +231335,12 @@ class DirectSpiral3d extends _TransitionSpiral3d__WEBPACK_IMPORTED_MODULE_13__.T
     clone() {
         return new DirectSpiral3d(this.localToWorld.clone(), this._spiralType, this.designProperties?.clone(), this._nominalL1, this._nominalR1, this._activeFractionInterval?.clone(), this._evaluator.clone());
     }
-    /** Apply `transform` to this spiral's local to world transform. */
-    tryTransformInPlace(transformA) {
-        const rigidData = this.applyRigidPartOfTransform(transformA);
+    /**
+     * Apply `transform` to this spiral's local to world transform.
+     * * Only the rigid part of the transform is applied.
+     */
+    tryTransformInPlace(transform) {
+        const rigidData = this.applyRigidPartOfTransform(transform);
         if (rigidData !== undefined) {
             this._nominalL1 *= rigidData.scale;
             this._nominalR1 *= rigidData.scale;
@@ -231595,9 +231745,12 @@ class IntegratedSpiral3d extends _TransitionSpiral3d__WEBPACK_IMPORTED_MODULE_15
     clone() {
         return new IntegratedSpiral3d(this._spiralType, this._evaluator, this.radius01.clone(), this.bearing01.clone(), this.activeFractionInterval.clone(), this.localToWorld.clone(), this._arcLength01, this._designProperties?.clone());
     }
-    /** Apply `transform` to this spiral's local to world transform. */
-    tryTransformInPlace(transformA) {
-        const rigidData = this.applyRigidPartOfTransform(transformA);
+    /**
+     * Apply `transform` to this spiral's local to world transform.
+     * * Only the rigid part of the transform is applied.
+     */
+    tryTransformInPlace(transform) {
+        const rigidData = this.applyRigidPartOfTransform(transform);
         if (rigidData !== undefined) {
             this._curvature01.x0 /= rigidData.scale;
             this._curvature01.x1 /= rigidData.scale;
@@ -282545,8 +282698,11 @@ function nullToUndefined(data) {
 function createTypedCurveCollection(collectionType) {
     if (collectionType === 1)
         return new _curve_Path__WEBPACK_IMPORTED_MODULE_14__.Path();
-    if (collectionType === 2 || collectionType === 3)
-        return new _curve_Loop__WEBPACK_IMPORTED_MODULE_12__.Loop();
+    if (collectionType === 2 || collectionType === 3) {
+        const loop = new _curve_Loop__WEBPACK_IMPORTED_MODULE_12__.Loop();
+        loop.isInner = collectionType === 3;
+        return loop;
+    }
     if (collectionType === 4)
         return new _curve_ParityRegion__WEBPACK_IMPORTED_MODULE_13__.ParityRegion();
     if (collectionType === 5)
@@ -282780,9 +282936,8 @@ class BGFBWriter {
         let cvType = 0;
         if (cv instanceof _curve_Path__WEBPACK_IMPORTED_MODULE_16__.Path)
             cvType = 1;
-        else if (cv instanceof _curve_Loop__WEBPACK_IMPORTED_MODULE_14__.Loop) {
+        else if (cv instanceof _curve_Loop__WEBPACK_IMPORTED_MODULE_14__.Loop)
             cvType = cv.isInner ? 3 : 2;
-        }
         else if (cv instanceof _curve_ParityRegion__WEBPACK_IMPORTED_MODULE_15__.ParityRegion)
             cvType = 4;
         else if (cv instanceof _curve_UnionRegion__WEBPACK_IMPORTED_MODULE_21__.UnionRegion)
@@ -283988,16 +284143,17 @@ var IModelJson;
             return undefined;
         }
         /** parse contents of a curve collection to a CurveCollection instance */
-        static parseCurveCollectionMembers(result, data) {
-            if (data && Array.isArray(data)) {
-                for (const c of data) {
-                    const g = Reader.parse(c);
-                    if (g instanceof _curve_GeometryQuery__WEBPACK_IMPORTED_MODULE_11__.GeometryQuery && ("curveCollection" === g.geometryCategory || "curvePrimitive" === g.geometryCategory))
-                        result.tryAddChild(g);
-                }
-                return result;
+        static parseCurveCollectionMembers(result, data, isInner = false) {
+            if (!data || !Array.isArray(data))
+                return undefined;
+            for (const c of data) {
+                const g = Reader.parse(c);
+                if (g instanceof _curve_GeometryQuery__WEBPACK_IMPORTED_MODULE_11__.GeometryQuery && ("curveCollection" === g.geometryCategory || "curvePrimitive" === g.geometryCategory))
+                    result.tryAddChild(g);
             }
-            return undefined;
+            if (isInner && result instanceof _curve_Loop__WEBPACK_IMPORTED_MODULE_14__.Loop)
+                result.isInner = true;
+            return result;
         }
         /** Parse content of `bsurf` to BSplineSurface3d or BSplineSurface3dH */
         static parseBsurf(data) {
@@ -284215,7 +284371,7 @@ var IModelJson;
                     return Reader.parseCurveCollectionMembers(new _curve_Path__WEBPACK_IMPORTED_MODULE_16__.Path(), json.path);
                 }
                 else if (json.hasOwnProperty("loop")) {
-                    return Reader.parseCurveCollectionMembers(new _curve_Loop__WEBPACK_IMPORTED_MODULE_14__.Loop(), json.loop);
+                    return Reader.parseCurveCollectionMembers(new _curve_Loop__WEBPACK_IMPORTED_MODULE_14__.Loop(), json.loop, json.hasOwnProperty("isInner") && true === json.isInner);
                 }
                 else if (json.hasOwnProperty("parityRegion")) {
                     return Reader.parseCurveCollectionMembers(new _curve_ParityRegion__WEBPACK_IMPORTED_MODULE_15__.ParityRegion(), json.parityRegion);
@@ -284541,7 +284697,7 @@ var IModelJson;
         }
         /** Convert strongly typed instance to tagged json */
         handleLoop(data) {
-            return { loop: this.collectChildren(data) };
+            return { loop: this.collectChildren(data), isInner: data.isInner ? true : undefined };
         }
         /** Convert strongly typed instance to tagged json */
         handleParityRegion(data) {
@@ -284557,12 +284713,10 @@ var IModelJson;
         }
         collectChildren(data) {
             const children = [];
-            if (data.children && Array.isArray(data.children)) {
-                for (const child of data.children) {
-                    const cdata = child.dispatchToGeometryHandler(this);
-                    if (cdata)
-                        children.push(cdata);
-                }
+            for (const child of data.children) {
+                const cdata = child.dispatchToGeometryHandler(this);
+                if (cdata)
+                    children.push(cdata);
             }
             return children;
         }
@@ -295800,23 +295954,9 @@ class ITwinLocalization {
             this.i18next.loadNamespaces(name, (err) => {
                 if (!err)
                     return resolve();
-                // Here we got a non-null err object.
-                // This method is called when the system has attempted to load the resources for the namespaces for each possible locale.
-                // For example 'fr-ca' might be the most specific locale, in which case 'fr' and 'en' are fallback locales.
-                // Using Backend from i18next-http-backend, err will be an array of strings of each namespace it tried to read and its locale.
-                // There might be errs for some other namespaces as well as this one. We resolve the promise unless there's an error for each possible locale.
-                let locales = this.getLanguageList().map((thisLocale) => `/${thisLocale}/`);
-                try {
-                    for (const thisError of err) {
-                        if (typeof thisError === "string")
-                            locales = locales.filter((thisLocale) => !thisError.includes(thisLocale));
-                    }
-                }
-                catch {
-                    locales = [];
-                }
-                // if we removed every locale from the array, it wasn't loaded.
-                if (locales.length === 0)
+                // i18next can return errors from other concurrent namespace loads in this callback.
+                const wasLoaded = this.getLanguageList().some((language) => this.i18next.hasResourceBundle(language, name));
+                if (!wasLoaded)
                     _itwin_core_bentley__WEBPACK_IMPORTED_MODULE_3__.Logger.logError("i18n", `No resources for namespace ${name} could be loaded`);
                 resolve();
             });
@@ -336830,7 +336970,7 @@ class WMS {
 (module) {
 
 "use strict";
-module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.14.0-dev.7","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s build:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts && npm run -s extract","extract":"betools extract --fileExt=ts --extractFrom=./src/test/example-code --recursive --out=../../generated-docs/extract","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run build:test-worker && vitest --run","cover":"npm run build:test-worker && vitest --run","build:test-worker":"vite build --config ./src/test/worker/vite.config.mts 1>&2","build:workers":"rimraf lib/workers/webpack && vite build --config ./src/workers/ImdlParser/vite.config.mts 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@bentley/aec-units-schema":"^1.0.3","@bentley/formats-schema":"^1.0.0","@bentley/units-schema":"^1.0.11","@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/object-storage-core":"^3.0.4","@itwin/eslint-plugin":"^6.0.0","@types/node":"~20.17.0","@types/sinon":"^17.0.2","@vitest/browser-playwright":"^4.1.10","@vitest/coverage-v8":"^4.1.10","cpx2":"^8.0.0","eslint":"^9.31.0","playwright":"~1.56.1","rimraf":"^6.0.1","sinon":"^17.0.2","typescript":"~5.6.2","vite":"^6.4.3","vitest":"^4.1.10","vite-plugin-static-copy":"2.2.0"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"^4.4.5","@loaders.gl/draco":"^4.4.5","fuse.js":"^3.3.0","wms-capabilities":"0.6.0"}}');
+module.exports = /*#__PURE__*/JSON.parse('{"name":"@itwin/core-frontend","version":"5.14.0-dev.9","description":"iTwin.js frontend components","main":"lib/cjs/core-frontend.js","module":"lib/esm/core-frontend.js","typings":"lib/cjs/core-frontend","license":"MIT","scripts":{"build":"npm run -s copy:public && npm run -s build:cjs && npm run -s build:esm && npm run -s build:workers && npm run -s copy:draco","build:cjs":"npm run -s copy:js:cjs && tsc 1>&2 --outDir lib/cjs","build:esm":"npm run -s copy:js:esm && tsc 1>&2 --module ES2022 --outDir lib/esm","clean":"rimraf -g lib .rush/temp/package-deps*.json","copy:public":"cpx \\"./src/public/**/*\\" ./lib/public","copy:js:cjs":"cpx \\"./src/**/*.js\\" ./lib/cjs","copy:js:esm":"cpx \\"./src/**/*.js\\" ./lib/esm","copy:draco":"cpx \\"./node_modules/@loaders.gl/draco/dist/libs/*\\" ./lib/public/scripts","docs":"betools docs --json=../../generated-docs/core/core-frontend/file.json --tsIndexFile=./core-frontend.ts --onlyJson --excludes=webgl/**/*,**/map/*.d.ts,**/tile/*.d.ts,**/*-css.ts && npm run -s extract","extract":"betools extract --fileExt=ts --extractFrom=./src/test/example-code --recursive --out=../../generated-docs/extract","extract-api":"betools extract-api --entry=core-frontend && npm run extract-extension-api","extract-extension-api":"eslint --no-inline-config -c extraction.eslint.config.js  \\"./src/**/*.ts\\" 1>&2","lint":"eslint \\"./src/**/*.ts\\" 1>&2","lint-fix":"eslint --fix -f visualstudio \\"./src/**/*.ts\\" 1>&2","lint-deprecation":"eslint --fix -f visualstudio --no-inline-config -c ../../common/config/eslint/eslint.config.deprecation-policy.js \\"./src/**/*.ts\\"","pseudolocalize":"betools pseudolocalize --englishDir ./src/public/locales/en --out ./public/locales/en-PSEUDO","test":"npm run build:test-worker && vitest --run","cover":"npm run build:test-worker && vitest --run","build:test-worker":"vite build --config ./src/test/worker/vite.config.mts 1>&2","build:workers":"rimraf lib/workers/webpack && vite build --config ./src/workers/ImdlParser/vite.config.mts 1>&2"},"repository":{"type":"git","url":"https://github.com/iTwin/itwinjs-core.git","directory":"core/frontend"},"keywords":["Bentley","BIM","iModel","digital-twin","iTwin"],"author":{"name":"Bentley Systems, Inc.","url":"http://www.bentley.com"},"peerDependencies":{"@itwin/appui-abstract":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*"},"//devDependencies":["NOTE: All peerDependencies should also be listed as devDependencies since peerDependencies are not considered by npm install","NOTE: All tools used by scripts in this package must be listed as devDependencies"],"devDependencies":{"@bentley/aec-units-schema":"^1.0.3","@bentley/formats-schema":"^1.0.0","@bentley/units-schema":"^1.0.11","@itwin/appui-abstract":"workspace:*","@itwin/build-tools":"workspace:*","@itwin/core-bentley":"workspace:*","@itwin/core-common":"workspace:*","@itwin/core-geometry":"workspace:*","@itwin/core-orbitgt":"workspace:*","@itwin/core-quantity":"workspace:*","@itwin/ecschema-metadata":"workspace:*","@itwin/ecschema-rpcinterface-common":"workspace:*","@itwin/object-storage-core":"^3.0.4","@itwin/eslint-plugin":"^6.0.0","@types/node":"~20.17.0","@types/sinon":"^17.0.2","@vitest/browser-playwright":"^4.1.10","@vitest/coverage-v8":"^4.1.10","cpx2":"^8.0.0","eslint":"^9.31.0","playwright":"~1.56.1","rimraf":"^6.0.1","sinon":"^17.0.2","typescript":"~5.6.2","vite":"^6.4.3","vitest":"^4.1.10","vite-plugin-static-copy":"2.2.0"},"//dependencies":["NOTE: these dependencies should be only for things that DO NOT APPEAR IN THE API","NOTE: core-frontend should remain UI technology agnostic, so no react/angular dependencies are allowed"],"dependencies":{"@itwin/core-i18n":"workspace:*","@itwin/webgl-compatibility":"workspace:*","@loaders.gl/core":"^4.4.5","@loaders.gl/draco":"^4.4.5","fuse.js":"^3.3.0","wms-capabilities":"0.6.0"}}');
 
 /***/ }