feat(feishu): wire clarify tool with interactive card buttons#29719
Open
bladeJumper wants to merge 1 commit into
Open
feat(feishu): wire clarify tool with interactive card buttons#29719bladeJumper wants to merge 1 commit into
bladeJumper wants to merge 1 commit into
Conversation
Implements the clarify tool on Feishu with interactive card buttons, mirroring the Telegram pattern from NousResearch#24199. - send_clarify: renders question + choice buttons as Feishu interactive card - _handle_clarify_card_action: routes button callbacks via hermes_clarify_action - choice: resolves immediately with choice text - other: flips to text-capture mode via mark_awaiting_text - Already-resolved guard prevents double-click / stale-button issues - Authorization check reuses _is_interactive_operator_authorized - Choice text lookup: stored state → clarify entry → fallback index Known limitation: Feishu shows both the tool-progress bubble (❓ clarify: ...) and the interactive card because Feishu lacks delete_message support. This can be revisited once Feishu implements message deletion. Closes NousResearch#12573 Closes NousResearch#21032 Supersedes NousResearch#23740 (which included framework code now merged in NousResearch#24199) Related NousResearch#24199, NousResearch#21893, NousResearch#503
Collaborator
This was referenced Jun 10, 2026
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
2 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.
Implements the clarify tool on Feishu with interactive card buttons, mirroring the Telegram pattern from #24199.
Changes
send_clarify: renders question + choice buttons as Feishu interactive card_handle_clarify_card_action: route button callbacks viahermes_clarify_actionmark_awaiting_text_is_interactive_operator_authorizedKnown limitation
Feishu shows both the tool-progress bubble (
❓ clarify: ...) and the interactive card because Feishu lacksdelete_messagesupport. This can be revisited once Feishu implements message deletion.Testing
tests/gateway/test_feishu_clarify_buttons.pyCloses #12573
Closes #21032
Supersedes #23740
Related #24199, #21893, #503