fix(report): newbuild-consistent медиана + obj_class/Comfort dedup + recommend_mix regression (#1955 #1960) #1978

Merged

bot-backend merged 3 commits from fix/report-data-1953 into main

2026-06-27 11:41:37 +00:00

bot-backend commented

2026-06-27 10:57:20 +00:00

Collaborator

Батч 2 аудита отчёта (эпик #1953). code-reviewer ✅ APPROVE (включая re-review дельты).

#1960 «Медиана рынка» = 64k → новостроечный базис

Карточка брала median_12m из MV по кадастровому КВАРТАЛУ (1 ДКП-сделка = 63694), хотя весь остальной отчёт — новостроечный Objective/DOM.РФ. Новая цепочка базисов (parcels.py, документирована в коде):

Objective median по району → 2. geo_radius_price.median (Objective 3км, тот же источник что и весь отчёт) → 3. ekb_districts (ДДУ ref) → 4. квартальная rosreestr median_12m только при deals_count ≥ 5.
Прод-verify для 66:41:0205010:287: базис2 geo_radius = 132690 (n=12085 лотов, 14 ЖК) вместо 63694. Добавлен median_price_basis в payload (медиана теперь nullable — все консьюмеры защищены гардами; подпись «12 мес»→«новостройки»).

#1955 «Что хорошо продаётся»: дубли бакетов + фантом Comfort/Комфорт

Миграция data/sql/172_fix_v_bucket_success_score_obj_class.sql (CREATE OR REPLACE VIEW, идемпотентна): COALESCE(o.obj_class,'Comfort') → COALESCE(o.obj_class,'не указан'). Прод-аудит: источник имеет только канонический русский (Комфорт 723 / NULL 397 / Типовой 105 / Бизнес 84 / Элит 7 / Премиум 6) — английский Comfort создавался ИСКЛЮЧИТЕЛЬНО литералом COALESCE (фантом-расщепление). Все 11 колонок/GROUP BY/HAVING идентичны оригиналу (byte-diff).
obj_class проброшен в success-ranking SELECT + dict + TS SuccessRankingBucket.obj_class. (UI-лейбл с классом — отдельный фронт-тикет.)

Регрессия от миграции 172 (поймана ревью, исправлена `a184d38`)

recommend_mix success-boost (analytics_queries.py, /analytics/recommend квартирография) матчил obj_class = COALESCE(:cls,'Comfort') — опирался на баговый английский литерал. После 172 → 0 строк (тихая деградация). Фикс: перевод класса _class_to_db_vocab (target_class_db) + дефолт COALESCE(:cls,'Комфорт') (также чинит латентный Business/Elite). Прод-verify: 0 строк → 5 строк (восстановлено). Тест теперь реально ловит регрессию (раньше мок прятал её через patch→[]).

Безопасность / контракт

Миграция 172: воркер вручную применил CREATE OR REPLACE VIEW на проде (идемпотентно, без потери данных); deploy переприменит и запишет в _schema_migrations. ⚠️ На проде сейчас view 172 + СТАРЫЙ код recommend_mix → success-boost пуст — деплой этой ветки и есть фактическое восстановление.
ruff чисто, CAST-rule соблюдён, pytest -k "recommend or bucket or success or median or analyze" → 269 passed / 0 failed (+ новые тесты на регрессию и 4-й fallback). frontend tsc --noEmit exit 0.

Батч 2 аудита отчёта (эпик #1953). code-reviewer ✅ APPROVE (включая re-review дельты). ## #1960 «Медиана рынка» = 64k → новостроечный базис Карточка брала `median_12m` из MV по кадастровому КВАРТАЛУ (1 ДКП-сделка = 63694), хотя весь остальной отчёт — новостроечный Objective/DOM.РФ. Новая цепочка базисов (`parcels.py`, документирована в коде): 1. Objective median по району → 2. `geo_radius_price.median` (Objective 3км, тот же источник что и весь отчёт) → 3. `ekb_districts` (ДДУ ref) → 4. квартальная rosreestr `median_12m` **только** при `deals_count ≥ 5`. Прод-verify для 66:41:0205010:287: базис2 `geo_radius` = **132690** (n=12085 лотов, 14 ЖК) вместо 63694. Добавлен `median_price_basis` в payload (медиана теперь nullable — все консьюмеры защищены гардами; подпись «12 мес»→«новостройки»). ## #1955 «Что хорошо продаётся»: дубли бакетов + фантом Comfort/Комфорт - **Миграция `data/sql/172_fix_v_bucket_success_score_obj_class.sql`** (`CREATE OR REPLACE VIEW`, идемпотентна): `COALESCE(o.obj_class,'Comfort')` → `COALESCE(o.obj_class,'не указан')`. Прод-аудит: источник имеет только канонический русский (Комфорт 723 / NULL 397 / Типовой 105 / Бизнес 84 / Элит 7 / Премиум 6) — английский `Comfort` создавался ИСКЛЮЧИТЕЛЬНО литералом COALESCE (фантом-расщепление). Все 11 колонок/GROUP BY/HAVING идентичны оригиналу (byte-diff). - **obj_class проброшен** в success-ranking SELECT + dict + TS `SuccessRankingBucket.obj_class`. (UI-лейбл с классом — отдельный фронт-тикет.) ## Регрессия от миграции 172 (поймана ревью, исправлена `a184d38`) `recommend_mix` success-boost (`analytics_queries.py`, /analytics/recommend квартирография) матчил `obj_class = COALESCE(:cls,'Comfort')` — опирался на баговый английский литерал. После 172 → 0 строк (тихая деградация). Фикс: перевод класса `_class_to_db_vocab` (`target_class_db`) + дефолт `COALESCE(:cls,'Комфорт')` (также чинит латентный Business/Elite). Прод-verify: 0 строк → **5 строк** (восстановлено). Тест теперь реально ловит регрессию (раньше мок прятал её через patch→[]). ## Безопасность / контракт - Миграция 172: воркер вручную применил `CREATE OR REPLACE VIEW` на проде (идемпотентно, без потери данных); deploy переприменит и запишет в `_schema_migrations`. ⚠️ **На проде сейчас view 172 + СТАРЫЙ код recommend_mix → success-boost пуст** — деплой этой ветки и есть фактическое восстановление. - ruff чисто, CAST-rule соблюдён, `pytest -k "recommend or bucket or success or median or analyze"` → 269 passed / 0 failed (+ новые тесты на регрессию и 4-й fallback). frontend `tsc --noEmit` exit 0.

bot-backend added 2 commits 2026-06-27 10:57:20 +00:00

fix(report): newbuild-consistent district median + obj_class dedup (#1953 ) 453a1f08da

FIX A (#1955) «Что хорошо продаётся»: убран фантомный класс 'Comfort'.
- Миграция 172: v_bucket_success_score COALESCE(obj_class,'Comfort')
  → COALESCE(obj_class,'не указан'). Английский литерал заполнял 397 NULL
  и сливался отдельным классом от русского 'Комфорт' → визуальные дубли
  бакетов в UI. Источник уже канонически-русский (проверено на проде),
  synonym-mapping не нужен.
- parcels.py: obj_class протаскивается в success-ranking query + dict.
- TS SuccessRankingBucket.obj_class добавлен.

FIX B (#1960) «Медиана рынка» = 64k (квартальная росреестровская n=1 ДКП):
- district.median_price_per_m2 больше не COALESCE(median_12m, ekb_ref) в SQL.
  Basis-приоритет (newbuild-first): Objective по имени района →
  geo_radius (Objective в 3км) → ekb_districts reference →
  квартальная росреестровская медиана ТОЛЬКО при deals_count≥5.
  Для 66:41:0205010:287: 64k → 132690 (geo_radius, newbuild-consistent).
- median_price_basis добавлен в payload + TS type (nullable median).
- Frontend null-guards для нового nullable median.

Tests: +4 (geo_radius basis, objective-приоритет, deals-guard, obj_class
passthrough); обновлены district-моки в 9 analyze-тестах под новую
SQL-сигнатуру.

fix(analytics): recommend_mix success-boost regression from view 172 (#1955 )

CI / changes (pull_request) Successful in 6s

Details

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

Details

CI / openapi-codegen-check (pull_request) Failing after 1m51s

Details

CI / backend-tests (pull_request) Successful in 13m47s

Details

a184d38aa6

Миграция 172 убрала english 'Comfort' из v_bucket_success_score (NULL-class
→ 'не указан'), но _bucket_success_ranking в analytics_queries.py матчил
COALESCE(:cls, 'Comfort') — после 172 ни одна строка не равна 'Comfort' →
recommend_mix success-boost (#25, /analytics/recommend квартирография) тихо
возвращал [] (silent degradation). LIVE на проде с момента ручного применения
172 (graceful empty, без краша).

Fix:
- call site (recommend_mix): передаём target_class_db (уже переведённый через
  _class_to_db_vocab english→русский), а не сырой english target_class.
  Чинит и латентный pre-existing баг: Business/Elite никогда не матчили
  русский источник.
- SQL default 'Comfort' → 'Комфорт' (массовый класс ЕКБ, 723 объекта).
  NULL-class ('не указан') в default-путь намеренно не попадают
  (документировано в docstring).

Prod-verified: OLD 'Comfort' default → 0 строк; NEW 'Комфорт' default → 5.

Tests:
- _bucket_success_ranking: реальный запрос (не замокан) с русским default —
  SQL содержит 'Комфорт', не 'Comfort'; ranking непустой.
- recommend_mix: english 'Business' переводится в 'Бизнес' перед ranking
  (раньше тест патчил _bucket_success_ranking→[] и баг не ловил;
  helper теперь умеет patch_success_ranking=False).
- #1960: позитивный тест выбора quarter_rosreestr basis (deals≥5, все
  higher fallback'и None) → median_price_basis='quarter_rosreestr'.