Skip to content

feat(messagingx): channel-agnostic chat provider + Telegram implement… - #18

Merged
ihsanarifr merged 1 commit into
mainfrom
feature/messagingx-telegram
Jul 6, 2026
Merged

feat(messagingx): channel-agnostic chat provider + Telegram implement…#18
ihsanarifr merged 1 commit into
mainfrom
feature/messagingx-telegram

Conversation

@ihsanarifr

Copy link
Copy Markdown
Member

…ation

Adds a Provider interface (Channel/Poll/Send) so an app can receive group messages and send replies without binding to a channel; WhatsApp can implement the same interface later. Telegram provider uses the Bot API (getUpdates long-poll + sendMessage), configurable base URL/HTTP client for testing.

Tests (httptest mock Telegram API): Poll parses updates and computes the next offset (ignoring non-message updates); Send posts the correct chat_id/text; ok:false responses surface as errors. Needed by Cuanku's msg-worker (US3).

Summary

Checklist

  • Tests added or updated (run make test → green)
  • Coverage thresholds met (make coverage — core ≥ 85%, security ≥ 95%)
  • Godoc comments on all exported identifiers
  • CHANGELOG.md updated under [Unreleased]
  • No breaking changes OR marked with ! in commit / BREAKING CHANGE: footer
  • go mod tidy run — no stale entries in go.sum
  • make lint passes with zero new warnings
  • Benchmarks added for new hot-path functions (if applicable)

Related Issues

…ation

Adds a Provider interface (Channel/Poll/Send) so an app can receive group
messages and send replies without binding to a channel; WhatsApp can implement
the same interface later. Telegram provider uses the Bot API (getUpdates
long-poll + sendMessage), configurable base URL/HTTP client for testing.

Tests (httptest mock Telegram API): Poll parses updates and computes the next
offset (ignoring non-message updates); Send posts the correct chat_id/text;
ok:false responses surface as errors. Needed by Cuanku's msg-worker (US3).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@ihsanarifr
ihsanarifr merged commit 7778299 into main Jul 6, 2026
6 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