Skip to content

Notification System#41

Open
zubeirom wants to merge 8 commits intomasterfrom
user/zmohamed/notificationsystem
Open

Notification System#41
zubeirom wants to merge 8 commits intomasterfrom
user/zmohamed/notificationsystem

Conversation

@zubeirom
Copy link
Copy Markdown
Collaborator

notification.system.mp4

Zubeir Mohamed and others added 8 commits March 29, 2026 20:56
…ication

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…ing for input

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
- Add Notification.isSupported() guard for robustness
- Use BrowserWindow.fromWebContents() instead of getAllWindows()[0]
- Add 15s per-session notification cooldown to prevent spam
- Truncate display name to 100 chars in notification body
- Guard selectSession against stale/killed sessions

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Set AppUserModelId so Windows uses the correct app icon
- Pass logo.png as notification icon via nativeImage

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- Add bell icon with badge count to ActivityBar above theme toggle
- Notification dropdown lists sessions waiting for input with CLI icon and timestamp
- Clicking a notification selects that session
- Remove SELECT_SESSION IPC (toast click does nothing special)
- Revert setAppUserModelId (pre-existing icon issue, not our fix)
- Add waitingSince tracking to store for timestamp display
- Bigger toolbar buttons (h-8, 16px icons, 13px text)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- In-app toast slides in top-right when a non-selected session needs input and window is focused
- Native Windows toast fires when window is unfocused/minimized
- Toast shows CLI icon, session name, click to select session
- Auto-dismisses after 8 seconds, or click X to dismiss

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant