ac-sqs 3.3.0 → 3.3.2

package/CHANGELOG.md

@@ -1,3 +1,20 @@
+
+## [3.3.2](https://github.com/admiralcloud/ac-sqs/compare/v3.3.1..v3.3.2) (2025-09-26 20:15:26)
+
+
+### Bug Fix
+
+* **App:** Several minor improvement for createQueues | MP | [ef2abc7c6bc2f08e1328cb1c94be479362836323](https://github.com/admiralcloud/ac-sqs/commit/ef2abc7c6bc2f08e1328cb1c94be479362836323)    
+Several minor improvement for createQueues
+
+## [3.3.1](https://github.com/admiralcloud/ac-sqs/compare/v3.3.0..v3.3.1) (2025-09-26 17:45:21)
+
+
+### Bug Fix
+
+* **App:** Fixed createSQSlists function | MP | [246023f5eb622a33b7a261d45d68080bcfa29be6](https://github.com/admiralcloud/ac-sqs/commit/246023f5eb622a33b7a261d45d68080bcfa29be6)    
+Improved with logging and improved function names  
+Related issues:
  
 # [3.3.0](https://github.com/admiralcloud/ac-sqs/compare/v3.2.2..v3.3.0) (2025-09-26 17:27:53)
 

package/README.md

@@ -112,7 +112,7 @@ An array of metadata to get. By default only "ApproximateNumberOfMessages" is re
 
 https://docs.aws.amazon.com/AWSSimpleQueueService/latest/APIReference/API_GetQueueAttributes.html
 
-## createQueue
+## createQueues
 Call with all lists and let the function check if the queues exist. If not, the missing queue is created.
 
 # Test

package/index.js

@@ -2,7 +2,7 @@ const _ = require('lodash')
 const { v4: uuidV4 } = require('uuid')
 const { setTimeout: sleep } = require('timers/promises')
 
-const { SQSClient, SendMessageCommand, SendMessageBatchCommand, ReceiveMessageCommand, DeleteMessageBatchCommand, GetQueueAttributesCommand, ChangeMessageVisibilityBatchCommand } = require('@aws-sdk/client-sqs')
+const { SQSClient, SendMessageCommand, SendMessageBatchCommand, ReceiveMessageCommand, DeleteMessageBatchCommand, GetQueueAttributesCommand, ChangeMessageVisibilityBatchCommand, GetQueueUrlCommand, CreateQueueCommand } = require('@aws-sdk/client-sqs')
 const { S3Client, GetObjectCommand, PutObjectCommand, DeleteObjectsCommand } = require("@aws-sdk/client-s3")
 
 class ACSQS {
@@ -146,9 +146,10 @@ class ACSQS {
       ReceiptHandle: messageData.receiptHandle,
       VisibilityTimeout: visibilityTimeout
     }))
+    const { queueUrl } = this.getQueueUrl(config)
 
     return {
-      QueueUrl: await this.getQueueUrl(config),
+      QueueUrl: queueUrl,
       Entries: entries
     }
   }
@@ -318,14 +319,18 @@ class ACSQS {
     return response
   }
 
-  async getQueueUrl({ name, fifo, localPrefix, suffix }) {
-    let queueUrl = `https://sqs.${this.region}.amazonaws.com/${this.account}/` 
-    if (localPrefix) queueUrl += `local_${localPrefix}_`
-    if (process.env['NODE_ENV'] === 'test') queueUrl += 'test_'
-    queueUrl += name
-    if (suffix) queueUrl += suffix
-    if (fifo) queueUrl += '.fifo'
-    return queueUrl
+  getQueueUrl({ name, fifo, localPrefix, suffix }) {
+    // Build queue name using array filtering for cleaner concatenation
+    const queueName = [
+      localPrefix && `local_${localPrefix}_`,
+      process.env['NODE_ENV'] === 'test' && 'test_',
+      name,
+      suffix,
+      fifo && '.fifo'
+    ].filter(Boolean).join('')
+    
+    const queueUrl = `https://sqs.${this.region}.amazonaws.com/${this.account}/${queueName}`
+    return { queueName, queueUrl }
   }
 
   async getQueueAttributes({ name, attributes = ['ApproximateNumberOfMessages'], throwError }) {
@@ -334,8 +339,9 @@ class ACSQS {
       this.logger.error('ACSQS | getQueueAttributes | configurationMissing | %s', name)
       throw new Error('configurationForListMissing')
     }
+    const { queueUrl } = this.getQueueUrl(config)
     let sqsParams = {
-      QueueUrl: await this.getQueueUrl(config),
+      QueueUrl: queueUrl,
       AttributeNames: attributes
     }
     if (config.debug) this.logger.debug('ACSQS | getQueueAttributes | Payload %j', sqsParams)
@@ -349,7 +355,7 @@ class ACSQS {
     }
   }
 
-  async createQueue({ lists }) {
+  async createQueues({ lists, debug }) {
     for (const list of lists) {
       const config = _.find(this.availableLists, { name: list.name })
       if (!config) {
@@ -357,14 +363,34 @@ class ACSQS {
         throw new Error('configurationForListMissing')
       }
 
-      const queueUrl = await this.getQueueUrl(config)
-      const command = new CreateQueueCommand({ QueueName: queueUrl })
+      const { queueName } = this.getQueueUrl(config)
+        if (debug) this.logger.info('ACSQS | createQueues | %s | %s', list.name, queueName)
+      const input = {
+        QueueName: queueName
+      }
+      const checkCommand = new GetQueueUrlCommand(input)
       try {
-        await this.sqs.send(command)
+        await this.sqs.send(checkCommand)
+        continue
       }
-      catch(e) {
-        this.logger.error('AWS | createQueue | %s | %s', list.name, e?.message)
-        if (this.throwError) throw e
+      catch {
+        if (!_.isEmpty(_.get(config, 'attributes'))) {
+          input.Attributes = config.attributes
+        }
+        if (config?.fifo) input.Attributes = { ...input.Attributes, FifoQueue: 'true' }
+        if (config?.visibilityTimeout) input.Attributes = { ...input.Attributes, VisibilityTimeout: config.visibilityTimeout }
+        if (config?.delay) input.Attributes = { ...input.Attributes, DelaySeconds: config.delay }
+
+
+        const command = new CreateQueueCommand(input)
+        try {
+          await this.sqs.send(command)
+          if (debug) this.logger.info('ACSQS | createQueues | Created | %s | %s', list.name, queueName)
+        }
+        catch(e) {
+          this.logger.error('AWS | createQueue | %s | %s', list.name, e?.message)
+          if (this.throwError) throw e
+        } 
       }
     }
   }
@@ -390,8 +416,9 @@ class ACSQS {
       message = `s3:${key}`
     }
 
+    const { queueUrl } = this.getQueueUrl(config)
     const sqsParams = {
-      QueueUrl: await this.getQueueUrl(config),
+      QueueUrl: queueUrl,
       MessageBody: message
     }
     if (messageGroupId) _.set(sqsParams, 'MessageGroupId', messageGroupId)
@@ -448,8 +475,9 @@ class ACSQS {
       return item
     })
   
+    const { queueUrl } = this.getQueueUrl(config)
     const sqsParams = {
-      QueueUrl: await this.getQueueUrl(config),
+      QueueUrl: queueUrl,
       Entries: entries
     }
     if (debug || config.debug) this.logger.error('ACSQS | sendSQSMessageBatch | Payload %j', sqsParams)
@@ -473,8 +501,9 @@ class ACSQS {
     }
     const visibilityTimeout = _.get(config, 'visibilityTimeout') // if set, will activate visibilityTimeout management
   
+    const { queueUrl } = this.getQueueUrl(config)
     const sqsParams = {
-      QueueUrl: await this.getQueueUrl(config),
+      QueueUrl: queueUrl,
       MaxNumberOfMessages: _.get(config, 'batchSize', 10),
       VisibilityTimeout: _.get(config, 'visibilityTimeout', 30),
       WaitTimeSeconds: _.get(config, 'waitTime', 20)
@@ -539,8 +568,9 @@ class ACSQS {
       this.removeVisibilityTracking(messageId)
     }
 
-    let sqsParams = {
-      QueueUrl: await this.getQueueUrl(config),
+    const { queueUrl } = this.getQueueUrl(config)
+    const sqsParams = {
+      QueueUrl: queueUrl,
       Entries: entries
     }
     if (debug || config.debug) this.logger.debug('ACSQS | deleteSQSMessages | Payload %j', sqsParams)

package/package.json

@@ -3,7 +3,7 @@
   "author": "Mark Poepping (https://www.admiralcloud.com)",
   "license": "MIT",
   "repository": "admiralcloud/ac-sqs",
-  "version": "3.3.0",
+  "version": "3.3.2",
   "dependencies": {
     "@aws-sdk/client-s3": "^3.896.0",
     "@aws-sdk/client-sqs": "^3.896.0",