Remove unused devDependencies from package.json and package-lock.json; update README.md to streamline document processing instructions.

Author: adamrefaey

backend/README.md

@@ -87,78 +87,4 @@ This script sets a resource policy that:
 1. Allows only authenticated Cognito users to access your API
 2. Denies any non-HTTPS requests to your API
 
-If you need more complex permissions, you can modify the policy object in the script.
-
-# Medical Document Processor Test Controller
-
-This module provides a test controller for uploading and processing medical documents using the DocumentProcessorService.
-
-## Available Endpoints
-
-### Test Form
-```
-GET /api/document-processor/test-form
-```
-Provides a simple HTML form for testing the document processor functionality:
-- Upload PDF or image files (JPEG, PNG, TIFF)
-- See extracted text and medical analysis results
-
-### Upload Document
-```
-POST /api/document-processor/upload
-```
-API endpoint for uploading and processing medical documents:
-- Accepts PDF or image files (JPEG, PNG, TIFF)
-- Returns extracted text and medical analysis
-
-**Request parameters:**
-- `file` - The file to process (multipart/form-data)
-- `userId` - Optional user ID for tracking/analytics (defaults to "test-user-id")
-
-**Response:**
-```json
-{
-  "extractedText": {
-    "rawText": "...",
-    "lines": [...],
-    "tables": [...],
-    "keyValuePairs": [...]
-  },
-  "analysis": {
-    "keyMedicalTerms": [...],
-    "labValues": [...],
-    "diagnoses": [...],
-    "metadata": {
-      "isMedicalReport": true,
-      "confidence": 0.95,
-      "missingInformation": []
-    }
-  },
-  "processingMetadata": {
-    "processingTimeMs": 2500,
-    "fileType": "application/pdf",
-    "fileSize": 245000
-  }
-}
-```
-
-### Test Status
-```
-GET /api/document-processor/test
-```
-Simple endpoint to verify the controller is working properly.
-
-## Testing
-
-To test the document processing functionality:
-
-1. Ensure your backend server is running
-2. Navigate to `http://localhost:3000/api/document-processor/test-form` in your browser
-3. Upload a medical document (PDF, JPEG, PNG, or TIFF)
-4. View the results of text extraction and medical analysis
-
-## Notes
-
-- Maximum file size: 10 MB
-- Supported file types: PDF, JPEG, PNG, TIFF
-- For testing purposes, authentication is bypassed for these endpoints
+If you need more complex permissions, you can modify the policy object in the script.

backend/src/app.module.spec.ts

@@ -5,10 +5,10 @@ import { JwtModule } from '@nestjs/jwt';
 import { ReportsService } from './reports/reports.service';
 import { vi, describe, it, expect } from 'vitest';
 import configuration from './config/configuration';
-import { AwsBedrockService } from './document-processor/services/aws-bedrock.service';
 import { PerplexityService } from './services/perplexity.service';
 import { AwsSecretsService } from './services/aws-secrets.service';
 import { AwsTextractService } from './document-processor/services/aws-textract.service';
+import { AwsBedrockService } from './document-processor/services/aws-bedrock.service';
 
 describe('AppModule', () => {
   it('should compile the module', async () => {
@@ -32,13 +32,6 @@ describe('AppModule', () => {
         findOne: vi.fn().mockResolvedValue({}),
         updateStatus: vi.fn().mockResolvedValue({}),
       })
-      .overrideProvider(AwsBedrockService)
-      .useValue({
-        listAvailableModels: vi.fn().mockResolvedValue({
-          models: [],
-          currentModelId: 'test-model-id',
-        }),
-      })
       .overrideProvider(PerplexityService)
       .useValue({
         askQuestion: vi.fn().mockResolvedValue({}),
@@ -52,6 +45,20 @@ describe('AppModule', () => {
         extractText: vi.fn().mockResolvedValue({}),
         processBatch: vi.fn().mockResolvedValue([]),
       })
+      .overrideProvider(AwsBedrockService)
+      .useValue({
+        generateResponse: vi.fn().mockResolvedValue('test response'),
+        analyzeMedicalDocument: vi.fn().mockResolvedValue({
+          keyMedicalTerms: [],
+          labValues: [],
+          diagnoses: [],
+          metadata: {
+            isMedicalReport: true,
+            confidence: 0.9,
+            missingInformation: [],
+          },
+        }),
+      })
       .compile();
 
     expect(module).toBeDefined();

backend/src/app.module.ts

@@ -1,4 +1,4 @@
-import { Module, NestModule, MiddlewareConsumer, RequestMethod } from '@nestjs/common';
+import { Module, NestModule, MiddlewareConsumer } from '@nestjs/common';
 import { ConfigModule } from '@nestjs/config';
 import configuration from './config/configuration';
 import { AppController } from './app.controller';
@@ -26,9 +26,6 @@ import { DocumentProcessorModule } from './document-processor/document-processor
 })
 export class AppModule implements NestModule {
   configure(consumer: MiddlewareConsumer) {
-    consumer
-      .apply(AuthMiddleware)
-      .exclude({ path: 'health', method: RequestMethod.GET })
-      .forRoutes('*');
+    consumer.apply(AuthMiddleware).forRoutes('*'); // Apply to all routes
   }
 }

backend/src/services/README.md

@@ -169,7 +169,7 @@ async processReport(fileBuffer: Buffer, fileType: string, userId: string) {
 ## Rate Limiting
 
 Both services implement rate limiting based on user ID:
-- AWS Textract: 10 document requests per minute by default (configurable)
+- AWS Textract: 20 document requests per minute by default (configurable)
 - AWS Bedrock: 20 model invocations per minute by default (configurable)
 
 ## Batch Processing

package-lock.json

@@ -10,8 +10,6 @@
     "build:frontend": "cd frontend && npm run build"
   },
   "devDependencies": {
-    "@nestjs/testing": "^11.0.13",
-    "@types/jest": "^29.5.14",
     "husky": "^9.1.7"
   }
 }