fix(tg): Трейдинг text-button -> human report (was old treasury report) - #202
Conversation
…d report) - shared send_full_report helper, intercept before treasury intent in accounts.py, inline callback unified
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
| for msg in format_report(snap): | ||
| api.send_message(chat_id, msg) | ||
| api.send_message(chat_id, "⏳ LLM-аналитика готовится…") | ||
| import threading |
There was a problem hiding this comment.
🔥 The Roast: import threading внутри функции send_full_report — это как хранить запасной ключ в сейфе, ключ от которого у тебя в кармане. Работает, но каждый вызов тащит повторный импорт, и PEP 8 уже подписала petition на развод.
🩹 The Fix: Перенеси import threading в начало файла. Модули из stdlib не должны импортироваться внутри функции без веской причины.
📏 Severity: nitpick
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review Roast 🔥Verdict: 1 Issue Found | Recommendation: Address before merge Overview
Issue Details (click to expand)
🏆 Best part: Тесты покрывают новый путь кнопки текстовой клавиатуры — наконец-то кто-то думает про regression. 💀 Worst part: 📊 Overall: Как первый снег в августе — неожиданно, но в целом приятно. Одна мелочь исправить, и будет чисто. Files Reviewed (7 files)
Fix these issues in Kilo Cloud Reviewed by free · Input: 144.2K · Output: 21.4K · Cached: 317.2K |
Текстовая кнопка Трейдинг вела на старый treasury-отчёт; теперь перехват до treasury-интента и единый хелпер отправки человеческого отчёта.