@friggframework/core 2.0.0--canary.461.af4cde8.0 → 2.0.0--canary.461.74b99c9.0

package/database/config.js

@@ -4,13 +4,28 @@
  */
 
 /**
- * Determines database type from app definition
- * Reads backend/index.js Definition.database configuration
+ * Determines database type from DATABASE_URL or app definition
+ * 
+ * Detection order:
+ * 1. DATABASE_URL protocol (for migration handlers - avoids loading app definition)
+ * 2. App definition (backend/index.js Definition.database configuration)
  *
  * @returns {'mongodb'|'postgresql'} Database type
  * @throws {Error} If database type cannot be determined or app definition missing
  */
 function getDatabaseType() {
+    // First, try to detect from DATABASE_URL (migration handlers don't need app definition)
+    const databaseUrl = process.env.DATABASE_URL;
+    if (databaseUrl) {
+        if (databaseUrl.startsWith('postgresql://') || databaseUrl.startsWith('postgres://')) {
+            return 'postgresql';
+        }
+        if (databaseUrl.startsWith('mongodb://') || databaseUrl.startsWith('mongodb+srv://')) {
+            return 'mongodb';
+        }
+    }
+    
+    // Fallback: Load app definition
     try {
         const path = require('node:path');
         const fs = require('node:fs');

package/handlers/routers/db-migration.js

@@ -85,11 +85,13 @@ router.use(validateApiKey);
 router.post(
     '/db-migrate',
     catchAsyncError(async (req, res) => {
-        const { dbType, stage } = req.body;
+        // Migration infrastructure is PostgreSQL-only, so hardcode dbType
+        const dbType = 'postgresql';
+        const { stage } = req.body;
         // TODO: Extract userId from JWT token when auth is implemented
         const userId = req.body.userId || 'admin';
 
-        console.log(`Migration trigger request: dbType=${dbType}, stage=${stage}, userId=${userId}`);
+        console.log(`Migration trigger request: dbType=${dbType}, stage=${stage || 'auto-detect'}, userId=${userId}`);
 
         try {
             const result = await triggerMigrationUseCase.execute({

package/handlers/routers/db-migration.test.js

@@ -28,7 +28,7 @@ describe('Database Migration Router - Adapter Layer', () => {
     it('should load without requiring app definition (critical bug fix)', () => {
         // Before fix: createProcessRepository() → getDatabaseType() → loads app definition → requires integrations → CRASH
         // After fix: ProcessRepositoryPostgres instantiated directly → no app definition → SUCCESS
-
+        
         expect(() => {
             require('./db-migration');
         }).not.toThrow();
@@ -40,4 +40,12 @@ describe('Database Migration Router - Adapter Layer', () => {
         expect(typeof router).toBe('function');
         expect(router.stack).toBeDefined();
     });
+
+    it('should hardcode dbType as postgresql (migrations are PostgreSQL-only)', () => {
+        // Migration infrastructure is only created for PostgreSQL deployments
+        // So we can safely hardcode dbType instead of requiring it from request
+        const router = require('./db-migration').router;
+        expect(router).toBeDefined();
+        // Test will pass if handler doesn't crash when dbType is omitted from request
+    });
 });

package/handlers/workers/db-migration.js

@@ -104,32 +104,32 @@ function sanitizeDatabaseUrl(url) {
  */
 function extractMigrationParams(event) {
     let processId = null;
-    let dbType = null;
     let stage = null;
+    
+    // Migration infrastructure is PostgreSQL-only, so hardcode dbType
+    const dbType = 'postgresql';
 
     // Check if this is an SQS event
     if (event.Records && event.Records.length > 0) {
         // SQS event - extract from message body
         const message = JSON.parse(event.Records[0].body);
         processId = message.processId;
-        dbType = message.dbType;
         stage = message.stage;
 
         console.log('SQS event detected');
         console.log(`  Process ID: ${processId}`);
-        console.log(`  DB Type: ${dbType}`);
+        console.log(`  DB Type: ${dbType} (hardcoded - PostgreSQL-only)`);
         console.log(`  Stage: ${stage}`);
     } else {
         // Direct invocation - use event properties or environment variables
         processId = event.processId || null;
-        dbType = event.dbType || process.env.DB_TYPE || 'postgresql';
         stage = event.stage || process.env.STAGE || 'production';
 
         console.log('Direct invocation detected');
         if (processId) {
             console.log(`  Process ID: ${processId}`);
         }
-        console.log(`  DB Type: ${dbType}`);
+        console.log(`  DB Type: ${dbType} (hardcoded - PostgreSQL-only)`);
         console.log(`  Stage: ${stage}`);
     }
 

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.461.af4cde8.0",
+    "version": "2.0.0--canary.461.74b99c9.0",
     "dependencies": {
         "@aws-sdk/client-apigatewaymanagementapi": "^3.588.0",
         "@aws-sdk/client-kms": "^3.588.0",
@@ -37,9 +37,9 @@
         }
     },
     "devDependencies": {
-        "@friggframework/eslint-config": "2.0.0--canary.461.af4cde8.0",
-        "@friggframework/prettier-config": "2.0.0--canary.461.af4cde8.0",
-        "@friggframework/test": "2.0.0--canary.461.af4cde8.0",
+        "@friggframework/eslint-config": "2.0.0--canary.461.74b99c9.0",
+        "@friggframework/prettier-config": "2.0.0--canary.461.74b99c9.0",
+        "@friggframework/test": "2.0.0--canary.461.74b99c9.0",
         "@prisma/client": "^6.17.0",
         "@types/lodash": "4.17.15",
         "@typescript-eslint/eslint-plugin": "^8.0.0",
@@ -79,5 +79,5 @@
     "publishConfig": {
         "access": "public"
     },
-    "gitHead": "af4cde8379559b787e0a3e968c7a1c0b90823a5b"
+    "gitHead": "74b99c9d56dce5e4ea9f74f2db3a6087132578d3"
 }