jihoson/The-Ouroboros
1
0
Fork 0
Code Issues Pull Requests Actions 10 Packages Projects Releases Wiki Activity

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
Conversation 0 Commits 1 Files Changed 2 +215
agentson commented 2026-02-08 21:26:32 +09:00
Collaborator
Copy Link

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
Details
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.
Reviewers
No Reviewers
Labels
Clear labels
bug
Something is not working
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 wontfix
This won't be fixed
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
agentson jihoson
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#108
Delete Branch "feature/issue-81-telegram-playbook-notify"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?