build: Add eslint-plugin-unicorn

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

23 files changed, 122 insertions, 107 deletions

diff --git a/packages/main/.eslintrc.cjs b/packages/main/.eslintrc.cjs
index 548ea34..cab227c 100644
--- a/packages/main/.eslintrc.cjs
+++ b/packages/main/.eslintrc.cjs
@@ -3,4 +3,8 @@ module.exports = {
     node: true,
     browser: false,
   },
+  rules: {
+    // This is an application, so we're allowed to `exit` from it.
+    'unicorn/no-process-exit': 'off',
+  },
 };
diff --git a/packages/main/esbuild.config.js b/packages/main/esbuild.config.js
index d5f6f1e..ef0aa71 100644
--- a/packages/main/esbuild.config.js
+++ b/packages/main/esbuild.config.js
@@ -1,7 +1,7 @@
 import getRepoInfo from 'git-repo-info';
 
 import { node } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 const externalPackages = ['electron'];
@@ -14,7 +14,7 @@ const gitInfo = getRepoInfo();
 
 export default getEsbuildConfig(
   {
-    absWorkingDir: fileURLToDirname(import.meta.url),
+    absWorkingDir: fileUrlToDirname(import.meta.url),
     entryPoints: ['src/index.ts'],
     outfile: 'dist/index.cjs',
     format: 'cjs',
@@ -23,9 +23,9 @@ export default getEsbuildConfig(
     external: externalPackages,
   },
   {
-    VITE_DEV_SERVER_URL: process.env.VITE_DEV_SERVER_URL || null,
+    VITE_DEV_SERVER_URL: process.env.VITE_DEV_SERVER_URL || undefined,
     GIT_SHA: gitInfo.abbreviatedSha,
     GIT_BRANCH: gitInfo.branch,
-    BUILD_DATE: new Date().getTime(),
+    BUILD_DATE: Date.now(),
   },
 );
diff --git a/packages/main/jest.config.js b/packages/main/jest.config.js
index b86463c..9aaf344 100644
--- a/packages/main/jest.config.js
+++ b/packages/main/jest.config.js
@@ -1,3 +1,4 @@
-import rootConfig from '../../config/jest.config.base.js';
+import baseConfig from '../../config/jest.config.base.js';
 
-export default rootConfig;
+// eslint-disable-next-line unicorn/prefer-export-from -- Can't export from default.
+export default baseConfig;
diff --git a/packages/main/src/controllers/__tests__/initConfig.spec.ts b/packages/main/src/controllers/__tests__/initConfig.spec.ts
index 9a5f85e..11e7690 100644
--- a/packages/main/src/controllers/__tests__/initConfig.spec.ts
+++ b/packages/main/src/controllers/__tests__/initConfig.spec.ts
@@ -133,7 +133,7 @@ describe('when it has loaded the config', () => {
   });
 
   it('should throttle saving changes to the config file', () => {
-    mocked(persistenceService.writeConfig).mockResolvedValue(undefined);
+    mocked(persistenceService.writeConfig).mockResolvedValue();
     config.setThemeSource('dark');
     jest.advanceTimersByTime(lessThanThrottleMs);
     config.setThemeSource('light');
diff --git a/packages/main/src/controllers/initConfig.ts b/packages/main/src/controllers/initConfig.ts
index e83b8da..915f451 100644
--- a/packages/main/src/controllers/initConfig.ts
+++ b/packages/main/src/controllers/initConfig.ts
@@ -38,7 +38,7 @@ export default async function initConfig(
 ): Promise<Disposer> {
   log.trace('Initializing config controller');
 
-  let lastSnapshotOnDisk: ConfigSnapshotOut | null = null;
+  let lastSnapshotOnDisk: ConfigSnapshotOut | undefined;
 
   async function readConfig(): Promise<boolean> {
     const result = await persistenceService.readConfig();
@@ -46,8 +46,8 @@ export default async function initConfig(
       try {
         applySnapshot(config, result.data);
         lastSnapshotOnDisk = getSnapshot(config);
-      } catch (err) {
-        log.error('Failed to apply config snapshot', result.data, err);
+      } catch (error) {
+        log.error('Failed to apply config snapshot', result.data, error);
       }
     }
     return result.found;
@@ -70,8 +70,8 @@ export default async function initConfig(
     debounce((snapshot) => {
       // We can compare snapshots by reference, since it is only recreated on store changes.
       if (lastSnapshotOnDisk !== snapshot) {
-        writeConfig().catch((err) => {
-          log.error('Failed to write config on config change', err);
+        writeConfig().catch((error) => {
+          log.error('Failed to write config on config change', error);
         });
       }
     }, debounceTime),
@@ -80,8 +80,8 @@ export default async function initConfig(
   const disposeWatcher = persistenceService.watchConfig(async () => {
     try {
       await readConfig();
-    } catch (err) {
-      log.error('Failed to read config', err);
+    } catch (error) {
+      log.error('Failed to read config', error);
     }
   }, debounceTime);
 
diff --git a/packages/main/src/devTools.ts b/packages/main/src/devTools.ts
index 69f1514..4ca1bf3 100644
--- a/packages/main/src/devTools.ts
+++ b/packages/main/src/devTools.ts
@@ -49,7 +49,8 @@ export async function installDevToolsExtensions(): Promise<void> {
     /* eslint-disable-next-line
       import/no-extraneous-dependencies,
       global-require,
-      @typescript-eslint/no-var-requires
+      @typescript-eslint/no-var-requires,
+      unicorn/prefer-module
     */
   } = require('electron-devtools-installer') as typeof import('electron-devtools-installer');
   await installExtension([REACT_DEVELOPER_TOOLS, REDUX_DEVTOOLS], {
diff --git a/packages/main/src/index.ts b/packages/main/src/index.ts
index 1f80e44..02e6cda 100644
--- a/packages/main/src/index.ts
+++ b/packages/main/src/index.ts
@@ -19,9 +19,9 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import { arch } from 'os';
-import { join } from 'path';
-import { URL } from 'url';
+import { arch } from 'node:os';
+import path from 'node:path';
+import { URL } from 'node:url';
 
 import {
   ServiceToMainIpcMessage,
@@ -101,11 +101,14 @@ app.setAboutPanelOptions({
   version: '',
 });
 
+// eslint-disable-next-line unicorn/prefer-module -- Electron apps run in a commonjs environment.
+const thisDir = __dirname;
+
 function getResourcePath(relativePath: string): string {
-  return join(__dirname, relativePath);
+  return path.join(thisDir, relativePath);
 }
 
-const baseUrl = `file://${__dirname}`;
+const baseUrl = `file://${thisDir}`;
 function getResourceUrl(relativePath: string): string {
   return new URL(relativePath, baseUrl).toString();
 }
@@ -117,15 +120,15 @@ const serviceInject: WebSource = {
   url: getResourceUrl(serviceInjectRelativePath),
 };
 
-let mainWindow: BrowserWindow | null = null;
+let mainWindow: BrowserWindow | undefined;
 
 const store = createMainStore();
 init(store)
   .then((disposeCompositionRoot) => {
     app.on('will-quit', disposeCompositionRoot);
   })
-  .catch((err) => {
-    log.log('Failed to initialize application', err);
+  .catch((error) => {
+    log.log('Failed to initialize application', error);
   });
 
 const rendererBaseUrl = getResourceUrl('../renderer/');
@@ -136,7 +139,7 @@ function shouldCancelMainWindowRequest(url: string, method: string): boolean {
   let normalizedUrl: string;
   try {
     normalizedUrl = new URL(url).toString();
-  } catch (_err) {
+  } catch {
     return true;
   }
   if (isDevelopment) {
@@ -233,7 +236,7 @@ async function createWindow(): Promise<unknown> {
         'from webContents',
         event.sender.id,
       );
-      return null;
+      throw new Error('Invalid IPC call');
     }
     return getSnapshot(store.shared);
   });
@@ -262,22 +265,22 @@ async function createWindow(): Promise<unknown> {
             .then((data) => {
               serviceInject.code = data;
             })
-            .catch((err) => {
-              log.error('Error while reloading', serviceInjectPath, err);
+            .catch((error) => {
+              log.error('Error while reloading', serviceInjectPath, error);
             })
             .then(() => {
               browserView.webContents.reload();
             })
-            .catch((err) => {
-              log.error('Failed to reload browserView', err);
+            .catch((error) => {
+              log.error('Failed to reload browserView', error);
             });
           break;
         default:
           log.error('Unexpected action from UI renderer:', actionToDispatch);
           break;
       }
-    } catch (err) {
-      log.error('Error while dispatching renderer action', rawAction, err);
+    } catch (error) {
+      log.error('Error while dispatching renderer action', rawAction, error);
     }
   });
 
@@ -285,9 +288,18 @@ async function createWindow(): Promise<unknown> {
     webContents.send(MainToRendererIpcMessage.SharedStorePatch, patch);
   });
 
-  ipcMain.handle(ServiceToMainIpcMessage.ApiExposedInMainWorld, (event) =>
-    event.sender.id === browserView.webContents.id ? serviceInject : null,
-  );
+  ipcMain.handle(ServiceToMainIpcMessage.ApiExposedInMainWorld, (event) => {
+    if (event.sender.id !== browserView.webContents.id) {
+      log.warn(
+        'Unexpected',
+        ServiceToMainIpcMessage.ApiExposedInMainWorld,
+        'from webContents',
+        event.sender.id,
+      );
+      throw new Error('Invalid IPC call');
+    }
+    return serviceInject;
+  });
 
   browserView.webContents.on('ipc-message', (_event, channel, ...args) => {
     try {
@@ -303,8 +315,8 @@ async function createWindow(): Promise<unknown> {
           log.error('Unknown IPC message:', channel, args);
           break;
       }
-    } catch (err) {
-      log.error('Error while processing IPC message:', channel, args, err);
+    } catch (error) {
+      log.error('Error while processing IPC message:', channel, args, error);
     }
   });
 
@@ -316,9 +328,7 @@ async function createWindow(): Promise<unknown> {
 
   browserView.webContents.session.webRequest.onBeforeSendHeaders(
     ({ url, requestHeaders }, callback) => {
-      const requestUserAgent = url.match(
-        /^[^:]+:\/\/accounts\.google\.[^./]+\//,
-      )
+      const requestUserAgent = /^[^:]+:\/\/accounts\.google\.[^./]+\//.test(url)
         ? chromelessUserAgent
         : userAgent;
       callback({
@@ -332,15 +342,15 @@ async function createWindow(): Promise<unknown> {
 
   browserView.webContents
     .loadURL('https://gitlab.com/say-hi-to-sophie/sophie')
-    .catch((err) => {
-      log.error('Failed to load browser', err);
+    .catch((error) => {
+      log.error('Failed to load browser', error);
     });
 
   return mainWindow.loadURL(pageUrl);
 }
 
 app.on('second-instance', () => {
-  if (mainWindow !== null) {
+  if (mainWindow !== undefined) {
     if (!mainWindow.isVisible()) {
       mainWindow.show();
     }
@@ -363,14 +373,14 @@ app
     if (isDevelopment) {
       try {
         await installDevToolsExtensions();
-      } catch (err) {
-        log.error('Failed to install devtools extensions', err);
+      } catch (error) {
+        log.error('Failed to install devtools extensions', error);
       }
     }
 
     return createWindow();
   })
-  .catch((err) => {
-    log.error('Failed to create window', err);
+  .catch((error) => {
+    log.error('Failed to create window', error);
     process.exit(1);
   });
diff --git a/packages/main/src/services/impl/ConfigPersistenceServiceImpl.ts b/packages/main/src/services/impl/ConfigPersistenceServiceImpl.ts
index e92f706..a11a9da 100644
--- a/packages/main/src/services/impl/ConfigPersistenceServiceImpl.ts
+++ b/packages/main/src/services/impl/ConfigPersistenceServiceImpl.ts
@@ -17,9 +17,9 @@
  *
  * SPDX-License-Identifier: AGPL-3.0-only
  */
-import { watch } from 'fs';
-import { readFile, stat, writeFile } from 'fs/promises';
-import { join } from 'path';
+import { watch } from 'node:fs';
+import { readFile, stat, writeFile } from 'node:fs/promises';
+import path from 'node:path';
 
 import JSON5 from 'json5';
 import throttle from 'lodash-es/throttle';
@@ -39,26 +39,26 @@ export default class ConfigPersistenceServiceImpl
 
   private writingConfig = false;
 
-  private timeLastWritten: Date | null = null;
+  private timeLastWritten: Date | undefined;
 
   constructor(
     private readonly userDataDir: string,
     private readonly configFileName: string = 'config.json5',
   ) {
     this.configFileName = configFileName;
-    this.configFilePath = join(this.userDataDir, this.configFileName);
+    this.configFilePath = path.join(this.userDataDir, this.configFileName);
   }
 
   async readConfig(): Promise<ReadConfigResult> {
     let configStr;
     try {
       configStr = await readFile(this.configFilePath, 'utf8');
-    } catch (err) {
-      if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
+    } catch (error) {
+      if ((error as NodeJS.ErrnoException).code === 'ENOENT') {
         log.debug('Config file', this.configFilePath, 'was not found');
         return { found: false };
       }
-      throw err;
+      throw error;
     }
     log.info('Read config file', this.configFilePath);
     return {
@@ -92,8 +92,8 @@ export default class ConfigPersistenceServiceImpl
         const stats = await stat(this.configFilePath);
         mtime = stats.mtime;
         log.trace('Config file last modified at', mtime);
-      } catch (err) {
-        if ((err as NodeJS.ErrnoException).code === 'ENOENT') {
+      } catch (error) {
+        if ((error as NodeJS.ErrnoException).code === 'ENOENT') {
           log.debug(
             'Config file',
             this.configFilePath,
@@ -101,11 +101,11 @@ export default class ConfigPersistenceServiceImpl
           );
           return;
         }
-        throw err;
+        throw error;
       }
       if (
         !this.writingConfig &&
-        (this.timeLastWritten === null || mtime > this.timeLastWritten)
+        (this.timeLastWritten === undefined || mtime > this.timeLastWritten)
       ) {
         log.debug(
           'Found a config file modified at',
diff --git a/packages/main/src/stores/Config.ts b/packages/main/src/stores/Config.ts
index 06dbdeb..ca90c0c 100644
--- a/packages/main/src/stores/Config.ts
+++ b/packages/main/src/stores/Config.ts
@@ -18,12 +18,7 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
-import {
-  config as originalConfig,
-  ConfigSnapshotIn,
-  ConfigSnapshotOut,
-  ThemeSource,
-} from '@sophie/shared';
+import { config as originalConfig, ThemeSource } from '@sophie/shared';
 import { Instance } from 'mobx-state-tree';
 
 export const config = originalConfig.actions((self) => ({
@@ -34,4 +29,4 @@ export const config = originalConfig.actions((self) => ({
 
 export interface Config extends Instance<typeof config> {}
 
-export type { ConfigSnapshotIn, ConfigSnapshotOut };
+export type { ConfigSnapshotIn, ConfigSnapshotOut } from '@sophie/shared';
diff --git a/packages/main/src/utils/log.ts b/packages/main/src/utils/log.ts
index 5218721..0a632d8 100644
--- a/packages/main/src/utils/log.ts
+++ b/packages/main/src/utils/log.ts
@@ -18,8 +18,9 @@
  * SPDX-License-Identifier: AGPL-3.0-only
  */
 
+// eslint-disable-next-line unicorn/import-style -- Import the type `ChalkInstance` separately.
 import chalk, { ChalkInstance } from 'chalk';
-import loglevel, { Logger } from 'loglevel';
+import loglevel from 'loglevel';
 import prefix from 'loglevel-plugin-prefix';
 
 if (import.meta.env?.DEV) {
@@ -54,7 +55,7 @@ prefix.apply(loglevel, {
   },
 });
 
-export function getLogger(loggerName: string): Logger {
+export function getLogger(loggerName: string): loglevel.Logger {
   return loglevel.getLogger(loggerName);
 }
 
diff --git a/packages/preload/esbuild.config.js b/packages/preload/esbuild.config.js
index d888987..87e91d8 100644
--- a/packages/preload/esbuild.config.js
+++ b/packages/preload/esbuild.config.js
@@ -1,9 +1,9 @@
 import { chrome } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 export default getEsbuildConfig({
-  absWorkingDir: fileURLToDirname(import.meta.url),
+  absWorkingDir: fileUrlToDirname(import.meta.url),
   entryPoints: ['src/index.ts'],
   outfile: 'dist/index.cjs',
   format: 'cjs',
diff --git a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
index b0af280..88b0077 100644
--- a/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
+++ b/packages/preload/src/contextBridge/__tests__/createSophieRenderer.spec.ts
@@ -45,7 +45,7 @@ const { default: createSophieRenderer } = await import(
 );
 
 const event: Electron.IpcRendererEvent =
-  null as unknown as Electron.IpcRendererEvent;
+  undefined as unknown as Electron.IpcRendererEvent;
 
 const snapshot: SharedStoreSnapshotIn = {
   shouldUseDarkColors: true,
@@ -183,7 +183,7 @@ describe('SharedStoreConnector', () => {
 
     it('should catch listener errors', () => {
       mocked(listener.onPatch).mockImplementation(() => {
-        throw new Error();
+        throw new Error('listener error');
       });
       expect(() => onSharedStorePatch(event, patch)).not.toThrow();
     });
@@ -193,7 +193,7 @@ describe('SharedStoreConnector', () => {
     describe('after the listener threw in onPatch', () => {
       beforeEach(() => {
         mocked(listener.onPatch).mockImplementation(() => {
-          throw new Error();
+          throw new Error('listener error');
         });
         onSharedStorePatch(event, patch);
         listener.onPatch.mockRestore();
@@ -205,7 +205,9 @@ describe('SharedStoreConnector', () => {
 
   describe('when a listener failed to register due to IPC error', () => {
     beforeEach(async () => {
-      mocked(ipcRenderer.invoke).mockRejectedValue(new Error());
+      mocked(ipcRenderer.invoke).mockRejectedValue(
+        new Error('ipcRenderer error'),
+      );
       try {
         await sut.onSharedStoreChange(listener);
       } catch {
@@ -237,7 +239,7 @@ describe('SharedStoreConnector', () => {
     beforeEach(async () => {
       mocked(ipcRenderer.invoke).mockResolvedValueOnce(snapshot);
       mocked(listener.onSnapshot).mockImplementation(() => {
-        throw new Error();
+        throw new Error('listener error');
       });
       try {
         await sut.onSharedStoreChange(listener);
diff --git a/packages/preload/src/contextBridge/createSophieRenderer.ts b/packages/preload/src/contextBridge/createSophieRenderer.ts
index b97503d..3174fed 100644
--- a/packages/preload/src/contextBridge/createSophieRenderer.ts
+++ b/packages/preload/src/contextBridge/createSophieRenderer.ts
@@ -34,7 +34,7 @@ import type { IJsonPatch } from 'mobx-state-tree';
 class SharedStoreConnector {
   private onSharedStoreChangeCalled = false;
 
-  private listener: SharedStoreListener | null = null;
+  private listener: SharedStoreListener | undefined;
 
   constructor(private readonly allowReplaceListener: boolean) {
     ipcRenderer.on(
@@ -43,9 +43,9 @@ class SharedStoreConnector {
         try {
           // `mobx-state-tree` will validate the patch, so we can safely cast here.
           this.listener?.onPatch(patch as IJsonPatch);
-        } catch (err) {
-          log.error('Shared store listener onPatch failed', err);
-          this.listener = null;
+        } catch (error) {
+          log.error('Shared store listener onPatch failed', error);
+          this.listener = undefined;
         }
       },
     );
@@ -57,14 +57,14 @@ class SharedStoreConnector {
     }
     this.onSharedStoreChangeCalled = true;
     let success = false;
-    let snapshot: unknown | null = null;
+    let snapshot: unknown;
     try {
       snapshot = await ipcRenderer.invoke(
         RendererToMainIpcMessage.GetSharedStoreSnapshot,
       );
       success = true;
-    } catch (err) {
-      log.error('Failed to get initial shared store snapshot', err);
+    } catch (error) {
+      log.error('Failed to get initial shared store snapshot', error);
     }
     if (success) {
       if (sharedStore.is(snapshot)) {
@@ -84,10 +84,10 @@ function dispatchAction(actionToDispatch: Action): void {
   const parsedAction = action.parse(actionToDispatch);
   try {
     ipcRenderer.send(RendererToMainIpcMessage.DispatchAction, parsedAction);
-  } catch (err) {
+  } catch (error) {
     // Do not leak IPC failure details into the main world.
     const message = 'Failed to dispatch action';
-    log.error(message, actionToDispatch, err);
+    log.error(message, actionToDispatch, error);
     throw new Error(message);
   }
 }
diff --git a/packages/renderer/src/components/BrowserViewPlaceholder.tsx b/packages/renderer/src/components/BrowserViewPlaceholder.tsx
index 8f055e7..58407ee 100644
--- a/packages/renderer/src/components/BrowserViewPlaceholder.tsx
+++ b/packages/renderer/src/components/BrowserViewPlaceholder.tsx
@@ -44,15 +44,15 @@ export default observer(() => {
     [store],
   );
 
-  const resizeObserverRef = useRef<ResizeObserver | null>(null);
+  const resizeObserverRef = useRef<ResizeObserver | undefined>();
 
   const ref = useCallback(
     (element: HTMLElement | null) => {
-      if (resizeObserverRef.current !== null) {
+      if (resizeObserverRef.current !== undefined) {
         resizeObserverRef.current.disconnect();
       }
       if (element === null) {
-        resizeObserverRef.current = null;
+        resizeObserverRef.current = undefined;
         return;
       }
       resizeObserverRef.current = new ResizeObserver(onResize);
diff --git a/packages/renderer/src/components/StoreProvider.tsx b/packages/renderer/src/components/StoreProvider.tsx
index bb8495c..3360a43 100644
--- a/packages/renderer/src/components/StoreProvider.tsx
+++ b/packages/renderer/src/components/StoreProvider.tsx
@@ -22,11 +22,12 @@ import React, { createContext, useContext } from 'react';
 
 import type { RendererStore } from '../stores/RendererStore';
 
-const StoreContext = createContext<RendererStore | null>(null);
+// eslint-disable-next-line unicorn/no-useless-undefined -- `createContext` expects 1 parameter.
+const StoreContext = createContext<RendererStore | undefined>(undefined);
 
 export function useStore(): RendererStore {
   const store = useContext(StoreContext);
-  if (store === null) {
+  if (store === undefined) {
     throw new Error('useStore can only be called inside of StoreProvider');
   }
   return store;
diff --git a/packages/renderer/src/index.tsx b/packages/renderer/src/index.tsx
index d900e50..a42a30c 100644
--- a/packages/renderer/src/index.tsx
+++ b/packages/renderer/src/index.tsx
@@ -45,8 +45,8 @@ if (isDevelopment) {
 const store = createAndConnectRendererStore(window.sophieRenderer);
 
 if (isDevelopment) {
-  exposeToReduxDevtools(store).catch((err) => {
-    log.error('Cannot initialize redux devtools', err);
+  exposeToReduxDevtools(store).catch((error) => {
+    log.error('Cannot initialize redux devtools', error);
   });
 }
 
diff --git a/packages/renderer/src/stores/RendererStore.ts b/packages/renderer/src/stores/RendererStore.ts
index 0b78ce1..4cc5163 100644
--- a/packages/renderer/src/stores/RendererStore.ts
+++ b/packages/renderer/src/stores/RendererStore.ts
@@ -86,8 +86,8 @@ export function createAndConnectRendererStore(
         applyPatch(store.shared, patch);
       },
     })
-    .catch((err) => {
-      log.error('Failed to connect to shared store', err);
+    .catch((error) => {
+      log.error('Failed to connect to shared store', error);
     });
 
   return store;
diff --git a/packages/renderer/vite.config.js b/packages/renderer/vite.config.js
index e20e0f1..cb0203c 100644
--- a/packages/renderer/vite.config.js
+++ b/packages/renderer/vite.config.js
@@ -1,15 +1,15 @@
 /* eslint-disable no-process-env */
 /* eslint-env node */
 
-import { builtinModules } from 'module';
-import { join } from 'path';
+import { builtinModules } from 'node:module';
+import path from 'node:path';
 
 import react from '@vitejs/plugin-react';
 
 import { banner, chrome } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 
-const thisDir = fileURLToDirname(import.meta.url);
+const thisDir = fileUrlToDirname(import.meta.url);
 
 const mode = process.env.MODE || 'development';
 
@@ -24,7 +24,7 @@ export default {
   logLevel: 'info',
   mode,
   root: thisDir,
-  cacheDir: join(thisDir, '../../.vite'),
+  cacheDir: path.join(thisDir, '../../.vite'),
   plugins: [
     react({
       babel: {
diff --git a/packages/service-inject/esbuild.config.js b/packages/service-inject/esbuild.config.js
index 795b0f6..d8698ac 100644
--- a/packages/service-inject/esbuild.config.js
+++ b/packages/service-inject/esbuild.config.js
@@ -1,9 +1,9 @@
 import { chrome } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 export default getEsbuildConfig({
-  absWorkingDir: fileURLToDirname(import.meta.url),
+  absWorkingDir: fileUrlToDirname(import.meta.url),
   entryPoints: ['src/index.ts'],
   outfile: 'dist/index.js',
   format: 'iife',
diff --git a/packages/service-preload/esbuild.config.js b/packages/service-preload/esbuild.config.js
index d888987..87e91d8 100644
--- a/packages/service-preload/esbuild.config.js
+++ b/packages/service-preload/esbuild.config.js
@@ -1,9 +1,9 @@
 import { chrome } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 export default getEsbuildConfig({
-  absWorkingDir: fileURLToDirname(import.meta.url),
+  absWorkingDir: fileUrlToDirname(import.meta.url),
   entryPoints: ['src/index.ts'],
   outfile: 'dist/index.cjs',
   format: 'cjs',
diff --git a/packages/service-preload/src/index.ts b/packages/service-preload/src/index.ts
index 2bbfefd..bb4a62d 100644
--- a/packages/service-preload/src/index.ts
+++ b/packages/service-preload/src/index.ts
@@ -61,6 +61,6 @@ async function fetchAndExecuteInjectScript(): Promise<void> {
   await webFrame.executeJavaScriptInIsolatedWorld(0, [injectSource]);
 }
 
-fetchAndExecuteInjectScript().catch((err) => {
-  log.error('Failed to fetch inject source:', err);
+fetchAndExecuteInjectScript().catch((error) => {
+  log.error('Failed to fetch inject source:', error);
 });
diff --git a/packages/service-shared/esbuild.config.js b/packages/service-shared/esbuild.config.js
index 2b0dec8..62e3d2a 100644
--- a/packages/service-shared/esbuild.config.js
+++ b/packages/service-shared/esbuild.config.js
@@ -1,9 +1,9 @@
 import { chrome, node } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 export default getEsbuildConfig({
-  absWorkingDir: fileURLToDirname(import.meta.url),
+  absWorkingDir: fileUrlToDirname(import.meta.url),
   entryPoints: ['src/index.ts'],
   outfile: 'dist/index.mjs',
   format: 'esm',
diff --git a/packages/shared/esbuild.config.js b/packages/shared/esbuild.config.js
index 44501bd..7a79ce6 100644
--- a/packages/shared/esbuild.config.js
+++ b/packages/shared/esbuild.config.js
@@ -1,9 +1,9 @@
 import { chrome, node } from '../../config/buildConstants.js';
-import fileURLToDirname from '../../config/fileURLToDirname.js';
+import fileUrlToDirname from '../../config/fileUrlToDirname.js';
 import getEsbuildConfig from '../../config/getEsbuildConfig.js';
 
 export default getEsbuildConfig({
-  absWorkingDir: fileURLToDirname(import.meta.url),
+  absWorkingDir: fileUrlToDirname(import.meta.url),
   entryPoints: ['src/index.ts'],
   outfile: 'dist/index.mjs',
   format: 'esm',