@friggframework/core 2.0.0--canary.461.e85a4f6.0 → 2.0.0--canary.461.97d93c9.0

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({
@@ -173,5 +175,21 @@ router.get(
     })
 );
 
-module.exports = router;
+// Minimal Lambda handler (avoids app-handler-helpers which loads core/index.js → user/**)
+const serverlessHttp = require('serverless-http');
+const express = require('express');
+const cors = require('cors');
+
+const app = express();
+app.use(cors());
+app.use(express.json());
+app.use(router);
+app.use((err, req, res, next) => {
+    console.error('Migration Router Error:', err);
+    res.status(500).json({ message: 'Internal Server Error' });
+});
+
+const handler = serverlessHttp(app);
+
+module.exports = { handler, router };
 

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

@@ -28,21 +28,24 @@ 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();
     });
 
-    it('should export Express router', () => {
-        const router = require('./db-migration');
+    it('should export handler and router', () => {
+        const { handler, router } = require('./db-migration');
+        expect(typeof handler).toBe('function');
         expect(typeof router).toBe('function');
         expect(router.stack).toBeDefined();
     });
 
-    it('should have separate handler file for Lambda', () => {
-        // db-migration.handler.js wraps router with createAppHandler
-        // This keeps db-migration.js free of app-handler-helpers dependency
-        expect(() => require('./db-migration.handler')).not.toThrow();
+    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.e85a4f6.0",
+    "version": "2.0.0--canary.461.97d93c9.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.e85a4f6.0",
-        "@friggframework/prettier-config": "2.0.0--canary.461.e85a4f6.0",
-        "@friggframework/test": "2.0.0--canary.461.e85a4f6.0",
+        "@friggframework/eslint-config": "2.0.0--canary.461.97d93c9.0",
+        "@friggframework/prettier-config": "2.0.0--canary.461.97d93c9.0",
+        "@friggframework/test": "2.0.0--canary.461.97d93c9.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": "e85a4f699f625a261b925af41d7d056476e504ef"
+    "gitHead": "97d93c9143829a56c166c50c1127961fba83bfa0"
 }