3 files changed, 35 insertions, 35 deletions

diff --git a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
index ff77a63..a38dbac 100644
--- a/packages/preload/src/contextBridge/__tests__/SophieRendererImpl.spec.ts
+++ b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
@@ -19,9 +19,6 @@
  */
 
 import { jest } from '@jest/globals';
-import { mocked } from 'jest-mock';
-import log from 'loglevel';
-import type { IJsonPatch } from 'mobx-state-tree';
 import {
   Action,
   MainToRendererIpcMessage,
@@ -29,6 +26,9 @@ import {
   SharedStoreSnapshotIn,
   SophieRenderer,
 } from '@sophie/shared';
+import { mocked } from 'jest-mock';
+import log from 'loglevel';
+import type { IJsonPatch } from 'mobx-state-tree';
 
 jest.unstable_mockModule('electron', () => ({
   ipcRenderer: {
@@ -40,7 +40,7 @@ jest.unstable_mockModule('electron', () => ({
 
 const { ipcRenderer } = await import('electron');
 
-const { createSophieRenderer } = await import('../SophieRendererImpl');
+const { default: createSophieRenderer } = await import('../createSophieRenderer');
 
 const event: Electron.IpcRendererEvent = null as unknown as Electron.IpcRendererEvent;
 
@@ -81,10 +81,10 @@ describe('createSophieRenderer', () => {
   });
 });
 
-describe('SophieRendererImpl', () => {
+describe('SharedStoreConnector', () => {
   let sut: SophieRenderer;
-  let onSharedStorePatch: (event1: Electron.IpcRendererEvent, patch1: unknown) => void;
-  let listener = {
+  let onSharedStorePatch: (eventArg: Electron.IpcRendererEvent, patchArg: unknown) => void;
+  const listener = {
     // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars
     onSnapshot: jest.fn((_snapshot: SharedStoreSnapshotIn) => {}),
     // eslint-disable-next-line @typescript-eslint/no-unused-vars, no-unused-vars
@@ -93,9 +93,9 @@ describe('SophieRendererImpl', () => {
 
   beforeEach(() => {
     sut = createSophieRenderer(false);
-    onSharedStorePatch = mocked(ipcRenderer.on).mock.calls.find(([channel]) => {
-      return channel === MainToRendererIpcMessage.SharedStorePatch;
-    })?.[1]!;
+    [, onSharedStorePatch] = mocked(ipcRenderer.on).mock.calls.find(
+      ([channel]) => channel === MainToRendererIpcMessage.SharedStorePatch,
+    )!;
   });
 
   describe('onSharedStoreChange', () => {
@@ -140,15 +140,15 @@ describe('SophieRendererImpl', () => {
     });
   });
 
-  function itRefusesToRegisterAnotherListener() {
+  function itRefusesToRegisterAnotherListener(): void {
     it('should refuse to register another listener', async () => {
       await expect(sut.onSharedStoreChange(listener)).rejects.toBeInstanceOf(Error);
     });
   }
 
   function itDoesNotPassPatchesToTheListener(
-    name: string = 'should not pass patches to the listener',
-  ) {
+    name = 'should not pass patches to the listener',
+  ): void {
     it(name, () => {
       onSharedStorePatch(event, patch);
       expect(listener.onPatch).not.toBeCalled();
diff --git a/packages/preload/src/contextBridge/SophieRendererImpl.ts b/packages/preload/src/contextBridge/createSophieRenderer.ts
index f3c07c5..2055080 100644
--- a/packages/preload/src/contextBridge/SophieRendererImpl.ts
+++ b/packages/preload/src/contextBridge/createSophieRenderer.ts
@@ -18,9 +18,6 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import { ipcRenderer } from 'electron';
-import log from 'loglevel';
-import type { IJsonPatch } from 'mobx-state-tree';
 import {
   Action,
   action,
@@ -30,9 +27,12 @@ import {
   SharedStoreListener,
   SophieRenderer,
 } from '@sophie/shared';
+import { ipcRenderer } from 'electron';
+import log from 'loglevel';
+import type { IJsonPatch } from 'mobx-state-tree';
 
-class SophieRendererImpl implements SophieRenderer {
-  private onSharedStoreChangeCalled: boolean = false;
+class SharedStoreConnector {
+  private onSharedStoreChangeCalled = false;
 
   private listener: SharedStoreListener | null = null;
 
@@ -71,26 +71,26 @@ class SophieRendererImpl implements SophieRenderer {
     }
     throw new Error('Failed to connect to shared store');
   }
+}
 
-  dispatchAction(actionToDispatch: Action): void {
-    // Let the full zod parse error bubble up to the main world,
-    // since all data it may contain was provided from the main world.
-    const parsedAction = action.parse(actionToDispatch);
-    try {
-      ipcRenderer.send(RendererToMainIpcMessage.DispatchAction, parsedAction);
-    } catch (err) {
-      // Do not leak IPC failure details into the main world.
-      const message = 'Failed to dispatch action';
-      log.error(message, actionToDispatch, err);
-      throw new Error(message);
-    }
+function dispatchAction(actionToDispatch: Action): void {
+  // Let the full zod parse error bubble up to the main world,
+  // since all data it may contain was provided from the main world.
+  const parsedAction = action.parse(actionToDispatch);
+  try {
+    ipcRenderer.send(RendererToMainIpcMessage.DispatchAction, parsedAction);
+  } catch (err) {
+    // Do not leak IPC failure details into the main world.
+    const message = 'Failed to dispatch action';
+    log.error(message, actionToDispatch, err);
+    throw new Error(message);
   }
 }
 
-export function createSophieRenderer(allowReplaceListener: boolean): SophieRenderer {
-  const impl = new SophieRendererImpl(allowReplaceListener);
+export default function createSophieRenderer(allowReplaceListener: boolean): SophieRenderer {
+  const connector = new SharedStoreConnector(allowReplaceListener);
   return {
-    onSharedStoreChange: impl.onSharedStoreChange.bind(impl),
-    dispatchAction: impl.dispatchAction.bind(impl),
+    onSharedStoreChange: connector.onSharedStoreChange.bind(connector),
+    dispatchAction,
   };
 }
diff --git a/packages/preload/src/index.ts b/packages/preload/src/index.ts
index de91742..f13220c 100644
--- a/packages/preload/src/index.ts
+++ b/packages/preload/src/index.ts
@@ -20,7 +20,7 @@
 
 import { contextBridge } from 'electron';
 
-import { createSophieRenderer } from './contextBridge/SophieRendererImpl';
+import createSophieRenderer from './contextBridge/createSophieRenderer';
 
 const isDevelopment = import.meta.env.MODE === 'development';