@salesforce/lds-runtime-mobile 1.451.0-dev3 → 1.451.0-dev5

package/dist/main.js

@@ -42663,19 +42663,32 @@ class AbstractResourceRequestActionHandler {
                     actionDataChanged = true;
                 }
                 else if (!this.isServerGeneratedError(response.body)) {
-                    // Killswitch CLOSED (default) — only an edge/gateway-injected 400 (no
+                    // Killswitch CLOSED (default) — an edge/gateway-injected 400 (no
                     // recognizable server error body) is retried with the SAME key, so the
                     // server can dedupe a write that may already have committed at the origin.
-                    // A real, server-generated 400 (UiApiError[] or an enhancedErrorType body,
-                    // e.g. a field-validation failure) falls through to the error path below:
-                    // retrying it unchanged would only be rejected again, so the draft is
-                    // errored for the user to fix or discard — the same terminal outcome as
-                    // before this fix, minus the duplicating key rotation. The key is only ever
-                    // rotated by the recognized idempotency error codes above, by a backdating
-                    // collision (new body), or when the user edits an errored draft
-                    // (mergeActions/handleReplaceAction).
+                    // Rotating here is what duplicated the WOLI records (@W-23445154), so the
+                    // key is preserved.
                     shouldRetry = true;
                 }
+                else {
+                    // Killswitch CLOSED — a REAL, server-generated 400 (UiApiError[] or an
+                    // enhancedErrorType body, e.g. a field-validation failure). The origin
+                    // understood the request and rejected it, so it definitively did NOT
+                    // commit — there is no already-written record to duplicate, which is what
+                    // makes rotation safe here (unlike the edge-injected case above).
+                    //
+                    // We still do NOT retry (shouldRetry stays false): the request as-is would
+                    // just be rejected again and wedge the queue. The draft errors for the user
+                    // to fix or discard. But we DO rotate the key: the server's idempotency
+                    // layer caches the 400 against the key, so if the same request is later
+                    // resubmitted unchanged after the server-side validation rule changes to
+                    // allow it, reusing the key would replay that cached 400 and the now-valid
+                    // write would never be applied. Rotating gives the retried request a fresh
+                    // key so the origin re-evaluates it. @W-23445154
+                    updatedAction.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                    reportIdempotencyKeyRotated(updatedAction.targetId, 'server-validation-400');
+                    actionDataChanged = true;
+                }
             }
             if (response.status === HttpStatusCode$1.BadRequest &&
                 this.isBackdatingError(response.body, action)) {
@@ -59137,7 +59150,7 @@ function buildServiceDescriptor$b(luvio) {
         },
     };
 }
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020
 
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
@@ -59163,7 +59176,7 @@ function buildServiceDescriptor$a(notifyRecordUpdateAvailable, getNormalizedLuvi
         },
     };
 }
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020
 
 function findExecutableOperation(input) {
   const operations = input.query.definitions.filter(
@@ -61875,4 +61888,4 @@ register({
 });
 
 export { O11Y_NAMESPACE_LDS_MOBILE, getRuntime, ingest$1o as ingestDenormalizedRecordRepresentation, initializeOneStore, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020

package/dist/types/instrumentation/metrics.d.ts

@@ -24,7 +24,7 @@ export declare function reportDraftActionEvent(state: 'added' | 'uploading' | 'c
  * the action's targetId/tag or any request body/field values.
  */
 export declare function reportDraftActionUploadError(error: unknown, handlerId: string, attempt: number): void;
-export type IdempotencyKeyRotationReason = 'idempotency-error' | 'backdating-collision' | 'merge-actions' | 'generic-400-killswitch';
+export type IdempotencyKeyRotationReason = 'idempotency-error' | 'backdating-collision' | 'merge-actions' | 'server-validation-400' | 'generic-400-killswitch';
 /**
  * Emits a single greppable Splunk log line (and an o11y counter) every time a draft's
  * Idempotency-Key is rotated — i.e. replaced with a fresh uuid rather than reused on

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lds-runtime-mobile",
-    "version": "1.451.0-dev3",
+    "version": "1.451.0-dev5",
     "license": "SEE LICENSE IN LICENSE.txt",
     "description": "LDS runtime for mobile/hybrid environments.",
     "main": "dist/main.js",
@@ -35,11 +35,11 @@
         "@conduit-client/service-bindings-imperative": "3.26.1-dev3",
         "@conduit-client/service-bindings-lwc": "3.26.1-dev3",
         "@conduit-client/service-provisioner": "3.26.1-dev3",
-        "@salesforce/lds-adapters-uiapi": "^1.451.0-dev3",
-        "@salesforce/lds-bindings": "^1.451.0-dev3",
-        "@salesforce/lds-instrumentation": "^1.451.0-dev3",
-        "@salesforce/lds-luvio-service": "^1.451.0-dev3",
-        "@salesforce/lds-luvio-uiapi-records-service": "^1.451.0-dev3",
+        "@salesforce/lds-adapters-uiapi": "^1.451.0-dev5",
+        "@salesforce/lds-bindings": "^1.451.0-dev5",
+        "@salesforce/lds-instrumentation": "^1.451.0-dev5",
+        "@salesforce/lds-luvio-service": "^1.451.0-dev5",
+        "@salesforce/lds-luvio-uiapi-records-service": "^1.451.0-dev5",
         "@salesforce/user": "0.0.21",
         "o11y": "250.7.0",
         "o11y_schema": "256.126.0"
@@ -59,16 +59,16 @@
         "@conduit-client/service-pubsub": "3.26.1-dev3",
         "@conduit-client/service-store": "3.26.1-dev3",
         "@conduit-client/utils": "3.26.1-dev3",
-        "@salesforce/lds-adapters-graphql": "^1.451.0-dev3",
-        "@salesforce/lds-drafts": "^1.451.0-dev3",
-        "@salesforce/lds-durable-records": "^1.451.0-dev3",
-        "@salesforce/lds-network-adapter": "^1.451.0-dev3",
-        "@salesforce/lds-network-nimbus": "^1.451.0-dev3",
-        "@salesforce/lds-store-binary": "^1.451.0-dev3",
-        "@salesforce/lds-store-nimbus": "^1.451.0-dev3",
-        "@salesforce/lds-store-sql": "^1.451.0-dev3",
-        "@salesforce/lds-utils-adapters": "^1.451.0-dev3",
-        "@salesforce/nimbus-plugin-lds": "^1.451.0-dev3",
+        "@salesforce/lds-adapters-graphql": "^1.451.0-dev5",
+        "@salesforce/lds-drafts": "^1.451.0-dev5",
+        "@salesforce/lds-durable-records": "^1.451.0-dev5",
+        "@salesforce/lds-network-adapter": "^1.451.0-dev5",
+        "@salesforce/lds-network-nimbus": "^1.451.0-dev5",
+        "@salesforce/lds-store-binary": "^1.451.0-dev5",
+        "@salesforce/lds-store-nimbus": "^1.451.0-dev5",
+        "@salesforce/lds-store-sql": "^1.451.0-dev5",
+        "@salesforce/lds-utils-adapters": "^1.451.0-dev5",
+        "@salesforce/nimbus-plugin-lds": "^1.451.0-dev5",
         "babel-plugin-dynamic-import-node": "^2.3.3",
         "wait-for-expect": "^3.0.2"
     },

package/sfdc/main.js

@@ -42663,19 +42663,32 @@ class AbstractResourceRequestActionHandler {
                     actionDataChanged = true;
                 }
                 else if (!this.isServerGeneratedError(response.body)) {
-                    // Killswitch CLOSED (default) — only an edge/gateway-injected 400 (no
+                    // Killswitch CLOSED (default) — an edge/gateway-injected 400 (no
                     // recognizable server error body) is retried with the SAME key, so the
                     // server can dedupe a write that may already have committed at the origin.
-                    // A real, server-generated 400 (UiApiError[] or an enhancedErrorType body,
-                    // e.g. a field-validation failure) falls through to the error path below:
-                    // retrying it unchanged would only be rejected again, so the draft is
-                    // errored for the user to fix or discard — the same terminal outcome as
-                    // before this fix, minus the duplicating key rotation. The key is only ever
-                    // rotated by the recognized idempotency error codes above, by a backdating
-                    // collision (new body), or when the user edits an errored draft
-                    // (mergeActions/handleReplaceAction).
+                    // Rotating here is what duplicated the WOLI records (@W-23445154), so the
+                    // key is preserved.
                     shouldRetry = true;
                 }
+                else {
+                    // Killswitch CLOSED — a REAL, server-generated 400 (UiApiError[] or an
+                    // enhancedErrorType body, e.g. a field-validation failure). The origin
+                    // understood the request and rejected it, so it definitively did NOT
+                    // commit — there is no already-written record to duplicate, which is what
+                    // makes rotation safe here (unlike the edge-injected case above).
+                    //
+                    // We still do NOT retry (shouldRetry stays false): the request as-is would
+                    // just be rejected again and wedge the queue. The draft errors for the user
+                    // to fix or discard. But we DO rotate the key: the server's idempotency
+                    // layer caches the 400 against the key, so if the same request is later
+                    // resubmitted unchanged after the server-side validation rule changes to
+                    // allow it, reusing the key would replay that cached 400 and the now-valid
+                    // write would never be applied. Rotating gives the retried request a fresh
+                    // key so the origin re-evaluates it. @W-23445154
+                    updatedAction.data.headers[HTTP_HEADER_IDEMPOTENCY_KEY] = uuidv4();
+                    reportIdempotencyKeyRotated(updatedAction.targetId, 'server-validation-400');
+                    actionDataChanged = true;
+                }
             }
             if (response.status === HttpStatusCode$1.BadRequest &&
                 this.isBackdatingError(response.body, action)) {
@@ -59137,7 +59150,7 @@ function buildServiceDescriptor$b(luvio) {
         },
     };
 }
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020
 
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
@@ -59163,7 +59176,7 @@ function buildServiceDescriptor$a(notifyRecordUpdateAvailable, getNormalizedLuvi
         },
     };
 }
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020
 
 function findExecutableOperation(input) {
   const operations = input.query.definitions.filter(
@@ -61875,4 +61888,4 @@ register({
 });
 
 export { O11Y_NAMESPACE_LDS_MOBILE, getRuntime, ingest$1o as ingestDenormalizedRecordRepresentation, initializeOneStore, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.451.0-dev3-debcd48191
+// version: 1.451.0-dev5-24c4844020

package/sfdc/types/instrumentation/metrics.d.ts

@@ -24,7 +24,7 @@ export declare function reportDraftActionEvent(state: 'added' | 'uploading' | 'c
  * the action's targetId/tag or any request body/field values.
  */
 export declare function reportDraftActionUploadError(error: unknown, handlerId: string, attempt: number): void;
-export type IdempotencyKeyRotationReason = 'idempotency-error' | 'backdating-collision' | 'merge-actions' | 'generic-400-killswitch';
+export type IdempotencyKeyRotationReason = 'idempotency-error' | 'backdating-collision' | 'merge-actions' | 'server-validation-400' | 'generic-400-killswitch';
 /**
  * Emits a single greppable Splunk log line (and an o11y counter) every time a draft's
  * Idempotency-Key is rotated — i.e. replaced with a fresh uuid rather than reused on