feat(sf-legacy): EGRN table + ЗОУИТ + инженерные сети в Землю tab (доп. к #363) #364

Closed
lekss361 wants to merge 1 commit from feat/legacy-expose-new-b5-fields into main
Owner

Summary

Дополнение к #363 (Обзор tab inline patch). Worker отдельно расширил Землю tab с new B5 fields:

Files (5)

  • frontend/src/types/site-finder.ts — 4 type aliases: UtilitiesSummary, NspdZouitOverlap, NspdEngineeringPoint, EgrnData; расширение ParcelAnalysis
  • frontend/src/components/site-finder/EgrnPropertyTable.tsx — NEW 10-row ЕГРН table (кад. стоимость, площадь, ВРИ, статус, дата обновления); пропускает строки со значением "—"
  • frontend/src/components/site-finder/NspdZouitOverlapsBlock.tsx — рендер зон ЗОУИТ; пусто → зелёный «не обнаружено»; охранные зоны → warning/danger; дедуп по name
  • frontend/src/components/site-finder/NspdEngineeringNearbyBlock.tsx — инженерные сети из utilities.summary + nspd_engineering_nearby list + охранная зона ЛЭП flag
  • frontend/src/components/site-finder/LandTab.tsx — добавлены 3 новых блока вверху (ЕГРН → ЗОУИТ → Инженерные сети), conditional render

Note про overlap с #363

#363 добавляет EGRN compact + Сети рядом в Обзор tab (inline summary).
Этот PR добавляет full ЕГРН table + полный ЗОУИТ list + полные инженерные сети в Землю tab (детализация).

Дублирование scope минимальное — types/site-finder.ts может конфликтнуть при merge. Решение: merge #363 first → этот PR rebase или close если не нужно.

Verify

  • /legacy/site-finder → ввод cad → Земля tab → 3 новых блока вверху
  • Conditional render — пустые поля скрыты или показывают «не обнаружено»

User может выбрать: merge только #363 (минимальный) или оба (полное покрытие Обзор + Землю).

## Summary Дополнение к **#363** (Обзор tab inline patch). Worker отдельно расширил **Землю** tab с new B5 fields: ## Files (5) - `frontend/src/types/site-finder.ts` — 4 type aliases: `UtilitiesSummary`, `NspdZouitOverlap`, `NspdEngineeringPoint`, `EgrnData`; расширение `ParcelAnalysis` - `frontend/src/components/site-finder/EgrnPropertyTable.tsx` — NEW 10-row ЕГРН table (кад. стоимость, площадь, ВРИ, статус, дата обновления); пропускает строки со значением "—" - `frontend/src/components/site-finder/NspdZouitOverlapsBlock.tsx` — рендер зон ЗОУИТ; пусто → зелёный «не обнаружено»; охранные зоны → warning/danger; дедуп по name - `frontend/src/components/site-finder/NspdEngineeringNearbyBlock.tsx` — инженерные сети из `utilities.summary` + `nspd_engineering_nearby` list + охранная зона ЛЭП flag - `frontend/src/components/site-finder/LandTab.tsx` — добавлены 3 новых блока вверху (ЕГРН → ЗОУИТ → Инженерные сети), conditional render ## Note про overlap с #363 #363 добавляет EGRN compact + Сети рядом в **Обзор** tab (inline summary). Этот PR добавляет full ЕГРН table + полный ЗОУИТ list + полные инженерные сети в **Землю** tab (детализация). Дублирование scope минимальное — types/site-finder.ts может конфликтнуть при merge. Решение: merge #363 first → этот PR rebase или close если не нужно. ## Verify - `/legacy/site-finder` → ввод cad → Земля tab → 3 новых блока вверху - Conditional render — пустые поля скрыты или показывают «не обнаружено» User может выбрать: merge только #363 (минимальный) или оба (полное покрытие Обзор + Землю).
lekss361 added 1 commit 2026-05-18 04:25:21 +00:00
feat(sf-legacy): expose new B5 fields (gate_verdict + utilities + zouit + recommendation) в старые табы
Some checks failed
CI / backend (pull_request) Failing after 1m15s
CI / frontend (pull_request) Successful in 1m45s
9a26575a26
- Добавлены типы в ParcelAnalysis: utilities, nspd_zouit_overlaps, nspd_engineering_nearby, egrn
- Новый компонент NspdZouitOverlapsBlock — зоны с ограничениями ЗОУИТ (warn/danger)
- Новый компонент NspdEngineeringNearbyBlock — инженерные сети + охранная зона ЛЭП
- Новый компонент EgrnPropertyTable — данные участка из ЕГРН (10 полей)
- LandTab: условный рендер трёх новых блоков, расширен hasAny guard
- OverviewTab и MarketTab уже содержали GateVerdictBanner/ScoreBreakdownPanel и SuccessRecommendationBlock соответственно
Author
Owner

⚠️ Merge blocked: conflict with main after #363 merge (as PR description predicted).

mergeable=false, update_pr_branch returned 409: merge failed because of conflict. The conflict is on frontend/src/types/site-finder.ts (both #363 and #364 add type aliases for B5 fields).

Decision needed from user/author:

  • If both Обзор + Землю scope is desired → rebase #364, resolve types/site-finder.ts conflict (keep #363's additions, layer #364's UtilitiesSummary/NspdZouitOverlap/NspdEngineeringPoint/EgrnData on top), push fixup.
  • If #363 alone covers the requirement → close #364.

Code review deferred until branch is updated. Will re-evaluate on next polling iteration.

⚠️ **Merge blocked: conflict with main after #363 merge** (as PR description predicted). `mergeable=false`, `update_pr_branch` returned `409: merge failed because of conflict`. The conflict is on `frontend/src/types/site-finder.ts` (both #363 and #364 add type aliases for B5 fields). Decision needed from user/author: - If both Обзор + Землю scope is desired → rebase #364, resolve `types/site-finder.ts` conflict (keep #363's additions, layer #364's `UtilitiesSummary`/`NspdZouitOverlap`/`NspdEngineeringPoint`/`EgrnData` on top), push fixup. - If #363 alone covers the requirement → close #364. Code review deferred until branch is updated. Will re-evaluate on next polling iteration.
lekss361 closed this pull request 2026-05-18 04:30:50 +00:00
Author
Owner

Closing — base SHA ae0fce3 устарел, лежит до _upserted ruff fix (5960368), CI lint fail RUF059 Unpacked variable upserted is never used в tests/services/test_cadastre_bulk.py:1165/1200. User confirmed lint fail.

Scope этого PR (расширенный Землю tab) — bonus, не критичный для user request «не только старые данные». Минимальная версия (Обзор tab EGRN+utilities) уже merged в #363. CompetitorTable filter — отдельным PR #365 (fix/sf-legacy-filter-stale-competitors).

Если нужно расширить Землю tab позже — reopen через rebase onto latest main.

Closing — base SHA `ae0fce3` устарел, лежит до `_upserted` ruff fix (`5960368`), CI lint fail `RUF059 Unpacked variable upserted is never used` в `tests/services/test_cadastre_bulk.py:1165/1200`. User confirmed lint fail. Scope этого PR (расширенный Землю tab) — bonus, не критичный для user request «не только старые данные». Минимальная версия (Обзор tab EGRN+utilities) уже merged в **#363**. CompetitorTable filter — отдельным **PR #365** (`fix/sf-legacy-filter-stale-competitors`). Если нужно расширить Землю tab позже — reopen через rebase onto latest main.
Some checks failed
CI / backend (pull_request) Failing after 1m15s
CI / frontend (pull_request) Successful in 1m45s

Pull request closed

Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: lekss361/gendesign#364
No description provided.