Fix AC coverage for TP1X-class (oneUiVersion "7.0 Air conditioner")

custom_components/localthings/climate.py

@@ -41,6 +41,7 @@ from .registry.capabilities.airconditioner import (
     HREF_TEMP_CURRENT as TEMP_CURRENT_HREF,
     HREF_TEMP_DESIRED as TEMP_DESIRED_HREF,
     HREF_TEMP_CONTROL as TEMP_CONTROL_HREF,
+    HREF_TEMPS_VS as TEMPS_VS_HREF,
     HREF_WIND_STRENGTH as WIND_STRENGTH_HREF,
     HREF_WIND_DIRECTION as WIND_DIRECTION_HREF,
     HREF_CONVENIENT as CONVENIENT_HREF,
@@ -125,6 +126,22 @@ def _num(value):
         return None
 
 
+def _temps_vs_item(rep: dict) -> dict:
+    """First item of the vendor `/temperatures/vs/0` items[] array.
+
+    Newer AC firmware (Tizen Lite, oneUiVersion "7.0 Air conditioner", e.g.
+    model TP1X_DA-AC-RAC-01011) does NOT expose the OCF-standard
+    /temperature/current/0 + /temperature/desired/0 pair; it reports current
+    and target under a single `/temperatures/vs/0` resource whose
+    `x.com.samsung.da.items[0]` carries current/desired/minimum/maximum/
+    increment/unit. Returns {} when absent, so callers fall through cleanly.
+    """
+    items = rep.get('x.com.samsung.da.items')
+    if isinstance(items, (list, tuple)) and items and isinstance(items[0], dict):
+        return items[0]
+    return {}
+
+
 class LocalThingsClimate(LocalThingsEntity, ClimateEntity):
     """Composite climate entity for a Samsung air conditioner."""
 
@@ -172,24 +189,41 @@ class LocalThingsClimate(LocalThingsEntity, ClimateEntity):
 
     # -- temperature --------------------------------------------------------
 
+    def _temps_vs(self) -> dict:
+        """Vendor `/temperatures/vs/0` items[0] (empty {} when absent)."""
+        return _temps_vs_item(self._rep(TEMPS_VS_HREF))
+
     @property
     def temperature_unit(self) -> str:
         raw = self._rep(TEMP_DESIRED_HREF).get('units')
+        if raw is None:
+            raw = self._temps_vs().get('x.com.samsung.da.unit')
         return (UnitOfTemperature.FAHRENHEIT
                 if normalize_temp_unit(raw, '°C') == '°F'
                 else UnitOfTemperature.CELSIUS)
 
     @property
     def current_temperature(self):
-        return _num(self._rep(TEMP_CURRENT_HREF).get('temperature'))
+        v = _num(self._rep(TEMP_CURRENT_HREF).get('temperature'))
+        if v is None:
+            v = _num(self._temps_vs().get('x.com.samsung.da.current'))
+        return v
 
     @property
     def target_temperature(self):
-        return _num(self._rep(TEMP_DESIRED_HREF).get('temperature'))
+        v = _num(self._rep(TEMP_DESIRED_HREF).get('temperature'))
+        if v is None:
+            v = _num(self._temps_vs().get('x.com.samsung.da.desired'))
+        return v
 
     def _range(self) -> list | None:
         r = self._rep(TEMP_DESIRED_HREF).get('range')
-        return r if (isinstance(r, (list, tuple)) and len(r) == 2) else None
+        if isinstance(r, (list, tuple)) and len(r) == 2:
+            return r
+        item = self._temps_vs()
+        lo = _num(item.get('x.com.samsung.da.minimum'))
+        hi = _num(item.get('x.com.samsung.da.maximum'))
+        return [lo, hi] if (lo is not None and hi is not None) else None
 
     @property
     def min_temp(self) -> float:
@@ -203,7 +237,10 @@ class LocalThingsClimate(LocalThingsEntity, ClimateEntity):
 
     @property
     def target_temperature_step(self) -> float:
-        return _num(self._rep(TEMP_CONTROL_HREF).get('increment')) or 1.0
+        return (_num(self._rep(TEMP_CONTROL_HREF).get('increment'))
+                or _num(self._rep(TEMP_CONTROL_HREF).get('x.com.samsung.da.increment'))
+                or _num(self._temps_vs().get('x.com.samsung.da.increment'))
+                or 1.0)
 
     # -- hvac mode ----------------------------------------------------------
 

custom_components/localthings/registry/by_type/airconditioner.py

@@ -24,6 +24,7 @@ REGISTRY = DeviceRegistry(
         airconditioner.AIR_PURIFY,
         airconditioner.AUTO_CLEAN,
         airconditioner.AIR_FILTER,
+        airconditioner.DISPLAY_LIGHT,
         *airconditioner.COVERAGE,
     ]),
 )

custom_components/localthings/registry/capabilities/airconditioner.py

@@ -34,10 +34,12 @@ HREF_TEMP_CONTROL = '/temperature/control/vs/0'   # target_temperature_step
 HREF_WIND_STRENGTH = '/wind/strength/vs/0'        # fan_mode
 HREF_WIND_DIRECTION = '/wind/direction/vs/0'      # swing_mode
 HREF_CONVENIENT = '/mode/convenient/vs/0'         # preset_mode
+HREF_TEMPS_VS = '/temperatures/vs/0'              # vendor temp fallback (items[] array)
 
 CLIMATE_CONSUMED_HREFS = [
     HREF_POWER, HREF_POWER_VS, HREF_TEMP_CURRENT, HREF_TEMP_DESIRED,
-    HREF_TEMP_CONTROL, HREF_WIND_STRENGTH, HREF_WIND_DIRECTION, HREF_CONVENIENT,
+    HREF_TEMP_CONTROL, HREF_TEMPS_VS, HREF_WIND_STRENGTH, HREF_WIND_DIRECTION,
+    HREF_CONVENIENT,
 ]
 
 
@@ -146,6 +148,20 @@ AIR_FILTER = Capability(
     ),
 )
 
+DISPLAY_LIGHT = Capability(
+    href='/light/vs/0',
+    poll_tier='cold',
+    entities=(
+        SwitchDesc(key='display_light', field='mode',
+                   name='Display light', icon='mdi:led-on',
+                   entity_category='config',
+                   value_fn=lambda v: v == 'On',
+                   write_fn=lambda p, rep, href=None: (
+                       ['light', 'vs', '0'],
+                       {'mode': 'On' if p == 'On' else 'Off'})),
+    ),
+)
+
 # ---------------------------------------------------------------------------
 # AC-scoped coverage: the CLIMATE_CONSUMED_HREFS above (read by the climate
 # entity) plus vendor duplicates / all-zero-ambiguous / plumbing resources.
@@ -155,8 +171,6 @@ AIR_FILTER = Capability(
 # bound so discover() reports no coverage gap.
 # ---------------------------------------------------------------------------
 _AC_IGNORED = [
-    # Vendor superset that duplicates the OCF /temperature/current+desired pair.
-    '/temperatures/vs/0',
     # All-zero and ambiguously encoded on this model (2-value arrays); the
     # 'don't guess' rule -- leave unmodeled rather than invent entities.
     '/sensors/vs/0',
@@ -164,6 +178,24 @@ _AC_IGNORED = [
     '/humidity/vs/0',
     # Presence-personalization plumbing (empty item list here).
     '/personality/presence/vs/0',
+    # --- TP1X-class (oneUiVersion "7.0 Air conditioner") housekeeping / opaque
+    # blobs. These carry no user-actionable state or no documented write
+    # contract, so per the 'don't guess' rule they are ignored rather than
+    # modeled. (Temperature for this class lives at /temperatures/vs/0, now
+    # consumed by the climate entity -- see CLIMATE_CONSUMED_HREFS.)
+    '/airlevelcheck/vs/0',         # periodic air-quality sensing scheduler plumbing
+    '/aisleep/vs/0',               # AI-sleep feedback state (no actionable control)
+    '/availablecontrolsets/vs/0',  # opaque hex-encoded control-set bitmap
+    '/da/softreset/vs/0',          # soft-reset trigger plumbing
+    '/keepnormalstate/vs/0',       # internal keep-normal flag
+    '/mds/absencemonitoring/vs/0', # motion-detection sensor plumbing (empty here)
+    '/mds/absencestate/vs/0',      # motion-detection state (empty here)
+    '/option/muteonce/vs/0',       # one-shot buzzer mute (no persistent state)
+    '/remotedatacontrol/vs/0',     # remote data-control session status
+    '/remotetemperature/vs/0',     # external temp-sensor feed (unset on this unit)
+    '/reserverulesets/vs/0',       # opaque hex-encoded schedule reservation blob
+    '/selfcheck/vs/0',             # self-diagnosis action (start/cancel, no simple state)
+    '/welcome/temperature/vs/0',   # welcome-cooling plumbing
 ]
 
 # Built as bare no-entity caps; folded into the AC registry (not global).