Add DELETE method for report and update UI handling for report deletion

Author: adamrefaey

backend/src/iac/backend-stack.ts

@@ -454,6 +454,18 @@ export class BackendStack extends cdk.Stack {
       },
     });
 
+    const deleteReportIntegration = new apigateway.Integration({
+      type: apigateway.IntegrationType.HTTP_PROXY,
+      integrationHttpMethod: 'DELETE',
+      uri: `${serviceUrl}/api/reports/{id}`,
+      options: {
+        ...integrationOptions,
+        requestParameters: {
+          'integration.request.path.id': 'method.request.path.id',
+        },
+      },
+    });
+
     const patchReportStatusIntegration = new apigateway.Integration({
       type: apigateway.IntegrationType.HTTP_PROXY,
       integrationHttpMethod: 'PATCH',
@@ -517,6 +529,13 @@ export class BackendStack extends cdk.Stack {
       },
     });
 
+    reportIdResource.addMethod('DELETE', deleteReportIntegration, {
+      ...methodOptions,
+      requestParameters: {
+        'method.request.path.id': true,
+      },
+    });
+
     reportStatusResource.addMethod('PATCH', patchReportStatusIntegration, {
       ...methodOptions,
       requestParameters: {

backend/src/reports/reports.controller.ts

@@ -10,6 +10,7 @@ import {
   UnauthorizedException,
   Post,
   NotFoundException,
+  Delete,
 } from '@nestjs/common';
 import {
   ApiTags,
@@ -192,6 +193,25 @@ export class ReportsController {
     return this.reportsService.saveReport(filePath, userId, originalFilename, fileSize);
   }
 
+  @ApiOperation({ summary: 'Delete a report' })
+  @ApiResponse({
+    status: 200,
+    description: 'Report deleted successfully',
+  })
+  @ApiResponse({
+    status: 404,
+    description: 'Report not found',
+  })
+  @ApiParam({
+    name: 'id',
+    description: 'Report ID',
+  })
+  @Delete(':id')
+  async deleteReport(@Param('id') id: string, @Req() request: RequestWithUser): Promise<void> {
+    const userId = this.extractUserId(request);
+    await this.reportsService.deleteReport(id, userId);
+  }
+
   private extractUserId(request: RequestWithUser): string {
     // The user object is attached to the request by our middleware
     const user = request.user;

frontend/src/pages/Reports/ReportDetailPage.tsx

@@ -80,12 +80,13 @@ const ReportDetailPage: React.FC = () => {
 
   // Handle close button
   const handleClose = () => {
-    history.goBack();
+    history.push('/tabs/home');
   };
 
   // Handle action buttons
-  const handleDiscard = () => {
-    history.goBack();
+  const handleDiscard = async () => {
+    await axios.delete(`${API_URL}/api/reports/${reportId}`, await getAuthConfig());
+    history.push('/tabs/home');
   };
 
   const handleNewUpload = () => {

frontend/src/pages/Reports/components/AiAnalysisTab.tsx

@@ -45,11 +45,13 @@ const AiAnalysisTab: React.FC<AiAnalysisTabProps> = ({
       {isLowConfidence && <LowConfidenceNotice />}
 
       {/* Flagged values section */}
-      <FlaggedValuesSection
-        flaggedValues={flaggedValues}
-        isExpanded={flaggedValuesExpanded}
-        onToggle={toggleFlaggedValues}
-      />
+      {flaggedValues && (
+        <FlaggedValuesSection
+          flaggedValues={flaggedValues}
+          isExpanded={flaggedValuesExpanded}
+          onToggle={toggleFlaggedValues}
+        />
+      )}
 
       {/* Normal values section */}
       <NormalValuesSection