Fix reconnection bugs: prevent memory leaks and stack overflow

Co-authored-by: kevinelliott <123112+kevinelliott@users.noreply.github.com>

Author: Copilot

cmd/web/script.js

@@ -225,32 +225,43 @@ function connectWebSocket() {
             document.getElementById('status-text').textContent = 'Connection lost - attempting to reconnect...';
         }
 
+        // Clear any existing reconnection timer to prevent memory leaks
+        if (wsReconnectInterval) {
+            clearTimeout(wsReconnectInterval);
+            wsReconnectInterval = null;
+        }
+
         // Set up reconnection with exponential backoff
-        if (!wsReconnectInterval) {
-            let reconnectAttempts = 0;
-            let delay = 1000; // Start with 1 second
-            
-            const attemptReconnect = () => {
-                reconnectAttempts++;
-                const maxAttempts = 30;
-
-                if (reconnectAttempts > maxAttempts) {
-                    wsReconnectInterval = null;
-                    document.getElementById('status-text').textContent = 'Connection failed - please refresh the page';
-                    return;
-                }
+        let reconnectAttempts = 0;
+        let delay = 1000; // Start with 1 second
+        
+        const attemptReconnect = () => {
+            reconnectAttempts++;
+            const maxAttempts = 30;
+
+            if (reconnectAttempts > maxAttempts) {
+                wsReconnectInterval = null;
+                document.getElementById('status-text').textContent = 'Connection failed - please refresh the page';
+                return;
+            }
 
-                console.log('Reconnect attempt ' + reconnectAttempts + '/' + maxAttempts + ' (delay: ' + delay + 'ms)');
-                document.getElementById('status-text').textContent = 'Reconnecting (' + reconnectAttempts + '/' + maxAttempts + ')...';
-                connectWebSocket();
-                
-                // Double the delay for next attempt, cap at 30 seconds
-                delay = Math.min(delay * 2, 30000);
-                wsReconnectInterval = setTimeout(attemptReconnect, delay);
-            };
+            console.log('Reconnect attempt ' + reconnectAttempts + '/' + maxAttempts + ' (delay: ' + delay + 'ms)');
+            document.getElementById('status-text').textContent = 'Reconnecting (' + reconnectAttempts + '/' + maxAttempts + ')...';
+            
+            // Close existing WebSocket if it exists to avoid resource leaks
+            if (ws) {
+                ws.onclose = null; // Remove handler to prevent triggering another reconnection
+                ws.close();
+            }
+            
+            connectWebSocket();
 
+            // Double the delay for next attempt, cap at 30 seconds
+            delay = Math.min(delay * 2, 30000);
             wsReconnectInterval = setTimeout(attemptReconnect, delay);
-        }
+        };
+        
+        wsReconnectInterval = setTimeout(attemptReconnect, delay);
     };
 }