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

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

@@ -196,8 +196,6 @@ 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);
 
         for (const integration of appDefinition.integrations) {
@@ -477,13 +475,9 @@ class IntegrationBuilder extends InfrastructureBuilder {
     }
 
     /**
-     * Create the app-level FIFO queue + FIFO DLQ + worker used by integration
-     * events declared with `dispatch: 'queue'`.
-     *
-     * A single queue serves every integration: `MessageGroupId = integrationId`
-     * (set by the producer) serializes events per integration while letting
-     * distinct integrations process in parallel. Always created, like the
-     * InternalErrorQueue — idle cost is ~$0.
+     * App-level FIFO queue + DLQ + worker for `dispatch: 'queue'` events. One
+     * queue serves all integrations; MessageGroupId = integrationId serializes
+     * per integration. Always created, like the InternalErrorQueue.
      */
     createUserActionQueue(result, functionPackageConfig, usePrismaLayer = true) {
         const queueName =
@@ -504,10 +498,8 @@ class IntegrationBuilder extends InfrastructureBuilder {
             },
         };
 
-        // ContentBasedDeduplication is intentionally OMITTED. The producer
-        // supplies a unique MessageDeduplicationId per send, so two distinct
-        // requests with identical bodies both run — content-based dedup would
-        // hash the body and silently drop the second within the 5-min window.
+        // No ContentBasedDeduplication: the producer sends a unique dedup id so
+        // two distinct requests with identical bodies both run (not deduped).
         result.resources.FriggUserActionQueue = {
             Type: 'AWS::SQS::Queue',
             Properties: {
@@ -524,13 +516,10 @@ class IntegrationBuilder extends InfrastructureBuilder {
             },
         };
 
-        // Expose the queue URL to ALL Lambdas — both the HTTP handlers that
-        // produce events and the worker that consumes them.
         result.environment.USER_ACTION_QUEUE_URL = {
             Ref: 'FriggUserActionQueue',
         };
 
-        // Any message in the FIFO DLQ is a failed eventual event.
         result.resources.FriggUserActionDLQAlarm = {
             Type: 'AWS::CloudWatch::Alarm',
             Properties: {
@@ -555,9 +544,8 @@ class IntegrationBuilder extends InfrastructureBuilder {
             },
         };
 
-        // Single app-level worker. NO reservedConcurrency: FIFO already
-        // serializes per MessageGroupId; capping at 1 would serialize across
-        // ALL integrations and kill cross-integration parallelism.
+        // No reservedConcurrency: FIFO already serializes per group; capping
+        // would serialize across all integrations.
         result.functions.userActionQueueWorker = {
             handler:
                 'node_modules/@friggframework/core/handlers/workers/user-action-worker.userActionQueueWorker',

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

@@ -751,9 +751,7 @@ describe('IntegrationBuilder', () => {
 
             const functionKeys = Object.keys(result.functions);
 
-            // Expected order: dlqProcessor + userActionQueueWorker (both
-            // app-level, created before the integration loop), then webhook,
-            // integration, queueWorker.
+            // App-level functions (dlqProcessor, userActionQueueWorker) come first.
             expect(functionKeys).toEqual([
                 'dlqProcessor',
                 'userActionQueueWorker',

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

@@ -426,9 +426,7 @@ function generateIAMCloudFormation(options = {}) {
             ],
             Resource: [
                 { 'Fn::Sub': 'arn:aws:sqs:*:${AWS::AccountId}:*frigg*' },
-                // IAM wildcard matching is case-sensitive; the app-level FIFO
-                // queue is named "...-FriggUserActionQueue.fifo", which the
-                // lowercase "*frigg*" glob would not match.
+                // Case-sensitive: lowercase "*frigg*" won't match "FriggUserActionQueue.fifo".
                 { 'Fn::Sub': 'arn:aws:sqs:*:${AWS::AccountId}:*Frigg*' },
                 {
                     'Fn::Sub':

package/infrastructure/domains/security/templates/frigg-deployment-iam-stack.yaml

@@ -205,8 +205,7 @@ Resources:
               - 'sqs:UntagQueue'
             Resource:
               - !Sub 'arn:aws:sqs:*:${AWS::AccountId}:*frigg*'
-              # Case-sensitive glob: the app-level FIFO queue is named
-              # "...-FriggUserActionQueue.fifo" which "*frigg*" would not match.
+              # Case-sensitive: lowercase "*frigg*" won't match "FriggUserActionQueue.fifo".
               - !Sub 'arn:aws:sqs:*:${AWS::AccountId}:*Frigg*'
               - !Sub 'arn:aws:sqs:*:${AWS::AccountId}:internal-error-queue-*'
           

package/infrastructure/domains/shared/utilities/base-definition-factory.js

@@ -196,9 +196,7 @@ function createBaseDefinition(
                     ],
                     Resource: [
                         { 'Fn::GetAtt': ['InternalErrorQueue', 'Arn'] },
-                        // Explicit ARN: the per-integration glob below ends in
-                        // "-*Queue" and does NOT match the ".fifo" suffix, so the
-                        // app-level FIFO queue must be listed directly.
+                        // Explicit: the "-*Queue" glob below doesn't match ".fifo".
                         { 'Fn::GetAtt': ['FriggUserActionQueue', 'Arn'] },
                         {
                             'Fn::Join': [

package/infrastructure/domains/shared/utilities/base-definition-factory.test.js

@@ -134,8 +134,7 @@ describe('Base Definition Factory', () => {
             );
             expect(sqsPermission).toBeDefined();
 
-            // The per-integration glob ends in "-*Queue" and won't match the
-            // ".fifo" app-level queue, so its ARN must be listed explicitly.
+            // The "-*Queue" glob doesn't match ".fifo", so the ARN is explicit.
             expect(sqsPermission.Resource).toContainEqual({
                 'Fn::GetAtt': ['FriggUserActionQueue', 'Arn'],
             });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.608.03436383054a.0",
+  "version": "2.0.0--canary.608.e6b65ff.0",
   "bin": {
     "frigg": "./frigg-cli/index.js"
   },
@@ -25,9 +25,9 @@
     "@babel/eslint-parser": "^7.18.9",
     "@babel/parser": "^7.25.3",
     "@babel/traverse": "^7.25.3",
-    "@friggframework/core": "2.0.0--canary.608.03436383054a.0",
-    "@friggframework/schemas": "2.0.0--canary.608.03436383054a.0",
-    "@friggframework/test": "2.0.0--canary.608.03436383054a.0",
+    "@friggframework/core": "2.0.0--canary.608.e6b65ff.0",
+    "@friggframework/schemas": "2.0.0--canary.608.e6b65ff.0",
+    "@friggframework/test": "2.0.0--canary.608.e6b65ff.0",
     "@hapi/boom": "^10.0.1",
     "@inquirer/prompts": "^5.3.8",
     "axios": "^1.18.0",
@@ -55,8 +55,8 @@
     "validate-npm-package-name": "^5.0.0"
   },
   "devDependencies": {
-    "@friggframework/eslint-config": "2.0.0--canary.608.03436383054a.0",
-    "@friggframework/prettier-config": "2.0.0--canary.608.03436383054a.0",
+    "@friggframework/eslint-config": "2.0.0--canary.608.e6b65ff.0",
+    "@friggframework/prettier-config": "2.0.0--canary.608.e6b65ff.0",
     "aws-sdk-client-mock": "^4.1.0",
     "aws-sdk-client-mock-jest": "^4.1.0",
     "jest": "^30.1.3",
@@ -88,5 +88,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "03436383054a3ee70bd4de8f13a1907890a627da"
+  "gitHead": "e6b65ff3bf4a3b3e7777a1f2a6cf1cecd24bedd7"
 }