Merge pull request #79 from alpsla/fix/v9-tool-parsers

Fix/v9 tool parsers

Author: alpsla

apps/api/src/__tests__/middleware/auth-middleware.test.ts

@@ -9,8 +9,9 @@ const mockSupabase = {
   eq: jest.fn().mockReturnThis()
 };
 
-jest.mock('@supabase/supabase-js', () => ({
-  createClient: jest.fn(() => mockSupabase)
+// Mock the database package's getSupabase function
+jest.mock('@codequal/database/supabase/client', () => ({
+  getSupabase: jest.fn(() => mockSupabase)
 }));
 
 import { Request, Response, NextFunction } from 'express';
@@ -24,7 +25,8 @@ describe('Authentication Middleware', () => {
 
   beforeEach(() => {
     mockRequest = {
-      headers: {}
+      headers: {},
+      path: '/api/test'  // Default path to avoid undefined errors
     } as any;
 
     mockResponse = {

apps/api/src/index.ts

@@ -36,6 +36,7 @@ import usageStatsRoutes from './routes/usage-stats';
 import researcherRoutes from './routes/researcher';
 import metricsRoutes from './routes/metrics';
 import monitoringPublicRoutes from './routes/monitoring-public';
+import v9AnalyzeRoutes from './routes/v9-analyze';
 import { errorHandler } from './middleware/error-handler';
 // import { i18nMiddleware, translateResponse, validateLanguage } from './middleware/i18n-middleware';
 import { requestLogger } from './middleware/request-logger';
@@ -214,6 +215,12 @@ app.use('/stripe', stripeWebhookRoutes);
 // Note: This must come before the auth middleware to allow public access
 app.use('/api/monitoring/public', monitoringPublicRoutes);
 
+// V9 Unified Analysis API (for development/testing - no auth required)
+// TODO: Add proper authentication before production deployment
+if (process.env.NODE_ENV !== 'production') {
+  app.use('/api/v9', v9AnalyzeRoutes);
+  logger.info('V9 API routes mounted at /api/v9 (no auth - dev mode)');
+}
 
 // Internal API routes (requires user authentication)
 app.use('/api', authMiddleware);

apps/api/src/middleware/auth-middleware.ts

@@ -34,6 +34,12 @@ export const authMiddleware = async (
       return next();
     }
 
+    // Skip auth for V9 routes in development mode
+    // Note: V9 routes have their own auth when needed
+    if (process.env.NODE_ENV !== 'production' && (req.path.startsWith('/v9') || req.baseUrl?.includes('/v9'))) {
+      return next();
+    }
+
     // Temporary: Allow internal researcher testing
     if (req.path === '/researcher/research' && req.headers['x-internal-test'] === 'true') {
       (req as any).user = {