fix(telegram): omit General topic id for model picker#19961
Conversation
|
Likely duplicate of #12873 — same fix: route send_model_picker() thread_id through _message_thread_id_for_send() to omit General topic id=1. |
1 similar comment
|
Likely duplicate of #12873 — same fix: route send_model_picker() thread_id through _message_thread_id_for_send() to omit General topic id=1. |
|
Thanks for catching this — the General-topic mapping ( The parallel code path in the Closing as superseded — thanks for the contribution and for keeping us focused on this bug class! |
Summary
send_model_picker()/modelinline keyboard sends from passingmessage_thread_id=1for the General forum topicTest Plan
python -m pytest tests/gateway/test_telegram_approval_buttons.py -q -o 'addopts='\n\n## Context\nTelegram represents the General forum topic internally as thread id1, but the Bot API rejects outboundmessage_thread_id=1withMessage thread not found. Normal Telegram text sends already route through_message_thread_id_for_send()to omit this value; the model picker path was bypassing that helper and falling back to the long text model list when the inline keyboard send failed.