@salesforce/lds-runtime-mobile 1.229.0-dev4 → 1.229.0-dev6

package/dist/main.js

@@ -5077,8 +5077,12 @@ function uuidv4() {
 
 const HTTP_HEADER_RETRY_AFTER = 'Retry-After';
 const HTTP_HEADER_IDEMPOTENCY_KEY = 'Idempotency-Key';
+const ERROR_CODE_IDEMPOTENCY_FEATURE_NOT_ENABLED = 'IDEMPOTENCY_FEATURE_NOT_ENABLED';
+const ERROR_CODE_IDEMPOTENCY_NOT_SUPPORTED = 'IDEMPOTENCY_NOT_SUPPORTED';
 const ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER = 'IDEMPOTENCY_KEY_USED_DIFFERENT_USER';
 const ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST = 'IDEMPOTENCY_CONCURRENT_REQUEST';
+const ERROR_CODE_IDEMPOTENCY_KEY_ALREADY_USED = 'IDEMPOTENCY_KEY_ALREADY_USED';
+const ERROR_CODE_IDEMPOTENCY_BACKEND_OPERATION_ERROR = 'IDEMPOTENCY_BACKEND_OPERATION_ERROR';
 /**
  * Get the retry after in milliseconds from the response headers, undefined if not specified.
  * The header could have two different format.
@@ -5108,7 +5112,9 @@ function buildLuvioOverrideForDraftAdapters(luvio, handler, extractTargetIdFromC
     const dispatchResourceRequest = async function (resourceRequest, _context) {
         const resourceRequestCopy = clone$1(resourceRequest);
         resourceRequestCopy.headers = resourceRequestCopy.headers || {};
-        resourceRequestCopy.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        if (handler.hasIdempotencySupport()) {
+            resourceRequestCopy.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        }
         // enable return extra fields for record creation and record update http call
         if (resourceRequest.basePath === '/ui-api/records' &&
             (resourceRequest.method === 'post' || resourceRequest.method === 'patch')) {
@@ -5944,6 +5950,7 @@ class AbstractResourceRequestActionHandler {
         // the luvio store redirect table, during which a new draft might be enqueued
         // which would not see a necessary mapping.
         this.ephemeralRedirects = {};
+        this.isIdempotencySupported = true;
     }
     enqueue(data) {
         return this.draftQueue.enqueue(this.handlerId, data);
@@ -5973,21 +5980,43 @@ class AbstractResourceRequestActionHandler {
                         retryDelayInMs = getRetryAfterInMs(response.headers);
                         shouldRetry = true;
                         break;
-                    case HttpStatusCode.ServerError:
+                    case HttpStatusCode.ServerError: {
                         shouldRetry = true;
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, false, ERROR_CODE_IDEMPOTENCY_BACKEND_OPERATION_ERROR)) {
+                            this.isIdempotencySupported = false;
+                            retryDelayInMs = 0;
+                            actionDataChanged = true;
+                        }
                         break;
+                    }
                     case 409 /* IdempotentWriteSpecificHttpStatusCode.Conflict */: {
-                        const errorCode = response.body[0].errorCode;
-                        if (errorCode === ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER) {
-                            updatedAction.data.headers = updatedAction.data.headers || {};
-                            updatedAction.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                        if (this.isUiApiErrors(response.body)) {
+                            const errorCode = response.body[0].errorCode;
+                            if (this.handleIdempotencyServerError(response.body, updatedAction, true, ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER)) {
+                                retryDelayInMs = 0;
+                                actionDataChanged = true;
+                            }
+                            else if (errorCode === ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST) {
+                                retryDelayInMs = getRetryAfterInMs(response.headers);
+                            }
+                            shouldRetry = true;
+                        }
+                        break;
+                    }
+                    case HttpStatusCode.BadRequest: {
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, false, ERROR_CODE_IDEMPOTENCY_FEATURE_NOT_ENABLED, ERROR_CODE_IDEMPOTENCY_NOT_SUPPORTED)) {
                             retryDelayInMs = 0;
                             actionDataChanged = true;
+                            shouldRetry = true;
                         }
-                        else if (errorCode === ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST) {
-                            retryDelayInMs = getRetryAfterInMs(response.headers);
+                        break;
+                    }
+                    case 422 /* IdempotentWriteSpecificHttpStatusCode.UnProcessableEntity */: {
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, true, ERROR_CODE_IDEMPOTENCY_KEY_ALREADY_USED)) {
+                            retryDelayInMs = 0;
+                            actionDataChanged = true;
+                            shouldRetry = true;
                         }
-                        shouldRetry = true;
                         break;
                     }
                 }
@@ -6006,6 +6035,27 @@ class AbstractResourceRequestActionHandler {
             return ProcessActionResult.NETWORK_ERROR;
         }
     }
+    // true if response is an idempotency server error. updates or deletes idempotency key if the reponse is idempotency related error. Idempotency related error is in format of UiApiError array.
+    handleIdempotencyServerError(responseBody, action, updateIdempotencyKey, ...targetErrorCodes) {
+        if (this.isUiApiErrors(responseBody)) {
+            const errorCode = responseBody[0].errorCode;
+            if (targetErrorCodes.includes(errorCode)) {
+                action.data.headers = action.data.headers || {};
+                if (updateIdempotencyKey) {
+                    action.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                }
+                else {
+                    delete action.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY];
+                }
+                return true;
+            }
+        }
+        return false;
+    }
+    // checks if the body is an array of UiApiError. Sometimes the body has `enhancedErrorType` field as an error indicator(one example is the field validation failure). In such case Action being processed updates to an Error Action.
+    isUiApiErrors(body) {
+        return body !== undefined && Array.isArray(body) && body.length > 0 && body[0].errorCode;
+    }
     async buildPendingAction(request, queue) {
         const targetId = await this.getIdFromRequest(request);
         if (targetId === undefined) {
@@ -6219,6 +6269,10 @@ class AbstractResourceRequestActionHandler {
             ...targetData,
             body: this.mergeRequestBody(targetBody, sourceBody),
         };
+        // Updates Idempotency key if target has one
+        if (targetData.headers && targetData.headers[HTTP_HEADER_IDEMPOTENCY_KEY]) {
+            merged.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        }
         // overlay metadata
         merged.metadata = { ...targetMetadata, ...sourceMetadata };
         // put status back to pending to auto upload if queue is active and targed is at the head.
@@ -6255,6 +6309,9 @@ class AbstractResourceRequestActionHandler {
     getDraftIdsFromAction(action) {
         return [action.targetId];
     }
+    hasIdempotencySupport() {
+        return this.isIdempotencySupported;
+    }
     async ingestResponses(responses, action) {
         const luvio = this.getLuvio();
         await luvio.handleSuccessResponse(() => {
@@ -10215,16 +10272,18 @@ function injectFilter(filterNode, idState, parentPath, isParentPolymorphic, obje
                         if (!isIdField) {
                             let subSelectionNodes = [];
                             let subFieldsHasId = false;
-                            filterNode.value.fields.forEach((subFieldNode) => {
-                                // Check if the filter field has the 'Id'
-                                if (isFieldAnIdField(subFieldNode.name.value, objectInfos[objectInfoName])) {
-                                    subFieldsHasId = true;
-                                    updateIDInfo(subFieldNode, idState, draftFunctions);
-                                }
-                                // try injecting the fields within predicate no matter it has relation or not.
-                                let subResults = injectFilter(subFieldNode, idState, curPath, isPolymorphicField, objectInfos, pathToObjectApiNamesMap, draftFunctions, existingFields ? existingFields[0] : undefined);
-                                subSelectionNodes = subSelectionNodes.concat(subResults);
-                            });
+                            if (isSpanning) {
+                                filterNode.value.fields.forEach((subFieldNode) => {
+                                    // Check if the filter field has the 'Id'
+                                    if (isFieldAnIdField(subFieldNode.name.value, objectInfos[objectInfoName])) {
+                                        subFieldsHasId = true;
+                                        updateIDInfo(subFieldNode, idState, draftFunctions);
+                                    }
+                                    // try injecting the fields within predicate no matter it has relation or not.
+                                    let subResults = injectFilter(subFieldNode, idState, curPath, isPolymorphicField, objectInfos, pathToObjectApiNamesMap, draftFunctions, existingFields ? existingFields[0] : undefined);
+                                    subSelectionNodes = subSelectionNodes.concat(subResults);
+                                });
+                            }
                             if (!subFieldsHasId) {
                                 // Check if the query field has the 'Id'
                                 const existingIdsInQuery = existingFields &&
@@ -10253,6 +10312,7 @@ function injectFilter(filterNode, idState, parentPath, isParentPolymorphic, obje
                             }
                             //Inject Conditions: 1. Same field does not exist 2. Same fields has different children. 3. Filter spanning field does not have Id. 4. InLineFragment does not have the '__typename' field
                             if (!existingFields ||
+                                existingFields.length === 0 ||
                                 subSelectionNodes.length > 0 ||
                                 (isSpanning && !subFieldsHasId) ||
                                 (isInlineFragment && !isTypeNameExisting)) {
@@ -12955,7 +13015,7 @@ function draftAwareGraphQLAdapterFactory(userId, objectInfoService, store, luvio
         // Fulfilled snapshot (this only happens in this code path if
         // the error is network error or 504), otherwise we spread over
         // the non-eval'ed snapshot (which will be either Fulfilled or Stale)
-        return nonEvaluatedSnapshot.state === 'Error'
+        const resultSnapshot = nonEvaluatedSnapshot.state === 'Error'
             ? createLocalEvalSnapshot(gqlResult, seenRecords, recordId, rebuildWithLocalEval)
             : {
                 ...nonEvaluatedSnapshot,
@@ -12964,6 +13024,22 @@ function draftAwareGraphQLAdapterFactory(userId, objectInfoService, store, luvio
                 seenRecords,
                 rebuildWithLocalEval,
             };
+        const { refresh, state } = resultSnapshot;
+        if (state !== 'Error' && refresh) {
+            // after refreshing a graphql snapshot, we want to force a rebuild regardless
+            // of if the call failed or not or if the data changed or not because we want
+            // to make sure any potential new drafts are picked up
+            resultSnapshot.refresh = {
+                ...refresh,
+                resolve: (config) => {
+                    return refresh.resolve(config).finally(() => {
+                        luvio.storePublish(resultSnapshot.recordId, undefined);
+                        luvio.storeBroadcast();
+                    });
+                },
+            };
+        }
+        return resultSnapshot;
     };
 }
 
@@ -16845,4 +16921,4 @@ register({
 });
 
 export { getRuntime, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.229.0-dev4-07d38781a
+// version: 1.229.0-dev6-53f3d6911

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lds-runtime-mobile",
-    "version": "1.229.0-dev4",
+    "version": "1.229.0-dev6",
     "license": "SEE LICENSE IN LICENSE.txt",
     "description": "LDS runtime for mobile/hybrid environments.",
     "main": "dist/main.js",
@@ -32,25 +32,25 @@
         "release:corejar": "yarn build && ../core-build/scripts/core.js --name=lds-runtime-mobile"
     },
     "dependencies": {
-        "@salesforce/lds-adapters-uiapi": "1.229.0-dev4",
-        "@salesforce/lds-bindings": "1.229.0-dev4",
-        "@salesforce/lds-instrumentation": "1.229.0-dev4",
-        "@salesforce/lds-priming": "1.229.0-dev4",
+        "@salesforce/lds-adapters-uiapi": "1.229.0-dev6",
+        "@salesforce/lds-bindings": "1.229.0-dev6",
+        "@salesforce/lds-instrumentation": "1.229.0-dev6",
+        "@salesforce/lds-priming": "1.229.0-dev6",
         "@salesforce/user": "0.0.21",
         "o11y": "244.0.0"
     },
     "devDependencies": {
-        "@salesforce/lds-adapters-graphql": "1.229.0-dev4",
-        "@salesforce/lds-drafts": "1.229.0-dev4",
-        "@salesforce/lds-drafts-adapters-uiapi": "1.229.0-dev4",
-        "@salesforce/lds-graphql-eval": "1.229.0-dev4",
-        "@salesforce/lds-network-adapter": "1.229.0-dev4",
-        "@salesforce/lds-network-nimbus": "1.229.0-dev4",
-        "@salesforce/lds-store-binary": "1.229.0-dev4",
-        "@salesforce/lds-store-nimbus": "1.229.0-dev4",
-        "@salesforce/lds-store-sql": "1.229.0-dev4",
-        "@salesforce/lds-utils-adapters": "1.229.0-dev4",
-        "@salesforce/nimbus-plugin-lds": "1.229.0-dev4",
+        "@salesforce/lds-adapters-graphql": "1.229.0-dev6",
+        "@salesforce/lds-drafts": "1.229.0-dev6",
+        "@salesforce/lds-drafts-adapters-uiapi": "1.229.0-dev6",
+        "@salesforce/lds-graphql-eval": "1.229.0-dev6",
+        "@salesforce/lds-network-adapter": "1.229.0-dev6",
+        "@salesforce/lds-network-nimbus": "1.229.0-dev6",
+        "@salesforce/lds-store-binary": "1.229.0-dev6",
+        "@salesforce/lds-store-nimbus": "1.229.0-dev6",
+        "@salesforce/lds-store-sql": "1.229.0-dev6",
+        "@salesforce/lds-utils-adapters": "1.229.0-dev6",
+        "@salesforce/nimbus-plugin-lds": "1.229.0-dev6",
         "babel-plugin-dynamic-import-node": "^2.3.3",
         "wait-for-expect": "^3.0.2"
     },

package/sfdc/main.js

@@ -5077,8 +5077,12 @@ function uuidv4() {
 
 const HTTP_HEADER_RETRY_AFTER = 'Retry-After';
 const HTTP_HEADER_IDEMPOTENCY_KEY = 'Idempotency-Key';
+const ERROR_CODE_IDEMPOTENCY_FEATURE_NOT_ENABLED = 'IDEMPOTENCY_FEATURE_NOT_ENABLED';
+const ERROR_CODE_IDEMPOTENCY_NOT_SUPPORTED = 'IDEMPOTENCY_NOT_SUPPORTED';
 const ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER = 'IDEMPOTENCY_KEY_USED_DIFFERENT_USER';
 const ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST = 'IDEMPOTENCY_CONCURRENT_REQUEST';
+const ERROR_CODE_IDEMPOTENCY_KEY_ALREADY_USED = 'IDEMPOTENCY_KEY_ALREADY_USED';
+const ERROR_CODE_IDEMPOTENCY_BACKEND_OPERATION_ERROR = 'IDEMPOTENCY_BACKEND_OPERATION_ERROR';
 /**
  * Get the retry after in milliseconds from the response headers, undefined if not specified.
  * The header could have two different format.
@@ -5108,7 +5112,9 @@ function buildLuvioOverrideForDraftAdapters(luvio, handler, extractTargetIdFromC
     const dispatchResourceRequest = async function (resourceRequest, _context) {
         const resourceRequestCopy = clone$1(resourceRequest);
         resourceRequestCopy.headers = resourceRequestCopy.headers || {};
-        resourceRequestCopy.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        if (handler.hasIdempotencySupport()) {
+            resourceRequestCopy.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        }
         // enable return extra fields for record creation and record update http call
         if (resourceRequest.basePath === '/ui-api/records' &&
             (resourceRequest.method === 'post' || resourceRequest.method === 'patch')) {
@@ -5944,6 +5950,7 @@ class AbstractResourceRequestActionHandler {
         // the luvio store redirect table, during which a new draft might be enqueued
         // which would not see a necessary mapping.
         this.ephemeralRedirects = {};
+        this.isIdempotencySupported = true;
     }
     enqueue(data) {
         return this.draftQueue.enqueue(this.handlerId, data);
@@ -5973,21 +5980,43 @@ class AbstractResourceRequestActionHandler {
                         retryDelayInMs = getRetryAfterInMs(response.headers);
                         shouldRetry = true;
                         break;
-                    case HttpStatusCode.ServerError:
+                    case HttpStatusCode.ServerError: {
                         shouldRetry = true;
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, false, ERROR_CODE_IDEMPOTENCY_BACKEND_OPERATION_ERROR)) {
+                            this.isIdempotencySupported = false;
+                            retryDelayInMs = 0;
+                            actionDataChanged = true;
+                        }
                         break;
+                    }
                     case 409 /* IdempotentWriteSpecificHttpStatusCode.Conflict */: {
-                        const errorCode = response.body[0].errorCode;
-                        if (errorCode === ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER) {
-                            updatedAction.data.headers = updatedAction.data.headers || {};
-                            updatedAction.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                        if (this.isUiApiErrors(response.body)) {
+                            const errorCode = response.body[0].errorCode;
+                            if (this.handleIdempotencyServerError(response.body, updatedAction, true, ERROR_CODE_IDEMPOTENCY_KEY_USED_DIFFERENT_USER)) {
+                                retryDelayInMs = 0;
+                                actionDataChanged = true;
+                            }
+                            else if (errorCode === ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST) {
+                                retryDelayInMs = getRetryAfterInMs(response.headers);
+                            }
+                            shouldRetry = true;
+                        }
+                        break;
+                    }
+                    case HttpStatusCode.BadRequest: {
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, false, ERROR_CODE_IDEMPOTENCY_FEATURE_NOT_ENABLED, ERROR_CODE_IDEMPOTENCY_NOT_SUPPORTED)) {
                             retryDelayInMs = 0;
                             actionDataChanged = true;
+                            shouldRetry = true;
                         }
-                        else if (errorCode === ERROR_CODE_IDEMPOTENCY_CONCURRENT_REQUEST) {
-                            retryDelayInMs = getRetryAfterInMs(response.headers);
+                        break;
+                    }
+                    case 422 /* IdempotentWriteSpecificHttpStatusCode.UnProcessableEntity */: {
+                        if (this.handleIdempotencyServerError(response.body, updatedAction, true, ERROR_CODE_IDEMPOTENCY_KEY_ALREADY_USED)) {
+                            retryDelayInMs = 0;
+                            actionDataChanged = true;
+                            shouldRetry = true;
                         }
-                        shouldRetry = true;
                         break;
                     }
                 }
@@ -6006,6 +6035,27 @@ class AbstractResourceRequestActionHandler {
             return ProcessActionResult.NETWORK_ERROR;
         }
     }
+    // true if response is an idempotency server error. updates or deletes idempotency key if the reponse is idempotency related error. Idempotency related error is in format of UiApiError array.
+    handleIdempotencyServerError(responseBody, action, updateIdempotencyKey, ...targetErrorCodes) {
+        if (this.isUiApiErrors(responseBody)) {
+            const errorCode = responseBody[0].errorCode;
+            if (targetErrorCodes.includes(errorCode)) {
+                action.data.headers = action.data.headers || {};
+                if (updateIdempotencyKey) {
+                    action.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                }
+                else {
+                    delete action.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY];
+                }
+                return true;
+            }
+        }
+        return false;
+    }
+    // checks if the body is an array of UiApiError. Sometimes the body has `enhancedErrorType` field as an error indicator(one example is the field validation failure). In such case Action being processed updates to an Error Action.
+    isUiApiErrors(body) {
+        return body !== undefined && Array.isArray(body) && body.length > 0 && body[0].errorCode;
+    }
     async buildPendingAction(request, queue) {
         const targetId = await this.getIdFromRequest(request);
         if (targetId === undefined) {
@@ -6219,6 +6269,10 @@ class AbstractResourceRequestActionHandler {
             ...targetData,
             body: this.mergeRequestBody(targetBody, sourceBody),
         };
+        // Updates Idempotency key if target has one
+        if (targetData.headers && targetData.headers[HTTP_HEADER_IDEMPOTENCY_KEY]) {
+            merged.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+        }
         // overlay metadata
         merged.metadata = { ...targetMetadata, ...sourceMetadata };
         // put status back to pending to auto upload if queue is active and targed is at the head.
@@ -6255,6 +6309,9 @@ class AbstractResourceRequestActionHandler {
     getDraftIdsFromAction(action) {
         return [action.targetId];
     }
+    hasIdempotencySupport() {
+        return this.isIdempotencySupported;
+    }
     async ingestResponses(responses, action) {
         const luvio = this.getLuvio();
         await luvio.handleSuccessResponse(() => {
@@ -10215,16 +10272,18 @@ function injectFilter(filterNode, idState, parentPath, isParentPolymorphic, obje
                         if (!isIdField) {
                             let subSelectionNodes = [];
                             let subFieldsHasId = false;
-                            filterNode.value.fields.forEach((subFieldNode) => {
-                                // Check if the filter field has the 'Id'
-                                if (isFieldAnIdField(subFieldNode.name.value, objectInfos[objectInfoName])) {
-                                    subFieldsHasId = true;
-                                    updateIDInfo(subFieldNode, idState, draftFunctions);
-                                }
-                                // try injecting the fields within predicate no matter it has relation or not.
-                                let subResults = injectFilter(subFieldNode, idState, curPath, isPolymorphicField, objectInfos, pathToObjectApiNamesMap, draftFunctions, existingFields ? existingFields[0] : undefined);
-                                subSelectionNodes = subSelectionNodes.concat(subResults);
-                            });
+                            if (isSpanning) {
+                                filterNode.value.fields.forEach((subFieldNode) => {
+                                    // Check if the filter field has the 'Id'
+                                    if (isFieldAnIdField(subFieldNode.name.value, objectInfos[objectInfoName])) {
+                                        subFieldsHasId = true;
+                                        updateIDInfo(subFieldNode, idState, draftFunctions);
+                                    }
+                                    // try injecting the fields within predicate no matter it has relation or not.
+                                    let subResults = injectFilter(subFieldNode, idState, curPath, isPolymorphicField, objectInfos, pathToObjectApiNamesMap, draftFunctions, existingFields ? existingFields[0] : undefined);
+                                    subSelectionNodes = subSelectionNodes.concat(subResults);
+                                });
+                            }
                             if (!subFieldsHasId) {
                                 // Check if the query field has the 'Id'
                                 const existingIdsInQuery = existingFields &&
@@ -10253,6 +10312,7 @@ function injectFilter(filterNode, idState, parentPath, isParentPolymorphic, obje
                             }
                             //Inject Conditions: 1. Same field does not exist 2. Same fields has different children. 3. Filter spanning field does not have Id. 4. InLineFragment does not have the '__typename' field
                             if (!existingFields ||
+                                existingFields.length === 0 ||
                                 subSelectionNodes.length > 0 ||
                                 (isSpanning && !subFieldsHasId) ||
                                 (isInlineFragment && !isTypeNameExisting)) {
@@ -12955,7 +13015,7 @@ function draftAwareGraphQLAdapterFactory(userId, objectInfoService, store, luvio
         // Fulfilled snapshot (this only happens in this code path if
         // the error is network error or 504), otherwise we spread over
         // the non-eval'ed snapshot (which will be either Fulfilled or Stale)
-        return nonEvaluatedSnapshot.state === 'Error'
+        const resultSnapshot = nonEvaluatedSnapshot.state === 'Error'
             ? createLocalEvalSnapshot(gqlResult, seenRecords, recordId, rebuildWithLocalEval)
             : {
                 ...nonEvaluatedSnapshot,
@@ -12964,6 +13024,22 @@ function draftAwareGraphQLAdapterFactory(userId, objectInfoService, store, luvio
                 seenRecords,
                 rebuildWithLocalEval,
             };
+        const { refresh, state } = resultSnapshot;
+        if (state !== 'Error' && refresh) {
+            // after refreshing a graphql snapshot, we want to force a rebuild regardless
+            // of if the call failed or not or if the data changed or not because we want
+            // to make sure any potential new drafts are picked up
+            resultSnapshot.refresh = {
+                ...refresh,
+                resolve: (config) => {
+                    return refresh.resolve(config).finally(() => {
+                        luvio.storePublish(resultSnapshot.recordId, undefined);
+                        luvio.storeBroadcast();
+                    });
+                },
+            };
+        }
+        return resultSnapshot;
     };
 }
 
@@ -16845,4 +16921,4 @@ register({
 });
 
 export { getRuntime, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.229.0-dev4-07d38781a
+// version: 1.229.0-dev6-53f3d6911