fix(main): Inconsistent RendererBridge snapshot

Make sure that the snapshot we send to the renderer process corresponds
to the state of the SharedStore after the last completed action.

Signed-off-by: Kristóf Marussy <kristof@marussy.com>

1 files changed, 6 insertions, 3 deletions

diff --git a/packages/main/src/infrastructure/electron/RendererBridge.ts b/packages/main/src/infrastructure/electron/RendererBridge.ts
index 8633b9d..c26aa94 100644
--- a/packages/main/src/infrastructure/electron/RendererBridge.ts
+++ b/packages/main/src/infrastructure/electron/RendererBridge.ts
@@ -53,7 +53,11 @@ export default class RendererBridge {
     this.disposeOnPatch = onPatch(store.shared, (patch) => {
       if (patches === undefined) {
         // Update unprotected stores (outside an action) right away.
-        listener([patch]);
+        try {
+          listener([patch]);
+        } finally {
+          this.snapshot = getSnapshot(store.shared);
+        }
       } else {
         patches.push(patch);
       }
@@ -77,8 +81,6 @@ export default class RendererBridge {
         abort(undefined);
         return;
       }
-      // Make shure that the saved snapshot is consistent with the patches we're going to send.
-      this.snapshot = getSnapshot(store.shared);
       topLevelCall = call;
       patches = [];
       try {
@@ -91,6 +93,7 @@ export default class RendererBridge {
         } finally {
           topLevelCall = undefined;
           patches = undefined;
+          this.snapshot = getSnapshot(store.shared);
         }
       }
     });