@friggframework/devtools 2.0.0--canary.454.726ab78.1 → 2.0.0--canary.454.cbce887.0

package/frigg-cli/utils/prisma-runner.js

@@ -191,11 +191,11 @@ async function runPrismaMigrate(command = 'dev', verbose = false) {
 
 /**
  * Runs Prisma db push for MongoDB
- * Interactive - will prompt user if data loss detected
  * @param {boolean} verbose - Enable verbose output
+ * @param {boolean} nonInteractive - Run in non-interactive mode (accepts data loss, for Lambda/CI)
  * @returns {Promise<Object>} { success: boolean, output?: string, error?: string }
  */
-async function runPrismaDbPush(verbose = false) {
+async function runPrismaDbPush(verbose = false, nonInteractive = false) {
     return new Promise((resolve) => {
         try {
             const schemaPath = getPrismaSchemaPath('mongodb');
@@ -209,20 +209,52 @@ async function runPrismaDbPush(verbose = false) {
                 '--skip-generate' // We generate separately
             ];
 
+            // Add non-interactive flag for Lambda/CI environments
+            if (nonInteractive) {
+                args.push('--accept-data-loss');
+            }
+
             if (verbose) {
                 console.log(chalk.gray(`Running: npx ${args.join(' ')}`));
             }
 
-            console.log(chalk.yellow('⚠️  Interactive mode: You may be prompted if schema changes cause data loss'));
+            if (nonInteractive) {
+                console.log(chalk.yellow('⚠️  Non-interactive mode: Data loss will be automatically accepted'));
+            } else {
+                console.log(chalk.yellow('⚠️  Interactive mode: You may be prompted if schema changes cause data loss'));
+            }
 
             const proc = spawn('npx', args, {
-                stdio: 'inherit', // Interactive mode - user can respond to prompts
+                stdio: nonInteractive ? 'pipe' : 'inherit', // Use pipe for non-interactive to capture output
                 env: {
                     ...process.env,
                     PRISMA_HIDE_UPDATE_MESSAGE: '1'
                 }
             });
 
+            let stdout = '';
+            let stderr = '';
+
+            // Capture output in non-interactive mode
+            if (nonInteractive) {
+                if (proc.stdout) {
+                    proc.stdout.on('data', (data) => {
+                        stdout += data.toString();
+                        if (verbose) {
+                            process.stdout.write(data);
+                        }
+                    });
+                }
+                if (proc.stderr) {
+                    proc.stderr.on('data', (data) => {
+                        stderr += data.toString();
+                        if (verbose) {
+                            process.stderr.write(data);
+                        }
+                    });
+                }
+            }
+
             proc.on('error', (error) => {
                 resolve({
                     success: false,
@@ -234,12 +266,13 @@ async function runPrismaDbPush(verbose = false) {
                 if (code === 0) {
                     resolve({
                         success: true,
-                        output: 'Database push completed successfully'
+                        output: nonInteractive ? stdout || 'Database push completed successfully' : 'Database push completed successfully'
                     });
                 } else {
                     resolve({
                         success: false,
-                        error: `Database push process exited with code ${code}`
+                        error: `Database push process exited with code ${code}`,
+                        output: stderr || stdout
                     });
                 }
             });

package/infrastructure/serverless-template.js

@@ -613,6 +613,20 @@ const createBaseDefinition = (AppDefinition, appEnvironmentVars, discoveredResou
                     { httpApi: { path: '/health/{proxy+}', method: 'GET' } },
                 ],
             },
+            dbMigrate: {
+                handler: 'node_modules/@friggframework/core/handlers/workers/db-migration.handler',
+                timeout: 300,  // 5 minutes for long-running migrations
+                memorySize: 512,  // Extra memory for Prisma CLI operations
+                reservedConcurrency: 1,  // Prevent concurrent migrations
+                description: 'Runs database migrations via Prisma (invoke manually from CI/CD)',
+                // No events - this function is invoked manually via AWS CLI
+                maximumEventAge: 60,  // Don't retry old migration requests (60 seconds)
+                maximumRetryAttempts: 0,  // Don't auto-retry failed migrations
+                tags: {
+                    Purpose: 'DatabaseMigration',
+                    ManagedBy: 'Frigg',
+                },
+            },
         },
         resources: {
             Resources: {

package/infrastructure/serverless-template.test.js

@@ -1762,6 +1762,97 @@ describe('composeServerlessDefinition', () => {
         });
     });
 
+    describe('Database Migration Lambda', () => {
+        it('should include dbMigrate function in all deployments', async () => {
+            const appDefinition = {
+                name: 'test-app',
+                integrations: []
+            };
+
+            const result = await composeServerlessDefinition(appDefinition);
+
+            // Check dbMigrate function exists
+            expect(result.functions.dbMigrate).toBeDefined();
+            expect(result.functions.dbMigrate.handler).toBe(
+                'node_modules/@friggframework/core/handlers/workers/db-migration.handler'
+            );
+        });
+
+        it('should configure dbMigrate with correct settings', async () => {
+            const appDefinition = {
+                integrations: []
+            };
+
+            const result = await composeServerlessDefinition(appDefinition);
+
+            const dbMigrate = result.functions.dbMigrate;
+
+            // Check timeout (5 minutes for long migrations)
+            expect(dbMigrate.timeout).toBe(300);
+
+            // Check memory allocation (extra for Prisma CLI)
+            expect(dbMigrate.memorySize).toBe(512);
+
+            // Check description
+            expect(dbMigrate.description).toContain('database migrations');
+            expect(dbMigrate.description).toContain('Prisma');
+        });
+
+        it('should not have HTTP events (manual invocation only)', async () => {
+            const appDefinition = {
+                integrations: []
+            };
+
+            const result = await composeServerlessDefinition(appDefinition);
+
+            const dbMigrate = result.functions.dbMigrate;
+
+            // Should have no events (manually invoked via AWS CLI)
+            expect(dbMigrate.events).toBeUndefined();
+        });
+
+        it('should include dbMigrate even with VPC enabled', async () => {
+            const appDefinition = {
+                vpc: {
+                    enable: true,
+                    management: 'discover'
+                },
+                integrations: []
+            };
+
+            const result = await composeServerlessDefinition(appDefinition);
+
+            // dbMigrate should exist with VPC configuration
+            expect(result.functions.dbMigrate).toBeDefined();
+
+            // Should have same VPC access as other functions
+            expect(result.provider.vpc).toBeDefined();
+        });
+
+        it('should include dbMigrate with database configuration', async () => {
+            const appDefinition = {
+                vpc: {
+                    enable: true,
+                    management: 'discover'
+                },
+                database: {
+                    postgres: {
+                        enable: true,
+                        management: 'discover'
+                    }
+                },
+                integrations: []
+            };
+
+            const result = await composeServerlessDefinition(appDefinition);
+
+            // dbMigrate should exist alongside database configuration
+            expect(result.functions.dbMigrate).toBeDefined();
+            expect(result.provider.environment.DB_TYPE).toBe('postgresql');
+            expect(result.provider.environment.DATABASE_URL).toBeDefined();
+        });
+    });
+
     describe('Edge Cases', () => {
         it('should handle empty app definition', async () => {
             const appDefinition = {};

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@friggframework/devtools",
   "prettier": "@friggframework/prettier-config",
-  "version": "2.0.0--canary.454.726ab78.1",
+  "version": "2.0.0--canary.454.cbce887.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.454.726ab78.1",
-    "@friggframework/test": "2.0.0--canary.454.726ab78.1",
+    "@friggframework/schemas": "2.0.0--canary.454.cbce887.0",
+    "@friggframework/test": "2.0.0--canary.454.cbce887.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.454.726ab78.1",
-    "@friggframework/prettier-config": "2.0.0--canary.454.726ab78.1",
+    "@friggframework/eslint-config": "2.0.0--canary.454.cbce887.0",
+    "@friggframework/prettier-config": "2.0.0--canary.454.cbce887.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": "726ab78ba26d816f9aadab7e26c1e0495da5c2e7"
+  "gitHead": "cbce8872aea93a375c1088a0dcb8d05a5d48d8b7"
 }