Skip to content

fix(gateway): escape Telegram chunk indicators for MarkdownV2 - #1477

Closed
starlash7 wants to merge 1 commit into
NousResearch:mainfrom
starlash7:fix/1456-telegram-mdv2-chunk-indicator
Closed

fix(gateway): escape Telegram chunk indicators for MarkdownV2#1477
starlash7 wants to merge 1 commit into
NousResearch:mainfrom
starlash7:fix/1456-telegram-mdv2-chunk-indicator

Conversation

@starlash7

Copy link
Copy Markdown

Summary

  • escape the trailing chunk indicator added to multi-part Telegram messages before sending with MarkdownV2
  • keep the shared chunking behavior unchanged for other adapters
  • add a regression test covering chunked formatted Telegram responses

Testing

  • ./.venv/bin/python -m pytest tests/gateway/test_telegram_format.py tests/gateway/test_platform_base.py
  • ./.venv/bin/python -m pytest tests/gateway/test_telegram_documents.py -k send_document_success -q

Closes #1456

@teknium1

Copy link
Copy Markdown
Contributor

Thanks for the contribution! This fix was already merged independently via PR #1478 — the chunk indicator escaping is now inlined in send() at line 325-331 of telegram.py, with test coverage at test_send_escapes_chunk_indicator_for_markdownv2. Closing as redundant.

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.

[Bug]: Telegram formatting falls back to plain text on chunked messages

2 participants