feat: add generic send_message method to TelegramClient (issue #59) #60

agentson · 2026-02-05T13:39:24+09:00

agentson commented

2026-02-05 13:39:24 +09:00

Summary

Add send_message(text, parse_mode) method to TelegramClient for generic message sending
Refactor _send_notification() to use the new method
Add comprehensive tests with 93% coverage

Changes

src/notifications/telegram_client.py: Add send_message() method
tests/test_telegram.py: Add 4 new tests for send_message()

Test plan

All tests pass (19/19)
Coverage ≥ 80% (93% for telegram_client.py)
Supports both HTML and Markdown parse modes
Returns bool for success/failure tracking
Gracefully handles disabled client and API errors

Closes #59

🤖 Generated with Claude Code

## Summary - Add `send_message(text, parse_mode)` method to TelegramClient for generic message sending - Refactor `_send_notification()` to use the new method - Add comprehensive tests with 93% coverage ## Changes - `src/notifications/telegram_client.py`: Add send_message() method - `tests/test_telegram.py`: Add 4 new tests for send_message() ## Test plan - [x] All tests pass (19/19) - [x] Coverage ≥ 80% (93% for telegram_client.py) - [x] Supports both HTML and Markdown parse modes - [x] Returns bool for success/failure tracking - [x] Gracefully handles disabled client and API errors ## Related Issues Closes #59 🤖 Generated with [Claude Code](https://claude.com/claude-code)

agentson added 1 commit 2026-02-05 13:39:24 +09:00

feat: add generic send_message method to TelegramClient (issue #59 )

CI / test (pull_request) Has been cancelled

Details

259f9d2e24

Add send_message(text, parse_mode) method that can be used for both
notifications and command responses. Refactor _send_notification to
use the new method.

Changes:
- Add send_message() method with return value for success/failure
- Refactor _send_notification() to call send_message()
- Add comprehensive tests for send_message()
- Coverage: 93% for telegram_client.py

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

jihoson merged commit c76b9d5c15 into main

2026-02-05 13:40:06 +09:00

jihoson referenced this issue from a commit

2026-02-05 13:40:09 +09:00

Merge pull request 'feat: add generic send_message method to TelegramClient (issue #59)' (#60) from feature/issue-59-send-message into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#60