Merge pull request #24 from a23bc/fix/engine-update-prompt

fix: ipatool update prompt, engine settings UX, account list polish (v0.1.1)

Author: a23bc

package.json

@@ -1,7 +1,7 @@
 {
   "name": "ipatool-gui",
   "productName": "IPATool GUI",
-  "version": "0.1.0",
+  "version": "0.1.1",
   "engines": {
     "node": ">=20.19"
   },

src/main/accounts.ts

@@ -196,7 +196,7 @@ export class AccountRegistry extends EventEmitter {
   async snapshot(): Promise<AccountsSnapshot> {
     const bridge = await slotBridge()
     const activeId = this.activeId
-    const profiles = sortAccounts(this.list(), activeId)
+    const profiles = sortAccounts(this.list())
     const shared = profiles.some((profile) => this.effectiveStore(profile) === 'os')
     return {
       // Unused slots are not listed: the switcher must not offer a nameless

src/main/engine.ts

@@ -26,7 +26,8 @@ import { gunzip } from 'node:zlib'
 import { promisify } from 'node:util'
 import { mkdir, rename, rm, writeFile, chmod } from 'node:fs/promises'
 import { app } from 'electron'
-import type { EngineDownloadProgress, EngineErrorCode, EngineRelease, EngineStatus } from '../shared/types'
+import type { EngineDownloadProgress, EngineErrorCode, EngineRelease, EngineStatus, EngineUpdateCheck } from '../shared/types'
+import { compareSemver } from '../shared/semver'
 import { parseSha256Sum, parseVersion } from '../shared/ipatool/parse'
 import { findTarFile, parseTar } from '../shared/tar'
 import {
@@ -287,6 +288,37 @@ export class EngineManager extends EventEmitter {
     }
   }
 
+  /**
+   * Compares the installed ipatool with the newest published release.
+   *
+   * `checkAppUpdate` is about *this* GUI; engine updates are separate and only
+   * look at majd/ipatool tags. Prereleases never count as "newer".
+   */
+  async checkUpdate(): Promise<EngineUpdateCheck> {
+    const current = (this.current.version ?? '').replace(/^v/, '')
+    try {
+      const list = await this.releases(15)
+      const stable = list
+        .filter((r) => !r.prerelease && r.hasAssetForThisPlatform)
+        .map((r) => r.version.replace(/^v/, ''))
+      const latest = stable.length > 0 ? (stable[0] ?? '') : ''
+      // releases() is newest-first; pick the max in case GitHub order shifts.
+      let newest = latest
+      for (const v of stable) {
+        if (compareSemver(v, newest) > 0) newest = v
+      }
+      const hasUpdate = current !== '' && newest !== '' && compareSemver(newest, current) > 0
+      return { current, latest: newest === '' ? null : newest, hasUpdate, error: null }
+    } catch (error) {
+      return {
+        current,
+        latest: null,
+        hasUpdate: false,
+        error: error instanceof Error ? error.message : String(error)
+      }
+    }
+  }
+
   /**
    * Downloads, verifies and installs ipatool. Emits progress through the
    * `status` event so the UI can show a real percentage.

src/main/index.ts

@@ -182,6 +182,21 @@ async function postStartup(): Promise<void> {
     /* surfaced through the engine status event */
   }
 
+  // Tell the UI when a newer ipatool is out - the version picker can already
+  // see it, but nothing used to say "you are behind".
+  if (engineManager.state.state === 'ready') {
+    void engineManager
+      .checkUpdate()
+      .then((result) => {
+        if (result.hasUpdate) {
+          for (const win of BrowserWindow.getAllWindows()) {
+            if (!win.isDestroyed()) win.webContents.send('engine:update', result)
+          }
+        }
+      })
+      .catch(() => undefined)
+  }
+
   // Only worth asking once the engine is actually usable. The probe confirms the
   // active account's session, learns its identity (or adopts a session the
   // terminal CLI created) and publishes the whole account list.

src/main/ipc.ts

@@ -217,6 +217,7 @@ export function registerIpc(): void {
   ipcMain.handle(IPC.EngineReleases, () =>
     wrap(() => engineManager.releases(30))
   )
+  ipcMain.handle(IPC.EngineCheckUpdate, () => engineManager.checkUpdate())
   ipcMain.handle(IPC.AppCheckUpdate, () => checkAppUpdate())
 
   /* ---------------------------------------------------------------- *

src/main/session.ts

@@ -118,6 +118,9 @@ export async function switchTo(accountId: string): Promise<{ snapshot: AccountsS
  */
 export async function refreshAll(onlyUnknown = false): Promise<AccountsSnapshot> {
   for (const profile of accounts.list()) {
+    // Never-used slots are hidden from the UI and have nothing to probe -
+    // running `auth info` on them only produces keyring errors in the log.
+    if (!accounts.hasRecordedSession(profile) && profile.remark.trim() === '') continue
     if (onlyUnknown && accounts.hasRecordedSession(profile)) continue
     try {
       await verifySession(profile.id)

src/preload/index.ts

@@ -53,6 +53,7 @@ import type { QueueAction, SearchRequest, PurchasesRequest, EnqueueOptions } fro
  */
 const EVENT_CHANNELS: ReadonlySet<string> = new Set<EventChannel>([
   'engine:status',
+  'engine:update',
   'engine:progress',
   'accounts:changed',
   'settings:changed',
@@ -99,6 +100,7 @@ const api: RendererApi = {
   installEngine: (version?: string, force?: boolean): Promise<EngineStatus> =>
     ipcRenderer.invoke(IPC.EngineInstall, version, force),
   listEngineReleases: (): Promise<Operation<EngineRelease[]>> => ipcRenderer.invoke(IPC.EngineReleases),
+  checkEngineUpdate: () => ipcRenderer.invoke(IPC.EngineCheckUpdate),
   uninstallEngine: (): Promise<EngineStatus> => ipcRenderer.invoke(IPC.EngineUninstall),
   checkAppUpdate: (): Promise<UpdateCheckResult> => ipcRenderer.invoke(IPC.AppCheckUpdate),
 

src/renderer/src/components/AccountManager.tsx

@@ -172,6 +172,7 @@ export function AccountManager(): ReactNode {
   const updateAccount = useAppStore((state) => state.updateAccount)
   const removeAccount = useAppStore((state) => state.removeAccount)
   const getAccounts = useAppStore((state) => state.getAccounts)
+  const refreshAccounts = useAppStore((state) => state.refreshAccounts)
 
   const open = useUiStore((state) => state.accountsOpen)
   const setOpen = useUiStore((state) => state.setAccountsOpen)
@@ -249,8 +250,18 @@ export function AccountManager(): ReactNode {
       width={560}
       footer={
         <>
-          <button type="button" className="btn" onClick={() => void getAccounts()}>
-            <Icon name="refresh" size={13} />
+          <button
+            type="button"
+            className="btn"
+            disabled={busy}
+            onClick={() => {
+              void run(async () => {
+                await refreshAccounts()
+                toast({ kind: 'success', message: t('accounts.verified') })
+              })
+            }}
+          >
+            {busy ? <Spinner size={13} /> : <Icon name="refresh" size={13} />}
             {t('accounts.refreshAll')}
           </button>
           <button type="button" className="btn btn-primary" onClick={onAdd} disabled={busy}>

src/renderer/src/components/Header.tsx

@@ -19,13 +19,17 @@ function useChromePadding(): { left: number; right: number } {
 
 function EnginePill(): ReactNode {
   const engine = useAppStore((state) => state.engine)
+  const engineUpdate = useAppStore((state) => state.engineUpdate)
   const t = useAppStore((state) => state.t)
   const setView = useUiStore((state) => state.setView)
 
   let label: string
   let tone: 'ok' | 'warn' | 'bad'
 
-  if (engineReady(engine)) {
+  if (engineReady(engine) && engineUpdate?.hasUpdate) {
+    label = t('engine.pill.ready', { version: engine.version ?? '' })
+    tone = 'warn'
+  } else if (engineReady(engine)) {
     label = t('engine.pill.ready', { version: engine.version ?? '' })
     tone = 'ok'
   } else if (engine.state === 'downloading') {
@@ -55,7 +59,11 @@ function EnginePill(): ReactNode {
       onClick={() => setView('settings')}
       className="no-drag flex h-[26px] shrink-0 items-center gap-1.5 rounded-full px-2.5 text-[11.5px] font-semibold transition-transform active:scale-[0.97]"
       style={{ background: colors.bg, color: colors.fg }}
-      title={engine.path ?? engine.message ?? label}
+      title={
+        engineUpdate?.hasUpdate
+          ? t('engine.updateAvailable', { version: engineUpdate.latest ?? '' })
+          : (engine.path ?? engine.message ?? label)
+      }
     >
       {engine.state === 'downloading' || engine.state === 'checking' ? (
         <Spinner size={11} />

src/renderer/src/components/SettingsControls.tsx

@@ -192,7 +192,12 @@ export function Choice<T extends string | number>({
   onChange: (value: T) => void
 }): ReactNode {
   return (
-    <div className="flex items-center gap-1 rounded-lg p-0.5" style={{ background: 'var(--panel-2)' }}>
+    // `self-start` keeps the pill bar hug its labels when the Field stacks
+    // children full-width (otherwise the track stretches across the card).
+    <div
+      className="flex w-fit items-center gap-1 rounded-lg p-0.5"
+      style={{ background: 'var(--panel-2)', alignSelf: 'flex-start' }}
+    >
       {options.map((option) => {
         const active = option.value === value
         return (