Доработка логики и дополнительные проверки

Author: saippuakauppias

CHANGELOG.md

@@ -45,9 +45,53 @@
   не останавливается ровно на пороге и не дребезжит вокруг него.
 - `source_url` — блюпринт можно импортировать по ссылке и обновлять из HACS.
 
+### Исправлено во втором раунде ревизии
+
+- **Перепутанные местами минимум и максимум тока поднимали потолок.** При
+  «минимум 80, максимум 6» блюпринт выдавал 80 А. Теперь при перепутанных
+  полях побеждает максимум: опустить пол безопасно, поднять потолок — нет.
+- **Потолок, не кратный шагу, был недостижим.** При максимуме 28 А и шаге 5 А
+  мы просили 28, станция округляла по-своему, и уставка переписывалась
+  на каждом пересчёте. Потолок теперь опускается до кратного шагу.
+- **Правильный счётчик сессии отвергался при маленькой цели.** Проверка
+  «не больше трёх целей» рвалась на цели в 2 кВт·ч, и зарядка не
+  останавливалась по достигнутой цели. Порог привязан к ёмкости батареи.
+- **Режим «доработать начатое» жёг максимум весь день.** В сочетании
+  с аварийной дозарядкой и выключенным «гасить в конце окна» ток держался
+  на 28 А до самой цели. Горизонты для этих двух режимов разведены.
+- **Watchdog молча отключался без сенсора мощности**, хотя описание обещало
+  откат на статус станции.
+- **Станция, не принимающая команду включения**, получала её на каждом
+  пересчёте, а хук старта слал уведомление каждые полчаса.
+- **Пустое значение режима зарядки** уходило в станцию как пустая опция:
+  `continue_on_error` глушил ошибку, и режим не выставлялся никогда.
+- **Нулевая зона нечувствительности** означала «писать всегда», а не «писать
+  при любом отличии».
+- **Триггер приезда машины** не знал про откат на идентификатор зоны, хотя
+  сама проверка местоположения его уже использовала.
+- Мелочи: в журнале больше не пишется «н/д (none)»; описания троттлинга,
+  watchdog и режима «доработать начатое» приведены в соответствие с кодом.
+
+### Изменено
+
+- **В зарядку, начатую вручную, автоматизация больше не вмешивается.** Раньше
+  она подбирала для неё ток, а гасить в конце окна отказывалась. Теперь при
+  заполненном «флаге сессии» чужая сессия не трогается совсем: ни уставка,
+  ни выключение. Исключение — неисправность станции. Кабель, воткнутый заранее
+  при выключенной зарядке, чужой сессией не считается и запускается по обычному
+  расписанию. Без флага сессии поведение прежнее.
+
+### Добавлено во втором раунде
+
+- Параметр **«Максимальный возраст показания температуры»** (по умолчанию
+  выключен) — защита от залипшего сенсора для режима холодной погоды.
+
 ### Тесты
 
-- 412 тестов вместо 243, 23 мутации вместо 16.
+- 474 теста вместо 243, 32 мутации вместо 16.
+- `tests/test_actions.py` — блок `actions:` теперь исполняется тестами:
+  состав и порядок команд, троттлинг, хуки, сброс уставки, шаблон журнала.
+  Раньше эта часть не проверялась вообще.
 - Тестовый движок приведён в соответствие с Home Assistant: `is_number`
   отвергает `inf` / `nan`, `states['']` бросает ошибку, `round(0)` возвращает
   целое. Раньше движок был мягче реального HA и мог скрыть регрессию.

CONTRIBUTING.md

@@ -8,7 +8,7 @@
 
 ## Обязательные требования к правкам
 
-1. **Тесты должны проходить.** `pytest` — 412 тестов, полный прогон около 3.5 минут.
+1. **Тесты должны проходить.** `pytest` — 474 теста, полный прогон около 7 минут.
 2. **Новая переменная — новый тест.** Как минимум нормальный случай и границы.
 3. **Порядок объявления переменных имеет значение.** Home Assistant рендерит
    `variables:` сверху вниз; ссылка на переменную, объявленную ниже, молча даст

README.md

@@ -53,7 +53,7 @@
 - **Работа без связи с машиной**: при недоступных данных о заряде автоматизация
   не падает, а продолжает заряжать по запасному плану.
 - **Троттлинг команд** — станции, плохо переносящие частые команды, получают
-  не больше одной команды в заданный интервал.
+  не больше одной команды каждого вида в заданный интервал.
 - **Аварийная дозарядка** ниже критического процента вне окна — максимальным
   током и с запасом над порогом, чтобы не остановиться ровно на нём.
 - **Режим холодной погоды**: ниже заданной температуры растяжка отключается.
@@ -66,12 +66,15 @@
 
 ```
 нужно_кВтч   = (цель % − текущий %) / 100 × ёмкость / КПД
-осталось_ч   = до конца окна − резерв          (переход через полночь учтён;
-                                               вне окна растяжки нет)
+осталось_ч   = до конца окна − резерв          (переход через полночь учтён)
 ток          = нужно_кВтч × 1000 / осталось_ч / (U × число фаз)
 ток          = округлить вверх до шага, зажать в [минимум, максимум]
 ```
 
+Вне окна зарядка идёт по двум разным поводам, и горизонт для них разный:
+аварийная дозарядка спешит и берёт максимум, а «доработать начатое» после
+конца окна доливает мягко, чтобы не создавать пик в дневном тарифе.
+
 Ключевое — это замкнутая обратная связь. Если машина берёт меньше, чем задано,
 на следующем тике процент окажется ниже плана: `нужно_кВтч` останется прежним,
 а `осталось_ч` уменьшится, и ток вырастет. Тот же механизм компенсирует неточность
@@ -209,6 +212,20 @@ https://github.com/saippuakauppias/ha-ev-smart-charging/blob/main/blueprints/aut
 Аналогично с геолокацией: потеря позиции запрещает старт новой сессии, но не рвёт
 уже идущую. Это отличает «трекер замолчал» от «машина уехала».
 
+### Ручная зарядка
+
+Если заполнить поле «Флаг сессии», автоматизация начнёт отличать свою зарядку
+от начатой вручную и **в чужую не вмешивается вовсе**: не подбирает ток и не
+гасит её в конце окна. Включили сами — значит и ток выставили какой хотели.
+Исключение одно: при неисправности станции зарядка прекращается в любом случае.
+
+Воткнутый заранее кабель при выключенной зарядке чужой сессией не считается —
+это обычный способ поставить машину ждать ночи, и в окне она начнёт заряжаться
+своим порядком.
+
+Без этого флага отличить одно от другого нечем: Home Assistant не запоминает,
+кто щёлкнул выключателем.
+
 Тот же принцип применён к статусу станции. Если сенсор статуса стал нечитаемым
 (`unknown` / `unavailable`), новая сессия **не начинается** — отличить
 подключённый кабель от пустой розетки нечем, а включение вслепую вдобавок
@@ -220,6 +237,7 @@ https://github.com/saippuakauppias/ha-ev-smart-charging/blob/main/blueprints/aut
 
 | Ситуация | Что было бы | Что делает блюпринт |
 |---|---|---|
+| Минимальный и максимальный ток перепутаны местами | Ток выше расчётного для проводки | Побеждает максимум: поднять потолок нельзя, опустить пол можно |
 | В поле энергии сессии указан накопительный счётчик | «Уже залито 4500 кВт·ч» → цель считается достигнутой | Считает сенсор недоступным и заряжает по резервному плану |
 | Сняты все галочки дней недели | Окно не откроется никогда | Трактует как «все дни» |
 | Сенсор напряжения врёт (мВ вместо В) | Ток падает до минимума на всю ночь | Отбрасывает значения вне 175–280 В |
@@ -296,8 +314,8 @@ https://github.com/saippuakauppias/ha-ev-smart-charging/blob/main/blueprints/aut
 
 ```bash
 pip install -r requirements-dev.txt
-pytest -m "not slow"    # 388 тестов, около 5 секунд
-pytest -m slow          # мутационное тестирование, около 3.5 минут
+pytest -m "not slow"    # 441 тест, около 10 секунд
+pytest -m slow          # мутационное тестирование, около 7 минут
 ```
 
 | Файл | Что проверяет |
@@ -312,6 +330,7 @@ pytest -m slow          # мутационное тестирование, ок
 | `test_throttle_and_alarms.py` | Троттлинг команд, зона нечувствительности, контроль исправности |
 | `test_triggers.py` | Шаблонные триггеры, в том числе на пустом наборе сущностей |
 | `test_night_simulation.py` | Сквозная симуляция ночи в замкнутом контуре, включая приезд и отъезд машины, неисправность, потерю связи, мороз и watchdog |
+| `test_actions.py` | Команды, уходящие станции: состав, порядок, троттлинг, хуки |
 | `test_mutations.py` | Проверка самих тестов: ловят ли они намеренно сломанный блюпринт |
 
 Последний файл самый ценный. Он моделирует батарею и прогоняет регулятор
@@ -325,7 +344,7 @@ pytest -m slow          # мутационное тестирование, ок
 
 Зелёный набор тестов сам по себе ничего не доказывает — тесты бывают
 бессодержательными. `test_mutations.py` намеренно ломает блюпринт по одному месту
-за раз (23 мутации, каждая соответствует обещанию из этого README) и убеждается,
+за раз (32 мутации, каждая соответствует обещанию из этого README) и убеждается,
 что остальной набор это замечает. Выжившая мутация означает, что соответствующее
 поведение не покрыто.