feat: add Telegram playbook notifications (issue #81) #108

Merged
jihoson merged 1 commits from feature/issue-81-telegram-playbook-notify into main 2026-02-08 21:27:46 +09:00
Collaborator

Summary

  • notify_playbook_generated(): 장 전 플레이북 생성 알림 (MEDIUM) — market, stock/scenario count, token usage
  • notify_scenario_matched(): 장중 시나리오 매칭 알림 (HIGH) — stock, action, condition, confidence
  • notify_playbook_failed(): 플레이북 생성 실패 알림 (HIGH) — market, reason (200자 truncation)
  • 기존 _send_notification + NotificationMessage 패턴 일관 준수
  • 6 new tests (25 total): 3 format + 3 priority 검증

Test plan

  • test_playbook_generated_format: 필드 포맷 확인
  • test_scenario_matched_format: 필드 포맷 확인
  • test_playbook_failed_format: 필드 포맷 확인
  • test_playbook_generated_priority: MEDIUM emoji 확인
  • test_playbook_failed_priority: HIGH emoji 확인
  • test_scenario_matched_priority: HIGH emoji 확인
  • 전체 25 tests passed

Closes #81

## Summary - `notify_playbook_generated()`: 장 전 플레이북 생성 알림 (MEDIUM) — market, stock/scenario count, token usage - `notify_scenario_matched()`: 장중 시나리오 매칭 알림 (HIGH) — stock, action, condition, confidence - `notify_playbook_failed()`: 플레이북 생성 실패 알림 (HIGH) — market, reason (200자 truncation) - 기존 `_send_notification` + `NotificationMessage` 패턴 일관 준수 - 6 new tests (25 total): 3 format + 3 priority 검증 ## Test plan - [x] test_playbook_generated_format: 필드 포맷 확인 - [x] test_scenario_matched_format: 필드 포맷 확인 - [x] test_playbook_failed_format: 필드 포맷 확인 - [x] test_playbook_generated_priority: MEDIUM emoji 확인 - [x] test_playbook_failed_priority: HIGH emoji 확인 - [x] test_scenario_matched_priority: HIGH emoji 확인 - [x] 전체 25 tests passed Closes #81
agentson added 1 commit 2026-02-08 21:26:32 +09:00
feat: add Telegram playbook notifications (issue #81)
Some checks failed
CI / test (pull_request) Has been cancelled
e8634b93c3
- notify_playbook_generated(): market, stock/scenario count, token usage (MEDIUM)
- notify_scenario_matched(): stock, action, condition, confidence (HIGH)
- notify_playbook_failed(): market, reason with 200-char truncation (HIGH)
- 6 new tests: 3 format + 3 priority validations

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
jihoson merged commit 815d675529 into main 2026-02-08 21:27:46 +09:00
Sign in to join this conversation.
No Reviewers
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#108