chore: in frontend, xxStore files: moving autoRefresh as an optional functionality instead of starting timers all the time

Author: HardMax71

frontend/src/lib/admin/stores/__tests__/eventsStore.test.ts

@@ -99,6 +99,12 @@ describe('EventsStore', () => {
     });
 
     function createStore() {
+        teardown = effect_root(() => {
+            store = createEventsStore({ autoRefresh: false });
+        });
+    }
+
+    function createStoreWithAutoRefresh() {
         teardown = effect_root(() => {
             store = createEventsStore();
         });
@@ -108,6 +114,7 @@ describe('EventsStore', () => {
         vi.unstubAllGlobals();
         store?.cleanup();
         teardown?.();
+        vi.clearAllTimers();
     });
 
     describe('initial state', () => {
@@ -386,7 +393,7 @@ describe('EventsStore', () => {
 
     describe('auto-refresh', () => {
         it('fires loadAll on 30s interval', async () => {
-            createStore();
+            createStoreWithAutoRefresh();
             vi.clearAllMocks();
 
             await vi.advanceTimersByTimeAsync(30000);
@@ -397,7 +404,7 @@ describe('EventsStore', () => {
         });
 
         it('stops on teardown', async () => {
-            createStore();
+            createStoreWithAutoRefresh();
             await vi.advanceTimersByTimeAsync(30000);
             expect(mocks.browseEventsApiV1AdminEventsBrowsePost).toHaveBeenCalled();
 

frontend/src/lib/admin/stores/__tests__/executionsStore.test.ts

@@ -56,19 +56,10 @@ describe('ExecutionsStore', () => {
         setupDefaultMocks();
     });
 
-    /**
-     * Creates a store with auto-refresh timers immediately cleared.
-     * The $effect fires synchronously inside effect_root, starting a
-     * setInterval. We clear all timers, reset mocks, and re-apply defaults
-     * so individual tests control timing explicitly.
-     */
     function createStore() {
         teardown = effect_root(() => {
-            store = createExecutionsStore();
+            store = createExecutionsStore({ autoRefresh: false });
         });
-        vi.clearAllTimers();
-        vi.clearAllMocks();
-        setupDefaultMocks();
     }
 
     function createStoreWithAutoRefresh() {

frontend/src/lib/admin/stores/__tests__/sagasStore.test.ts

@@ -29,13 +29,20 @@ describe('SagasStore', () => {
     });
 
     function createStore() {
+        teardown = effect_root(() => {
+            store = createSagasStore({ autoRefresh: false });
+        });
+    }
+
+    function createStoreWithAutoRefresh() {
         teardown = effect_root(() => {
             store = createSagasStore();
         });
     }
 
     afterEach(() => {
         teardown?.();
+        vi.clearAllTimers();
     });
 
     describe('initial state', () => {
@@ -190,7 +197,7 @@ describe('SagasStore', () => {
 
     describe('auto-refresh', () => {
         it('fires loadSagas on interval', async () => {
-            createStore();
+            createStoreWithAutoRefresh();
             vi.clearAllMocks();
 
             await vi.advanceTimersByTimeAsync(5000);
@@ -206,7 +213,7 @@ describe('SagasStore', () => {
                 data: { sagas, total: 1 },
             });
 
-            createStore();
+            createStoreWithAutoRefresh();
             await store.loadExecutionSagas('exec-target');
             vi.clearAllMocks();
 
@@ -224,7 +231,7 @@ describe('SagasStore', () => {
         });
 
         it('stops when refreshEnabled set to false', async () => {
-            createStore();
+            createStoreWithAutoRefresh();
             await vi.advanceTimersByTimeAsync(5000);
             expect(mocks.listSagasApiV1SagasGet).toHaveBeenCalled();
 

frontend/src/lib/admin/stores/eventsStore.svelte.ts

@@ -39,13 +39,15 @@ class EventsStore {
 
     pagination = createPaginationState({ initialPageSize: 10 });
 
-    constructor() {
-        $effect(() => {
-            const id = setInterval(() => this.loadAll(), 30_000);
-            return () => {
-                clearInterval(id);
-            };
-        });
+    constructor({ autoRefresh = true }: { autoRefresh?: boolean } = {}) {
+        if (autoRefresh) {
+            $effect(() => {
+                const id = setInterval(() => this.loadAll(), 30_000);
+                return () => {
+                    clearInterval(id);
+                };
+            });
+        }
     }
 
     async loadAll(): Promise<void> {
@@ -235,6 +237,6 @@ class EventsStore {
     }
 }
 
-export function createEventsStore(): EventsStore {
-    return new EventsStore();
+export function createEventsStore(options?: { autoRefresh?: boolean }): EventsStore {
+    return new EventsStore(options);
 }

frontend/src/lib/admin/stores/executionsStore.svelte.ts

@@ -23,13 +23,15 @@ class ExecutionsStore {
 
     pagination = createPaginationState({ initialPageSize: 20 });
 
-    constructor() {
-        $effect(() => {
-            const id = setInterval(() => this.loadData(), 5000);
-            return () => {
-                clearInterval(id);
-            };
-        });
+    constructor({ autoRefresh = true }: { autoRefresh?: boolean } = {}) {
+        if (autoRefresh) {
+            $effect(() => {
+                const id = setInterval(() => this.loadData(), 5000);
+                return () => {
+                    clearInterval(id);
+                };
+            });
+        }
     }
 
     async loadData(): Promise<void> {
@@ -80,6 +82,6 @@ class ExecutionsStore {
     }
 }
 
-export function createExecutionsStore(): ExecutionsStore {
-    return new ExecutionsStore();
+export function createExecutionsStore(options?: { autoRefresh?: boolean }): ExecutionsStore {
+    return new ExecutionsStore(options);
 }

frontend/src/lib/admin/stores/sagasStore.svelte.ts

@@ -20,14 +20,16 @@ class SagasStore {
 
     pagination = createPaginationState({ initialPageSize: 10 });
 
-    constructor() {
-        $effect(() => {
-            if (!this.refreshEnabled) return;
-            const id = setInterval(() => this.loadSagas(), this.refreshRate * 1000);
-            return () => {
-                clearInterval(id);
-            };
-        });
+    constructor({ autoRefresh = true }: { autoRefresh?: boolean } = {}) {
+        if (autoRefresh) {
+            $effect(() => {
+                if (!this.refreshEnabled) return;
+                const id = setInterval(() => this.loadSagas(), this.refreshRate * 1000);
+                return () => {
+                    clearInterval(id);
+                };
+            });
+        }
     }
 
     async loadSagas(): Promise<void> {
@@ -77,6 +79,6 @@ class SagasStore {
     }
 }
 
-export function createSagasStore(): SagasStore {
-    return new SagasStore();
+export function createSagasStore(options?: { autoRefresh?: boolean }): SagasStore {
+    return new SagasStore(options);
 }