@friggframework/devtools 2.0.0--canary.490.b93c127.0 → 2.0.0--canary.490.83d98eb.0

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

@@ -220,16 +220,6 @@ class CloudFormationDiscovery {
             discovered.existingLogicalIds = [];
         }
         
-        // DEBUG: Log all VPC-related resources CloudFormation returns
-        console.log('  DEBUG: VPC-related resources from CloudFormation:');
-        resources.filter(r => 
-            r.LogicalResourceId.includes('VPCEndpoint') || 
-            r.LogicalResourceId.includes('Route') ||
-            r.LogicalResourceId.includes('RouteTable')
-        ).forEach(r => {
-            console.log(`    - ${r.LogicalResourceId} (${r.ResourceType}): ${r.PhysicalResourceId}`);
-        });
-        
         for (const resource of resources) {
             const { LogicalResourceId, PhysicalResourceId, ResourceType } = resource;
 
@@ -239,9 +229,6 @@ class CloudFormationDiscovery {
                 LogicalResourceId.includes('Migration') ||
                 LogicalResourceId.startsWith('VPCEndpoint')) {
                 discovered.existingLogicalIds.push(LogicalResourceId);
-                if (LogicalResourceId.includes('Route') || LogicalResourceId.includes('VPCEndpoint')) {
-                    console.log(`  DEBUG: Added to existingLogicalIds: ${LogicalResourceId}`);
-                }
             }
 
             // Debug Aurora detection
@@ -375,14 +362,19 @@ class CloudFormationDiscovery {
                 
                 // CRITICAL: Extract subnet ID from association physical ID by querying EC2
                 // Physical ID is the association ID (rtbassoc-xxx), need to query to get subnet
+                console.log(`  DEBUG: Checking if this is subnet 1 association: LogicalResourceId="${LogicalResourceId}", matches=${LogicalResourceId === 'FriggSubnet1RouteAssociation' || LogicalResourceId === 'FriggPrivateSubnet1RouteTableAssociation'}`);
+                console.log(`  DEBUG: this.provider=${!!this.provider}, getEC2Client=${!!(this.provider && this.provider.getEC2Client)}`);
+                
                 if (this.provider && this.provider.getEC2Client && 
                     (LogicalResourceId === 'FriggSubnet1RouteAssociation' || LogicalResourceId === 'FriggPrivateSubnet1RouteTableAssociation')) {
                     // Store association ID to query later (after loop)
                     discovered._subnet1AssociationId = PhysicalResourceId;
+                    console.log(`  DEBUG: Set _subnet1AssociationId = ${PhysicalResourceId}`);
                 }
                 if (this.provider && this.provider.getEC2Client && 
                     (LogicalResourceId === 'FriggSubnet2RouteAssociation' || LogicalResourceId === 'FriggPrivateSubnet2RouteTableAssociation')) {
                     discovered._subnet2AssociationId = PhysicalResourceId;
+                    console.log(`  DEBUG: Set _subnet2AssociationId = ${PhysicalResourceId}`);
                 }
             }
 

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

@@ -872,5 +872,14 @@ describe('CloudFormationDiscovery', () => {
             expect(result.existingLogicalIds).not.toContain('VPCEndpointS3');
         });
     });
+
+    describe('Subnet extraction from route table associations', () => {
+        it.skip('should extract subnet IDs from route table associations when route table Associations array is empty', async () => {
+            // TODO: This test needs proper mock setup for EC2 client
+            // The code already exists (lines 563-608 in cloudformation-discovery.js)
+            // Skipping until we can properly mock the EC2 client chain
+        });
+
+    });
 });
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.490.b93c127.0",
+  "version": "2.0.0--canary.490.83d98eb.0",
   "bin": {
     "frigg": "./frigg-cli/index.js"
   },
@@ -16,9 +16,9 @@
     "@babel/eslint-parser": "^7.18.9",
     "@babel/parser": "^7.25.3",
     "@babel/traverse": "^7.25.3",
-    "@friggframework/core": "2.0.0--canary.490.b93c127.0",
-    "@friggframework/schemas": "2.0.0--canary.490.b93c127.0",
-    "@friggframework/test": "2.0.0--canary.490.b93c127.0",
+    "@friggframework/core": "2.0.0--canary.490.83d98eb.0",
+    "@friggframework/schemas": "2.0.0--canary.490.83d98eb.0",
+    "@friggframework/test": "2.0.0--canary.490.83d98eb.0",
     "@hapi/boom": "^10.0.1",
     "@inquirer/prompts": "^5.3.8",
     "axios": "^1.7.2",
@@ -46,8 +46,8 @@
     "validate-npm-package-name": "^5.0.0"
   },
   "devDependencies": {
-    "@friggframework/eslint-config": "2.0.0--canary.490.b93c127.0",
-    "@friggframework/prettier-config": "2.0.0--canary.490.b93c127.0",
+    "@friggframework/eslint-config": "2.0.0--canary.490.83d98eb.0",
+    "@friggframework/prettier-config": "2.0.0--canary.490.83d98eb.0",
     "aws-sdk-client-mock": "^4.1.0",
     "aws-sdk-client-mock-jest": "^4.1.0",
     "jest": "^30.1.3",
@@ -79,5 +79,5 @@
   "publishConfig": {
     "access": "public"
   },
-  "gitHead": "b93c1271110aaecf2fb36d5b9b39d35fad1c4dc5"
+  "gitHead": "83d98eb69c36a73f76db1a7289e6b282178b6b5b"
 }