соседи (#2111) #2113

Merged

bot-backend merged 1 commit from feat/secondary-max-info into main

2026-06-30 13:45:44 +00:00

bot-backend commented

2026-06-30 13:30:12 +00:00

Collaborator

Что

MEDIUM/LOW-пункты аудита #2111: попапы/карточки показывали данные, которые есть в ответе, но не доходили до UI. Все поля едут через существующие loose dict[str, Any] passthrough'ы (competitors, neighbors_summary) — без изменения контракта/codegen.

Конкурент (MarketLayers): «Застройщик» (dev_name) + «Ср. площадь кв.» (avg_area_pd — SQL уже считал в obj_pricing CTE, прокидываем явно в _competitor_with_coords).
§3 «Инж. сооружения рядом» (NspdEngineeringNearbyBlock): вторичная строка Назначение / Характеристики / Адрес / № ЕГРН (EngineeringStructure уже несёт эти поля после #2112).
Польз. точка (CustomPoiLayer): scope-метка («Глобальная точка» / «Привязана к участку …») + «Добавлена … · изменена …» (formatDate с NaN-guard).
Соседи (NeighborsBlock): «Назначение» + «Статус» (cad_buildings.purpose/status → добавлены в neighbors CTE SELECT + neighbor dict; прод-verified: колонки есть, ~100% заполнено).
Utility material → RU («сталь/металл/бетон/железобетон…», unknown → passthrough) — «человеческим языком» (ui-microcopy), завершает #2112.

Контракт / тесты

competitors / neighbors_summary — list[dict[str, Any]] passthrough → OpenAPI не меняется, codegen не нужен. FE-типы (ParcelAnalysisCompetitor.avg_area_pd, NeighborBuilding.purpose/status) — optional+nullable (backward-compat).
Юнит-тесты: _utility_characteristics material RU (known→RU, case-insensitive, unknown→passthrough); _competitor_with_coords avg_area_pd passthrough. Backend 36 relevant tests зелёные, ruff clean. Frontend tsc/eslint/prettier ✅.
Pre-push review: ✅ APPROVE (без блокеров).
⚠️ neighbor purpose/status покрыты только DB-gated EXPLAIN-тестом (integration), unit-покрытия нет by design — прод-verified вместо этого.

После merge+deploy — скриншот-верификация попапов на проде.

## Что MEDIUM/LOW-пункты аудита [#2111](https://git.gendsgn.ru/lekss361/gendesign/issues/2111): попапы/карточки показывали данные, которые **есть в ответе, но не доходили до UI**. Все поля едут через существующие loose `dict[str, Any]` passthrough'ы (`competitors`, `neighbors_summary`) — **без изменения контракта/codegen**. - **Конкурент** (`MarketLayers`): «Застройщик» (`dev_name`) + «Ср. площадь кв.» (`avg_area_pd` — SQL уже считал в `obj_pricing` CTE, прокидываем явно в `_competitor_with_coords`). - **§3 «Инж. сооружения рядом»** (`NspdEngineeringNearbyBlock`): вторичная строка Назначение / Характеристики / Адрес / № ЕГРН (`EngineeringStructure` уже несёт эти поля после #2112). - **Польз. точка** (`CustomPoiLayer`): scope-метка («Глобальная точка» / «Привязана к участку …») + «Добавлена … · изменена …» (`formatDate` с NaN-guard). - **Соседи** (`NeighborsBlock`): «Назначение» + «Статус» (`cad_buildings.purpose`/`status` → добавлены в neighbors CTE SELECT + neighbor dict; **прод-verified: колонки есть, ~100% заполнено**). - **Utility material → RU** («сталь/металл/бетон/железобетон…», unknown → passthrough) — «человеческим языком» (ui-microcopy), завершает #2112. ## Контракт / тесты - `competitors` / `neighbors_summary` — `list[dict[str, Any]]` passthrough → OpenAPI не меняется, codegen не нужен. FE-типы (`ParcelAnalysisCompetitor.avg_area_pd`, `NeighborBuilding.purpose/status`) — optional+nullable (backward-compat). - Юнит-тесты: `_utility_characteristics` material RU (known→RU, case-insensitive, unknown→passthrough); `_competitor_with_coords` avg_area_pd passthrough. Backend **36 relevant tests зелёные**, ruff clean. Frontend `tsc`/eslint/prettier ✅. - Pre-push review: ✅ APPROVE (без блокеров). - ⚠️ neighbor `purpose`/`status` покрыты только DB-gated EXPLAIN-тестом (integration), unit-покрытия нет by design — прод-verified вместо этого. После merge+deploy — скриншот-верификация попапов на проде.

Rows
Columns