europa-tech-srl/arnold-trader-app
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

[CL-AUDIT-TRADING] H12 cleanup: удаление мёртвого затенённого indicators.py #308

Merged
andrei merged 1 commit from feature/claude-postaudit-followups into master 2026-06-10 13:56:42 +00:00
Conversation 0 Commits 1 Files changed 2 +3 -54
andrei commented 2026-06-10 13:51:48 +00:00
Owner
Copy link

Что сделано

  • Удалён backend/app/strategies/indicators.py — мёртвый файл, затенённый пакетом app/strategies/indicators/ (при одинаковом имени Python выбирает пакет; все from . import indicators / from app.strategies.indicators import ... резолвятся в пакет).
  • В TRADING-LOGIC-AUDIT.md follow-up H12 помечен выполненным; follow-up по унификации leverage caps помечен как требующий sign-off (money-path).

Зачем

Follow-up H12 из торгового аудита (PR #303): баг деления в мёртвом файле не имел live-эффекта, но файл вводил в заблуждение при чтении кода и аудитах.

План тестирования

  • Smoke: from app.strategies.indicators import atr, adx, ema_slope, rsi — OK (резолв в пакет).
  • pytest -k "indicator or regime or momentum" — 176 passed.
  • pytest -k "synth or features or ml" — 87 passed.
  • ruff check app/strategies app/ml app/services/regime.py — clean.

Где могу ошибаться

  • Если какой-то внешний скрипт импортировал файл по прямому пути app/strategies/indicators.py (не через import-систему) — но grep по repo таких ссылок не нашёл.
## Что сделано - Удалён `backend/app/strategies/indicators.py` — мёртвый файл, затенённый пакетом `app/strategies/indicators/` (при одинаковом имени Python выбирает пакет; все `from . import indicators` / `from app.strategies.indicators import ...` резолвятся в пакет). - В `TRADING-LOGIC-AUDIT.md` follow-up H12 помечен выполненным; follow-up по унификации leverage caps помечен как требующий sign-off (money-path). ## Зачем Follow-up H12 из торгового аудита (PR #303): баг деления в мёртвом файле не имел live-эффекта, но файл вводил в заблуждение при чтении кода и аудитах. ## План тестирования - Smoke: `from app.strategies.indicators import atr, adx, ema_slope, rsi` — OK (резолв в пакет). - `pytest -k "indicator or regime or momentum"` — 176 passed. - `pytest -k "synth or features or ml"` — 87 passed. - `ruff check app/strategies app/ml app/services/regime.py` — clean. ## Где могу ошибаться - Если какой-то внешний скрипт импортировал файл по прямому пути `app/strategies/indicators.py` (не через import-систему) — но grep по repo таких ссылок не нашёл.
[CL-AUDIT-TRADING] H12 cleanup: удалён мёртвый затенённый app/strategies/indicators.py
All checks were successful
Arnold Forgejo CI / secret-scan (pull_request) Successful in 3s
Details
Forgejo Smoke Test / Smoke (pull_request) Successful in 1s
Details
Arnold Forgejo CI / frontend-audit (pull_request) Successful in 11s
Details
Arnold Forgejo CI / backend-tests (pull_request) Successful in 3m43s
Details
7ef92c5796 
Пакет app/strategies/indicators/ затеняет одноимённый файл (package wins),
все import'ы резолвятся в пакет — файл недостижим. Баг H12 существовал
только в мёртвом файле. Smoke: from app.strategies.indicators import
atr, adx, ema_slope, rsi — OK. pytest -k 'indicator or regime or momentum'
176 passed; -k 'synth or features or ml' 87 passed; ruff clean.
andrei merged commit a9305c68d0 into master 2026-06-10 13:56:42 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
europa-tech-srl/arnold-trader-app!308
No description provided.