Skip to content

Surface ComfoConnect alarms in Home Assistant#151

Open
PrzemyslawKlys wants to merge 3 commits into
michaelarnauts:masterfrom
EvotecIT:pr/alarm-notifications
Open

Surface ComfoConnect alarms in Home Assistant#151
PrzemyslawKlys wants to merge 3 commits into
michaelarnauts:masterfrom
EvotecIT:pr/alarm-notifications

Conversation

@PrzemyslawKlys

Copy link
Copy Markdown

Summary

  • surface ComfoConnect alarm notifications in Home Assistant instead of only writing them to the log
  • add a diagnostic problem binary sensor with active alarm details in attributes
  • fix the fan platform import for aiocomfoconnect==0.1.15 by using AioComfoConnectNotConnected
  • keep alarm callback details out of warning-level logs now that Home Assistant has a user-facing notification

Details

The integration already receives structured alarm callbacks from aiocomfoconnect, but the callback only logged the active errors. This adds a persistent notification, a comfoconnect_alarm event, and an Active alarms diagnostic binary sensor so alarms are visible in Home Assistant.

A reset/recheck state reported as alarm 100 is shown as ComfoConnect is checking alarms; active faults are shown as ComfoConnect needs attention.

This also fixes a setup failure in current installs where fan.py imports ComfoConnectNotConnected, which is not exported by aiocomfoconnect==0.1.15.

Validation

  • python -m ruff check custom_components
  • python -m ruff format --check custom_components
  • python -m compileall custom_components\comfoconnect
  • tested on an AERISnext 450 R VV: active ODA temperature/humidity sensor alarms appeared in the Home Assistant notification panel; reset briefly showed the recheck state, then the active faults returned

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant