refactor(auto-update): should triggle download when checking for update manually (#5262)

refactor(auto-update): streamline update check logic and enhance error handling

- Moved update check logic from IPC handler to AppUpdater class for better organization.
- Improved error handling during update checks to ensure graceful failure.
- Removed redundant conditions in the AboutSettings component for cleaner rendering.

src/main/ipc.ts

@@ -178,19 +178,7 @@ export function registerIpc(mainWindow: BrowserWindow, app: Electron.App) {
 
   // check for update
   ipcMain.handle(IpcChannel.App_CheckForUpdate, async () => {
-    if (isWin && 'PORTABLE_EXECUTABLE_DIR' in process.env) {
-      return {
-        currentVersion: app.getVersion(),
-        updateInfo: null
-      }
-    }
-
-    const update = await appUpdater.autoUpdater.checkForUpdates()
-
-    return {
-      currentVersion: appUpdater.autoUpdater.currentVersion,
-      updateInfo: update?.updateInfo
-    }
+    await appUpdater.checkForUpdates()
   })
 
   // zip

src/main/services/AppUpdater.ts

@@ -1,3 +1,4 @@
+import { isWin } from '@main/constant'
 import { IpcChannel } from '@shared/IpcChannel'
 import { UpdateInfo } from 'builder-util-runtime'
 import { app, BrowserWindow, dialog } from 'electron'
@@ -60,6 +61,35 @@ export default class AppUpdater {
     autoUpdater.autoInstallOnAppQuit = isActive
   }
 
+  public async checkForUpdates() {
+    if (isWin && 'PORTABLE_EXECUTABLE_DIR' in process.env) {
+      return {
+        currentVersion: app.getVersion(),
+        updateInfo: null
+      }
+    }
+
+    try {
+      const update = await this.autoUpdater.checkForUpdates()
+      if (update?.updateInfo && !this.autoUpdater.autoDownload) {
+        // 如果 autoDownload 为 false，则需要再调用下面的函数触发下
+        // do not use await, because it will block the return of this function
+        this.autoUpdater.downloadUpdate()
+      }
+
+      return {
+        currentVersion: this.autoUpdater.currentVersion,
+        updateInfo: update?.updateInfo
+      }
+    } catch (error) {
+      logger.error('Failed to check for update:', error)
+      return {
+        currentVersion: app.getVersion(),
+        updateInfo: null
+      }
+    }
+  }
+
   public async showUpdateDialog(mainWindow: BrowserWindow) {
     if (!this.releaseInfo) {
       return

src/renderer/src/pages/settings/AboutSettings.tsx

@@ -160,7 +160,7 @@ const AboutSettings: FC = () => {
           </>
         )}
       </SettingGroup>
-      {autoCheckUpdate && hasNewVersion && update.info && (
+      {hasNewVersion && update.info && (
         <SettingGroup theme={theme}>
           <SettingRow>
             <SettingRowTitle>