feat(desktop): add standalone update settings tab

Expose update status, channel selection, automatic checks, and notifications in a dedicated desktop settings tab.

Keep download and installation behind explicit user actions.

Author: elkaix

apps/desktop/src/main.ts

@@ -45,10 +45,13 @@ import {
   installDownloadedUpdateNow,
   markUpdateNotified,
   setAutoUpdate,
+  setNotifyUpdate,
+  setUpdateChannel,
   skipUpdate,
   startUpdateDownload,
   undoSkippedUpdate,
   updateReleaseNotesUrl,
+  type UpdateChannel,
 } from './updater'
 import { windowAppearanceOptions } from './window-options'
 import { createDesktopLifecycle, type DesktopLifecycle } from './window-lifecycle'
@@ -200,6 +203,13 @@ function updateVersion(value: unknown): string {
   return value
 }
 
+function updateChannel(value: unknown): UpdateChannel {
+  if (value !== 'stable' && value !== 'beta' && value !== 'nightly') {
+    throw new TypeError('update channel must be stable, beta, or nightly')
+  }
+  return value
+}
+
 /** Install navigation and permission policy before the first renderer loads. */
 function hardenSession(): void {
   const desktopSession = session.defaultSession
@@ -267,6 +277,15 @@ ipcMain.handle('pythinker:update:set-auto', (event, enabled: unknown) => {
   if (typeof enabled !== 'boolean') throw new TypeError('automatic update checks must be a boolean')
   return setAutoUpdate(enabled)
 })
+ipcMain.handle('pythinker:update:set-channel', (event, channel: unknown) => {
+  assertTrustedSender(event)
+  return setUpdateChannel(updateChannel(channel))
+})
+ipcMain.handle('pythinker:update:set-notify', (event, enabled: unknown) => {
+  assertTrustedSender(event)
+  if (typeof enabled !== 'boolean') throw new TypeError('update notifications must be a boolean')
+  return setNotifyUpdate(enabled)
+})
 ipcMain.handle('pythinker:update:check', (event) => {
   assertTrustedSender(event)
   return checkForUpdatesNow()

apps/desktop/src/preload.ts

@@ -4,6 +4,9 @@ contextBridge.exposeInMainWorld('pythinkerDesktop', {
   platform: process.platform,
   getUpdateState: () => ipcRenderer.invoke('pythinker:update:get'),
   setAutoUpdate: (enabled: boolean) => ipcRenderer.invoke('pythinker:update:set-auto', enabled),
+  setUpdateChannel: (channel: 'stable' | 'beta' | 'nightly') =>
+    ipcRenderer.invoke('pythinker:update:set-channel', channel),
+  setNotifyUpdate: (enabled: boolean) => ipcRenderer.invoke('pythinker:update:set-notify', enabled),
   checkForUpdates: () => ipcRenderer.invoke('pythinker:update:check'),
   downloadUpdate: () => ipcRenderer.invoke('pythinker:update:download'),
   cancelUpdateDownload: () => ipcRenderer.invoke('pythinker:update:cancel'),

apps/desktop/src/updater.ts

@@ -12,8 +12,12 @@ const INITIAL_CHECK_DELAY_MS = 10_000
 const CHECK_INTERVAL_MS = 4 * 60 * 60 * 1_000
 const RELEASE_REPOSITORY_PATH = '/PyModel/pythinker-desktop-releases/releases/tag/'
 
+export type UpdateChannel = 'stable' | 'beta' | 'nightly'
+
 export interface UpdateSettings {
   readonly autoUpdate: boolean
+  readonly channel: UpdateChannel
+  readonly notifyUpdate: boolean
   readonly notifiedVersion?: string
   readonly skippedVersion?: string
   readonly pendingInstallVersion?: string
@@ -44,6 +48,8 @@ export type UpdateState = {
   bytesPerSecond?: number
   message?: string
   autoUpdate: boolean
+  channel: UpdateChannel
+  notifyUpdate: boolean
   notifiedVersion?: string
   skippedVersion?: string
   completedVersion?: string
@@ -54,12 +60,20 @@ export type UpdateTelemetryTrack = (
   properties?: Readonly<Record<string, string>>,
 ) => void
 
-const DEFAULT_SETTINGS: UpdateSettings = { autoUpdate: true }
+const DEFAULT_SETTINGS: UpdateSettings = {
+  autoUpdate: true,
+  channel: 'stable',
+  notifyUpdate: true,
+}
 
 function optionalString(value: unknown): string | undefined {
   return typeof value === 'string' && value.length > 0 ? value : undefined
 }
 
+function updateChannel(value: unknown): UpdateChannel {
+  return value === 'beta' || value === 'nightly' ? value : 'stable'
+}
+
 export function readUpdateSettings(dir: string): UpdateSettings {
   try {
     const parsed: unknown = JSON.parse(readFileSync(join(dir, UPDATE_SETTINGS_FILE), 'utf8'))
@@ -68,6 +82,8 @@ export function readUpdateSettings(dir: string): UpdateSettings {
       if (typeof source['autoUpdate'] === 'boolean') {
         return {
           autoUpdate: source['autoUpdate'],
+          channel: updateChannel(source['channel']),
+          notifyUpdate: typeof source['notifyUpdate'] === 'boolean' ? source['notifyUpdate'] : true,
           notifiedVersion: optionalString(source['notifiedVersion']),
           skippedVersion: optionalString(source['skippedVersion']),
           pendingInstallVersion: optionalString(source['pendingInstallVersion']),
@@ -125,6 +141,8 @@ let state: UpdateState = {
   status: app.isPackaged ? 'idle' : 'disabled',
   installedVersion: app.getVersion(),
   autoUpdate: settings.autoUpdate,
+  channel: settings.channel,
+  notifyUpdate: settings.notifyUpdate,
 }
 let getWindow: (() => BrowserWindow | undefined) | undefined
 let initialCheckTimer: ReturnType<typeof setTimeout> | undefined
@@ -183,6 +201,8 @@ function updateState(next: Partial<UpdateState>): void {
     ...state,
     ...next,
     autoUpdate: settings.autoUpdate,
+    channel: settings.channel,
+    notifyUpdate: settings.notifyUpdate,
     notifiedVersion: settings.notifiedVersion,
     skippedVersion: settings.skippedVersion,
     completedVersion: settings.completedVersion,
@@ -238,6 +258,9 @@ function hasUpdateConfig(): boolean {
 function configureExplicitConsent(): void {
   autoUpdater.autoDownload = false
   autoUpdater.autoInstallOnAppQuit = false
+  autoUpdater.channel = settings.channel === 'stable' ? null : settings.channel
+  autoUpdater.allowPrerelease = settings.channel !== 'stable'
+  autoUpdater.allowDowngrade = false
 }
 
 function disableUpdates(): UpdateState {
@@ -366,6 +389,8 @@ export function initUpdater(
     status: app.isPackaged ? 'idle' : 'disabled',
     installedVersion: app.getVersion(),
     autoUpdate: settings.autoUpdate,
+    channel: settings.channel,
+    notifyUpdate: settings.notifyUpdate,
     notifiedVersion: settings.notifiedVersion,
     skippedVersion: settings.skippedVersion,
     completedVersion: settings.completedVersion,
@@ -420,6 +445,48 @@ export function setAutoUpdate(enabled: boolean): UpdateState {
   return state
 }
 
+export function setUpdateChannel(channel: UpdateChannel): UpdateState {
+  if (settings.channel === channel) return state
+  if (state.status === 'checking' || state.status === 'downloading' || state.status === 'downloaded') {
+    throw new Error('Cannot change update channel while an update operation is active')
+  }
+
+  persistSettings({
+    ...settings,
+    channel,
+    notifiedVersion: undefined,
+    skippedVersion: undefined,
+  })
+
+  if (app.isPackaged && hasUpdateConfig()) {
+    try {
+      configureExplicitConsent()
+    } catch (error) {
+      stateError(error)
+      return state
+    }
+  }
+
+  updateState({
+    status: app.isPackaged && hasUpdateConfig() ? 'idle' : 'disabled',
+    availableVersion: undefined,
+    releaseDate: undefined,
+    releaseNotes: undefined,
+    lastCheckedAt: undefined,
+    percent: undefined,
+    transferred: undefined,
+    total: undefined,
+    bytesPerSecond: undefined,
+    message: undefined,
+  })
+  return state
+}
+
+export function setNotifyUpdate(enabled: boolean): UpdateState {
+  persistSettings({ ...settings, notifyUpdate: enabled })
+  return state
+}
+
 export async function checkForUpdatesNow(): Promise<UpdateState> {
   if (!app.isPackaged) {
     updateState({ status: 'disabled' })

apps/desktop/tests/updater.spec.ts

@@ -56,6 +56,9 @@ afterEach(() => {
   vi.clearAllMocks()
   autoUpdater.autoDownload = undefined as unknown as boolean
   autoUpdater.autoInstallOnAppQuit = undefined as unknown as boolean
+  autoUpdater.allowPrerelease = undefined as unknown as boolean
+  autoUpdater.allowDowngrade = undefined as unknown as boolean
+  autoUpdater.channel = null
 })
 
 function temporaryDirectory(): string {
@@ -66,27 +69,56 @@ function temporaryDirectory(): string {
 
 describe('update settings', () => {
   it('defaults automatic updates to enabled when the file is missing', () => {
-    expect(readUpdateSettings(temporaryDirectory())).toEqual({ autoUpdate: true })
+    expect(readUpdateSettings(temporaryDirectory())).toEqual({
+      autoUpdate: true,
+      channel: 'stable',
+      notifyUpdate: true,
+    })
   })
 
   it('defaults automatic updates to enabled when the file is corrupt', () => {
     const directory = temporaryDirectory()
     writeFileSync(join(directory, 'update-settings.json'), '{not-json', 'utf8')
 
-    expect(readUpdateSettings(directory)).toEqual({ autoUpdate: true })
+    expect(readUpdateSettings(directory)).toEqual({
+      autoUpdate: true,
+      channel: 'stable',
+      notifyUpdate: true,
+    })
+  })
+
+  it('round-trips automatic checks, update notifications, and the channel', () => {
+    const directory = temporaryDirectory()
+    writeUpdateSettings(directory, {
+      autoUpdate: false,
+      channel: 'beta',
+      notifyUpdate: false,
+    })
+
+    expect(readUpdateSettings(directory)).toEqual({
+      autoUpdate: false,
+      channel: 'beta',
+      notifyUpdate: false,
+    })
   })
 
-  it('round-trips the automatic-updates setting', () => {
+  it('uses safe defaults for missing or invalid new settings', () => {
     const directory = temporaryDirectory()
-    writeUpdateSettings(directory, { autoUpdate: false })
+    writeFileSync(join(directory, 'update-settings.json'), '{"autoUpdate":false,"channel":"preview"}\n', 'utf8')
 
-    expect(readUpdateSettings(directory)).toEqual({ autoUpdate: false })
+    expect(readUpdateSettings(directory)).toEqual({
+      autoUpdate: false,
+      channel: 'stable',
+      notifyUpdate: true,
+    })
   })
 
   it('persists update notification, skip, install, and completion receipts separately', () => {
     const directory = temporaryDirectory()
     const value = {
       autoUpdate: true,
+      channel: 'nightly' as const,
+      notifyUpdate: true,
       notifiedVersion: '1.2.3',
       skippedVersion: '1.2.3',
       pendingInstallVersion: '1.3.0',
@@ -106,7 +138,13 @@ describe('update telemetry transitions', () => {
     const track = (event: string): void => {
       events.push(event)
     }
-    const previous: UpdateState = { status: 'idle', installedVersion: '1.0.0', autoUpdate: true }
+    const previous: UpdateState = {
+      status: 'idle',
+      installedVersion: '1.0.0',
+      autoUpdate: true,
+      channel: 'stable',
+      notifyUpdate: true,
+    }
 
     trackUpdateTransition(previous, { ...previous, status: 'checking' }, track)
     trackUpdateTransition(previous, { ...previous, status: 'available', availableVersion: '0.2.0' }, track)
@@ -136,7 +174,11 @@ describe('release notes URL', () => {
 describe('packaged builds without update metadata', () => {
   it('disables updates without wiring updater events', () => {
     const directory = temporaryDirectory()
-    writeUpdateSettings(directory, { autoUpdate: false })
+    writeUpdateSettings(directory, {
+      autoUpdate: false,
+      channel: 'stable',
+      notifyUpdate: true,
+    })
     vi.mocked(app.getPath).mockReturnValue(directory)
     Object.defineProperty(app, 'isPackaged', { configurable: true, value: true })
     Object.defineProperty(process, 'resourcesPath', { configurable: true, value: directory })
@@ -154,6 +196,45 @@ describe('packaged builds without update metadata', () => {
 })
 
 describe('strict update consent', () => {
+  it('changes channels and notification preference without checking, downloading, installing, or downgrading', async () => {
+    vi.resetModules()
+    const directory = temporaryDirectory()
+    writeFileSync(join(directory, 'app-update.yml'), '', 'utf8')
+    writeFileSync(
+      join(directory, 'update-settings.json'),
+      '{"autoUpdate":false,"channel":"beta","notifyUpdate":true}\n',
+      'utf8',
+    )
+    const { app: localApp } = await import('electron')
+    const { default: localElectronUpdater } = await import('electron-updater')
+    const {
+      initUpdater: initLocalUpdater,
+      setNotifyUpdate: setLocalNotifyUpdate,
+      setUpdateChannel: setLocalUpdateChannel,
+    } = await import('../src/updater')
+    const localAutoUpdater = localElectronUpdater.autoUpdater
+    vi.mocked(localApp.getPath).mockReturnValue(directory)
+    Object.defineProperty(localApp, 'isPackaged', { configurable: true, value: true })
+    Object.defineProperty(process, 'resourcesPath', { configurable: true, value: directory })
+
+    initLocalUpdater(() => undefined)
+    expect(localAutoUpdater.channel).toBe('beta')
+    expect(localAutoUpdater.allowPrerelease).toBe(true)
+    expect(localAutoUpdater.allowDowngrade).toBe(false)
+
+    expect(setLocalUpdateChannel('nightly')).toMatchObject({ channel: 'nightly', status: 'idle' })
+    expect(setLocalNotifyUpdate(false)).toMatchObject({ notifyUpdate: false })
+    expect(setLocalUpdateChannel('stable')).toMatchObject({ channel: 'stable', status: 'idle' })
+    expect(localAutoUpdater.channel).toBeNull()
+    expect(localAutoUpdater.allowPrerelease).toBe(false)
+    expect(localAutoUpdater.allowDowngrade).toBe(false)
+    expect(localAutoUpdater.autoDownload).toBe(false)
+    expect(localAutoUpdater.autoInstallOnAppQuit).toBe(false)
+    expect(localAutoUpdater.checkForUpdates).not.toHaveBeenCalled()
+    expect(localAutoUpdater.downloadUpdate).not.toHaveBeenCalled()
+    expect(localAutoUpdater.quitAndInstall).not.toHaveBeenCalled()
+  })
+
   it('manual check cannot download an update', async () => {
     vi.resetModules()
     const directory = temporaryDirectory()

apps/pythinker-code/dist-web/.web-bundle-manifest.json

@@ -1,4 +1,4 @@
 {
-  "sourceHash": "9b3dc1691590f09af45eb3cab7d356ecf76b4ebe1b2ca9ec4d89b9b50c610d9a",
+  "sourceHash": "f1c3ebe2f168111a6fd779f6593566ba221abc86d20272fffb04554f39958da5",
   "sourceFileCount": 399
 }