♻️ Drop unused site from coordinator topology tuples #49

Merged
mat merged 1 commit from refactor/drop-site-tuples into main 2026-07-10 18:16:07 +00:00
Owner

Pure, behavior-preserving refactor.

coordinator.sensor_devices / switch_devices stored (site, device) tuples, but every consumer discarded the site (for _site, device in ...). They now hold plain device dicts. Touches coordinator.py, sensor.py, switch.py, __init__.py.

⚠️ Overlaps with #48 in _fetch_all (both edit the switch-append branch). Whichever of #48 / this PR merges second needs a trivial rebase (one 2-line hunk).

  • 32/32 tests pass, ruff clean.
Pure, behavior-preserving refactor. `coordinator.sensor_devices` / `switch_devices` stored `(site, device)` tuples, but every consumer discarded the site (`for _site, device in ...`). They now hold plain device dicts. Touches `coordinator.py`, `sensor.py`, `switch.py`, `__init__.py`. ⚠️ Overlaps with #48 in `_fetch_all` (both edit the switch-append branch). Whichever of #48 / this PR merges **second** needs a trivial rebase (one 2-line hunk). - 32/32 tests pass, ruff clean.
♻️ Drop unused site from coordinator topology tuples
All checks were successful
Validate / lint-ruff (push) Successful in 7s
Validate / test-pytest (push) Successful in 3m0s
Validate / type-check-mypy (push) Successful in 3m7s
Validate / lint-ruff (pull_request) Successful in 7s
Validate / test-pytest (pull_request) Successful in 3m2s
Validate / type-check-mypy (pull_request) Successful in 3m8s
a064ababd0
mat merged commit 17117ff2e5 into main 2026-07-10 18:16:07 +00:00
mat referenced this pull request from a commit 2026-07-13 13:03:58 +00:00
Sign in to join this conversation.
No reviewers
No labels
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
mat/homeassistant-comwatt!49
No description provided.