fix(ptica): выручка и сделки в KPI лидов названы по своему охвату (#2464) #2963

Merged

bot-backend merged 1 commit from fix/2464-leads-window-naming into main

2026-08-20 09:44:15 +00:00

bot-backend commented

2026-08-20 09:16:46 +00:00

Collaborator

Пункт эпика #2464: admin_leads.py:152.

Что не так

В ответе /admin/leads/stats соседствуют величины двух видов, и соглашение читается прямо по именам:

leads_total, sources_total          — за всё время
leads_window, converted_window,
conv_pct_window                     — за окно months
revenue_total, deals_total          — ??? считаются по CTE window_leads

Последние две считались за окно, а суффиксом обещали итог. Админка из-за этого печатала карточку «Revenue (всего)» с 12-месячной цифрой — рядом с честными «За 12 мес» и «Conversion (12 мес)».

Правка

revenue_total → revenue_window, deals_total → 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), поэтому if not row не срабатывает. Правка там — согласованность, а не наблюдаемая починка, и тест её не покрывает. Ветка оставлена как защита на случай смены формы запроса; расходиться с основной ей нельзя — именно так пропажа поля и возникла.

Тест

Проверяет инвариант, а не набор имён: на данных, где итог заведомо не равен окну (5 сделок на 17 млн, из них в окне 2 на 2 млн), каждое поле *_total обязано совпасть с итогом.

Это принципиально: проверка «есть ли ключ revenue_window» дала бы на origin/main KeyError, то есть «возможности нет». Здесь красное — по неверному значению:

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 идут.

Пункт эпика #2464: `admin_leads.py:152`. ## Что не так В ответе `/admin/leads/stats` соседствуют величины двух видов, и соглашение читается прямо по именам: ``` leads_total, sources_total — за всё время leads_window, converted_window, conv_pct_window — за окно months revenue_total, deals_total — ??? считаются по CTE window_leads ``` Последние две считались **за окно**, а суффиксом обещали итог. Админка из-за этого печатала карточку **«Revenue (всего)»** с 12-месячной цифрой — рядом с честными «За 12 мес» и «Conversion (12 мес)». ## Правка `revenue_total` → `revenue_window`, `deals_total` → `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), поэтому `if not row` не срабатывает. Правка там — согласованность, а не наблюдаемая починка, и тест её **не покрывает**. Ветка оставлена как защита на случай смены формы запроса; расходиться с основной ей нельзя — именно так пропажа поля и возникла. ## Тест Проверяет **инвариант**, а не набор имён: на данных, где итог заведомо не равен окну (5 сделок на 17 млн, из них в окне 2 на 2 млн), каждое поле `*_total` обязано совпасть с итогом. Это принципиально: проверка «есть ли ключ `revenue_window`» дала бы на `origin/main` `KeyError`, то есть «возможности нет». Здесь красное — по неверному **значению**: ``` 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 идут.

bot-backend added 1 commit 2026-08-20 09:16:46 +00:00

fix(ptica): выручка и сделки в KPI лидов названы по своему охвату (#2464 )

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

Details

CI / changes (pull_request) Successful in 9s

Details

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

Details

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

Details

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

Details

CI / frontend-tests (pull_request) Successful in 1m33s

Details

CI / openapi-codegen-check (pull_request) Successful in 2m25s

Details

CI / backend-tests (pull_request) Successful in 17m36s

Details

5f0567b154

В ответе /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>