@salesforce/lds-runtime-mobile 1.100.3 → 1.100.5

package/dist/NimbusDraftQueue.d.ts

@@ -7,7 +7,10 @@ import type { DraftAction, DraftQueue, ProcessActionResult, DraftQueueState, Dra
  * to a concrete implementation running in jscore.
  */
 export declare class NimbusDraftQueue implements DraftQueue {
-    enqueue<Response>(handlerId: string, data: unknown): Promise<PendingDraftAction<Response>>;
+    enqueue<Data, Response>(handlerId: string, data: unknown): Promise<{
+        action: PendingDraftAction<Data>;
+        data: Response;
+    }>;
     registerOnChangedListener(_listener: DraftQueueChangeListener): () => Promise<void>;
     processNextAction(): Promise<ProcessActionResult>;
     getQueueActions(): Promise<DraftAction<unknown, unknown>[]>;

package/dist/main.js

@@ -4344,6 +4344,44 @@ function makeStoreEval(preconditioner, objectInfoService, userId, contextProvide
     return wrapStartEndEvents(storeEval);
 }
 
+/**
+ * Copyright (c) 2022, Salesforce, Inc.,
+ * All rights reserved.
+ * For full license text, see the LICENSE.txt file
+ */
+
+class AsyncWorkerPool {
+    constructor(concurrency) {
+        this.queue = [];
+        this.activeWorkers = 0;
+        this.concurrency = concurrency;
+    }
+    push(workFn) {
+        return new Promise((resolve, reject) => {
+            this.queue.push(async () => {
+                return workFn().then(resolve).catch(reject);
+            });
+            this.work();
+        });
+    }
+    cancel() {
+        this.queue = [];
+        // TODO [W-12513105]: thread cancellation through to active workers
+    }
+    work() {
+        while (this.queue.length > 0 && this.activeWorkers < this.concurrency) {
+            this.activeWorkers += 1;
+            const next = this.queue.shift();
+            if (next) {
+                next().finally(() => {
+                    this.activeWorkers -= 1;
+                    this.work();
+                });
+            }
+        }
+    }
+}
+
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
  * All rights reserved.
@@ -4528,14 +4566,13 @@ const { isArray: isArray$3 } = Array;
 function buildLuvioOverrideForDraftAdapters(luvio, handler, extractTargetIdFromCacheKey, options = {}) {
     // override this to create and enqueue a new draft action, and return synthetic response
     const dispatchResourceRequest = async function (resourceRequest, _context) {
-        const action = await handler.enqueue(resourceRequest).catch((err) => {
+        const { data } = await handler.enqueue(resourceRequest).catch((err) => {
             throw transformErrorToDraftSynthesisError(err);
         });
-        const record = await handler.getDataForAction(action);
-        if (record === undefined) {
+        if (data === undefined) {
             return Promise.reject(createDraftSynthesisErrorResponse());
         }
-        return createOkResponse(record);
+        return createOkResponse(data);
     };
     // override this to use an infinitely large ttl so the cache entry never expires
     const publishStoreMetadata = function (key, storeMetadataParams) {
@@ -4762,6 +4799,7 @@ class DurableDraftQueue {
         this.timeoutHandler = undefined;
         this.handlers = {};
         this.draftStore = draftStore;
+        this.workerPool = new AsyncWorkerPool(1);
     }
     addHandler(handler) {
         const id = handler.handlerId;
@@ -4779,10 +4817,6 @@ class DurableDraftQueue {
         const handler = customActionHandler(executor, id, this);
         return this.addHandler(handler);
     }
-    async getDataForAction(action) {
-        const handler = this.getHandler(action.handler);
-        return handler.getDataForAction(action);
-    }
     getQueueState() {
         return this.state;
     }
@@ -4861,20 +4895,26 @@ class DurableDraftQueue {
         });
     }
     async enqueue(handlerId, data) {
-        let queue = await this.getQueueActions();
-        const handler = this.getHandler(handlerId);
-        const pendingAction = (await handler.buildPendingAction(data, queue));
-        await this.draftStore.writeAction(pendingAction);
-        queue = await this.getQueueActions();
-        await this.notifyChangedListeners({
-            type: DraftQueueEventType.ActionAdded,
-            action: pendingAction,
+        return this.workerPool.push(async () => {
+            let queue = await this.getQueueActions();
+            const handler = this.getHandler(handlerId);
+            const pendingAction = (await handler.buildPendingAction(data, queue));
+            await this.draftStore.writeAction(pendingAction);
+            queue = await this.getQueueActions();
+            await this.notifyChangedListeners({
+                type: DraftQueueEventType.ActionAdded,
+                action: pendingAction,
+            });
+            await handler.handleActionEnqueued(pendingAction, queue);
+            if (this.state === DraftQueueState.Started) {
+                this.processNextAction();
+            }
+            const actionData = (await handler.getDataForAction(pendingAction));
+            return {
+                action: pendingAction,
+                data: actionData,
+            };
         });
-        await handler.handleActionEnqueued(pendingAction, queue);
-        if (this.state === DraftQueueState.Started) {
-            this.processNextAction();
-        }
-        return pendingAction;
     }
     registerOnChangedListener(listener) {
         this.draftQueueChangedListeners.push(listener);
@@ -4886,27 +4926,29 @@ class DurableDraftQueue {
         };
     }
     async actionCompleted(action) {
-        const handler = this.getHandler(action.handler);
-        let queue = await this.getQueueActions();
-        const queueOperations = handler.getQueueOperationsForCompletingDrafts(queue, action);
-        const idAndKeyMappings = handler.getRedirectMappings(action);
-        const keyMappings = idAndKeyMappings === undefined
-            ? undefined
-            : idAndKeyMappings.map((m) => {
-                return { draftKey: m.draftKey, canonicalKey: m.canonicalKey };
+        return this.workerPool.push(async () => {
+            const handler = this.getHandler(action.handler);
+            let queue = await this.getQueueActions();
+            const queueOperations = handler.getQueueOperationsForCompletingDrafts(queue, action);
+            const idAndKeyMappings = handler.getRedirectMappings(action);
+            const keyMappings = idAndKeyMappings === undefined
+                ? undefined
+                : idAndKeyMappings.map((m) => {
+                    return { draftKey: m.draftKey, canonicalKey: m.canonicalKey };
+                });
+            await this.draftStore.completeAction(queueOperations, keyMappings);
+            queue = await this.getQueueActions();
+            this.retryIntervalMilliseconds = 0;
+            this.uploadingActionId = undefined;
+            await handler.handleActionCompleted(action, queueOperations, queue, values$1(this.handlers));
+            await this.notifyChangedListeners({
+                type: DraftQueueEventType.ActionCompleted,
+                action,
             });
-        await this.draftStore.completeAction(queueOperations, keyMappings);
-        queue = await this.getQueueActions();
-        this.retryIntervalMilliseconds = 0;
-        this.uploadingActionId = undefined;
-        await handler.handleActionCompleted(action, queueOperations, queue, values$1(this.handlers));
-        await this.notifyChangedListeners({
-            type: DraftQueueEventType.ActionCompleted,
-            action,
+            if (this.state === DraftQueueState.Started) {
+                this.processNextAction();
+            }
         });
-        if (this.state === DraftQueueState.Started) {
-            this.processNextAction();
-        }
     }
     async actionFailed(action, retry) {
         this.uploadingActionId = undefined;
@@ -5356,6 +5398,13 @@ class AbstractResourceRequestActionHandler {
         this.draftQueue = draftQueue;
         this.networkAdapter = networkAdapter;
         this.getLuvio = getLuvio;
+        // NOTE[W-12567340]: This property stores in-memory mappings between draft
+        // ids and canonical ids for the current session. Having a local copy of
+        // these mappings is necessary to avoid a race condition between publishing
+        // new mappings to the durable store and those mappings being loaded into
+        // the luvio store redirect table, during which a new draft might be enqueued
+        // which would not see a necessary mapping.
+        this.ephemeralRedirects = {};
     }
     enqueue(data) {
         return this.draftQueue.enqueue(this.handlerId, data);
@@ -5423,7 +5472,8 @@ class AbstractResourceRequestActionHandler {
     }
     getQueueOperationsForCompletingDrafts(queue, action) {
         const queueOperations = [];
-        if (action.data.method === 'post') {
+        const redirects = this.getRedirectMappings(action);
+        if (redirects !== undefined) {
             const { length } = queue;
             for (let i = 0; i < length; i++) {
                 const queueAction = queue[i];
@@ -5433,69 +5483,64 @@ class AbstractResourceRequestActionHandler {
                     continue;
                 }
                 if (isResourceRequestAction(queueAction)) {
-                    const redirects = this.getRedirectMappings(action);
-                    if (redirects !== undefined) {
-                        let queueOperationMutated = false;
-                        let updatedActionTag = undefined;
-                        let updatedActionTargetId = undefined;
-                        const { tag: queueActionTag, data: queueActionRequest, id: queueActionId, } = queueAction;
-                        let { basePath, body } = queueActionRequest;
-                        let stringifiedBody = stringify$4(body);
-                        // for each redirected ID/key we loop over the operation to see if it needs
-                        // to be updated
-                        for (const { draftId, draftKey, canonicalId, canonicalKey } of redirects) {
-                            if (basePath.search(draftId) >= 0 ||
-                                stringifiedBody.search(draftId) >= 0) {
-                                basePath = basePath.replace(draftId, canonicalId);
-                                stringifiedBody = stringifiedBody.replace(draftId, canonicalId);
-                                queueOperationMutated = true;
-                            }
-                            // if the action is performed on a previous draft id, we need to replace the action
-                            // with a new one at the updated canonical key
-                            if (queueActionTag === draftKey) {
-                                updatedActionTag = canonicalKey;
-                                updatedActionTargetId = canonicalId;
-                            }
+                    let queueOperationMutated = false;
+                    let updatedActionTag = undefined;
+                    let updatedActionTargetId = undefined;
+                    const { tag: queueActionTag, data: queueActionRequest, id: queueActionId, } = queueAction;
+                    let { basePath, body } = queueActionRequest;
+                    let stringifiedBody = stringify$4(body);
+                    // for each redirected ID/key we loop over the operation to see if it needs
+                    // to be updated
+                    for (const { draftId, draftKey, canonicalId, canonicalKey } of redirects) {
+                        if (basePath.search(draftId) >= 0 || stringifiedBody.search(draftId) >= 0) {
+                            basePath = basePath.replace(draftId, canonicalId);
+                            stringifiedBody = stringifiedBody.replace(draftId, canonicalId);
+                            queueOperationMutated = true;
                         }
-                        if (queueOperationMutated) {
-                            if (updatedActionTag !== undefined &&
-                                updatedActionTargetId !== undefined) {
-                                const updatedAction = {
-                                    ...queueAction,
-                                    tag: updatedActionTag,
-                                    targetId: updatedActionTargetId,
-                                    data: {
-                                        ...queueActionRequest,
-                                        basePath: basePath,
-                                        body: parse$4(stringifiedBody),
-                                    },
-                                };
-                                // item needs to be replaced with a new item at the new record key
-                                queueOperations.push({
-                                    type: QueueOperationType.Delete,
-                                    id: queueActionId,
-                                });
-                                queueOperations.push({
-                                    type: QueueOperationType.Add,
-                                    action: updatedAction,
-                                });
-                            }
-                            else {
-                                const updatedAction = {
-                                    ...queueAction,
-                                    data: {
-                                        ...queueActionRequest,
-                                        basePath: basePath,
-                                        body: parse$4(stringifiedBody),
-                                    },
-                                };
-                                // item needs to be updated
-                                queueOperations.push({
-                                    type: QueueOperationType.Update,
-                                    id: queueActionId,
-                                    action: updatedAction,
-                                });
-                            }
+                        // if the action is performed on a previous draft id, we need to replace the action
+                        // with a new one at the updated canonical key
+                        if (queueActionTag === draftKey) {
+                            updatedActionTag = canonicalKey;
+                            updatedActionTargetId = canonicalId;
+                        }
+                    }
+                    if (queueOperationMutated) {
+                        if (updatedActionTag !== undefined && updatedActionTargetId !== undefined) {
+                            const updatedAction = {
+                                ...queueAction,
+                                tag: updatedActionTag,
+                                targetId: updatedActionTargetId,
+                                data: {
+                                    ...queueActionRequest,
+                                    basePath: basePath,
+                                    body: parse$4(stringifiedBody),
+                                },
+                            };
+                            // item needs to be replaced with a new item at the new record key
+                            queueOperations.push({
+                                type: QueueOperationType.Delete,
+                                id: queueActionId,
+                            });
+                            queueOperations.push({
+                                type: QueueOperationType.Add,
+                                action: updatedAction,
+                            });
+                        }
+                        else {
+                            const updatedAction = {
+                                ...queueAction,
+                                data: {
+                                    ...queueActionRequest,
+                                    basePath: basePath,
+                                    body: parse$4(stringifiedBody),
+                                },
+                            };
+                            // item needs to be updated
+                            queueOperations.push({
+                                type: QueueOperationType.Update,
+                                id: queueActionId,
+                                action: updatedAction,
+                            });
                         }
                     }
                 }
@@ -5515,6 +5560,9 @@ class AbstractResourceRequestActionHandler {
         const body = action.response.body;
         const canonicalId = this.getIdFromResponseBody(body);
         const draftId = action.targetId;
+        if (draftId !== undefined && canonicalId !== undefined && draftId !== canonicalId) {
+            this.ephemeralRedirects[draftId] = canonicalId;
+        }
         return [
             {
                 draftId,
@@ -5809,7 +5857,9 @@ class DraftManager {
             targetId,
             tag,
         })
-            .then(this.buildDraftQueueItem);
+            .then((result) => {
+            return this.buildDraftQueueItem(result.action);
+        });
     }
     /**
      * Get the current state of each of the DraftActions in the DraftQueue
@@ -11296,12 +11346,18 @@ class UiApiActionHandler extends AbstractResourceRequestActionHandler {
                 const fieldName = fieldNames[i];
                 const fieldValue = bodyFields[fieldName];
                 if (typeof fieldValue === 'string' && this.isDraftId(fieldValue)) {
-                    const draftKey = this.buildTagForTargetId(fieldValue);
-                    const canonicalKey = this.getLuvio().storeGetCanonicalKey(draftKey);
-                    if (draftKey !== canonicalKey) {
-                        const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
-                        if (canonicalId !== undefined) {
-                            bodyFields[fieldName] = canonicalId;
+                    const canonicalId = this.ephemeralRedirects[fieldValue];
+                    if (canonicalId !== undefined) {
+                        bodyFields[fieldName] = canonicalId;
+                    }
+                    else {
+                        const draftKey = this.buildTagForTargetId(fieldValue);
+                        const canonicalKey = this.getLuvio().storeGetCanonicalKey(draftKey);
+                        if (draftKey !== canonicalKey) {
+                            const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
+                            if (canonicalId !== undefined) {
+                                bodyFields[fieldName] = canonicalId;
+                            }
                         }
                     }
                 }
@@ -11309,17 +11365,24 @@ class UiApiActionHandler extends AbstractResourceRequestActionHandler {
         }
         if (request.method === 'patch' || request.method === 'delete') {
             const recordId = request.urlParams['recordId'];
-            const recordKey = this.buildTagForTargetId(recordId);
-            const canonicalKey = this.getLuvio().storeGetCanonicalKey(recordKey);
-            if (recordKey !== canonicalKey) {
-                const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
-                if (canonicalId === undefined) {
-                    // could not resolve id from request -- return the request un-modified
-                    return request;
-                }
+            const canonicalId = this.ephemeralRedirects[recordId];
+            if (canonicalId !== undefined) {
                 resolvedBasePath = resolvedBasePath.replace(recordId, canonicalId);
                 resolvedUrlParams = { ...resolvedUrlParams, recordId: canonicalId };
             }
+            else {
+                const recordKey = this.buildTagForTargetId(recordId);
+                const canonicalKey = this.getLuvio().storeGetCanonicalKey(recordKey);
+                if (recordKey !== canonicalKey) {
+                    const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
+                    if (canonicalId === undefined) {
+                        // could not resolve id from request -- return the request un-modified
+                        return request;
+                    }
+                    resolvedBasePath = resolvedBasePath.replace(recordId, canonicalId);
+                    resolvedUrlParams = { ...resolvedUrlParams, recordId: canonicalId };
+                }
+            }
         }
         return {
             ...request,
@@ -11655,14 +11718,13 @@ function performQuickActionDraftEnvironment(luvio, env, handler) {
             // only override requests to createRecord endpoint
             return env.dispatchResourceRequest(request, context, eventObservers);
         }
-        const action = await handler.enqueue(request).catch((err) => {
+        const { data } = await handler.enqueue(request).catch((err) => {
             throw createDraftSynthesisErrorResponse(err.message);
         });
-        const record = await handler.getDataForAction(action);
-        if (record === undefined) {
+        if (data === undefined) {
             return Promise.reject(createDraftSynthesisErrorResponse());
         }
-        return createOkResponse(record);
+        return createOkResponse(data);
     };
     return create$2(env, {
         dispatchResourceRequest: { value: dispatchResourceRequest },
@@ -14667,34 +14729,6 @@ function setupInspection(luvio) {
  * For full license text, see the LICENSE.txt file
  */
 
-class AsyncWorkerPool {
-    constructor(concurrency) {
-        this.queue = [];
-        this.activeWorkers = 0;
-        this.concurrency = concurrency;
-    }
-    push(workFn) {
-        this.queue.push(workFn);
-        this.work();
-    }
-    cancel() {
-        this.queue = [];
-        // TODO [W-12513105]: thread cancellation through to active workers
-    }
-    work() {
-        while (this.queue.length > 0 && this.activeWorkers < this.concurrency) {
-            this.activeWorkers += 1;
-            const next = this.queue.shift();
-            if (next) {
-                next().finally(() => {
-                    this.activeWorkers -= 1;
-                    this.work();
-                });
-            }
-        }
-    }
-}
-
 class EventEmitter {
     constructor() {
         // @ts-ignore typescript doesn't like us setting this to an empty object for some reason
@@ -15247,4 +15281,4 @@ register({
 });
 
 export { getRuntime, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.100.3-8176c84f6
+// version: 1.100.5-bd2d67d6a

package/package.json

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

package/sfdc/NimbusDraftQueue.d.ts

@@ -7,7 +7,10 @@ import type { DraftAction, DraftQueue, ProcessActionResult, DraftQueueState, Dra
  * to a concrete implementation running in jscore.
  */
 export declare class NimbusDraftQueue implements DraftQueue {
-    enqueue<Response>(handlerId: string, data: unknown): Promise<PendingDraftAction<Response>>;
+    enqueue<Data, Response>(handlerId: string, data: unknown): Promise<{
+        action: PendingDraftAction<Data>;
+        data: Response;
+    }>;
     registerOnChangedListener(_listener: DraftQueueChangeListener): () => Promise<void>;
     processNextAction(): Promise<ProcessActionResult>;
     getQueueActions(): Promise<DraftAction<unknown, unknown>[]>;

package/sfdc/main.js

@@ -4344,6 +4344,44 @@ function makeStoreEval(preconditioner, objectInfoService, userId, contextProvide
     return wrapStartEndEvents(storeEval);
 }
 
+/**
+ * Copyright (c) 2022, Salesforce, Inc.,
+ * All rights reserved.
+ * For full license text, see the LICENSE.txt file
+ */
+
+class AsyncWorkerPool {
+    constructor(concurrency) {
+        this.queue = [];
+        this.activeWorkers = 0;
+        this.concurrency = concurrency;
+    }
+    push(workFn) {
+        return new Promise((resolve, reject) => {
+            this.queue.push(async () => {
+                return workFn().then(resolve).catch(reject);
+            });
+            this.work();
+        });
+    }
+    cancel() {
+        this.queue = [];
+        // TODO [W-12513105]: thread cancellation through to active workers
+    }
+    work() {
+        while (this.queue.length > 0 && this.activeWorkers < this.concurrency) {
+            this.activeWorkers += 1;
+            const next = this.queue.shift();
+            if (next) {
+                next().finally(() => {
+                    this.activeWorkers -= 1;
+                    this.work();
+                });
+            }
+        }
+    }
+}
+
 /**
  * Copyright (c) 2022, Salesforce, Inc.,
  * All rights reserved.
@@ -4528,14 +4566,13 @@ const { isArray: isArray$3 } = Array;
 function buildLuvioOverrideForDraftAdapters(luvio, handler, extractTargetIdFromCacheKey, options = {}) {
     // override this to create and enqueue a new draft action, and return synthetic response
     const dispatchResourceRequest = async function (resourceRequest, _context) {
-        const action = await handler.enqueue(resourceRequest).catch((err) => {
+        const { data } = await handler.enqueue(resourceRequest).catch((err) => {
             throw transformErrorToDraftSynthesisError(err);
         });
-        const record = await handler.getDataForAction(action);
-        if (record === undefined) {
+        if (data === undefined) {
             return Promise.reject(createDraftSynthesisErrorResponse());
         }
-        return createOkResponse(record);
+        return createOkResponse(data);
     };
     // override this to use an infinitely large ttl so the cache entry never expires
     const publishStoreMetadata = function (key, storeMetadataParams) {
@@ -4762,6 +4799,7 @@ class DurableDraftQueue {
         this.timeoutHandler = undefined;
         this.handlers = {};
         this.draftStore = draftStore;
+        this.workerPool = new AsyncWorkerPool(1);
     }
     addHandler(handler) {
         const id = handler.handlerId;
@@ -4779,10 +4817,6 @@ class DurableDraftQueue {
         const handler = customActionHandler(executor, id, this);
         return this.addHandler(handler);
     }
-    async getDataForAction(action) {
-        const handler = this.getHandler(action.handler);
-        return handler.getDataForAction(action);
-    }
     getQueueState() {
         return this.state;
     }
@@ -4861,20 +4895,26 @@ class DurableDraftQueue {
         });
     }
     async enqueue(handlerId, data) {
-        let queue = await this.getQueueActions();
-        const handler = this.getHandler(handlerId);
-        const pendingAction = (await handler.buildPendingAction(data, queue));
-        await this.draftStore.writeAction(pendingAction);
-        queue = await this.getQueueActions();
-        await this.notifyChangedListeners({
-            type: DraftQueueEventType.ActionAdded,
-            action: pendingAction,
+        return this.workerPool.push(async () => {
+            let queue = await this.getQueueActions();
+            const handler = this.getHandler(handlerId);
+            const pendingAction = (await handler.buildPendingAction(data, queue));
+            await this.draftStore.writeAction(pendingAction);
+            queue = await this.getQueueActions();
+            await this.notifyChangedListeners({
+                type: DraftQueueEventType.ActionAdded,
+                action: pendingAction,
+            });
+            await handler.handleActionEnqueued(pendingAction, queue);
+            if (this.state === DraftQueueState.Started) {
+                this.processNextAction();
+            }
+            const actionData = (await handler.getDataForAction(pendingAction));
+            return {
+                action: pendingAction,
+                data: actionData,
+            };
         });
-        await handler.handleActionEnqueued(pendingAction, queue);
-        if (this.state === DraftQueueState.Started) {
-            this.processNextAction();
-        }
-        return pendingAction;
     }
     registerOnChangedListener(listener) {
         this.draftQueueChangedListeners.push(listener);
@@ -4886,27 +4926,29 @@ class DurableDraftQueue {
         };
     }
     async actionCompleted(action) {
-        const handler = this.getHandler(action.handler);
-        let queue = await this.getQueueActions();
-        const queueOperations = handler.getQueueOperationsForCompletingDrafts(queue, action);
-        const idAndKeyMappings = handler.getRedirectMappings(action);
-        const keyMappings = idAndKeyMappings === undefined
-            ? undefined
-            : idAndKeyMappings.map((m) => {
-                return { draftKey: m.draftKey, canonicalKey: m.canonicalKey };
+        return this.workerPool.push(async () => {
+            const handler = this.getHandler(action.handler);
+            let queue = await this.getQueueActions();
+            const queueOperations = handler.getQueueOperationsForCompletingDrafts(queue, action);
+            const idAndKeyMappings = handler.getRedirectMappings(action);
+            const keyMappings = idAndKeyMappings === undefined
+                ? undefined
+                : idAndKeyMappings.map((m) => {
+                    return { draftKey: m.draftKey, canonicalKey: m.canonicalKey };
+                });
+            await this.draftStore.completeAction(queueOperations, keyMappings);
+            queue = await this.getQueueActions();
+            this.retryIntervalMilliseconds = 0;
+            this.uploadingActionId = undefined;
+            await handler.handleActionCompleted(action, queueOperations, queue, values$1(this.handlers));
+            await this.notifyChangedListeners({
+                type: DraftQueueEventType.ActionCompleted,
+                action,
             });
-        await this.draftStore.completeAction(queueOperations, keyMappings);
-        queue = await this.getQueueActions();
-        this.retryIntervalMilliseconds = 0;
-        this.uploadingActionId = undefined;
-        await handler.handleActionCompleted(action, queueOperations, queue, values$1(this.handlers));
-        await this.notifyChangedListeners({
-            type: DraftQueueEventType.ActionCompleted,
-            action,
+            if (this.state === DraftQueueState.Started) {
+                this.processNextAction();
+            }
         });
-        if (this.state === DraftQueueState.Started) {
-            this.processNextAction();
-        }
     }
     async actionFailed(action, retry) {
         this.uploadingActionId = undefined;
@@ -5356,6 +5398,13 @@ class AbstractResourceRequestActionHandler {
         this.draftQueue = draftQueue;
         this.networkAdapter = networkAdapter;
         this.getLuvio = getLuvio;
+        // NOTE[W-12567340]: This property stores in-memory mappings between draft
+        // ids and canonical ids for the current session. Having a local copy of
+        // these mappings is necessary to avoid a race condition between publishing
+        // new mappings to the durable store and those mappings being loaded into
+        // the luvio store redirect table, during which a new draft might be enqueued
+        // which would not see a necessary mapping.
+        this.ephemeralRedirects = {};
     }
     enqueue(data) {
         return this.draftQueue.enqueue(this.handlerId, data);
@@ -5423,7 +5472,8 @@ class AbstractResourceRequestActionHandler {
     }
     getQueueOperationsForCompletingDrafts(queue, action) {
         const queueOperations = [];
-        if (action.data.method === 'post') {
+        const redirects = this.getRedirectMappings(action);
+        if (redirects !== undefined) {
             const { length } = queue;
             for (let i = 0; i < length; i++) {
                 const queueAction = queue[i];
@@ -5433,69 +5483,64 @@ class AbstractResourceRequestActionHandler {
                     continue;
                 }
                 if (isResourceRequestAction(queueAction)) {
-                    const redirects = this.getRedirectMappings(action);
-                    if (redirects !== undefined) {
-                        let queueOperationMutated = false;
-                        let updatedActionTag = undefined;
-                        let updatedActionTargetId = undefined;
-                        const { tag: queueActionTag, data: queueActionRequest, id: queueActionId, } = queueAction;
-                        let { basePath, body } = queueActionRequest;
-                        let stringifiedBody = stringify$4(body);
-                        // for each redirected ID/key we loop over the operation to see if it needs
-                        // to be updated
-                        for (const { draftId, draftKey, canonicalId, canonicalKey } of redirects) {
-                            if (basePath.search(draftId) >= 0 ||
-                                stringifiedBody.search(draftId) >= 0) {
-                                basePath = basePath.replace(draftId, canonicalId);
-                                stringifiedBody = stringifiedBody.replace(draftId, canonicalId);
-                                queueOperationMutated = true;
-                            }
-                            // if the action is performed on a previous draft id, we need to replace the action
-                            // with a new one at the updated canonical key
-                            if (queueActionTag === draftKey) {
-                                updatedActionTag = canonicalKey;
-                                updatedActionTargetId = canonicalId;
-                            }
+                    let queueOperationMutated = false;
+                    let updatedActionTag = undefined;
+                    let updatedActionTargetId = undefined;
+                    const { tag: queueActionTag, data: queueActionRequest, id: queueActionId, } = queueAction;
+                    let { basePath, body } = queueActionRequest;
+                    let stringifiedBody = stringify$4(body);
+                    // for each redirected ID/key we loop over the operation to see if it needs
+                    // to be updated
+                    for (const { draftId, draftKey, canonicalId, canonicalKey } of redirects) {
+                        if (basePath.search(draftId) >= 0 || stringifiedBody.search(draftId) >= 0) {
+                            basePath = basePath.replace(draftId, canonicalId);
+                            stringifiedBody = stringifiedBody.replace(draftId, canonicalId);
+                            queueOperationMutated = true;
                         }
-                        if (queueOperationMutated) {
-                            if (updatedActionTag !== undefined &&
-                                updatedActionTargetId !== undefined) {
-                                const updatedAction = {
-                                    ...queueAction,
-                                    tag: updatedActionTag,
-                                    targetId: updatedActionTargetId,
-                                    data: {
-                                        ...queueActionRequest,
-                                        basePath: basePath,
-                                        body: parse$4(stringifiedBody),
-                                    },
-                                };
-                                // item needs to be replaced with a new item at the new record key
-                                queueOperations.push({
-                                    type: QueueOperationType.Delete,
-                                    id: queueActionId,
-                                });
-                                queueOperations.push({
-                                    type: QueueOperationType.Add,
-                                    action: updatedAction,
-                                });
-                            }
-                            else {
-                                const updatedAction = {
-                                    ...queueAction,
-                                    data: {
-                                        ...queueActionRequest,
-                                        basePath: basePath,
-                                        body: parse$4(stringifiedBody),
-                                    },
-                                };
-                                // item needs to be updated
-                                queueOperations.push({
-                                    type: QueueOperationType.Update,
-                                    id: queueActionId,
-                                    action: updatedAction,
-                                });
-                            }
+                        // if the action is performed on a previous draft id, we need to replace the action
+                        // with a new one at the updated canonical key
+                        if (queueActionTag === draftKey) {
+                            updatedActionTag = canonicalKey;
+                            updatedActionTargetId = canonicalId;
+                        }
+                    }
+                    if (queueOperationMutated) {
+                        if (updatedActionTag !== undefined && updatedActionTargetId !== undefined) {
+                            const updatedAction = {
+                                ...queueAction,
+                                tag: updatedActionTag,
+                                targetId: updatedActionTargetId,
+                                data: {
+                                    ...queueActionRequest,
+                                    basePath: basePath,
+                                    body: parse$4(stringifiedBody),
+                                },
+                            };
+                            // item needs to be replaced with a new item at the new record key
+                            queueOperations.push({
+                                type: QueueOperationType.Delete,
+                                id: queueActionId,
+                            });
+                            queueOperations.push({
+                                type: QueueOperationType.Add,
+                                action: updatedAction,
+                            });
+                        }
+                        else {
+                            const updatedAction = {
+                                ...queueAction,
+                                data: {
+                                    ...queueActionRequest,
+                                    basePath: basePath,
+                                    body: parse$4(stringifiedBody),
+                                },
+                            };
+                            // item needs to be updated
+                            queueOperations.push({
+                                type: QueueOperationType.Update,
+                                id: queueActionId,
+                                action: updatedAction,
+                            });
                         }
                     }
                 }
@@ -5515,6 +5560,9 @@ class AbstractResourceRequestActionHandler {
         const body = action.response.body;
         const canonicalId = this.getIdFromResponseBody(body);
         const draftId = action.targetId;
+        if (draftId !== undefined && canonicalId !== undefined && draftId !== canonicalId) {
+            this.ephemeralRedirects[draftId] = canonicalId;
+        }
         return [
             {
                 draftId,
@@ -5809,7 +5857,9 @@ class DraftManager {
             targetId,
             tag,
         })
-            .then(this.buildDraftQueueItem);
+            .then((result) => {
+            return this.buildDraftQueueItem(result.action);
+        });
     }
     /**
      * Get the current state of each of the DraftActions in the DraftQueue
@@ -11296,12 +11346,18 @@ class UiApiActionHandler extends AbstractResourceRequestActionHandler {
                 const fieldName = fieldNames[i];
                 const fieldValue = bodyFields[fieldName];
                 if (typeof fieldValue === 'string' && this.isDraftId(fieldValue)) {
-                    const draftKey = this.buildTagForTargetId(fieldValue);
-                    const canonicalKey = this.getLuvio().storeGetCanonicalKey(draftKey);
-                    if (draftKey !== canonicalKey) {
-                        const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
-                        if (canonicalId !== undefined) {
-                            bodyFields[fieldName] = canonicalId;
+                    const canonicalId = this.ephemeralRedirects[fieldValue];
+                    if (canonicalId !== undefined) {
+                        bodyFields[fieldName] = canonicalId;
+                    }
+                    else {
+                        const draftKey = this.buildTagForTargetId(fieldValue);
+                        const canonicalKey = this.getLuvio().storeGetCanonicalKey(draftKey);
+                        if (draftKey !== canonicalKey) {
+                            const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
+                            if (canonicalId !== undefined) {
+                                bodyFields[fieldName] = canonicalId;
+                            }
                         }
                     }
                 }
@@ -11309,17 +11365,24 @@ class UiApiActionHandler extends AbstractResourceRequestActionHandler {
         }
         if (request.method === 'patch' || request.method === 'delete') {
             const recordId = request.urlParams['recordId'];
-            const recordKey = this.buildTagForTargetId(recordId);
-            const canonicalKey = this.getLuvio().storeGetCanonicalKey(recordKey);
-            if (recordKey !== canonicalKey) {
-                const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
-                if (canonicalId === undefined) {
-                    // could not resolve id from request -- return the request un-modified
-                    return request;
-                }
+            const canonicalId = this.ephemeralRedirects[recordId];
+            if (canonicalId !== undefined) {
                 resolvedBasePath = resolvedBasePath.replace(recordId, canonicalId);
                 resolvedUrlParams = { ...resolvedUrlParams, recordId: canonicalId };
             }
+            else {
+                const recordKey = this.buildTagForTargetId(recordId);
+                const canonicalKey = this.getLuvio().storeGetCanonicalKey(recordKey);
+                if (recordKey !== canonicalKey) {
+                    const canonicalId = extractRecordIdFromStoreKey(canonicalKey);
+                    if (canonicalId === undefined) {
+                        // could not resolve id from request -- return the request un-modified
+                        return request;
+                    }
+                    resolvedBasePath = resolvedBasePath.replace(recordId, canonicalId);
+                    resolvedUrlParams = { ...resolvedUrlParams, recordId: canonicalId };
+                }
+            }
         }
         return {
             ...request,
@@ -11655,14 +11718,13 @@ function performQuickActionDraftEnvironment(luvio, env, handler) {
             // only override requests to createRecord endpoint
             return env.dispatchResourceRequest(request, context, eventObservers);
         }
-        const action = await handler.enqueue(request).catch((err) => {
+        const { data } = await handler.enqueue(request).catch((err) => {
             throw createDraftSynthesisErrorResponse(err.message);
         });
-        const record = await handler.getDataForAction(action);
-        if (record === undefined) {
+        if (data === undefined) {
             return Promise.reject(createDraftSynthesisErrorResponse());
         }
-        return createOkResponse(record);
+        return createOkResponse(data);
     };
     return create$2(env, {
         dispatchResourceRequest: { value: dispatchResourceRequest },
@@ -14667,34 +14729,6 @@ function setupInspection(luvio) {
  * For full license text, see the LICENSE.txt file
  */
 
-class AsyncWorkerPool {
-    constructor(concurrency) {
-        this.queue = [];
-        this.activeWorkers = 0;
-        this.concurrency = concurrency;
-    }
-    push(workFn) {
-        this.queue.push(workFn);
-        this.work();
-    }
-    cancel() {
-        this.queue = [];
-        // TODO [W-12513105]: thread cancellation through to active workers
-    }
-    work() {
-        while (this.queue.length > 0 && this.activeWorkers < this.concurrency) {
-            this.activeWorkers += 1;
-            const next = this.queue.shift();
-            if (next) {
-                next().finally(() => {
-                    this.activeWorkers -= 1;
-                    this.work();
-                });
-            }
-        }
-    }
-}
-
 class EventEmitter {
     constructor() {
         // @ts-ignore typescript doesn't like us setting this to an empty object for some reason
@@ -15247,4 +15281,4 @@ register({
 });
 
 export { getRuntime, registerReportObserver, reportGraphqlQueryParseError };
-// version: 1.100.3-8176c84f6
+// version: 1.100.5-bd2d67d6a