@salesforce/lds-ads-bridge 1.228.1 → 1.229.0-dev10

package/dist/ads-bridge-perf.js

@@ -16,12 +16,15 @@ var SnapshotState;
 const { create: create$1, entries, freeze: freeze$1, keys: keys$2, values } = Object;
 const { isArray: isArray$1 } = Array;
 const { parse: parse$1, stringify: stringify$1 } = JSON;
+const WeakSetCtor = WeakSet;
 
+const deeplyFrozen = new WeakSetCtor();
 function deepFreeze(value) {
-    // No need to freeze primitives
-    if (typeof value !== 'object' || value === null) {
+    // No need to freeze primitives or already frozen stuff
+    if (typeof value !== 'object' || value === null || deeplyFrozen.has(value)) {
         return;
     }
+    deeplyFrozen.add(value);
     if (isArray$1(value)) {
         for (let i = 0, len = value.length; i < len; i += 1) {
             deepFreeze(value[i]);
@@ -427,7 +430,7 @@ function createResourceParamsImpl(config, configMetadata) {
     }
     return resourceParams;
 }
-// engine version: 0.145.2-6a13677c
+// engine version: 0.146.0-dev5-a2ec6e3f
 
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
@@ -474,7 +477,7 @@ const callbacks$1 = [];
 function register(r) {
     callbacks$1.forEach((callback) => callback(r));
 }
-// version: 1.228.1-4e6356f71
+// version: 1.229.0-dev10-bc9ef2513
 
 /**
  * Returns true if the value acts like a Promise, i.e. has a "then" function,
@@ -5504,7 +5507,7 @@ function buildAdapterValidationConfig(displayName, paramsMeta) {
 }
 const keyPrefix = 'UiApi';
 
-const { assign, create, freeze, keys: keys$1 } = Object;
+const { assign, create, freeze, isFrozen, keys: keys$1 } = Object;
 const { hasOwnProperty: hasOwnProperty$1 } = Object.prototype;
 const { split, endsWith } = String.prototype;
 const { isArray } = Array;
@@ -8275,7 +8278,7 @@ function extractTrackedFieldsToTrie(recordId, node, root, config, visitedRecordI
                 extractTrackedFieldsToTrie(spanningLink.data.__ref, spanning, next, config, spanningVisitedRecordIds, depth + 1);
                 // For a spanning record that is detected to be a circular reference, we add the field along with Id and Name.
                 // It's possible for spanning record lookup fields to sometimes be circular, and sometimes not - depending on the value of the lookup field.
-                // For more information on scenarios that caused this fix: https://salesforce.quip.com/OvzNAh3eNIWY
+                // For more information on scenarios that caused this fix: search "LDS Recursive Spanning Fields Problem" in Quip
                 if (keys$1(next.children).length === 0) {
                     addScalarFieldId(next);
                     addScalarFieldName(next);
@@ -8422,22 +8425,12 @@ function _markMissingPath(record, path) {
     const fieldValueRepresentation = record.object('fields');
     const fieldName = path.shift();
     if (fieldValueRepresentation.isUndefined(fieldName) === true) {
-        // TODO [W-6900046]: remove cast, make RecordRepresentationNormalized['fields'] accept
-        // an undefined/non-present __ref if isMissing is present
-        fieldValueRepresentation.write(fieldName, {
-            __ref: undefined,
-            isMissing: true,
-        });
+        writeMissingFieldToStore(fieldValueRepresentation, fieldName);
         return;
     }
     const link = fieldValueRepresentation.link(fieldName);
     if (link.isPending()) {
-        // TODO [W-6900046]: remove cast, make RecordRepresentationNormalized['fields'] accept
-        // an undefined/non-present __ref if isMissing is present
-        fieldValueRepresentation.write(fieldName, {
-            __ref: undefined,
-            isMissing: true,
-        });
+        writeMissingFieldToStore(fieldValueRepresentation, fieldName);
     }
     else if (path.length > 0 && link.isMissing() === false) {
         const fieldValue = link.follow();
@@ -8453,6 +8446,19 @@ function _markMissingPath(record, path) {
         }
     }
 }
+/**
+ * Graph Node Directly modifies store entries, which is generally a non-starter.
+ * Until we can refactor this mess, you need to use this function to safely mark the RecordRepresentation
+ * as a seenId in the store when you perform this mutation.
+ */
+function writeMissingFieldToStore(field, fieldName) {
+    // TODO [W-6900046]: remove cast, make RecordRepresentationNormalized['fields'] accept
+    // an undefined/non-present __ref if isMissing is present
+    field.write(fieldName, {
+        __ref: undefined,
+        isMissing: true,
+    });
+}
 /**
  * Tells you if an objectApiName is supported by UI API or not.
  * Note: Luvio does not currently support all the entities, the list is limited to UI API supported entities
@@ -8584,8 +8590,11 @@ function mergeAndRefreshLowerVersionRecord(luvio, incoming, existing, incomingTr
     return existing;
 }
 function mergeRecordConflict(luvio, incoming, existing, recordConflictMap) {
-    const incomingNode = luvio.wrapNormalizedGraphNode(incoming);
-    const existingNode = luvio.wrapNormalizedGraphNode(existing);
+    const recordKey = keyBuilder$35(luvio, {
+        recordId: incoming.id,
+    });
+    const incomingNode = luvio.wrapNormalizedGraphNode(incoming, recordKey);
+    const existingNode = luvio.wrapNormalizedGraphNode(existing, recordKey);
     const incomingTrackedFieldsTrieRoot = {
         name: incoming.apiName,
         children: {},
@@ -8594,9 +8603,6 @@ function mergeRecordConflict(luvio, incoming, existing, recordConflictMap) {
         name: existing.apiName,
         children: {},
     };
-    const recordKey = keyBuilder$35(luvio, {
-        recordId: incoming.id,
-    });
     const trackedFieldsConfig = {
         maxDepth: configurationForRestAdapters.getTrackedFieldDepthOnCacheMergeConflict(),
         onlyFetchLeafNodeIdAndName: configurationForRestAdapters.getTrackedFieldLeafNodeIdAndNameOnly(),
@@ -8919,7 +8925,7 @@ function fulfill(existing, incoming) {
         const batchRequestWithSingleRequest = isSingleBatchRecordRequest(existingUrlParams) &&
             isSingleRecordRequest(urlParams) &&
             incomingUrlRecords[0] === existingUrlRecords[0];
-        if (!batchRequestWithSingleRequest) {
+        if (!batchRequestWithSingleRequest || isRestrictedPathCondition(existingPath, path)) {
             return false;
         }
     }
@@ -8962,6 +8968,12 @@ function isSingleBatchRecordRequest(urlParams) {
 function isSingleRecordRequest(urlParams) {
     return hasOwnProperty$1.call(urlParams, 'recordId');
 }
+function isRestrictedPathCondition(existingPath, path) {
+    // should not dedupe getRecordUi and getRecord as both of their representation is different
+    // records call cannot digest response of getRecordUi
+    return ((existingPath.includes('/record-ui') && path.includes('/records')) ||
+        (existingPath.includes('/records') && path.includes('/record-ui')));
+}
 
 const createResourceRequest$16 = function getUiApiRecordsByRecordIdCreateResourceRequest(config) {
     return {

package/dist/adsBridge.js

@@ -375,4 +375,4 @@ function withAdsBridge(callback) {
 }
 
 export { instrument, withAdsBridge };
-// version: 1.228.1-4e6356f71
+// version: 1.229.0-dev10-bc9ef2513

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lds-ads-bridge",
-    "version": "1.228.1",
+    "version": "1.229.0-dev10",
     "license": "SEE LICENSE IN LICENSE.txt",
     "description": "Bridge to sync data between LDS and ADS",
     "main": "dist/adsBridge.js",
@@ -29,7 +29,7 @@
         "release:corejar": "yarn build && ../core-build/scripts/core.js --name=lds-ads-bridge"
     },
     "devDependencies": {
-        "@salesforce/lds-adapters-uiapi": "*",
-        "@salesforce/lds-uiapi-record-utils": "*"
+        "@salesforce/lds-adapters-uiapi": "1.229.0-dev10",
+        "@salesforce/lds-uiapi-record-utils": "1.229.0-dev10"
     }
 }