fix(tradein): startup topic-check реально валидирует message_thread_id #3513

Merged

lekss361 merged 1 commit from fix/3471-topic-check-really-validates into main

2026-09-13 10:41:17 +00:00

bot-backend commented

2026-09-13 10:32:14 +00:00

Collaborator

Summary

sendChatAction не валидирует message_thread_id — живой прод-замер (#3471) показал, что она подтверждает даже заведомо несуществующую тему ({"ok":true,"result":true}). verify_chat_and_topic была ложно-зелёной ровно в сценарии, ради которого её писали: удалённая/переименованная тема проходила проверку, а реальная отправка потом падала на каждом сообщении.
Заменил проверку темы на sendMessage+deleteMessage: sendMessage с несуществующим message_thread_id мгновенно отвечает 400 Bad Request: message thread not found (тоже проверено на проде), а с валидным создаёт сообщение, которое сразу стирается deleteMessage — в истории треда не остаётся ничего.
Поправлен неверный докстринг send_chat_action, утверждавший обратное.
Поведение вокруг сохранено: тот же путь через клиент (ретранслятор/ретраи), тот же формат лог-строк, та же семантика возврата (bool, не роняет процесс).

Test plan

Живой прод-замер (ssh selectel -> docker exec tradein-tgbot): sendMessage с мусорным message_thread_id=987654 -> 400 message thread not found; с валидным message_thread_id=2 (support) -> успех, зонд-сообщение сразу удалено deleteMessage.
Новый тест test_verify_chat_and_topic_is_red_when_topic_deleted_even_though_send_chat_action_lies явно проверен на падение против до-фиксной реализации (git checkout HEAD -- client.py + прогон теста -> assert True is False), затем зелёный после возврата фикса.
pytest tests/services/tgbot/test_topic_check_and_group_rate_limit.py tests/test_alerts_become_events.py -- 34 passed.
ruff check на изменённые файлы -- чисто.

Closes #3471

🤖 Generated with Claude Code

https://claude.ai/code/session_01JY6iWDnGDthdvsMWgK1BMG

## Summary - `sendChatAction` не валидирует `message_thread_id` — живой прод-замер (#3471) показал, что она подтверждает даже заведомо несуществующую тему (`{"ok":true,"result":true}`). `verify_chat_and_topic` была ложно-зелёной ровно в сценарии, ради которого её писали: удалённая/переименованная тема проходила проверку, а реальная отправка потом падала на каждом сообщении. - Заменил проверку темы на `sendMessage`+`deleteMessage`: `sendMessage` с несуществующим `message_thread_id` мгновенно отвечает `400 Bad Request: message thread not found` (тоже проверено на проде), а с валидным создаёт сообщение, которое сразу стирается `deleteMessage` — в истории треда не остаётся ничего. - Поправлен неверный докстринг `send_chat_action`, утверждавший обратное. - Поведение вокруг сохранено: тот же путь через клиент (ретранслятор/ретраи), тот же формат лог-строк, та же семантика возврата (`bool`, не роняет процесс). ## Test plan - [x] Живой прод-замер (ssh selectel -> docker exec tradein-tgbot): `sendMessage` с мусорным `message_thread_id=987654` -> `400 message thread not found`; с валидным `message_thread_id=2` (support) -> успех, зонд-сообщение сразу удалено `deleteMessage`. - [x] Новый тест `test_verify_chat_and_topic_is_red_when_topic_deleted_even_though_send_chat_action_lies` явно проверен на падение против до-фиксной реализации (`git checkout HEAD -- client.py` + прогон теста -> `assert True is False`), затем зелёный после возврата фикса. - [x] `pytest tests/services/tgbot/test_topic_check_and_group_rate_limit.py tests/test_alerts_become_events.py` -- 34 passed. - [x] `ruff check` на изменённые файлы -- чисто. Closes #3471 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01JY6iWDnGDthdvsMWgK1BMG

bot-backend added 1 commit 2026-09-13 10:32:15 +00:00

fix(tradein): startup topic-check реально валидирует message_thread_id

CI Trade-In / changes (pull_request) Successful in 10s

Details

CI / changes (pull_request) Successful in 13s

Details

CI Trade-In / browser-tests (pull_request) Has been skipped

Details

CI Trade-In / frontend-checks (pull_request) Has been skipped

Details

CI / backend-tests (pull_request) Has been skipped

Details

CI / frontend-tests (pull_request) Has been skipped

Details

CI / openapi-codegen-check (pull_request) Has been skipped

Details

CI Trade-In / backend-tests (pull_request) Successful in 4m51s

Details

a2e8c46db7

sendChatAction не проверяет message_thread_id — живой прод-замер (#3471)
показал, что она подтверждает даже заведомо несуществующую тему
({"ok":true,"result":true}). verify_chat_and_topic поэтому была
ложно-зелёной ровно в сценарии, ради которого её писали: удалённая или
переименованная тема форума проходила проверку, а реальная отправка
потом падала на каждом сообщении.

Заменил проверку темы на sendMessage+deleteMessage: sendMessage с
несуществующим message_thread_id мгновенно отвечает 400 "message
thread not found" (тоже подтверждено на проде), а с валидным создаёт
сообщение, которое сразу стирается deleteMessage — в истории треда не
остаётся ничего. Docstring send_chat_action поправлен — он утверждал
обратное.

Новый тест воспроизводит замеренное поведение (sendChatAction лжёт,
sendMessage честно отказывает) и явно проверен на падение против
до-фиксной реализации (git checkout HEAD -- client.py + прогон теста
дал assert True is False).

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01JY6iWDnGDthdvsMWgK1BMG