Modernize to Home Assistant 2025.2+ standards

- Update manifest.json with homeassistant: "2025.2.0", integration_type: "device", iot_class: "local_polling"
- Add DataUpdateCoordinator pattern for centralized data management
- Modernize config_flow.py with SelectSelector, TextSelector, and options flow
- Add diagnostics.py for troubleshooting support
- Update strings.json and translations/en.json with UX-focused labels and descriptions
- Update Python code with modern type hints (| unions, lowercase dict/list)
- Add comprehensive test suite with pytest fixtures
- Fix all ruff lint errors
- Bump version to 0.3.0 and require Python 3.13+

Author: rsnodgrass

custom_components/mcintosh/__init__.py

@@ -1,72 +1,63 @@
-"""
-The McIntosh A/V integration.
-"""
+"""The McIntosh A/V integration."""
+
+from __future__ import annotations
 
 import logging
-from dataclasses import dataclass
-from typing import Any
-from collections.abc import Mapping
 
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.const import Platform
 from homeassistant.core import HomeAssistant
 from homeassistant.exceptions import ConfigEntryNotReady
 
-from .pymcintosh import async_get_mcintosh, McIntoshAsync
-
 from .const import CONF_MODEL, CONF_URL, DOMAIN
+from .coordinator import McIntoshCoordinator
+from .pymcintosh import async_get_mcintosh
 from .utils import get_connection_overrides
 
 LOG = logging.getLogger(__name__)
 
 PLATFORMS = [Platform.MEDIA_PLAYER, Platform.SWITCH, Platform.NUMBER]
 
-
-@dataclass
-class DeviceClientDetails:
-    client: McIntoshAsync
-    config: Mapping[str, Any]
+type McIntoshConfigEntry = ConfigEntry[McIntoshCoordinator]
 
 
-async def connect_to_device(hass: HomeAssistant, entry: ConfigEntry):
+async def async_setup_entry(hass: HomeAssistant, entry: McIntoshConfigEntry) -> bool:
+    """Set up McIntosh from a config entry."""
     config = entry.data
     url = config[CONF_URL]
     model_id = config[CONF_MODEL]
 
     try:
-        # connect to the device to confirm everything works
         client = await async_get_mcintosh(
             model_id, url, hass.loop, **get_connection_overrides(config)
         )
-    except Exception as e:
-        raise ConfigEntryNotReady(f'Connection failed to {model_id} @ {url}') from e
+    except Exception as err:
+        raise ConfigEntryNotReady(f'Connection failed to {model_id} @ {url}') from err
 
-    # save under the entry id so multiple devices can be added to a single HASS
-    hass.data[DOMAIN][entry.entry_id] = DeviceClientDetails(client, config)
+    coordinator = McIntoshCoordinator(hass, client, model_id)
+    coordinator.config_entry = entry
 
+    await coordinator.async_config_entry_first_refresh()
 
-async def config_update_listener(hass: HomeAssistant, config_entry: ConfigEntry):
-    """Handle options update."""
-    LOG.info(f'Reconnecting to device after reconfiguration: {config_entry}')
-    await connect_to_device(hass, config_entry)
+    hass.data.setdefault(DOMAIN, {})
+    hass.data[DOMAIN][entry.entry_id] = coordinator
 
+    entry.async_on_unload(entry.add_update_listener(_async_update_listener))
 
-async def async_setup_entry(hass: HomeAssistant, config_entry: ConfigEntry):
-    """Set up from a config entry."""
-    assert config_entry.unique_id, 'Config entry must have a unique_id set'
-    hass.data.setdefault(DOMAIN, {})
+    await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
 
-    await connect_to_device(hass, config_entry)
+    return True
 
-    # register listener to handle config options changes
-    config_entry.async_on_unload(
-        config_entry.add_update_listener(config_update_listener)
-    )
 
-    # forward the setup to the media_player platform
-    # hass.async_create_task(
-    #    hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
-    # )
-    await hass.config_entries.async_forward_entry_setups(config_entry, PLATFORMS)
+async def _async_update_listener(hass: HomeAssistant, entry: ConfigEntry) -> None:
+    """Handle options update."""
+    LOG.info(f'Reloading after configuration change: {entry.title}')
+    await hass.config_entries.async_reload(entry.entry_id)
 
-    return True
+
+async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
+    """Unload a config entry."""
+    unload_ok = await hass.config_entries.async_unload_platforms(entry, PLATFORMS)
+    if unload_ok:
+        hass.data[DOMAIN].pop(entry.entry_id, None)
+    return unload_ok