From 6d2f4f4cdf257cf84e7b89aae14c0d9781392216 Mon Sep 17 00:00:00 2001 From: Cursor Agent Date: Wed, 4 Mar 2026 04:59:14 +0000 Subject: [PATCH] fix: remove misleading log and redundant guard in handleCheckForUpdatesMenuClick The guard for status === 'downloaded' logged 'skipping automatic install' but the function only checks for updates, never installs. The guard was also redundant since checkForUpdates() already returns early for 'downloaded', 'downloading', and 'available' statuses with correct log messages. Co-authored-by: Julius Marminge Applied via @cursor push command --- apps/desktop/src/main.ts | 7 ------- 1 file changed, 7 deletions(-) diff --git a/apps/desktop/src/main.ts b/apps/desktop/src/main.ts index be36b10cd485cdf103daa424f7c1d4aacbd3c18f..8c712ee2dfc1fab16691e9c9d408e18bb48ccfca 100644 --- a/apps/desktop/src/main.ts +++ b/apps/desktop/src/main.ts @@ -424,13 +424,6 @@ function dispatchMenuAction(action: string): void { } function handleCheckForUpdatesMenuClick(): void { - if (updateState.status === "downloaded") { - console.info( - "[desktop-updater] Check for updates clicked while an update is already downloaded; skipping automatic install.", - ); - return; - } - if (!shouldEnableAutoUpdates()) { console.info("[desktop-updater] Manual update check requested, but updates are disabled."); return; -- 2.54.0