@friggframework/devtools 2.0.0--canary.461.12ba2eb.0 → 2.0.0--canary.461.499527f.0

package/infrastructure/domains/database/aurora-builder.js

@@ -156,7 +156,10 @@ class AuroraBuilder extends InfrastructureBuilder {
                     SecretStringTemplate: JSON.stringify({ username: dbConfig.username || 'postgres' }),
                     GenerateStringKey: 'password',
                     PasswordLength: 32,
-                    ExcludeCharacters: '"@/\\',
+                    // Exclude URL-special characters for Prisma connection string compatibility
+                    // Prisma docs: https://www.prisma.io/docs/reference/database-reference/connection-urls#special-characters
+                    // Exclude: " @ : / ? # [ ] % \ (all have special meaning in URLs or need escaping)
+                    ExcludeCharacters: '"@:/?#[]%\\\\',
                 },
                 Tags: [
                     { Key: 'Name', Value: '${self:service}-${self:provider.stage}-db-secret' },
@@ -288,7 +291,10 @@ class AuroraBuilder extends InfrastructureBuilder {
                         SecretStringTemplate: JSON.stringify({ username: dbConfig.username || 'postgres' }),
                         GenerateStringKey: 'password',
                         PasswordLength: 32,
-                        ExcludeCharacters: '"@/\\\\',
+                        // Exclude URL-special characters for Prisma connection string compatibility
+                        // Prisma docs: https://www.prisma.io/docs/reference/database-reference/connection-urls#special-characters
+                        // Exclude: " @ : / ? # [ ] % \ (all have special meaning in URLs or need escaping)
+                        ExcludeCharacters: '"@:/?#[]%\\\\',
                     },
                     Tags: [
                         { Key: 'Name', Value: '${self:service}-${self:provider.stage}-db-secret' },

package/infrastructure/domains/database/aurora-builder.test.js

@@ -572,7 +572,7 @@ describe('AuroraBuilder', () => {
                 expect(jsonOutput).not.toContain('[object Object]');
             });
 
-            it('should properly escape ExcludeCharacters for valid JSON in CloudFormation template', async () => {
+            it('should exclude URL-special characters from password generation for Prisma compatibility', async () => {
                 const appDefinition = {
                     database: {
                         postgres: {
@@ -591,18 +591,15 @@ describe('AuroraBuilder', () => {
                 const result = await auroraBuilder.build(appDefinition, discoveredResources);
 
                 const excludeChars = result.resources.FriggDBSecret.Properties.GenerateSecretString.ExcludeCharacters;
-
-                // Should properly escape the backslash so it's valid JSON
-                // In JavaScript string: '"@/\\' represents the string: "@/\
-                // When serialized to JSON, backslash must be doubled: '"@/\\'
-                expect(excludeChars).toBe('"@/\\\\');
-
+                
+                // Must exclude URL-special characters that would break Prisma connection strings
+                // Prisma docs: https://www.prisma.io/docs/reference/database-reference/connection-urls#special-characters
+                // These characters have special meaning in URLs and must be excluded or the password must be URL-encoded
+                // Exclude: " @ : / ? # [ ] % (and \ for JSON escaping)
+                expect(excludeChars).toBe('"@:/?#[]%\\\\');
+                
                 // Verify it can be JSON-stringified without errors
                 expect(() => JSON.stringify(result.resources.FriggDBSecret)).not.toThrow();
-
-                // Verify the JSON output has the correct escape sequence
-                const jsonOutput = JSON.stringify(result.resources.FriggDBSecret);
-                expect(jsonOutput).toContain('\\"@/\\\\\\\\');  // In JSON string: "\"@/\\\\"
             });
         });
     });

package/infrastructure/domains/database/migration-builder.test.js

@@ -183,31 +183,26 @@ describe('MigrationBuilder', () => {
 
             const result = await builder.build(appDef, {});
 
-            // Both migration functions should have package configs
+            // Both migration functions should have the same package config
             expect(result.functions.dbMigrationWorker.package).toBeDefined();
             expect(result.functions.dbMigrationRouter.package).toBeDefined();
 
-            // Check worker package config
-            const workerPackage = result.functions.dbMigrationWorker.package;
-            expect(workerPackage.individually).toBe(true);
-            expect(workerPackage.exclude).toBeDefined();
-            expect(Array.isArray(workerPackage.exclude)).toBe(true);
-            
-            // Verify critical exclusions for size optimization
-            expect(workerPackage.exclude).toContain('test/**');
-            expect(workerPackage.exclude).toContain('**/*.test.js');
-            expect(workerPackage.exclude).toContain('node_modules/**/node_modules/**');
-            expect(workerPackage.exclude).toContain('node_modules/esbuild/**');
-            expect(workerPackage.exclude).toContain('node_modules/typescript/**');
-            expect(workerPackage.exclude).toContain('node_modules/@friggframework/devtools/**');
-            expect(workerPackage.exclude).toContain('src/**'); // Migration handlers don't need backend source
-
-            // Check router package config
-            const routerPackage = result.functions.dbMigrationRouter.package;
-            expect(routerPackage.individually).toBe(true);
-            expect(routerPackage.exclude).toBeDefined();
-            expect(routerPackage.exclude).toContain('test/**');
-            expect(routerPackage.exclude).toContain('node_modules/**/node_modules/**');
+            // They should share the same config object (migrationPackageConfig)
+            expect(result.functions.dbMigrationWorker.package).toBe(result.functions.dbMigrationRouter.package);
+
+            // Verify the shared package config has critical exclusions for size optimization
+            const packageConfig = result.functions.dbMigrationWorker.package;
+            expect(packageConfig.individually).toBe(true);
+            expect(Array.isArray(packageConfig.exclude)).toBe(true);
+
+            // Critical exclusions to prevent Lambda size limit errors
+            expect(packageConfig.exclude).toContain('test/**');
+            expect(packageConfig.exclude).toContain('**/*.test.js');
+            expect(packageConfig.exclude).toContain('node_modules/**/node_modules/**');
+            expect(packageConfig.exclude).toContain('node_modules/esbuild/**');
+            expect(packageConfig.exclude).toContain('node_modules/typescript/**');
+            expect(packageConfig.exclude).toContain('node_modules/@friggframework/devtools/**');
+            expect(packageConfig.exclude).toContain('src/**'); // Migration handlers don't need backend source
         });
 
         it('should add queue URL to environment', async () => {

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.461.12ba2eb.0",
+  "version": "2.0.0--canary.461.499527f.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.12ba2eb.0",
-    "@friggframework/test": "2.0.0--canary.461.12ba2eb.0",
+    "@friggframework/schemas": "2.0.0--canary.461.499527f.0",
+    "@friggframework/test": "2.0.0--canary.461.499527f.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.12ba2eb.0",
-    "@friggframework/prettier-config": "2.0.0--canary.461.12ba2eb.0",
+    "@friggframework/eslint-config": "2.0.0--canary.461.499527f.0",
+    "@friggframework/prettier-config": "2.0.0--canary.461.499527f.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": "12ba2eb2888f4999e52cf4b8f6eadc9f1beacfb0"
+  "gitHead": "499527f5e554f19da6d79c9cfa9f3d2e77857467"
 }