fix(updater): make update downloads non-blocking and announce new version

Return from app_update_download immediately while download continues in the background, so the Settings UI can show live progress without a long pending RPC. Also add a one-time startup toast when the app version changes to confirm a successful update.

Made-with: Cursor

Author: beautyfree

src/main/app-updater.ts

@@ -29,6 +29,7 @@ let remoteVersion: string | null = null;
 let lastError: string | null = null;
 let state: AppUpdateStatusJson["state"] = "idle";
 let updateDownloaded = false;
+let downloadPromise: Promise<string[]> | null = null;
 
 function snapshot(): AppUpdateStatusJson {
 	return {
@@ -170,20 +171,32 @@ export async function downloadUpdate(): Promise<AppUpdateStatusJson> {
 	// before the first `download-progress` event fires (those can lag 2–3s on
 	// slow networks).
 	setState("downloading");
-	try {
-		const result = await autoUpdater.downloadUpdate();
-		console.log("[updater] downloadUpdate() resolved with:", result);
-		// If update-downloaded event didn't fire yet (race on some platforms),
-		// force the state transition based on the promise resolution.
-		if (state !== "ready") {
-			updateDownloaded = true;
-			downloadProgress = 100;
-			setState("ready");
-		}
-	} catch (err) {
-		console.warn("[updater] downloadUpdate() rejected:", err);
-		lastError = (err as Error)?.message ?? String(err);
-		setState("error");
+	// Important UX behavior: return immediately so renderer request doesn't sit
+	// in pending for minutes on large updates. Progress/finish/error keeps coming
+	// through updater events pushed via `app_update_status_changed`.
+	if (!downloadPromise) {
+		downloadPromise = autoUpdater.downloadUpdate();
+		void downloadPromise
+			.then((result) => {
+				console.log("[updater] downloadUpdate() resolved with:", result);
+				// If update-downloaded event didn't fire yet (race on some
+				// platforms), force the state transition from the resolved promise.
+				if (state !== "ready") {
+					updateDownloaded = true;
+					downloadProgress = 100;
+					setState("ready");
+				}
+			})
+			.catch((err) => {
+				console.warn("[updater] downloadUpdate() rejected:", err);
+				lastError = (err as Error)?.message ?? String(err);
+				setState("error");
+			})
+			.finally(() => {
+				downloadPromise = null;
+			});
+	} else {
+		console.log("[updater] download already in progress");
 	}
 	return snapshot();
 }

src/mainview/App.tsx

@@ -17,18 +17,21 @@ import SettingsPage from './pages/Settings'
 import OnboardingWizard from './components/OnboardingWizard'
 import { useTheme } from './hooks/useTheme'
 import CloseConfirmDialog from './components/CloseConfirmDialog'
+import { useToast } from './components/ToastProvider'
 
 const GITHUB_REPO_URL =
   'https://github.com/beautyfree/skiller-skills-desktop-manager'
 const STAR_PROMPT_MIN_LAUNCHES = 3
 const STAR_PROMPT_MIN_AGE_MS = 24 * 60 * 60 * 1000
 const STAR_PROMPT_RESHOW_LAUNCHES = 3
 const STAR_PROMPT_RESHOW_MS = 3 * 24 * 60 * 60 * 1000
+const LAST_SEEN_VERSION_STORAGE_KEY = 'skiller.last_seen_app_version'
 
 function AppInner() {
   const queryClient = useQueryClient()
   const location = useLocation()
-  const { i18n } = useTranslation()
+  const { i18n, t } = useTranslation()
+  const { toast } = useToast()
   useTheme()
   const appOpenedTrackedRef = useRef(false)
   const [telemetryReady, setTelemetryReady] = useState(false)
@@ -45,6 +48,32 @@ function AppInner() {
     }
   })
 
+  useEffect(() => {
+    let cancelled = false
+    void invoke('get_app_version')
+      .then((currentVersion) => {
+        if (cancelled) return
+        try {
+          const previousVersion = localStorage.getItem(LAST_SEEN_VERSION_STORAGE_KEY)
+          if (previousVersion && previousVersion !== currentVersion) {
+            toast(
+              t('settings.updateWelcomeToast', {
+                from: previousVersion,
+                to: currentVersion,
+              }),
+            )
+          }
+          localStorage.setItem(LAST_SEEN_VERSION_STORAGE_KEY, currentVersion)
+        } catch {
+          // Ignore storage errors in private/locked environments.
+        }
+      })
+      .catch(() => {})
+    return () => {
+      cancelled = true
+    }
+  }, [t, toast])
+
   useEffect(() => {
     if (!telemetryReady || appOpenedTrackedRef.current) return
     appOpenedTrackedRef.current = true

src/mainview/i18n/en.ts

@@ -321,6 +321,7 @@ const en = {
     updateVersionLine: "Current {{current}} — latest {{latest}}",
     updateVersionOnly: "Version {{version}}",
     updateLastChecked: "Last checked {{time}}",
+    updateWelcomeToast: "Updated successfully: {{from}} → {{to}}",
     // Marketplace Cache
     onboardingTitle: "Onboarding",
     onboardingDescription: