В ответе /admin/leads/stats соседствуют величины двух видов, и соглашение
читается прямо по именам: `_window` — за окно months, `_total` — за всё время.
revenue_total и deals_total его нарушали: считались по CTE window_leads, то есть
за окно, а суффиксом обещали итог — рядом с честными leads_total и sources_total.
Админка из-за этого печатала карточку «Revenue (всего)» с 12-месячной цифрой.
Переименованы в revenue_window / deals_window. Потребитель ровно один —
frontend/src/app/admin/leads/page.tsx; там же подпись карточки теперь берёт
ширину окна из window_months, а не утверждает «всего». Сгенерированные типы
перегенерированы: полей эндпоинта в схеме нет (возвращает dict[str, Any]),
менялось только описание, поэтому рационал убран из docstring в комментарий —
docstring уходит в OpenAPI description и дальше во фронтовые типы.
Заодно: window_months отдавался ТОЛЬКО в непустой ветке ответа, формы
различались. Про эту ветку сказано прямо и в коде, и в тесте: она СЕГОДНЯ
недостижима — запрос агрегатный и на пустых таблицах возвращает обычную строку
(проверено: leads_total=0, leads_window=0, строка truthy). То есть правка там —
согласованность, а не наблюдаемая починка, и тест её НЕ покрывает.
Тест проверяет инвариант, а не набор имён: на данных, где итог заведомо не равен
окну (5 сделок на 17 млн, из них в окне 2 на 2 млн), каждое поле `*_total`
обязано совпасть с итогом. Против origin/main красное по неверному ЗНАЧЕНИЮ,
а не по отсутствию ключа:
revenue_total = 2 000 000 при итоге 17 000 000 → падает
revenue_window отсутствует → падает
оконные величины не поехали — контроль, зелёный с обеих сторон
форма ответа на пустых данных — контроль, зелёный с обеих сторон
Прогоны: tests/sql (живой Postgres) 4 passed rc=0; tsc --noEmit rc=0.
Четыре nodeid в skip_allowlist.txt — нужен Postgres, в CI идут.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
(:bind || ' months')::interval is a psycopg v3 SyntaxError — the driver
sees '::' immediately after the bind placeholder token and chokes.
Replace all 6 occurrences (admin_leads.py + analytics_queries.py) with
make_interval(months => :bind) which is unambiguous to the parser and
semantically identical.
Both metrics were querying prinzip_deals without any date filter,
returning all-time figures while the surrounding stats (leads_window,
converted_window, conv_pct_window) were scoped to the last N months.
Now both subqueries restrict to deals linked to leads in window_leads
(via deal_id IN (...)), making all «за период» figures consistent.
