From 15e9f113e5339520ffb4b2bf081520bf5c8026e2 Mon Sep 17 00:00:00 2001
From: Matt Van Horn <455140+mvanhorn@users.noreply.github.com>
Date: Mon, 23 Mar 2026 23:34:17 -0700
Subject: [PATCH] fix: add error logging to empty catch blocks in
 useMessagesElectric

Replaces 4 silent catch blocks with console.warn calls so sync
failures are observable in the browser console. Cleanup catches
at lines 147/155 stay silent since PGlite may already be closed.

Fixes #905
---
 surfsense_web/hooks/use-messages-electric.ts | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/surfsense_web/hooks/use-messages-electric.ts b/surfsense_web/hooks/use-messages-electric.ts
index 728503de9..4928bed63 100644
--- a/surfsense_web/hooks/use-messages-electric.ts
+++ b/surfsense_web/hooks/use-messages-electric.ts
@@ -57,8 +57,8 @@ export function useMessagesElectric(
 							handle.initialSyncPromise,
 							new Promise((resolve) => setTimeout(resolve, 3000)),
 						]);
-					} catch {
-						// Timeout
+					} catch (err) {
+						console.warn("[useMessagesElectric] Sync timeout:", err);
 					}
 				}
 
@@ -70,8 +70,8 @@ export function useMessagesElectric(
 				syncHandleRef.current = handle;
 				await fetchMessages();
 				await setupLiveQuery();
-			} catch {
-				// Sync failed
+			} catch (err) {
+				console.warn("[useMessagesElectric] Sync failed:", err);
 			}
 		}
 
@@ -88,8 +88,8 @@ export function useMessagesElectric(
 				if (mounted && result.rows) {
 					handleMessagesUpdate(result.rows);
 				}
-			} catch {
-				// Query failed
+			} catch (err) {
+				console.warn("[useMessagesElectric] Query failed:", err);
 			}
 		}
 
@@ -130,8 +130,8 @@ export function useMessagesElectric(
 						liveQueryRef.current = liveQuery;
 					}
 				}
-			} catch {
-				// Live query failed
+			} catch (err) {
+				console.warn("[useMessagesElectric] Live query failed:", err);
 			}
 		}