Skip to content

PR-011: Notifications - #10

Draft
raychrisgdp wants to merge 9 commits into
mainfrom
feature/notifications
Draft

PR-011: Notifications#10
raychrisgdp wants to merge 9 commits into
mainfrom
feature/notifications

Conversation

@raychrisgdp

Copy link
Copy Markdown
Owner

Summary

Implement task reminder system with 24h/6h/overdue alerts, quiet hours, deduplication, and persisted history. Support desktop notifications (local) and in-app feed (Docker). Optional agent-run notifications when agent system exists.

Changes

  • Add Notification model with types (reminder_24h, reminder_6h, overdue, agent_started, agent_completed, agent_failed)
  • Implement NotificationService with scheduler, quiet hours, and deduplication
  • Add desktop notification channel via plyer (with graceful fallback)
  • Add in-app notification channel for Docker environments
  • Create notification API endpoints (GET /api/v1/notifications, PATCH read)
  • Add configuration settings for notification scheduling and quiet hours
  • Background scheduler runs every 60s on app startup

Test Results

  • Total Tests: 216
  • Passed: 216 ✅
  • Failed: 0 ❌
  • Coverage: Full

Acceptance Criteria

  • ✅ AC1: Scheduling and Overdue
  • ✅ AC2: Deduplication and History
  • ✅ AC3: Delivery Channels
  • ✅ AC4: Quiet Hours

See test results for details.

Related

  • Spec: PR-011-notifications.md
  • Depends on: PR-002 (Task CRUD API)
  • Optional: PR-003B (Agent tool calling), PR-014 (Multi-agent orchestration)

- Add Notification model with types (reminder_24h, reminder_6h, overdue, agent_started, agent_completed, agent_failed)
- Implement NotificationService with scheduler, quiet hours, and deduplication
- Add desktop notification channel via plyer (with graceful fallback)
- Add in-app notification channel for Docker environments
- Create notification API endpoints (GET /api/v1/notifications, PATCH read)
- Add configuration settings for notification scheduling and quiet hours
- Background scheduler runs every 60s on app startup
- 216 tests passing (100%)
- All acceptance criteria met

See PR-011-TEST-RESULTS.md for detailed test results
@raychrisgdp
raychrisgdp marked this pull request as draft January 4, 2026 05:36
@raychrisgdp raychrisgdp self-assigned this Jan 4, 2026
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