Skip to content

fix(telegram): format media captions#76

Merged
bogdanovich merged 2 commits into
mainfrom
codex/telegram-media-caption-format
Jun 17, 2026
Merged

fix(telegram): format media captions#76
bogdanovich merged 2 commits into
mainfrom
codex/telegram-media-caption-format

Conversation

@bogdanovich

Copy link
Copy Markdown
Owner

Summary

  • format Telegram media captions with the same markdown conversion path as normal text messages
  • set parse mode for photo/video/audio/voice/document captions and the first caption in media groups
  • retry once with plain caption text if Telegram rejects parsed entities

Testing

  • go test ./pkg/channels/telegram -count=1
  • golangci-lint run ./pkg/channels/telegram/...

@bogdanovich bogdanovich left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated review by Reviewer agent.

Comment thread pkg/channels/telegram/telegram.go Outdated
@bogdanovich

bogdanovich commented Jun 12, 2026

Copy link
Copy Markdown
Owner Author

Automated review by Reviewer agent.

Review Summary

No high-confidence issues found in the Telegram media caption formatting change.

The previously reported findings are resolved in the current PR head.

@bogdanovich

Copy link
Copy Markdown
Owner Author

Addressed the Telegram album-caption parse fallback issue.

Changes:

  • added the same plain-text retry path for sendSingleImageMediaGroup() when Telegram rejects formatted captions
  • added a regression test covering a multi-image album where the first formatted caption fails and the resend succeeds without parse_mode

Validation:

  • go test -tags goolm,stdjson ./pkg/channels/telegram -count=1
  • golangci-lint run ./pkg/channels/telegram --build-tags=goolm,stdjson --allow-parallel-runners

@bogdanovich bogdanovich merged commit 03a2dc6 into main Jun 17, 2026
4 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