@configura/web-api 1.6.1-alpha.0 → 1.6.1-alpha.4

package/dist/CfgProduct.d.ts

@@ -1,9 +1,9 @@
 import { AggregatedLoadingObservable, LengthUnit, Observable, SingleArgCallback } from "@configura/web-utilities";
-import { AdditionalProductConfiguration, CatalogueParams, MeasureParam, MtrlApplication, Prices, Transform } from "./CatalogueAPI.js";
+import { AdditionalProductConfiguration, AdditionalProductRef, CatalogueParams, MeasureParam, MtrlApplication, Prices, Transform } from "./CatalogueAPI.js";
 import { CfgMeasureDefinition } from "./CfgMeasure.js";
 import { _CfgFeatureInternal } from "./productConfiguration/CfgFeature.js";
-import { ProductConfigurationBubbleMode, _CfgOptionInternal } from "./productConfiguration/CfgOption.js";
-import { CfgProductConfiguration, _CfgProductConfigurationInternal } from "./productConfiguration/CfgProductConfiguration.js";
+import { ProductConfigurationBubbleMode } from "./productConfiguration/CfgOption.js";
+import { CfgProductConfiguration } from "./productConfiguration/CfgProductConfiguration.js";
 import { ProductLoader } from "./productLoader.js";
 import { SyncGroupsApplyMode } from "./syncGroups/SyncGroupsApplyMode.js";
 import { SyncGroupsHandler } from "./syncGroups/SyncGroupsHandler.js";
@@ -20,16 +20,17 @@ export declare type CfgProductSettings = {
      */
     strictSelectOneSelectionCount: boolean;
     /**
-     * Activating this will make setAPI throw an error if the number of actually selected options
-     * on Features (excluding Group) are not exactly equal to the number of options passed in.
-     * Note: This check is not always reliable for Options with multiple Features each, which we
-     * believe is a rare use case.
-     */
-    strictSetApiSelectionMatch: boolean;
-    /**
-     * Controls if SyncGroups are applied Fast or Strict. Fast keeps the number of validate calls
-     * to the server down, Strict is closer to how CET works. We recommend you use Fast as cases
-     * where the results differ should be rare in real use cases.
+     * Controls if SyncGroups are applied Faster or Stricter.
+     *
+     * Fast - Tries to minimize the number of validates calls to the AI lowering the response times
+     * for selecting options. Might not always give the expected result for complex products.
+     *
+     * Strict - Apply the SyncGroups rules in a stricter fashion to be as close to CET as possible,
+     * which might result in longer response times and more validation calls to the API when
+     * selecting options.
+     *
+     * The SDK will default to Strict, but we recommend that you try out Fast since cases where the
+     * results differ should be rare in most real uses cases and the speedup can be quite large.
      */
     syncGroupsApplyMode: SyncGroupsApplyMode | undefined;
 };
@@ -56,8 +57,6 @@ export declare type CfgPrice = {
     currency: string;
     fractionDigits: number;
 };
-export declare type CfgPathSegment = string;
-export declare type CfgPath = CfgPathSegment[];
 export declare type RevalidateResult = {
     wasAborted: boolean;
     requestDidValidate: boolean;
@@ -77,13 +76,10 @@ export declare class _CfgProductInternal {
     private readonly _rawUnit;
     private _rawProductData;
     readonly loadingObservable: AggregatedLoadingObservable;
-    readonly refKey: string | undefined;
-    readonly refDescription: string | undefined;
     readonly parent: _CfgProductInternal | undefined;
-    readonly transform: Transform | undefined;
-    anchor: MeasureParam | undefined;
+    private _additionalProductRef;
     private readonly _syncGroupHandler;
-    static make: (productLoaderRaw: ProductLoader, productLoaderForGroupedLoad: ProductLoader | undefined, lang: string, catId: CatalogueParams, partNumber: string, settings: CfgProductSettings, optional: boolean, loadingObservable: AggregatedLoadingObservable, refKey: string | undefined, refDescription: string | undefined, parent: _CfgProductInternal | undefined, root: _CfgProductInternal | undefined, transform: Transform | undefined, anchor: MeasureParam | undefined) => Promise<_CfgProductInternal>;
+    static make: (productLoaderRaw: ProductLoader, productLoaderForGroupedLoad: ProductLoader | undefined, lang: string, catId: CatalogueParams, partNumber: string, settings: CfgProductSettings, optional: boolean, loadingObservable: AggregatedLoadingObservable, parent: _CfgProductInternal | undefined, root: _CfgProductInternal | undefined, additionalProductRef: AdditionalProductRef | undefined) => Promise<_CfgProductInternal>;
     private constructor();
     readonly root: _CfgProductInternal;
     private _destroyed;
@@ -96,6 +92,7 @@ export declare class _CfgProductInternal {
     readonly isAdditionalProduct: boolean;
     clone(parent?: _CfgProductInternal, root?: _CfgProductInternal): Promise<_CfgProductInternal>;
     destroy: () => void;
+    _updateAdditionalProdRef(p: AdditionalProductRef): void;
     get description(): string | undefined;
     get rootNodeSources(): RootNodeSource[] | undefined;
     get mtrlApplications(): MtrlApplication[] | undefined;
@@ -105,6 +102,9 @@ export declare class _CfgProductInternal {
     private _measureDefinitions;
     get measureDefinitions(): CfgMeasureDefinition[];
     private _unit;
+    get refKey(): string | undefined;
+    get transform(): Transform | undefined;
+    get anchor(): MeasureParam | undefined;
     /** @throws an error if the actual unit sent by the server was not a LengthUnit */
     get unit(): LengthUnit;
     get aggregatedPrice(): CfgPrice;
@@ -136,15 +136,9 @@ export declare class _CfgProductInternal {
     _configurationHasChanged: (freshRef: CfgProductConfiguration, bubbleMode: ProductConfigurationBubbleMode) => Promise<void>;
     getApiSelection: () => AdditionalProductConfiguration;
     setApiSelection: (s: AdditionalProductConfiguration, doValidate: boolean, productLoaderForGroupedLoad?: ProductLoader | undefined) => Promise<boolean>;
-    copyFrom: (otherProduct: _CfgProductInternal, doValidate: boolean, productLoaderForGroupedLoad?: ProductLoader | undefined) => Promise<boolean>;
+    copyFrom: (source: _CfgProductInternal, doValidate: boolean, productLoaderForGroupedLoad?: ProductLoader | undefined) => Promise<boolean>;
     private _setApiSelectionWithOtherProduct;
     get syncGroupHandler(): SyncGroupsHandler | undefined;
-    get path(): CfgPath;
-    getFromPath(path: CfgPath): _CfgProductInternal | _CfgProductConfigurationInternal | _CfgFeatureInternal | _CfgOptionInternal;
-    getProductFromPath(path: CfgPath): _CfgProductInternal;
-    getProductConfigurationFromPath(path: CfgPath): _CfgProductConfigurationInternal;
-    getFeatureFromPath(path: CfgPath): _CfgFeatureInternal;
-    getOptionFromPath(path: CfgPath): _CfgOptionInternal;
     structureCompare: (other: _CfgProductInternal, strictOrder?: boolean, descriptionMatch?: boolean) => boolean;
     tryMatchSelection: (other: _CfgProductInternal, descriptionMatch?: boolean, productLoaderForGroupedLoad?: ProductLoader | undefined) => Promise<boolean>;
     /** Only features in selected options and selected additional products. */

package/dist/CfgProduct.js

@@ -7,20 +7,18 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-import { AggregatedLoadingObservable, compareArrays, count, Observable, toLengthUnit, } from "@configura/web-utilities";
+import { AggregatedLoadingObservable, assert, assertDefined, compareArrays, count, Observable, toLengthUnit, } from "@configura/web-utilities";
 import { CfgMeasureDefinition } from "./CfgMeasure.js";
-import { _CfgFeatureInternal } from "./productConfiguration/CfgFeature.js";
-import { ProductConfigurationBubbleMode, _CfgOptionInternal, } from "./productConfiguration/CfgOption.js";
-import { CfgProductConfiguration, _CfgProductConfigurationInternal, } from "./productConfiguration/CfgProductConfiguration.js";
+import { ProductConfigurationBubbleMode } from "./productConfiguration/CfgOption.js";
+import { CfgProductConfiguration } from "./productConfiguration/CfgProductConfiguration.js";
 import { collectAdditionalProductRefs } from "./productConfiguration/utilitiesProductConfiguration.js";
 import { wrapWithCache } from "./productLoader.js";
 import { SyncGroupsHandler } from "./syncGroups/SyncGroupsHandler.js";
-import { comparePricesObjects, correctDefaultsOnCatalogueParams, isSameProductRef, makeProductKey, } from "./utilitiesCatalogueData.js";
+import { comparePricesObjects, correctDefaultsOnCatalogueParams, isSameCatalogueParams, isSameProductRef, makeProductKey, } from "./utilitiesCatalogueData.js";
 function completeSettings(incompleteSettings) {
-    var _a, _b;
+    var _a;
     return {
         strictSelectOneSelectionCount: (_a = incompleteSettings === null || incompleteSettings === void 0 ? void 0 : incompleteSettings.strictSelectOneSelectionCount) !== null && _a !== void 0 ? _a : false,
-        strictSetApiSelectionMatch: (_b = incompleteSettings === null || incompleteSettings === void 0 ? void 0 : incompleteSettings.strictSetApiSelectionMatch) !== null && _b !== void 0 ? _b : false,
         syncGroupsApplyMode: incompleteSettings === null || incompleteSettings === void 0 ? void 0 : incompleteSettings.syncGroupsApplyMode,
     };
 }
@@ -53,7 +51,8 @@ function isDescriptionMatch(l, r) {
  * the class that should be used and interacted with.
  */
 export class _CfgProductInternal {
-    constructor(initSuccess, initFail, _productLoaderRaw, lang, catId, partNumber, settings, optional, selected, rootFeatureRefs, allRawFeatures, uuid, _rawUnit, _rawProductData, apiSelection, loadingObservable, refKey, refDescription, parent, root, transform, anchor, _syncGroupHandler) {
+    constructor(initSuccess, initFail, _productLoaderRaw, lang, catId, partNumber, settings, optional, selected, rootFeatureRefs, allRawFeatures, uuid, _rawUnit, _rawProductData, apiSelection, loadingObservable, parent, root, _additionalProductRef, _syncGroupHandler) {
+        var _a;
         this._productLoaderRaw = _productLoaderRaw;
         this.lang = lang;
         this.catId = catId;
@@ -63,11 +62,8 @@ export class _CfgProductInternal {
         this._rawUnit = _rawUnit;
         this._rawProductData = _rawProductData;
         this.loadingObservable = loadingObservable;
-        this.refKey = refKey;
-        this.refDescription = refDescription;
         this.parent = parent;
-        this.transform = transform;
-        this.anchor = anchor;
+        this._additionalProductRef = _additionalProductRef;
         this._syncGroupHandler = _syncGroupHandler;
         this._destroyed = false;
         this.additionalProducts = [];
@@ -154,8 +150,8 @@ export class _CfgProductInternal {
         this.setApiSelection = (s, doValidate, productLoaderForGroupedLoad) => __awaiter(this, void 0, void 0, function* () {
             return this._setApiSelectionWithOtherProduct(s, doValidate, productLoaderForGroupedLoad, undefined);
         });
-        this.copyFrom = (otherProduct, doValidate, productLoaderForGroupedLoad) => __awaiter(this, void 0, void 0, function* () {
-            return yield this._setApiSelectionWithOtherProduct(otherProduct.getApiSelection(), doValidate, productLoaderForGroupedLoad, otherProduct);
+        this.copyFrom = (source, doValidate, productLoaderForGroupedLoad) => __awaiter(this, void 0, void 0, function* () {
+            return yield this._setApiSelectionWithOtherProduct(source.getApiSelection(), doValidate, productLoaderForGroupedLoad, source);
         });
         this._setApiSelectionWithOtherProduct = (s, doValidate, productLoaderForGroupedLoad, sourceProduct) => __awaiter(this, void 0, void 0, function* () {
             // Wrap with cache will make getProduct for this function call use the same server call
@@ -166,6 +162,7 @@ export class _CfgProductInternal {
             let change = false;
             if (sourceProduct !== undefined) {
                 this._rawProductData = sourceProduct.rawProductData;
+                this.configuration._internal.populateFeatures(sourceProduct.configuration.rootFeatureRefs);
                 change = true; // We can not know if this is an actual change, so we assume it is
             }
             const configurationChange = yield this.configuration._internal.setApiSelection(s.selOptions, false);
@@ -186,27 +183,19 @@ export class _CfgProductInternal {
                 throw new Error(`Additional products are not same length. This product: "${this.key}". This product additional products count: ${additionalProductsCount}. Passed apiSelection additional products count: ${apiSelectionAdditionalProductsCount}`);
             }
             const sourceProductAdditionalProducts = sourceProduct === null || sourceProduct === void 0 ? void 0 : sourceProduct.additionalProducts;
-            if (sourceProductAdditionalProducts &&
-                additionalProductsCount !== sourceProductAdditionalProducts.length) {
-                throw new Error(`Passed sourceProduct does not have the same number of additional products as this.`);
-            }
+            assert(!sourceProductAdditionalProducts ||
+                additionalProductsCount === sourceProductAdditionalProducts.length, `Passed sourceProduct does not have the same number of additional products as this.`);
             if ((yield Promise.all(apiSelectionAdditionalProducts.map((apiSelectionAdditionalProduct, index) => {
                 var _a;
                 const refKey = apiSelectionAdditionalProduct.refKey;
-                if (refKey === undefined) {
-                    throw new Error("Additional product api configurations must have refKey.");
-                }
+                assertDefined(refKey, "Additional product api configurations must have refKey.");
                 const i = additionalProducts.findIndex((a) => refKey === a.refKey);
-                if (i === -1) {
-                    throw new Error(`Additional product not found. This product: "${this.key}". refKey of not found additional product: "${refKey}"`);
-                }
+                assert(i !== -1, `Additional product not found. This product: "${this.key}". refKey of not found additional product: "${refKey}"`);
                 let sourceProductAdditionalProduct = undefined;
                 if (sourceProductAdditionalProducts !== undefined) {
                     sourceProductAdditionalProduct =
                         (_a = sourceProductAdditionalProducts.find((a) => refKey === a.refKey)) === null || _a === void 0 ? void 0 : _a._internal;
-                    if (sourceProductAdditionalProduct === undefined) {
-                        throw new Error("Additional product not found in sourceProduct");
-                    }
+                    assertDefined(sourceProductAdditionalProduct, "Additional product not found in sourceProduct");
                 }
                 const additionalProduct = additionalProducts.splice(i, 1)[0]; // Splicing like this is okay because this is done synchronous. The setCon. is what is async.
                 return additionalProduct._internal._setApiSelectionWithOtherProduct(apiSelectionAdditionalProduct, doValidate, productLoaderForGroupedLoad, sourceProductAdditionalProduct);
@@ -372,14 +361,19 @@ export class _CfgProductInternal {
             let i = currentAdditionalProducts.length;
             while (i--) {
                 const currentAdditionalProduct = currentAdditionalProducts[i];
-                const refKey = currentAdditionalProduct.refKey;
-                const j = additionalProductRefs.findIndex((p) => p.prodRef.refKey === refKey);
+                const j = additionalProductRefs.findIndex((p) => {
+                    const prodRef = p.prodRef;
+                    return (prodRef.refKey === currentAdditionalProduct.refKey &&
+                        prodRef.partNumber === currentAdditionalProduct.partNumber &&
+                        isSameCatalogueParams(prodRef.catId, currentAdditionalProduct.catId));
+                });
                 if (j === -1) {
                     currentAdditionalProduct.destroy();
                     currentAdditionalProducts.splice(i, 1);
                     change = true;
                 }
                 else {
+                    currentAdditionalProduct._internal._updateAdditionalProdRef(additionalProductRefs[j].prodRef);
                     additionalProductRefs.splice(j, 1);
                 }
             }
@@ -392,7 +386,7 @@ export class _CfgProductInternal {
                     const additionalProductRef = p.prodRef;
                     return {
                         originalIndex: p.originalIndex,
-                        product: CfgProduct._makeNewRefFrom(yield _CfgProductInternal.make(productLoaderRaw, productLoaderForGroupedLoad, lang, additionalProductRef.catId, additionalProductRef.partNumber, this.settings, additionalProductRef.optional === true, this.loadingObservable, additionalProductRef.refKey, additionalProductRef.refDescription, this, this.root, additionalProductRef.transform, additionalProductRef.anchor)),
+                        product: CfgProduct._makeNewRefFrom(yield _CfgProductInternal.make(productLoaderRaw, productLoaderForGroupedLoad, lang, additionalProductRef.catId, additionalProductRef.partNumber, this.settings, additionalProductRef.optional === true, this.loadingObservable, this, this.root, additionalProductRef)),
                     };
                 }))()));
                 if (this._destroyed) {
@@ -411,7 +405,7 @@ export class _CfgProductInternal {
             }
         });
         this.root = root !== null && root !== void 0 ? root : this;
-        this.key = makeProductKey(catId, refKey || partNumber);
+        this.key = makeProductKey(catId, (_a = _additionalProductRef === null || _additionalProductRef === void 0 ? void 0 : _additionalProductRef.refKey) !== null && _a !== void 0 ? _a : partNumber);
         this._selected = optional ? selected : undefined;
         this.isAdditionalProduct = parent !== undefined;
         this._configuration = CfgProductConfiguration.make(initSuccess, initFail, rootFeatureRefs, allRawFeatures, apiSelection, this, this.root);
@@ -425,7 +419,7 @@ export class _CfgProductInternal {
                 var _a;
                 const p = new _CfgProductInternal(() => {
                     initSuccess(p);
-                }, initFail, this._productLoaderRaw, this.lang, this.catId, this.partNumber, this.settings, this.optional, this.selected, this._configuration.rootFeatureRefs, this._configuration.allRawFeatures, this.uuid, this._rawUnit, this._rawProductData, this.configuration.getApiSelection(), new AggregatedLoadingObservable(), this.refKey, this.refDescription, parent, root, this.transform, this.anchor, (_a = this._syncGroupHandler) === null || _a === void 0 ? void 0 : _a.clone());
+                }, initFail, this._productLoaderRaw, this.lang, this.catId, this.partNumber, this.settings, this.optional, this.selected, this._configuration.rootFeatureRefs, this._configuration.allRawFeatures, this.uuid, this._rawUnit, this._rawProductData, this.configuration.getApiSelection(), new AggregatedLoadingObservable(), parent, root, this._additionalProductRef, (_a = this._syncGroupHandler) === null || _a === void 0 ? void 0 : _a.clone());
             });
             for (const additionalProduct of this.additionalProducts) {
                 product.additionalProducts.push(CfgProduct._makeNewRefFrom(yield additionalProduct._internal.clone(product, root || product)));
@@ -433,8 +427,15 @@ export class _CfgProductInternal {
             return product;
         });
     }
+    _updateAdditionalProdRef(p) {
+        this._additionalProductRef = p;
+        if (p.optional !== this.optional) {
+            this._selected = p.optional ? false : undefined;
+        }
+    }
     get description() {
-        return this.refDescription || this._rawProductData.description;
+        var _a, _b;
+        return (_b = (_a = this._additionalProductRef) === null || _a === void 0 ? void 0 : _a.refDescription) !== null && _b !== void 0 ? _b : this._rawProductData.description;
     }
     get rootNodeSources() {
         return this._rawProductData.models;
@@ -460,6 +461,18 @@ export class _CfgProductInternal {
         }
         return this._measureDefinitions;
     }
+    get refKey() {
+        var _a;
+        return (_a = this._additionalProductRef) === null || _a === void 0 ? void 0 : _a.refKey;
+    }
+    get transform() {
+        var _a;
+        return (_a = this._additionalProductRef) === null || _a === void 0 ? void 0 : _a.transform;
+    }
+    get anchor() {
+        var _a;
+        return (_a = this._additionalProductRef) === null || _a === void 0 ? void 0 : _a.anchor;
+    }
     /** @throws an error if the actual unit sent by the server was not a LengthUnit */
     get unit() {
         if (this._unit === undefined) {
@@ -540,68 +553,17 @@ export class _CfgProductInternal {
     get syncGroupHandler() {
         return this.root._syncGroupHandler;
     }
-    get path() {
-        if (this.parent === undefined || this.refKey === undefined) {
-            return [];
-        }
-        return [...this.parent.path, "p", this.refKey];
-    }
-    getFromPath(path) {
-        path = path.slice();
-        const s = path.shift();
-        switch (s) {
-            case undefined:
-                return this;
-            case "p":
-                const refKey = path.shift();
-                const additionalProduct = this.additionalProducts.find((p) => p.refKey === refKey);
-                if (additionalProduct === undefined) {
-                    throw new Error(`Additional product not found "p, ${refKey}, ${path.join(", ")}"`);
-                }
-                return additionalProduct._internal.getFromPath(path);
-            case "c":
-                return this.configuration._internal.getFromPath(path);
-            default:
-                throw new Error(`Unexpected path segment ${s}`);
-        }
-    }
-    getProductFromPath(path) {
-        const p = this.getFromPath(path);
-        if (p instanceof _CfgProductInternal) {
-            return p;
-        }
-        throw new Error(`Path did not lead to a Product "${p.path.join(", ")}"`);
-    }
-    getProductConfigurationFromPath(path) {
-        const c = this.getFromPath(path);
-        if (c instanceof _CfgProductConfigurationInternal) {
-            return c;
-        }
-        throw new Error(`Path did not lead to a ProductConfiguration "${c.path.join(", ")}"`);
-    }
-    getFeatureFromPath(path) {
-        const f = this.getFromPath(path);
-        if (f instanceof _CfgFeatureInternal) {
-            return f;
-        }
-        throw new Error(`Path did not lead to a Feature "${f.path.join(", ")}"`);
-    }
-    getOptionFromPath(path) {
-        const o = this.getFromPath(path);
-        if (o instanceof _CfgOptionInternal) {
-            return o;
-        }
-        throw new Error(`Path did not lead to a Option "${o.path.join(", ")}"`);
-    }
 }
 _CfgProductInternal.make = (productLoaderRaw, productLoaderForGroupedLoad, // Used when instantiating the current product
-lang, catId, partNumber, settings, optional, loadingObservable, refKey, refDescription, parent, root, transform, anchor) => __awaiter(void 0, void 0, void 0, function* () {
+lang, catId, partNumber, settings, optional, loadingObservable, parent, root, additionalProductRef) => __awaiter(void 0, void 0, void 0, function* () {
     // Wrap with cache will make getProduct for this function call use the same server call
     // for the same product with the same params. Not retained for future calls, only used
     // at this initial load.
     productLoaderForGroupedLoad =
         productLoaderForGroupedLoad || wrapWithCache(productLoaderRaw);
-    const syncGroupHandler = root === undefined ? SyncGroupsHandler.make(settings.syncGroupsApplyMode) : undefined;
+    const syncGroupHandler = root === undefined
+        ? SyncGroupsHandler.make(settings.syncGroupsApplyMode, loadingObservable)
+        : undefined;
     const productResponse = yield productLoaderForGroupedLoad.getProduct(Object.assign(Object.assign({ lang }, correctDefaultsOnCatalogueParams(catId)), { partNumber }));
     const { productData, rootFeatureRefs, features: allRawFeatures, uuid, unit, } = productResponse;
     const product = yield new Promise((initSuccess, initFail) => {
@@ -610,7 +572,7 @@ lang, catId, partNumber, settings, optional, loadingObservable, refKey, refDescr
             // But we can not set the api selection synchronously. And the product configuration needs "this". So we use this callback.
             // Feel free to find a nicer more readable solution :)
             initSuccess(p);
-        }, initFail, productLoaderRaw, lang, catId, partNumber, settings, optional, !optional, rootFeatureRefs, allRawFeatures, uuid, unit, productData, productData.partsData.selOptions || [], loadingObservable, refKey, refDescription, parent, root, transform, anchor, syncGroupHandler);
+        }, initFail, productLoaderRaw, lang, catId, partNumber, settings, optional, !optional, rootFeatureRefs, allRawFeatures, uuid, unit, productData, productData.partsData.selOptions || [], loadingObservable, parent, root, additionalProductRef, syncGroupHandler);
     });
     yield product._syncAndLoadAdditionalProducts(productLoaderForGroupedLoad);
     // Product is guaranteed to be root
@@ -665,7 +627,7 @@ export class CfgProduct {
     }
     static make(productLoader, lang, catId, partNumber, settings) {
         return __awaiter(this, void 0, void 0, function* () {
-            return this._makeNewRefFrom(yield _CfgProductInternal.make(productLoader, undefined, lang, catId, partNumber, completeSettings(settings), false, new AggregatedLoadingObservable(), undefined, undefined, undefined, undefined, undefined, undefined));
+            return this._makeNewRefFrom(yield _CfgProductInternal.make(productLoader, undefined, lang, catId, partNumber, completeSettings(settings), false, new AggregatedLoadingObservable(), undefined, undefined, undefined));
         });
     }
     /**

package/dist/productConfiguration/CfgFeature.d.ts

@@ -1,6 +1,6 @@
 import { LengthUnit, Observable, SingleArgCallback } from "@configura/web-utilities";
 import { Feature, SelectedOption, SyncGroup } from "../CatalogueAPI.js";
-import { CfgPath, CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
+import { CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
 import { CfgMtrlApplication } from "../material/CfgMtrlApplication.js";
 import { CfgOption, ProductConfigurationBubbleMode, _CfgOptionInternal } from "./CfgOption.js";
 import { _CfgProductConfigurationInternal } from "./CfgProductConfiguration.js";
@@ -96,8 +96,6 @@ export declare class _CfgFeatureInternal {
     setApiSelection: (apiOptionSelectionMap: {
         [index: string]: SelectedOption;
     } | undefined) => Promise<boolean>;
-    get path(): CfgPath;
-    getFromPath(path: CfgPath): _CfgFeatureInternal | _CfgOptionInternal;
     /** Pushes to refresh stretch. Does not cause validation. */
     pushStretch: () => Promise<boolean>;
     structureCompare: (other: _CfgFeatureInternal, strictOrder?: boolean, descriptionMatch?: boolean) => boolean;

package/dist/productConfiguration/CfgFeature.js

@@ -7,7 +7,7 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
-import { compareArrays, convertLength, count, Observable, someMatch, toLengthUnit, } from "@configura/web-utilities";
+import { assertDefined, compareArrays, convertLength, count, Observable, someMatch, toLengthUnit, } from "@configura/web-utilities";
 import { CfgProduct } from "../CfgProduct.js";
 import { CfgMtrlApplication } from "../material/CfgMtrlApplication.js";
 import { CfgMtrlApplicationSource } from "../material/CfgMtrlApplicationSource.js";
@@ -189,18 +189,6 @@ export class _CfgFeatureInternal {
                         console.warn(wrongCountWarning);
                     }
                 }
-                if (!isGroup) {
-                    const apiSelectionCount = Object.keys(apiOptionSelectionMap).length;
-                    if (selectionCount !== apiSelectionCount) {
-                        const wrongCountWarning = `All provided Options are expected to be selected. Feature key: "${this.key}". Expected: ${apiSelectionCount} Actual: ${selectionCount}`;
-                        if (this.rootProduct.settings.strictSetApiSelectionMatch) {
-                            throw new Error(wrongCountWarning);
-                        }
-                        else {
-                            console.warn(wrongCountWarning);
-                        }
-                    }
-                }
             }
             if (change) {
                 if (isAllOptionsAffectedByAnySelection) {
@@ -311,10 +299,8 @@ export class _CfgFeatureInternal {
                 ProductConfigurationBubbleMode.ValidateAndBubbleSelectedAndApplySyncGroups) {
                 const product = this.rootProduct;
                 const syncGroupHandler = product.syncGroupHandler;
-                if (syncGroupHandler === undefined) {
-                    throw new Error(`No sync group handler when bubble mode ${ProductConfigurationBubbleMode.ValidateAndBubbleSelectedAndApplySyncGroups}`);
-                }
-                return yield syncGroupHandler.selectOption(product, optionInternal.path, on, wrapWithCache(product._productLoaderRaw));
+                assertDefined(syncGroupHandler, `Sync group handler is required for bubble mode ${ProductConfigurationBubbleMode.ValidateAndBubbleSelectedAndApplySyncGroups}`);
+                return yield syncGroupHandler.selectOption(product, optionInternal, on, wrapWithCache(product._productLoaderRaw));
             }
             if (!on) {
                 if (this.selectionType === SelectionType.Group) {
@@ -502,23 +488,6 @@ export class _CfgFeatureInternal {
         }
         return this._options;
     }
-    get path() {
-        return [...this.parent.path, this.code];
-    }
-    getFromPath(path) {
-        path = path.slice();
-        const s = path.shift();
-        switch (s) {
-            case undefined:
-                return this;
-            default:
-                const option = this.options.find((o) => o.code === s);
-                if (option === undefined) {
-                    throw new Error(`Option not found ${s}, ${path.join(", ")}`);
-                }
-                return option._internal.getFromPath(path);
-        }
-    }
     /**
      * Make fresh references to all options on this feature.
      * Also includes currently selected options.

package/dist/productConfiguration/CfgOption.d.ts

@@ -1,6 +1,6 @@
 import { LengthUnit, Observable, SingleArgCallback } from "@configura/web-utilities";
 import { Feature, Option, SelectedOption } from "../CatalogueAPI.js";
-import { CfgPath, CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
+import { CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
 import { CfgMtrlApplication } from "../material/CfgMtrlApplication.js";
 import { NumericValuesSelection } from "../utilitiesNumericValues.js";
 import { CfgFeature, _CfgFeatureInternal } from "./CfgFeature.js";
@@ -14,19 +14,19 @@ export declare enum ProductConfigurationBubbleMode {
      */
     BubbleSelected = "BubbleSelected",
     /**
-     * Bubble to the closest CfgProduct, let it revalidate, then that will continue
-     * the bubble after validate.
+     * Bubble to the closest CfgProduct, let it revalidate, then that will continue the bubble
+     * after validate.
      */
     Validate = "Validate",
     /**
-     * Bubble to the closest CfgProduct, let it revalidate, then that will continue
-     * the bubble after validate. If this is select it will turns on all ancestors all the way up.
+     * Bubble to the closest CfgProduct, let it revalidate, then that will continue the bubble
+     * after validate. If this is select it will turn on all ancestors all the way up.
      * So with this mode it is possible to select an option where its parents are not selected.
      */
     ValidateAndBubbleSelected = "ValidateAndBubbleSelected",
     /**
-     * Like ValidateAndBubbleSelected, but SyncGroups are applied after
-     * ValidateAndBubbleSelected has been done
+     * Like ValidateAndBubbleSelected, but SyncGroups are applied after ValidateAndBubbleSelected
+     * has been done
      */
     ValidateAndBubbleSelectedAndApplySyncGroups = "ValidateAndBubbleSelectedAndApplySyncGroups",
     /**
@@ -53,10 +53,9 @@ export declare enum ProductConfigurationBubbleMode {
     ToRoot = "ToRoot"
 }
 /**
- * This class is meant to only be used through CfgOption. It should
- * never be instantiated on its own. Normally the internal state of this class
- * should never be directly modified. CfgOption is the class that
- * should be used and interacted with.
+ * This class is only meant to be used through CfgOption. It should never be instantiated on its
+ * own. Normally the internal state of this class should never be directly modified. CfgOption is
+ * the class that should be used and interacted with.
  */
 export declare class _CfgOptionInternal {
     readonly rawOption: Option;
@@ -80,6 +79,7 @@ export declare class _CfgOptionInternal {
     get unit(): LengthUnit;
     get description(): string;
     get selected(): boolean;
+    get selectedChangeInProgress(): boolean;
     get ancestorsSelected(): boolean;
     get mtrlApplications(): CfgMtrlApplication[];
     get thumbnail(): string | undefined;
@@ -90,8 +90,6 @@ export declare class _CfgOptionInternal {
     _childHasChanged: (freshRef: CfgFeature, bubbleMode: ProductConfigurationBubbleMode) => Promise<void>;
     getApiSelection: () => SelectedOption;
     setApiSelection: (apiOptionSelection: SelectedOption | undefined) => Promise<boolean>;
-    get path(): CfgPath;
-    getFromPath(path: CfgPath): _CfgFeatureInternal | _CfgOptionInternal;
     structureCompare: (other: _CfgOptionInternal, strictOrder?: boolean, descriptionMatch?: boolean) => boolean;
     tryMatchSelection: (other: CfgOption, descriptionMatch?: boolean) => Promise<boolean>;
     keyMatch: (other: _CfgOptionInternal, descriptionMatch?: boolean) => boolean;
@@ -127,8 +125,21 @@ export declare class CfgOption {
     get unit(): LengthUnit;
     get description(): string;
     get selected(): boolean;
+    /**
+     * Selection state is in progress to be changed. This can be used in GUI
+     * to display the state as transitioning, or as already changed.
+     * If selectedChangeInProgress and:
+     * selected is true, it means that this is about to get unselected
+     * selected is false, it means that this is about to get selected
+     */
+    get selectedChangeInProgress(): boolean;
     /** Are all ancestors up to the CfgProductConfiguration selected? Includes self. */
     get ancestorsSelected(): boolean;
+    /**
+     * Selects this Option.
+     * Only Options belonging to Features that are "select many" can be deselected.
+     * Calling this will cause a validation call to the server.
+     */
     setSelected: (on: boolean) => Promise<boolean>;
     get thumbnail(): string | undefined;
     get upcharge(): number | undefined;