refactor: add centralized error logging and handling across main and renderer process

Author: antonreshetov

src/main/db/index.ts

@@ -5,6 +5,7 @@ import Database from 'better-sqlite3'
 import { format } from 'date-fns'
 import fs from 'fs-extra'
 import { store } from '../store'
+import { log } from '../utils'
 
 const DB_NAME = 'massCode.db'
 const isDev = process.env.NODE_ENV === 'development'
@@ -136,7 +137,7 @@ export function useDB() {
     return db
   }
   catch (error) {
-    console.error('Database initialization failed:', error)
+    log('Database initialization failed', error)
     throw error
   }
 }
@@ -167,7 +168,7 @@ export function reloadDB() {
     console.log(`Database successfully reloaded: ${dbPath}`)
   }
   catch (error) {
-    console.error('Error while reloading the database:', error)
+    log('Error while reloading the database', error)
     throw error
   }
 }
@@ -201,7 +202,7 @@ export function clearDB() {
     stmt()
   }
   catch (error) {
-    console.error('Error while clearing the database:', error)
+    log('Error while clearing the database', error)
     throw error
   }
 }
@@ -228,7 +229,7 @@ export async function moveDB(path: string) {
     reloadDB()
   }
   catch (error) {
-    console.error('Error while moving the database:', error)
+    log('Error while moving the database', error)
     throw error
   }
 }
@@ -256,7 +257,7 @@ export async function createBackup(manual = false) {
     return backupFilePath
   }
   catch (error) {
-    console.error('Error creating database backup:', error)
+    log('Error creating database backup', error)
     throw error
   }
 }
@@ -338,12 +339,12 @@ export async function startAutoBackup() {
         }
       }
       catch (error) {
-        console.error('Error during scheduled backup:', error)
+        log('Error during scheduled backup', error)
       }
     }, intervalMs)
   }
   catch (error) {
-    console.error('Error starting auto backup:', error)
+    log('Error starting auto backup', error)
   }
 }
 
@@ -377,7 +378,7 @@ export async function restoreFromBackup(backupFilePath: string) {
     console.warn(`Database restored from backup: ${backupFilePath}`)
   }
   catch (error) {
-    console.error('Error restoring database from backup:', error)
+    log('Error restoring database from backup', error)
     throw error
   }
 }
@@ -415,7 +416,7 @@ export async function getBackupList() {
     )
   }
   catch (error) {
-    console.error('Error getting backup list:', error)
+    log('Error getting backup list', error)
     return []
   }
 }
@@ -454,7 +455,7 @@ export async function moveBackupStorage(newPath: string) {
     store.preferences.set('backup.path', newPath)
   }
   catch (error) {
-    console.error('Error while moving backup storage:', error)
+    log('Error while moving backup storage', error)
     throw error
   }
 }

src/main/index.ts

@@ -9,6 +9,7 @@ import { registerIPC } from './ipc'
 import { mainMenu } from './menu/main'
 import { store } from './store'
 import { checkForUpdates } from './updates'
+import { log } from './utils'
 
 process.env.ELECTRON_DISABLE_SECURITY_WARNINGS = 'true' // Отключаем security warnings
 
@@ -73,12 +74,41 @@ if (!gotTheLock) {
   app.quit()
 }
 else {
-  app.whenReady().then(() => {
-    createWindow()
-    registerIPC()
-    initApi()
-    checkForUpdates()
-    startAutoBackup()
+  app.whenReady().then(async () => {
+    try {
+      createWindow()
+    }
+    catch (error) {
+      log('Error creating window', error)
+    }
+
+    try {
+      registerIPC()
+    }
+    catch (error) {
+      log('Error registering IPC', error)
+    }
+
+    try {
+      initApi()
+    }
+    catch (error) {
+      log('Error initializing API', error)
+    }
+
+    try {
+      checkForUpdates()
+    }
+    catch (error) {
+      log('Error checking for updates', error)
+    }
+
+    try {
+      await startAutoBackup()
+    }
+    catch (error) {
+      log('Error starting auto backup', error)
+    }
 
     if (store.app.get('isAutoMigratedFromJson')) {
       return
@@ -92,7 +122,7 @@ else {
       store.app.set('isAutoMigratedFromJson', true)
     }
     catch (err) {
-      console.error('Error on auto migration JSON to SQLite:', err)
+      log('Error on auto migration JSON to SQLite', err)
     }
   })
 
@@ -126,4 +156,22 @@ else {
   app.on('open-url', (_, url) => {
     BrowserWindow.getFocusedWindow()?.webContents.send('system:deep-link', url)
   })
+
+  // Global error handlers
+  process.on('uncaughtException', (err) => {
+    BrowserWindow.getFocusedWindow()?.webContents.send('system:error', {
+      source: 'main',
+      message: err.message,
+      stack: err.stack,
+    })
+  })
+
+  process.on('unhandledRejection', (reason) => {
+    const err = reason instanceof Error ? reason : new Error(String(reason))
+    BrowserWindow.getFocusedWindow()?.webContents.send('system:error', {
+      source: 'main',
+      message: err.message,
+      stack: err.stack,
+    })
+  })
 }

src/main/types/ipc.ts

@@ -41,6 +41,7 @@ type SystemAction =
   | 'open-external'
   | 'deep-link'
   | 'update-available'
+  | 'error'
 type PrettierAction = 'format'
 type FsAction = 'assets'
 

src/main/utils/index.ts

@@ -0,0 +1,14 @@
+import { BrowserWindow } from 'electron'
+
+export function log(context: string, error: unknown): void {
+  const message = error instanceof Error ? error.message : String(error)
+  const stack = error instanceof Error ? error.stack : undefined
+
+  console.error(`[${context}] ${message}`, error)
+
+  BrowserWindow.getFocusedWindow()?.webContents.send('system:error', {
+    context,
+    message,
+    stack,
+  })
+}

src/renderer/ipc/listeners/system.ts

@@ -48,4 +48,8 @@ export function registerSystemListeners() {
       },
     })
   })
+
+  ipc.on('system:error', (_, payload) => {
+    console.error(`[system][${payload.context}]`, payload)
+  })
 }