update comments

Author: borys3kk

src/hooks/useSearchHighlightAndScroll.ts

@@ -238,11 +238,7 @@ function useSearchHighlightAndScroll({
             }
             setNewSearchResultKeys(newKeys);
 
-            // Clear consumed manual highlight flags synchronously so subsequent detect runs
-            // don't re-highlight the same IDs. A previous timer-based reset effect lost a race
-            // with the reset of `newSearchResultKeys` (its cleanup kept canceling the timer
-            // before it could fire), leaving flags stuck at `true` and causing every later add
-            // to re-animate every prior expense.
+            // Clear consumed manual highlight flags so subsequent detect runs don't re-highlight the same IDs.
             if (consumedManualIDs.length > 0) {
                 mergeTransactionIdsHighlightOnSearchRoute(queryJSON.type, Object.fromEntries(consumedManualIDs.map((id) => [id, false])));
             }

tests/unit/useSearchHighlightAndScrollTest.ts

@@ -332,9 +332,6 @@ describe('useSearchHighlightAndScroll', () => {
         expect(result.current.newSearchResultKeys?.size).toBe(1);
         expect([...(result.current.newSearchResultKeys ?? new Set())]).toContain('transactions_3');
 
-        // The hook now clears the consumed manual highlight flag synchronously inside the
-        // detect effect (right after `setNewSearchResultKeys`), instead of via a timer.
-        // No need to wait — the merge has already fired by the time `rerender` returns.
         expect(spyOnMergeTransactionIdsHighlightOnSearchRoute).toHaveBeenCalledWith(baseProps.queryJSON.type, {'3': false});
     });