fix: source_list HTML entity decoding and optional excluded_sources filter

Resolves #1

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: cmos486

README.md

@@ -72,6 +72,32 @@ Before installing, make sure your TV is configured:
   <img src="https://raw.githubusercontent.com/cmos486/Bravia-REST-API/main/images/integration_installed.jpg" alt="Integration installed in Home Assistant" width="600"/>
 </p>
 
+### Excluded Sources
+
+Optionally hide specific apps or inputs from the source list dropdown. Excluded sources
+are still launchable via the `bravia_rest_api.open_app` service and remain visible in the
+`installed_apps` attribute.
+
+**Option A: UI (Options Flow)**
+
+1. Go to **Settings > Devices & Services > Bravia REST API**
+2. Click **Configure**
+3. Select the sources you want to exclude from the dropdown
+4. Click **Submit**
+
+> If the TV is off when you open the options, a text field is shown instead — enter one source name per line.
+
+**Option B: YAML**
+
+```yaml
+bravia_rest_api:
+  excluded_sources:
+    - "Timers & Clock"
+    - "Eco Dashboard"
+```
+
+> If both UI and YAML are configured, the UI selection takes priority.
+
 ---
 
 ## 🎮 Entities Created

custom_components/bravia_rest_api/__init__.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import html
 import logging
 from typing import Any
 
@@ -14,7 +15,15 @@
 from homeassistant.helpers.aiohttp_client import async_get_clientsession
 
 from .bravia_client import BraviaClient, BraviaError
-from .const import CONF_PSK, DOMAIN, IRCC_CODES, POWER_SAVING_OFF, POWER_SAVING_PICTURE_OFF
+from .const import (
+    CONF_EXCLUDED_SOURCES,
+    CONF_PSK,
+    DOMAIN,
+    IRCC_CODES,
+    POWER_SAVING_OFF,
+    POWER_SAVING_PICTURE_OFF,
+    YAML_CONFIG_KEY,
+)
 from .coordinator import BraviaCoordinator
 
 _LOGGER = logging.getLogger(__name__)
@@ -73,6 +82,26 @@
     }
 )
 
+CONFIG_SCHEMA = vol.Schema(
+    {
+        DOMAIN: vol.Schema(
+            {
+                vol.Optional(CONF_EXCLUDED_SOURCES, default=[]): vol.All(
+                    cv.ensure_list, [cv.string]
+                ),
+            }
+        )
+    },
+    extra=vol.ALLOW_EXTRA,
+)
+
+
+async def async_setup(hass: HomeAssistant, config: dict[str, Any]) -> bool:
+    """Store YAML configuration if present."""
+    if DOMAIN in config:
+        hass.data[YAML_CONFIG_KEY] = config[DOMAIN]
+    return True
+
 
 def _get_coordinator(hass: HomeAssistant, entity_id: str) -> BraviaCoordinator | None:
     """Find the coordinator for a given entity_id."""
@@ -98,13 +127,25 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
 
     await hass.config_entries.async_forward_entry_setups(entry, PLATFORM_LIST)
 
+    # Refresh entities when options change (e.g. excluded_sources)
+    entry.async_on_unload(entry.add_update_listener(_async_update_listener))
+
     # Register services (only once, on first entry)
     if not hass.services.has_service(DOMAIN, SERVICE_OPEN_APP):
         _register_services(hass)
 
     return True
 
 
+async def _async_update_listener(
+    hass: HomeAssistant, entry: ConfigEntry
+) -> None:
+    """Handle options update without restarting the coordinator."""
+    coordinator = hass.data[DOMAIN].get(entry.entry_id)
+    if coordinator:
+        coordinator.async_set_updated_data(coordinator.data)
+
+
 async def async_unload_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     """Unload a config entry."""
     if unload_ok := await hass.config_entries.async_unload_platforms(
@@ -144,19 +185,19 @@ async def handle_open_app(call: ServiceCall) -> None:
             return
 
         if not app_uri and app_name:
-            # Look up by name (case-insensitive)
+            # Look up by name (case-insensitive, HTML-decoded)
             name_lower = app_name.lower()
             if coordinator.data and coordinator.data.app_list:
                 for app in coordinator.data.app_list:
-                    if app.get("title", "").lower() == name_lower:
+                    if html.unescape(app.get("title", "")).lower() == name_lower:
                         app_uri = app.get("uri")
                         break
             if not app_uri:
                 _LOGGER.error(
                     "App '%s' not found. Available: %s",
                     app_name,
                     ", ".join(
-                        a.get("title", "?")
+                        html.unescape(a.get("title", "?"))
                         for a in (coordinator.data.app_list if coordinator.data else [])
                     ),
                 )

custom_components/bravia_rest_api/config_flow.py

@@ -2,6 +2,7 @@
 
 from __future__ import annotations
 
+import html
 import logging
 from typing import Any
 from urllib.parse import urlparse
@@ -10,17 +11,25 @@
 import voluptuous as vol
 
 from homeassistant.components import ssdp
-from homeassistant.config_entries import ConfigFlow, ConfigFlowResult
+from homeassistant.config_entries import (
+    ConfigEntry,
+    ConfigFlow,
+    ConfigFlowResult,
+    OptionsFlow,
+)
 from homeassistant.const import CONF_HOST
+from homeassistant.core import callback
+from homeassistant.helpers import config_validation as cv
 from homeassistant.helpers.aiohttp_client import async_get_clientsession
+from homeassistant.helpers.selector import TextSelector, TextSelectorConfig
 
 from .bravia_client import (
     BraviaAuthError,
     BraviaClient,
     BraviaConnectionError,
     BraviaError,
 )
-from .const import CONF_MAC, CONF_PSK, DOMAIN
+from .const import CONF_EXCLUDED_SOURCES, CONF_MAC, CONF_PSK, DOMAIN
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -37,6 +46,12 @@ class BraviaRestApiConfigFlow(ConfigFlow, domain=DOMAIN):
 
     VERSION = 1
 
+    @staticmethod
+    @callback
+    def async_get_options_flow(config_entry: ConfigEntry) -> OptionsFlow:
+        """Create the options flow."""
+        return BraviaRestApiOptionsFlow(config_entry)
+
     def __init__(self) -> None:
         """Initialize the config flow."""
         self._discovered_host: str | None = None
@@ -151,3 +166,74 @@ async def async_step_ssdp(
         self.context["title_placeholders"] = {"name": model or host}
 
         return await self.async_step_user()
+
+
+class BraviaRestApiOptionsFlow(OptionsFlow):
+    """Handle options for Bravia REST API."""
+
+    def __init__(self, config_entry: ConfigEntry) -> None:
+        """Initialize options flow."""
+        self.config_entry = config_entry
+
+    async def async_step_init(
+        self, user_input: dict[str, Any] | None = None
+    ) -> ConfigFlowResult:
+        """Manage the options."""
+        if user_input is not None:
+            excluded = user_input.get(CONF_EXCLUDED_SOURCES, [])
+            if isinstance(excluded, str):
+                # Text area fallback: split by newlines
+                excluded = [s.strip() for s in excluded.splitlines() if s.strip()]
+            return self.async_create_entry(
+                title="", data={CONF_EXCLUDED_SOURCES: excluded}
+            )
+
+        # Build available sources from coordinator
+        coordinator = self.hass.data.get(DOMAIN, {}).get(
+            self.config_entry.entry_id
+        )
+        sources: list[str] = []
+        if coordinator and coordinator.data:
+            data = coordinator.data
+            for inp in data.external_inputs:
+                custom_label = inp.get("label", "")
+                label = inp.get("title", "")
+                name = custom_label if custom_label else label
+                if name:
+                    sources.append(name)
+            for app in data.app_list:
+                title = html.unescape(app.get("title", ""))
+                if title:
+                    sources.append(title)
+
+        current_excluded = self.config_entry.options.get(
+            CONF_EXCLUDED_SOURCES, []
+        )
+
+        if sources:
+            # Multi-select with current sources
+            source_lower_map = {s.lower(): s for s in sources}
+            valid_default = [
+                source_lower_map[e.lower()]
+                for e in current_excluded
+                if e.lower() in source_lower_map
+            ]
+            schema = vol.Schema(
+                {
+                    vol.Optional(
+                        CONF_EXCLUDED_SOURCES, default=valid_default
+                    ): cv.multi_select({s: s for s in sorted(sources)}),
+                }
+            )
+        else:
+            # TV off fallback: multiline text input
+            schema = vol.Schema(
+                {
+                    vol.Optional(
+                        CONF_EXCLUDED_SOURCES,
+                        default="\n".join(current_excluded),
+                    ): TextSelector(TextSelectorConfig(multiline=True)),
+                }
+            )
+
+        return self.async_show_form(step_id="init", data_schema=schema)

custom_components/bravia_rest_api/const.py

@@ -6,6 +6,9 @@
 
 CONF_PSK: Final = "psk"
 CONF_MAC: Final = "mac"
+CONF_EXCLUDED_SOURCES: Final = "excluded_sources"
+
+YAML_CONFIG_KEY: Final = f"{DOMAIN}_yaml"
 
 DEFAULT_PORT: Final = 80
 DEFAULT_SCAN_INTERVAL: Final = 15  # seconds

custom_components/bravia_rest_api/manifest.json

@@ -7,7 +7,7 @@
     "issue_tracker": "https://github.com/cmos486/Bravia-REST-API/issues",
     "integration_type": "device",
     "iot_class": "local_polling",
-    "version": "1.3.1",
+    "version": "1.3.2",
     "requirements": [],
     "homeassistant": "2024.1.0",
     "ssdp": [

custom_components/bravia_rest_api/media_player.py

@@ -3,6 +3,7 @@
 from __future__ import annotations
 
 import asyncio
+import html
 import logging
 import socket
 from typing import Any
@@ -18,7 +19,7 @@
 from homeassistant.helpers.entity_platform import AddEntitiesCallback
 
 from .bravia_client import BraviaError
-from .const import CONF_MAC, DOMAIN, WOL_PORT
+from .const import CONF_EXCLUDED_SOURCES, CONF_MAC, DOMAIN, WOL_PORT, YAML_CONFIG_KEY
 from .coordinator import BraviaCoordinator
 from .entity import BraviaEntity
 
@@ -96,7 +97,7 @@ def _build_source_map(self) -> None:
                 self._sources[name] = {"uri": uri, "type": SOURCE_TYPE_INPUT}
 
         for app in data.app_list:
-            title = app.get("title", "")
+            title = html.unescape(app.get("title", ""))
             uri = app.get("uri", "")
             if title and uri:
                 self._sources[title] = {"uri": uri, "type": SOURCE_TYPE_APP}
@@ -141,13 +142,24 @@ def source(self) -> str | None:
             if src["uri"] == uri:
                 return name
 
-        return title or uri or None
+        return html.unescape(title) if title else uri or None
+
+    def _get_excluded_sources(self) -> set[str]:
+        """Get excluded sources (case-insensitive). Options flow wins over YAML."""
+        excluded = self._entry.options.get(CONF_EXCLUDED_SOURCES)
+        if excluded is None:
+            yaml_cfg = self.hass.data.get(YAML_CONFIG_KEY, {})
+            excluded = yaml_cfg.get(CONF_EXCLUDED_SOURCES, [])
+        return {s.lower() for s in excluded}
 
     @property
     def source_list(self) -> list[str]:
         """Return the list of available sources."""
         self._build_source_map()
-        return list(self._sources.keys())
+        excluded = self._get_excluded_sources()
+        if not excluded:
+            return list(self._sources.keys())
+        return [name for name in self._sources if name.lower() not in excluded]
 
     @property
     def media_title(self) -> str | None:
@@ -164,7 +176,7 @@ def extra_state_attributes(self) -> dict[str, Any]:
         data = self.coordinator.data
         if data and data.app_list:
             attrs["installed_apps"] = {
-                app.get("title", "Unknown"): app.get("uri", "")
+                html.unescape(app.get("title", "Unknown")): app.get("uri", "")
                 for app in data.app_list
                 if app.get("title") and app.get("uri")
             }

custom_components/bravia_rest_api/strings.json

@@ -23,6 +23,20 @@
             "cannot_connect": "Cannot connect to the discovered TV."
         }
     },
+    "options": {
+        "step": {
+            "init": {
+                "title": "Bravia REST API Options",
+                "description": "Configure source list filtering.",
+                "data": {
+                    "excluded_sources": "Excluded Sources"
+                },
+                "data_description": {
+                    "excluded_sources": "Sources to hide from the source list dropdown. If the TV is off, enter one source name per line."
+                }
+            }
+        }
+    },
     "entity": {
         "button": {
             "reboot": {

custom_components/bravia_rest_api/translations/es.json

@@ -23,6 +23,20 @@
             "cannot_connect": "No se puede conectar a la TV detectada."
         }
     },
+    "options": {
+        "step": {
+            "init": {
+                "title": "Opciones Bravia REST API",
+                "description": "Configura el filtrado de la lista de fuentes.",
+                "data": {
+                    "excluded_sources": "Fuentes Excluidas"
+                },
+                "data_description": {
+                    "excluded_sources": "Fuentes a ocultar del desplegable de fuentes. Si la TV esta apagada, introduce un nombre de fuente por linea."
+                }
+            }
+        }
+    },
     "entity": {
         "button": {
             "reboot": {