admin-valentin/agent-ecosystem
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat: enable notification action buttons on Windows (not just macOS)

Browse source 
Electron's 'action' event fires on both macOS and Windows per docs.
Removed isMac guard — now Allow/Deny buttons appear on Windows too.
Linux excluded (libnotify doesn't fire the 'action' event).
This commit is contained in:
iliya 2026-03-21 13:44:50 +02:00
parent 95b282577f
commit 020aaab1c6
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
src/main/services/team/TeamProvisioningService.ts
View file

@ -5380,16 +5380,21 @@ export class TeamProvisioningService {
if (!ElectronNotification.isSupported()) return;
const isMac = process.platform === 'darwin';
const isLinux = process.platform === 'linux';
const iconPath = isMac ? undefined : getAppIconPath();
const teamLabel = run.request.displayName ?? run.teamName;
const body = this.formatToolApprovalBody(approval.toolName, approval.toolInput);
// Actions (Allow/Deny buttons) supported on macOS and Windows.
// Linux libnotify doesn't fire the 'action' event — users get click-to-focus.
const supportsActions = !isLinux;
const notification = new ElectronNotification({
title: `Tool Approval — ${teamLabel}`,
body,
sound: config.notifications.soundEnabled ? 'default' : undefined,
...(iconPath ? { icon: iconPath } : {}),
...(isMac
...(supportsActions
? {
actions: [
{ type: 'button' as const, text: 'Allow' },
@ -5417,8 +5422,9 @@ export class TeamProvisioningService {
notification.on('close', cleanup);
// macOS action buttons: Allow (index 0) / Deny (index 1)
if (isMac) {
// Action buttons: Allow (index 0) / Deny (index 1)
// 'action' event fires on macOS and Windows (not Linux)
if (supportsActions) {
notification.on('action', (_event, index) => {
cleanup();
const allow = index === 0;