@friggframework/devtools 2.0.0--canary.608.ba60ba6.0 → 2.0.0--canary.608.03436383054a.0

package/infrastructure/domains/integration/integration-builder.js

@@ -198,11 +198,7 @@ class IntegrationBuilder extends InfrastructureBuilder {
 
         // App-level FIFO queue for dispatch:'queue' events — one per app,
         // independent of per-integration queue ownership.
-        this.createUserActionQueue(
-            result,
-            functionPackageConfig,
-            usePrismaLayer
-        );
+        this.createUserActionQueue(result, functionPackageConfig, usePrismaLayer);
 
         for (const integration of appDefinition.integrations) {
             const integrationName = integration.Definition.name;
@@ -489,11 +485,7 @@ class IntegrationBuilder extends InfrastructureBuilder {
      * distinct integrations process in parallel. Always created, like the
      * InternalErrorQueue — idle cost is ~$0.
      */
-    createUserActionQueue(
-        result,
-        functionPackageConfig,
-        usePrismaLayer = true
-    ) {
+    createUserActionQueue(result, functionPackageConfig, usePrismaLayer = true) {
         const queueName =
             '${self:service}-${self:provider.stage}-FriggUserActionQueue.fifo';
         const dlqName =

package/infrastructure/domains/integration/integration-builder.test.js

@@ -1,6 +1,6 @@
 /**
  * Tests for Integration Builder
- *
+ * 
  * Tests integration-specific Lambda functions and SQS queues
  */
 
@@ -17,7 +17,9 @@ describe('IntegrationBuilder', () => {
     describe('shouldExecute()', () => {
         it('should return true when integrations array has items', () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             expect(integrationBuilder.shouldExecute(appDefinition)).toBe(true);
@@ -83,7 +85,9 @@ describe('IntegrationBuilder', () => {
 
         it('should error when integration is missing Definition', () => {
             const appDefinition = {
-                integrations: [{ someOtherField: 'value' }],
+                integrations: [
+                    { someOtherField: 'value' },
+                ],
             };
 
             const result = integrationBuilder.validate(appDefinition);
@@ -96,7 +100,9 @@ describe('IntegrationBuilder', () => {
 
         it('should error when integration Definition is missing name', () => {
             const appDefinition = {
-                integrations: [{ Definition: {} }],
+                integrations: [
+                    { Definition: {} },
+                ],
             };
 
             const result = integrationBuilder.validate(appDefinition);
@@ -126,7 +132,9 @@ describe('IntegrationBuilder', () => {
     describe('build()', () => {
         it('should create HTTP handler for integration', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'hubspot' } }],
+                integrations: [
+                    { Definition: { name: 'hubspot' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -139,7 +147,9 @@ describe('IntegrationBuilder', () => {
 
         it('should configure HTTP API event for integration', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'salesforce' } }],
+                integrations: [
+                    { Definition: { name: 'salesforce' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -156,7 +166,9 @@ describe('IntegrationBuilder', () => {
 
         it('should create SQS queue for integration', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'slack' } }],
+                integrations: [
+                    { Definition: { name: 'slack' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -167,39 +179,39 @@ describe('IntegrationBuilder', () => {
 
         it('should configure queue with correct retention and visibility timeout', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
-            expect(
-                result.resources.TestQueue.Properties.MessageRetentionPeriod
-            ).toBe(345600);
-            expect(
-                result.resources.TestQueue.Properties.VisibilityTimeout
-            ).toBe(1800);
+            expect(result.resources.TestQueue.Properties.MessageRetentionPeriod).toBe(345600);
+            expect(result.resources.TestQueue.Properties.VisibilityTimeout).toBe(1800);
         });
 
         it('should configure redrive policy to internal error queue', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
-            expect(result.resources.TestQueue.Properties.RedrivePolicy).toEqual(
-                {
-                    maxReceiveCount: 3,
-                    deadLetterTargetArn: {
-                        'Fn::GetAtt': ['InternalErrorQueue', 'Arn'],
-                    },
-                }
-            );
+            expect(result.resources.TestQueue.Properties.RedrivePolicy).toEqual({
+                maxReceiveCount: 3,
+                deadLetterTargetArn: {
+                    'Fn::GetAtt': ['InternalErrorQueue', 'Arn'],
+                },
+            });
         });
 
         it('should create queue worker function', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'hubspot' } }],
+                integrations: [
+                    { Definition: { name: 'hubspot' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -209,7 +221,9 @@ describe('IntegrationBuilder', () => {
 
         it('should configure queue worker with SQS event', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -227,7 +241,9 @@ describe('IntegrationBuilder', () => {
 
         it('should set queue worker timeout to 600 seconds', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -237,19 +253,21 @@ describe('IntegrationBuilder', () => {
 
         it('should set queue worker reserved concurrency', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'test' } }],
+                integrations: [
+                    { Definition: { name: 'test' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
-            expect(result.functions.testQueueWorker.reservedConcurrency).toBe(
-                20
-            );
+            expect(result.functions.testQueueWorker.reservedConcurrency).toBe(20);
         });
 
         it('should add queue URL to environment variables', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'slack' } }],
+                integrations: [
+                    { Definition: { name: 'slack' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -261,14 +279,14 @@ describe('IntegrationBuilder', () => {
 
         it('should add queue name to custom variables', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'stripe' } }],
+                integrations: [
+                    { Definition: { name: 'stripe' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
-            expect(result.custom.StripeQueue).toBe(
-                '${self:service}--${self:provider.stage}-StripeQueue'
-            );
+            expect(result.custom.StripeQueue).toBe('${self:service}--${self:provider.stage}-StripeQueue');
         });
 
         it('should handle multiple integrations', async () => {
@@ -303,7 +321,9 @@ describe('IntegrationBuilder', () => {
 
         it('should capitalize integration name for queue reference', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'myIntegration' } }],
+                integrations: [
+                    { Definition: { name: 'myIntegration' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -314,7 +334,9 @@ describe('IntegrationBuilder', () => {
 
         it('should handle integration names with hyphens', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'my-integration' } }],
+                integrations: [
+                    { Definition: { name: 'my-integration' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
@@ -339,8 +361,7 @@ describe('IntegrationBuilder', () => {
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
-            const retention =
-                result.resources.TestQueue.Properties.MessageRetentionPeriod;
+            const retention = result.resources.TestQueue.Properties.MessageRetentionPeriod;
 
             // The max SQS DelaySeconds is 900. Retention must comfortably
             // exceed this to ensure delayed messages are never silently lost.
@@ -356,9 +377,7 @@ describe('IntegrationBuilder', () => {
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
-            const maxReceiveCount =
-                result.resources.TestQueue.Properties.RedrivePolicy
-                    .maxReceiveCount;
+            const maxReceiveCount = result.resources.TestQueue.Properties.RedrivePolicy.maxReceiveCount;
 
             // Should allow at least 2 retries (maxReceiveCount >= 3)
             expect(maxReceiveCount).toBeGreaterThanOrEqual(3);
@@ -374,9 +393,7 @@ describe('IntegrationBuilder', () => {
             const result = await integrationBuilder.build(appDefinition, {});
             const sqsEvent = result.functions.testQueueWorker.events[0].sqs;
 
-            expect(sqsEvent.functionResponseType).toBe(
-                'ReportBatchItemFailures'
-            );
+            expect(sqsEvent.functionResponseType).toBe('ReportBatchItemFailures');
         });
     });
 
@@ -389,18 +406,10 @@ describe('IntegrationBuilder', () => {
             const result = await integrationBuilder.build(appDefinition, {});
 
             expect(result.resources.DLQMessageAlarm).toBeDefined();
-            expect(result.resources.DLQMessageAlarm.Type).toBe(
-                'AWS::CloudWatch::Alarm'
-            );
-            expect(result.resources.DLQMessageAlarm.Properties.MetricName).toBe(
-                'ApproximateNumberOfMessagesVisible'
-            );
-            expect(
-                result.resources.DLQMessageAlarm.Properties.ComparisonOperator
-            ).toBe('GreaterThanThreshold');
-            expect(result.resources.DLQMessageAlarm.Properties.Threshold).toBe(
-                500
-            );
+            expect(result.resources.DLQMessageAlarm.Type).toBe('AWS::CloudWatch::Alarm');
+            expect(result.resources.DLQMessageAlarm.Properties.MetricName).toBe('ApproximateNumberOfMessagesVisible');
+            expect(result.resources.DLQMessageAlarm.Properties.ComparisonOperator).toBe('GreaterThanThreshold');
+            expect(result.resources.DLQMessageAlarm.Properties.Threshold).toBe(500);
         });
 
         it('should wire alarm to InternalErrorBridgeTopic for notifications', async () => {
@@ -410,9 +419,9 @@ describe('IntegrationBuilder', () => {
 
             const result = await integrationBuilder.build(appDefinition, {});
 
-            expect(
-                result.resources.DLQMessageAlarm.Properties.AlarmActions
-            ).toEqual([{ Ref: 'InternalErrorBridgeTopic' }]);
+            expect(result.resources.DLQMessageAlarm.Properties.AlarmActions).toEqual([
+                { Ref: 'InternalErrorBridgeTopic' },
+            ]);
         });
 
         it('should create a DLQ processor Lambda triggered by InternalErrorQueue', async () => {
@@ -426,9 +435,7 @@ describe('IntegrationBuilder', () => {
             expect(result.functions.dlqProcessor.events[0].sqs.arn).toEqual({
                 'Fn::GetAtt': ['InternalErrorQueue', 'Arn'],
             });
-            expect(
-                result.functions.dlqProcessor.events[0].sqs.functionResponseType
-            ).toBe('ReportBatchItemFailures');
+            expect(result.functions.dlqProcessor.events[0].sqs.functionResponseType).toBe('ReportBatchItemFailures');
         });
 
         it('DLQ processor should have skipEsbuild, short timeout, and low concurrency', async () => {
@@ -440,9 +447,7 @@ describe('IntegrationBuilder', () => {
 
             expect(result.functions.dlqProcessor.skipEsbuild).toBe(true);
             expect(result.functions.dlqProcessor.package).toBeDefined();
-            expect(result.functions.dlqProcessor.timeout).toBeLessThanOrEqual(
-                60
-            );
+            expect(result.functions.dlqProcessor.timeout).toBeLessThanOrEqual(60);
             expect(result.functions.dlqProcessor.reservedConcurrency).toBe(1);
         });
     });
@@ -469,7 +474,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'hubspot',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -489,7 +494,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'salesforce',
                             webhooks: { enabled: true },
-                        },
+                        }
                     },
                 ],
             };
@@ -506,7 +511,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'slack',
                             webhooks: false,
-                        },
+                        }
                     },
                 ],
             };
@@ -523,7 +528,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'test',
                             webhooks: { enabled: false },
-                        },
+                        }
                     },
                 ],
             };
@@ -540,7 +545,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'stripe',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -711,7 +716,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'asana',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -737,7 +742,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'test',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -765,19 +770,19 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'hubspot',
                             webhooks: true,
-                        },
+                        }
                     },
                     {
                         Definition: {
                             name: 'salesforce',
                             webhooks: false,
-                        },
+                        }
                     },
                     {
                         Definition: {
                             name: 'slack',
                             webhooks: { enabled: true },
-                        },
+                        }
                     },
                 ],
             };
@@ -807,7 +812,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'test',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -824,7 +829,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'test',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -832,26 +837,24 @@ describe('IntegrationBuilder', () => {
             const result = await integrationBuilder.build(appDefinition, {});
 
             expect(result.functions.testWebhook.package).toBeDefined();
-            expect(result.functions.testWebhook.package.exclude).toContain(
-                'node_modules/aws-sdk/**'
-            );
-            expect(result.functions.testWebhook.package.exclude).toContain(
-                'node_modules/@prisma/**'
-            );
+            expect(result.functions.testWebhook.package.exclude).toContain('node_modules/aws-sdk/**');
+            expect(result.functions.testWebhook.package.exclude).toContain('node_modules/@prisma/**');
         });
     });
 
     describe('Prisma Layer Configuration', () => {
         it('should attach Prisma Lambda layer to queue worker functions', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'hubspot' } }],
+                integrations: [
+                    { Definition: { name: 'hubspot' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
             // Queue workers need Prisma layer for database operations
             expect(result.functions.hubspotQueueWorker.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
         });
 
@@ -867,26 +870,28 @@ describe('IntegrationBuilder', () => {
             const result = await integrationBuilder.build(appDefinition, {});
 
             expect(result.functions.hubspotQueueWorker.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
             expect(result.functions.salesforceQueueWorker.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
             expect(result.functions.slackQueueWorker.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
         });
 
         it('should attach Prisma layer to HTTP handlers for database access', async () => {
             const appDefinition = {
-                integrations: [{ Definition: { name: 'stripe' } }],
+                integrations: [
+                    { Definition: { name: 'stripe' } },
+                ],
             };
 
             const result = await integrationBuilder.build(appDefinition, {});
 
             // HTTP handlers also need Prisma for integration queries
             expect(result.functions.stripe.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
         });
 
@@ -897,7 +902,7 @@ describe('IntegrationBuilder', () => {
                         Definition: {
                             name: 'hubspot',
                             webhooks: true,
-                        },
+                        }
                     },
                 ],
             };
@@ -906,7 +911,7 @@ describe('IntegrationBuilder', () => {
 
             // Webhook handlers need Prisma for credential lookups
             expect(result.functions.hubspotWebhook.layers).toEqual([
-                { Ref: 'PrismaLambdaLayer' },
+                { Ref: 'PrismaLambdaLayer' }
             ]);
         });
 
@@ -1041,10 +1046,7 @@ describe('IntegrationBuilder', () => {
 
         it('omits the Prisma layer on the worker when usePrismaLambdaLayer=false', async () => {
             const result = await integrationBuilder.build(
-                {
-                    usePrismaLambdaLayer: false,
-                    integrations: [{ Definition: { name: 'test' } }],
-                },
+                { usePrismaLambdaLayer: false, integrations: [{ Definition: { name: 'test' } }] },
                 {}
             );
 
@@ -1054,3 +1056,4 @@ describe('IntegrationBuilder', () => {
         });
     });
 });
+

package/infrastructure/domains/security/iam-generator.js

@@ -14,7 +14,7 @@ function generateIAMCloudFormation(options = {}) {
         appName = 'Frigg',
         features = {},
         userPrefix = 'frigg-deployment-user',
-        stackName = 'frigg-deployment-iam',
+        stackName = 'frigg-deployment-iam'
     } = options;
 
     const deploymentUserName = userPrefix;
@@ -765,7 +765,8 @@ function getFeatureSummary(appDefinition) {
     const features = {
         core: true, // Always enabled
         vpc: appDefinition.vpc?.enable === true,
-        kms: appDefinition.encryption?.fieldLevelEncryptionMethod === 'kms',
+        kms:
+            appDefinition.encryption?.fieldLevelEncryptionMethod === 'kms',
         ssm: appDefinition.ssm?.enable === true,
         websockets: appDefinition.websockets?.enable === true,
     };
@@ -784,10 +785,7 @@ function getFeatureSummary(appDefinition) {
  * @returns {Object} Basic IAM policy document
  */
 function generateBasicIAMPolicy() {
-    const basicPolicyPath = path.join(
-        __dirname,
-        'templates/iam-policy-basic.json'
-    );
+    const basicPolicyPath = path.join(__dirname, 'templates/iam-policy-basic.json');
     return require(basicPolicyPath);
 }
 
@@ -796,10 +794,7 @@ function generateBasicIAMPolicy() {
  * @returns {Object} Full IAM policy document
  */
 function generateFullIAMPolicy() {
-    const fullPolicyPath = path.join(
-        __dirname,
-        'templates/iam-policy-full.json'
-    );
+    const fullPolicyPath = path.join(__dirname, 'templates/iam-policy-full.json');
     return require(fullPolicyPath);
 }