fix(lib): avoid duplicate query refetches

CHANGELOG.md

@@ -15,6 +15,7 @@ All notable changes are documented here. Format based on [Keep a Changelog](http
 - `force_cancel()` action in `StreamActions` for recipients (once contract support is merged)
 
 ### Fixed
+- `refreshStreamData` now invalidates active queries once instead of immediately refetching the same queries a second time
 - `scValToU64`/`scValToI128` and `streamsBySender`/`streamsByRecipient` now boundary-check the RPC
   response shape instead of trusting it blindly; the streams list surfaces load failures inline
   instead of silently logging to console

lib/queryClient.test.ts

@@ -21,10 +21,13 @@ describe('refreshStreamData', () => {
     mockRefetchQueries.mockResolvedValue(undefined);
   });
 
-  it('invalidates and refetches active queries after a stream mutation succeeds', async () => {
+  it('refetches active queries only once after a stream mutation succeeds', async () => {
     await refreshStreamData();
 
-    expect(queryClient.invalidateQueries).toHaveBeenCalledWith({ refetchType: 'active' });
-    expect(queryClient.refetchQueries).toHaveBeenCalledWith({ type: 'active' });
+    expect(queryClient.invalidateQueries).toHaveBeenCalledTimes(1);
+    expect(queryClient.invalidateQueries).toHaveBeenCalledWith({
+      refetchType: 'active',
+    });
+    expect(queryClient.refetchQueries).not.toHaveBeenCalled();
   });
 });

lib/queryClient.ts

@@ -11,10 +11,7 @@ export const queryClient = new QueryClient({
 
 export async function refreshStreamData(): Promise<void> {
   try {
-    await Promise.all([
-      queryClient.invalidateQueries({ refetchType: 'active' }),
-      queryClient.refetchQueries({ type: 'active' }),
-    ]);
+    await queryClient.invalidateQueries({ refetchType: 'active' });
   } catch (error) {
     console.warn('Failed to refresh stream data after a transaction.', error);
   }