Merge pull request #731 from extolkom/main

fix: ensure proper cleanup of SSE events with AbortController

Author: ogazboiz

frontend/src/app/streams/[id]/page.tsx

@@ -147,17 +147,17 @@ export default function StreamDetailsPage() {
 
   // Handle SSE events
   useEffect(() => {
-    if (streamEvents.length > 0) {
-      const controller = new AbortController();
-      const updateData = async () => {
-        await Promise.all([
-          fetchStream(controller.signal),
-          fetchEvents(eventsPage, controller.signal),
-        ]);
-      };
-      void updateData();
-      return () => controller.abort();
-    }
+    const controller = new AbortController();
+
+    const refreshStreamData = async () => {
+      if (streamEvents.length > 0) {
+        await Promise.all([fetchStream(controller.signal), fetchEvents(eventsPage, controller.signal)]);
+      }
+    };
+
+    refreshStreamData();
+
+    return () => controller.abort();
   }, [streamEvents, fetchStream, fetchEvents, eventsPage]);
 
   // Live claimable counter

frontend/src/components/TransactionTracker.tsx

@@ -150,14 +150,19 @@ export default function TransactionTracker({
 
   // Reset state when returning to idle
   useEffect(() => {
+    let timer: ReturnType<typeof setTimeout> | undefined;
+
     if (status === "idle") {
-      const timer = setTimeout(() => {
+      timer = setTimeout(() => {
         setPollCount(0);
         setStreamData(null);
         setPreviousStreamData(null);
       }, 0);
-      return () => clearTimeout(timer);
     }
+
+    return () => {
+      if (timer) clearTimeout(timer);
+    };
   }, [status]);
 
   // Render based on status

frontend/tsconfig.json

@@ -5,6 +5,8 @@
     "allowJs": true,
     "skipLibCheck": true,
     "strict": true,
+    "noImplicitReturns": true,
+    "noFallthroughCasesInSwitch": true,
     "noEmit": true,
     "esModuleInterop": true,
     "module": "esnext",