@salesforce/lds-drafts 1.100.3 → 1.100.4

package/dist/DurableDraftQueue.d.ts

@@ -16,6 +16,7 @@ export declare class DurableDraftQueue implements DraftQueue {
     private replacingAction?;
     private uploadingActionId?;
     private timeoutHandler;
+    private workerPool;
     private handlers;
     private getHandler;
     constructor(draftStore: DraftStore);

package/dist/actionHandlers/AbstractResourceRequestActionHandler.d.ts

@@ -11,6 +11,9 @@ export declare abstract class AbstractResourceRequestActionHandler<ResponseType,
     protected readonly draftQueue: DraftQueue;
     protected readonly networkAdapter: NetworkAdapter;
     protected readonly getLuvio: () => Luvio;
+    ephemeralRedirects: {
+        [key: string]: string;
+    };
     constructor(draftQueue: DraftQueue, networkAdapter: NetworkAdapter, getLuvio: () => Luvio);
     enqueue(data: ResourceRequest): Promise<PendingDraftAction<ResourceRequest>>;
     handleAction(action: DraftAction<ResourceRequest, ResponseType>, actionCompleted: (action: CompletedDraftAction<ResourceRequest, ResponseType>) => Promise<void>, actionErrored: (action: DraftAction<ResourceRequest, ResponseType>, retry: boolean) => Promise<void>): Promise<ProcessActionResult>;

package/dist/ldsDrafts.js

@@ -5,6 +5,7 @@
  */
 
 import { HttpStatusCode, StoreKeyMap } from '@luvio/engine';
+import { AsyncWorkerPool } from '@salesforce/lds-utils-adapters';
 
 var DraftActionStatus;
 (function (DraftActionStatus) {
@@ -424,6 +425,7 @@ class DurableDraftQueue {
         this.timeoutHandler = undefined;
         this.handlers = {};
         this.draftStore = draftStore;
+        this.workerPool = new AsyncWorkerPool(1);
     }
     addHandler(handler) {
         const id = handler.handlerId;
@@ -523,20 +525,22 @@ 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();
+            }
+            return pendingAction;
         });
-        await handler.handleActionEnqueued(pendingAction, queue);
-        if (this.state === DraftQueueState.Started) {
-            this.processNextAction();
-        }
-        return pendingAction;
     }
     registerOnChangedListener(listener) {
         this.draftQueueChangedListeners.push(listener);
@@ -548,27 +552,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(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(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;
@@ -1018,6 +1024,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);
@@ -1085,7 +1098,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];
@@ -1095,69 +1109,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(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(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(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(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(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(stringifiedBody),
+                                },
+                            };
+                            // item needs to be updated
+                            queueOperations.push({
+                                type: QueueOperationType.Update,
+                                id: queueActionId,
+                                action: updatedAction,
+                            });
                         }
                     }
                 }
@@ -1177,6 +1186,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,

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@salesforce/lds-drafts",
-    "version": "1.100.3",
+    "version": "1.100.4",
     "license": "SEE LICENSE IN LICENSE.txt",
     "description": "LDS Drafts",
     "main": "dist/ldsDrafts.js",
@@ -25,6 +25,7 @@
     },
     "dependencies": {
         "@luvio/engine": "0.135.4",
-        "@luvio/environments": "0.135.4"
+        "@luvio/environments": "0.135.4",
+        "@salesforce/lds-utils-adapters": "^1.100.4"
     }
 }