fix(site-finder): NO_ENGINEERING_NEARBY is a warning, not a verdict blocker (#1617) #1697

Merged
lekss361 merged 1 commit from fix/gate-verdict-no-blocker-1617 into main 2026-06-17 18:11:45 +00:00
Owner

Проблема (#1617)

Финальная сборка вердикта (elif warnings: label="С ограничениями") понижала вердикт на ЛЮБОЙ warning. NO_ENGINEERING_NEARBY (нет инж.сетей рядом) — информационный, но форсил «С ограничениями» даже для чистого Ж-участка.

Решение (warning ≠ блокер — выбор владельца)

INFORMATIONAL_WARNING_CODES = frozenset({"NO_ENGINEERING_NEARBY"}). Понижение считается по verdict_relevant_warnings (без информационных). Код остаётся в warnings[] для UI.

Before/after

  • Чистый Ж-2, нет ЗОУИТ, нет инж.сетей → было «С ограничениями», стало «Можно»
  • Ж-2 + реальный ЗОУИТ-warning → по-прежнему «С ограничениями»
  • Ж-3 + ZOUIT_OVERLAP_SUB17 → по-прежнему «Нельзя» (блокер не тронут)

Verify

ruff clean; pytest test_gate_verdict.py102 passed, 1 skipped (+2 теста: реальный warning всё ещё понижает, блокер всё ещё реджектит).

Closes #1617

## Проблема (#1617) Финальная сборка вердикта (`elif warnings: label="С ограничениями"`) понижала вердикт на ЛЮБОЙ warning. `NO_ENGINEERING_NEARBY` (нет инж.сетей рядом) — информационный, но форсил «С ограничениями» даже для чистого Ж-участка. ## Решение (warning ≠ блокер — выбор владельца) `INFORMATIONAL_WARNING_CODES = frozenset({"NO_ENGINEERING_NEARBY"})`. Понижение считается по `verdict_relevant_warnings` (без информационных). Код остаётся в `warnings[]` для UI. ## Before/after - Чистый Ж-2, нет ЗОУИТ, нет инж.сетей → было «С ограничениями», стало **«Можно»** - Ж-2 + реальный ЗОУИТ-warning → по-прежнему «С ограничениями» - Ж-3 + ZOUIT_OVERLAP_SUB17 → по-прежнему «Нельзя» (блокер не тронут) ## Verify `ruff` clean; `pytest test_gate_verdict.py` → **102 passed, 1 skipped** (+2 теста: реальный warning всё ещё понижает, блокер всё ещё реджектит). Closes #1617
lekss361 added 1 commit 2026-06-17 18:00:27 +00:00
fix(site-finder): NO_ENGINEERING_NEARBY is a warning, not a verdict blocker (#1617)
Some checks failed
CI / changes (push) Successful in 9s
CI / changes (pull_request) Successful in 7s
CI / frontend-tests (push) Has been skipped
CI / backend-tests (pull_request) Has been cancelled
CI / openapi-codegen-check (push) Failing after 1m40s
CI / frontend-tests (pull_request) Has been cancelled
CI / openapi-codegen-check (pull_request) Has been cancelled
CI / backend-tests (push) Failing after 8m47s
ac19ef4823
Introduce INFORMATIONAL_WARNING_CODES frozenset; verdict assembly now only
downgrades to «С ограничениями» when verdict-relevant warnings exist.
NO_ENGINEERING_NEARBY still appears in warnings[] for display but a clean
residential parcel (Ж-zone, no ЗОУИТ) correctly receives verdict_label=«Можно».
lekss361 merged commit f55e83a150 into main 2026-06-17 18:11:45 +00:00
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#1697
No description provided.