@izara_project/izara-core-library-asynchronous-flow 1.0.33 → 1.0.35

package/package.json

@@ -3,7 +3,7 @@
   "author": "Sven Mason <thebarbariansven@gmail.com>",
   "license": "AGPL-3.0-or-later",
   "homepage": "https://bitbucket.org/izara-core-libraries/izara-core-library-asynchronous-flow#readme",
-  "version": "1.0.33",
+  "version": "1.0.35",
   "description": "Shared asynchronous flow logic",
   "type": "module",
   "main": "index.js",

package/src/asyncFlowSharedLib.js

@@ -105,6 +105,10 @@ export function createParentFlowId(prefix = '') {
  * @throws {NoRetryError} If the stripped value equals the prefix (invalid)
  */
 export function explodePendingStepId(pendingStepId, prefix = '') {
+  if (!pendingStepId) {
+    throw NoRetryError('pendingStepId is required');
+  }
+
   if (prefix === '') {
     return pendingStepId;
   } else {
@@ -178,6 +182,22 @@ export async function checkUniqueRequestProcessing(
   overwriteUniqueRequestId = null,
   uniqueRequestIdFieldName = 'UniqueRequestId' // if set will check/add this fieldname with _izContext.uniqueRequestId value
 ) {
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:checkUniqueRequestProcessing] Input: ',
+    {
+      _izContext,
+      tableName,
+      primaryKey,
+      prefix,
+      overwriteUniqueRequestId,
+      uniqueRequestIdFieldName
+    }
+  );
+
+  if (!tableName || !primaryKey) {
+    throw NoRetryError('tableName and primaryKey are required');
+  }
+
   try {
     _izContext.logger.debug(
       'current uniqueRequestId:',
@@ -298,6 +318,20 @@ export async function checkTimeCacheComplete(
   timeCacheComplete,
   prefix
 ) {
+  _izContext.logger.debug('[Lib:AsyncFlow:checkTimeCacheComplete] Input: ', {
+    _izContext,
+    fullMainTableName,
+    keyValues,
+    timeCacheComplete,
+    prefix
+  });
+
+  if (!fullMainTableName || !keyValues || !timeCacheComplete || !prefix) {
+    throw NoRetryError(
+      'fullMainTableName, keyValues, timeCacheComplete and prefix are required'
+    );
+  }
+
   let [returnValue, newTimeCacheComplete, uniqueRequestId] =
     await checkAndGetTimeCacheComplete(
       _izContext,
@@ -329,16 +363,19 @@ export async function checkAndGetTimeCacheComplete(
   timeCacheComplete,
   prefix
 ) {
-  _izContext.logger.debug('event checkTimeCacheComplete', {
-    fullMainTableName: fullMainTableName,
-    keyValues: keyValues,
-    timeCacheComplete: timeCacheComplete,
-    prefix: prefix
-  });
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:checkAndGetTimeCacheComplete] Input: ',
+    {
+      fullMainTableName: fullMainTableName,
+      keyValues: keyValues,
+      timeCacheComplete: timeCacheComplete,
+      prefix: prefix
+    }
+  );
 
-  let uniqueRequestIdFieldName = createFieldNameUniqueRequestId('cache');
-  let cacheCompleteFieldName = createFieldNameCacheComplete(prefix);
-  let statusFieldName = createFieldNameStatus(prefix);
+  const uniqueRequestIdFieldName = createFieldNameUniqueRequestId('cache');
+  const cacheCompleteFieldName = createFieldNameCacheComplete(prefix);
+  const statusFieldName = createFieldNameStatus(prefix);
 
   let getTimeCacheComplete = await dynamodbSharedLib.getItem(
     _izContext,
@@ -436,6 +473,12 @@ export function validateStartKeyParam(
   partitionKeyFieldName,
   sortKeyFieldName
 ) {
+  _izContext.logger.debug('[Lib:AsyncFlow:validateStartKeyParam] Input: ', {
+    startKey,
+    partitionKeyFieldName,
+    sortKeyFieldName
+  });
+
   const stringNotEmptyRegex = /^[A-Za-z0-9_-]+$/;
 
   if (
@@ -482,12 +525,16 @@ export async function validateMultipleInvocations(
   numberInvocation,
   queueName
 ) {
-  _izContext.logger.debug('Lib:validateMultipleInvocations', {
-    messageProperty: messageProperty,
-    passOnStartKey: passOnStartKey,
-    numberInvocation: numberInvocation,
-    queueName: queueName
-  });
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:validateMultipleInvocations] Input: ',
+    {
+      messageProperty: messageProperty,
+      passOnStartKey: passOnStartKey,
+      numberInvocation: numberInvocation,
+      queueName: queueName
+    }
+  );
+
   try {
     const maxProcessInvocationCountImport = 20;
     if (numberInvocation >= maxProcessInvocationCountImport) {
@@ -504,7 +551,7 @@ export async function validateMultipleInvocations(
     }
     messageProperty['numberInvocation'] = numberInvocation;
 
-    let messageReInvokeFunction = {
+    const messageReInvokeFunction = {
       MessageBody: JSON.stringify(messageProperty),
       QueueUrl: await sqsSharedLib.sqsQueueUrl(_izContext, queueName)
     };

package/src/awaitingMultipleSteps.js

@@ -43,16 +43,17 @@ export async function createAwaitingMultipleStepsWithAdditionalAttributes(
   pendingStepId
 ) {
   _izContext.logger.debug(
-    'Lib:createAwaitingMultipleStepsWithAdditionalAttributes',
+    '[Lib:AsyncFlow:createAwaitingMultipleStepsWithAdditionalAttributes] Input: ',
     {
-      records: records,
-      pendingStepId: pendingStepId
+      records,
+      pendingStepId
     }
   );
-  let promiseArray = [];
 
-  for (let record of records) {
-    let attributesWhenCreate = {
+  const promiseArray = [];
+
+  for (const record of records) {
+    const attributesWhenCreate = {
       awaitingStepId: record.awaitingStepId,
       pendingStepId: pendingStepId,
       timestamp: Date.now(),
@@ -101,10 +102,13 @@ export async function initAwaitingStepsWithAttrs(
   records,
   pendingStepId
 ) {
-  _izContext.logger.debug('Lib:initAwaitingStepsWithAttrs', {
-    records,
-    pendingStepId
-  });
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:initAwaitingStepsWithAttrs] Input: ',
+    {
+      records,
+      pendingStepId
+    }
+  );
 
   const timestamp = Date.now();
   const promiseArray = [];
@@ -155,22 +159,26 @@ export async function createAwaitingMultipleSteps(
   records, // array of awaitingStepIds
   pendingStepId
 ) {
-  _izContext.logger.debug('Lib:createAwaitingMultipleSteps', {
-    records: records,
-    pendingStepId: pendingStepId
-  });
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:createAwaitingMultipleSteps] Input: ',
+    {
+      records: records,
+      pendingStepId: pendingStepId
+    }
+  );
 
-  let promiseArray = [];
+  const promiseArray = [];
 
-  for (let record of records) {
-    let attributesWhenCreate = {
+  for (const record of records) {
+    const attributesWhenCreate = {
       awaitingStepId: record,
       pendingStepId: pendingStepId,
       timestamp: Date.now()
     };
-    _izContext.logger.debug('putItem:AwaitingMultipleSteps +++++++', {
-      awaitingStepId: attributesWhenCreate.awaitingStepId
-    });
+    _izContext.logger.debug(
+      '[Lib:AsyncFlow:createAwaitingMultipleSteps] attributesWhenCreate: ',
+      attributesWhenCreate
+    );
     promiseArray.push(
       dynamodbSharedLib.putItem(
         _izContext,
@@ -179,9 +187,13 @@ export async function createAwaitingMultipleSteps(
       )
     );
 
-    _izContext.logger.debug('putItem:AwaitingMultipleStepByPending', {
-      awaitingStepId: attributesWhenCreate.awaitingStepId
-    });
+    _izContext.logger.debug(
+      '[Lib:AsyncFlow:createAwaitingMultipleSteps] putItem:AwaitingMultipleStepByPending',
+      {
+        awaitingStepId: attributesWhenCreate.awaitingStepId
+      }
+    );
+
     promiseArray.push(
       dynamodbSharedLib.putItem(
         _izContext,
@@ -193,6 +205,7 @@ export async function createAwaitingMultipleSteps(
       )
     );
   }
+
   await Promise.all(promiseArray);
 }
 
@@ -229,6 +242,11 @@ export async function createNewAwaitingMultipleStep(
     errorsFound: []
   };
 
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:createNewAwaitingMultipleStep] attributesWhenCreate: ',
+    attributesWhenCreate
+  );
+
   await dynamodbSharedLib.putItem(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleSteps'),
@@ -246,6 +264,11 @@ export async function createNewAwaitingMultipleStep(
     attributesForPending
   );
 
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:createNewAwaitingMultipleStep] Returning: ',
+    parentFlowId
+  );
+
   return parentFlowId;
 }
 
@@ -260,9 +283,23 @@ export async function findPendingStepIdsAwaitingMultipleSteps(
   _izContext,
   awaitingStepId
 ) {
-  let pendingStepIds = [];
+  _izContext.logger.debug(
+    '[Lib:findPendingStepIdsAwaitingMultipleSteps] Input parameters',
+    {
+      awaitingStepId: awaitingStepId
+    }
+  );
 
-  let listPendingStepIds = await dynamodbSharedLib.query(
+  if (!awaitingStepId) {
+    _izContext.logger.error(
+      '[Lib:findPendingStepIdsAwaitingMultipleSteps] awaitingStepId is required'
+    );
+    throw new NoRetryError('awaitingStepId is required');
+  }
+
+  const pendingStepIds = [];
+
+  const listPendingStepIds = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleSteps'),
     {
@@ -270,10 +307,15 @@ export async function findPendingStepIdsAwaitingMultipleSteps(
     }
   );
 
-  for (let idx = 0; idx < listPendingStepIds.Items.length; idx++) {
-    pendingStepIds.push(listPendingStepIds.Items[idx].pendingStepId);
+  for (const item of listPendingStepIds.Items) {
+    pendingStepIds.push(item.pendingStepId);
   }
 
+  _izContext.logger.debug(
+    '[Lib:findPendingStepIdsAwaitingMultipleSteps] Pending step IDs',
+    pendingStepIds
+  );
+
   return pendingStepIds;
 }
 
@@ -288,14 +330,33 @@ export async function findPendingStepsAwaitingMultipleSteps(
   _izContext,
   awaitingStepId
 ) {
-  let listPendingSteps = await dynamodbSharedLib.query(
+  _izContext.logger.debug(
+    '[Lib:findPendingStepsAwaitingMultipleSteps] Input parameters',
+    {
+      awaitingStepId: awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    _izContext.logger.error(
+      '[Lib:findPendingStepsAwaitingMultipleSteps] awaitingStepId is required'
+    );
+    throw new NoRetryError('awaitingStepId is required');
+  }
+
+  const listPendingSteps = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleSteps'),
     {
       awaitingStepId: awaitingStepId
     }
   );
-  _izContext.logger.debug('Record of awaitingStepId', listPendingSteps);
+
+  _izContext.logger.debug(
+    '[Lib:findPendingStepsAwaitingMultipleSteps] Record of awaitingStepId',
+    listPendingSteps
+  );
+
   return listPendingSteps.Items;
 }
 
@@ -311,18 +372,40 @@ export async function findPendingStepAwaitingMultipleSteps(
   _izContext,
   awaitingStepId
 ) {
-  let listPendingSteps = await dynamodbSharedLib.query(
+  _izContext.logger.debug(
+    '[Lib:findPendingStepAwaitingMultipleSteps] Input parameters',
+    {
+      awaitingStepId: awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    _izContext.logger.error(
+      '[Lib:findPendingStepAwaitingMultipleSteps] awaitingStepId is required'
+    );
+    throw new NoRetryError('awaitingStepId is required');
+  }
+
+  const listPendingSteps = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleSteps'),
     {
       awaitingStepId: awaitingStepId
     }
   );
-  _izContext.logger.debug('Record of awaitingStepId', listPendingSteps);
+
+  _izContext.logger.debug(
+    '[Lib:findPendingStepAwaitingMultipleSteps] Record of awaitingStepId',
+    listPendingSteps
+  );
 
   if (listPendingSteps.Items.length !== 1) {
+    _izContext.logger.error(
+      '[Lib:findPendingStepAwaitingMultipleSteps] listPendingSteps not equals 1'
+    );
     throw new NoRetryError('listPendingSteps not equals 1');
   }
+
   return listPendingSteps.Items[0];
 }
 
@@ -343,7 +426,21 @@ export async function findAwaitingMultipleStepByPending(
   _izContext,
   pendingStepId
 ) {
-  let listAwaitingSteps = await dynamodbSharedLib.query(
+  _izContext.logger.debug(
+    '[Lib:findAwaitingMultipleStepByPending] Input parameters',
+    {
+      pendingStepId: pendingStepId
+    }
+  );
+
+  if (!pendingStepId) {
+    _izContext.logger.error(
+      '[Lib:findAwaitingMultipleStepByPending] pendingStepId is required'
+    );
+    throw new NoRetryError('pendingStepId is required');
+  }
+
+  const listAwaitingSteps = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleStepByPending'),
     {
@@ -351,12 +448,24 @@ export async function findAwaitingMultipleStepByPending(
     }
   );
   _izContext.logger.debug(
-    'Record of AwaitingMultipleStepByPending',
+    '[Lib:findAwaitingMultipleStepByPending] Record of AwaitingMultipleStepByPending',
     listAwaitingSteps
   );
+
   return listAwaitingSteps.Items;
 }
 
+/**
+ * Updates the completion status of a specific awaiting step within a multiple-step pending process.
+ * Marks the given awaiting step as complete and stores any encountered errors or additional attributes.
+ *
+ * @param {Object} _izContext - The execution context object containing logger and other utilities.
+ * @param {string} pendingStepId - The unique identifier of the parent pending step.
+ * @param {string|null} [currentAwaitingStepId=null] - The unique identifier of the specific awaiting step being processed. If null, no update is performed.
+ * @param {Array<Object|string>} [errorsFound=[]] - An array of errors encountered during the step's execution.
+ * @param {Object} [additionalAttributes={}] - Any extra data to save along with the completion status.
+ * @returns {Promise<void>} Resolves when the update operation is complete.
+ */
 export async function checkAllAwaitingStepsFinishedShared(
   _izContext,
   pendingStepId,
@@ -364,11 +473,14 @@ export async function checkAllAwaitingStepsFinishedShared(
   errorsFound = [],
   additionalAttributes = {}
 ) {
-  _izContext.logger.debug('Lib:checkAllAwaitingStepsFinishedWithError', {
-    pendingStepId: pendingStepId,
-    currentAwaitingStepId: currentAwaitingStepId,
-    errorsFound: errorsFound
-  });
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinishedShared] Input parameters',
+    {
+      pendingStepId: pendingStepId,
+      currentAwaitingStepId: currentAwaitingStepId,
+      errorsFound: errorsFound
+    }
+  );
 
   if (currentAwaitingStepId) {
     await dynamodbSharedLib.updateItem(
@@ -381,7 +493,7 @@ export async function checkAllAwaitingStepsFinishedShared(
       {
         complete: true,
         errorsFound: errorsFound,
-        ...additionalAttributes
+        additionalAttributes: additionalAttributes
       }
     );
   }
@@ -404,6 +516,15 @@ export async function checkAllAwaitingStepsFinished(
   errorsFound = [],
   additionalAttributes = {}
 ) {
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinished] Input parameters',
+    {
+      pendingStepId: pendingStepId,
+      currentAwaitingStepId: currentAwaitingStepId,
+      errorsFound: errorsFound
+    }
+  );
+
   await checkAllAwaitingStepsFinishedShared(
     _izContext,
     pendingStepId,
@@ -424,6 +545,9 @@ export async function checkAllAwaitingStepsFinished(
     if (!item.complete) return false;
   }
 
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinished] All steps are complete'
+  );
   return true;
 }
 
@@ -451,12 +575,21 @@ export async function checkAllAwaitingStepsFinishedWithReturnParams(
   errorsFound = [],
   additionalAttributes = {}
 ) {
-  _izContext.logger.debug('Lib:checkAllAwaitingStepsFinishedWithReturnParams', {
-    pendingStepId,
-    currentAwaitingStepId,
-    errorsFound,
-    additionalAttributes
-  });
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinishedWithReturnParams] Input parameters',
+    {
+      pendingStepId,
+      currentAwaitingStepId,
+      errorsFound,
+      additionalAttributes
+    }
+  );
+
+  if (!pendingStepId || !currentAwaitingStepId) {
+    throw NoRetryError(
+      '[Lib:checkAllAwaitingStepsFinishedWithReturnParams] pendingStepId or currentAwaitingStepId is required'
+    );
+  }
 
   await checkAllAwaitingStepsFinishedShared(
     _izContext,
@@ -473,7 +606,10 @@ export async function checkAllAwaitingStepsFinishedWithReturnParams(
     { limit: 50 }
   );
 
-  _izContext.logger.debug('awaitingStepItems:::', awaitingStepItems);
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinishedWithReturnParams] awaitingStepItems:::',
+    awaitingStepItems
+  );
 
   // Pass 1: check completeness of all other steps before collecting any data
   for (const item of awaitingStepItems.Items) {
@@ -487,6 +623,10 @@ export async function checkAllAwaitingStepsFinishedWithReturnParams(
     }
   }
 
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinishedWithReturnParams] Pass 1: all steps are complete'
+  );
+
   // Pass 2: all steps are complete — collect errors and attributes
   const collectedAttributes = {};
   const collectedErrors = [];
@@ -505,6 +645,10 @@ export async function checkAllAwaitingStepsFinishedWithReturnParams(
     }
   }
 
+  _izContext.logger.debug(
+    '[Lib:checkAllAwaitingStepsFinishedWithReturnParams] Pass 2: all steps are complete — collect errors and attributes'
+  );
+
   return {
     isComplete: true,
     collectedAttributes,
@@ -521,17 +665,28 @@ export async function checkAllAwaitingStepsFinishedWithReturnParams(
  * @returns {Promise<boolean>} true if delete flow processed
  */
 export async function clearAllAwaitingSteps(_izContext, pendingStepId) {
+  _izContext.logger.debug('[Lib:clearAllAwaitingSteps] Input parameters', {
+    pendingStepId: pendingStepId
+  });
+
+  if (!pendingStepId) {
+    throw NoRetryError('[Lib:clearAllAwaitingSteps] pendingStepId is required');
+  }
+
   // ----- query items from GSI table
-  let listPendingStepIds = await dynamodbSharedLib.query(
+  const listPendingStepIds = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingMultipleStepByPending'),
     {
       pendingStepId: pendingStepId
     }
   );
-  _izContext.logger.debug('listPendingStepIds', listPendingStepIds);
+  _izContext.logger.debug(
+    '[Lib:clearAllAwaitingSteps] listPendingStepIds',
+    listPendingStepIds
+  );
 
-  let promiseArray = [];
+  const promiseArray = [];
 
   for (let idx = 0; idx < listPendingStepIds.Items.length; idx++) {
     _izContext.logger.debug(`delete item in dynamodb AwaitingMultipleSteps`);
@@ -547,7 +702,7 @@ export async function clearAllAwaitingSteps(_izContext, pendingStepId) {
     );
 
     _izContext.logger.debug(
-      `delete item in dynamodb AwaitingMultipleStepByPending`
+      '[Lib:clearAllAwaitingSteps] delete item in dynamodb AwaitingMultipleStepByPending'
     );
     promiseArray.push(
       dynamodbSharedLib.deleteItem(
@@ -585,19 +740,26 @@ export async function removeAwaitingMultipleStep(
   pendingStepId,
   errorsFound = []
 ) {
-  _izContext.logger.debug('Lib:removeAwaitingMultipleStep', {
+  _izContext.logger.debug('[Lib:removeAwaitingMultipleStep] Input parameters', {
     awaitingStepId: awaitingStepId,
-    pendingStepId: pendingStepId
+    pendingStepId: pendingStepId,
+    errorsFound: errorsFound
   });
 
-  let promiseArray = [];
+  if (!awaitingStepId || !pendingStepId) {
+    throw NoRetryError(
+      '[Lib:removeAwaitingMultipleStep] awaitingStepId or pendingStepId is required'
+    );
+  }
+
+  const promiseArray = [];
 
-  let keyValues = {
+  const keyValues = {
     awaitingStepId: awaitingStepId,
     pendingStepId: pendingStepId
   };
 
-  let keyValuesForByPending = {
+  const keyValuesForByPending = {
     pendingStepId: pendingStepId,
     awaitingStepId: awaitingStepId
   };
@@ -607,7 +769,11 @@ export async function removeAwaitingMultipleStep(
   });
 
   _izContext.logger.debug(
-    `delete item in dynamodb AwaitingMultipleSteps ==> awaitingStepId:${keyValues.awaitingStepId}`
+    '[Lib:removeAwaitingMultipleStep] delete item in dynamodb AwaitingMultipleSteps',
+    {
+      awaitingStepId: keyValues.awaitingStepId,
+      pendingStepId: keyValues.pendingStepId
+    }
   );
   promiseArray.push(
     dynamodbSharedLib.deleteItem(
@@ -635,6 +801,21 @@ export async function removeAwaitingMultipleStep(
   await Promise.all(promiseArray);
 }
 
+/**
+ * Creates multiple awaiting steps with additional attributes and publishes corresponding messages to an SNS topic.
+ * Each message is augmented with a new `parentFlowId` representing its awaiting step before being published.
+ *
+ * @param {Object} _izContext - The execution context object containing logger and other utilities.
+ * @param {Array<Object>} messages - An array of messages to be augmented and published.
+ * @param {string} [prefix=''] - An optional prefix used when generating new awaiting step IDs.
+ * @param {Object} [flowType={}] - Defines the target SNS topic. Must contain `flowTag` and `serviceTag`.
+ * @param {string} flowType.flowTag - The flow tag identifier.
+ * @param {string} flowType.serviceTag - The service tag identifier.
+ * @param {Object} [additionalAttributes={}] - Additional attributes to save with each new awaiting step.
+ * @param {string} pendingStepId - The unique identifier of the parent pending step that waits for these new steps.
+ * @returns {Promise<void>} Resolves when all awaiting steps are created and messages are published.
+ * @throws {Error} Throws a NoRetryError if `messages` is not an array, or if `flowType` is missing `flowTag` or `serviceTag`.
+ */
 export async function createAwaitingMultipleStepsWithAdditionalAttributesAndPublish(
   _izContext,
   messages,
@@ -643,30 +824,28 @@ export async function createAwaitingMultipleStepsWithAdditionalAttributesAndPubl
   additionalAttributes = {},
   pendingStepId
 ) {
+  _izContext.logger.debug(
+    '[Lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish] Input parameters',
+    {
+      messages,
+      prefix,
+      flowType,
+      additionalAttributes,
+      pendingStepId
+    }
+  );
+
   if (!Array.isArray(messages)) {
-    throw new Error('messages must be an array');
+    throw NoRetryError('messages must be an array');
   }
 
   const flowTag = flowType.flowTag;
   const serviceTag = flowType.serviceTag;
 
-  _izContext.logger.debug(
-    '[lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish] flowType',
-    {
-      flowTag,
-      serviceTag
-    }
-  );
-
   if (flowTag === undefined || serviceTag === undefined) {
-    throw new Error('flowType must have both flowTag and serviceTag');
+    throw NoRetryError('flowType must have both flowTag and serviceTag');
   }
 
-  _izContext.logger.debug(
-    'Lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish',
-    { count: messages.length, pendingStepId, flowType }
-  );
-
   const topicArn = snsSharedLib.snsTopicArnByFlowSchema(
     _izContext,
     flowType.flowTag,
@@ -674,8 +853,8 @@ export async function createAwaitingMultipleStepsWithAdditionalAttributesAndPubl
   );
 
   _izContext.logger.debug(
-    'Lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish',
-    { topicArn }
+    '[Lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish] Topic Arn',
+    topicArn
   );
 
   const messagesWithParentId = await Promise.all(
@@ -702,4 +881,8 @@ export async function createAwaitingMultipleStepsWithAdditionalAttributesAndPubl
       })
     )
   );
+
+  _izContext.logger.debug(
+    '[Lib:asyncFlow:createAwaitingMultipleStepsWithAdditionalAttributesAndPublish] Exit point'
+  );
 }

package/src/awaitingStep.js

@@ -41,24 +41,25 @@ export async function createAwaitingStep(
   additionalAttributes = {}, // save in top level
   callingFlowConfig = {} // optional
 ) {
-  _izContext.logger.debug('Lib createAwaitingStep:', {
+  _izContext.logger.debug('[Lib:AsyncFlow:createAwaitingStep] Input: ', {
     awaitingStepId,
     pendingStepId,
     additionalAttributes,
     callingFlowConfig
   });
 
-  let attributesWhenCreate = {
+  const attributesWhenCreate = {
     awaitingStepId: awaitingStepId,
     pendingStepId: pendingStepId,
     timestamp: Date.now(),
     uniqueRequestId: _izContext.uniqueRequestId
   };
   // loop if not empty
-  for (let additionalAttribute in additionalAttributes) {
+  for (const additionalAttribute in additionalAttributes) {
     attributesWhenCreate[additionalAttribute] =
       additionalAttributes[additionalAttribute]; // each  additionalAttributes
   }
+
   if (Object.keys(callingFlowConfig).length > 0) {
     Object.assign(attributesWhenCreate, { callingFlowConfig });
   }
@@ -81,9 +82,20 @@ export async function findPendingStepIdsAwaitingStep(
   _izContext,
   awaitingStepId
 ) {
-  let pendingStepIds = [];
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:findPendingStepIdsAwaitingStep] Input: ',
+    {
+      awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    throw NoRetryError('awaitingStepId is required');
+  }
+
+  const pendingStepIds = [];
 
-  let listPendingStepIds = await dynamodbSharedLib.query(
+  const listPendingStepIds = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingStep'),
     {
@@ -106,7 +118,18 @@ export async function findPendingStepIdsAwaitingStep(
  * @returns {Promise<AwaitingStepItem[]>}
  */
 export async function findPendingStepsAwaitingStep(_izContext, awaitingStepId) {
-  let listPendingSteps = await dynamodbSharedLib.query(
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:findPendingStepsAwaitingStep] Input: ',
+    {
+      awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    throw NoRetryError('awaitingStepId is required');
+  }
+
+  const listPendingSteps = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingStep'),
     {
@@ -126,17 +149,32 @@ export async function findPendingStepsAwaitingStep(_izContext, awaitingStepId) {
  * @throws {NoRetryError} if there is not exactly one item
  */
 export async function findPendingStepAwaitingStep(_izContext, awaitingStepId) {
-  let listPendingSteps = await dynamodbSharedLib.query(
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:findPendingStepAwaitingStep] Input: ',
+    {
+      awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    throw NoRetryError('awaitingStepId is required');
+  }
+
+  const listPendingSteps = await dynamodbSharedLib.query(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingStep'),
     {
       awaitingStepId: awaitingStepId
     }
   );
-  _izContext.logger.debug('Record of awaitingStepId', listPendingSteps);
+
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:findPendingStepAwaitingStep] Found records: ',
+    listPendingSteps
+  );
 
   if (listPendingSteps.Items.length !== 1) {
-    throw new NoRetryError('listPendingSteps not equals 1');
+    throw NoRetryError('listPendingSteps not equals 1');
   }
   return listPendingSteps.Items[0];
 }
@@ -154,6 +192,17 @@ export async function findPendingStepAwaitingStepWithCallingFlow(
   _izContext,
   awaitingStepId
 ) {
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:findPendingStepAwaitingStepWithCallingFlow] Input: ',
+    {
+      awaitingStepId
+    }
+  );
+
+  if (!awaitingStepId) {
+    throw NoRetryError('awaitingStepId is required');
+  }
+
   let callingFlowConfig = null;
   let awaitingStep = await findPendingStepAwaitingStep(
     _izContext,
@@ -184,6 +233,15 @@ export async function removeAwaitingStep(
   awaitingStepId,
   pendingStepId
 ) {
+  _izContext.logger.debug('[Lib:AsyncFlow:removeAwaitingStep] Input: ', {
+    awaitingStepId,
+    pendingStepId
+  });
+
+  if (!awaitingStepId || !pendingStepId) {
+    throw NoRetryError('awaitingStepId and pendingStepId are required');
+  }
+
   await dynamodbSharedLib.deleteItem(
     _izContext,
     dynamodbSharedLib.tableName(_izContext, 'AwaitingStep'),
@@ -213,6 +271,22 @@ export async function removeAwaitingStepWithCheckUniqueRequestId(
   checkUniqueRequestId,
   prefix = ''
 ) {
+  _izContext.logger.debug(
+    '[Lib:AsyncFlow:removeAwaitingStepWithCheckUniqueRequestId] Input: ',
+    {
+      awaitingStepId,
+      pendingStepId,
+      checkUniqueRequestId,
+      prefix
+    }
+  );
+
+  if (!awaitingStepId || !pendingStepId || !checkUniqueRequestId) {
+    throw NoRetryError(
+      'awaitingStepId, pendingStepId and checkUniqueRequestId are required'
+    );
+  }
+
   // if logicalElements add queryElements.logicalElements
   // set condition uniqueRequestId match with exists
   const queryElementsCondition = {