@friggframework/devtools 2.0.0--canary.461.2db86ce.0 → 2.0.0--canary.461.cdfbcf7.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: {
@@ -592,17 +592,14 @@ describe('AuroraBuilder', () => {
 
                 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.js

@@ -108,8 +108,9 @@ class MigrationBuilder extends InfrastructureBuilder {
                 '**/libquery_engine-darwin*',
                 '**/*-darwin-arm64*',
                 '**/*-darwin*',
-                '**/runtime/*.wasm',
-                '**/*.wasm*',
+                // Exclude WASM engines (but keep Prisma CLI WASM files in build/)
+                '**/runtime/*.wasm',  // Exclude query engine WASM (unused with binary engines)
+                // DO NOT exclude **/*.wasm* - Prisma CLI needs build/*.wasm files!
                 'src/**',
                 'test/**',
                 'layers/**',

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

@@ -186,7 +186,7 @@ describe('MigrationBuilder', () => {
             // Both migration functions should have the same package config
             expect(result.functions.dbMigrationWorker.package).toBeDefined();
             expect(result.functions.dbMigrationRouter.package).toBeDefined();
-            
+
             // They should share the same config object (migrationPackageConfig)
             expect(result.functions.dbMigrationWorker.package).toBe(result.functions.dbMigrationRouter.package);
 
@@ -194,7 +194,7 @@ describe('MigrationBuilder', () => {
             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');

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

@@ -333,9 +333,10 @@ function createBaseDefinition(
                         '**/*-darwin-arm64*',
                         '**/*-darwin*',
 
-                        // Exclude WASM engines
-                        '**/runtime/*.wasm',
-                        '**/*.wasm*',
+                        // Exclude WASM engines (but keep Prisma CLI WASM files in build/)
+                        // Note: Prisma CLI needs prisma/build/prisma_schema_build_bg.wasm for migrations
+                        '**/runtime/*.wasm',  // Exclude query engine WASM (unused with binary engines)
+                        // DO NOT exclude **/*.wasm* - Prisma CLI needs build/*.wasm files!
 
                         // Exclude backend source (dbMigrate doesn't need it)
                         'src/**',  // OK to exclude for migration handler only

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

@@ -106,6 +106,20 @@ describe('Base Definition Factory', () => {
             expect(result.functions.dbMigrate.layers).toBeUndefined(); // No Prisma layer
         });
 
+        it('should NOT exclude Prisma CLI WASM files from dbMigrate function', () => {
+            const result = createBaseDefinition({}, {}, {});
+
+            const excludeList = result.functions.dbMigrate.package.exclude;
+            
+            // Should exclude WASM files from runtime/ (query engine WASM)
+            expect(excludeList).toContain('**/runtime/*.wasm');
+            
+            // But should NOT exclude build/*.wasm files (Prisma CLI needs prisma_schema_build_bg.wasm)
+            expect(excludeList).not.toContain('**/*.wasm*');
+            expect(excludeList).not.toContain('**/*.wasm');
+            expect(excludeList).not.toContain('**/build/*.wasm');
+        });
+
         it('should include Prisma Lambda Layer', () => {
             const result = createBaseDefinition({}, {}, {});
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.461.2db86ce.0",
+  "version": "2.0.0--canary.461.cdfbcf7.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.2db86ce.0",
-    "@friggframework/test": "2.0.0--canary.461.2db86ce.0",
+    "@friggframework/schemas": "2.0.0--canary.461.cdfbcf7.0",
+    "@friggframework/test": "2.0.0--canary.461.cdfbcf7.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.2db86ce.0",
-    "@friggframework/prettier-config": "2.0.0--canary.461.2db86ce.0",
+    "@friggframework/eslint-config": "2.0.0--canary.461.cdfbcf7.0",
+    "@friggframework/prettier-config": "2.0.0--canary.461.cdfbcf7.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": "2db86ce72c7e8f37cfcf3576560288dd2322065e"
+  "gitHead": "cdfbcf7ed07ca84840c56a33d30d39a6c6b20628"
 }