@striae-org/striae 3.0.5 → 3.1.0

package/app/components/actions/case-export/core-export.ts

@@ -278,7 +278,7 @@ export async function exportCaseData(
             latestAnnotationDate = annotations.updatedAt;
           }
         }
-      } catch (error) {
+      } catch {
         // Continue without annotations for this file
       }
 

package/app/components/actions/case-export/download-handlers.ts

@@ -36,7 +36,7 @@ export async function downloadAllCasesAsJSON(user: User, exportData: AllCasesExp
   
   try {
     // Start audit workflow
-    const workflowId = auditService.startWorkflow('all-cases');
+    auditService.startWorkflow('all-cases');
     
     const dataStr = JSON.stringify(exportData, null, 2);
     
@@ -120,7 +120,7 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
   
   try {
     // Start audit workflow
-    const workflowId = auditService.startWorkflow('all-cases');
+    auditService.startWorkflow('all-cases');
     
     // Dynamic import of XLSX to avoid bundle size issues
     const XLSX = await import('xlsx');
@@ -207,7 +207,7 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
     XLSX.utils.book_append_sheet(workbook, summaryWorksheet, 'Summary');
 
     // Create a worksheet for each case
-    exportData.cases.forEach((caseData, index) => {
+    exportData.cases.forEach((caseData) => {
       if (caseData.summary?.exportError) {
         // For failed cases, create a simple error sheet
         const errorData = [
@@ -231,7 +231,7 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
       const metadataRows = generateMetadataRows(caseData);
       
       // Create case details with headers
-      const caseDetailsData = [
+      const caseDetailsData: Array<Array<string | number | boolean | null | undefined>> = [
         protectForensicData 
           ? [`CASE DATA - ${caseData.metadata.caseNumber} - PROTECTED`]
           : [`Case ${caseData.metadata.caseNumber} - Detailed Export`],
@@ -245,7 +245,7 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
 
       // Add file data if available
       if (caseData.files && caseData.files.length > 0) {
-        const fileRows: any[][] = [];
+        const fileRows: Array<Array<string | number | boolean | null | undefined>> = [];
         
         caseData.files.forEach(fileEntry => {
           const processedRows = processFileDataForTabular(fileEntry);
@@ -265,7 +265,7 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
       }
       
       // Clean sheet name for Excel compatibility
-      const sheetName = `Case_${caseData.metadata.caseNumber}`.replace(/[\\\/\?\*\[\]]/g, '_').substring(0, 31);
+      const sheetName = `Case_${caseData.metadata.caseNumber}`.replace(/[\\/?*\x5B\x5D]/g, '_').substring(0, 31);
       XLSX.utils.book_append_sheet(workbook, caseWorksheet, sheetName);
     });
 
@@ -303,8 +303,6 @@ export async function downloadAllCasesAsCSV(user: User, exportData: AllCasesExpo
     // Clean up
     window.URL.revokeObjectURL(url);
     
-    const passwordInfo = protectForensicData && exportPassword ? ` (Password: ${exportPassword})` : '';
-    
     // Log successful export audit event
     const endTime = Date.now();
     await auditService.logCaseExport(
@@ -366,7 +364,7 @@ export async function downloadCaseAsJSON(
   
   try {
     // Start audit workflow
-    const workflowId = auditService.startWorkflow(exportData.metadata.caseNumber);
+    auditService.startWorkflow(exportData.metadata.caseNumber);
     
     const jsonContent = await generateJSONContent(exportData, options.includeUserInfo, options.protectForensicData);
     const dataUri = 'data:application/json;charset=utf-8,' + encodeURIComponent(jsonContent);
@@ -445,7 +443,7 @@ export async function downloadCaseAsCSV(
   
   try {
     // Start audit workflow
-    const workflowId = auditService.startWorkflow(exportData.metadata.caseNumber);
+    auditService.startWorkflow(exportData.metadata.caseNumber);
     
     const csvContent = await generateCSVContent(exportData, options.protectForensicData);
     const dataUri = 'data:text/csv;charset=utf-8,' + encodeURIComponent(csvContent);
@@ -528,7 +526,7 @@ export async function downloadCaseAsZip(
   
   try {
     // Start audit workflow
-    const workflowId = auditService.startWorkflow(caseNumber);
+    auditService.startWorkflow(caseNumber);
     
     onProgress?.(10);
     
@@ -856,7 +854,7 @@ async function generateJSONContent(
   includeUserInfo: boolean = true, 
   protectForensicData: boolean = true
 ): Promise<string> {
-  let jsonData = { ...exportData };
+  const jsonData = { ...exportData };
   
   // Remove sensitive user info if not included
   if (!includeUserInfo) {

package/app/components/actions/case-export/metadata-helpers.ts

@@ -71,7 +71,7 @@ export function generateRandomPassword(): string {
 /**
  * Protect Excel worksheet from editing
  */
-export function protectExcelWorksheet(worksheet: any, sheetPassword?: string): string {
+export function protectExcelWorksheet(worksheet: Record<string, unknown>, sheetPassword?: string): string {
   // Generate random password if none provided
   const password = sheetPassword || generateRandomPassword();
   

package/app/components/actions/case-import/confirmation-import.ts

@@ -8,6 +8,21 @@ import { auditService } from '~/services/audit.service';
 
 const DATA_WORKER_URL = paths.data_worker_url;
 
+interface CaseDataFile {
+  id: string;
+  originalFilename?: string;
+}
+
+interface CaseDataResponse {
+  files?: CaseDataFile[];
+  originalImageIds?: Record<string, string>;
+}
+
+type AnnotationImportData = Record<string, unknown> & {
+  confirmationData?: unknown;
+  updatedAt?: string;
+};
+
 /**
  * Import confirmation data from JSON file
  */
@@ -98,7 +113,7 @@ export async function importConfirmationData(
       throw new Error(`Failed to fetch case data: ${caseResponse.status}`);
     }
 
-    const caseData = await caseResponse.json() as any; // Using any for flexibility with originalImageIds
+    const caseData = await caseResponse.json() as CaseDataResponse;
     
     // Build mapping from original image IDs to current image IDs
     const imageIdMapping = new Map<string, string>();
@@ -110,7 +125,7 @@ export async function importConfirmationData(
       }
     } else {
       // For regular cases, assume original IDs match current IDs
-      for (const file of caseData.files) {
+      for (const file of caseData.files || []) {
         imageIdMapping.set(file.id, file.id);
       }
     }
@@ -128,7 +143,7 @@ export async function importConfirmationData(
       }
 
       // Get the original filename for user-friendly messages
-      const currentFile = caseData.files.find((file: any) => file.id === currentImageId);
+      const currentFile = (caseData.files || []).find((file) => file.id === currentImageId);
       const displayFilename = currentFile?.originalFilename || currentImageId;
 
       // Get current annotation data for this image
@@ -139,22 +154,22 @@ export async function importConfirmationData(
         }
       });
 
-      let annotationData = {};
+      let annotationData: AnnotationImportData = {};
       if (annotationResponse.ok) {
-        annotationData = await annotationResponse.json();
+        annotationData = await annotationResponse.json() as AnnotationImportData;
       }
 
       // Check if confirmation data already exists
-      if ((annotationData as any).confirmationData) {
+      if (annotationData.confirmationData) {
         result.warnings?.push(`Image ${displayFilename} already has confirmation data - skipping`);
         continue;
       }
 
       // Validate that annotations haven't been modified after original export
       const importedConfirmationData = confirmations.length > 0 ? confirmations[0] : null;
-      if (importedConfirmationData && confirmationData.metadata.originalExportCreatedAt && (annotationData as any).updatedAt) {
+      if (importedConfirmationData && confirmationData.metadata.originalExportCreatedAt && annotationData.updatedAt) {
         const originalExportDate = new Date(confirmationData.metadata.originalExportCreatedAt);
-        const annotationUpdatedAt = new Date((annotationData as any).updatedAt);
+        const annotationUpdatedAt = new Date(annotationData.updatedAt);
         
         if (annotationUpdatedAt > originalExportDate) {
           // Format timestamps in user's timezone
@@ -305,7 +320,7 @@ export async function importConfirmationData(
     
     // First, try to extract basic metadata for audit purposes (if file is parseable)
     try {
-      const confirmationData: any = JSON.parse(await confirmationFile.text());
+      const confirmationData = JSON.parse(await confirmationFile.text()) as ConfirmationImportData;
       reviewingExaminerUidForAudit = confirmationData.metadata?.exportedByUid;
       totalConfirmationsForAudit = confirmationData.metadata?.totalConfirmations || 0;
       if (confirmationData.metadata?.signature) {

package/app/components/actions/case-import/orchestrator.ts

@@ -187,7 +187,7 @@ export async function importCaseForReview(
 
       if (!signatureResult.isValid) {
         throw new Error(
-          `Manifest signature validation failed: ${signatureResult.error || 'Unknown signature error'}. Import cannot proceed.`
+          'Manifest signature validation failed. Import cannot proceed.'
         );
       }
       
@@ -206,8 +206,7 @@ export async function importCaseForReview(
       
       if (!validation.isValid) {
         throw new Error(
-          `Comprehensive integrity validation failed: ${validation.summary}. ` +
-          `Errors: ${validation.errors.join(', ')}. Import cannot proceed.`
+          'Comprehensive integrity validation failed. Import cannot proceed.'
         );
       }
       
@@ -215,7 +214,7 @@ export async function importCaseForReview(
       onProgress?.(
         'Complete integrity verified',
         18,
-        `${validation.summary}. Signature verified${signatureKeyId ? ` (${signatureKeyId})` : ''}`
+        `Integrity validation passed. Signature verified${signatureKeyId ? ` (${signatureKeyId})` : ''}`
       );
       
     } else {

package/app/components/actions/case-import/validation.ts

@@ -100,7 +100,7 @@ export async function validateConfirmationHash(jsonContent: string, expectedHash
   try {
     // Validate input parameters
     if (!expectedHash || typeof expectedHash !== 'string') {
-      console.error('validateConfirmationHash: expectedHash is invalid:', expectedHash);
+      console.error('validateConfirmationHash: expected hash input is invalid');
       return false;
     }
 
@@ -126,8 +126,8 @@ export async function validateConfirmationHash(jsonContent: string, expectedHash
     }
     
     return actualHash.toUpperCase() === expectedHash.toUpperCase();
-  } catch (error) {
-    console.error('validateConfirmationHash: validation failed:', error);
+  } catch {
+    console.error('validateConfirmationHash: validation failed');
     return false;
   }
 }

package/app/components/actions/case-import/zip-processing.ts

@@ -43,35 +43,6 @@ function extractImageIdFromFilename(exportFilename: string): string | null {
   return filenameWithoutExt.substring(lastHyphenIndex + 1);
 }
 
-/**
- * Reconstruct original filename from export filename
- * Format: {originalFilename}-{id}.{extension} → {originalFilename}.{extension}
- * Example: "evidence-2b365c5e-0559-4d6a-564f-d40bf1770101.jpg" returns "evidence.jpg"
- */
-function reconstructOriginalFilename(exportFilename: string): string {
-  const lastDotIndex = exportFilename.lastIndexOf('.');
-  const extension = lastDotIndex === -1 ? '' : exportFilename.substring(lastDotIndex);
-  const filenameWithoutExt = lastDotIndex === -1 ? exportFilename : exportFilename.substring(0, lastDotIndex);
-  
-  // UUID pattern: 8-4-4-4-12 (36 chars including hyphens)
-  const uuidPattern = /^(.+)-([0-9a-f]{8}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{4}-[0-9a-f]{12})$/i;
-  const match = filenameWithoutExt.match(uuidPattern);
-  
-  if (match) {
-    return match[1] + extension; // Return the original filename part + extension
-  }
-  
-  // Fallback: remove everything after the last hyphen
-  const lastHyphenIndex = filenameWithoutExt.lastIndexOf('-');
-  
-  if (lastHyphenIndex === -1) {
-    return exportFilename; // No hyphen found, return as-is (backward compatibility)
-  }
-  
-  const originalBasename = filenameWithoutExt.substring(0, lastHyphenIndex);
-  return originalBasename + extension;
-}
-
 /**
  * Preview case information from ZIP file without importing
  */
@@ -84,8 +55,6 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
     // First, validate hash if forensic metadata exists
     let hashValid: boolean | undefined = undefined;
     let hashError: string | undefined = undefined;
-    let expectedHash: string | undefined = undefined;
-    let actualHash: string | undefined = undefined;
     let validationDetails: CaseImportPreview['validationDetails'];
     
     // Find the main data file (JSON or CSV)
@@ -132,7 +101,7 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
           const manifestForValidation = extractForensicManifestData(forensicManifest);
           if (!manifestForValidation) {
             hashValid = false;
-            hashError = 'Forensic manifest format is invalid or incomplete.';
+            hashError = 'Validation failed.';
 
             validationDetails = {
               hasForensicManifest: true,
@@ -143,8 +112,6 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
               integrityErrors: [hashError]
             };
           } else {
-            expectedHash = manifestForValidation.manifestHash;
-          
             // Extract image files for comprehensive validation
             const imageFiles: { [filename: string]: Blob } = {};
             const imagesFolder = zip.folder('images');
@@ -171,17 +138,16 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
             );
           
             hashValid = validation.isValid && signatureResult.isValid;
-            actualHash = validation.manifestValid ? expectedHash : 'validation_failed';
-          
+
             if (!hashValid) {
               const errorParts: string[] = [];
               if (!signatureResult.isValid) {
-                errorParts.push(`Signature validation failed: ${signatureResult.error}`);
+                errorParts.push('Signature validation failed.');
               }
               if (!validation.isValid) {
-                errorParts.push(`Comprehensive validation failed: ${validation.summary}. Errors: ${validation.errors.join(', ')}`);
+                errorParts.push('Integrity validation failed.');
               }
-              hashError = errorParts.join(' ');
+              hashError = errorParts.length > 0 ? errorParts.join(' ') : 'Validation failed.';
             }
 
             // Capture detailed validation information
@@ -206,7 +172,7 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
         } else {
           // No forensic manifest found - cannot validate
           hashValid = false;
-          hashError = 'No forensic manifest found. This case export does not support comprehensive integrity validation.';
+          hashError = 'Validation failed.';
           
           validationDetails = {
             hasForensicManifest: false,
@@ -215,8 +181,8 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
             integrityErrors: ['Export does not contain forensic manifest required for validation']
           };
         }
-      } catch (error) {
-        hashError = `Failed to validate forensic metadata: ${error instanceof Error ? error.message : 'Unknown error'}`;
+      } catch {
+        hashError = 'Validation failed.';
         hashValid = false;
         
         validationDetails = {
@@ -280,11 +246,9 @@ export async function previewCaseImport(zipFile: File, currentUser: User): Promi
       totalFiles,
       caseCreatedDate: caseData.metadata.caseCreatedDate,
       hasAnnotations: false, // We'll need to determine this during parsing if needed
-      validationSummary: hashValid ? 'Validation successful' : (hashError || 'Validation failed'),
+      validationSummary: hashValid ? 'Validation passed' : 'Validation failed',
       hashValid,
       hashError,
-      expectedHash,
-      actualHash,
       validationDetails
     };
     
@@ -301,7 +265,7 @@ export async function parseImportZip(zipFile: File, currentUser: User): Promise<
   caseData: CaseExportData;
   imageFiles: { [filename: string]: Blob };
   imageIdMapping: { [exportFilename: string]: string }; // exportFilename -> originalImageId
-  metadata?: any;
+  metadata?: Record<string, unknown>;
   cleanedContent?: string; // Add cleaned content for hash validation
 }> {
   // Dynamic import of JSZip to avoid bundle size issues
@@ -390,12 +354,12 @@ export async function parseImportZip(zipFile: File, currentUser: User): Promise<
     }
     
     // Extract forensic manifest if present
-    let metadata: any = undefined;
+    let metadata: Record<string, unknown> | undefined;
     const manifestFile = zip.file('FORENSIC_MANIFEST.json');
     
     if (manifestFile) {
       const manifestContent = await manifestFile.async('text');
-      metadata = { forensicManifest: JSON.parse(manifestContent) };
+      metadata = { forensicManifest: JSON.parse(manifestContent) as unknown };
     }
     
     return {

package/app/components/actions/case-manage.ts

@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-explicit-any */
 import { User } from 'firebase/auth';
 import { 
   canCreateCase, 

package/app/components/actions/confirm-export.ts

@@ -218,7 +218,7 @@ export async function exportConfirmationData(
     }
 
     // Try to get the forensic manifest createdAt timestamp from the original case export
-    let originalExportCreatedAt: string | undefined = forensicManifestCreatedAt;
+    const originalExportCreatedAt: string | undefined = forensicManifestCreatedAt;
     
     if (!originalExportCreatedAt) {
       console.warn(`No forensic manifest timestamp found for case ${caseNumber}. This case may have been imported before forensic linking was implemented, or the original export did not include a forensic manifest.`);
@@ -292,7 +292,7 @@ export async function exportConfirmationData(
     document.body.removeChild(a);
     URL.revokeObjectURL(url);
     
-    console.log(`Confirmation data exported for case ${caseNumber} with hash ${hash.toUpperCase()}`);
+    console.log(`Confirmation data exported for case ${caseNumber}`);
     
     // Log successful confirmation export
     const endTime = Date.now();

package/app/components/audit/user-audit-viewer.tsx

@@ -1,11 +1,18 @@
-import { useState, useEffect, useContext } from 'react';
+import { useState, useEffect, useContext, useCallback } from 'react';
 import { AuthContext } from '~/contexts/auth.context';
 import { auditService } from '~/services/audit.service';
 import { auditExportService } from '~/services/audit-export.service';
-import { ValidationAuditEntry, AuditAction, AuditResult, AuditTrail, UserData } from '~/types';
+import { ValidationAuditEntry, AuditAction, AuditResult, AuditTrail, UserData, WorkflowPhase } from '~/types';
 import { getUserData } from '~/utils/permissions';
 import styles from './user-audit.module.css';
 
+const isWorkflowPhase = (phase: unknown): phase is WorkflowPhase =>
+  phase === 'casework' ||
+  phase === 'case-export' ||
+  phase === 'case-import' ||
+  phase === 'confirmation' ||
+  phase === 'user-management';
+
 interface UserAuditViewerProps {
   isOpen: boolean;
   onClose: () => void;
@@ -30,13 +37,6 @@ export const UserAuditViewer = ({ isOpen, onClose, caseNumber, title }: UserAudi
   const [customEndDateInput, setCustomEndDateInput] = useState<string>('');
   const [auditTrail, setAuditTrail] = useState<AuditTrail | null>(null);
 
-  useEffect(() => {
-    if (isOpen && user) {
-      loadAuditData();
-      loadUserData();
-    }
-  }, [isOpen, user, dateRange, customStartDate, customEndDate, filterCaseNumber, caseNumber]);
-
   useEffect(() => {
     const handleEscape = (event: KeyboardEvent) => {
       if (event.key === 'Escape' && isOpen) {
@@ -52,7 +52,7 @@ export const UserAuditViewer = ({ isOpen, onClose, caseNumber, title }: UserAudi
     }
   }, [isOpen, onClose]);
 
-  const loadUserData = async () => {
+  const loadUserData = useCallback(async () => {
     if (!user) return;
     
     try {
@@ -62,9 +62,9 @@ export const UserAuditViewer = ({ isOpen, onClose, caseNumber, title }: UserAudi
       console.error('Failed to load user data:', error);
       // Don't set error state for user data failure, just log it
     }
-  };
+  }, [user]);
 
-  const loadAuditData = async () => {
+  const loadAuditData = useCallback(async () => {
     if (!user?.uid) return;
 
     setLoading(true);
@@ -137,7 +137,7 @@ export const UserAuditViewer = ({ isOpen, onClose, caseNumber, title }: UserAudi
             warningEvents: entries.filter(e => e.result === 'warning').length,
             workflowPhases: [...new Set(entries
               .map(e => e.details.workflowPhase)
-              .filter(Boolean))] as any[],
+              .filter(isWorkflowPhase))],
             participatingUsers: [...new Set(entries.map(e => e.userId))],
             startTimestamp: entries[entries.length - 1]?.timestamp || new Date().toISOString(),
             endTimestamp: entries[0]?.timestamp || new Date().toISOString(),
@@ -154,7 +154,21 @@ export const UserAuditViewer = ({ isOpen, onClose, caseNumber, title }: UserAudi
     } finally {
       setLoading(false);
     }
-  };
+  }, [
+    user,
+    dateRange,
+    customStartDate,
+    customEndDate,
+    caseNumber,
+    filterCaseNumber
+  ]);
+
+  useEffect(() => {
+    if (isOpen && user) {
+      loadAuditData();
+      loadUserData();
+    }
+  }, [isOpen, user, loadAuditData, loadUserData]);
 
   const handleApplyCaseFilter = () => {
     setFilterCaseNumber(caseNumberInput.trim());
@@ -449,10 +463,34 @@ Generated by Striae
   ).length;
   const loginSessions = auditEntries.filter(e => e.action === 'user-login').length;
 
+  const handleOverlayMouseDown = (event: React.MouseEvent<HTMLDivElement>) => {
+    if (event.target === event.currentTarget) {
+      onClose();
+    }
+  };
+
+  const handleOverlayKeyDown = (event: React.KeyboardEvent<HTMLDivElement>) => {
+    if (event.target !== event.currentTarget) {
+      return;
+    }
+
+    if (event.key === 'Enter' || event.key === ' ') {
+      event.preventDefault();
+      onClose();
+    }
+  };
+
   if (!isOpen) return null;
 
   return (
-    <div className={styles.overlay} onClick={(e) => e.target === e.currentTarget && onClose()}>
+    <div
+      className={styles.overlay}
+      onMouseDown={handleOverlayMouseDown}
+      onKeyDown={handleOverlayKeyDown}
+      role="button"
+      tabIndex={0}
+      aria-label="Close audit trail dialog"
+    >
       <div className={styles.modal}>
         <div className={styles.header}>
           <h2 className={styles.title}>

package/app/components/audit/user-audit.module.css

@@ -12,6 +12,7 @@
   justify-content: center;
   z-index: 1000;
   padding: 20px;
+  cursor: default;
 }
 
 .modal {
@@ -24,6 +25,7 @@
   box-shadow: 0 4px 20px rgba(0, 0, 0, 0.3);
   display: flex;
   flex-direction: column;
+  cursor: default;
 }
 
 .header {
@@ -100,7 +102,8 @@
 }
 
 /* Loading & Error States */
-.loading, .error {
+.loading,
+.error {
   text-align: center;
   padding: 40px 20px;
   color: var(--textBody);
@@ -117,8 +120,12 @@
 }
 
 @keyframes spin {
-  0% { transform: rotate(0deg); }
-  100% { transform: rotate(360deg); }
+  0% {
+    transform: rotate(0deg);
+  }
+  100% {
+    transform: rotate(360deg);
+  }
 }
 
 .error {
@@ -565,4 +572,4 @@
   padding: 32px 16px;
   color: var(--textLight);
   font-style: italic;
-}
+}