fix(site-finder): разгейтить geoportal зон-резолвер от gappy NSPD-дампа — починить видимость финмодели (#1881) #1891

Merged

bot-backend merged 3 commits from fix/zoning-resolution-decouple into main

2026-06-23 23:55:53 +00:00

bot-backend commented

2026-06-23 22:25:39 +00:00

Collaborator

Проблема (прод-замер 2026-06-24, postgres-gendesign MCP)

financial_estimate фаерит ~0% (0/542 анализов analyze-1.0 за 90д). Резолв ПЗЗ-зоны на участок проваливается на 94%: max_far есть у 15/542, can_build_mkd=true у 4/542, пересечение = 0.

В analyze два резолвера зоны:

_get_zoning (nspd_quarter_dumps, точный ST_Intersects) — терзоны не замощают квартал плотно, центроид участка падает в 45–86 м зазор между валидными зонами → nspd_zoning=None у 512/542.
get_or_fetch_zone_regulation → EKBGeoportalClient.zone_index_at (живой геопортал, cache-first) — РАБОТАЕТ, резолвит gap-участки (проверено: возвращает Ж-5/ПК-1/ЦС-3 там, где дамп пуст).

Но (2), дающий max_far, был загейчен за _nspd_zoning is not None (parcels.py:2074) — зазор дампа глушил рабочий резолвер. Плюс дамп кладёт кадастровый рег-номер в zone_code, а gate.is_residential_zone матчит ^Ж → не срабатывал.

Фикс (PR-A)

Убран _nspd_zoning is not None из условия — geoportal-резолвер работает при centroid (region-guard: только КН 66:41, геопортал ЕКБ-only).
Когда дамп зону не дал → синтезируем минимальный nspd_zoning из geoportal: zone_code = индекс зоны ("Ж-2") → is_residential_zone ^Ж срабатывает → can_build_mkd резолвится для Ж-* зон. Write-back в nspd_dump_data["nspd_zoning"] доходит до gate / финмоста / ответа (читают по ключу).
Когда дамп зону дал → zone_code не перетираем (raw_props.subcategory детектит жильё), только добавляем regulation-поля. Расширение на Ц-/ЦС- → PR-B.

Геопортал — authoritative (point-in-zone по ПОЛНОМУ слою ПЗЗ); «зазор» только в нашем кэше-дампе. Hot-path-safe (try/except, timeout 3с, cache-first).

Ожидаемый эффект

Прод-замер до: 66:41:0303006:930 (Ж-5 gap) → zone_code/max_far/financial=None, can_build=False. После деплоя ожидаем Ж-5 / max_far=4 / can_build=True / financial=PRESENT (провалидирую на проде).

Test plan

+7 тестов: синтез при dump=None+geoportal; нет синтеза при обоих None; dump zone_code не перетёрт; geoportal падает → деградация; is_residential_zone Ж-2=True / ПК-1/ЦС-3=False. Кейс синтеза ассертит РЕАЛЬНЫЙ gate can_build_mkd=True.
47 passed, 0 failed (вкл. pre-existing zoning+gate). ruff + mypy чисто.
Схема не менялась → api-types regen не нужен (openapi-codegen-check пройдёт).
code-review: 2× approve, 0 majors (adversarial gate-correctness lens прошёл все 5 проверок: write-back доходит, нет false-positive от authoritative-источника, region-guard, hot-path try/except).

⚠️ Известные follow-ups

PR-B: broaden is_residential_zone на Ц-/ЦС- МКД-зоны через zone_regulation_cache.main_vri (нужна ПЗЗ-валидация какие индексы разрешают МКД).
Latency: zone_index_at — живой WFS-вызов на каждый ЕКБ-analyze (~1-2с, был только у 6%). Приемлемо (gated 3с timeout); оптимизация (dump-buffer-first) — отдельно.

Refs #1881

🤖 Generated with Claude Code

## Проблема (прод-замер 2026-06-24, postgres-gendesign MCP) `financial_estimate` фаерит **~0%** (0/542 анализов analyze-1.0 за 90д). Резолв ПЗЗ-зоны на участок проваливается на **94%**: `max_far` есть у 15/542, `can_build_mkd=true` у 4/542, пересечение = 0. В analyze **два** резолвера зоны: 1. `_get_zoning` (`nspd_quarter_dumps`, точный `ST_Intersects`) — терзоны **не замощают квартал плотно**, центроид участка падает в **45–86 м зазор** между валидными зонами → `nspd_zoning=None` у 512/542. 2. `get_or_fetch_zone_regulation` → `EKBGeoportalClient.zone_index_at` (живой геопортал, cache-first) — **РАБОТАЕТ**, резолвит gap-участки (проверено: возвращает Ж-5/ПК-1/ЦС-3 там, где дамп пуст). Но (2), дающий `max_far`, был **загейчен** за `_nspd_zoning is not None` (`parcels.py:2074`) — зазор дампа глушил рабочий резолвер. Плюс дамп кладёт кадастровый рег-номер в `zone_code`, а `gate.is_residential_zone` матчит `^Ж` → не срабатывал. ## Фикс (PR-A) - Убран `_nspd_zoning is not None` из условия — geoportal-резолвер работает при centroid (region-guard: только КН `66:41`, геопортал ЕКБ-only). - Когда дамп зону не дал → **синтезируем** минимальный `nspd_zoning` из geoportal: `zone_code` = индекс зоны (`"Ж-2"`) → `is_residential_zone` `^Ж` срабатывает → `can_build_mkd` резолвится для Ж-* зон. Write-back в `nspd_dump_data["nspd_zoning"]` доходит до gate / финмоста / ответа (читают по ключу). - Когда дамп зону **дал** → `zone_code` не перетираем (raw_props.subcategory детектит жильё), только добавляем regulation-поля. Расширение на Ц-*/ЦС-* → **PR-B**. Геопортал — authoritative (point-in-zone по ПОЛНОМУ слою ПЗЗ); «зазор» только в нашем кэше-дампе. Hot-path-safe (try/except, timeout 3с, cache-first). ## Ожидаемый эффект Прод-замер до: `66:41:0303006:930` (Ж-5 gap) → `zone_code/max_far/financial=None, can_build=False`. После деплоя ожидаем **Ж-5 / max_far=4 / can_build=True / financial=PRESENT** (провалидирую на проде). ## Test plan - [x] +7 тестов: синтез при dump=None+geoportal; нет синтеза при обоих None; dump zone_code не перетёрт; geoportal падает → деградация; `is_residential_zone` Ж-2=True / ПК-1/ЦС-3=False. Кейс синтеза ассертит **РЕАЛЬНЫЙ** gate `can_build_mkd=True`. - [x] 47 passed, 0 failed (вкл. pre-existing zoning+gate). ruff + mypy чисто. - [x] Схема не менялась → api-types regen не нужен (openapi-codegen-check пройдёт). - [x] code-review: **2× approve, 0 majors** (adversarial gate-correctness lens прошёл все 5 проверок: write-back доходит, нет false-positive от authoritative-источника, region-guard, hot-path try/except). ## ⚠️ Известные follow-ups - **PR-B:** broaden `is_residential_zone` на Ц-*/ЦС-* МКД-зоны через `zone_regulation_cache.main_vri` (нужна ПЗЗ-валидация какие индексы разрешают МКД). - Latency: `zone_index_at` — живой WFS-вызов на каждый ЕКБ-analyze (~1-2с, был только у 6%). Приемлемо (gated 3с timeout); оптимизация (dump-buffer-first) — отдельно. Refs #1881 🤖 Generated with [Claude Code](https://claude.com/claude-code)

Rows
Columns