@salesforce/lwc-adapters-uiapi 1.309.0-dev15 → 1.309.0-dev17

package/dist/main.js

@@ -572,7 +572,7 @@ function createResourceParamsImpl(config, configMetadata) {
     }
     return resourceParams;
 }
-// engine version: 0.156.4-dev1-8df86bf0
+// engine version: 0.156.4-dev2-78889b7e
 
 /**
  * Returns true if the value acts like a Promise, i.e. has a "then" function,
@@ -11993,7 +11993,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');
@@ -12060,6 +12060,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);
@@ -12092,6 +12094,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) {
@@ -12542,15 +12546,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] = {
@@ -12564,7 +12602,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) ===
@@ -12581,14 +12619,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) ===
@@ -12598,7 +12636,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] = {
@@ -12637,7 +12675,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);
 }
@@ -27532,11 +27570,11 @@ function validateAdapterConfig$S(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;
@@ -61927,6 +61965,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.309.0-dev15",
+    "version": "1.309.0-dev17",
     "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.309.0-dev15"
+        "@salesforce/lds-adapters-uiapi": "^1.309.0-dev17"
     },
     "dependencies": {
-        "@luvio/lwc-luvio": "0.156.4-dev1",
-        "@salesforce/lds-default-luvio": "^1.309.0-dev15"
+        "@luvio/lwc-luvio": "0.156.4-dev2",
+        "@salesforce/lds-default-luvio": "^1.309.0-dev17"
     }
 }