PR #20526: feat(feishu): Add interactive card model picker (send_model_picker)

• Repository: NousResearch/hermes-agent
• Author: zycaskevin
• State: closed | merged: False
• Link: https://github.com/NousResearch/hermes-agent/pull/20526

Description (problem / solution / changelog)

Summary

Implements send_model_picker for the Feishu/Lark platform adapter, enabling an interactive card-based model selector — matching the UX that Telegram and Discord already have.

Closes #20520

What Changed

gateway/platforms/feishu.py

• Added _model_picker_state dict to track per-chat picker state
• Added send_model_picker() — sends an interactive provider selection card, matching the contract in gateway/run.py
• Added _build_model_picker_provider_card() — builds the top-level provider selection card (4 buttons per action group, Feishu limit)
• Added _build_model_picker_model_card() — builds the paginated model selection card with navigation
• Added _handle_model_picker_card_action() — synchronous dispatch handler for all picker button clicks:
  • mp:<slug> → switch to model list for selected provider
  • mg:<page> → paginate model list
  • mm:<idx> → show "processing..." card + schedule async switch
  • mb → back to provider list
  • mx → cancel
• Added _handle_model_picker_selection() — async callback that performs the actual model switch via on_model_selected
• Added helper methods: _cleanup_picker_state, _update_card, _picker_expired_response, _picker_error_response
• Modified _on_card_action_trigger to intercept model picker actions before approval handling

tests/gateway/test_feishu_model_picker.py (new)

• 12 unit tests covering:
  • Provider card structure and current-provider marking
  • Model card structure, pagination, back/cancel buttons
  • Synchronous dispatch: provider→model, page nav, back, cancel, expired state
  • Model selection schedules async work and returns processing card

All tests pass ✅

Design Decisions

• Navigation is synchronous — P2CardActionTriggerResponse replaces the card immediately on provider/page/back actions (fast, no loading state needed)
• Model switch is async — the actual on_model_selected callback may involve API calls; we show "⏳ processing..." card first, then update via edit_message when done
• Reuses existing infrastructure — _submit_on_loop, _feishu_send_with_retry, _finalize_send_result, P2CardActionTriggerResponse/CallBackCard patterns
• Feishu card constraints honored — 4 buttons per action group, wide screen mode enabled

Testing

pytest tests/gateway/test_feishu_model_picker.py -v
# 12 passed ✅

Before/After

Before: /model on Feishu → plain text list:

Current: deepseek-v4-pro on ollama-cloud

OpenRouter:
  openai/gpt-5.4, anthropic/claude-sonnet-4, ...

After: /model on Feishu → interactive card with buttons for provider selection, then paginated model list with navigation.

Changed files

• gateway/platforms/feishu.py (modified, +395/-0)
• tests/gateway/test_feishu_model_picker.py (added, +291/-0)