ha(ux): исправления по ревью — быстрый выход пробинга, prefill шаблона, ниты
- major: probe_template прекращает перебор на unreachable/no_slots/key_error (недоступный модуль: ~10 c вместо ~49 c), продолжает только при properties_missing (доступен, но чужой шаблон); +тест - m1: extra_state_attributes не падает до добавления в hass - m2: reconfigure берёт сохранённый шаблон как дефолт (не пересчитывает по модели); +тест - m3: device_capabilities/display_temperature переведены в необязательные зонды (trial не падает на молчащем caps; feature-модель трактует None как «не режем») - m4: async_sync_now(online_timeout) — честное имя/докстринг - ниты: удалены PRESET_CAP/as_dict/лишние плейсхолдеры, DIRECTION_KEYS упрощён, тавтология в тесте исправлена, en-тире - 59 components + 47 pyfglair/acceptance зелёные
This commit is contained in:
@@ -496,8 +496,6 @@ class FglairConfigFlow(ConfigFlow, domain=DOMAIN):
|
||||
default = str(data.get(CONF_TEMPLATE, default))
|
||||
placeholders = {
|
||||
"device_values": _device_values_text(self._trial_properties),
|
||||
"detected": default,
|
||||
"model": str(data.get(CONF_MODEL) or "-"),
|
||||
}
|
||||
if self._probed:
|
||||
placeholders["probe_note"] = (
|
||||
@@ -641,6 +639,12 @@ class FglairConfigFlow(ConfigFlow, domain=DOMAIN):
|
||||
# текущие конверсии, если путь их не задаёт.
|
||||
for key, value in self._reconfigure_entry.data.items():
|
||||
data.setdefault(key, value)
|
||||
# Сохранённый шаблон важнее пересчёта по модели: пользователь
|
||||
# мог выбрать его вручную (страница подтвердит/изменит).
|
||||
if self._reconfigure_entry.data.get(CONF_TEMPLATE):
|
||||
data[CONF_TEMPLATE] = self._reconfigure_entry.data[
|
||||
CONF_TEMPLATE
|
||||
]
|
||||
await self.async_set_unique_id(data[CONF_DSN], raise_on_progress=False)
|
||||
if self._reconfigure_entry is not None:
|
||||
self._abort_if_unique_id_mismatch()
|
||||
@@ -651,7 +655,11 @@ class FglairConfigFlow(ConfigFlow, domain=DOMAIN):
|
||||
)
|
||||
if result.ok:
|
||||
self._pending_data = data
|
||||
self._pending_data[CONF_TEMPLATE] = template.name
|
||||
if (
|
||||
self._reconfigure_entry is None
|
||||
or not self._reconfigure_entry.data.get(CONF_TEMPLATE)
|
||||
):
|
||||
self._pending_data[CONF_TEMPLATE] = template.name
|
||||
self._trial_properties = result.properties
|
||||
self._trial_answered = result.answered
|
||||
self._resolved_template = template
|
||||
|
||||
@@ -94,11 +94,6 @@ PRESET_PROP = {
|
||||
PRESET_ECO: Prop.ECONOMY_MODE,
|
||||
PRESET_BOOST: Prop.POWERFUL_MODE,
|
||||
}
|
||||
PRESET_CAP = {
|
||||
PRESET_ECO: CAP_ECONOMY,
|
||||
PRESET_BOOST: CAP_POWERFUL,
|
||||
}
|
||||
|
||||
SWING_MODES = ["off", "on"]
|
||||
|
||||
# switch-свойства: prop -> (translation_key, capability bit | None)
|
||||
@@ -218,7 +213,7 @@ ENTITY_DESCRIPTIONS = {
|
||||
"en": {
|
||||
"climate": "Air conditioner control: mode, fan, setpoint, louvers, presets",
|
||||
"room_temperature": "Room air temperature",
|
||||
"error_code": "Air conditioner error code (0 — no errors)",
|
||||
"error_code": "Air conditioner error code (0 = no errors)",
|
||||
"connection_state": "Connection state with the AC module",
|
||||
"connectivity": "Whether the module is reachable",
|
||||
"defrost": "Outdoor unit defrost in progress",
|
||||
|
||||
@@ -123,13 +123,14 @@ class FglairCoordinator(DataUpdateCoordinator[FglairData]):
|
||||
# После разрыва при следующем ONLINE нужна повторная синхронизация.
|
||||
self._synced = False
|
||||
|
||||
async def async_sync_now(self, timeout: float = 2.0) -> None:
|
||||
async def async_sync_now(self, online_timeout: float = 2.0) -> None:
|
||||
"""Гарантирует первичный GET-батч и ожидание ключевых свойств.
|
||||
|
||||
Ждёт ONLINE (ограниченно), затем — завершения синхронизации, чтобы
|
||||
сущности создавались с корректными feature-дефолтами.
|
||||
Ждёт ONLINE не дольше ``online_timeout``, затем — завершения
|
||||
синхронизации (сама синхронизация ограничена своим таймаутом),
|
||||
чтобы сущности создавались с корректными feature-дефолтами.
|
||||
"""
|
||||
deadline = self.hass.loop.time() + timeout
|
||||
deadline = self.hass.loop.time() + online_timeout
|
||||
while (
|
||||
self._sync_task is None
|
||||
and self.client.state != State.ONLINE
|
||||
|
||||
@@ -52,7 +52,7 @@ class FglairEntity(CoordinatorEntity[FglairCoordinator]):
|
||||
def extra_state_attributes(self) -> Optional[dict]:
|
||||
"""HA не поддерживает тултипы сущностей: краткое описание видно в
|
||||
атрибутах (More info → Атрибуты)."""
|
||||
if self._description_key is None:
|
||||
if self._description_key is None or self.hass is None:
|
||||
return None
|
||||
text = entity_description(self.hass.config.language, self._description_key)
|
||||
return {"description": text} if text else None
|
||||
|
||||
@@ -13,11 +13,8 @@ from pyfglair import Prop, Value
|
||||
from .const import FEATURES
|
||||
|
||||
DIRECTION_KEYS = {
|
||||
"af_vertical_direction": ("af_vertical_num_dir", Prop.AF_VERTICAL_NUM_DIR),
|
||||
"af_horizontal_direction": (
|
||||
"af_horizontal_num_dir",
|
||||
Prop.AF_HORIZONTAL_NUM_DIR,
|
||||
),
|
||||
"af_vertical_direction": Prop.AF_VERTICAL_NUM_DIR,
|
||||
"af_horizontal_direction": Prop.AF_HORIZONTAL_NUM_DIR,
|
||||
}
|
||||
|
||||
|
||||
@@ -44,7 +41,7 @@ def device_default(
|
||||
return True # возможностей не сообщили — не режем
|
||||
return bool(caps & (1 << bit))
|
||||
if key in DIRECTION_KEYS:
|
||||
num = _int_value(values, DIRECTION_KEYS[key][1])
|
||||
num = _int_value(values, DIRECTION_KEYS[key])
|
||||
return bool(num and num > 1)
|
||||
if prop is not None:
|
||||
if answered is not None:
|
||||
@@ -113,6 +110,3 @@ class FeatureSet:
|
||||
if key in self._explicit:
|
||||
return self._explicit[key]
|
||||
return device_default(key, self._values)
|
||||
|
||||
def as_dict(self) -> dict[str, bool]:
|
||||
return dict(self._explicit)
|
||||
|
||||
@@ -20,13 +20,13 @@ from pyfglair.templates import template_info
|
||||
REQUIRED_PROPS: tuple[Prop, ...] = (
|
||||
Prop.OPERATION_MODE,
|
||||
Prop.FAN_SPEED,
|
||||
Prop.DEVICE_CAPABILITIES,
|
||||
Prop.DISPLAY_TEMPERATURE,
|
||||
)
|
||||
|
||||
# Зондируемые свойства (в пределах выбранного шаблона): наличие в ответе —
|
||||
# информация для дефолтов фич и распознавания шаблона.
|
||||
OPTIONAL_PROPS: tuple[Prop, ...] = (
|
||||
Prop.DEVICE_CAPABILITIES,
|
||||
Prop.DISPLAY_TEMPERATURE,
|
||||
Prop.AF_VERTICAL_DIRECTION,
|
||||
Prop.AF_HORIZONTAL_DIRECTION,
|
||||
Prop.AF_VERTICAL_NUM_DIR,
|
||||
@@ -244,7 +244,11 @@ def probe_template(
|
||||
props_timeout=2.0,
|
||||
)
|
||||
if not result.ok:
|
||||
continue
|
||||
if result.reason == "properties_missing":
|
||||
# Модуль доступен, но свойств шаблона нет — пробуем дальше.
|
||||
continue
|
||||
# Недоступен/нет слотов/ключ — перебор шаблонов бессмыслен.
|
||||
return None, properties
|
||||
answered |= result.answered
|
||||
properties.update(result.properties)
|
||||
detected = classify_template(answered)
|
||||
|
||||
Reference in New Issue
Block a user