refactor: drop the comments the code already says

Keep only the lines naming a constraint the code cannot show: the extension
wiping its folder, and the move emptying the shared path.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>

Author: adhorodyski

src/libs/telemetry/ReceiptObservability.ts

@@ -178,10 +178,6 @@ function logReceiptDropped({
     });
 }
 
-/**
- * Records a stat failure behind checkFileExists. A locked device rejects the stat with a permission error, which is
- * not the same as a missing file, and both otherwise collapse into the same dropped line.
- */
 function logReceiptStatFailed(code: string | undefined) {
     Log.info(`${RECEIPT_LOG_PREFIX} stat failed`, false, {
         event: 'statFailed',

src/pages/Share/SubmitDetailsPage.tsx

@@ -564,12 +564,12 @@ function SubmitDetailsPage({
             return;
         }
         formHasBeenSubmitted.current = true;
-        // The share extension wipes its folder on the next share, so upload from the receipts folder instead. Adopting
-        // here rather than at ingestion keeps abandoned shares out of a folder nothing prunes.
+        // The share extension wipes its folder on the next share. Adopting at submit, not at ingestion, keeps cancelled
+        // shares out of a folder nothing prunes.
         ReceiptStorage.adopt(currentReceiptSource, currentReceiptName)
             .then((durableName) => {
                 const uri = ReceiptStorage.toLocalUri(durableName);
-                // The draft is what a retry, the preview and the size check all re-read, and the shared path is gone once the move lands.
+                // The shared path is empty once the move lands, and the draft is what a retry re-reads.
                 setMoneyRequestReceipt(CONST.IOU.OPTIMISTIC_TRANSACTION_ID, uri, currentReceiptName, true, currentReceiptType);
                 return uri;
             })
@@ -583,7 +583,6 @@ function SubmitDetailsPage({
                     currentReceiptName,
                     (file) => onSuccess(file, locationPermissionGranted),
                     () => {
-                        // Allow retry after a file-read failure.
                         formHasBeenSubmitted.current = false;
                         setIsConfirming(false);
                     },

tests/ui/SubmitDetailsPageTest.tsx

@@ -355,8 +355,7 @@ describe('SubmitDetailsPage', () => {
         expect(jest.mocked(readFileAsync)).not.toHaveBeenCalled();
     });
 
-    // Error #12 — the share extension wipes its folder on the next share, so the receipt must be read from the
-    // receipts folder, not from the app-group folder it was shared into.
+    // Error #12 — the share extension wipes its folder on the next share, so the read must come from the receipts folder.
     it('adopts the shared file into the receipts folder and uploads from there', async () => {
         // Given a share whose file adopt moves into the receipts folder
         const durableUri = 'file:///Documents/Receipts-Upload/shared_1234.jpg';
@@ -371,8 +370,7 @@ describe('SubmitDetailsPage', () => {
         expect(jest.mocked(readFileAsync).mock.calls.at(0)?.[0]).toBe(durableUri);
     });
 
-    // Error #12b — a failed move must not block the submit: the shared path still works for this session,
-    // so falling back keeps a rare failure from turning every share into a dead end.
+    // Error #12b — a failed move must not block the submit: the shared path still works for this session.
     it('submits with the shared path when adopting the file fails', async () => {
         // Given adopt rejects
         const logAlertSpy = jest.spyOn(Log, 'alert').mockImplementation(() => {});
@@ -389,8 +387,7 @@ describe('SubmitDetailsPage', () => {
         logAlertSpy.mockRestore();
     });
 
-    // Error #12c — adopt is a move, so the shared path is empty once it lands. A retry after a read failure has to read
-    // the durable copy, or it adopts a path that no longer exists and the advertised retry can never recover.
+    // Error #12c — adopt is a move, so a retry after a read failure has to work off the durable copy.
     it('retries from the durable copy after a file-read failure', async () => {
         // Given a share that adopts fine but fails its first read
         const durableUri = 'file:///Documents/Receipts-Upload/shared_1234.jpg';