Skip to content

fix(tg): Трейдинг text-button -> human report (was old treasury report) - #202

Merged
JoTalbot merged 1 commit into
mainfrom
agent/20260819-trading-button-fix
Aug 19, 2026
Merged

fix(tg): Трейдинг text-button -> human report (was old treasury report)#202
JoTalbot merged 1 commit into
mainfrom
agent/20260819-trading-button-fix

Conversation

@JoTalbot

Copy link
Copy Markdown
Owner

Текстовая кнопка Трейдинг вела на старый treasury-отчёт; теперь перехват до treasury-интента и единый хелпер отправки человеческого отчёта.

…d report) - shared send_full_report helper, intercept before treasury intent in accounts.py, inline callback unified
@JoTalbot
JoTalbot merged commit 1d87fb8 into main Aug 19, 2026
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@JoTalbot
JoTalbot deleted the agent/20260819-trading-button-fix branch August 19, 2026 11:27
@github-actions github-actions Bot added the tests label Aug 19, 2026
Comment thread tg_bot/trading_report.py
for msg in format_report(snap):
api.send_message(chat_id, msg)
api.send_message(chat_id, "⏳ LLM-аналитика готовится…")
import threading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔥 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.

@kilo-code-bot

kilo-code-bot Bot commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

Code Review Roast 🔥

Verdict: 1 Issue Found | Recommendation: Address before merge

Overview

Severity Count
🚨 critical 0
⚠️ warning 0
💡 suggestion 0
🤏 nitpick 1
Issue Details (click to expand)
File Line Roast
tg_bot/trading_report.py 552 import threading внутри функции send_full_report — импорт stdlib внутри тела функции на каждом вызове

🏆 Best part: Тесты покрывают новый путь кнопки текстовой клавиатуры — наконец-то кто-то думает про regression.

💀 Worst part: import threading внутри функции на строке 552. Это не баг, это вызов PEP 8 на дуэль.

📊 Overall: Как первый снег в августе — неожиданно, но в целом приятно. Одна мелочь исправить, и будет чисто.

Files Reviewed (7 files)
  • coordination/PROJECT_CONTEXT.md - 0 issues
  • coordination/sessions/20260819T010000Z-aios-arena-tg-trading-report.md - 0 issues
  • tests/test_trading_button_path.py - 0 issues
  • tests/test_trading_report.py - 0 issues
  • tg_bot/accounts.py - 0 issues
  • tg_bot/callbacks.py - 0 issues
  • tg_bot/trading_report.py - 1 issue

Fix these issues in Kilo Cloud


Reviewed by free · Input: 144.2K · Output: 21.4K · Cached: 317.2K

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant