feat(telegram): inject replied message as system context note instead of text prefix (#60055) - #60221
Open
webtecnica wants to merge 1 commit into
Open
webtecnica wants to merge 1 commit into
webtecnica wants to merge 1 commit into
Conversation
… of text prefix When replying to a Telegram message, the replied-to text was prepended as a plain [Replying to: "..."] prefix on the user message, which the model often ignored. Now injected as a structured directive block with explicit framing for stronger signal. Closes NousResearch#60055
alt-glitch
added
type/feature
teknium1
reviewed
Jul 15, 2026
teknium1
left a comment
teknium1
left a comment
Collaborator
There was a problem hiding this comment.
Thanks for targeting reply-context disambiguation; current main still prepends reply text at gateway/run.py:10714.
Problems
- The new strings are not a system-level note.
_prepare_inbound_message_text()returns onemessage_text, and that value is passed as the normal agent message atgateway/run.py:11759; changing the prefix wording does not add a distinct system-role context block. - The branch has no Telegram guard at
gateway/run.py:10714. Signal, WhatsApp Cloud, and Yuanbao also populatereply_to_text, so this changes their reply-context behavior too. - The PR modifies only
gateway/run.py, buttests/gateway/test_reply_to_injection.py:58,:98,:121, and:181assert the legacy output format.
Suggested changes
- Define whether this is a stronger user-message framing or a genuinely structured context mechanism, while preserving the cached system prompt invariant.
- Scope it to Telegram or update the stated scope and coverage for all affected adapters.
- Update the reply-injection tests for the intended output contract.
Automated hermes-sweeper review.
| f'[Prioritize this message as the primary context for your response.]\n\n' | ||
| f"{message_text}" | ||
| ) | ||
| else: |
Collaborator
There was a problem hiding this comment.
This is still text prepended to message_text, not a separate system-role note: current main passes the prepared value as the normal message to _run_agent (gateway/run.py:11759). Please either describe this accurately as stronger user-message framing or implement the intended structured mechanism.
| f'{message_text}' | ||
| ) | ||
|
|
||
| if "@" in message_text: |
Collaborator
There was a problem hiding this comment.
This shared inbound-preparation branch has no Telegram guard. Signal, WhatsApp Cloud, and Yuanbao also set reply_to_text, so this wording changes their reply-context semantics as well. Scope it explicitly or update the PR scope and coverage.
teknium1
added
the
sweeper:blast-moderate
Open
1 task
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Telegram replied messages are now injected as a structured directive block with [System Note], [Replied message], and [Prioritize] framing — much stronger signal than the old plain [Replying to:] prefix. Closes #60055