Fix TS2345: wrap cleanup to return void instead of Promise

useEffect cleanup functions must return void, not Promise<void>.
The arrow shorthand was implicitly returning the promise from
clearActiveTransactionIDs(). Wrapping in a block body discards it.

Made-with: Cursor

Author: mountiny

src/components/MoneyRequestReportView/MoneyRequestReportTransactionList.tsx

@@ -289,7 +289,9 @@ function MoneyRequestReportTransactionList({
     }, [visualOrderTransactionIDs]);
 
     useEffect(() => {
-        return () => clearActiveTransactionIDs();
+        return () => {
+            clearActiveTransactionIDs();
+        };
     }, []);
 
     const sortedTransactionsMap = useMemo(() => {

tests/unit/MoneyRequestReportTransactionListActiveTransactionIDsTest.tsx

@@ -38,7 +38,9 @@ function useActiveTransactionIDsEffect(visualOrderTransactionIDs: string[]) {
     }, [visualOrderTransactionIDs]);
 
     useEffect(() => {
-        return () => clearActiveTransactionIDs();
+        return () => {
+            clearActiveTransactionIDs();
+        };
     }, []);
 }