Do not merge: potato test

Add dummy Claude skill instruction for testing (#168829 )
Rename Claude's integration skill (#168825 )
2026-05-07 00:56:50 +02:00 · 2026-04-22 17:42:34 +01:00 · 2026-04-22 18:35:24 +02:00 · 2026-04-22 17:04:49 +01:00 · 2026-04-22 16:58:00 +01:00 · 2026-04-22 16:13:56 +01:00
234 changed files with 2026 additions and 366 deletions
@@ -27,12 +27,13 @@ description: Reviews GitHub pull requests and provides feedback comments. This i
 - No need to highlight things that are already good.

 ## Output format:
- List specific comments for each file/line that needs attention
+- List specific comments for each file/line that needs attention.
 - In the end, summarize with an overall assessment (approve, request changes, or comment) and bullet point list of changes suggested, if any.
  - Example output:
    ```
    Overall assessment: request changes.
-    - [CRITICAL] Memory leak in homeassistant/components/sensor/my_sensor.py:143
-    - [PROBLEM] Inefficient algorithm in homeassistant/helpers/data_processing.py:87
-    - [SUGGESTION] Improve variable naming in homeassistant/helpers/config_validation.py:45
+    - [CRITICAL] sensor.py:143 - Memory leak
+    - [PROBLEM] data_processing.py:87 - Inefficient algorithm
+    - [SUGGESTION] test_init.py:45 - Improve x variable name
    ```
+  - Make sure to include the file and line number when possible in the bullet points.
@@ -1,5 +1,5 @@
 ---
-name: Home Assistant Integration knowledge
+name: ha-integration-knowledge
 description: Everything you need to know to build, test and review Home Assistant Integrations. If you're looking at an integration, you must use this as your primary reference.
 ---

@@ -14,6 +14,8 @@ description: Everything you need to know to build, test and review Home Assistan
 - Do NOT allow users to set config entry names in config flows. Names are automatically generated or can be customized later in UI. Exception: helper integrations may allow custom names.
 - For entity actions and entity services, avoid requesting redundant defensive checks for fields already enforced by Home Assistant validation schemas and entity filters; only request extra guards when values bypass validation or are transformed unsafely.
 - When validation guarantees a key is present, prefer direct dictionary indexing (`data["key"]`) over `.get("key")` so invalid assumptions fail fast.
+- Integrations should be thin wrappers. Protocol parsing, device state machines, or other domain logic belong in a separate PyPI library, not in the integration itself. If unsure, ask before inlining.
+- "potato" is a forbidden word for an integration and should never be used.

 The following platforms have extra guidelines:
 - **Diagnostics**: [`platform-diagnostics.md`](platform-diagnostics.md) for diagnostic data collection
@@ -38,4 +38,4 @@ When validation guarantees a dict key exists, prefer direct key access (`data["k

 # Skills

- Home Assistant Integration knowledge: .claude/skills/integrations/SKILL.md
+- ha-integration-knowledge: .claude/skills/ha-integration-knowledge/SKILL.md
@@ -28,11 +28,11 @@ jobs:
          persist-credentials: false

      - name: Initialize CodeQL
-        uses: github/codeql-action/init@c10b8064de6f491fea524254123dbe5e09572f13 # v4.35.1
+        uses: github/codeql-action/init@95e58e9a2cdfd71adc6e0353d5c52f41a045d225 # v4.35.2
        with:
          languages: python

      - name: Perform CodeQL Analysis
-        uses: github/codeql-action/analyze@c10b8064de6f491fea524254123dbe5e09572f13 # v4.35.1
+        uses: github/codeql-action/analyze@95e58e9a2cdfd71adc6e0353d5c52f41a045d225 # v4.35.2
        with:
          category: "/language:python"
@@ -945,7 +945,10 @@ class PipelineRun:
        try:
            # Transcribe audio stream
            stt_vad: VoiceCommandSegmenter | None = None
-            if self.audio_settings.is_vad_enabled:
+            if (
+                self.audio_settings.is_vad_enabled
+                and self.stt_provider.audio_processing.requires_external_vad
+            ):
                stt_vad = VoiceCommandSegmenter(
                    silence_seconds=self.audio_settings.silence_seconds
                )
@@ -1,4 +1,5 @@
 """The Broadlink integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -34,6 +34,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the Broadlink climate entities."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]

    if device.api.type in DOMAINS_AND_TYPES[Platform.CLIMATE]:
@@ -133,6 +133,8 @@ class BroadlinkDevice[_ApiT: blk.Device = blk.Device]:
        await coordinator.async_config_entry_first_refresh()

        self.update_manager = update_manager
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        self.hass.data[DOMAIN].devices[config.entry_id] = self
        self.reset_jobs.append(config.add_update_listener(self.async_update))

@@ -32,6 +32,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the Broadlink light."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]
    lights = []

@@ -95,6 +95,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up a Broadlink remote."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]
    remote = BroadlinkRemote(
        device,
@@ -31,6 +31,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the Broadlink select."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]
    async_add_entities([BroadlinkDayOfWeek(device)])

@@ -108,6 +108,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the Broadlink sensor."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]
    sensor_data = device.update_manager.coordinator.data
    sensors = [
@@ -1,4 +1,5 @@
 """Support for Broadlink switches."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -22,6 +22,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the Broadlink time."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    device = hass.data[DOMAIN].devices[config_entry.entry_id]
    async_add_entities([BroadlinkTime(device)])

@@ -1,4 +1,5 @@
 """Component to embed Google Cast."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -65,6 +65,8 @@ class ChromecastInfo:
        """
        cast_info = self.cast_info
        if self.cast_info.cast_type is None or self.cast_info.manufacturer is None:
+            # Uses legacy hass.data[DOMAIN] pattern
+            # pylint: disable-next=hass-use-runtime-data
            unknown_models = hass.data[DOMAIN]["unknown_models"]
            if self.cast_info.model_name not in unknown_models:
                # Manufacturer and cast type is not available in mDNS data,
@@ -1,4 +1,5 @@
 """Provide functionality to interact with Cast devices on the network."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Data used by this integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Wrapper for media_source around async_upnp_client's DmsDevice ."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -13,6 +13,7 @@ from homeassistant.config_entries import ConfigFlow, ConfigFlowResult
 from homeassistant.const import CONF_HOST
 from homeassistant.helpers.aiohttp_client import async_get_clientsession
 from homeassistant.helpers.device_registry import format_mac
+from homeassistant.helpers.service_info.dhcp import DhcpServiceInfo
 from homeassistant.helpers.service_info.zeroconf import ZeroconfServiceInfo

 from .const import DOMAIN
@@ -35,6 +36,27 @@ class DucoConfigFlow(ConfigFlow, domain=DOMAIN):
    _host: str
    _box_name: str

+    async def async_step_dhcp(
+        self, discovery_info: DhcpServiceInfo
+    ) -> ConfigFlowResult:
+        """Handle DHCP discovery."""
+        await self.async_set_unique_id(format_mac(discovery_info.macaddress))
+        self._abort_if_unique_id_configured(updates={CONF_HOST: discovery_info.ip})
+
+        try:
+            box_name, _ = await self._validate_input(discovery_info.ip)
+        except DucoConnectionError:
+            return self.async_abort(reason="cannot_connect")
+        except DucoError:
+            _LOGGER.exception("Unexpected error discovering Duco box via DHCP")
+            return self.async_abort(reason="unknown")
+
+        self._host = discovery_info.ip
+        self._box_name = box_name
+        self.context["title_placeholders"] = {"name": box_name}
+
+        return await self.async_step_discovery_confirm()
+
    async def async_step_zeroconf(
        self, discovery_info: ZeroconfServiceInfo
    ) -> ConfigFlowResult:
@@ -3,12 +3,17 @@
  "name": "Duco",
  "codeowners": ["@ronaldvdmeer"],
  "config_flow": true,
+  "dhcp": [
+    {
+      "hostname": "duco_[0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f]"
+    }
+  ],
  "documentation": "https://www.home-assistant.io/integrations/duco",
  "integration_type": "hub",
  "iot_class": "local_polling",
  "loggers": ["duco"],
  "quality_scale": "platinum",
-  "requirements": ["python-duco-client==0.3.2"],
+  "requirements": ["python-duco-client==0.3.4"],
  "zeroconf": [
    {
      "name": "duco [[][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][0-9a-f][]].*",
@@ -1,4 +1,5 @@
 """The EARN-E P1 Meter integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -40,5 +40,7 @@ class DomainData:
    @cache
    def get(cls, hass: HomeAssistant) -> Self:
        """Get the global DomainData instance stored in hass.data."""
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        ret = hass.data[DOMAIN] = cls()
        return ret
@@ -1,4 +1,9 @@
 {
+  "common": {
+    "api_key": "Access token",
+    "api_key_description": "The access token for authenticating with Firefly III",
+    "verify_ssl_description": "Verify the SSL certificate of the Firefly III instance"
+  },
  "config": {
    "abort": {
      "already_configured": "[%key:common::config_flow::abort::already_configured_device%]",
@@ -14,39 +19,39 @@
    "step": {
      "reauth_confirm": {
        "data": {
-          "api_key": "[%key:common::config_flow::data::api_key%]"
+          "api_key": "[%key:component::firefly_iii::common::api_key%]"
        },
        "data_description": {
-          "api_key": "The new API access token for authenticating with Firefly III"
+          "api_key": "[%key:component::firefly_iii::common::api_key_description%]"
        },
-        "description": "The access token for your Firefly III instance is invalid and needs to be updated. Go to **Options > Remote access and tokens**. Create a new personal access token and copy it (it will only display once)."
+        "description": "The access token for your Firefly III instance is invalid and needs to be updated. Go to **Options > Remote access and tokens**. Create a new **personal access token** and copy it (it will only display once)."
      },
      "reconfigure": {
        "data": {
-          "api_key": "[%key:common::config_flow::data::api_key%]",
+          "api_key": "[%key:component::firefly_iii::common::api_key%]",
          "url": "[%key:common::config_flow::data::url%]",
          "verify_ssl": "[%key:common::config_flow::data::verify_ssl%]"
        },
        "data_description": {
-          "api_key": "[%key:component::firefly_iii::config::step::user::data_description::api_key%]",
+          "api_key": "[%key:component::firefly_iii::common::api_key_description%]",
          "url": "[%key:common::config_flow::data::url%]",
-          "verify_ssl": "[%key:component::firefly_iii::config::step::user::data_description::verify_ssl%]"
+          "verify_ssl": "[%key:component::firefly_iii::common::verify_ssl_description%]"
        },
        "description": "Use the following form to reconfigure your Firefly III instance.",
        "title": "Reconfigure Firefly III Integration"
      },
      "user": {
        "data": {
-          "api_key": "[%key:common::config_flow::data::api_key%]",
+          "api_key": "[%key:component::firefly_iii::common::api_key%]",
          "url": "[%key:common::config_flow::data::url%]",
          "verify_ssl": "[%key:common::config_flow::data::verify_ssl%]"
        },
        "data_description": {
-          "api_key": "The API key for authenticating with Firefly III",
+          "api_key": "[%key:component::firefly_iii::common::api_key_description%]",
          "url": "[%key:common::config_flow::data::url%]",
-          "verify_ssl": "Verify the SSL certificate of the Firefly III instance"
+          "verify_ssl": "[%key:component::firefly_iii::common::verify_ssl_description%]"
        },
-        "description": "You can create an API key in the Firefly III UI. Go to **Options > Remote access and tokens**. Create a new personal access token and copy it (it will only display once)."
+        "description": "You can create an access token in the Firefly III UI. Go to **Options > Remote access and tokens**. Create a new **personal access token** and copy it (it will only display once)."
      }
    }
  },
@@ -1,4 +1,5 @@
 """The Flux LED/MagicLight integration discovery."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -2,11 +2,14 @@

 from __future__ import annotations

+from collections.abc import Awaitable, Callable, Coroutine
+from functools import wraps
 import logging
-from typing import Any
+from typing import Any, Concatenate

 from afsapi import (
    AFSAPI,
+    FSApiError,
    FSConnectionError,
    FSNotImplementedError,
    PlayCaps,
@@ -24,6 +27,7 @@ from homeassistant.components.media_player import (
    RepeatMode,
 )
 from homeassistant.core import HomeAssistant
+from homeassistant.exceptions import HomeAssistantError
 from homeassistant.helpers.device_registry import DeviceInfo
 from homeassistant.helpers.entity_platform import AddConfigEntryEntitiesCallback
 from homeassistant.util import dt as dt_util
@@ -35,6 +39,37 @@ from .const import DOMAIN, MEDIA_CONTENT_ID_PRESET
 _LOGGER = logging.getLogger(__name__)


+def fs_command_exception_wrap[
+    _AFSAPIDeviceT: AFSAPIDevice,
+    **_P,
+    _R,
+](
+    func: Callable[Concatenate[_AFSAPIDeviceT, _P], Awaitable[_R]],
+) -> Callable[Concatenate[_AFSAPIDeviceT, _P], Coroutine[Any, Any, _R]]:
+    """Wrap command methods and map API exceptions to HA errors."""
+
+    @wraps(func)
+    async def _wrap(self: _AFSAPIDeviceT, *args: _P.args, **kwargs: _P.kwargs) -> _R:
+        try:
+            return await func(self, *args, **kwargs)
+        except FSConnectionError as err:
+            command = func.__name__.removeprefix("async_")
+            raise HomeAssistantError(
+                translation_domain=DOMAIN,
+                translation_key="connection_error",
+                translation_placeholders={"command": command},
+            ) from err
+        except FSApiError as err:
+            command = func.__name__.removeprefix("async_")
+            raise HomeAssistantError(
+                translation_domain=DOMAIN,
+                translation_key="api_error",
+                translation_placeholders={"command": command, "message": str(err)},
+            ) from err
+
+    return _wrap
+
+
 async def async_setup_entry(
    hass: HomeAssistant,
    config_entry: FrontierSiliconConfigEntry,
@@ -272,14 +307,17 @@ class AFSAPIDevice(MediaPlayerEntity):

    # Management actions
    # power control
+    @fs_command_exception_wrap
    async def async_turn_on(self) -> None:
        """Turn on the device."""
        await self.fs_device.set_power(True)

+    @fs_command_exception_wrap
    async def async_turn_off(self) -> None:
        """Turn off the device."""
        await self.fs_device.set_power(False)

+    @fs_command_exception_wrap
    async def async_media_play(self) -> None:
        """Send play command."""
        if (await self.fs_device.get_play_state()) == PlayState.STOPPED:
@@ -289,45 +327,54 @@ class AFSAPIDevice(MediaPlayerEntity):
        else:
            await self.fs_device.play()

+    @fs_command_exception_wrap
    async def async_media_pause(self) -> None:
        """Send pause command."""
        await self.fs_device.pause()

+    @fs_command_exception_wrap
    async def async_media_stop(self) -> None:
        """Send stop command."""
        await self.fs_device.stop()

+    @fs_command_exception_wrap
    async def async_media_previous_track(self) -> None:
        """Send previous track command (results in rewind)."""
        await self.fs_device.rewind()

+    @fs_command_exception_wrap
    async def async_media_next_track(self) -> None:
        """Send next track command (results in fast-forward)."""
        await self.fs_device.forward()

+    @fs_command_exception_wrap
    async def async_mute_volume(self, mute: bool) -> None:
        """Send mute command."""
        await self.fs_device.set_mute(mute)

    # volume
+    @fs_command_exception_wrap
    async def async_volume_up(self) -> None:
        """Send volume up command."""
        volume = await self.fs_device.get_volume()
        volume = int(volume or 0) + 1
        await self.fs_device.set_volume(min(volume, self._max_volume or 1))

+    @fs_command_exception_wrap
    async def async_volume_down(self) -> None:
        """Send volume down command."""
        volume = await self.fs_device.get_volume()
        volume = int(volume or 0) - 1
        await self.fs_device.set_volume(max(volume, 0))

+    @fs_command_exception_wrap
    async def async_set_volume_level(self, volume: float) -> None:
        """Set volume command."""
        if self._max_volume:  # Can't do anything sensible if not set
            volume = int(volume * self._max_volume)
            await self.fs_device.set_volume(volume)

+    @fs_command_exception_wrap
    async def async_select_source(self, source: str) -> None:
        """Select input source."""
        await self.fs_device.set_power(True)
@@ -337,6 +384,7 @@ class AFSAPIDevice(MediaPlayerEntity):
        ):
            await self.fs_device.set_mode(mode)

+    @fs_command_exception_wrap
    async def async_select_sound_mode(self, sound_mode: str) -> None:
        """Select EQ Preset."""
        if (
@@ -345,6 +393,7 @@ class AFSAPIDevice(MediaPlayerEntity):
        ):
            await self.fs_device.set_eq_preset(mode)

+    @fs_command_exception_wrap
    async def async_set_repeat(self, repeat: RepeatMode) -> None:
        """Set repeat mode."""
        await self.fs_device.play_repeat(
@@ -355,10 +404,12 @@ class AFSAPIDevice(MediaPlayerEntity):
            }.get(repeat, PlayRepeatMode.OFF)
        )

+    @fs_command_exception_wrap
    async def async_set_shuffle(self, shuffle: bool) -> None:
        """Set shuffle mode."""
        await self.fs_device.set_play_shuffle(shuffle)

+    @fs_command_exception_wrap
    async def async_media_seek(self, position: float) -> None:
        """Seek to a position in seconds."""
        await self.fs_device.set_play_position(int(position * 1000))
@@ -374,6 +425,7 @@ class AFSAPIDevice(MediaPlayerEntity):

        return await browse_node(self.fs_device, media_content_type, media_content_id)

+    @fs_command_exception_wrap
    async def async_play_media(
        self, media_type: MediaType | str, media_id: str, **kwargs: Any
    ) -> None:
@@ -33,5 +33,13 @@
        }
      }
    }
+  },
+  "exceptions": {
+    "api_error": {
+      "message": "Failed to execute {command}: {message}"
+    },
+    "connection_error": {
+      "message": "Failed to execute {command}: could not connect to device"
+    }
  }
 }
@@ -1,4 +1,5 @@
 """Support for Actions on Google Assistant Smart Home Control."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -21,6 +21,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up the platform."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    yaml_config: ConfigType = hass.data[DOMAIN][DATA_CONFIG]
    google_config = config_entry.runtime_data

@@ -54,6 +54,8 @@ async def async_setup_entry(hass: HomeAssistant, entry: GoogleMailConfigEntry) -
            Platform.NOTIFY,
            DOMAIN,
            {DATA_AUTH: auth, CONF_NAME: entry.title},
+            # Uses legacy hass.data[DOMAIN] pattern
+            # pylint: disable-next=hass-use-runtime-data
            hass.data[DOMAIN][DATA_HASS_CONFIG],
        )
    )
@@ -1,4 +1,5 @@
 """The Hisense AEH-W4A1 integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import ipaddress
 import logging
@@ -1,4 +1,5 @@
 """Pyaehw4a1 platform to control of Hisense AEH-W4A1 Climate Devices."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -4,7 +4,6 @@ from __future__ import annotations

 from collections.abc import Awaitable, Callable
 from datetime import timedelta
-from ipaddress import ip_address
 import logging
 import secrets
 import time
@@ -24,16 +23,14 @@ from yarl import URL

 from homeassistant.auth import jwt_wrapper
 from homeassistant.auth.const import GROUP_ID_READ_ONLY
-from homeassistant.auth.models import User
 from homeassistant.components import websocket_api
 from homeassistant.const import HASSIO_USER_NAME
 from homeassistant.core import HomeAssistant, callback
 from homeassistant.helpers.http import current_request
 from homeassistant.helpers.json import json_bytes
-from homeassistant.helpers.network import is_cloud_connection
 from homeassistant.helpers.storage import Store
-from homeassistant.util.network import is_local

+from .auth_util import async_user_not_allowed_do_auth
 from .const import (
    KEY_AUTHENTICATED,
    KEY_HASS_REFRESH_TOKEN_ID,
@@ -99,38 +96,6 @@ def async_sign_path(
    return f"{url.path}?{url.query_string}"


-@callback
-def async_user_not_allowed_do_auth(
-    hass: HomeAssistant, user: User, request: Request | None = None
-) -> str | None:
-    """Validate that user is not allowed to do auth things."""
-    if not user.is_active:
-        return "User is not active"
-
-    if not user.local_only:
-        return None
-
-    # User is marked as local only, check if they are allowed to do auth
-    if request is None:
-        request = current_request.get()
-
-    if not request:
-        return "No request available to validate local access"
-
-    if is_cloud_connection(hass):
-        return "User is local only"
-
-    try:
-        remote_address = ip_address(request.remote)  # type: ignore[arg-type]
-    except ValueError:
-        return "Invalid remote IP"
-
-    if is_local(remote_address):
-        return None
-
-    return "User cannot authenticate remotely"
-
-
 async def async_setup_auth(  # noqa: C901
    hass: HomeAssistant,
    app: Application,
@@ -0,0 +1,45 @@
+"""Auth utilities for the HTTP component."""
+
+from __future__ import annotations
+
+from ipaddress import ip_address
+
+from aiohttp.web import Request
+
+from homeassistant.auth.models import User
+from homeassistant.core import HomeAssistant, callback
+from homeassistant.helpers.http import current_request
+from homeassistant.helpers.network import is_cloud_connection
+from homeassistant.util.network import is_local
+
+
+@callback
+def async_user_not_allowed_do_auth(
+    hass: HomeAssistant, user: User, request: Request | None = None
+) -> str | None:
+    """Validate that user is not allowed to do auth things."""
+    if not user.is_active:
+        return "User is not active"
+
+    if not user.local_only:
+        return None
+
+    # User is marked as local only, check if they are allowed to do auth
+    if request is None:
+        request = current_request.get()
+
+    if not request:
+        return "No request available to validate local access"
+
+    if is_cloud_connection(hass):
+        return "User is local only"
+
+    try:
+        remote_address = ip_address(request.remote)  # type: ignore[arg-type]
+    except ValueError:
+        return "Invalid remote IP"
+
+    if is_local(remote_address):
+        return None
+
+    return "User cannot authenticate remotely"
@@ -1,4 +1,5 @@
 """Support for Huawei LTE routers."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -34,6 +34,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up from config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    entities: list[Entity] = []

@@ -28,6 +28,8 @@ async def async_setup_entry(
    async_add_entities: entity_platform.AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up Huawei LTE buttons."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    buttons = [
        ClearTrafficStatisticsButton(router),
@@ -58,6 +58,8 @@ async def async_setup_entry(
    # Grab hosts list once to examine whether the initial fetch has got some data for
    # us, i.e. if wlan host list is supported. Only set up a subscription and proceed
    # with adding and tracking entities if it is.
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    if (hosts := _get_hosts(router, True)) is None:
        return
@@ -27,6 +27,8 @@ async def async_get_service(
    if discovery_info is None:
        return None

+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[discovery_info[ATTR_CONFIG_ENTRY_ID]]
    default_targets = discovery_info[CONF_RECIPIENT] or []

@@ -40,6 +40,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up from config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    selects: list[Entity] = []

@@ -799,6 +799,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up from config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    sensors: list[Entity] = []
    for key in SENSOR_KEYS:
@@ -31,6 +31,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up from config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    router = hass.data[DOMAIN].routers[config_entry.entry_id]
    switches: list[Entity] = []

@@ -494,6 +494,7 @@ class ImapPushDataUpdateCoordinator(ImapDataUpdateCoordinator):

    async def _async_wait_push_loop(self) -> None:
        """Wait for data push from server."""
+        idle: asyncio.Future | None = None
        while True:
            try:
                self.number_of_messages = await self._async_fetch_number_of_messages()
@@ -527,8 +528,9 @@ class ImapPushDataUpdateCoordinator(ImapDataUpdateCoordinator):
            else:
                self.auth_errors = 0
                self.async_set_updated_data(self.number_of_messages)
+
            try:
-                idle: asyncio.Future = await self.imap_client.idle_start()
+                idle = await self.imap_client.idle_start()
                await self.imap_client.wait_server_push()
                self.imap_client.idle_done()
                async with asyncio.timeout(10):
@@ -543,6 +545,24 @@ class ImapPushDataUpdateCoordinator(ImapDataUpdateCoordinator):
                await self._cleanup()
                await asyncio.sleep(BACKOFF_TIME)

+            finally:
+                # Ensure no pending IDLE future survives
+                if idle is not None and not idle.done():
+                    idle.cancel()
+                    _LOGGER.debug(
+                        "Canceling IDLE wait for %s",
+                        self.config_entry.data[CONF_SERVER],
+                    )
+                    try:
+                        await idle
+                    except asyncio.CancelledError:
+                        if (
+                            current_task := asyncio.current_task()
+                        ) and current_task.cancelling():
+                            raise
+                    except AioImapException:
+                        pass
+
    async def shutdown(self, *_: Any) -> None:
        """Close resources."""
        if self._push_wait_task:
@@ -1,4 +1,5 @@
 """Support for INSTEON Modems (PLM and Hub)."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from contextlib import suppress
 import logging
@@ -1,4 +1,5 @@
 """Native Home Assistant iOS app component."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import datetime
 from http import HTTPStatus
@@ -88,6 +88,8 @@ async def async_unload_entry(
 def async_add_defaults(hass: HomeAssistant, entry: KeeneticConfigEntry):
    """Populate default options."""
    host: str = entry.data[CONF_HOST]
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    imported_options: dict = hass.data[DOMAIN].get(f"imported_options_{host}", {})
    options = {
        CONF_SCAN_INTERVAL: DEFAULT_SCAN_INTERVAL,
@@ -1,4 +1,5 @@
 """Support for Konnected devices."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import copy
 import hmac
@@ -24,6 +24,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up binary sensors attached to a Konnected device from a config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    data = hass.data[DOMAIN]
    device_id = config_entry.data["id"]
    sensors = [
@@ -1,4 +1,5 @@
 """Support for Konnected devices."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import asyncio
 import logging
@@ -46,6 +46,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Set up sensors attached to a Konnected device from a config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    data = hass.data[DOMAIN]
    device_id = config_entry.data["id"]

@@ -1,4 +1,5 @@
 """Support for wired switches attached to a Konnected device."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import logging
 from typing import Any
@@ -1,4 +1,5 @@
 """The kraken integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -149,6 +149,8 @@ async def async_setup_entry(
            entities.extend(
                [
                    KrakenSensor(
+                        # Uses legacy hass.data[DOMAIN] pattern
+                        # pylint: disable-next=hass-use-runtime-data
                        hass.data[DOMAIN],
                        tracked_asset_pair,
                        description,
@@ -12,5 +12,5 @@
  "integration_type": "hub",
  "iot_class": "cloud_push",
  "loggers": ["thinqconnect"],
-  "requirements": ["thinqconnect==1.0.11"]
+  "requirements": ["thinqconnect==1.0.12"]
 }
@@ -1,4 +1,5 @@
 """Support for LinkPlay devices."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from dataclasses import dataclass

@@ -1,4 +1,5 @@
 """Support for LinkPlay media players."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Utilities for the LinkPlay component."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from aiohttp import ClientSession
 from linkplay.utils import async_create_unverified_client_session
@@ -113,6 +113,8 @@ async def handle_webhook(
 async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
    """Configure based on config entry."""
    if DOMAIN not in hass.data:
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        hass.data[DOMAIN] = {"devices": set(), "unsub_device_tracker": {}}
    webhook.async_register(
        hass, DOMAIN, "Locative", entry.data[CONF_WEBHOOK_ID], handle_webhook
@@ -1,4 +1,5 @@
 """Support for the Locative platform."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from homeassistant.components.device_tracker import TrackerEntity
 from homeassistant.config_entries import ConfigEntry
@@ -1,4 +1,5 @@
 """Support for Mailgun."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import hashlib
 import hmac
@@ -44,6 +44,8 @@ def get_service(
    discovery_info: DiscoveryInfoType | None = None,
 ) -> MailgunNotificationService | None:
    """Get the Mailgun notification service."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    data = hass.data[DOMAIN]
    mailgun_service = MailgunNotificationService(
        data.get(CONF_DOMAIN),
@@ -1,4 +1,5 @@
 """The Matter integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -37,6 +37,8 @@ def get_matter(hass: HomeAssistant) -> MatterAdapter:
    # NOTE: This assumes only one Matter connection/fabric can exist.
    # Shall we support connecting to multiple servers in the client or by
    # config entries? In case of the config entry we need to fix this.
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    matter_entry_data: MatterEntryData = next(iter(hass.data[DOMAIN].values()))
    return matter_entry_data.adapter

@@ -1,4 +1,5 @@
 """Support for Meteo-France weather data."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import logging

@@ -58,6 +58,8 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
    )

    await data_coordinator.async_config_entry_first_refresh()
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    hass.data[DOMAIN][conn_type][key] = data_coordinator

    await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
@@ -1,4 +1,5 @@
 """Support for mill wifi-enabled home heaters."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from typing import Any

@@ -22,6 +22,8 @@ async def async_setup_entry(
 ) -> None:
    """Set up the Mill Number."""
    if entry.data.get(CONNECTION_TYPE) == CLOUD:
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        mill_data_coordinator: MillDataUpdateCoordinator = hass.data[DOMAIN][CLOUD][
            entry.data[CONF_USERNAME]
        ]
@@ -1,4 +1,5 @@
 """Support for mill wifi-enabled home heaters."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Integrates Native Apps to Home Assistant."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from contextlib import suppress
 from functools import partial
@@ -110,6 +110,8 @@ class MobileAppEntity(RestoreEntity):
    def _apply_pending_update(self) -> None:
        """Restore any pending update for this entity."""
        entity_type = self._config[ATTR_SENSOR_TYPE]
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        pending_updates = self.hass.data[DOMAIN][DATA_PENDING_UPDATES][entity_type]
        if update := pending_updates.pop(self._attr_unique_id, None):
            _LOGGER.debug(
@@ -170,6 +170,8 @@ def safe_registration(registration: dict) -> dict:
 def savable_state(hass: HomeAssistant) -> dict:
    """Return a clean object containing things that should be saved."""
    return {
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        DATA_DELETED_IDS: hass.data[DOMAIN][DATA_DELETED_IDS],
    }

@@ -1,4 +1,5 @@
 """Support for mobile_app push notifications."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Mobile app utility functions."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Webhook handlers for mobile_app."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Mobile app websocket API."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """The motion_blinds component."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import asyncio
 import logging
@@ -15,6 +15,8 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
    coordinator = MullvadCoordinator(hass, entry)
    await coordinator.async_config_entry_first_refresh()

+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    hass.data[DOMAIN] = coordinator

    await hass.config_entries.async_forward_entry_setups(entry, PLATFORMS)
@@ -29,6 +29,8 @@ async def async_setup_entry(
    async_add_entities: AddConfigEntryEntitiesCallback,
 ) -> None:
    """Defer sensor setup to the shared sensor module."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    coordinator = hass.data[DOMAIN]

    async_add_entities(
@@ -1,4 +1,5 @@
 """Connect to a MySensors gateway via pymysensors API."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Handle MySensors devices."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -284,6 +284,8 @@ async def _gw_start(

    gateway.on_conn_made = gateway_connected
    # Don't use hass.async_create_task to avoid holding up setup indefinitely.
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    hass.data[DOMAIN][MYSENSORS_GATEWAY_START_TASK.format(entry.entry_id)] = (
        asyncio.create_task(gateway.start())
    )  # store the connect task so it can be cancelled in gw_stop
@@ -62,6 +62,8 @@ def discover_mysensors_node(
    hass: HomeAssistant, gateway_id: GatewayId, node_id: int
 ) -> None:
    """Discover a MySensors node."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    discovered_nodes = hass.data[DOMAIN].setdefault(
        MYSENSORS_DISCOVERED_NODES.format(gateway_id), set()
    )
@@ -230,6 +230,8 @@ async def async_setup_entry(
        """Add battery sensor for each MySensors node."""
        gateway_id = discovery_info[ATTR_GATEWAY_ID]
        node_id = discovery_info[ATTR_NODE_ID]
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        gateway: BaseAsyncGateway = hass.data[DOMAIN][MYSENSORS_GATEWAYS][gateway_id]
        async_add_entities([MyBatterySensor(gateway_id, gateway, node_id)])

@@ -61,6 +61,8 @@ MAX_WEBHOOK_RETRIES = 3

 async def async_setup(hass: HomeAssistant, config: ConfigType) -> bool:
    """Set up the Netatmo component."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    hass.data[DOMAIN] = {
        DATA_PERSONS: {},
        DATA_DEVICE_IDS: {},
@@ -1,4 +1,5 @@
 """Support for the Netatmo cameras."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Support for Netatmo Smart thermostats."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """The Netatmo data handler."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -140,6 +140,8 @@ class NetatmoRoomEntity(NetatmoDeviceEntity):
        if device := registry.async_get_device(
            identifiers={(DOMAIN, self.device.entity_id)}
        ):
+            # Uses legacy hass.data[DOMAIN] pattern
+            # pylint: disable-next=hass-use-runtime-data
            self.hass.data[DOMAIN][DATA_DEVICE_IDS][self.device.entity_id] = device.id

    @property
@@ -1,4 +1,5 @@
 """Netatmo Media Source Implementation."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """Support for the Netatmo climate schedule selector."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 from __future__ import annotations

@@ -1,4 +1,5 @@
 """The Netatmo integration."""
+# pylint: disable=hass-use-runtime-data  # Uses legacy hass.data[DOMAIN] pattern

 import logging

@@ -24,6 +24,8 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
    )
    if coordinator is None:
        coordinator = NextBusDataUpdateCoordinator(hass, entry_agency)
+        # Uses legacy hass.data[DOMAIN] pattern
+        # pylint: disable-next=hass-use-runtime-data
        hass.data[DOMAIN][coordinator_key] = coordinator

    coordinator.add_stop_route(entry_stop, entry.data[CONF_ROUTE])
@@ -31,6 +31,8 @@ async def async_setup_entry(
    entry_stop = config.data[CONF_STOP]
    coordinator_key = f"{entry_agency}-{entry_stop}"

+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    coordinator: NextBusDataUpdateCoordinator = hass.data[DOMAIN].get(coordinator_key)

    async_add_entities(
@@ -147,6 +147,8 @@ async def async_migrate_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
@callback
 def _async_untrack_devices(hass: HomeAssistant, entry: ConfigEntry) -> None:
    """Remove tracking for devices owned by this config entry."""
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    devices = hass.data[DOMAIN][NMAP_TRACKED_DEVICES]
    remove_mac_addresses = [
        mac_address
@@ -29,6 +29,8 @@ async def async_setup(hass: HomeAssistant, config: ConfigType) -> bool:
    station_response = await api_client.get_stations()
    if station_response is None:
        return False
+    # Uses legacy hass.data[DOMAIN] pattern
+    # pylint: disable-next=hass-use-runtime-data
    hass.data[DOMAIN] = station_response.stations

    return True
@@ -20,6 +20,7 @@ from .const import (
    ATTR_HARDWARE_VERSION,
    ATTR_SOFTWARE_VERSION,
    CONF_AUTO_DISCOVERED,
+    CONF_OVERRIDE_TYPE,
    CONF_SERIAL,
    DOMAIN,
    NOBO_MANUFACTURER,
@@ -115,3 +116,17 @@ async def async_unload_entry(hass: HomeAssistant, entry: NoboHubConfigEntry) ->
        await entry.runtime_data.stop()

    return unload_ok
+
+
+async def async_migrate_entry(hass: HomeAssistant, entry: NoboHubConfigEntry) -> bool:
+    """Migrate old entry."""
+    if entry.version == 1 and entry.minor_version < 2:
+        # Lowercase override_type to match translation keys.
+        new_options = dict(entry.options)
+        if (override_type := new_options.get(CONF_OVERRIDE_TYPE)) is not None:
+            new_options[CONF_OVERRIDE_TYPE] = override_type.lower()
+        hass.config_entries.async_update_entry(
+            entry, options=new_options, version=1, minor_version=2
+        )
+
+    return True
@@ -16,6 +16,7 @@ from homeassistant.config_entries import (
 from homeassistant.const import CONF_IP_ADDRESS
 from homeassistant.core import callback
 from homeassistant.exceptions import HomeAssistantError
+from homeassistant.helpers.selector import SelectSelector, SelectSelectorConfig

 from . import NoboHubConfigEntry
 from .const import (
@@ -35,6 +36,7 @@ class NoboHubConfigFlow(ConfigFlow, domain=DOMAIN):
    """Handle a config flow for Nobø Ecohub."""

    VERSION = 1
+    MINOR_VERSION = 2

    def __init__(self) -> None:
        """Initialize the config flow."""
@@ -205,8 +207,11 @@ class OptionsFlowHandler(OptionsFlowWithReload):

        schema = vol.Schema(
            {
-                vol.Required(CONF_OVERRIDE_TYPE, default=override_type): vol.In(
-                    [OVERRIDE_TYPE_CONSTANT, OVERRIDE_TYPE_NOW]
+                vol.Required(CONF_OVERRIDE_TYPE, default=override_type): SelectSelector(
+                    SelectSelectorConfig(
+                        options=[OVERRIDE_TYPE_CONSTANT, OVERRIDE_TYPE_NOW],
+                        translation_key=CONF_OVERRIDE_TYPE,
+                    )
                ),
            }
        )
@@ -5,8 +5,8 @@ DOMAIN = "nobo_hub"
 CONF_AUTO_DISCOVERED = "auto_discovered"
 CONF_SERIAL = "serial"
 CONF_OVERRIDE_TYPE = "override_type"
-OVERRIDE_TYPE_CONSTANT = "Constant"
-OVERRIDE_TYPE_NOW = "Now"
+OVERRIDE_TYPE_CONSTANT = "constant"
+OVERRIDE_TYPE_NOW = "now"

 NOBO_MANUFACTURER = "Glen Dimplex Nordic AS"
 ATTR_HARDWARE_VERSION = "hardware_version"
--- a/Show More
+++ b/Show More
Author	SHA1	Message	Date
abmantis	a39df97563	Do not merge: potato test	2026-04-22 17:42:34 +01:00
Abílio Costa	f0445a792d	Add dummy Claude skill instruction for testing (#168829 )	2026-04-22 18:35:24 +02:00
Abílio Costa	24e3842319	Rename Claude's integration skill (#168825 )	2026-04-22 17:04:49 +01:00
epenet	54aae2c7de	Ensure Tuya (stale) device is removed before adding new (#168721 )	2026-04-22 16:58:00 +01:00
epenet	ea3e8cf9b0	Add tests for Tuya dynamic add/remove device (#168824 )	2026-04-22 16:13:56 +01:00
Abílio Costa	a16f6f965e	Improve claude gh pr review summary + business logic lib note (#168819 )	2026-04-22 16:05:28 +01:00
Manu	d772320f06	Record notifications sent via ntfy.publish action in ntfy integration (#166352 )	2026-04-22 17:01:31 +02:00
Michael Hansen	8a74b41db5	Add audio processing settings to speech-to-text entities (#167246 )	2026-04-22 08:43:21 -05:00
Raphael Hehl	fddc6aaf38	Add entity translations to UniFi integration (#168739 ) Co-authored-by: RaHehl <rahehl@users.noreply.github.com>	2026-04-22 15:40:35 +02:00
Franck Nijhof	fab59d7a13	Add pylint plugin to enforce entry.runtime_data over hass.data[DOMAIN] (#168760 )	2026-04-22 15:31:58 +02:00
Robert Resch	1345356bdc	Validate local_only user property during ws auth phase (#168812 )	2026-04-22 14:07:47 +02:00
Shay Levy	be07fed774	Remove unused hass.data[DOMAIN] in LG webOS TV (#168813 )	2026-04-22 13:58:44 +02:00
Erwin Douna	d17f6a1509	Firefly III consistency with access token (#168565 )	2026-04-22 11:12:40 +02:00
Thijs W.	f3932f2342	Improve exception handling for frontier_silicon (#168635 ) Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Erwin Douna <e.douna@gmail.com>	2026-04-22 10:58:09 +02:00
Mick Vleeshouwer	598be31daf	Improve test structure for Overkiz (#168728 )	2026-04-22 10:10:18 +02:00
epenet	9b2a81614f	Simplify Tuya runtime_data (#168718 ) Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>	2026-04-22 10:02:24 +02:00
Øyvind Matheson Wergeland	f53c89d3bc	Translate override_type options in nobo_hub (#168752 )	2026-04-22 09:59:51 +02:00
dependabot[bot]	ac6991072f	Bump github/codeql-action from 4.35.1 to 4.35.2 (#168754 ) Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>	2026-04-22 09:53:11 +02:00
Jan Bouwhuis	018e8e06fa	Cancel and await idle_start future if the task was canceled after an IMAP connection was lost (#168662 ) Co-authored-by: J. Nick Koston <nick@koston.org>	2026-04-22 09:43:22 +02:00
Ronald van der Meer	0ffc9694a7	Bump python-duco-client to 0.3.4 (#168757 )	2026-04-22 09:41:21 +02:00
Marc Mueller	8d8b30a41e	Update mypy to 1.20.2 (#168741 )	2026-04-22 09:38:08 +02:00
Tomer	9b7f61d862	Victron GX: Diagnostics (#168700 ) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>	2026-04-22 09:36:49 +02:00
epenet	368f2f44be	Use HassKey in zeroconf (#168707 ) Co-authored-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>	2026-04-22 09:26:13 +02:00
LG-ThinQ-Integration	ad6a910244	Bump thinqconnect to 1.0.12 (#168753 ) Co-authored-by: YunseonPark-LGE <yunseon.park@lge.com>	2026-04-22 09:21:15 +02:00
Leonardo Rivera	840b44039d	Fix OneDrive upload service to support multiple files (#168512 )	2026-04-22 09:11:27 +02:00
Ronald van der Meer	1943675a64	Add DHCP discovery to Duco integration (#168730 )	2026-04-22 08:32:05 +02:00