@friggframework/core 2.0.0--canary.461.f753a98.0 → 2.0.0--canary.461.740089b.0

package/handlers/routers/db-migration.js

@@ -173,5 +173,9 @@ router.get(
     })
 );
 
-module.exports = router;
+// Export handler for Lambda (like auth.js and user.js)
+const { createAppHandler } = require('../app-handler-helpers');
+const handler = createAppHandler('HTTP Event: DB Migration', router, true);
+
+module.exports = { handler, router };
 

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

@@ -0,0 +1,42 @@
+/**
+ * Adapter Layer Tests - Database Migration Router
+ * 
+ * CRITICAL TEST: Verify handler loads without app definition
+ * 
+ * Business logic is tested in:
+ * - database/use-cases/trigger-database-migration-use-case.test.js (14 tests)
+ * - database/use-cases/get-migration-status-use-case.test.js (11 tests)
+ * 
+ * Following hexagonal architecture principles:
+ * - Handlers are thin adapters (HTTP → Use Case → HTTP)
+ * - Use cases contain all business logic (fully tested)
+ * - Repositories are infrastructure adapters (tested separately)
+ */
+
+process.env.ADMIN_API_KEY = 'test-admin-key';
+process.env.DB_MIGRATION_QUEUE_URL = 'https://sqs.test/queue';
+
+// Mock infrastructure dependencies to prevent app definition loading
+jest.mock('../../integrations/repositories/process-repository-postgres', () => ({
+    ProcessRepositoryPostgres: jest.fn(() => ({
+        create: jest.fn(),
+        findById: jest.fn(),
+    })),
+}));
+
+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');
+        expect(typeof router).toBe('function');
+        expect(router.stack).toBeDefined();
+    });
+});

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

@@ -0,0 +1,50 @@
+/**
+ * Adapter Layer Tests - Database Migration Worker
+ * 
+ * CRITICAL TEST: Verify handler loads without app definition
+ * 
+ * Business logic is tested in:
+ * - database/use-cases/run-database-migration-use-case.test.js (22 tests)
+ * 
+ * Following hexagonal architecture principles:
+ * - Handlers are thin adapters (SQS → Use Case → Response)
+ * - Use cases contain all business logic (fully tested)
+ * - Repositories are infrastructure adapters (tested separately)
+ */
+
+process.env.DATABASE_URL = 'postgresql://test:test@localhost:5432/test';
+process.env.STAGE = 'test';
+
+// Mock infrastructure dependencies to prevent app definition loading
+jest.mock('../../integrations/repositories/process-repository-postgres', () => ({
+    ProcessRepositoryPostgres: jest.fn(() => ({
+        create: jest.fn(),
+        findById: jest.fn(),
+        updateState: jest.fn(),
+    })),
+}));
+
+jest.mock('../../integrations/use-cases/update-process-state', () => ({
+    UpdateProcessState: jest.fn(() => ({ execute: jest.fn() })),
+}));
+
+jest.mock('../../database/utils/prisma-runner', () => ({
+    runMigration: jest.fn(),
+    deployMigration: jest.fn(),
+}));
+
+describe('Database Migration Worker - 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 handler function', () => {
+        const { handler } = require('./db-migration');
+        expect(typeof handler).toBe('function');
+    });
+});

package/package.json

@@ -1,7 +1,7 @@
 {
     "name": "@friggframework/core",
     "prettier": "@friggframework/prettier-config",
-    "version": "2.0.0--canary.461.f753a98.0",
+    "version": "2.0.0--canary.461.740089b.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.f753a98.0",
-        "@friggframework/prettier-config": "2.0.0--canary.461.f753a98.0",
-        "@friggframework/test": "2.0.0--canary.461.f753a98.0",
+        "@friggframework/eslint-config": "2.0.0--canary.461.740089b.0",
+        "@friggframework/prettier-config": "2.0.0--canary.461.740089b.0",
+        "@friggframework/test": "2.0.0--canary.461.740089b.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": "f753a98f8601dcb06cd579c918a9b132fce2d61b"
+    "gitHead": "740089b293858dc4a11a55a4bb65e87bbe765db0"
 }