@striae-org/striae 4.3.2 → 4.3.4

package/app/routes/striae/striae.tsx

@@ -17,30 +17,29 @@ import { FilesModal } from '~/components/sidebar/files/files-modal';
 import { NotesEditorModal } from '~/components/sidebar/notes/notes-editor-modal';
 import { UserAuditViewer } from '~/components/audit/user-audit-viewer';
 import { fetchUserApi } from '~/utils/api';
-import { resolveEarliestAnnotationTimestamp } from '~/utils/ui';
 import { type AnnotationData, type FileData } from '~/types';
-import type * as CaseExportActions from '~/components/actions/case-export';
-import { checkCaseIsReadOnly, validateCaseNumber, renameCase, deleteCase, checkExistingCase, createNewCase, archiveCase, getCaseArchiveDetails } from '~/components/actions/case-manage';
+import { validateCaseNumber, renameCase, deleteCase, checkExistingCase, createNewCase, archiveCase, getCaseArchiveDetails } from '~/components/actions/case-manage';
 import { checkReadOnlyCaseExists, deleteReadOnlyCase } from '~/components/actions/case-review';
-import { canCreateCase, getLimitsDescription, getUserData } from '~/utils/data';
+import { canCreateCase } from '~/utils/data';
+import {
+  resolveEarliestAnnotationTimestamp,
+  CREATE_READ_ONLY_CASE_EXISTS_ERROR,
+  CLEAR_READ_ONLY_CASE_PARTIAL_FAILURE,
+  DELETE_CASE_CONFIRMATION,
+  DELETE_FILE_CONFIRMATION,
+  DELETE_CASE_FAILED,
+  DELETE_FILE_FAILED,
+  RENAME_CASE_FAILED
+} from '~/utils/ui';
+import { useStriaeResetHelpers } from './hooks/use-striae-reset-helpers';
+import { getExportProgressLabel, loadCaseExportActions } from './utils/case-export';
+import { resolveOpenCaseHelperText } from './utils/open-case-helper';
 import styles from './striae.module.css';
 
 interface StriaePage {
   user: User;
 }
 
-type CaseExportActionsModule = typeof CaseExportActions;
-
-let caseExportActionsPromise: Promise<CaseExportActionsModule> | null = null;
-
-const loadCaseExportActions = (): Promise<CaseExportActionsModule> => {
-  if (!caseExportActionsPromise) {
-    caseExportActionsPromise = import('~/components/actions/case-export');
-  }
-
-  return caseExportActionsPromise;
-};
-
 export const Striae = ({ user }: StriaePage) => {
   // Image and error states
   const [selectedImage, setSelectedImage] = useState<string>();
@@ -61,6 +60,7 @@ export const Striae = ({ user }: StriaePage) => {
   const [showNotes, setShowNotes] = useState(false);
   const [isUploading, setIsUploading] = useState(false);
   const [isReadOnlyCase, setIsReadOnlyCase] = useState(false);
+  const [isReviewOnlyCase, setIsReviewOnlyCase] = useState(false);
 
   // Annotation states
   const [activeAnnotations, setActiveAnnotations] = useState<Set<string>>(new Set());
@@ -98,24 +98,39 @@ export const Striae = ({ user }: StriaePage) => {
     archiveReason?: string;
   }>({ archived: false });
 
+  const {
+    clearSelectedImageState,
+    clearCaseContextState,
+    clearLoadedCaseState,
+  } = useStriaeResetHelpers({
+    setSelectedImage,
+    setSelectedFilename,
+    setImageId,
+    setAnnotationData,
+    setError,
+    setImageLoaded,
+    setCurrentCase,
+    setFiles,
+    setActiveAnnotations,
+    setIsBoxAnnotationMode,
+    setIsReadOnlyCase,
+    setIsReviewOnlyCase,
+    setArchiveDetails,
+    setShowNotes,
+    setIsAuditTrailOpen,
+    setIsRenameCaseModalOpen,
+  });
+
 
    useEffect(() => {
     // Set clear.jpg when case changes or is cleared
-    setSelectedImage('/clear.jpg');
-    setSelectedFilename(undefined);
-    setImageId(undefined);
-    setAnnotationData(null);    
-    setError(undefined);
-    setImageLoaded(false);
+    clearSelectedImageState();
     
     // Reset annotation and UI states when case is cleared
     if (!currentCase) {
-      setActiveAnnotations(new Set());
-      setIsBoxAnnotationMode(false);
-      setIsReadOnlyCase(false);
-      setArchiveDetails({ archived: false });
+      clearCaseContextState();
     }
-  }, [currentCase]);
+  }, [currentCase, clearSelectedImageState, clearCaseContextState]);
 
   // Fetch user company data when component mounts
   useEffect(() => {
@@ -154,18 +169,23 @@ export const Striae = ({ user }: StriaePage) => {
     const checkReadOnlyStatus = async () => {
       if (!currentCase || !user?.uid) {
         setIsReadOnlyCase(false);
+        setIsReviewOnlyCase(false);
         return;
       }
 
       try {
-        // Check if the case data itself has isReadOnly: true
-        const isReadOnly = await checkCaseIsReadOnly(user, currentCase);
-        setIsReadOnlyCase(isReadOnly);
+        // Imported review cases are tracked in the user's read-only case list.
+        // This includes archived ZIP imports and distinguishes them from manually archived regular cases.
+        const readOnlyCaseEntry = await checkReadOnlyCaseExists(user, currentCase);
         const details = await getCaseArchiveDetails(user, currentCase);
+        const reviewOnly = Boolean(readOnlyCaseEntry);
+        setIsReviewOnlyCase(reviewOnly);
+        setIsReadOnlyCase(reviewOnly || details.archived);
         setArchiveDetails(details);
       } catch (error) {
         console.error('Error checking read-only status:', error);
         setIsReadOnlyCase(false);
+        setIsReviewOnlyCase(false);
         setArchiveDetails({ archived: false });
       }
     };
@@ -250,11 +270,7 @@ export const Striae = ({ user }: StriaePage) => {
 
     if (includeImages) {
       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';
+        const label = getExportProgressLabel(progress);
         onProgress?.(Math.round(progress), label);
       });
       showNotification(`Case ${exportCaseNumber} exported successfully.`, 'success');
@@ -317,7 +333,7 @@ export const Striae = ({ user }: StriaePage) => {
     try {
       const existingReadOnlyCase = await checkReadOnlyCaseExists(user, newCaseName);
       if (existingReadOnlyCase) {
-        showNotification(`Case "${newCaseName}" already exists as a read-only review case.`, 'error');
+        showNotification(CREATE_READ_ONLY_CASE_EXISTS_ERROR(newCaseName), 'error');
         return;
       }
 
@@ -327,7 +343,7 @@ export const Striae = ({ user }: StriaePage) => {
       setIsRenameCaseModalOpen(false);
       showNotification(`Case renamed to ${newCaseName}.`, 'success');
     } catch (renameError) {
-      showNotification(renameError instanceof Error ? renameError.message : 'Failed to rename case.', 'error');
+      showNotification(renameError instanceof Error ? renameError.message : RENAME_CASE_FAILED, 'error');
     } finally {
       setIsRenamingCase(false);
     }
@@ -339,9 +355,7 @@ export const Striae = ({ user }: StriaePage) => {
       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 any image assets are already missing (404), they will be skipped and the case deletion will continue.`
-    );
+    const confirmed = window.confirm(DELETE_CASE_CONFIRMATION(currentCase));
 
     if (!confirmed) {
       return;
@@ -350,11 +364,7 @@ export const Striae = ({ user }: StriaePage) => {
     setIsDeletingCase(true);
     try {
       const deleteResult = await deleteCase(user, currentCase);
-      setCurrentCase('');
-      setFiles([]);
-      setShowNotes(false);
-      setIsAuditTrailOpen(false);
-      setIsRenameCaseModalOpen(false);
+      clearLoadedCaseState();
       if (deleteResult.missingImages.length > 0) {
         showNotification(
           `Case deleted. ${deleteResult.missingImages.length} image(s) were not found and were skipped during deletion.`,
@@ -364,7 +374,7 @@ export const Striae = ({ user }: StriaePage) => {
         showNotification('Case deleted successfully.', 'success');
       }
     } catch (deleteError) {
-      showNotification(deleteError instanceof Error ? deleteError.message : 'Failed to delete case.', 'error');
+      showNotification(deleteError instanceof Error ? deleteError.message : DELETE_CASE_FAILED, 'error');
     } finally {
       setIsDeletingCase(false);
     }
@@ -383,9 +393,7 @@ export const Striae = ({ user }: StriaePage) => {
 
     const selectedFile = files.find((file) => file.id === imageId);
     const selectedFileName = selectedFile?.originalFilename || imageId;
-    const confirmed = window.confirm(
-      `Are you sure you want to delete ${selectedFileName}? This action cannot be undone.`
-    );
+    const confirmed = window.confirm(DELETE_FILE_CONFIRMATION(selectedFileName));
 
     if (!confirmed) {
       return;
@@ -396,7 +404,7 @@ export const Striae = ({ user }: StriaePage) => {
       const deleteResult = await deleteFile(user, currentCase, imageId, 'User-requested deletion via navbar file management');
       const updatedFiles = files.filter((file) => file.id !== imageId);
       setFiles(updatedFiles);
-      handleImageSelect({ id: 'clear', originalFilename: '/clear.jpg', uploadedAt: '' });
+      clearSelectedImageState();
       setShowNotes(false);
       if (deleteResult.imageMissing) {
         showNotification(
@@ -407,13 +415,18 @@ export const Striae = ({ user }: StriaePage) => {
         showNotification('File deleted successfully.', 'success');
       }
     } catch (deleteError) {
-      showNotification(deleteError instanceof Error ? deleteError.message : 'Failed to delete file.', 'error');
+      showNotification(deleteError instanceof Error ? deleteError.message : DELETE_FILE_FAILED, 'error');
     } finally {
       setIsDeletingFile(false);
     }
   };
 
   const handleClearROCase = async () => {
+    if (!isReviewOnlyCase) {
+      showNotification('Only imported review cases can be cleared from workspace.', 'error');
+      return;
+    }
+
     if (!currentCase) {
       showNotification('No read-only case is currently loaded.', 'error');
       return;
@@ -431,15 +444,10 @@ export const Striae = ({ user }: StriaePage) => {
     try {
       const success = await deleteReadOnlyCase(user, caseToRemove);
       if (!success) {
-        showNotification(`Failed to fully clear read-only case "${caseToRemove}". Please try again.`, 'error');
+        showNotification(CLEAR_READ_ONLY_CASE_PARTIAL_FAILURE(caseToRemove), 'error');
         return;
       }
-      setCurrentCase('');
-      setFiles([]);
-      handleImageSelect({ id: 'clear', originalFilename: '/clear.jpg', uploadedAt: '' });
-      setShowNotes(false);
-      setIsAuditTrailOpen(false);
-      setIsRenameCaseModalOpen(false);
+      clearLoadedCaseState();
       showNotification(`Read-only case "${caseToRemove}" cleared.`, 'success');
     } catch (clearError) {
       showNotification(clearError instanceof Error ? clearError.message : 'Failed to clear read-only case.', 'error');
@@ -461,6 +469,7 @@ export const Striae = ({ user }: StriaePage) => {
     try {
       await archiveCase(user, currentCase, archiveReason);
       setIsReadOnlyCase(true);
+      setIsReviewOnlyCase(false);
       setArchiveDetails({
         archived: true,
         archivedAt: new Date().toISOString(),
@@ -506,7 +515,7 @@ export const Striae = ({ user }: StriaePage) => {
 
       const existingReadOnlyCase = await checkReadOnlyCaseExists(user, nextCaseNumber);
       if (existingReadOnlyCase) {
-        showNotification(`Case "${nextCaseNumber}" already exists as a read-only review case.`, 'error');
+        showNotification(CREATE_READ_ONLY_CASE_EXISTS_ERROR(nextCaseNumber), 'error');
         return;
       }
 
@@ -531,17 +540,8 @@ export const Striae = ({ user }: StriaePage) => {
 
   const handleOpenCaseModal = async () => {
     setIsOpenCaseModalOpen(true);
-    try {
-      const userData = await getUserData(user);
-      if (userData && !userData.permitted) {
-        const limitsDescription = await getLimitsDescription(user);
-        setOpenCaseHelperText(limitsDescription || 'Load an existing case or create a new one.');
-      } else {
-        setOpenCaseHelperText('Load an existing case or create a new one.');
-      }
-    } catch {
-      setOpenCaseHelperText('Load an existing case or create a new one.');
-    }
+    const helperText = await resolveOpenCaseHelperText(user);
+    setOpenCaseHelperText(helperText);
   };
 
   // Function to refresh annotation data (called when notes are saved)
@@ -566,10 +566,7 @@ export const Striae = ({ user }: StriaePage) => {
         }
       } else if (!result.caseNumber && !result.isReadOnly) {
         // Read-only case cleared - reset all UI state
-        setCurrentCase('');
-        setFiles([]);
-        handleImageSelect({ id: 'clear', originalFilename: '/clear.jpg', uploadedAt: '' });
-        setShowNotes(false);
+        clearLoadedCaseState();
       }
     }
   };
@@ -746,6 +743,7 @@ export const Striae = ({ user }: StriaePage) => {
         isUploading={isUploading}
         company={userCompany}
         isReadOnly={isReadOnlyCase}
+        isReviewOnlyCase={isReviewOnlyCase}
         currentCase={currentCase}
         currentFileName={selectedFilename}
         isCurrentImageConfirmed={isCurrentImageConfirmed}

package/app/routes/striae/utils/case-export.ts

@@ -0,0 +1,37 @@
+import type * as CaseExportActions from '~/components/actions/case-export';
+
+export type CaseExportActionsModule = typeof CaseExportActions;
+
+let caseExportActionsPromise: Promise<CaseExportActionsModule> | null = null;
+
+export const loadCaseExportActions = (): Promise<CaseExportActionsModule> => {
+  if (!caseExportActionsPromise) {
+    caseExportActionsPromise = import('~/components/actions/case-export').catch((error: unknown) => {
+      // Clear cached failures so transient chunk/network errors can recover on retry.
+      caseExportActionsPromise = null;
+      throw error;
+    });
+  }
+
+  return caseExportActionsPromise;
+};
+
+export const getExportProgressLabel = (progress: number): string => {
+  if (progress < 30) {
+    return 'Loading case data';
+  }
+
+  if (progress < 50) {
+    return 'Preparing archive';
+  }
+
+  if (progress < 80) {
+    return 'Adding images';
+  }
+
+  if (progress < 96) {
+    return 'Finalizing';
+  }
+
+  return 'Downloading';
+};

package/app/routes/striae/utils/open-case-helper.ts

@@ -0,0 +1,18 @@
+import type { User } from 'firebase/auth';
+import { getLimitsDescription, getUserData } from '~/utils/data';
+
+export const DEFAULT_OPEN_CASE_HELPER_TEXT = 'Load an existing case or create a new one.';
+
+export const resolveOpenCaseHelperText = async (user: User): Promise<string> => {
+  try {
+    const userData = await getUserData(user);
+    if (userData && !userData.permitted) {
+      const limitsDescription = await getLimitsDescription(user);
+      return limitsDescription || DEFAULT_OPEN_CASE_HELPER_TEXT;
+    }
+
+    return DEFAULT_OPEN_CASE_HELPER_TEXT;
+  } catch {
+    return DEFAULT_OPEN_CASE_HELPER_TEXT;
+  }
+};

package/app/services/audit/audit-console-logger.ts

@@ -1,6 +1,6 @@
 import { type ValidationAuditEntry } from '~/types';
 
-export const getAuditSecurityIssuesForConsole = (
+const getAuditSecurityIssuesForConsole = (
   entry: ValidationAuditEntry
 ): string[] => {
   const checks = entry.details.securityChecks;

package/app/services/audit/audit-export-csv.ts

@@ -42,7 +42,7 @@ export const AUDIT_CSV_ENTRY_HEADERS = [
   'Confirmed Files'
 ];
 
-export const formatForCSV = (value?: string | number | null): string => {
+const formatForCSV = (value?: string | number | null): string => {
   if (value === undefined || value === null) return '';
   const str = String(value);
   if (str.includes(',') || str.includes('"') || str.includes('\n')) {

package/app/services/audit/audit-export-signing.ts

@@ -22,14 +22,14 @@ interface SignAuditExportInput {
   hash: string;
 }
 
-export interface AuditExportSignature {
+interface AuditExportSignature {
   algorithm: string;
   keyId: string;
   signedAt: string;
   value: string;
 }
 
-export interface SignedAuditExportPayload {
+interface SignedAuditExportPayload {
   signatureMetadata: AuditExportSigningPayload;
   signature: AuditExportSignature;
 }

package/app/services/audit/audit-export.service.ts

@@ -8,7 +8,7 @@ import { type AuditExportContext, signAuditExport } from './audit-export-signing
  * Audit Export Service
  * Handles exporting audit trails to various formats for compliance and forensic analysis
  */
-export class AuditExportService {
+class AuditExportService {
   private static instance: AuditExportService;
 
   private constructor() {}

package/app/services/audit/audit-worker-client.ts

@@ -18,7 +18,7 @@ interface PersistAuditEntryResponse {
   filename: string;
 }
 
-export type PersistAuditEntryResult =
+type PersistAuditEntryResult =
   | {
       ok: true;
       entryCount: number;

package/app/services/audit/audit.service.ts

@@ -2,7 +2,6 @@ import type { User } from 'firebase/auth';
 import type {
   ValidationAuditEntry, 
   CreateAuditEntryParams, 
-  AuditTrail, 
   AuditQueryParams,
   WorkflowPhase,
   AuditAction,
@@ -18,7 +17,6 @@ import {
 import {
   applyAuditEntryFilters,
   applyAuditPagination,
-  generateAuditSummary,
   sortAuditEntriesNewestFirst
 } from './audit-query-helpers';
 import { logAuditEntryToConsole } from './audit-console-logger';
@@ -58,7 +56,7 @@ import {
  * Audit Service for ValidationAuditEntry system
  * Provides comprehensive audit logging throughout the confirmation workflow
  */
-export class AuditService {
+class AuditService {
   private static instance: AuditService;
   private auditBuffer: ValidationAuditEntry[] = [];
   private workflowId: string | null = null;
@@ -383,13 +381,15 @@ export class AuditService {
   public async logCaseCreation(
     user: User,
     caseNumber: string,
-    caseName: string
+    caseName: string,
+    renamedFromCaseNumber?: string
   ): Promise<void> {
     await this.logEventForUser(user,
       buildCaseCreationAuditParams({
         user,
         caseNumber,
-        caseName
+        caseName,
+        renamedFromCaseNumber
       })
     );
   }
@@ -721,37 +721,6 @@ export class AuditService {
     );
   }
 
-  /**
-   * Log user account deletion event
-   */
-  public async logAccountDeletion(
-    user: User,
-    result: AuditResult,
-    deletionReason: 'user-requested' | 'admin-initiated' | 'policy-violation' | 'inactive-account' = 'user-requested',
-    confirmationMethod: 'uid-email' | 'password' | 'admin-override' = 'uid-email',
-    casesCount?: number,
-    filesCount?: number,
-    dataRetentionPeriod?: number,
-    emailNotificationSent?: boolean,
-    sessionId?: string,
-    errors: string[] = []
-  ): Promise<void> {
-    // Wrapper that extracts user data and calls the simplified version
-    return this.logAccountDeletionSimple(
-      user.uid,
-      user.email || '',
-      result,
-      deletionReason,
-      confirmationMethod,
-      casesCount,
-      filesCount,
-      dataRetentionPeriod,
-      emailNotificationSent,
-      sessionId,
-      errors
-    );
-  }
-
   /**
    * Log user account deletion event with simplified user data
    */
@@ -1011,32 +980,6 @@ export class AuditService {
     return await this.getAuditEntries(queryParams, params?.requestingUser);
   }
 
-  /**
-   * Get audit trail for a case
-   */
-  public async getAuditTrail(caseNumber: string): Promise<AuditTrail | null> {
-    try {
-      // Implement retrieval from storage
-      const entries = await this.getAuditEntries({ caseNumber });
-      if (!entries || entries.length === 0) {
-        return null;
-      }
-
-      const summary = generateAuditSummary(entries);
-      const workflowId = this.workflowId || `${caseNumber}-archived`;
-
-      return {
-        caseNumber,
-        workflowId,
-        entries,
-        summary
-      };
-    } catch (error) {
-      console.error('🚨 Audit: Failed to get audit trail:', error);
-      return null;
-    }
-  }
-
   /**
    * Get audit entries based on query parameters
    */
@@ -1143,19 +1086,6 @@ export class AuditService {
     }
   }
 
-  /**
-   * Clear audit buffer (for testing)
-   */
-  public clearBuffer(): void {
-    this.auditBuffer = [];
-  }
-
-  /**
-   * Get current buffer size (for monitoring)
-   */
-  public getBufferSize(): number {
-    return this.auditBuffer.length;
-  }
 }
 
 // Export singleton instance

package/app/services/audit/builders/audit-event-builders-case-file.ts

@@ -6,6 +6,7 @@ interface BuildCaseCreationAuditParamsInput {
   user: User;
   caseNumber: string;
   caseName: string;
+  renamedFromCaseNumber?: string;
 }
 
 export const buildCaseCreationAuditParams = (
@@ -22,7 +23,9 @@ export const buildCaseCreationAuditParams = (
     caseNumber: input.caseNumber,
     workflowPhase: 'casework',
     caseDetails: {
+      oldCaseName: input.renamedFromCaseNumber,
       newCaseName: input.caseName,
+      createdByRename: Boolean(input.renamedFromCaseNumber),
       createdDate: new Date().toISOString(),
       totalFiles: 0,
       totalAnnotations: 0

package/app/services/audit/index.ts

@@ -1,2 +1,2 @@
-export { AuditService, auditService } from './audit.service';
-export { AuditExportService, auditExportService } from './audit-export.service';
+export { auditService } from './audit.service';
+export { auditExportService } from './audit-export.service';

package/app/types/audit.ts

@@ -44,7 +44,7 @@ export interface ValidationAuditEntry {
  * Detailed information for each audit entry
  * Contains action-specific data and metadata
  */
-export interface AuditDetails {
+interface AuditDetails {
   // Core identification
   fileName?: string;
   fileType?: AuditFileType;
@@ -194,9 +194,10 @@ export interface AuditQueryParams {
 /**
  * Case management specific audit details
  */
-export interface CaseAuditDetails {
+interface CaseAuditDetails {
   oldCaseName?: string;
   newCaseName?: string;
+  createdByRename?: boolean;
   totalFiles?: number;
   totalAnnotations?: number;
   confirmedFileNames?: string[];
@@ -210,7 +211,7 @@ export interface CaseAuditDetails {
 /**
  * File operation specific audit details
  */
-export interface FileAuditDetails {
+interface FileAuditDetails {
   fileId?: string;
   originalFileName?: string;
   fileSize: number;
@@ -225,7 +226,7 @@ export interface FileAuditDetails {
 /**
  * Annotation operation specific audit details
  */
-export interface AnnotationAuditDetails {
+interface AnnotationAuditDetails {
   annotationId?: string;
   annotationType?: 'measurement' | 'identification' | 'comparison' | 'note' | 'region';
   annotationData?: unknown; // The actual annotation data structure
@@ -238,7 +239,7 @@ export interface AnnotationAuditDetails {
 /**
  * User session specific audit details
  */
-export interface SessionAuditDetails {
+interface SessionAuditDetails {
   sessionId?: string;
   userAgent?: string;
   sessionDuration?: number;
@@ -249,7 +250,7 @@ export interface SessionAuditDetails {
 /**
  * Security incident specific audit details
  */
-export interface SecurityAuditDetails {
+interface SecurityAuditDetails {
   incidentType?: 'unauthorized-access' | 'data-breach' | 'malware' | 'injection' | 'brute-force' | 'privilege-escalation';
   severity?: 'low' | 'medium' | 'high' | 'critical';
   targetResource?: string;
@@ -272,7 +273,7 @@ export interface SecurityAuditDetails {
 /**
  * User profile and authentication specific audit details
  */
-export interface UserProfileAuditDetails {
+interface UserProfileAuditDetails {
   profileField?: 'displayName' | 'email' | 'organization' | 'role' | 'preferences' | 'avatar' | 'badgeId';
   oldValue?: string;
   newValue?: string;

package/app/utils/data/case-filters.ts

@@ -86,7 +86,7 @@ export function filterCasesForModal(
 ): CasesModalCaseItem[] {
   const archiveFilteredCases = preferences.showArchivedOnly
     ? cases.filter((entry) => entry.archived && !entry.isReadOnly)
-    : cases.filter((entry) => !entry.archived && !entry.isReadOnly);
+    : cases.filter((entry) => !entry.isReadOnly);
 
   return archiveFilteredCases.filter((entry) =>
     matchesConfirmationFilter(entry.caseNumber, preferences.confirmationFilter, caseConfirmationStatus)