rewrite comments as plain sentences without jargon

Author: mollfpr

src/components/ExportDownloadStatusManager.tsx

@@ -45,7 +45,7 @@ function ExportDownloadStatusManager() {
             markExportDownloadSurfaced(exportID);
             return;
         }
-        // The modal blocks dismissal while still preparing, so this is belt-and-suspenders.
+        // The modal already blocks dismissal while preparing, so this is an extra guard.
         if (exportDownload?.state === CONST.EXPORT_DOWNLOAD.STATE.PREPARING) {
             return;
         }

src/components/ExportDownloadStatusModal.tsx

@@ -73,7 +73,7 @@ function ExportDownloadStatusModal({exportID, isVisible, onClose, failedBody}: E
 
     // Build the secure download URL the same way downloadReportPDF does, so the host always follows
     // the app's current environment (instead of the env baked into a backend-built URL) and authenticates
-    // via the encryptedAuthToken — no separate OldDot sign-in needed.
+    // via the encryptedAuthToken, so no separate OldDot sign-in is needed.
     const downloadFile = () => {
         if (!fileName || !currentUserLogin) {
             return;

src/hooks/useExportActions.ts

@@ -141,7 +141,7 @@ function useExportActions({reportID, policy, onPDFModalOpen}: UseExportActionsPa
             },
             true,
         );
-        // Clear the selection now that the export has started; the app-level ExportDownloadStatusManager shows the modal.
+        // Clear the selection now that the export has started. The app-level ExportDownloadStatusManager shows the modal.
         clearSelectedTransactions(true);
     };
 

src/hooks/useSearchBulkActions.ts

@@ -847,7 +847,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
                     true,
                 );
             }
-            // Clear the selection now that the export has started; ExportDownloadStatusManager shows the modal.
+            // Clear the selection now that the export has started. The ExportDownloadStatusManager shows the modal.
             selectAllMatchingItems(false);
             clearSelectedTransactions(undefined, true);
         },
@@ -951,7 +951,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
                     exportColumnLabels: exportParameters.exportColumnLabels,
                     exportName,
                 });
-                // Clear the selection now that the export has started; ExportDownloadStatusManager shows the modal.
+                // Clear the selection now that the export has started. The ExportDownloadStatusManager shows the modal.
                 selectAllMatchingItems(false);
                 clearSelectedTransactions(undefined, true);
                 return;
@@ -2243,7 +2243,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
                         return;
                     }
                     exportReportsToPDF(selectedReportIDs);
-                    // Clear the selection now that the export has started; ExportDownloadStatusManager shows the modal.
+                    // Clear the selection now that the export has started. The ExportDownloadStatusManager shows the modal.
                     selectAllMatchingItems(false);
                     clearSelectedTransactions(undefined, true);
                 },
@@ -2263,7 +2263,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
                         return;
                     }
                     exportReceiptsToZip({reportIDs: selectedReportIDs});
-                    // Clear the selection now that the export has started; ExportDownloadStatusManager shows the modal.
+                    // Clear the selection now that the export has started. The ExportDownloadStatusManager shows the modal.
                     selectAllMatchingItems(false);
                     clearSelectedTransactions(undefined, true);
                 },
@@ -2286,7 +2286,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
                         return;
                     }
                     exportReceiptsToZip({transactionIDs});
-                    // Clear the selection now that the export has started; ExportDownloadStatusManager shows the modal.
+                    // Clear the selection now that the export has started. The ExportDownloadStatusManager shows the modal.
                     selectAllMatchingItems(false);
                     clearSelectedTransactions(undefined, true);
                 },

src/libs/actions/Export.ts

@@ -83,7 +83,7 @@ function clearStaleExportDownloads() {
                 if (exportDownload.shouldSendFromConcierge) {
                     continue;
                 }
-                // Keep preparing (in flight) and ready (finished but maybe unseen) exports; only clear failed leftovers.
+                // Keep preparing and ready exports so the manager can re-surface them. Only failed leftovers are cleared here.
                 if (exportDownload.state === CONST.EXPORT_DOWNLOAD.STATE.PREPARING || exportDownload.state === CONST.EXPORT_DOWNLOAD.STATE.READY) {
                     continue;
                 }

tests/unit/ExportDownloadStatusModalTest.tsx

@@ -144,7 +144,7 @@ describe('ExportDownloadStatusModal', () => {
         await waitForBatchedUpdatesWithAct();
 
         const expectedURLPart = `secure?secureType=csvexport&filename=${encodeURIComponent(CSV_FILE_NAME)}&downloadName=${encodeURIComponent(CSV_FILE_NAME)}`;
-        // shouldUnlink (arg 9) is left undefined so the platform default cleans up the temp file; appendTimestamp (arg 10) is false so the OS-recorded download time isn't duplicated in the name.
+        // shouldUnlink (arg 9) is left undefined so the platform default cleans up the temp file. appendTimestamp (arg 10) is false so the download time recorded by the OS is not duplicated in the name.
         expect(mockFileDownload).toHaveBeenCalledWith(
             expect.anything(),
             expect.stringContaining(expectedURLPart),
@@ -166,7 +166,7 @@ describe('ExportDownloadStatusModal', () => {
         await waitForBatchedUpdatesWithAct();
 
         const expectedURLPart = `secure?secureType=pdfreport&filename=${encodeURIComponent(PDF_FILE_NAME)}&downloadName=${encodeURIComponent(PDF_FILE_NAME)}`;
-        // shouldUnlink (arg 9) is left undefined so the platform default cleans up the temp file; appendTimestamp (arg 10) is false so the OS-recorded download time isn't duplicated in the name.
+        // shouldUnlink (arg 9) is left undefined so the platform default cleans up the temp file. appendTimestamp (arg 10) is false so the download time recorded by the OS is not duplicated in the name.
         expect(mockFileDownload).toHaveBeenCalledWith(
             expect.anything(),
             expect.stringContaining(expectedURLPart),
@@ -205,7 +205,7 @@ describe('ExportDownloadStatusModal', () => {
         expect(screen.getByText('exportDownload.readyTitle')).toBeTruthy();
         expect(screen.getByText('exportDownload.readyBody')).toBeTruthy();
         expect(screen.getByText('exportDownload.downloadFile')).toBeTruthy();
-        // The Close button is removed in the ready state; the modal is dismissible and Download closes it.
+        // The Close button is removed in the ready state. The modal is dismissible and Download closes it.
         expect(screen.queryByText('exportDownload.close')).toBeNull();
     });