fix: catch errors from redux migration

src/context/revoltjs/SyncManager.tsx

@@ -5,6 +5,8 @@ import { ClientboundNotification } from "revolt.js/dist/websocket/notifications"
 
 import { useEffect } from "preact/hooks";
 
+import { reportError } from "../../lib/ErrorBoundary";
+
 import { useApplicationState } from "../../mobx/State";
 
 import { useClient } from "./RevoltClient";
@@ -28,7 +30,11 @@ export default function SyncManager() {
         if (!client) return;
         function onPacket(packet: ClientboundNotification) {
             if (packet.type === "UserSettingsUpdate") {
+                try {
                     state.sync.apply(packet.update);
+                } catch (err) {
+                    reportError(err, "failed_sync_apply");
+                }
             }
         }
 

src/lib/ErrorBoundary.tsx

@@ -27,11 +27,7 @@ interface Props {
 
 const ERROR_URL = "https://reporting.revolt.chat";
 
-export default function ErrorBoundary({ children, section }: Props) {
+export function reportError(error: Error, section: string) {
-    const [error, ignoreError] = useErrorBoundary();
-
-    useEffect(() => {
-        if (error) {
     stackTrace.fromError(error).then((stackframes) =>
         axios.post(ERROR_URL, {
             stackframes,
@@ -43,6 +39,14 @@ export default function ErrorBoundary({ children, section }: Props) {
         }),
     );
 }
+
+export default function ErrorBoundary({ children, section }: Props) {
+    const [error, ignoreError] = useErrorBoundary();
+
+    useEffect(() => {
+        if (error) {
+            reportError(error, section);
+        }
     }, [error]);
 
     if (error) {

src/mobx/State.ts

@@ -4,6 +4,8 @@ import localforage from "localforage";
 import { makeAutoObservable, reaction } from "mobx";
 import { Client } from "revolt.js";
 
+import { reportError } from "../lib/ErrorBoundary";
+
 import { legacyMigrateForwards, LegacyState } from "./legacy/redux";
 
 import Persistent from "./interfaces/Persistent";
@@ -197,17 +199,21 @@ export default class State {
      */
     async hydrate() {
         // Migrate legacy Redux store.
+        try {
             let legacy = await localforage.getItem("state");
+            await localforage.removeItem("state");
             if (legacy) {
                 if (typeof legacy === "string") {
                     legacy = JSON.parse(legacy);
                 }
 
                 legacyMigrateForwards(legacy as Partial<LegacyState>, this);
-            await localforage.removeItem("state");
                 await this.save();
                 return;
             }
+        } catch (err) {
+            reportError(err, "redux_migration");
+        }
 
         // Load MobX store.
         const sync = (await localforage.getItem("sync")) as DataSync;