Skip to content

feat: add push notifications with VAPID#110

Merged
chriswritescode-dev merged 14 commits intomainfrom
dev
Feb 6, 2026
Merged

feat: add push notifications with VAPID#110
chriswritescode-dev merged 14 commits intomainfrom
dev

Conversation

@chriswritescode-dev
Copy link
Owner

Summary

  • Push Notifications - Background alerts for agent events when PWA is closed using VAPID
  • Icon Fixes - Replace SquareFill with proper Lucide icons for recording controls

Changes

Push Notifications

  • Add NotificationService with VAPID configuration and subscription management
  • Implement client visibility tracking to prevent duplicate notifications when app is open
  • Add notification routes for subscribe/test/manage subscriptions
  • Support permissionAsked, questionAsked, sessionError, and sessionIdle events
  • Configure browser service worker for push notification handling
  • Add PWA icons (192x192, 512x512) and maskable variants
  • Add comprehensive documentation in README and docs/features/notifications.md

Icon Improvements

  • Replace SquareFill with MicOff for stop recording buttons
  • Replace SquareFill with X for stop generation buttons
  • Better visual consistency with Lucide icon set

- Add structured logging throughout push notification flow
- Improve error handling with detailed diagnostics
- Add /api/notifications/debug endpoint for troubleshooting
- Enhanced test notification response with per-subscription results
- Add service worker logging and better install/activate handling
…ter defaults

- Add validation warnings for VAPID_SUBJECT format (requires mailto: prefix for iOS/Safari)
- Expose VAPID details via notifications API endpoint
- Update default VAPID_SUBJECT from mailto:admin@opencode.local to empty string
- Enhance VAPID logging with key lengths and subject info
- Improve documentation in .env.example with setup instructions
@chriswritescode-dev chriswritescode-dev merged commit ae644e8 into main Feb 6, 2026
2 checks passed
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