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

package/dist/CfgProduct.d.ts

@@ -2,8 +2,8 @@ import { AggregatedLoadingObservable, LengthUnit, Observable, SingleArgCallback
 import { AdditionalProductConfiguration, 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";
@@ -27,9 +27,17 @@ export declare type CfgProductSettings = {
      */
     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 +64,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;
@@ -139,12 +145,6 @@ export declare class _CfgProductInternal {
     copyFrom: (otherProduct: _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,11 +7,10 @@ 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";
@@ -166,6 +165,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 +186,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);
@@ -540,59 +532,6 @@ 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* () {

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,11 +7,12 @@ 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";
 import { wrapWithCache } from "../productLoader.js";
+import { SyncGroupsPathHelper } from "../syncGroups/SyncGroupsPathHelper.js";
 import { CfgOption, ProductConfigurationBubbleMode } from "./CfgOption.js";
 import { _CfgProductConfigurationInternal } from "./CfgProductConfiguration.js";
 import { getMtrlPreview } from "./utilitiesProductConfiguration.js";
@@ -311,10 +312,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, SyncGroupsPathHelper.getPath(optionInternal), on, wrapWithCache(product._productLoaderRaw));
             }
             if (!on) {
                 if (this.selectionType === SelectionType.Group) {
@@ -502,23 +501,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;
@@ -90,8 +89,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;
@@ -129,6 +126,11 @@ export declare class CfgOption {
     get selected(): 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;

package/dist/productConfiguration/CfgOption.js

@@ -22,19 +22,19 @@ export var ProductConfigurationBubbleMode;
      */
     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.
      */
     ProductConfigurationBubbleMode["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.
      */
     ProductConfigurationBubbleMode["ValidateAndBubbleSelected"] = "ValidateAndBubbleSelected";
     /**
-     * Like ValidateAndBubbleSelected, but SyncGroups are applied after
-     * ValidateAndBubbleSelected has been done
+     * Like ValidateAndBubbleSelected, but SyncGroups are applied after ValidateAndBubbleSelected
+     * has been done
      */
     ProductConfigurationBubbleMode["ValidateAndBubbleSelectedAndApplySyncGroups"] = "ValidateAndBubbleSelectedAndApplySyncGroups";
     /**
@@ -75,10 +75,9 @@ function doesChildrenShareOptionsCode(features) {
     return false;
 }
 /**
- * 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 class _CfgOptionInternal {
     constructor(rawOption, allRawFeatures, siblingHasDuplicateDescription, parent, parentConfiguration, parentProduct, rootProduct) {
@@ -180,9 +179,8 @@ export class _CfgOptionInternal {
             agg.push(...feature._internal._getFeaturesWithCode(code));
             return agg;
         }, []);
-        // Description based key helps when switching between
-        // products with similar feature-options tree and trying
-        // to retain made selection
+        // Description based key helps when switching between products with similar feature-options
+        // tree and trying to retain made selection.
         this.key =
             this.description +
                 (this.description === "" || siblingHasDuplicateDescription ? this.code : "");
@@ -229,9 +227,9 @@ export class _CfgOptionInternal {
                     this._numericValue = val;
                     change = true;
                 }
-                // It could be that even though our value did not change some sibling value did, and this
-                // could make it needed to bubble later. Maybe. A bit uncertain about why I did not put
-                // this in the if-statement above //Linus
+                // It could be that even though our value did not change some sibling value did, and
+                // this could make it needed to bubble later. Maybe. A bit uncertain about why I did
+                // not put this in the if-statement above //Linus
                 if (yield this.parent.pushStretch()) {
                     change = true;
                 }
@@ -310,23 +308,6 @@ export class _CfgOptionInternal {
         }
         return this._features;
     }
-    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 feature = this.features.find((o) => o.code === s);
-                if (feature === undefined) {
-                    throw new Error(`Feature not found ${s}, ${path.join(", ")}`);
-                }
-                return feature._internal.getFromPath(path);
-        }
-    }
     _freshRefDescendants() {
         const features = this._features || [];
         for (let i = 0; i < features.length; i++) {
@@ -347,7 +328,7 @@ export class CfgOption {
         this.isBackedBySame = (other) => this._internal === other._internal;
         this.setNumericValue = (val, doSelectOption) => __awaiter(this, void 0, void 0, function* () { return yield this._internal.setNumericValue(val, doSelectOption); });
         this.isAllowedNumericValue = (val) => this._internal.isAllowedNumericValue(val);
-        /*
+        /**
          * 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.

package/dist/productConfiguration/CfgProductConfiguration.d.ts

@@ -1,8 +1,8 @@
 import { LengthUnit, Observable, SingleArgCallback } from "@configura/web-utilities";
 import { Feature, FeatureRef, SelectedOption } from "../CatalogueAPI.js";
-import { CfgPath, CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
+import { CfgProduct, _CfgProductInternal } from "../CfgProduct.js";
 import { CfgFeature, _CfgFeatureInternal } from "./CfgFeature.js";
-import { ProductConfigurationBubbleMode, _CfgOptionInternal } from "./CfgOption.js";
+import { ProductConfigurationBubbleMode } from "./CfgOption.js";
 export declare type ProductConfigurationChangeNotification = {
     freshRef: CfgProductConfiguration;
 };
@@ -43,8 +43,6 @@ export declare class _CfgProductConfigurationInternal {
      * outside we want notifications to bubble all the way to the root.
      */
     setApiSelection: (selectedOptions: SelectedOption[], bubbleToRoot: boolean) => Promise<boolean>;
-    get path(): CfgPath;
-    getFromPath(path: CfgPath): _CfgProductConfigurationInternal | _CfgFeatureInternal | _CfgOptionInternal;
     structureCompare: (other: _CfgProductConfigurationInternal, strictOrder?: boolean, descriptionMatch?: boolean) => boolean;
     /**
      * When used internally the notifications are taken care off by the caller, but if set from

package/dist/productConfiguration/CfgProductConfiguration.js

@@ -215,23 +215,6 @@ export class _CfgProductConfigurationInternal {
             features[i] = CfgFeature._makeNewRefFrom(featureInternal);
         }
     }
-    get path() {
-        return [...this.parentProduct.path, "c"];
-    }
-    getFromPath(path) {
-        path = path.slice();
-        const s = path.shift();
-        switch (s) {
-            case undefined:
-                return this;
-            default:
-                const feature = this.features.find((f) => f.code === s);
-                if (feature === undefined) {
-                    throw new Error(`Feature not found ${s}, ${path.join(", ")}`);
-                }
-                return feature._internal.getFromPath(path);
-        }
-    }
 }
 export class CfgProductConfiguration {
     /**

package/dist/syncGroups/SyncGroupsApplier.js

@@ -412,17 +412,18 @@ class OntoSyncState {
         if (optionCode === currentSyncGroupOptionCode) {
             return false;
         }
-        // featureDidJustComeIntoScope, in CET there is a feature that if a feature appears which can not be set
-        // to the current sync group value, then it will set in the opposite direction. Like if the sync group was
-        // empty. To avoid bouncing back and forth we will need to enforce that a sync group can only be updated
-        // once per transaction
+        // featureDidJustComeIntoScope, in CET there is a feature that if a feature appears which
+        // can not be set to the current sync group value, then it will set in the opposite
+        // direction. Like if the sync group was empty. To avoid bouncing back and forth we will
+        // need to enforce that a sync group can only be updated once per transaction
         if (!(activeSelectionForce ||
             currentSyncGroupOptionCode === undefined ||
             (featureDidJustComeIntoScope &&
                 feature.options.every((o) => currentSyncGroupOptionCode !== o.code)))) {
             return false;
         }
-        syncState.logDebug();
+        // TODO: Remove
+        //syncState.logDebug();
         transaction.addSyncGroupAffectedForSelectOne(syncCode);
         syncState.setForSelectOne(syncCode, optionCode);
         return true;

package/dist/syncGroups/SyncGroupsApplyMode.d.ts

@@ -1,15 +1,21 @@
 /**
  * The SyncGroupsApplyMode controls how many SyncGroups can be updated in the SyncState for one
- * run of rootProductOntoSyncState (C.). "Fast" is default and will make any SyncGroup that
- * should be updated update before running syncStateOntoRootProduct (B.) This way several
- * SyncGroups can be applied in one go before sending validation calls to the server. The
- * downside of this is that is not exactly how CET (the desktop software) works. CET will apply
- * Features to the SyncState as soon as it gets the chance. This is the same as running with
- * "Strict" here. This will potentially generate a lot more validate calls, increasing delay
- * and cost. So if you don't have to, stick with "Fast".
+ * run of rootProductOntoSyncState.
+ *
+ * "Fast" will update any SyncGroup that should be updated before running syncStateOntoRootProduct
+ * This way several SyncGroups can be applied in one go before sending the validation calls to the * server, making the whole process faster.
+ *
+ * The downside of this is that is not exactly how CET (the desktop software) works. CET will
+ * instead apply Features to the SyncState as soon as it gets the chance.
+ *
+ * "Strict" tries to behave exactly as CET. This will potentially generate a lot more validate
+ * calls, increasing delay and cost.
+ *
+ * Strict is the safer option of the two, but we still recommend trying out Fast since it should
+ * work fine in most cases.
  */
 export declare enum SyncGroupsApplyMode {
-    Fast = "Fast",
-    Strict = "Strict"
+    Strict = "Strict",
+    Fast = "Fast"
 }
 //# sourceMappingURL=SyncGroupsApplyMode.d.ts.map

package/dist/syncGroups/SyncGroupsApplyMode.js

@@ -1,15 +1,21 @@
 /**
  * The SyncGroupsApplyMode controls how many SyncGroups can be updated in the SyncState for one
- * run of rootProductOntoSyncState (C.). "Fast" is default and will make any SyncGroup that
- * should be updated update before running syncStateOntoRootProduct (B.) This way several
- * SyncGroups can be applied in one go before sending validation calls to the server. The
- * downside of this is that is not exactly how CET (the desktop software) works. CET will apply
- * Features to the SyncState as soon as it gets the chance. This is the same as running with
- * "Strict" here. This will potentially generate a lot more validate calls, increasing delay
- * and cost. So if you don't have to, stick with "Fast".
+ * run of rootProductOntoSyncState.
+ *
+ * "Fast" will update any SyncGroup that should be updated before running syncStateOntoRootProduct
+ * This way several SyncGroups can be applied in one go before sending the validation calls to the * server, making the whole process faster.
+ *
+ * The downside of this is that is not exactly how CET (the desktop software) works. CET will
+ * instead apply Features to the SyncState as soon as it gets the chance.
+ *
+ * "Strict" tries to behave exactly as CET. This will potentially generate a lot more validate
+ * calls, increasing delay and cost.
+ *
+ * Strict is the safer option of the two, but we still recommend trying out Fast since it should
+ * work fine in most cases.
  */
 export var SyncGroupsApplyMode;
 (function (SyncGroupsApplyMode) {
-    SyncGroupsApplyMode["Fast"] = "Fast";
     SyncGroupsApplyMode["Strict"] = "Strict";
+    SyncGroupsApplyMode["Fast"] = "Fast";
 })(SyncGroupsApplyMode || (SyncGroupsApplyMode = {}));

package/dist/syncGroups/SyncGroupsHandler.d.ts

@@ -1,6 +1,7 @@
-import { CfgPath, _CfgProductInternal } from "../CfgProduct.js";
+import { _CfgProductInternal } from "../CfgProduct.js";
 import { ProductLoader } from "../productLoader.js";
 import { SyncGroupsApplyMode } from "./SyncGroupsApplyMode.js";
+import { CfgPath } from "./SyncGroupsPathHelper.js";
 import { SyncGroupsTransaction } from "./SyncGroupsTransaction.js";
 export declare type SyncCode = string;
 export declare type OptionCode = string;

package/dist/syncGroups/SyncGroupsHandler.js

@@ -19,7 +19,7 @@ export class SyncGroupsHandler {
         this._syncState = _syncState;
         this.updateMode = updateMode;
     }
-    static make(updateMode = SyncGroupsApplyMode.Fast) {
+    static make(updateMode = SyncGroupsApplyMode.Strict) {
         return new SyncGroupsHandler(new SyncGroupsState(), updateMode);
     }
     clone() {

package/dist/syncGroups/SyncGroupsPathHelper.d.ts

@@ -0,0 +1,27 @@
+import { _CfgProductInternal } from "../CfgProduct.js";
+import { _CfgFeatureInternal } from "../productConfiguration/CfgFeature.js";
+import { _CfgOptionInternal } from "../productConfiguration/CfgOption.js";
+import { _CfgProductConfigurationInternal } from "../productConfiguration/CfgProductConfiguration.js";
+export declare type CfgPathSegment = string;
+/**
+ * Represents the path to a node in the configuration tree.
+ *
+ * The format consists of a number of CfgPathSegments, each of which is currently a string. The
+ * exact meaning of the strings vary depending on where they are located in the path.
+ *
+ * @warning
+ * The format is strictly internal and can change in later versions of the SDK.
+ * Data stored in a CfgPath should be thus be considered transient and never stored or transmitted.
+ */
+export declare type CfgPath = CfgPathSegment[];
+declare type PathItemType = _CfgProductInternal | _CfgProductConfigurationInternal | _CfgFeatureInternal | _CfgOptionInternal;
+export declare class SyncGroupsPathHelper {
+    static getPath(item: PathItemType): CfgPath;
+    static getFromPath(path: CfgPath, root: PathItemType): PathItemType;
+    static getProductFromPath(path: CfgPath, item: _CfgProductInternal): _CfgProductInternal;
+    static getProductConfigurationFromPath(path: CfgPath, item: _CfgProductInternal): _CfgProductConfigurationInternal;
+    static getFeatureFromPath(path: CfgPath, item: PathItemType): _CfgFeatureInternal;
+    static getOptionFromPath(path: CfgPath, item: PathItemType): _CfgOptionInternal;
+}
+export {};
+//# sourceMappingURL=SyncGroupsPathHelper.d.ts.map

package/dist/syncGroups/SyncGroupsPathHelper.js

@@ -0,0 +1,89 @@
+import { assert, assertDefined } from "@configura/web-utilities";
+import { _CfgProductInternal } from "../CfgProduct.js";
+import { _CfgFeatureInternal } from "../productConfiguration/CfgFeature.js";
+import { _CfgOptionInternal } from "../productConfiguration/CfgOption.js";
+import { _CfgProductConfigurationInternal } from "../productConfiguration/CfgProductConfiguration.js";
+export class SyncGroupsPathHelper {
+    static getPath(item) {
+        if (item instanceof _CfgProductInternal) {
+            const parent = item.parent;
+            const refKey = item.refKey;
+            if (parent === undefined || refKey === undefined) {
+                return [];
+            }
+            return [...this.getPath(parent), "p", refKey];
+        }
+        if (item instanceof _CfgProductConfigurationInternal) {
+            return [...this.getPath(item.parentProduct), "c"];
+        }
+        if (item instanceof _CfgFeatureInternal) {
+            return [...this.getPath(item.parent), item.code];
+        }
+        if (item instanceof _CfgOptionInternal) {
+            return [...this.getPath(item.parent), item.code];
+        }
+        throw new Error("This should not happen");
+    }
+    static getFromPath(path, root) {
+        path = path.slice();
+        let item = root;
+        while (path.length > 0) {
+            const segment = path.shift();
+            if (item instanceof _CfgProductInternal) {
+                switch (segment) {
+                    case "p":
+                        const refKey = path.shift();
+                        const additional = item.additionalProducts.find((p) => p.refKey === refKey);
+                        assertDefined(additional, `Additional product not found "p, ${refKey}, ${path.join(", ")}"`);
+                        item = additional._internal;
+                        break;
+                    case "c":
+                        assert(item instanceof _CfgProductInternal, "Was not product");
+                        item = item.configuration._internal;
+                        break;
+                    default:
+                        throw new Error(`Unexpected path segment ${segment}`);
+                }
+            }
+            else if (item instanceof _CfgProductConfigurationInternal) {
+                const feature = item.features.find((f) => f.code === segment);
+                assertDefined(feature, `Feature not found ${segment}, ${path.join(", ")}`);
+                item = feature._internal;
+            }
+            else if (item instanceof _CfgFeatureInternal) {
+                const option = item.options.find((o) => o.code === segment);
+                assertDefined(option, `Option not found ${segment}, ${path.join(", ")}`);
+                item = option._internal;
+            }
+            else if (item instanceof _CfgOptionInternal) {
+                const feature = item.features.find((o) => o.code === segment);
+                assertDefined(feature, `Feature not found ${segment}, ${path.join(", ")}`);
+                item = feature._internal;
+            }
+            else {
+                throw new Error("This should not happen (unexpected type)");
+            }
+        }
+        return item;
+    }
+    static getProductFromPath(path, item) {
+        const p = this.getFromPath(path, item);
+        assert(p instanceof _CfgProductInternal, `Path did not lead to a Product`);
+        return p;
+    }
+    static getProductConfigurationFromPath(path, item) {
+        const c = this.getFromPath(path, item);
+        assert(c instanceof _CfgProductConfigurationInternal, `Path did not lead to a ProductConfiguration"`);
+        return c;
+    }
+    static getFeatureFromPath(path, item) {
+        const f = this.getFromPath(path, item);
+        assert(f instanceof _CfgFeatureInternal, `Path did not lead to a Feature`);
+        return f;
+    }
+    static getOptionFromPath(path, item) {
+        const o = this.getFromPath(path, item);
+        assert(o instanceof _CfgOptionInternal, `Path did not lead to a Option`);
+        return o;
+    }
+}

package/dist/syncGroups/SyncGroupsState.d.ts

@@ -15,6 +15,6 @@ export declare class SyncGroupsState {
     setForSelectMany(syncCode: SyncCode, optionCode: OptionCode, selected: boolean): void;
     getForSelectOne(syncCode: SyncCode): OptionCode | undefined;
     getForSelectMany(syncCode: SyncCode, optionCode: OptionCode): boolean | undefined;
-    logDebug(): void;
+    logDebug(group?: string, code?: string, selected?: boolean): void;
 }
 //# sourceMappingURL=SyncGroupsState.d.ts.map

package/dist/syncGroups/SyncGroupsState.js

@@ -31,7 +31,7 @@ export class SyncGroupsState {
     }
     setForSelectOne(syncCode, optionCode) {
         this._selectOne.set(syncCode, optionCode);
-        this.logDebug();
+        this.logDebug(syncCode, optionCode);
     }
     setForSelectMany(syncCode, optionCode, selected) {
         let forSyncCode = this._selectMany.get(syncCode);
@@ -40,7 +40,7 @@ export class SyncGroupsState {
             this._selectMany.set(syncCode, forSyncCode);
         }
         forSyncCode.set(optionCode, selected);
-        this.logDebug();
+        this.logDebug(syncCode, optionCode, selected);
     }
     getForSelectOne(syncCode) {
         return this._selectOne.get(syncCode);
@@ -49,13 +49,60 @@ export class SyncGroupsState {
         var _a;
         return (_a = this._selectMany.get(syncCode)) === null || _a === void 0 ? void 0 : _a.get(optionCode);
     }
-    logDebug() {
-        console.table(Array.from(this._selectOne.entries()));
-        console.table(Array.from(this._selectMany.entries()).reduce((a, [groupCode, optionCodeToSelected]) => {
+    logDebug(group, code, selected) {
+        const isMany = selected !== undefined;
+        const selectOne = Array.from(this._selectOne.entries());
+        const selectMany = Array.from(this._selectMany.entries()).reduce((a, [groupCode, optionCodeToSelected]) => {
             for (const [optionCode, selected] of optionCodeToSelected) {
                 a.push([groupCode, optionCode, selected]);
             }
             return a;
-        }, []));
+        }, []);
+        let columnWidth = 1;
+        selectOne.forEach((e) => {
+            columnWidth = Math.max(columnWidth, e[0].length);
+        });
+        selectMany.forEach((e) => {
+            columnWidth = Math.max(columnWidth, e[0].length);
+        });
+        const padding = Array(columnWidth).join(" ");
+        const styleBold = "font-weight: bold";
+        const styleBoldThis = "color:blue; font-weight: bold";
+        const styleBoldOn = "color:green; font-weight: bold";
+        const styleBoldOff = "color:red; font-weight: bold";
+        const styleThis = "color:blue";
+        const styleOn = "color:green";
+        const styleOff = "color:red";
+        const output = [];
+        if (group === undefined) {
+            output.push("");
+        }
+        else if (isMany) {
+            output.push(...[
+                `%c${code} %cin %c${group} %cset to %c${selected ? "on" : "off"}\n`,
+                styleBoldThis,
+                "",
+                styleBoldThis,
+                "",
+                selected ? styleBoldOn : styleBoldOff,
+            ]);
+        }
+        else {
+            output.push(...[`%c${group}%c set to %c${code}\n`, styleBoldThis, "", styleBoldThis]);
+        }
+        output[0] = output[0] + "%cSync State (single)";
+        output.push(styleBold);
+        selectOne.forEach((e) => {
+            output[0] = output[0] + `\n  %c${(padding + e[0]).slice(-columnWidth)}: ${e[1]}`;
+            output.push(!isMany && group === e[0] ? styleThis : "");
+        });
+        output[0] = output[0] + "\n%cSync State (multi)";
+        output.push(styleBold);
+        selectMany.forEach((e) => {
+            output[0] = output[0] + `\n  %c${(padding + e[0]).slice(-columnWidth)}: %c${e[1]}`;
+            output.push(isMany && group === e[0] ? styleThis : "");
+            output.push(e[2] === true ? styleOn : styleOff);
+        });
+        console.log(...output);
     }
 }

package/dist/syncGroups/SyncGroupsTransaction.d.ts

@@ -1,9 +1,10 @@
-import { CfgPath, _CfgProductInternal } from "../CfgProduct.js";
+import { _CfgProductInternal } from "../CfgProduct.js";
 import { _CfgFeatureInternal } from "../productConfiguration/CfgFeature.js";
 import { _CfgOptionInternal } from "../productConfiguration/CfgOption.js";
 import { ProductLoader } from "../productLoader.js";
 import { SyncGroupsApplyMode } from "./SyncGroupsApplyMode.js";
 import { OptionCode, SyncCode } from "./SyncGroupsHandler.js";
+import { CfgPath } from "./SyncGroupsPathHelper.js";
 import { SyncGroupsState } from "./SyncGroupsState.js";
 /**
  * A transaction is normally limited to one user interaction. Like opening a product or

package/dist/syncGroups/SyncGroupsTransaction.js

@@ -7,8 +7,10 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, P, ge
         step((generator = generator.apply(thisArg, _arguments || [])).next());
     });
 };
+import { assert } from "@configura/web-utilities";
 import { SelectionType } from "../productConfiguration/CfgFeature.js";
 import { SyncGroupsApplier } from "./SyncGroupsApplier.js";
+import { SyncGroupsPathHelper } from "./SyncGroupsPathHelper.js";
 /**
  * A transaction is normally limited to one user interaction. Like opening a product or
  * selecting an option. This object is used to keep data for one transaction. In particular
@@ -57,32 +59,24 @@ export class SyncGroupsTransaction {
     }
     selectOption(optionPath, on) {
         return __awaiter(this, void 0, void 0, function* () {
-            const targetOption = this.target.getOptionFromPath(optionPath);
+            const targetOption = SyncGroupsPathHelper.getOptionFromPath(optionPath, this.target);
             return yield SyncGroupsApplier.selectOption(this, targetOption, on);
         });
     }
     addSelectOneFeatureAffected(feature) {
-        if (feature.selectionType !== SelectionType.SelectOne) {
-            throw new Error("Can only be used for SelectOne Feature");
-        }
+        assert(feature.selectionType === SelectionType.SelectOne, "Can only be used for SelectOne Feature");
         this.affectedSelectOneFeatures.add(feature);
     }
     addSelectManyOptionAffected(option) {
-        if (option.parent.selectionType !== SelectionType.SelectMany) {
-            throw new Error("Can only be used for option in SelectMany Feature");
-        }
+        assert(option.parent.selectionType === SelectionType.SelectMany, "Can only be used for option in SelectMany Feature");
         this.affectedSelectManyOptions.add(option);
     }
     hasSelectOneFeatureBeenAffected(feature) {
-        if (feature.selectionType !== SelectionType.SelectOne) {
-            throw new Error("Can only be used for SelectOne Feature");
-        }
+        assert(feature.selectionType === SelectionType.SelectOne, "Can only be used for SelectOne Feature");
         return this.affectedSelectOneFeatures.has(feature);
     }
     hasSelectManyOptionBeenAffected(option) {
-        if (option.parent.selectionType !== SelectionType.SelectMany) {
-            throw new Error("Can only be used for option in SelectMany Feature");
-        }
+        assert(option.parent.selectionType === SelectionType.SelectMany, "Can only be used for option in SelectMany Feature");
         return this.affectedSelectManyOptions.has(option);
     }
     addSyncGroupAffectedForSelectOne(syncCode) {

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@configura/web-api",
-	"version": "1.6.1-alpha.0",
+	"version": "1.6.1-alpha.1",
 	"license": "Apache-2.0",
 	"type": "module",
 	"main": "dist/index.js",
@@ -23,7 +23,7 @@
 		"access": "public"
 	},
 	"dependencies": {
-		"@configura/web-utilities": "^1.6.1-alpha.0"
+		"@configura/web-utilities": "^1.6.1-alpha.1"
 	},
-	"gitHead": "aa99963ec2c2e4ab4ef1795e7ad6204fef31da75"
+	"gitHead": "afaf1e342f0648246a1dcb6f7bc636d4d1979bec"
 }