wesandClaude Opus 4.8
34155fd7f9
The OneControl panel's command characteristic is a streaming (Write Without Response) endpoint. Bleak 3.x changed write_gatt_char to default to write-with-response when the char advertises the "write" property, so every command (incl. switch turn_on/off) got rejected by the panel with ATT 0x0E (Unlikely Error), surfaced as BleakGATTProtocolError. Force response=False on the command write (matching the auth key write) to restore control. Also commits the productionized custom_components integration (config flow, coordinator, switch/sensor/cover entities, key-seed TEA auth, COBS codec) and the matching src/ RE client/COBS fixes (big-endian framing, table-driven CRC8, status-event decoding) that were developed but never tracked. Verified live on the campsite HAOS Pi: switch.exterior_lights / interior_lights toggle the physical panel with no GATT error. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
41 lines
1.5 KiB
Python
41 lines
1.5 KiB
Python
from homeassistant.components.switch import SwitchEntity
|
|
from homeassistant.config_entries import ConfigEntry
|
|
from homeassistant.core import HomeAssistant
|
|
from homeassistant.helpers.entity_platform import AddEntitiesCallback
|
|
from homeassistant.helpers.update_coordinator import CoordinatorEntity
|
|
|
|
from .coordinator import OneControlCoordinator
|
|
from .const import DOMAIN, SWITCH_DEVICES
|
|
|
|
|
|
async def async_setup_entry(
|
|
hass: HomeAssistant, entry: ConfigEntry, async_add_entities: AddEntitiesCallback
|
|
) -> None:
|
|
coordinator: OneControlCoordinator = hass.data[DOMAIN][entry.entry_id]
|
|
async_add_entities(
|
|
OneControlSwitch(coordinator, device_id, name)
|
|
for device_id, name in SWITCH_DEVICES.items()
|
|
)
|
|
|
|
|
|
class OneControlSwitch(CoordinatorEntity, SwitchEntity):
|
|
def __init__(self, coordinator: OneControlCoordinator, device_id: int, name: str) -> None:
|
|
super().__init__(coordinator)
|
|
self._device_id = device_id
|
|
self._attr_name = name
|
|
self._attr_unique_id = f"{coordinator.address}_{device_id}"
|
|
|
|
@property
|
|
def is_on(self) -> bool | None:
|
|
return self.coordinator.get_switch_state(self._device_id)
|
|
|
|
@property
|
|
def available(self) -> bool:
|
|
return self.coordinator.is_connected
|
|
|
|
async def async_turn_on(self, **kwargs) -> None:
|
|
await self.coordinator.async_set_switch(self._device_id, True)
|
|
|
|
async def async_turn_off(self, **kwargs) -> None:
|
|
await self.coordinator.async_set_switch(self._device_id, False)
|