From b6facd02a7680731953c1f7e201d0cb4ea82d831 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?R=C5=ABdolfs=20O=C5=A1i=C5=86=C5=A1?= <rudolfs@osins.org>
Date: Tue, 24 Mar 2026 10:28:20 -0600
Subject: [PATCH] Fix update checker not detecting new versions

enable() was called before invoke("version") resolved, so the first
checkNewVersion() could run while sanitizedCurrentVersion was still
undefined. Even though newVersion is a derived value, Svelte 5 may
not reliably recompute it when state changes asynchronously on a
module-level class instance.

Moving enable() inside the invoke callback ensures sanitizedCurrentVersion
is always set before any version comparison runs.
---
 src/lib/updateChecker.svelte.ts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/lib/updateChecker.svelte.ts b/src/lib/updateChecker.svelte.ts
index e88192168..34c022697 100644
--- a/src/lib/updateChecker.svelte.ts
+++ b/src/lib/updateChecker.svelte.ts
@@ -58,9 +58,6 @@ class UpdateChecker {
 
   public static init(): UpdateChecker {
     const updateChecker = new UpdateChecker();
-    if (isEnabledStore.value) {
-      updateChecker.enable();
-    }
 
     void invoke<string>("version").then(currentVersion => {
       updateChecker.currentVersion = currentVersion.toString();
@@ -68,6 +65,9 @@ class UpdateChecker {
       if (version) {
         updateChecker.sanitizedCurrentVersion = version.toString();
       }
+      if (isEnabledStore.value) {
+        updateChecker.enable();
+      }
     });
 
     return updateChecker;