DS: re-homar el dispositivo "Común" fuera de la primera persiana al actualizar (v0.98.1) (#141)

El dispositivo compartido de persianas (recuentos + sol, SHUTTERS_DEVICE_ID)
existe desde la v0.90.0, cuando vivía anidado bajo la primera persiana. La
v0.98.0 movió sus entidades a la entrada "Común" auto-creada, pero Home Assistant
solo AÑADE entradas de configuración a un dispositivo (nunca poda la vieja), así
que el dispositivo seguía mostrándose dentro de esa persiana en vez de como su
propio hub.

Al configurar la entrada Común se retira ahora cualquier enlace que no sea el
suyo del dispositivo compartido (las entidades de la Común ya están adjuntas, así
que el dispositivo nunca se colapsa y sus entity_ids no cambian). Idempotente:
no hace nada en instalaciones nuevas ni una vez re-homado.

Test de regresión por la vía real de recarga de la entrada, verificando que el
enlace obsoleto se retira y la entidad compartida conserva su entity_id y su
entrada.


Claude-Session: https://claude.ai/code/session_01HjFYRJN939VaxK75qjLgoW

Co-authored-by: Claude <noreply@anthropic.com>

Author: woody-box

CHANGELOG.md

@@ -4,6 +4,21 @@ Todas las versiones notables de la integración `custom_components/dynamic_home`
 Formato basado en [Keep a Changelog](https://keepachangelog.com/es/1.1.0/) y
 [SemVer](https://semver.org/lang/es/).
 
+## [0.98.1] — 2026-07-04
+
+### Fixed
+- **DS · la sección "Dynamic Shutter · Común" seguía anidada dentro de la primera
+  persiana** tras actualizar desde una versión anterior a la v0.98.0. El
+  dispositivo compartido (recuentos + sol) existe desde la v0.90.0, cuando vivía
+  colgado de la primera persiana; la v0.98.0 movió sus entidades a la entrada
+  "Común" auto-creada, pero Home Assistant solo **añade** entradas de
+  configuración a un dispositivo (nunca poda la vieja), así que el dispositivo
+  seguía mostrándose **dentro** de esa persiana en vez de como su propio hub. Al
+  arrancar, la entrada Común ahora **retira el enlace obsoleto** de la persiana y
+  se queda como única dueña del dispositivo. Las entidades no se tocan: conservan
+  su `entity_id`. Las instalaciones nuevas no se ven afectadas (el dispositivo
+  nace ya bajo la Común).
+
 ## [0.98.0] — 2026-07-03
 
 ### Added

custom_components/dynamic_home/__init__.py

@@ -11,6 +11,7 @@
 from homeassistant.config_entries import ConfigEntry
 from homeassistant.core import HomeAssistant, ServiceCall, SupportsResponse
 from homeassistant.helpers import config_validation as cv
+from homeassistant.helpers import device_registry as dr
 from homeassistant.helpers import service as service_helper
 
 from . import const
@@ -68,6 +69,28 @@ async def _ensure_common_entry(hass: HomeAssistant) -> None:
         hass.data[const.DOMAIN].pop("_common_creating", None)
 
 
+def _reassign_common_device(hass: HomeAssistant, entry: ConfigEntry) -> None:
+    """Re-home the shared "Dynamic Shutter · Común" device onto its own entry.
+
+    The shared shutter device (SHUTTERS_DEVICE_ID: counts + sun + note) predates
+    v0.98.0, when it lived nested under the first shutter entry. v0.98.0 moved
+    its entities to the auto-created Común singleton, but Home Assistant only ever
+    *adds* config entries to a device — it never prunes the stale one — so the
+    device keeps the old shutter in its ``config_entries`` and stays displayed
+    *inside* that shutter instead of standing alone as its own hub. Drop every
+    entry that isn't ours; the Común entities are already attached, so the device
+    always keeps this entry and never collapses. Entity links are untouched.
+    Idempotent: a no-op on fresh installs and once already re-homed.
+    """
+    dev_reg = dr.async_get(hass)
+    device = dev_reg.async_get_device(
+        identifiers={(const.DOMAIN, const.SHUTTERS_DEVICE_ID)})
+    if device is None:
+        return
+    for cfg_id in device.config_entries - {entry.entry_id}:
+        dev_reg.async_update_device(device.id, remove_config_entry_id=cfg_id)
+
+
 async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     """Set up Dynamic Home from a config entry."""
     hub: SdhbHub = hass.data.setdefault(const.DOMAIN, {}).setdefault(
@@ -115,6 +138,10 @@ async def async_setup_entry(hass: HomeAssistant, entry: ConfigEntry) -> bool:
     hass.data[const.DOMAIN][entry.entry_id] = coordinator
 
     await hass.config_entries.async_forward_entry_setups(entry, _platforms(entry))
+    # The shared shutter device predates its own entry; on upgrade strip the stale
+    # shutter link so it shows as its own "Común" hub, not nested (v0.98.1).
+    if module == const.MODULE_SHUTTER_COMMON:
+        _reassign_common_device(hass, entry)
     entry.async_on_unload(entry.add_update_listener(_async_options_updated))
     _async_register_services(hass)
     # First shutter -> make sure the shared "Dynamic Shutter · Común" entry exists.

custom_components/dynamic_home/manifest.json

@@ -8,5 +8,5 @@
   "iot_class": "local_polling",
   "issue_tracker": "https://github.com/woody-box/Dynamic-Home/issues",
   "requirements": [],
-  "version": "0.98.0"
+  "version": "0.98.1"
 }

tests/test_ds_integration.py

@@ -1383,6 +1383,55 @@ async def test_common_entry_autocreated_and_removed_with_last_shutter(
                 if e.data.get(const.CONF_MODULE) == const.MODULE_SHUTTER_COMMON]
 
 
+async def test_common_device_rehomed_off_shutter_on_upgrade(
+        hass: HomeAssistant) -> None:
+    # v0.98.1: after upgrading, the shared shutter device (counts + sun) keeps a
+    # stale link to the first shutter in its config_entries, so it stays nested
+    # under that shutter instead of standing alone. The shared entities already
+    # belong to the Común entry (v0.98.0); only the device link is stale. A
+    # reload of the Común entry must strip it without churning any entity.
+    from homeassistant.helpers import device_registry as dr
+    from homeassistant.helpers import entity_registry as er
+    _seed(hass)
+    ea = MockConfigEntry(domain=const.DOMAIN, data={**SHUTTER}, title="A")
+    ea.add_to_hass(hass)
+    # Pre-create the Común as a normal (reloadable) entry so the shutter's
+    # auto-create step finds it and we exercise the real setup path on reload.
+    common = MockConfigEntry(
+        domain=const.DOMAIN, title="Dynamic Shutter · Común",
+        unique_id="shutter_common_singleton",
+        data={const.CONF_NAME: "Común",
+              const.CONF_MODULE: const.MODULE_SHUTTER_COMMON})
+    common.add_to_hass(hass)
+    assert await hass.config_entries.async_setup(ea.entry_id)
+    await hass.async_block_till_done()
+    if common.state is not ConfigEntryState.LOADED:
+        assert await hass.config_entries.async_setup(common.entry_id)
+        await hass.async_block_till_done()
+    dev_reg = dr.async_get(hass)
+    reg = er.async_get(hass)
+    device = dev_reg.async_get_device(
+        identifiers={(const.DOMAIN, const.SHUTTERS_DEVICE_ID)})
+    eid = reg.async_get_entity_id(
+        "sensor", const.DOMAIN, f"{const.DOMAIN}_covers_open")
+
+    # Simulate the leftover pre-0.98 link: the shutter still pinned to the device.
+    dev_reg.async_update_device(device.id, add_config_entry_id=ea.entry_id)
+    assert ea.entry_id in dev_reg.async_get(device.id).config_entries
+
+    # Reload the Común entry -> its setup strips the stale shutter link.
+    assert await hass.config_entries.async_reload(common.entry_id)
+    await hass.async_block_till_done()
+
+    device = dev_reg.async_get_device(
+        identifiers={(const.DOMAIN, const.SHUTTERS_DEVICE_ID)})
+    assert device.config_entries == {common.entry_id}   # only the Común owns it
+    # The shared entity was untouched: same entity_id, still on the Común entry.
+    assert reg.async_get_entity_id(
+        "sensor", const.DOMAIN, f"{const.DOMAIN}_covers_open") == eid
+    assert reg.async_get(eid).config_entry_id == common.entry_id
+
+
 # --- v0.98.0: the "Común" screen + global switches ---
 async def test_global_switch_fans_out_to_all_shutters(hass: HomeAssistant) -> None:
     from homeassistant.helpers import entity_registry as er