fix(ptica): «0 мм осадков» перестаёт означать «данных нет» (#2464) (#2947)

backend/app/services/weather_cache.py

@@ -161,8 +161,15 @@ def _fetch_weather_remote(lat: float, lon: float) -> dict[str, Any] | None:
                     "avg_max_c": round(sum(t_max) / len(t_max), 1) if t_max else None,
                     "avg_min_c": round(sum(t_min) / len(t_min), 1) if t_min else None,
                 },
-                "precipitation_total_mm": round(sum(precip), 1) if precip else 0,
-                "precipitation_days": sum(1 for p in precip if p and p > 0.5),
+                # #2464: было `if precip else 0`. Ноль здесь означал бы «осадков не
+                # ожидается» — утверждение о погоде. Но пустой `precip` значит, что
+                # open-meteo не отдал ряд осадков вовсе, то есть мы НЕ ЗНАЕМ. Все шесть
+                # соседних агрегатов в этом же словаре при пустых данных дают None
+                # (min_c/max_c/avg_*/uv_index_max/max_speed_m_s) — осадки были
+                # единственным исключением, и именно они рисуются на фронте как
+                # измеренная величина (ptica-adapt заворачивает их в `real(...)`).
+                "precipitation_total_mm": round(sum(precip), 1) if precip else None,
+                "precipitation_days": (sum(1 for p in precip if p and p > 0.5) if precip else None),
                 "uv_index_max": round(max(uv), 1) if uv else None,
                 "wind": {
                     "dominant_direction_deg": (
@@ -251,7 +258,10 @@ def _fetch_seasonal_remote(lat: float, lon: float) -> dict[str, Any] | None:
                     "avg_precip_per_day_mm": (
                         round(sum(precip) / len(precip), 1) if precip else None
                     ),
-                    "total_precip_mm": round(sum(precip), 0) if precip else 0,
+                    # #2464: та же правка, что у прогноза выше. Особенно наглядно
+                    # здесь: соседняя строка avg_precip_per_day_mm считается из ЭТОГО ЖЕ
+                    # списка и при пустом даёт None, а сумма давала 0.
+                    "total_precip_mm": round(sum(precip), 0) if precip else None,
                     "days_observed": len(vals["t_max"]),
                 }
             return {

backend/tests/services/test_weather_cache.py

@@ -594,3 +594,87 @@ class TestWindDirectionAllNone:
         # Один сэмпл 90° → circular mean ровно 90° → rose[2] = "Восток"
         assert wind["dominant_direction_deg"] == 90
         assert wind["dominant_direction_label"] == "Восток"
+
+
+# ──────────────────────────────────────────────────────────────────────────────
+# 8. осадков нет в ответе → precipitation_* = None, а не 0 (#2464)
+#
+# Тот же класс, что раздел 7 выше (wind_d all-None → None, не 0.0°): «ноль» здесь
+# был бы утверждением о погоде («сухо»), тогда как пустой ряд значит, что мы просто
+# не знаем. Все шесть соседних агрегатов того же словаря при пустых данных дают None
+# — осадки были единственным исключением, и именно они рисуются на фронте как
+# измеренная величина (ptica-adapt заворачивал их в `real(...)` безусловно).
+# ──────────────────────────────────────────────────────────────────────────────
+
+
+def _make_forecast_response_precip_all_none() -> dict[str, Any]:
+    """Forecast, где все precipitation_sum = null (Open-Meteo так делает штатно)."""
+    return {
+        "daily": {
+            "time": ["2026-06-12", "2026-06-13", "2026-06-14"],
+            "temperature_2m_max": [20.0, 21.0, 22.0],
+            "temperature_2m_min": [10.0, 11.0, 12.0],
+            "precipitation_sum": [None, None, None],
+            "uv_index_max": [5.0, 5.0, 5.0],
+            "winddirection_10m_dominant": [180, 180, 180],
+            "windspeed_10m_max": [3.0, 4.0, 3.0],
+        }
+    }
+
+
+def _make_forecast_response_precip_missing() -> dict[str, Any]:
+    """Forecast без ключа precipitation_sum вовсе."""
+    return {
+        "daily": {
+            "time": ["2026-06-12", "2026-06-13", "2026-06-14"],
+            "temperature_2m_max": [20.0, 21.0, 22.0],
+            "temperature_2m_min": [10.0, 11.0, 12.0],
+            "uv_index_max": [5.0, 5.0, 5.0],
+            "winddirection_10m_dominant": [180, 180, 180],
+            "windspeed_10m_max": [3.0, 4.0, 3.0],
+        }
+    }
+
+
+class TestPrecipitationUnknownIsNotZero:
+    """Регрессия: без ряда осадков не должно возвращаться 0 (= «сухо»)."""
+
+    def _call_with_payload(self, payload: dict[str, Any]) -> dict[str, Any] | None:
+        get = MagicMock(return_value=_make_httpx_response(payload))
+        client_ctx = MagicMock()
+        client_ctx.__enter__ = MagicMock(return_value=MagicMock(get=get))
+        client_ctx.__exit__ = MagicMock(return_value=None)
+        with patch("app.services.weather_cache.httpx.Client", return_value=client_ctx):
+            return weather_cache.get_weather_cached(56.84, 60.59)
+
+    def test_all_none_precip_gives_none_not_zero(self) -> None:
+        result = self._call_with_payload(_make_forecast_response_precip_all_none())
+
+        assert result is not None, "ответ должен строиться и без ряда осадков"
+        assert result["precipitation_total_mm"] is None, (
+            f"ожидался None, получено {result['precipitation_total_mm']!r} — "
+            "ноль здесь читается как «осадков не будет»"
+        )
+        assert result["precipitation_days"] is None
+
+    def test_missing_precip_key_gives_none(self) -> None:
+        result = self._call_with_payload(_make_forecast_response_precip_missing())
+
+        assert result is not None
+        assert result["precipitation_total_mm"] is None
+        assert result["precipitation_days"] is None
+
+    def test_real_dry_week_is_still_a_measured_zero(self) -> None:
+        """КОНТРОЛЬ: настоящая сухая неделя — это измеренный 0.0, а не None.
+
+        Без этой проверки правку можно было бы «сделать» так, что осадки всегда
+        None, и оба теста выше стали бы зелёными по неверной причине.
+        """
+        payload = _make_forecast_response_precip_all_none()
+        payload["daily"]["precipitation_sum"] = [0.0, 0.0, 0.0]
+
+        result = self._call_with_payload(payload)
+
+        assert result is not None
+        assert result["precipitation_total_mm"] == 0.0
+        assert result["precipitation_days"] == 0

frontend/src/components/site-finder/EnvironmentTab.tsx

@@ -361,16 +361,23 @@ function WeatherBlock({
 
       <div style={{ display: "flex", flexDirection: "column", gap: 2 }}>
         <div style={{ fontSize: 11, color: "#6b7280" }}>Осадки</div>
-        <div style={{ fontSize: 14, fontWeight: 600, color: "#374151" }}>
-          {weather.precipitation_total_mm} мм
-          <span style={{ fontSize: 11, fontWeight: 400, color: "#6b7280" }}>
-            {" "}
-            (за {weather.forecast_days} дн.)
-          </span>
-        </div>
-        <div style={{ fontSize: 11, color: "#6b7280" }}>
-          {weather.precipitation_days} дн. с осадками
-        </div>
+        {weather.precipitation_total_mm != null ? (
+          <>
+            <div style={{ fontSize: 14, fontWeight: 600, color: "#374151" }}>
+              {weather.precipitation_total_mm} мм
+              <span style={{ fontSize: 11, fontWeight: 400, color: "#6b7280" }}>
+                {" "}
+                (за {weather.forecast_days} дн.)
+              </span>
+            </div>
+            <div style={{ fontSize: 11, color: "#6b7280" }}>
+              {weather.precipitation_days} дн. с осадками
+            </div>
+          </>
+        ) : (
+          /* #2464: «0 мм» здесь читалось бы как «сухо». Данных нет — так и пишем. */
+          <div style={{ fontSize: 13, color: "#9ca3af" }}>нет данных</div>
+        )}
       </div>
 
       {weather.uv_index_max != null && (

frontend/src/components/site-finder/SeasonalWeatherBlock.tsx

@@ -50,7 +50,10 @@ function SeasonCard({
         (экстремумы ↓{stats.min_t_c} / ↑{stats.max_t_c}°C)
       </div>
       <div style={{ fontSize: 12, color: "#374151" }}>
-        {stats.total_precip_mm} мм осадков
+        {/* #2464: null — нормали по осадкам не пришли, это не «ноль миллиметров». */}
+        {stats.total_precip_mm != null
+          ? `${stats.total_precip_mm} мм осадков`
+          : "осадки — нет данных"}
       </div>
     </div>
   );

frontend/src/components/site-finder/analysis/Section5Atmosphere.tsx

@@ -356,8 +356,9 @@ function WeatherWindBlock({
 }: {
   wind: ParcelAnalysisWind;
   uvMax: number | null | undefined;
-  precipMm: number;
-  precipDays: number;
+  // #2464: null — ряда осадков не было. «0 мм» читалось бы как «сухо».
+  precipMm: number | null;
+  precipDays: number | null;
   forecastDays: number;
   source: string;
 }) {
@@ -397,7 +398,9 @@ function WeatherWindBlock({
           style={{ display: "inline", marginRight: 4, verticalAlign: "middle" }}
           aria-hidden
         />
-        {precipMm} мм · {precipDays} дн. с осадками за {forecastDays} дн.
+        {precipMm != null
+          ? `${precipMm} мм · ${precipDays} дн. с осадками за ${forecastDays} дн.`
+          : "осадки — нет данных"}
       </div>
       {uvMax != null && (
         <div style={{ fontSize: 12, color: "var(--fg-secondary, #5B6066)" }}>

frontend/src/components/site-finder/ptica/ptica-adapt.ts

@@ -1413,9 +1413,21 @@ export function adaptEnvironmentDrawer(
         },
         {
           k: "Осадки (сумма)",
-          field: real(`${formatInt(wt.precipitation_total_mm)} мм`),
+          // #2464: раньше стояло real(...) безусловно, и при отсутствии ряда осадков
+          // «0 мм» подавалось как ИЗМЕРЕННОЕ значение. Образец рядом — avg_min_c
+          // строкой выше: есть значение → real, нет → notReal.
+          field:
+            wt.precipitation_total_mm != null
+              ? real(`${formatInt(wt.precipitation_total_mm)} мм`)
+              : notReal("нет данных об осадках"),
+        },
+        {
+          k: "Дней с осадками",
+          field:
+            wt.precipitation_days != null
+              ? real(formatInt(wt.precipitation_days))
+              : notReal("нет данных об осадках"),
         },
-        { k: "Дней с осадками", field: real(formatInt(wt.precipitation_days)) },
       ]
     : [{ k: "Погода", field: notReal("нет данных погоды") }];
 

frontend/src/types/site-finder.ts

@@ -95,8 +95,9 @@ export interface ParcelAnalysisWeather {
     avg_max_c: number | null;
     avg_min_c: number | null;
   };
-  precipitation_total_mm: number;
-  precipitation_days: number;
+  /** #2464: null — open-meteo не отдал ряд осадков. Ноль означал бы «сухо». */
+  precipitation_total_mm: number | null;
+  precipitation_days: number | null;
   uv_index_max: number | null;
   wind: ParcelAnalysisWind;
   source: string;
@@ -120,7 +121,8 @@ export interface SeasonStats {
   max_t_c: number;
   min_t_c: number;
   avg_precip_per_day_mm: number;
-  total_precip_mm: number;
+  /** #2464: null — данных по осадкам нет, это НЕ «ноль миллиметров». */
+  total_precip_mm: number | null;
   days_observed: number;
 }