refactor: apply audit cleanup (reuse + simplify)

- domain-check: drop the redundant isIpLiteral guard (isLoopbackIp already
  validates and returns false for non-literals).
- session.ts: use shared getErrorMessage instead of the local error ternary
  (the file already imports it).
- tray.ts: use shared sleep() instead of a hand-rolled setTimeout promise.
- updater.ts: distinguish the synchronous-throw log from the async-rejection
  log on the manual update check.

Author: waleedlatif1

apps/desktop/src/main/browser-agent/session.ts

@@ -198,7 +198,7 @@ function createTabView(): WebContentsView {
         void tab.view.webContents.loadURL(details.url).catch(() => {})
       } catch (error) {
         logger.warn('Could not open browser popup in a new tab', {
-          error: error instanceof Error ? error.message : String(error),
+          error: getErrorMessage(error),
         })
       }
     }

apps/desktop/src/main/tray.ts

@@ -1,5 +1,6 @@
 import { join } from 'node:path'
 import { createLogger } from '@sim/logger'
+import { sleep } from '@sim/utils/helpers'
 import type { MenuItemConstructorOptions, NativeImage } from 'electron'
 import { app, Menu, nativeImage, session, Tray } from 'electron'
 import { isSafeInternalPath } from '@/main/config'
@@ -285,10 +286,7 @@ export function installTray(deps: TrayDeps): TrayHandle | null {
   const EMPTY_CACHE_POP_GRACE_MS = 600
   const popMenu = async () => {
     if (cachedChats.length === 0) {
-      await Promise.race([
-        refreshChats(),
-        new Promise((resolve) => setTimeout(resolve, EMPTY_CACHE_POP_GRACE_MS)),
-      ])
+      await Promise.race([refreshChats(), sleep(EMPTY_CACHE_POP_GRACE_MS)])
     }
     if (tray.isDestroyed()) return
     tray.popUpContextMenu(Menu.buildFromTemplate(buildTrayMenuTemplate(deps, cachedChats)))

apps/desktop/src/main/updater.ts

@@ -238,6 +238,6 @@ export function checkForUpdatesInteractive(deps: UpdaterDeps): void {
         })
       })
   } catch (error) {
-    logger.error('Manual update check failed', { error })
+    logger.error('Manual update check threw synchronously', { error })
   }
 }

apps/sim/lib/mcp/domain-check.ts

@@ -104,7 +104,7 @@ export function validateMcpDomain(url: string | undefined): void {
 function isLocalhostHostname(hostname: string): boolean {
   const clean = hostname.toLowerCase()
   if (clean === 'localhost') return true
-  return isIpLiteral(clean) && isLoopbackIp(clean)
+  return isLoopbackIp(clean)
 }
 
 /**