@friggframework/serverless-plugin 2.0.0--canary.579.2d1eba8.0 → 2.0.0--canary.580.1003d8d.0

package/index.js

@@ -66,12 +66,37 @@ class FriggServerlessPlugin {
   }
 
   extractQueueDefinitions() {
+    // Each custom.*Queue entry is the resolved QueueName. The matching
+    // CloudFormation resource (under resources.Resources) has the same
+    // logical ID and carries the Properties block we want to mirror onto
+    // LocalStack (VisibilityTimeout, MessageRetentionPeriod,
+    // RedrivePolicy, …). Deployed AWS applies those via CloudFormation;
+    // locally they'd be silently dropped and LocalStack would fall back
+    // to AWS defaults — notably a 30s VisibilityTimeout which
+    // re-delivers in-flight messages while a long-running queue worker
+    // is still processing them.
+    const resources =
+      this.serverless.service.resources &&
+      this.serverless.service.resources.Resources
+        ? this.serverless.service.resources.Resources
+        : {};
+
     return Object.keys(this.serverless.service.custom)
       .filter((key) => key.endsWith('Queue'))
-      .map((key) => ({
-        key,
-        name: this.serverless.service.custom[key],
-      }));
+      .map((key) => {
+        const resource = resources[key];
+        const properties =
+          resource &&
+          resource.Type === 'AWS::SQS::Queue' &&
+          resource.Properties
+            ? resource.Properties
+            : undefined;
+        return {
+          key,
+          name: this.serverless.service.custom[key],
+          ...(properties ? { properties } : {}),
+        };
+      });
   }
 
   createLocalStackSQSClient() {

package/index.test.js

@@ -183,6 +183,75 @@ describe('FriggServerlessPlugin', () => {
 
       expect(queues).toEqual([]);
     });
+
+    it('should attach Properties from matching CloudFormation resources', () => {
+      plugin = new FriggServerlessPlugin(mockServerless, mockOptions);
+      mockServerless.service.custom = {
+        HubspotQueue: 'svc--dev-HubspotQueue',
+      };
+      mockServerless.service.resources = {
+        Resources: {
+          HubspotQueue: {
+            Type: 'AWS::SQS::Queue',
+            Properties: {
+              QueueName: 'svc--dev-HubspotQueue',
+              VisibilityTimeout: 1800,
+              MessageRetentionPeriod: 345600,
+              RedrivePolicy: {
+                maxReceiveCount: 3,
+                deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+              },
+            },
+          },
+        },
+      };
+
+      const queues = plugin.extractQueueDefinitions();
+
+      expect(queues).toHaveLength(1);
+      expect(queues[0]).toEqual({
+        key: 'HubspotQueue',
+        name: 'svc--dev-HubspotQueue',
+        properties: {
+          QueueName: 'svc--dev-HubspotQueue',
+          VisibilityTimeout: 1800,
+          MessageRetentionPeriod: 345600,
+          RedrivePolicy: {
+            maxReceiveCount: 3,
+            deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+          },
+        },
+      });
+    });
+
+    it('should omit properties when resources.Resources is absent', () => {
+      plugin = new FriggServerlessPlugin(mockServerless, mockOptions);
+      mockServerless.service.custom = { LegacyQueue: 'legacy-queue' };
+      mockServerless.service.resources = undefined;
+
+      const queues = plugin.extractQueueDefinitions();
+
+      expect(queues).toEqual([{ key: 'LegacyQueue', name: 'legacy-queue' }]);
+    });
+
+    it('should ignore non-SQS CloudFormation resources with matching logical IDs', () => {
+      plugin = new FriggServerlessPlugin(mockServerless, mockOptions);
+      mockServerless.service.custom = { MisnamedQueue: 'misnamed-queue' };
+      mockServerless.service.resources = {
+        Resources: {
+          MisnamedQueue: {
+            Type: 'AWS::SNS::Topic',
+            Properties: { TopicName: 'not-an-sqs-queue' },
+          },
+        },
+      };
+
+      const queues = plugin.extractQueueDefinitions();
+
+      expect(queues).toEqual([
+        { key: 'MisnamedQueue', name: 'misnamed-queue' },
+      ]);
+    });
   });
 
   describe('Hooks', () => {

package/lib/localstack-queue-service.js

@@ -2,19 +2,73 @@
  * Infrastructure Service - LocalStack Queue Management
  *
  * Handles SQS queue creation in LocalStack for offline development.
+ *
+ * On deployed AWS, CloudFormation applies queue properties
+ * (VisibilityTimeout, MessageRetentionPeriod, RedrivePolicy) from the
+ * `Resources` block in serverless.yml. LocalStack gets those same
+ * properties here so local emulation matches production behavior —
+ * otherwise the queue defaults to a 30s VisibilityTimeout which re-
+ * delivers in-flight messages while a long-running queue worker is
+ * still processing them.
  */
 class LocalStackQueueService {
   constructor(sqsClient) {
     this.sqs = sqsClient;
   }
 
+  /**
+   * Whitelist of CloudFormation `AWS::SQS::Queue` Properties that map
+   * onto SQS `CreateQueue` Attributes. Everything else (tags, inline
+   * refs, etc.) is dropped on the way to LocalStack.
+   * @private
+   */
+  static PROPERTY_ATTRIBUTE_KEYS = [
+    'DelaySeconds',
+    'MaximumMessageSize',
+    'MessageRetentionPeriod',
+    'ReceiveMessageWaitTimeSeconds',
+    'VisibilityTimeout',
+    'RedrivePolicy',
+    'RedriveAllowPolicy',
+    'KmsMasterKeyId',
+    'KmsDataKeyReusePeriodSeconds',
+    'SqsManagedSseEnabled',
+    'FifoQueue',
+    'ContentBasedDeduplication',
+    'DeduplicationScope',
+    'FifoThroughputLimit',
+  ];
+
+  /**
+   * Serialize a CloudFormation `Properties` object into the
+   * `Attributes` shape the SQS `CreateQueue` API accepts (string
+   * values only; object values like `RedrivePolicy` get JSON-encoded).
+   * @private
+   */
+  _propertiesToAttributes(properties = {}) {
+    const attributes = {};
+    for (const key of LocalStackQueueService.PROPERTY_ATTRIBUTE_KEYS) {
+      const value = properties[key];
+      if (value === undefined || value === null) continue;
+      attributes[key] =
+        typeof value === 'object' ? JSON.stringify(value) : String(value);
+    }
+    return attributes;
+  }
+
   /**
    * @param {string} queueName - Name of queue to create
+   * @param {Object} [attributes] - SQS CreateQueue Attributes (already
+   *   stringified); missing/empty attributes fall back to AWS defaults.
    * @returns {Promise<string>} Queue URL
    */
-  async createQueue(queueName) {
+  async createQueue(queueName, attributes) {
     return new Promise((resolve, reject) => {
-      this.sqs.createQueue({ QueueName: queueName }, (err, data) => {
+      const params = { QueueName: queueName };
+      if (attributes && Object.keys(attributes).length > 0) {
+        params.Attributes = attributes;
+      }
+      this.sqs.createQueue(params, (err, data) => {
         if (err) {
           reject(new Error(`Failed to create queue ${queueName}: ${err.message}`));
         } else {
@@ -25,13 +79,17 @@ class LocalStackQueueService {
   }
 
   /**
-   * @param {Array<{key: string, name: string}>} queues - Queue definitions
+   * @param {Array<{key: string, name: string, properties?: Object}>} queues
+   *   Queue definitions. `properties` mirrors the CloudFormation
+   *   `AWS::SQS::Queue` Properties block; extracted by the plugin from
+   *   `serverless.service.resources.Resources`.
    * @returns {Promise<Array<{key: string, url: string}>>} Created queues with URLs
    */
   async createQueues(queues) {
     const results = await Promise.all(
       queues.map(async (queue) => {
-        const url = await this.createQueue(queue.name);
+        const attributes = this._propertiesToAttributes(queue.properties);
+        const url = await this.createQueue(queue.name, attributes);
         console.log(`Queue ${queue.name} created successfully. URL: ${url}`);
         return { key: queue.key, url };
       })

package/lib/localstack-queue-service.test.js

@@ -27,6 +27,41 @@ describe('LocalStackQueueService', () => {
       );
     });
 
+    it('should pass Attributes when provided', async () => {
+      mockSQS.createQueue.mockImplementation((params, callback) => {
+        callback(null, { QueueUrl: 'url' });
+      });
+
+      await service.createQueue('test-queue', {
+        VisibilityTimeout: '1800',
+        MessageRetentionPeriod: '345600',
+      });
+
+      expect(mockSQS.createQueue).toHaveBeenCalledWith(
+        {
+          QueueName: 'test-queue',
+          Attributes: {
+            VisibilityTimeout: '1800',
+            MessageRetentionPeriod: '345600',
+          },
+        },
+        expect.any(Function)
+      );
+    });
+
+    it('should omit Attributes when empty', async () => {
+      mockSQS.createQueue.mockImplementation((params, callback) => {
+        callback(null, { QueueUrl: 'url' });
+      });
+
+      await service.createQueue('test-queue', {});
+
+      expect(mockSQS.createQueue).toHaveBeenCalledWith(
+        { QueueName: 'test-queue' },
+        expect.any(Function)
+      );
+    });
+
     it('should reject with error on failure', async () => {
       const error = new Error('SQS Error');
       mockSQS.createQueue.mockImplementation((params, callback) => {
@@ -39,6 +74,43 @@ describe('LocalStackQueueService', () => {
     });
   });
 
+  describe('_propertiesToAttributes', () => {
+    it('maps CloudFormation Properties onto SQS Attributes and stringifies', () => {
+      const attrs = service._propertiesToAttributes({
+        VisibilityTimeout: 1800,
+        MessageRetentionPeriod: 345600,
+        RedrivePolicy: {
+          maxReceiveCount: 3,
+          deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+        },
+        QueueName: 'should-be-dropped',
+        Tags: [{ Key: 'ignored', Value: 'yes' }],
+      });
+
+      expect(attrs).toEqual({
+        VisibilityTimeout: '1800',
+        MessageRetentionPeriod: '345600',
+        RedrivePolicy: JSON.stringify({
+          maxReceiveCount: 3,
+          deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+        }),
+      });
+    });
+
+    it('skips undefined and null values', () => {
+      const attrs = service._propertiesToAttributes({
+        VisibilityTimeout: 60,
+        MessageRetentionPeriod: undefined,
+        KmsMasterKeyId: null,
+      });
+      expect(attrs).toEqual({ VisibilityTimeout: '60' });
+    });
+
+    it('returns an empty object when properties are missing', () => {
+      expect(service._propertiesToAttributes()).toEqual({});
+    });
+  });
+
   describe('createQueues', () => {
     it('should create multiple queues and return results', async () => {
       mockSQS.createQueue.mockImplementation((params, callback) => {
@@ -69,6 +141,58 @@ describe('LocalStackQueueService', () => {
       consoleLogSpy.mockRestore();
     });
 
+    it('forwards CloudFormation Properties as SQS Attributes', async () => {
+      const captured = [];
+      mockSQS.createQueue.mockImplementation((params, callback) => {
+        captured.push(params);
+        callback(null, { QueueUrl: 'url' });
+      });
+      jest.spyOn(console, 'log').mockImplementation();
+
+      await service.createQueues([
+        {
+          key: 'HubspotQueue',
+          name: 'my-service--dev-HubspotQueue',
+          properties: {
+            QueueName: 'my-service--dev-HubspotQueue',
+            VisibilityTimeout: 1800,
+            MessageRetentionPeriod: 345600,
+            RedrivePolicy: {
+              maxReceiveCount: 3,
+              deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+            },
+          },
+        },
+      ]);
+
+      expect(captured[0]).toEqual({
+        QueueName: 'my-service--dev-HubspotQueue',
+        Attributes: {
+          VisibilityTimeout: '1800',
+          MessageRetentionPeriod: '345600',
+          RedrivePolicy: JSON.stringify({
+            maxReceiveCount: 3,
+            deadLetterTargetArn: 'arn:aws:sqs:us-east-1:x:dlq',
+          }),
+        },
+      });
+    });
+
+    it('creates queues with defaults when properties are missing (back-compat)', async () => {
+      const captured = [];
+      mockSQS.createQueue.mockImplementation((params, callback) => {
+        captured.push(params);
+        callback(null, { QueueUrl: 'url' });
+      });
+      jest.spyOn(console, 'log').mockImplementation();
+
+      await service.createQueues([
+        { key: 'LegacyQueue', name: 'legacy-queue' },
+      ]);
+
+      expect(captured[0]).toEqual({ QueueName: 'legacy-queue' });
+    });
+
     it('should handle empty queue array', async () => {
       const results = await service.createQueues([]);
       expect(results).toEqual([]);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@friggframework/serverless-plugin",
-  "version": "2.0.0--canary.579.2d1eba8.0",
+  "version": "2.0.0--canary.580.1003d8d.0",
   "description": "Plugin to help dynamically load frigg resources",
   "main": "index.js",
   "scripts": {
@@ -11,5 +11,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "2d1eba87ebd7c2e83b0e95a87a41eb9037417694"
+  "gitHead": "1003d8deff8f587be3cd9210f1c35c5dc92531be"
 }