fix eslint comments

Author: OlimpiaZurek

src/pages/iou/request/step/IOURequestStepDistance/hooks/useDistanceTransactionBackup.ts

@@ -43,7 +43,7 @@ function useDistanceTransactionBackup({transaction, isCreatingNewRequest, isEdit
         return () => {
             // If the user cancels out of the modal without saving changes, then the original transaction
             // needs to be restored from the backup so that all changes are removed.
-            // eslint-disable-next-line react-hooks/exhaustive-deps
+            // eslint-disable-next-line react-hooks/exhaustive-deps -- ref reads are stable; we intentionally read the latest `.current` at unmount
             if (transactionWasSavedRef.current) {
                 removeBackupTransaction(transaction?.transactionID);
                 return;
@@ -56,7 +56,7 @@ function useDistanceTransactionBackup({transaction, isCreatingNewRequest, isEdit
             }
             openReport({reportID: transaction?.reportID, introSelected, betas});
         };
-        // eslint-disable-next-line react-hooks/exhaustive-deps
+        // eslint-disable-next-line react-hooks/exhaustive-deps -- mount/unmount-only effect: backup on mount, restore-or-drop on unmount, never re-runs
     }, []);
 }
 

src/pages/iou/request/step/IOURequestStepDistance/hooks/useOdometerReadingsState.ts

@@ -153,7 +153,7 @@ function useOdometerReadingsState({currentTransaction, isEditing, selectedTab, i
             const endValue = currentEnd !== null && currentEnd !== undefined ? currentEnd.toString() : '';
 
             if (startValue || endValue) {
-                // eslint-disable-next-line react-hooks/set-state-in-effect
+                // eslint-disable-next-line react-hooks/set-state-in-effect -- intentional: syncing local form state from the Onyx transaction (an external source) when the user navigates back to the page
                 setStartReading(startValue);
                 setEndReading(endValue);
                 startReadingRef.current = startValue;

src/pages/iou/request/step/IOURequestStepDistance/hooks/useOdometerTransactionBackup.ts

@@ -38,19 +38,19 @@ function useOdometerTransactionBackup({
         createBackupTransaction(transaction, isTransactionDraft, true);
 
         return () => {
-            // eslint-disable-next-line react-hooks/exhaustive-deps
+            // eslint-disable-next-line react-hooks/exhaustive-deps -- ref reads are stable; we intentionally read the latest `.current` at unmount to decide between bypass / drop / restore
             if (backupHandledManuallyRef.current) {
                 return;
             }
-            // eslint-disable-next-line react-hooks/exhaustive-deps
+            // eslint-disable-next-line react-hooks/exhaustive-deps -- ref reads are stable; we intentionally read the latest `.current` at unmount
             if (didSaveEditingConfirmationRef.current) {
                 removeBackupTransactionWithImageCleanup(transactionID, isTransactionDraft);
                 return;
             }
             restoreOriginalTransactionFromBackupWithImageCleanup(transactionID, isTransactionDraft);
         };
         // We only want to create the backup once on mount and restore/remove it on unmount
-        // eslint-disable-next-line react-hooks/exhaustive-deps
+        // eslint-disable-next-line react-hooks/exhaustive-deps -- mount/unmount-only effect, never re-runs
     }, []);
 }