Refactor services.py
async_setup_services should be run in async_setup, not in async_setup_entry (https://developers.home-assistant.io/docs/core/integration-quality-scale/rules/action-setup/) and async_unload_services seems to have been dropped entirely (https://github.com/home-assistant/core/pull/146048#discussion_r2121194482). Haven't been able to find where the recommendation this comment refers to is, but I do see that `async_unload_services is no longer used anywhere in the core-repo.
Looking at some of the integrations in core that implement async_setup_services, they rarely contain more than hass.services.async_register-calls within the function itself.
Prerequisite: Ensure good test coverage for services before refactoring to avoid introducing regressions.
Main tasks:
- Move call to
async_setup_servicesfromasync_setup_entrytoasync_setup - Move function definitions out of
async_setup_services - Investigate if
iter_objectscan be simplified and/or makes more sense as a function insideZaptecManager - remove
async_unload_services