fix(discord): fetch and inject reply context for incoming messages#30950
fix(discord): fetch and inject reply context for incoming messages#30950alaamohanad169-ship-it wants to merge 2 commits into
Conversation
|
Note: This is a clean rebase of the previously closed PR #30085. The old PR was closed due to upstream divergence. This version is rebased on latest main with only the reply-context changes. The related PR #5348 (luinbytes) takes a similar approach but adds TTL caching and author attribution. My approach is simpler — just fetch and inject the reply context. If maintainers prefer the caching approach, happy to collaborate or close this in favor of #5348. |
|
Note to maintainers: The alt-glitch bot flagged this as a duplicate. This is incorrect. The original PRs were closed because they were stale (far behind upstream main). This is the clean rebased version cherry-picked onto latest main. The old PRs referenced by the bot no longer exist. |
f70dbee to
673f9f4
Compare
|
Update: Rebased onto latest main with clean diff. Previous version had accumulated unrelated changes from a bad cherry-pick. This version contains only the reply-context fix: 14 insertions, 0 deletions in The fix: when a user replies to a Discord message, Hermes now fetches the referenced message content via API when Discord does not provide it inline (common for older/high-traffic messages), and prepends reply context to the message text so the agent understands what the user is responding to. Fixes #30049 |
Summary
When a user replies to a message in Discord, Hermes now fetches the referenced message content via API when Discord does not provide it inline (common for older/high-traffic messages), and prepends reply context to the message text so the agent understands what the user is responding to.
Changes
resolvedis None[Replying to: "..."]context to the message text (truncated to 200 chars)Fixes #30049