maximunited / imou_life / 26684280811

90%

push

github

fix: auto-cleanup orphan devices on entry setup (#60)

* fix: auto-cleanup orphan devices on entry setup

When a config entry is deleted and re-added, the old device registry
entry (keyed by the previous entry_id) was left behind as an orphan.
This adds cleanup at the start of async_setup_entry that removes any
device whose (DOMAIN, entry_id) identifier no longer matches a live
config entry. Rate-limited or temporarily unavailable devices are safe
since their config entries still exist.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

* fix: narrow exception handling and add tests for orphan cleanup

Address CodeRabbit and Qodo review feedback:
- Catch (KeyError, AttributeError) instead of broad Exception
- Log actual error details instead of generic message
- Add 6 unit tests covering orphan removal, active device
  retention, foreign device safety, and error handling

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Sonnet 4.5 <noreply@anthropic.com>

15 of 15 new or added lines in 1 file covered. (100.0%)

1616 of 1779 relevant lines covered (90.84%)

3.63 hits per line