fix(ptica): пустой слой риск-зон перестаёт означать «рисков нет» (#2934) #2940

Merged
bot-backend merged 1 commit from fix/2934-risk-coverage-signal into main 2026-08-19 16:34:27 +00:00
8 changed files with 318 additions and 139 deletions

View file

@ -160,6 +160,9 @@ def make_empty_result(
"harvest_triggered": harvest_triggered, "harvest_triggered": harvest_triggered,
"total_features": total_features, "total_features": total_features,
"harvest_eta_seconds": harvest_eta_seconds, "harvest_eta_seconds": harvest_eta_seconds,
# Дампа нет — про риск-слои не известно ничего. None, а не 0:
# ноль означал бы «спросили и не нашли».
"risks_count": None,
}, },
} }
@ -297,6 +300,20 @@ def get_quarter_dump_data(
"harvest_triggered": False, "harvest_triggered": False,
"total_features": total_features, "total_features": total_features,
"harvest_eta_seconds": None, "harvest_eta_seconds": None,
# #2934: сколько объектов риск-слоёв лежит в дампе КВАРТАЛА. Нужен, чтобы
# отличить «слой опрошен, на участке чисто» от «слой не дал ничего вообще».
# Пустой `nspd_risk_zones` сам по себе этих случаев не различает, и фронт
# рисовал по нему зелёное «Риски не обнаружены».
#
# Замер 19.08: risks_count = 0 у ВСЕХ 669 дампов (для сравнения zouit_count > 0
# у 581), то есть сегодня признак всегда говорит «не подтверждено». Это и есть
# правда: одиннадцать слоёв природного риска ни разу не вернули ни одного
# объекта.
#
# Намеренно НЕ отдаём layers_fetched: `layers_fetched.append(...)` в
# nspd_client.py:956 стоит ДО запроса, поэтому как признак «слой опрошен» он
# лжёт. Пока append не перенесён после успешного ответа, наружу его нельзя.
"risks_count": risks_count,
} }
if parcel_wkt is None: if parcel_wkt is None:

View file

@ -399,8 +399,7 @@ def _make_zouit_row(
def test_cad_zouit_overlaps_includes_geom_geojson() -> None: def test_cad_zouit_overlaps_includes_geom_geojson() -> None:
"""#255: каждый overlap содержит geom_geojson (string из ST_AsGeoJSON).""" """#255: каждый overlap содержит geom_geojson (string из ST_AsGeoJSON)."""
geojson_str = ( geojson_str = (
'{"type":"Polygon","coordinates":' '{"type":"Polygon","coordinates":' "[[[60.6,56.8],[60.7,56.8],[60.7,56.9],[60.6,56.8]]]}"
"[[[60.6,56.8],[60.7,56.8],[60.7,56.9],[60.6,56.8]]]}"
) )
rows = [ rows = [
_make_zouit_row( _make_zouit_row(
@ -549,3 +548,35 @@ def test_early_exit_partial_counts() -> None:
assert result["nspd_zoning"]["zone_code"] == "66:41-7.2" assert result["nspd_zoning"]["zone_code"] == "66:41-7.2"
assert result["nspd_zouit_overlaps"] == [] assert result["nspd_zouit_overlaps"] == []
assert result["nspd_engineering_nearby"] == [] assert result["nspd_engineering_nearby"] == []
# ── #2934: признак покрытия риск-слоёв доезжает до ответа ──────────────────────
#
# Пустой `nspd_risk_zones` неоднозначен: «слой опрошен, на участке чисто» и «слой
# не дал ничего вообще» выглядят одинаково. Фронт рисовал по нему зелёную плашку
# «Риски не обнаружены», то есть утверждал результат проверки, которой не было.
#
# Замер на проде 19.08.2026: risks_count = 0 у ВСЕХ 669 дампов (для сравнения
# zouit_count > 0 у 581). Одиннадцать слоёв природного риска ни разу не вернули
# ни одного объекта.
def test_empty_result_reports_unknown_risk_coverage() -> None:
"""Дампа нет → про риск-слои неизвестно НИЧЕГО.
Именно None, а не 0: ноль означал бы «спросили и не нашли».
"""
result = make_empty_result()
assert (
"risks_count" in result["nspd_dump"]
), "признак покрытия не отдаётся — фронт не сможет отличить «чисто» от «не спрашивали»"
assert result["nspd_dump"]["risks_count"] is None
def test_empty_result_stays_independent_between_calls() -> None:
"""Контроль: новое поле не сломало mutation-safety (см. шапку файла)."""
a = make_empty_result()
a["nspd_dump"]["risks_count"] = 42
assert make_empty_result()["nspd_dump"]["risks_count"] is None

View file

@ -20,7 +20,11 @@ export interface MarketLayerToggle {
key: MarketLayerKey; key: MarketLayerKey;
label: string; label: string;
color: string; color: string;
count: number; /**
* Число объектов слоя. `null` слой ничего не отдал и «0» рядом с измеренными
* счётчиками соседей читалось бы как измеренный ноль (#2934); показываем «н/д».
*/
count: number | null;
} }
interface Props { interface Props {
@ -114,7 +118,9 @@ export function CpLayerControlPanel({
{/* Empty state */} {/* Empty state */}
{data.dump_available && totalCount === 0 && ( {data.dump_available && totalCount === 0 && (
<div style={{ color: "#6b7280", fontSize: 11, marginBottom: 6 }}> <div
style={{ color: "#6b7280", fontSize: 11, marginBottom: 6 }}
>
0 точек подключения в этом квартале 0 точек подключения в этом квартале
</div> </div>
)} )}
@ -243,7 +249,8 @@ export function CpLayerControlPanel({
fontSize: 11, fontSize: 11,
}} }}
> >
Охранных зон: {data.summary.protection_zones_intersecting} Охранных зон:{" "}
{data.summary.protection_zones_intersecting}
</span> </span>
)} )}
</div> </div>
@ -308,7 +315,9 @@ export function CpLayerControlPanel({
}} }}
/> />
{ml.label} {ml.label}
<span style={{ color: "#9ca3af" }}>{ml.count}</span> <span style={{ color: "#9ca3af" }}>
{ml.count ?? "н/д"}
</span>
</label> </label>
); );
})} })}

View file

@ -107,7 +107,10 @@ export function LandTab({ data }: Props) {
<SectionLabel style={{ marginBottom: 10 }}> <SectionLabel style={{ marginBottom: 10 }}>
Природные риски (НСПД) Природные риски (НСПД)
</SectionLabel> </SectionLabel>
<NspdRiskZonesBlock riskZones={data.nspd_risk_zones} /> <NspdRiskZonesBlock
riskZones={data.nspd_risk_zones}
quarterRisksCount={data.nspd_dump?.risks_count}
/>
</div> </div>
)} )}

View file

@ -5,6 +5,15 @@ import type { RiskZone } from "@/types/nspd";
interface Props { interface Props {
riskZones: RiskZone[] | null | undefined; riskZones: RiskZone[] | null | undefined;
parcelAreaSqm?: number | null; parcelAreaSqm?: number | null;
/**
* #2934: сколько объектов риск-слоёв в дампе квартала. Пустой `riskZones` без
* этого признака неоднозначен, и блок рисовал по нему зелёное «Риски не
* обнаружены» то есть утверждал результат проверки, которой не было.
*
* `undefined`/`null` дампа нет; `0` слой не дал объектов на весь квартал;
* `>0` слой отработал, и отсутствие пересечений с участком уже настоящий факт.
*/
quarterRisksCount?: number | null;
} }
// Severity mapping: layer key suffix → severity tier // Severity mapping: layer key suffix → severity tier
@ -45,10 +54,52 @@ function formatArea(sqm: number | null): string | null {
return `${Math.round(sqm).toLocaleString("ru-RU")} м²`; return `${Math.round(sqm).toLocaleString("ru-RU")} м²`;
} }
export function NspdRiskZonesBlock({ riskZones, parcelAreaSqm }: Props) { export function NspdRiskZonesBlock({
riskZones,
parcelAreaSqm,
quarterRisksCount,
}: Props) {
const zones = riskZones ?? []; const zones = riskZones ?? [];
if (zones.length === 0) { if (zones.length === 0) {
// Зелёный success-стиль оставляем ТОЛЬКО когда слой реально что-то отдал по
// кварталу: тогда «на участке ничего не пересекается» — измеренный факт.
// Замер 19.08: risks_count = 0 у всех 669 дампов, слои risk_* не вернули ни
// одного объекта, — то есть сегодня показывается нейтральная ветка.
const layerAnswered = (quarterRisksCount ?? 0) > 0;
if (!layerAnswered) {
return (
<div
style={{
display: "flex",
alignItems: "center",
gap: 8,
background: "#f8fafc",
border: "1px solid #e2e8f0",
borderRadius: 8,
padding: "10px 14px",
}}
>
<span
style={{
background: "#e2e8f0",
color: "#475569",
borderRadius: 6,
padding: "2px 10px",
fontSize: 12,
fontWeight: 600,
}}
>
Не проверено
</span>
<span style={{ fontSize: 13, color: "#475569" }}>
Слои природного риска НСПД не вернули объектов по этому кварталу
отсутствие риска не подтверждено
</span>
</div>
);
}
return ( return (
<div <div
style={{ style={{

View file

@ -949,7 +949,12 @@ export function SiteMap({
key: "risk", key: "risk",
label: "Зоны риска", label: "Зоны риска",
color: MARKET_COLORS.risk, color: MARKET_COLORS.risk,
count: riskZoneList.length, // #2934: «0» в одном ряду с измеренными «Конкуренты 14» и
// «Красные линии 2» читается как измеренный ноль. Слои
// природного риска НСПД не вернули ни одного объекта на 669
// дампов (замер 19.08), поэтому при пустом слое показываем
// «н/д», а не число.
count: riskZoneList.length > 0 ? riskZoneList.length : null,
}, },
{ {
key: "opportunity", key: "opportunity",

View file

@ -0,0 +1,53 @@
/**
* #2934: блок природных рисков не должен утверждать «рисков нет», когда слой
* ничего не отдал.
*
* Бэкенд ВСЕГДА присылает `nspd_risk_zones` (пустым списком), поэтому условие
* показа в LandTab (`!== undefined`) истинно всегда, а ветка `zones.length === 0`
* рисовала зелёную плашку «Риски не обнаружены» с текстом «Риск-зоны НСПД на
* участке не выявлены».
*
* Замер на проде 19.08.2026: risks_count = 0 у всех 669 дампов, слои risk_* НСПД
* не вернули ни одного объекта то есть эта плашка показывалась всегда и всегда
* без основания.
*/
import { render, screen } from "@testing-library/react";
import { describe, expect, it } from "vitest";
import { NspdRiskZonesBlock } from "../NspdRiskZonesBlock";
import type { RiskZone } from "@/types/nspd";
const ZONE: RiskZone = {
layer: "risk_flooding",
subtype: "затопление",
geom_wkt: null,
intersection_area_sqm: 1234,
} as unknown as RiskZone;
describe("NspdRiskZonesBlock (#2934)", () => {
it("слой не дал объектов по кварталу → «не проверено», а не «рисков нет»", () => {
render(<NspdRiskZonesBlock riskZones={[]} quarterRisksCount={0} />);
expect(screen.getByText("Не проверено")).toBeInTheDocument();
expect(screen.queryByText("Риски не обнаружены")).not.toBeInTheDocument();
});
it("дампа нет → тоже «не проверено» (признак null)", () => {
render(<NspdRiskZonesBlock riskZones={[]} quarterRisksCount={null} />);
expect(screen.getByText("Не проверено")).toBeInTheDocument();
});
it("слой отработал по кварталу, но участок не пересекает → «рисков не обнаружено» законно", () => {
render(<NspdRiskZonesBlock riskZones={[]} quarterRisksCount={7} />);
expect(screen.getByText("Риски не обнаружены")).toBeInTheDocument();
});
it("контроль: найденные зоны рисуются как прежде", () => {
render(<NspdRiskZonesBlock riskZones={[ZONE]} quarterRisksCount={7} />);
expect(screen.queryByText("Не проверено")).not.toBeInTheDocument();
expect(screen.queryByText("Риски не обнаружены")).not.toBeInTheDocument();
});
});

View file

@ -65,6 +65,16 @@ export interface NspdDumpMeta {
// auto-stop re-poll после ETA*1.5. null когда dump уже available или harvest // auto-stop re-poll после ETA*1.5. null когда dump уже available или harvest
// не запущен (lock уже взят другим запросом). // не запущен (lock уже взят другим запросом).
harvest_eta_seconds?: number | null; harvest_eta_seconds?: number | null;
/**
* #2934: сколько объектов риск-слоёв лежит в дампе КВАРТАЛА.
*
* Отличает «слой опрошен, на участке чисто» (>0) от «слой не дал ничего вообще»
* (0). Пустой `nspd_risk_zones` сам по себе этих случаев не различает, а UI
* рисовал по нему зелёное «Риски не обнаружены».
*
* `null` дампа нет, про риск-слои не известно ничего.
*/
risks_count?: number | null;
} }
// Risk zones (issue #94 TIER 3) — from /analyze nspd_risk_zones field // Risk zones (issue #94 TIER 3) — from /analyze nspd_risk_zones field