sdaduanbilei/cc-haha
1
0
Fork 0
mirror of https://github.com/NanmiCoder/cc-haha synced 2026-07-16 13:03:31 +08:00
Code Issues Packages Projects Releases Wiki Activity

chore(desktop): leave notification permission code unchanged

Browse Source 
Drop the explanatory comment added earlier - notifications work as-is, so the
notification service stays byte-for-byte identical to main.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
程序员阿江(Relakkes) 2026-06-03 14:13:04 +08:00
parent 7e1e1b4ae6
commit 9322d9aba3
1 changed files with 0 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
desktop/electron/services/notifications.ts
View File

@ -23,15 +23,6 @@ export function validateNotificationOptions(value: unknown): value is DesktopNot
&& (record.extra === undefined || (typeof record.extra === 'object' && record.extra !== null && !Array.isArray(record.extra)))
}
// NOTE: Electron has no API to read the macOS user authorization status from the
// main process (`systemPreferences.getNotificationSettings()` does not exist; the
// only option is the native `macos-notification-state` module, which we avoid to
// keep the cross-platform build dependency-free). `Notification.isSupported()`
// reflects OS capability, NOT whether the user granted permission, so this can
// report 'granted' even when the user denied notifications in System Settings.
// The authoritative delivery signal is the 'failed' lifecycle event handled in
// sendDesktopNotification(); on macOS, events only emit correctly for code-signed
// builds (unsigned/ad-hoc builds emit 'failed').
export function notificationPermissionState(
NotificationClass: ElectronNotificationConstructor,
): NotificationPermissionState {