@striae-org/striae 4.1.0 → 4.2.1

package/app/components/sidebar/cases/case-sidebar.tsx

@@ -1,53 +1,29 @@
 import type { User } from 'firebase/auth';
-import type * as CaseExportActions from '../../actions/case-export';
 import { useState, useEffect, useMemo, useCallback } from 'react';
 import styles from './cases.module.css';
-import { Toast } from '~/components/toast/toast';
-import { CasesModal } from './cases-modal';
 import { FilesModal } from '../files/files-modal';
-import { CaseExport, type ExportFormat } from '../case-export/case-export';
 import { ImageUploadZone } from '../upload/image-upload-zone';
-import { UserAuditViewer } from '~/components/audit/user-audit-viewer';
-import {
-  validateCaseNumber,
-  checkExistingCase,
-  createNewCase,
-  renameCase,
-  deleteCase,
-} from '../../actions/case-manage';
 import {
   fetchFiles,
   deleteFile,
 } from '../../actions/image-manage';
 import { 
-  checkReadOnlyCaseExists 
-} from '../../actions/case-review';
-import { 
-  canCreateCase, 
   canUploadFile, 
-  getLimitsDescription,
-  getUserData,
-  getFileAnnotations
+  ensureCaseConfirmationSummary,
+  getCaseConfirmationSummary
 } from '~/utils/data';
-import { type FileData, type CaseActionType } from '~/types';
+import { type FileData } from '~/types';
 
 interface CaseSidebarProps {
   user: User;
   onImageSelect: (file: FileData) => void;
-  onCaseChange: (caseNumber: string) => void;
+  onOpenCase: () => void;
   imageLoaded: boolean;
   setImageLoaded: (loaded: boolean) => void;
   onNotesClick: () => void;
   files: FileData[];
   setFiles: React.Dispatch<React.SetStateAction<FileData[]>>;
-  caseNumber: string;
-  setCaseNumber: (caseNumber: string) => void;
   currentCase: string | null;
-  setCurrentCase: (caseNumber: string) => void;
-  error: string;
-  setError: (error: string) => void;
-  successAction: CaseActionType;
-  setSuccessAction: (action: CaseActionType) => void;
   isReadOnly?: boolean;
   isConfirmed?: boolean;
   confirmationSaveVersion?: number;
@@ -57,37 +33,16 @@ interface CaseSidebarProps {
   onUploadComplete?: (result: { successCount: number; failedFiles: string[] }) => void;
 }
 
-const SUCCESS_MESSAGE_TIMEOUT = 3000;
-
-type CaseExportActionsModule = typeof CaseExportActions;
-
-let caseExportActionsPromise: Promise<CaseExportActionsModule> | null = null;
-
-const loadCaseExportActions = (): Promise<CaseExportActionsModule> => {
-  if (!caseExportActionsPromise) {
-    caseExportActionsPromise = import('../../actions/case-export');
-  }
-
-  return caseExportActionsPromise;
-};
-
 export const CaseSidebar = ({ 
   user, 
   onImageSelect, 
-  onCaseChange,
+  onOpenCase,
   imageLoaded,
   setImageLoaded,
   onNotesClick,
   files,
   setFiles,
-  caseNumber,
-  setCaseNumber,
   currentCase,
-  setCurrentCase,
-  error,
-  setError,
-  successAction,
-  setSuccessAction,
   isReadOnly = false,
   isConfirmed = false,
   confirmationSaveVersion = 0,
@@ -97,27 +52,11 @@ export const CaseSidebar = ({
   onUploadComplete
 }: CaseSidebarProps) => {
   
-  const [isDeletingCase, setIsDeletingCase] = useState(false);
-  const [isRenaming, setIsRenaming] = useState(false);
-  const [isLoading, setIsLoading] = useState(false);
-  const [isModalOpen, setIsModalOpen] = useState(false);
   const [, setFileError] = useState('');
-  const [newCaseName, setNewCaseName] = useState('');
-  const [showCaseActions, setShowCaseActions] = useState(false);
-  const [showCaseManagement, setShowCaseManagement] = useState(false);
-  const [canCreateNewCase, setCanCreateNewCase] = useState(true);
-  const [isToastVisible, setIsToastVisible] = useState(false);
-  const [toastMessage, setToastMessage] = useState('');
-  const [toastType, setToastType] = useState<'success' | 'error' | 'warning'>('success');
   const [canUploadNewFile, setCanUploadNewFile] = useState(true);
-  const [createCaseError, setCreateCaseError] = useState('');
   const [uploadFileError, setUploadFileError] = useState('');
-  const [limitsDescription, setLimitsDescription] = useState('');
-  const [permissionChecking, setPermissionChecking] = useState(false);
-  const [isExportModalOpen, setIsExportModalOpen] = useState(false);
   const [isFilesModalOpen, setIsFilesModalOpen] = useState(false);
   const [deletingFileId, setDeletingFileId] = useState<string | null>(null);
-  const [isAuditTrailOpen, setIsAuditTrailOpen] = useState(false);
   const [fileConfirmationStatus, setFileConfirmationStatus] = useState<{
     [fileId: string]: { includeConfirmation: boolean; isConfirmed: boolean }
   }>({});
@@ -131,45 +70,6 @@ export const CaseSidebar = ({
     [files]
   );
 
-  const calculateCaseConfirmationStatus = useCallback((
-    statuses: { [fileId: string]: { includeConfirmation: boolean; isConfirmed: boolean } }
-  ) => {
-    const filesRequiringConfirmation = files
-      .map((file) => statuses[file.id] || { includeConfirmation: false, isConfirmed: false })
-      .filter((status) => status.includeConfirmation);
-
-    const allConfirmedFiles = filesRequiringConfirmation.every((status) => status.isConfirmed);
-
-    return {
-      includeConfirmation: filesRequiringConfirmation.length > 0,
-      isConfirmed: filesRequiringConfirmation.length > 0 ? allConfirmedFiles : false,
-    };
-  }, [files]);
-
-  // Function to check user permissions (extracted for reuse)
-  const checkUserPermissions = useCallback(async () => {
-    setPermissionChecking(true);
-    try {
-      const casePermission = await canCreateCase(user);
-      setCanCreateNewCase(casePermission.canCreate);
-      setCreateCaseError(casePermission.reason || '');
-
-      // Only show limits description for restricted accounts
-      const userData = await getUserData(user);
-      if (userData && !userData.permitted) {
-        const description = await getLimitsDescription(user);
-        setLimitsDescription(description);
-      } else {
-        setLimitsDescription(''); // Clear the description for permitted users
-      }
-    } catch (error) {
-      console.error('Error checking user permissions:', error);
-      setCreateCaseError('Unable to verify account permissions');
-    } finally {
-      setPermissionChecking(false);
-    }
-  }, [user]);
-
   // Function to check file upload permissions (extracted for reuse)
   const checkFileUploadPermissions = useCallback(async (fileCount?: number) => {
     if (currentCase) {
@@ -190,11 +90,6 @@ export const CaseSidebar = ({
     }
   }, [currentCase, files.length, user]);
 
-  // Check user permissions on mount and when user changes
-  useEffect(() => {
-    checkUserPermissions();
-  }, [checkUserPermissions]);
-
   // Check file upload permissions when currentCase or files change
   useEffect(() => {
     checkFileUploadPermissions();
@@ -202,7 +97,6 @@ export const CaseSidebar = ({
    
   useEffect(() => {
     if (currentCase) {
-      setIsLoading(true);
       fetchFiles(user, currentCase, { skipValidation: true })
         .then(loadedFiles => {
           setFiles(loadedFiles);
@@ -210,9 +104,6 @@ export const CaseSidebar = ({
         .catch(err => {
           console.error('Failed to load files:', err);
           setFileError(err instanceof Error ? err.message : 'Failed to load files');
-        })
-        .finally(() => {
-          setIsLoading(false);
         });
     } else {
       setFiles([]);
@@ -232,43 +123,24 @@ export const CaseSidebar = ({
         return;
       }
 
-      // Fetch all annotations in parallel
-      const annotationPromises = files.map(async (file) => {
-        try {
-          const annotations = await getFileAnnotations(user, currentCase, file.id);
-          return {
-            fileId: file.id,
-            includeConfirmation: annotations?.includeConfirmation ?? false,
-            isConfirmed: !!annotations?.confirmationData,
-          };
-        } catch (err) {
-          console.error(`Error fetching annotations for file ${file.id}:`, err);
-          return {
-            fileId: file.id,
-            includeConfirmation: false,
-            isConfirmed: false,
-          };
-        }
+      const caseSummary = await ensureCaseConfirmationSummary(user, currentCase, files).catch((error) => {
+        console.error(`Error fetching confirmation summary for case ${currentCase}:`, error);
+        return null;
       });
 
-      // Wait for all fetches to complete
-      const results = await Promise.all(annotationPromises);
-
-      // Build the statuses map from results
-      const statuses: { [fileId: string]: { includeConfirmation: boolean; isConfirmed: boolean } } = {};
-      results.forEach((result) => {
-        statuses[result.fileId] = {
-          includeConfirmation: result.includeConfirmation,
-          isConfirmed: result.isConfirmed,
-        };
-      });
+      if (!caseSummary) {
+        return;
+      }
 
       if (isCancelled) {
         return;
       }
 
-      setFileConfirmationStatus(statuses);
-      setCaseConfirmationStatus(calculateCaseConfirmationStatus(statuses));
+      setFileConfirmationStatus(caseSummary.filesById);
+      setCaseConfirmationStatus({
+        includeConfirmation: caseSummary.includeConfirmation,
+        isConfirmed: caseSummary.isConfirmed
+      });
     };
 
     fetchConfirmationStatuses();
@@ -276,7 +148,7 @@ export const CaseSidebar = ({
     return () => {
       isCancelled = true;
     };
-  }, [currentCase, fileIdsKey, user, files, calculateCaseConfirmationStatus]);
+  }, [currentCase, fileIdsKey, user, files]);
 
   // Refresh only selected file confirmation status after confirmation-related data is persisted
   useEffect(() => {
@@ -288,24 +160,18 @@ export const CaseSidebar = ({
       }
 
       try {
-        const annotations = await getFileAnnotations(user, currentCase, selectedFileId);
-        const selectedStatus = {
-          includeConfirmation: annotations?.includeConfirmation ?? false,
-          isConfirmed: !!annotations?.confirmationData,
-        };
+        const caseSummary =
+          await getCaseConfirmationSummary(user, currentCase) ||
+          await ensureCaseConfirmationSummary(user, currentCase, files);
 
         if (isCancelled) {
           return;
         }
 
-        setFileConfirmationStatus((previous) => {
-          const next = {
-            ...previous,
-            [selectedFileId]: selectedStatus,
-          };
-
-          setCaseConfirmationStatus(calculateCaseConfirmationStatus(next));
-          return next;
+        setFileConfirmationStatus(caseSummary.filesById);
+        setCaseConfirmationStatus({
+          includeConfirmation: caseSummary.includeConfirmation,
+          isConfirmed: caseSummary.isConfirmed
         });
       } catch (err) {
         console.error(`Error refreshing confirmation status for file ${selectedFileId}:`, err);
@@ -317,90 +183,7 @@ export const CaseSidebar = ({
     return () => {
       isCancelled = true;
     };
-  }, [currentCase, fileIdsKey, user, selectedFileId, confirmationSaveVersion, files.length, calculateCaseConfirmationStatus]);
-
-  useEffect(() => {
-    if (error) {
-      setToastMessage(error);
-      setToastType('error');
-      setIsToastVisible(true);
-    }
-  }, [error]);
-
-  useEffect(() => {
-    if (successAction) {
-      setToastMessage(`Case ${currentCase} ${successAction} successfully!`);
-      setToastType('success');
-      setIsToastVisible(true);
-    }
-    // currentCase intentionally omitted: we capture its value at the time successAction changes
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [successAction]);
-  
-  const handleCase = async () => {
-    setIsLoading(true);
-    setError('');
-    setCreateCaseError(''); // Clear permission errors when starting new operation
-    
-    if (!validateCaseNumber(caseNumber)) {
-      setError('Invalid case number format');
-      setIsLoading(false);
-      return;
-    }
-
-    try {
-      const existingCase = await checkExistingCase(user, caseNumber);
-      
-      if (existingCase) {
-        // Loading existing case - always allowed
-        setCurrentCase(caseNumber);
-        onCaseChange(caseNumber);
-        const files = await fetchFiles(user, caseNumber, { skipValidation: true });
-        setFiles(files);
-        setCaseNumber('');
-        setSuccessAction('loaded');
-        setShowCaseManagement(false);
-        setTimeout(() => setSuccessAction(null), SUCCESS_MESSAGE_TIMEOUT);
-        return;
-      }
-
-      // Check if a read-only case with this number exists
-      const existingReadOnlyCase = await checkReadOnlyCaseExists(user, caseNumber);
-      if (existingReadOnlyCase) {
-        setError(`Case "${caseNumber}" already exists as a read-only review case. You cannot create a case with the same number.`);
-        setIsLoading(false);
-        return;
-      }
-
-      // Creating new case - check permissions
-      if (!canCreateNewCase) {
-        setError(createCaseError || 'You cannot create more cases.');
-        setCreateCaseError(''); // Clear duplicate error
-        setIsLoading(false);
-        return;
-      }
-
-      const newCase = await createNewCase(user, caseNumber);
-      setCurrentCase(newCase.caseNumber);
-      onCaseChange(newCase.caseNumber);
-      setFiles([]);
-      setCaseNumber('');
-      setSuccessAction('created');
-      setShowCaseManagement(false);
-      setTimeout(() => setSuccessAction(null), SUCCESS_MESSAGE_TIMEOUT);
-      
-      // Refresh permissions after successful case creation
-      // This updates the UI for users with limited permissions
-      await checkUserPermissions();
-    } catch (err) {
-      setError(err instanceof Error ? err.message : 'Failed to load/create case');
-      console.error(err);
-    } finally {
-      setIsLoading(false);
-    }
-  };
-
-
+  }, [currentCase, fileIdsKey, user, selectedFileId, confirmationSaveVersion, files]);
 
   const handleFileDelete = async (fileId: string) => {
     // Don't allow file deletion for read-only cases
@@ -414,11 +197,15 @@ export const CaseSidebar = ({
     setDeletingFileId(fileId);
     
     try {
-      await deleteFile(user, currentCase, fileId);
+      const deleteResult = await deleteFile(user, currentCase, fileId);
       const updatedFiles = files.filter(f => f.id !== fileId);
       setFiles(updatedFiles);      
       onImageSelect({ id: 'clear', originalFilename: '/clear.jpg', uploadedAt: '' });
       setImageLoaded(false);
+
+      if (deleteResult.imageMissing) {
+        setFileError(`File record deleted. Image asset "${deleteResult.fileName}" was not found and was skipped.`);
+      }
       
       // Refresh file upload permissions after successful file deletion
       // Pass the new file count directly to avoid state update timing issues
@@ -430,79 +217,6 @@ export const CaseSidebar = ({
     }
   };
 
-  const handleRenameCase = async () => {
-  // Don't allow renaming read-only cases
-  if (isReadOnly) {
-    return;
-  }
-
-  if (!currentCase || !newCaseName) return;
-  
-  if (!validateCaseNumber(newCaseName)) {
-    setError('Invalid new case number format');
-    return;
-  }
-  
-  setIsRenaming(true);
-  setError('');
-  
-  try {
-    // Check if a read-only case with the new name exists
-    const existingReadOnlyCase = await checkReadOnlyCaseExists(user, newCaseName);
-    if (existingReadOnlyCase) {
-      setError(`Case "${newCaseName}" already exists as a read-only review case. You cannot rename to this case number.`);
-      setIsRenaming(false);
-      return;
-    }
-
-    await renameCase(user, currentCase, newCaseName);
-    setCurrentCase(newCaseName);
-    onCaseChange(newCaseName);
-    setNewCaseName('');
-    setSuccessAction('loaded');
-    setTimeout(() => setSuccessAction(null), SUCCESS_MESSAGE_TIMEOUT);
-  } catch (err) {
-    setError(err instanceof Error ? err.message : 'Failed to rename case');
-  } finally {
-    setIsRenaming(false);
-  }
-};
-
-  const handleDeleteCase = async () => {
-  // Don't allow deleting read-only cases
-  if (isReadOnly) {
-    return;
-  }
-
-  if (!currentCase) return;
-  
-  const confirmed = window.confirm(
-    `Are you sure you want to delete case ${currentCase}? This will permanently delete all associated files and cannot be undone.`
-  );
-  
-  if (!confirmed) return;
-  
-  setIsDeletingCase(true);
-  setError('');
-  
-  try {
-    await deleteCase(user, currentCase);
-    setCurrentCase('');
-    onCaseChange('');
-    setFiles([]);
-    setSuccessAction('deleted');
-    setTimeout(() => setSuccessAction(null), SUCCESS_MESSAGE_TIMEOUT);
-    
-    // Refresh permissions after successful case deletion
-    // This allows users with limited permissions to create a new case
-    await checkUserPermissions();
-  } catch (err) {
-    setError(err instanceof Error ? err.message : 'Failed to delete case');
-  } finally {
-    setIsDeletingCase(false);
-  }
-};
-
 const handleImageSelect = (file: FileData) => {
     onImageSelect(file);
     // Prevent notes from opening against stale image state while selection loads.
@@ -539,155 +253,10 @@ const handleImageSelect = (file: FileData) => {
     ? 'Select an image first'
     : undefined;
 
-  const handleExport = async (exportCaseNumber: string, format: ExportFormat, includeImages?: boolean, onProgress?: (progress: number, label: string) => void) => {
-    try {
-      const caseExportActions = await loadCaseExportActions();
-
-      if (includeImages) {
-        // ZIP export with images - only available for single case exports
-        await caseExportActions.downloadCaseAsZip(user, exportCaseNumber, format, (progress) => {
-          const label = progress < 30 ? 'Loading case data' :
-                        progress < 50 ? 'Preparing archive' :
-                        progress < 80 ? 'Adding images' :
-                        progress < 96 ? 'Finalizing' : 'Downloading';
-          onProgress?.(Math.round(progress), label);
-        });
-      } else {
-        // Standard data-only export
-        onProgress?.(5, 'Loading case data');
-        const exportData = await caseExportActions.exportCaseData(
-          user,
-          exportCaseNumber,
-          { includeMetadata: true },
-          (current, total, label) => {
-            const p = total > 0 ? Math.round(10 + (current / total) * 60) : 10;
-            onProgress?.(p, label);
-          }
-        );
-        onProgress?.(75, 'Preparing download');
-
-        // Download the exported data in the selected format
-        if (format === 'json') {
-          await caseExportActions.downloadCaseAsJSON(user, exportData);
-        } else {
-          await caseExportActions.downloadCaseAsCSV(user, exportData);
-        }
-        onProgress?.(100, 'Complete');
-      }
-      
-    } catch (error) {
-      console.error('Export failed:', error);
-      throw error; // Re-throw to be handled by the modal
-    }
-  };
-
-  const handleExportAll = async (onProgress: (current: number, total: number, caseName: string) => void, format: ExportFormat) => {
-    try {
-      const caseExportActions = await loadCaseExportActions();
-
-      // Export all cases with progress callback
-      const exportData = await caseExportActions.exportAllCases(user, {
-        includeMetadata: true
-      }, onProgress);
-      
-      // Download the exported data in the selected format
-      if (format === 'json') {
-        await caseExportActions.downloadAllCasesAsJSON(user, exportData);
-      } else {
-        await caseExportActions.downloadAllCasesAsCSV(user, exportData);
-      }
-      
-    } catch (error) {
-      console.error('Export all failed:', error);
-      throw error; // Re-throw to be handled by the modal
-    }
-  };
-
 return (
     <>
     <div className={styles.caseSection}>
-        {currentCase && !showCaseManagement ? (
-          <div className={`${styles.caseLoad} mb-4`}>
-            <button
-              className={styles.switchCaseButton}
-              onClick={() => setShowCaseManagement(true)}
-              disabled={isUploading}
-              title={isUploading ? "Cannot switch cases while uploading files" : undefined}
-            >
-              Switch Case
-            </button>
-          </div>
-        ) : (
-          <>
-            <h4>Case Management</h4>
-            {limitsDescription && (
-              <p className={styles.limitsInfo}>
-                {limitsDescription}
-              </p>
-            )}
-            <div className={`${styles.caseInput} mb-4`}>
-              <input
-                type="text"
-                value={caseNumber}
-                onChange={(e) => setCaseNumber(e.target.value)}
-                placeholder="Case #"
-              />
-            </div>
-            <div className={`${styles.caseLoad} mb-4`}>
-              <button
-                onClick={handleCase}
-                disabled={isLoading || !caseNumber || permissionChecking || (isReadOnly && !!currentCase) || isUploading}
-                title={
-                  isUploading
-                    ? "Cannot load/create cases while uploading files"
-                    : (isReadOnly && currentCase)
-                    ? "Cannot load/create cases while reviewing a read-only case. Clear the current case first."
-                    : (!canCreateNewCase ? createCaseError : undefined)
-                }
-              >
-                {isLoading ? 'Loading...' : permissionChecking ? 'Checking permissions...' : 'Load/Create Case'}
-              </button>
-            </div>
-            <div className={styles.caseInput}>
-              <button
-                onClick={() => setIsModalOpen(true)}
-                className={styles.listButton}
-                disabled={isUploading}
-                title={isUploading ? "Cannot list cases while uploading files" : undefined}
-              >
-                List All Cases
-              </button>
-            </div>
-            {currentCase && (
-              <div className="mb-4">
-                <button
-                  className={styles.cancelSwitchButton}
-                  onClick={() => setShowCaseManagement(false)}
-                  disabled={isUploading}
-                >
-                  Cancel
-                </button>
-              </div>
-            )}
-          </>
-        )}
-    <CasesModal
-        isOpen={isModalOpen}
-        onClose={() => setIsModalOpen(false)}
-        onSelectCase={setCaseNumber}
-        currentCase={currentCase || ''}
-        user={user}
-      />
-      
-      <CaseExport
-        isOpen={isExportModalOpen}
-        onClose={() => setIsExportModalOpen(false)}
-        onExport={handleExport}
-        onExportAll={handleExportAll}
-        currentCaseNumber={currentCase || ''}
-        isReadOnly={isReadOnly}
-      />
-      
+
       <FilesModal
         isOpen={isFilesModalOpen}
         onClose={() => setIsFilesModalOpen(false)}
@@ -697,21 +266,29 @@ return (
         setFiles={setFiles}
         isReadOnly={isReadOnly}
         selectedFileId={selectedFileId}
+        confirmationSaveVersion={confirmationSaveVersion}
       />
       
         <div className={styles.filesSection}>
-      <div className={isReadOnly && currentCase ? styles.readOnlyContainer : styles.caseHeader}>
-        <h4 className={`${styles.caseNumber} ${
-          currentCase && caseConfirmationStatus.includeConfirmation 
-            ? caseConfirmationStatus.isConfirmed 
-              ? styles.caseConfirmed 
-              : styles.caseNotConfirmed
-            : ''
-        }`}>
-          {currentCase || 'No Case Selected'}
-        </h4>
-        {isReadOnly && currentCase && (
-          <div className={styles.readOnlyBadge}>(Read-Only)</div>
+        <div className={currentCase ? (isReadOnly ? styles.readOnlyContainer : styles.caseHeader) : styles.emptyCaseHeader}>
+        {currentCase ? (
+          <h4 className={`${styles.caseNumber} ${
+            caseConfirmationStatus.includeConfirmation 
+              ? caseConfirmationStatus.isConfirmed 
+                ? styles.caseConfirmed 
+                : styles.caseNotConfirmed
+              : ''
+          }`}>
+            {currentCase}
+          </h4>
+        ) : (
+          <button
+            type="button"
+            className={styles.openCaseButton}
+            onClick={onOpenCase}
+          >
+            Open Case
+          </button>
         )}
       </div>
       {currentCase && (
@@ -743,25 +320,18 @@ return (
         </div>
       )}
       
-      {!currentCase ? (
-        <p className={styles.emptyState}>Create or select a case to view files</p>
-      ) : files.length === 0 ? (
-        <p className={styles.emptyState}>No files found for {currentCase}</p>
-      ) : (
-        <>
-          {!canUploadNewFile && (
-            <div className={styles.limitReached}>
-              <p>Upload limit reached for this case</p>
-            </div>
-          )}
-          <ul className={styles.fileList}>
-            {files.map((file) => {
+      {currentCase ? (
+        <ul className={styles.fileList}>
+          {files.length === 0 ? (
+            <li className={styles.fileListMessage}>No files found for {currentCase}</li>
+          ) : (
+            files.map((file) => {
               const confirmationStatus = fileConfirmationStatus[file.id];
               let confirmationClass = '';
-              
+
               if (confirmationStatus?.includeConfirmation) {
-                confirmationClass = confirmationStatus.isConfirmed 
-                  ? styles.fileItemConfirmed 
+                confirmationClass = confirmationStatus.isConfirmed
+                  ? styles.fileItemConfirmed
                   : styles.fileItemNotConfirmed;
               }
 
@@ -776,11 +346,11 @@ return (
                       title={isUploading ? "Cannot select files while uploading" : undefined}
                     >
                     <span className={styles.fileName}>{file.originalFilename}</span>
-                  </button>              
+                  </button>
                   <button
                     onClick={() => {
                       if (window.confirm('Are you sure you want to delete this file? This action cannot be undone.')) {
-                        handleFileDelete(file.id);                                        
+                        handleFileDelete(file.id);
                       }
                     }}
                     className={styles.deleteButton}
@@ -793,12 +363,14 @@ return (
                   </button>
                 </li>
               );
-            })}
-          </ul>
-        </>
+            })
+          )}
+        </ul>
+      ) : (
+        <div className={styles.fileListPlaceholder}>Select a case to view files</div>
       )}
     </div>
-    <div className={`${styles.sidebarToggle} mb-4`}>
+    <div className={styles.sidebarToggle}>
     <button
           onClick={onNotesClick}
           disabled={isImageNotesDisabled}
@@ -807,99 +379,7 @@ return (
           Image Notes
         </button>
         </div>
-          {currentCase && (
-        <div className={styles.caseActionsSection}>
-          <button
-            onClick={() => setShowCaseActions(!showCaseActions)}
-            className={styles.caseActionsButton}
-            disabled={isUploading}
-            title={isUploading ? "Cannot access case actions while uploading" : undefined}
-          >
-            {showCaseActions ? 'Hide Case Actions' : 'Case Actions'}
-          </button>
-          
-          {showCaseActions && !isUploading && (
-            <div className={styles.caseActionsContent}>
-              {/* Export Case Data Section */}
-              <div className={styles.exportSection}>
-                <button 
-                  onClick={() => setIsExportModalOpen(true)}
-                  className={styles.exportButton}
-                  disabled={isUploading}
-                  title={isUploading ? "Cannot export while uploading" : undefined}
-                >
-                  Export Case Data
-                </button>
-              </div>
-              
-              {/* Audit Trail Section - Available for all cases */}
-              <div className={styles.auditTrailSection}>
-                <button
-                  onClick={() => setIsAuditTrailOpen(true)}
-                  className={styles.auditTrailButton}
-                  disabled={isUploading}
-                  title={isUploading ? "Cannot view audit trail while uploading" : undefined}
-                >
-                  Audit Trail
-                </button>
-              </div>
-              
-              {/* Rename/Delete Section - Only for owned cases */}
-              {!isReadOnly && (
-                <div className={styles.renameDeleteSection}>
-                  <div className={`${styles.caseRename} mb-4`}>
-                    <input
-                      type="text"
-                      value={newCaseName}
-                      onChange={(e) => setNewCaseName(e.target.value)}
-                      placeholder="New Case Number"
-                      disabled={isUploading}
-                    />
-                    <button
-                      onClick={handleRenameCase}
-                      disabled={isRenaming || !newCaseName || isUploading}
-                      title={isUploading ? "Cannot rename while uploading" : undefined}
-                    >
-                      {isRenaming ? 'Renaming...' : 'Rename Case'}
-                    </button>
-                  </div>
-                  
-                  <div className={styles.deleteCaseSection}>
-                    <button
-                      onClick={handleDeleteCase}
-                      disabled={isDeletingCase || isUploading}
-                      className={styles.deleteWarningButton}
-                      title={isUploading ? "Cannot delete while uploading" : undefined}
-                    >
-                        {isDeletingCase ? 'Deleting...' : 'Delete Case'}
-                      </button>
-                  </div>
-                </div>
-              )}
-            </div>
-          )}
-        </div>
-      )}
-      
-      {/* Unified Audit Viewer */}
-      <UserAuditViewer 
-        caseNumber={currentCase || ''}
-        isOpen={isAuditTrailOpen}
-        onClose={() => setIsAuditTrailOpen(false)}
-        title={`Audit Trail - Case ${currentCase}`}
-      />
-      
       </div>
-    <Toast
-      message={toastMessage}
-      type={toastType}
-      isVisible={isToastVisible}
-      onClose={() => {
-        setIsToastVisible(false);
-        setError('');
-        setSuccessAction(null);
-      }}
-    />
     </>
   );
 };