chore: refactor logs (#646)

Author: peppescg

main/src/graceful-exit.ts

@@ -77,8 +77,6 @@ export async function stopAllServers(
 
   // Store the servers that are about to be shut down
   shutdownStore.set('lastShutdownServers', servers)
-  log.info(`Stored ${servers.length} servers for shutdown tracking`)
-
   log.info(`Stopping ${servers.length} servers...`)
 
   // First, initiate stop for all servers

main/src/main.ts

@@ -66,18 +66,12 @@ app.on('ready', () => {
       return
     }
 
-    log.info('Simulating a new release for testing purposes')
     mainWindow.webContents.send('update-downloaded')
   }, 2000)
 })
 
-autoUpdater.on('before-quit-for-update', () => {
-  log.info('🔄 before-quit-for-update event fired')
-})
-
 autoUpdater.on('update-downloaded', (_, __, releaseName) => {
   log.info('🔄 Update downloaded - showing dialog')
-  log.info(`📦 Release info: ${releaseName}`)
 
   if (!mainWindow) {
     log.error('MainWindow not available for update dialog')
@@ -108,12 +102,8 @@ autoUpdater.on('update-downloaded', (_, __, releaseName) => {
   dialog
     .showMessageBox(mainWindow, dialogOpts)
     .then(async (returnValue) => {
-      log.info(
-        `🎯 User clicked: ${returnValue.response === 0 ? 'Restart' : 'Later'}`
-      )
-
       if (returnValue.response === 0) {
-        log.info('🎯 User clicked: Restart')
+        log.debug('User clicked: Restart')
         isUpdateInProgress = true
 
         log.info('🛑 Removing quit listeners to avoid interference')
@@ -123,13 +113,10 @@ autoUpdater.on('update-downloaded', (_, __, releaseName) => {
         isQuitting = true
         tearingDown = true
 
-        log.info('🔄 Starting restart process...')
-
         try {
           log.info('🛑 Starting graceful shutdown before update...')
           mainWindow?.webContents.send('graceful-exit')
 
-          log.info('⏳ Waiting for renderer...')
           await delay(500)
 
           const port = getToolhivePort()
@@ -152,7 +139,9 @@ autoUpdater.on('update-downloaded', (_, __, releaseName) => {
         }
       } else {
         isUpdateInProgress = false
-        log.info('⏰ User chose Later - showing toast notification')
+        log.info(
+          'User deferred update installation - showing toast notification'
+        )
         if (mainWindow) {
           mainWindow.webContents.send('update-downloaded')
         }
@@ -239,7 +228,6 @@ if (!gotTheLock) {
 } else {
   app.on('second-instance', () => {
     // Someone tried to run a second instance, focus our window instead
-    log.info('Second instance attempted, focusing existing window')
     if (mainWindow) {
       if (mainWindow.isMinimized()) mainWindow.restore()
       mainWindow.focus()

main/src/system-tray.ts

@@ -213,11 +213,6 @@ const createClickHandler = () => {
 
   const toggleWindow = (window: BrowserWindow) => {
     setTimeout(() => {
-      log.info('TOGGLE WIN: ', {
-        winIsVisible: window.isVisible(),
-        winIsMinimized: window.isMinimized(),
-        check: window.isVisible() && !window.isMinimized(),
-      })
       showWindowWithFocus(window)
     }, 50)
   }

main/src/toolhive-manager.ts

@@ -58,10 +58,6 @@ function findFreePort(): Promise<number> {
 }
 
 export async function startToolhive(tray?: Tray): Promise<void> {
-  log.info(
-    `[startToolhive] Starting - Current state: isRunning=${isToolhiveRunning()}, isRestarting=${isRestarting}`
-  )
-
   if (!existsSync(binPath)) {
     log.error(`ToolHive binary not found at: ${binPath}`)
     return
@@ -135,9 +131,6 @@ export async function restartToolhive(tray?: Tray): Promise<void> {
 }
 
 export function stopToolhive(): void {
-  log.info(
-    `[stopToolhive] Current state: isRunning=${isToolhiveRunning()}, PID: ${toolhiveProcess?.pid}`
-  )
   if (toolhiveProcess && !toolhiveProcess.killed) {
     log.info('Stopping ToolHive process...')
     toolhiveProcess.kill()

renderer/src/common/components/error/connection-refused-error.tsx

@@ -64,9 +64,6 @@ export function ConnectionRefusedError() {
   })
 
   useEffect(() => {
-    if (!containerStatus?.available) {
-      log.info('Container engines are not available')
-    }
     if (containerStatus?.available && !isRestarting) {
       restartToolhive()
     }

renderer/src/routes/__root.tsx

@@ -78,7 +78,6 @@ export const Route = createRootRouteWithContext<{
 
     try {
       isUpdateInProgress = await window.electronAPI.isUpdateInProgress()
-      log.info(`[beforeLoad] Update in progress: ${isUpdateInProgress}`)
     } catch (e) {
       log.debug(`[beforeLoad] Update check API not available: ${e}`)
     }
@@ -92,11 +91,9 @@ export const Route = createRootRouteWithContext<{
       queryKey: ['is-toolhive-running'],
       queryFn: async () => {
         const res = await window.electronAPI.isToolhiveRunning()
-        const appVersion = await window.electronAPI.getAppVersion()
         if (!res) {
           log.error('ToolHive is not running')
         }
-        log.info(`ToolHive version ${appVersion} is running`)
         return res
       },
       retry: 3,