fix: Onboarding persistence and update interval (#1251)

Co-authored-by: littlekirkycode <littlekirky@googlemail.com>

Author: littlekirkycode

apps/code/src/main/services/updates/service.test.ts

@@ -686,21 +686,21 @@ describe("UpdatesService", () => {
       expect(mockAutoUpdater.checkForUpdates).toHaveBeenCalled();
     });
 
-    it("performs check every hour", async () => {
+    it("performs check every 24 hours", async () => {
       await initializeService(service);
 
       const initialCallCount =
         mockAutoUpdater.checkForUpdates.mock.calls.length;
 
-      // Advance 1 hour
-      await vi.advanceTimersByTimeAsync(60 * 60 * 1000);
+      // Advance 24 hours
+      await vi.advanceTimersByTimeAsync(24 * 60 * 60 * 1000);
 
       expect(mockAutoUpdater.checkForUpdates.mock.calls.length).toBe(
         initialCallCount + 1,
       );
 
-      // Advance another hour
-      await vi.advanceTimersByTimeAsync(60 * 60 * 1000);
+      // Advance another 24 hours
+      await vi.advanceTimersByTimeAsync(24 * 60 * 60 * 1000);
 
       expect(mockAutoUpdater.checkForUpdates.mock.calls.length).toBe(
         initialCallCount + 2,

apps/code/src/main/services/updates/service.ts

@@ -22,7 +22,7 @@ export class UpdatesService extends TypedEventEmitter<UpdatesEvents> {
   private static readonly SERVER_HOST = "https://update.electronjs.org";
   private static readonly REPO_OWNER = "PostHog";
   private static readonly REPO_NAME = "code";
-  private static readonly CHECK_INTERVAL_MS = 60 * 60 * 1000; // 1 hour
+  private static readonly CHECK_INTERVAL_MS = 24 * 60 * 60 * 1000; // 24 hours
   private static readonly CHECK_TIMEOUT_MS = 60 * 1000; // 1 minute timeout for checks
   private static readonly DISABLE_ENV_FLAG = "ELECTRON_DISABLE_AUTO_UPDATE";
   private static readonly SUPPORTED_PLATFORMS = ["darwin", "win32"];

apps/code/src/renderer/features/auth/stores/authStore.ts

@@ -966,7 +966,6 @@ export const useAuthStore = create<AuthState>()(
             needsProjectSelection: false,
             needsScopeReauth: false,
             hasCodeAccess: null,
-            hasCompletedOnboarding: false,
             selectedPlan: null,
             selectedOrgId: null,
           });

apps/code/src/renderer/features/settings/components/sections/AdvancedSettings.tsx

@@ -1,3 +1,4 @@
+import { useAuthStore } from "@features/auth/stores/authStore";
 import { SettingRow } from "@features/settings/components/SettingRow";
 import { useSettingsStore } from "@features/settings/stores/settingsStore";
 import { useFeatureFlag } from "@hooks/useFeatureFlag";
@@ -15,6 +16,20 @@ export function AdvancedSettings() {
 
   return (
     <Flex direction="column">
+      <SettingRow
+        label="Reset onboarding"
+        description="Re-run the onboarding tutorial on next app restart"
+      >
+        <Button
+          variant="soft"
+          size="1"
+          onClick={() =>
+            useAuthStore.setState({ hasCompletedOnboarding: false })
+          }
+        >
+          Reset
+        </Button>
+      </SettingRow>
       <SettingRow
         label="Clear application storage"
         description="This will remove all locally stored application data"