pywiim 2.2.7__tar.gz → 2.2.9__tar.gz

{pywiim-2.2.7/pywiim.egg-info → pywiim-2.2.9}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pywiim
-Version: 2.2.7
+Version: 2.2.9
 Summary: Python library for WiiM/LinkPlay device communication
 Author-email: Michael Cumming <mjcumming@gmail.com>
 License: MIT

{pywiim-2.2.7 → pywiim-2.2.9}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "pywiim"
-version = "2.2.7"
+version = "2.2.9"
 description = "Python library for WiiM/LinkPlay device communication"
 readme = "README.md"
 requires-python = ">=3.11"

{pywiim-2.2.7 → pywiim-2.2.9}/pywiim/__init__.py

@@ -62,6 +62,7 @@ from .discovery import (
     discover_devices,
     discover_via_ssdp,
     validate_device,
+    validate_device_strict,
 )
 from .exceptions import (
     WiiMConnectionError,
@@ -87,7 +88,7 @@ from .profiles import (
 from .role import RoleDetectionResult, detect_role
 from .state import GroupStateSynchronizer, StateSynchronizer
 
-__version__ = "2.2.7"
+__version__ = "2.2.9"
 __all__ = [
     # Main client
     "WiiMClient",
@@ -111,6 +112,7 @@ __all__ = [
     "discover_devices",
     "discover_via_ssdp",
     "validate_device",
+    "validate_device_strict",
     # Backoff
     "BackoffController",
     # Normalization

{pywiim-2.2.7 → pywiim-2.2.9}/pywiim/discovery.py

@@ -22,6 +22,7 @@ except ImportError:
     async_search = None  # type: ignore[assignment]
 
 from .client import WiiMClient
+from .exceptions import WiiMConnectionError
 
 _LOGGER = logging.getLogger(__name__)
 
@@ -32,6 +33,7 @@ __all__ = [
     "is_known_linkplay",
     "is_linkplay_device",
     "validate_device",
+    "validate_device_strict",
 ]
 
 # Known LinkPlay/WiiM server patterns (devices we're CERTAIN are LinkPlay)
@@ -485,6 +487,32 @@ async def validate_device(device: DiscoveredDevice) -> DiscoveredDevice:
     return device
 
 
+async def validate_device_strict(device: DiscoveredDevice) -> DiscoveredDevice:
+    """Validate a discovered device and raise if it is not usable.
+
+    ``validate_device()`` is intentionally soft-fail for bulk discovery: it
+    returns the input device with ``validated=False`` when the host does not
+    pass the LinkPlay/WiiM probe. Use this helper for manual setup flows or
+    other code paths where an unvalidated device is an error.
+
+    Args:
+        device: Device to validate.
+
+    Returns:
+        Updated device with full information.
+
+    Raises:
+        WiiMConnectionError: If the host cannot be validated as LinkPlay/WiiM.
+    """
+    validated_device = await validate_device(device)
+    if not validated_device.validated:
+        raise WiiMConnectionError(
+            f"Device at {device.ip} did not validate as a LinkPlay/WiiM device",
+            operation_context="device_validation",
+        )
+    return validated_device
+
+
 async def discover_devices(
     methods: list[str] | None = None,
     validate: bool = True,

{pywiim-2.2.7 → pywiim-2.2.9}/pywiim/player/coverart.py

@@ -359,12 +359,30 @@ class CoverArtManager:
             _LOGGER.debug("Built metadata update from %s: %s", source_name, update)
         return update
 
-    def _apply_metadata_update(self, update: dict[str, Any], *, source_name: str) -> None:
+    def _apply_metadata_update(
+        self,
+        update: dict[str, Any],
+        *,
+        source_name: str,
+        state_source: str = "http",
+    ) -> None:
         """Apply enrichment metadata to synchronizer, status model, and callbacks."""
         if not update:
             return
 
-        self.player._state_synchronizer.update_from_http(update, timestamp=time.time())
+        timestamp = time.time()
+        if state_source == "upnp":
+            self.player._state_synchronizer.update_from_upnp(
+                update,
+                timestamp=timestamp,
+                force_metadata_update=True,
+            )
+        else:
+            self.player._state_synchronizer.update_from_http(
+                update,
+                timestamp=timestamp,
+                force_metadata_update=True,
+            )
         merged = self.player._state_synchronizer.get_merged_state()
         if self.player._status_model:
             if "title" in update:
@@ -406,7 +424,7 @@ class CoverArtManager:
         self.player._getinfoex_supported = True
         update = self._build_metadata_update(merged_state, info, source_name="GetInfoEx")
         if update:
-            self._apply_metadata_update(update, source_name="GetInfoEx")
+            self._apply_metadata_update(update, source_name="GetInfoEx", state_source="upnp")
         return update
 
     async def _fetch_artwork_from_metainfo(self, merged_state: dict[str, Any]) -> None:

{pywiim-2.2.7 → pywiim-2.2.9}/pywiim/state.py

@@ -310,6 +310,7 @@ class StateSynchronizer:
         data: dict[str, Any],
         timestamp: float | None = None,
         source: str = "http",
+        force_metadata_update: bool = False,
     ) -> None:
         """Update state from HTTP polling data.
 
@@ -319,6 +320,9 @@ class StateSynchronizer:
             source: Source identifier for the update. Defaults to "http" for normal
                 HTTP polling. Use "propagated" when state is propagated from master
                 to slave in a group.
+            force_metadata_update: Apply metadata fields even when transport is paused/idle.
+                Use for explicit enrichment probes that return valid metadata for the current
+                paused track; normal status polling should leave this False.
         """
         ts = timestamp or time.time()
 
@@ -374,7 +378,7 @@ class StateSynchronizer:
             )
 
         # Extract metadata (preserve if playing)
-        if not self._should_clear_metadata():
+        if force_metadata_update or not self._should_clear_metadata():
             for field_name in ["title", "artist", "album", "image_url"]:
                 if field_name in data:
                     value = data.get(field_name)
@@ -395,12 +399,16 @@ class StateSynchronizer:
         self,
         data: dict[str, Any],
         timestamp: float | None = None,
+        force_metadata_update: bool = False,
     ) -> None:
         """Update state from UPnP event data.
 
         Args:
             data: Dictionary with state fields from UPnP event
             timestamp: Timestamp of the update (defaults to now)
+            force_metadata_update: Apply metadata fields even when transport is paused/idle.
+                Use for explicit enrichment probes like GetInfoEx; ordinary UPnP events should
+                leave this False so metadata-free stop/idle updates do not clear current fields.
         """
         ts = timestamp or time.time()
 
@@ -453,7 +461,7 @@ class StateSynchronizer:
             )
 
         # Extract metadata (preserve if playing)
-        if not self._should_clear_metadata():
+        if force_metadata_update or not self._should_clear_metadata():
             for field_name in ["title", "artist", "album", "image_url"]:
                 if field_name in data:
                     value = data.get(field_name)

{pywiim-2.2.7 → pywiim-2.2.9/pywiim.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: pywiim
-Version: 2.2.7
+Version: 2.2.9
 Summary: Python library for WiiM/LinkPlay device communication
 Author-email: Michael Cumming <mjcumming@gmail.com>
 License: MIT