fix(telegram): honor message.quote for partial-quote reply context

[briandevans]

Summary

When a Telegram user replies using Telegram's native partial-quote feature (selecting a substring of a prior message), _build_message_event was injecting the entire replied-to message into reply_to_text. Now it prefers message.quote.text (the user-selected substring) and falls back to the full replied-to message only when no native quote is present.

The bug

gateway/platforms/telegram.py::_build_message_event extracted reply context as:

if message.reply_to_message:
    reply_to_id = str(message.reply_to_message.message_id)
    reply_to_text = message.reply_to_message.text or message.reply_to_message.caption or None

message.reply_to_message.text is always the full prior message body. If Hermes sent a multi-section message and the user replied to one quoted bullet with "mark this one as done", the agent saw:

[Replying to: "<entire previous Hermes message — every bullet>"]

mark this one as done

python-telegram-bot >=21 exposes Message.quote: Optional[TextQuote] which carries TextQuote.text for the user-selected substring (PTB docs: "For replies that quote part of the original message, the quoted part of the message."). The repo pins python-telegram-bot[webhooks]>=22.6,<23 (pyproject.toml:58) so quote is reliably available — the production code just wasn't reading it.

The fix

Read getattr(message, "quote", None) and prefer quote.text when truthy. Fall back to reply_to_message.text or reply_to_message.caption for replies without a native quote (older PTB, or replies that don't select a substring) and for replied-to media messages where only the caption is set.

getattr + is not None checks keep the path safe if quote is ever absent or empty.

Test plan

• Focused regression: tests/gateway/test_telegram_reply_quote.py — 4 tests covering the partial-quote path, no-quote fallback, caption fallback, and an empty-string quote.text defensive case
• Adjacent suite: tests/gateway/test_dm_topics.py, test_telegram_reply_mode.py, test_reply_to_injection.py (70 tests passed)
• Regression guard: with the fix reverted, test_native_partial_quote_used_as_reply_to_text fails with AssertionError: 'Briefing:\\n- Item A:...' == 'Item B: rotate keys' — proves the test exercises the new code path

Run focused tests:

uv run --with pytest --with pytest-xdist --with pytest-asyncio python3 -m pytest \
  tests/gateway/test_telegram_reply_quote.py -v

Related

• Fixes [Bug]: Telegram native partial quotes are expanded to the full replied-to message #22619
• Builds on the reply-context plumbing introduced in feat(gateway): inject reply-to message context when quoted message is not in session history #1594; only the inbound extraction site is changed, the MessageEvent schema and downstream _prepare_inbound_message_text injection are unchanged.

Sibling code paths that may need the same fix: none in this adapter — _build_message_event is the single inbound extraction site for Telegram updates. Other adapters (Discord, Slack, Matrix, etc.) have their own reply-context paths and would need a per-platform check before extending.

[Copilot]

Pull request overview

This PR fixes Telegram reply-context extraction so that when a user replies with Telegram’s native partial-quote feature, the gateway prefers the user-selected substring (message.quote.text) for reply_to_text instead of injecting the entire replied-to message body.

Changes:

• Update TelegramAdapter._build_message_event to prefer message.quote.text when present/truthy, with a fallback to reply_to_message.text or reply_to_message.caption.
• Add a focused regression test suite covering partial-quote, no-quote fallback, caption fallback, and empty-quote defensive behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
gateway/platforms/telegram.py	Prefer Telegram native partial quote text for reply_to_text, falling back to full replied-to message text/caption.
tests/gateway/test_telegram_reply_quote.py	Adds regression coverage for partial-quote handling and fallbacks.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[teknium1]

Merged via salvage PR #22676. Your commit was cherry-picked onto current main with your authorship preserved in git log (rebase-merge). Thanks for the contribution!