@friggframework/devtools 2.0.0--canary.461.1b7292f.0 → 2.0.0--canary.461.e6fc7c7.0

package/infrastructure/domains/shared/cloudformation-discovery.js

@@ -28,7 +28,7 @@ class CloudFormationDiscovery {
         try {
             // Try to get the stack
             const stack = await this.provider.describeStack(stackName);
-            
+
             // Get stack resources
             const resources = await this.provider.listStackResources(stackName);
 
@@ -51,7 +51,7 @@ class CloudFormationDiscovery {
             if (error.message && error.message.includes('does not exist')) {
                 return null;
             }
-            
+
             // Other errors - log and return null
             console.warn(`⚠️  CloudFormation discovery failed: ${error.message}`);
             return null;
@@ -73,18 +73,18 @@ class CloudFormationDiscovery {
 
         // VPC outputs
         if (outputMap.VpcId) {
-            discovered.vpcId = outputMap.VpcId;
+            discovered.defaultVpcId = outputMap.VpcId; // VpcBuilder expects 'defaultVpcId'
         }
-        
+
         if (outputMap.PrivateSubnetIds) {
             // Handle comma-separated subnet IDs
             discovered.privateSubnetIds = outputMap.PrivateSubnetIds.split(',').map(id => id.trim());
         }
-        
+
         if (outputMap.PublicSubnetId) {
             discovered.publicSubnetId = outputMap.PublicSubnetId;
         }
-        
+
         if (outputMap.SecurityGroupId) {
             discovered.securityGroupId = outputMap.SecurityGroupId;
         }
@@ -114,19 +114,25 @@ class CloudFormationDiscovery {
             // Security Group - use to get VPC ID
             if (LogicalResourceId === 'FriggLambdaSecurityGroup' && ResourceType === 'AWS::EC2::SecurityGroup') {
                 discovered.securityGroupId = PhysicalResourceId;
+                console.log(`  ✓ Found security group in stack: ${PhysicalResourceId}`);
                 // Query security group to get VPC ID
-                if (this.provider && !discovered.vpcId) {
+                if (this.provider && !discovered.defaultVpcId) {
                     try {
+                        console.log(`  Querying security group to get VPC ID...`);
+                        const { DescribeSecurityGroupsCommand } = require('@aws-sdk/client-ec2');
                         const sgDetails = await this.provider.getEC2Client().send(
-                            new (require('@aws-sdk/client-ec2').DescribeSecurityGroupsCommand)({
+                            new DescribeSecurityGroupsCommand({
                                 GroupIds: [PhysicalResourceId]
                             })
                         );
                         if (sgDetails.SecurityGroups && sgDetails.SecurityGroups.length > 0) {
-                            discovered.vpcId = sgDetails.SecurityGroups[0].VpcId;
+                            discovered.defaultVpcId = sgDetails.SecurityGroups[0].VpcId; // VpcBuilder expects 'defaultVpcId'
+                            console.log(`  ✓ Extracted VPC ID from security group: ${discovered.defaultVpcId}`);
+                        } else {
+                            console.warn(`  ⚠️  Security group query returned no results`);
                         }
                     } catch (error) {
-                        console.warn(`Could not get VPC from security group: ${error.message}`);
+                        console.warn(`  ⚠️  Could not get VPC from security group: ${error.message}`);
                     }
                 }
             }

package/infrastructure/domains/shared/cloudformation-discovery.test.js

@@ -48,7 +48,7 @@ describe('CloudFormationDiscovery', () => {
             const result = await cfDiscovery.discoverFromStack('test-stack');
 
             expect(result).toEqual({
-                vpcId: 'vpc-123',
+                defaultVpcId: 'vpc-123', // VpcBuilder expects 'defaultVpcId', not 'vpcId'
                 privateSubnetIds: ['subnet-1', 'subnet-2'],
                 publicSubnetId: 'subnet-3',
                 securityGroupId: 'sg-123',
@@ -197,7 +197,7 @@ describe('CloudFormationDiscovery', () => {
             const result = await cfDiscovery.discoverFromStack('test-stack');
 
             expect(result).toEqual({
-                vpcId: 'vpc-123',
+                defaultVpcId: 'vpc-123', // VpcBuilder expects 'defaultVpcId'
                 defaultKmsKeyId: 'arn:aws:kms:us-east-1:123456789:key/abc',
                 auroraClusterId: 'test-cluster',
                 natGatewayId: 'nat-123',

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.461.1b7292f.0",
+  "version": "2.0.0--canary.461.e6fc7c7.0",
   "dependencies": {
     "@aws-sdk/client-ec2": "^3.835.0",
     "@aws-sdk/client-kms": "^3.835.0",
@@ -11,8 +11,8 @@
     "@babel/eslint-parser": "^7.18.9",
     "@babel/parser": "^7.25.3",
     "@babel/traverse": "^7.25.3",
-    "@friggframework/schemas": "2.0.0--canary.461.1b7292f.0",
-    "@friggframework/test": "2.0.0--canary.461.1b7292f.0",
+    "@friggframework/schemas": "2.0.0--canary.461.e6fc7c7.0",
+    "@friggframework/test": "2.0.0--canary.461.e6fc7c7.0",
     "@hapi/boom": "^10.0.1",
     "@inquirer/prompts": "^5.3.8",
     "axios": "^1.7.2",
@@ -34,8 +34,8 @@
     "serverless-http": "^2.7.0"
   },
   "devDependencies": {
-    "@friggframework/eslint-config": "2.0.0--canary.461.1b7292f.0",
-    "@friggframework/prettier-config": "2.0.0--canary.461.1b7292f.0",
+    "@friggframework/eslint-config": "2.0.0--canary.461.e6fc7c7.0",
+    "@friggframework/prettier-config": "2.0.0--canary.461.e6fc7c7.0",
     "aws-sdk-client-mock": "^4.1.0",
     "aws-sdk-client-mock-jest": "^4.1.0",
     "jest": "^30.1.3",
@@ -70,5 +70,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "1b7292f30e11b350a075d5dcecded3c8c0f21167"
+  "gitHead": "e6fc7c76e03cc5a0698efc4705b005d02d941654"
 }