@salesforce/lwc-adapters-uiapi 1.320.0 → 1.322.0

package/dist/main.js

@@ -572,7 +572,7 @@ function createResourceParamsImpl(config, configMetadata) {
     }
     return resourceParams;
 }
-// engine version: 0.156.4-a259372b
+// engine version: 0.156.5-f5fd8c7a
 
 /**
  * Returns true if the value acts like a Promise, i.e. has a "then" function,
@@ -12156,7 +12156,7 @@ const RECORD_REPRESENTATION_ERROR_STORE_METADATA_PARAMS = {
     version: RECORD_REPRESENTATION_ERROR_VERSION,
 };
 function isGraphNode(node) {
-    return node !== null && node.type === 'Node';
+    return !!node && node.type === 'Node';
 }
 function addScalarFieldId(current) {
     addScalarField(current, 'Id');
@@ -12223,6 +12223,8 @@ function extractTrackedFieldsToTrie(recordId, node, root, config, visitedRecordI
                 const spanning = spanningLink.follow();
                 // W-8058425, do not include external lookups added by getTrackedFields
                 if (isExternalLookupFieldKey(spanning)) {
+                    // NOTE: the logic to get here is mimicked in RecordRepresentation/merge::mergePendingFields as of
+                    // W-11899329 due to issues with external lookups being marked pending but never fetched
                     continue;
                 }
                 extractTrackedFieldsToTrie(spanningLink.data.__ref, spanning, next, config, spanningVisitedRecordIds, depth + 1);
@@ -12255,6 +12257,8 @@ function extractTrackedFieldsToTrie(recordId, node, root, config, visitedRecordI
                     const { fields } = state;
                     // W-8058425, do not include external lookups added by getTrackedFields
                     if (includes.call(fields, 'ExternalId')) {
+                        // NOTE: the logic to get here is mimicked in RecordRepresentation/merge::mergePendingFields as of
+                        // W-11899329 due to issues with external lookups being marked pending but never fetched
                         continue;
                     }
                     for (let s = 0, len = fields.length; s < len; s += 1) {
@@ -12705,15 +12709,49 @@ function ingestRecordResponse(luvio, response, recordId, recordIngest, conflictM
 
 // This function sets fields that we are refreshing to pending
 // These values will go into the store
-function mergePendingFields(newRecord, oldRecord) {
-    // TODO [W-6900046]: avoid casting to any by updating
-    // RecordRepresentationNormalized['fields'] to include `pending:true` property
+function mergePendingFields(newRecord, oldRecord, existingNode) {
     const mergedFields = { ...newRecord.fields };
     const merged = { ...newRecord, fields: mergedFields };
     const existingFields = keys(oldRecord.fields);
     for (let i = 0, len = existingFields.length; i < len; i += 1) {
         const spanningFieldName = existingFields[i];
         if (newRecord.fields[spanningFieldName] === undefined) {
+            /*
+             * Per W-8058425 external lookups fields are excluded from the tracked fields. However, as covered in
+             * W-11899329, situations can arise in which a merge conflict occurs when the existing record has a
+             * reference to an external lookup field.  The exclusion ultimately results in a snapshot stuck in the
+             * pending state. This is an approach to prevent that situation.
+             *
+             * The same logic checks for W-8058425 to "continue" as it relates to not tracking external lookups is
+             * mimicked here as it relates to not marking them as pending.
+             */
+            // make sure external lookups are NOT marked as pending when `existingNode` is provided
+            if (isGraphNode(existingNode)) {
+                // get the node for the spanning field
+                const fieldValueRep = existingNode
+                    .object('fields')
+                    .link(spanningFieldName);
+                const field = fieldValueRep.follow();
+                if (isGraphNode(field)) {
+                    if (field.isScalar('value') === false) {
+                        const record = field
+                            .link('value')
+                            .follow();
+                        if (isExternalLookupFieldKey(record)) {
+                            continue;
+                        }
+                    }
+                }
+                else {
+                    const state = fieldValueRep.linkData();
+                    if (state !== undefined) {
+                        const { fields } = state;
+                        if (includes.call(fields, 'ExternalId')) {
+                            continue;
+                        }
+                    }
+                }
+            }
             // TODO [W-6900046]: fix above casting issue so we're not stuffing arbitrary things
             // into RecordRepresentationNormalized['fields']
             mergedFields[spanningFieldName] = {
@@ -12727,7 +12765,7 @@ function mergePendingFields(newRecord, oldRecord) {
 // This method gets called
 // when incoming record has a higher version
 // than the record that is currently in the store
-function mergeAndRefreshHigherVersionRecord(incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap) {
+function mergeAndRefreshHigherVersionRecord(incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap, existingNode) {
     // If the higher version (incoming) does not contain a superset of fields as existing
     // then we need to refresh to get updated versions of fields in existing
     if (isSuperRecordFieldTrie(incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot) ===
@@ -12744,14 +12782,14 @@ function mergeAndRefreshHigherVersionRecord(incoming, existing, incomingTrackedF
         };
         // We want to mark fields in the store as pending
         // Because we don't want to emit any data to components
-        return mergePendingFields(incoming, existing);
+        return mergePendingFields(incoming, existing, existingNode);
     }
     return incoming;
 }
 // This method gets called
 // when incoming record has a lower version
 // than the record that is currently in the store
-function mergeAndRefreshLowerVersionRecord(luvio, incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap) {
+function mergeAndRefreshLowerVersionRecord(luvio, incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap, existingNode) {
     // If the higher version (existing) does not have a superset of fields as incoming
     // then we need to refresh to get updated versions of fields on incoming
     if (isSuperRecordFieldTrie(existingTrackedFieldsTrieRoot, incomingTrackedFieldsTrieRoot) ===
@@ -12761,7 +12799,7 @@ function mergeAndRefreshLowerVersionRecord(luvio, incoming, existing, incomingTr
         if (isSupportedEntity(incoming.apiName) === false) {
             return mergeRecordFields(existing, incoming);
         }
-        const merged = mergePendingFields(existing, incoming);
+        const merged = mergePendingFields(existing, incoming, existingNode);
         // update the conflict map to resolve the record conflict in resolveConflict
         if (recordConflictMap) {
             recordConflictMap.conflicts[incoming.id] = {
@@ -12800,7 +12838,7 @@ function mergeRecordConflict(luvio, incoming, existing, recordConflictMap) {
     extractTrackedFieldsToTrie(recordKey, incomingNode, incomingTrackedFieldsTrieRoot, trackedFieldsConfig);
     extractTrackedFieldsToTrie(recordKey, existingNode, existingTrackedFieldsTrieRoot, trackedFieldsConfig);
     if (incoming.weakEtag > existing.weakEtag) {
-        return mergeAndRefreshHigherVersionRecord(incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap);
+        return mergeAndRefreshHigherVersionRecord(incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap, existingNode);
     }
     return mergeAndRefreshLowerVersionRecord(luvio, incoming, existing, incomingTrackedFieldsTrieRoot, existingTrackedFieldsTrieRoot, recordConflictMap);
 }
@@ -19362,7 +19400,7 @@ function validate$1E(obj, path = 'RecordLayoutSectionUserStateRepresentation') {
 }
 
 const TTL$B = 900000;
-const VERSION$2u = "4ba42e1fa0fb00cf78fce86082da41c9";
+const VERSION$2u = "12123f1aca1b5a48303b1d099f9f5629";
 function validate$1D(obj, path = 'RecordLayoutUserStateRepresentation') {
     const v_error = (() => {
         if (typeof obj !== 'object' || ArrayIsArray(obj) || obj === null) {
@@ -19375,8 +19413,29 @@ function validate$1D(obj, path = 'RecordLayoutUserStateRepresentation') {
         }
         const obj_id = obj.id;
         const path_id = path + '.id';
-        if (typeof obj_id !== 'string') {
-            return new TypeError('Expected "string" but received "' + typeof obj_id + '" (at "' + path_id + '")');
+        let obj_id_union0 = null;
+        const obj_id_union0_error = (() => {
+            if (typeof obj_id !== 'string') {
+                return new TypeError('Expected "string" but received "' + typeof obj_id + '" (at "' + path_id + '")');
+            }
+        })();
+        if (obj_id_union0_error != null) {
+            obj_id_union0 = obj_id_union0_error.message;
+        }
+        let obj_id_union1 = null;
+        const obj_id_union1_error = (() => {
+            if (obj_id !== null) {
+                return new TypeError('Expected "null" but received "' + typeof obj_id + '" (at "' + path_id + '")');
+            }
+        })();
+        if (obj_id_union1_error != null) {
+            obj_id_union1 = obj_id_union1_error.message;
+        }
+        if (obj_id_union0 && obj_id_union1) {
+            let message = 'Object doesn\'t match union (at "' + path_id + '")';
+            message += '\n' + obj_id_union0.split('\n').map((line) => '\t' + line).join('\n');
+            message += '\n' + obj_id_union1.split('\n').map((line) => '\t' + line).join('\n');
+            return new TypeError(message);
         }
         const obj_layoutType = obj.layoutType;
         const path_layoutType = path + '.layoutType';
@@ -27695,11 +27754,11 @@ function validateAdapterConfig$Q(untrustedConfig, _configPropertyNames) {
     if (config === null) {
         return null;
     }
-    // recordTypeId coercion is nuts: if `null` (but not undefined) then use MASTER record type id
     let recordTypeId = config.recordTypeId;
     if (recordTypeId === undefined) {
         // must check untrusted bc config has been coerced
-        if (untrustedConfig.recordTypeId !== null) {
+        if (untrustedConfig.recordTypeId !== null &&
+            untrustedConfig.recordTypeId !== undefined) {
             return null;
         }
         recordTypeId = MAIN_RECORD_TYPE_ID;
@@ -58363,6 +58422,23 @@ function getInContextFragmentType$d(fragment, fragmentMap) {
     return sharedGetFragmentType(fragment, fragmentMap);
 }
 
+function getFieldType$b(field) {
+    switch (field.name.value) {
+        case '__typename': {
+            return {
+                isArray: false,
+                typename: 'String',
+            };
+        }
+        default: {
+            return {
+                isArray: false,
+                typename: 'Setup__SetupAggregateConnection',
+            };
+        }
+    }
+}
+
 const name$2 = 'Setup__SetupQueryAggregate';
 const VERSION$e = '0592284764c8e58016880e291b9ffb64';
 function keyBuilder$g(luvio, path, data) {
@@ -58480,24 +58556,6 @@ function getTypeCacheKeys$b(cacheKeySink, astNode, state) {
     }
     // Deal with mapped types' cache keys
 }
-function getFieldType$b(field) {
-    switch (field.name.value) {
-        case '__typename': {
-            return {
-                isArray: false,
-                typename: 'String'
-            };
-        }
-        case 'recordQueryAggregate': {
-            return {
-                isArray: false,
-                typename: 'Setup__SetupAggregateConnection'
-            };
-        }
-        default:
-            return undefined;
-    }
-}
 function ingestFieldByType$2(typename, parentKey, requestedField, sink, fieldKey, fieldData, state) {
     // TODO: add validation logic to only allow nullable fields to be null in the future
     if (fieldData === null) {
@@ -60545,6 +60603,12 @@ function selectTypeLinkWithPagination(resolvedLink, sel, fieldData, reader, key,
             return;
         }
     }
+    else if (totalEdges === 0 && !listIsComplete) {
+        // empty edge list, but the list isn't complete?
+        if (process.env.NODE_ENV !== 'production') {
+            throw new Error("page size of zero, but pageInfo .hasNextPage is true.");
+        }
+    }
     else {
         if (startOffset === undefined || endOffset === undefined) {
             reader.markMissingLink(fieldData.__ref);

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lwc-adapters-uiapi",
-    "version": "1.320.0",
+    "version": "1.322.0",
     "license": "SEE LICENSE IN LICENSE.txt",
     "description": "UIAPI adapters with LWC bindings",
     "module": "dist/main.js",
@@ -31,10 +31,10 @@
         "clean": "rm -rf dist src/generated"
     },
     "devDependencies": {
-        "@salesforce/lds-adapters-uiapi": "^1.320.0"
+        "@salesforce/lds-adapters-uiapi": "^1.322.0"
     },
     "dependencies": {
-        "@luvio/lwc-luvio": "0.156.4",
-        "@salesforce/lds-default-luvio": "^1.320.0"
+        "@luvio/lwc-luvio": "0.156.5",
+        "@salesforce/lds-default-luvio": "^1.322.0"
     }
 }