Windows desktop app icon format fixes (.ico)

Fixes on record

Merged to main. A stamp counts the issues a fix is linked to.

• #50808 fix(desktop): set AppUserModelID on Windows so notifications fire

  merged to main

  files: apps/desktop/electron/main.cjs

  Show the diff

  • apps/desktop/electron/main.cjs

    @@ -620,6 +620,16 @@ function previewFileMetadata(filePath, mimeType) {
     }
     
     app.setName(APP_NAME)
    +// Windows toast notifications silently no-op unless an AppUserModelID is set:
    +// `new Notification().show()` returns without error and nothing appears. The
    +// AUMID must match the installed Start Menu shortcut's AUMID, which
    +// electron-builder derives from the build `appId` (com.nousresearch.hermes) —
    +// keep this string in sync with package.json `build.appId`. macOS/Linux don't
    +// need this, so gate it on Windows. (Fixes: desktop approval/turn notifications
    +// never firing on Windows.)
    +if (IS_WINDOWS) {
    +  app.setAppUserModelId('com.nousresearch.hermes')
    +}
     // Seed the native About panel with the live Hermes version. This is refreshed
     // on every open via the explicit "About" menu handler (refreshAboutPanel), so
     // an in-place `hermes update` mid-session is reflected without an app restart;

• #75937 fix(desktop): make the Windows app icon match native icon size

  merged to main

  files: apps/desktop/assets/icon.ico, apps/desktop/electron/main.ts