Repository navigation
Conversation
WeChat adapter's secondary content-fingerprint dedup was dropping intentionally-repeated slash commands (e.g. /approve sent twice). The dedup layer now skips text starting with '/', allowing commands to be processed regardless of prior dedup state. Fixes #29779
alt-glitch
added
type/bug
JerryLiu369
added a commit
to JerryLiu369/hermes-agent
that referenced
this pull request
Sep 12, 2026
Simpler approach: if text starts with "/", skip the content-fingerprint dedup entirely. Message-ID dedup (300s TTL) still handles iLink network retries, so the 3-second sliding window for commands is unnecessary. This is the approach that NousResearch#29787, NousResearch#29855, NousResearch#35434, and NousResearch#36707 all targeted — the earlier PRs were closed for bundled unrelated changes, not for the fix strategy itself. Tests: - test_slash_commands_bypass_content_dedup: two /approve with different message IDs → both reach the handler - test_slash_command_msg_id_dedup_still_applies: same message_id → deduped by message_id cache (not affected by this change) - Existing test_duplicate_content_with_different_message_ids_is_dropped still passes (regular text dedup unchanged)
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
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.
Summary
WeChat adapter's secondary content-fingerprint dedup was dropping intentionally-repeated slash commands (e.g. /approve sent twice).
Fix
Skip content dedup for text starting with
/— commands are always processed regardless of dedup state.Testing
/approvetwice via WeChat — second one should now reach the agentCloses #29779