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
This commit is contained in:
Matt Van Horn 2026-03-23 23:34:17 -07:00
parent 5da2d95d80
commit 15e9f113e5
No known key found for this signature in database

View file

@ -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);
}
}