@salesforce/lds-ads-bridge 1.266.0-dev2 → 1.266.0-dev21

package/dist/ads-bridge-perf.js

@@ -21,7 +21,9 @@ const WeakSetCtor = WeakSet;
 const deeplyFrozen = new WeakSetCtor();
 function deepFreeze(value) {
     // No need to freeze primitives or already frozen stuff
-    if (typeof value !== 'object' || value === null || deeplyFrozen.has(value)) {
+    if (typeof value !== 'object' ||
+        value === null ||
+        deeplyFrozen.has(value)) {
         return;
     }
     deeplyFrozen.add(value);
@@ -383,6 +385,7 @@ function ingestShape(input, path, luvio, store, timestamp, ttlToUse, key, normal
             namespace,
             version,
             representationName,
+            ingestionTimestamp: timestamp,
         };
         luvio.publishStoreMetadata(key, storeMetadataParams);
     }
@@ -430,7 +433,7 @@ function createResourceParamsImpl(config, configMetadata) {
     }
     return resourceParams;
 }
-// engine version: 0.154.6-a3a5150a
+// engine version: 0.154.7-dev8-fca5df34
 
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
@@ -478,7 +481,7 @@ const callbacks$1 = [];
 function register(r) {
     callbacks$1.forEach((callback) => callback(r));
 }
-// version: 1.266.0-dev2-7c2f3615f
+// version: 1.266.0-dev21-b2a247476
 
 /**
  * Returns true if the value acts like a Promise, i.e. has a "then" function,
@@ -4794,15 +4797,17 @@ function splitQualifiedFieldApiName(fieldApiName) {
 /**
  * Returns the field API name, qualified with an object name if possible.
  * @param value The value from which to get the qualified field API name.
+ * @param onlyQualifiedFieldNames - Whether or not this function should skip fieldApiName that do not include the delimiter '.'
  * @return The qualified field API name.
  */
-function getFieldApiName(value) {
+function getFieldApiName(value, onlyQualifiedFieldNames = false) {
     // Note: tightening validation logic changes behavior from userland getting
     // a server-provided error to the adapter noop'ing. In 224 we decided to not
-    // change the behavior.
+    // change the behavior. In 250 we decided to add the 'onlyQualifiedFieldName' flag to tighten the logic
+    // optionally to avoid issues with persisted invalid field names.
     if (isString(value)) {
         const trimmed = value.trim();
-        if (trimmed.length > 0) {
+        if (trimmed.length > 0 && (onlyQualifiedFieldNames ? trimmed.indexOf('.') > -1 : true)) {
             return trimmed;
         }
     }
@@ -4815,15 +4820,19 @@ function getFieldApiName(value) {
 /**
  * Returns the field API name.
  * @param value The value from which to get the field API name.
+ * @param options Option bag. onlyQualifiedFieldNames is a boolean that allows this function to skip returning invalid FieldApiNames.
  * @returns The field API name.
  */
-function getFieldApiNamesArray(value) {
+function getFieldApiNamesArray(value, options = { onlyQualifiedFieldNames: false }) {
     const valueArray = isArray(value) ? value : [value];
     const array = [];
     for (let i = 0, len = valueArray.length; i < len; i += 1) {
         const item = valueArray[i];
-        const apiName = getFieldApiName(item);
+        const apiName = getFieldApiName(item, options.onlyQualifiedFieldNames);
         if (apiName === undefined) {
+            if (options.onlyQualifiedFieldNames) {
+                continue; // Just skips invalid field names rather than failing to return an array at all
+            }
             return undefined;
         }
         push$1.call(array, apiName);
@@ -4920,7 +4929,7 @@ function toSortedStringArray(value) {
 }
 
 const VERSION$2j = "d1e589a127fb1060c89070cdb6f500b2";
-function validate$1Q(obj, path = 'RecordTypeInfoRepresentation') {
+function validate$1R(obj, path = 'RecordTypeInfoRepresentation') {
     const v_error = (() => {
         if (typeof obj !== 'object' || ArrayIsArray(obj) || obj === null) {
             return new TypeError('Expected "object" but received "' + typeof obj + '" (at "' + path + '")');
@@ -5012,7 +5021,7 @@ function equals$14(existing, incoming) {
 }
 
 const VERSION$2i = "195d918987a35f45e1aa4dce9a11d8c5";
-function validate$1P(obj, path = 'FieldValueRepresentation') {
+function validate$1Q(obj, path = 'FieldValueRepresentation') {
     const v_error = (() => {
         if (typeof obj !== 'object' || ArrayIsArray(obj) || obj === null) {
             return new TypeError('Expected "object" but received "' + typeof obj + '" (at "' + path + '")');
@@ -5181,7 +5190,7 @@ const getTypeCacheKeys$1Y = (rootKeySet, luvio, input, _fullPathFactory) => {
 
 const TTL$E = 120000;
 const VERSION$2h = "79cb5ac9f44542f683d00245fdfe500d";
-function validate$1O(obj, path = 'RecordCollectionRepresentation') {
+function validate$1P(obj, path = 'RecordCollectionRepresentation') {
     const v_error = (() => {
         if (typeof obj !== 'object' || ArrayIsArray(obj) || obj === null) {
             return new TypeError('Expected "object" but received "' + typeof obj + '" (at "' + path + '")');
@@ -5454,7 +5463,7 @@ function equals$12(existing, incoming) {
 }
 const ingest$1U = function RecordCollectionRepresentationIngest(input, path, luvio, store, timestamp) {
     if (process.env.NODE_ENV !== 'production') {
-        const validateError = validate$1O(input);
+        const validateError = validate$1P(input);
         if (validateError !== null) {
             throw validateError;
         }
@@ -5490,7 +5499,7 @@ const keyBuilderFromType$B = function RecordRepresentationKeyBuilderFromType(luv
 
 const TTL$D = 30000;
 const VERSION$2g = "98c5b18512e48ca8d28727549507e4ba";
-function validate$1N(obj, path = 'RecordRepresentation') {
+function validate$1O(obj, path = 'RecordRepresentation') {
     const v_error = (() => {
         if (typeof obj !== 'object' || ArrayIsArray(obj) || obj === null) {
             return new TypeError('Expected "object" but received "' + typeof obj + '" (at "' + path + '")');
@@ -5620,7 +5629,7 @@ function validate$1N(obj, path = 'RecordRepresentation') {
         const path_recordTypeInfo = path + '.recordTypeInfo';
         let obj_recordTypeInfo_union0 = null;
         const obj_recordTypeInfo_union0_error = (() => {
-            const referencepath_recordTypeInfoValidationError = validate$1Q(obj_recordTypeInfo, path_recordTypeInfo);
+            const referencepath_recordTypeInfoValidationError = validate$1R(obj_recordTypeInfo, path_recordTypeInfo);
             if (referencepath_recordTypeInfoValidationError !== null) {
                 let message = 'Object doesn\'t match RecordTypeInfoRepresentation (at "' + path_recordTypeInfo + '")\n';
                 message += referencepath_recordTypeInfoValidationError.message.split('\n').map((line) => '\t' + line).join('\n');
@@ -8078,7 +8087,7 @@ function normalize$P(input, existing, path, luvio, store, timestamp, fieldsTrie,
 function makeIngest(fieldsTrie, optionalFieldsTrie, recordConflictMap) {
     return (input, path, luvio, store, timestamp) => {
         if (process.env.NODE_ENV !== 'production') {
-            const validateError = validate$1P(input);
+            const validateError = validate$1Q(input);
             if (validateError !== null) {
                 throw validateError;
             }
@@ -8171,7 +8180,7 @@ const createRecordIngest = (fieldsTrie, optionalFieldsTrie, recordConflictMap) =
     const childNormalize = createChildRecordNormalize(fieldsTrie, optionalFieldsTrie, recordConflictMap);
     return (input, path, luvio, store, timestamp) => {
         if (process.env.NODE_ENV !== 'production') {
-            const validateError = validate$1N(input);
+            const validateError = validate$1O(input);
             if (validateError !== null) {
                 throw validateError;
             }
@@ -8200,6 +8209,7 @@ const createRecordIngest = (fieldsTrie, optionalFieldsTrie, recordConflictMap) =
             representationName: RepresentationType$X,
             namespace: keyPrefix,
             version: VERSION$2g,
+            ingestionTimestamp: timestamp,
         });
         return createLink$1(key);
     };
@@ -8394,17 +8404,19 @@ function onResourceError(luvio, config, key, err) {
 function buildNetworkSnapshot$18(luvio, config, serverRequestCount = 0, options) {
     const { request, key, allTrackedFields, resourceParams } = prepareRequest$6(luvio, config);
     return luvio.dispatchResourceRequest(request, options).then((response) => {
+        // W-11964675 - Condition to dedupe a very specific set of requests for
+        // Komaci - a batch record request with a single record followed by a single
+        // record request. The fulfill logic sends the same network response, so
+        // there is some TypeScript massaging to extract the RecordRepresentation
+        //
+        // W-14381091 - Ensure hoisting the response body happens prior to
+        // calling `luvio.handleSuccessResponse`, since both arguments capture
+        // the response.
+        if (isSingleBatchRecordResponse(response.body)) {
+            response.body = response.body.results[0]
+                .result;
+        }
         return luvio.handleSuccessResponse(() => {
-            // W-11964675 - Condition to dedupe a very specific set of requests for
-            // Komaci - a batch record request with a single record followed by a single
-            // record request. The fulfill logic sends the same network response, so
-            // there is some TypeScript massaging to extract the RecordRepresentation
-            if (isSingleBatchRecordResponse(response.body)) {
-                let recordResponse = response;
-                recordResponse.body = response.body.results[0]
-                    .result;
-                return onResourceSuccess(luvio, config, key, allTrackedFields, recordResponse, serverRequestCount + 1);
-            }
             return onResourceSuccess(luvio, config, key, allTrackedFields, response, serverRequestCount + 1);
         }, () => {
             const cache = new StoreKeyMap();

package/dist/adsBridge.js

@@ -372,4 +372,4 @@ function withAdsBridge(callback) {
 }
 
 export { instrument, withAdsBridge };
-// version: 1.266.0-dev2-7c2f3615f
+// version: 1.266.0-dev21-b2a247476

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lds-ads-bridge",
-    "version": "1.266.0-dev2",
+    "version": "1.266.0-dev21",
     "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": "^1.266.0-dev2",
-        "@salesforce/lds-uiapi-record-utils": "^1.266.0-dev2"
+        "@salesforce/lds-adapters-uiapi": "^1.266.0-dev21",
+        "@salesforce/lds-uiapi-record-utils": "^1.266.0-dev21"
     }
 }