python-roborock 3.14.1__tar.gz → 3.14.2__tar.gz

{python_roborock-3.14.1 → python_roborock-3.14.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: python-roborock
-Version: 3.14.1
+Version: 3.14.2
 Summary: A package to control Roborock vacuums.
 Project-URL: Repository, https://github.com/humbertogontijo/python-roborock
 Project-URL: Documentation, https://python-roborock.readthedocs.io/

{python_roborock-3.14.1 → python_roborock-3.14.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-roborock"
-version = "3.14.1"
+version = "3.14.2"
 description = "A package to control Roborock vacuums."
 authors = [{ name = "humbertogontijo", email = "humbertogontijo@users.noreply.github.com" }, {name="Lash-L"}, {name="allenporter"}]
 requires-python = ">=3.11, <4"

{python_roborock-3.14.1 → python_roborock-3.14.2}/roborock/data/containers.py

@@ -228,6 +228,10 @@ class HomeDataProduct(RoborockBase):
     def product_nickname(self) -> RoborockProductNickname:
         return SHORT_MODEL_TO_ENUM.get(self.model.split(".")[-1], RoborockProductNickname.PEARLPLUS)
 
+    def summary_info(self) -> str:
+        """Return a string with key product information for logging purposes."""
+        return f"{self.name} (model={self.model}, category={self.category})"
+
 
 @dataclass
 class HomeDataDevice(RoborockBase):
@@ -263,6 +267,10 @@ class HomeDataDevice(RoborockBase):
     share_type: Any | None = None
     share_expired_time: int | None = None
 
+    def summary_info(self) -> str:
+        """Return a string with key device information for logging purposes."""
+        return f"{self.name} (pv={self.pv}, fv={self.fv}, online={self.online})"
+
 
 @dataclass
 class HomeDataRoom(RoborockBase):

{python_roborock-3.14.1 → python_roborock-3.14.2}/roborock/devices/device.py

@@ -183,7 +183,6 @@ class RoborockDevice(ABC, TraitsMixin):
         if self._unsub:
             raise ValueError("Already connected to the device")
         unsub = await self._channel.subscribe(self._on_message)
-        self._logger.info("Connecting to device")
         if self.v1_properties is not None:
             try:
                 await self.v1_properties.discover_features()

{python_roborock-3.14.1 → python_roborock-3.14.2}/roborock/devices/device_manager.py

@@ -78,12 +78,13 @@ class DeviceManager:
         home_data = cache_data.home_data
 
         device_products = home_data.device_products
-        _LOGGER.debug("Discovered %d devices %s", len(device_products), home_data)
+        _LOGGER.debug("Discovered %d devices", len(device_products))
 
         # These are connected serially to avoid overwhelming the MQTT broker
         new_devices = {}
         start_tasks = []
         for duid, (device, product) in device_products.items():
+            _LOGGER.debug("[%s] Discovered device %s %s", duid, product.summary_info(), device.summary_info())
             if duid in self._devices:
                 continue
             new_device = self._device_creator(home_data, device, product)

{python_roborock-3.14.1 → python_roborock-3.14.2}/roborock/devices/local_channel.py

@@ -179,7 +179,6 @@ class LocalChannel(Channel):
         if self._is_connected:
             self._logger.debug("Unexpected call to connect when already connected")
             return
-        self._logger.debug("Connecting to %s:%s", self._host, _PORT)
         loop = asyncio.get_running_loop()
         protocol = _LocalProtocol(self._data_received, self._connection_lost)
         try:

{python_roborock-3.14.1 → python_roborock-3.14.2}/roborock/devices/v1_channel.py

@@ -373,7 +373,7 @@ class V1Channel(Channel):
         # Wire up the new channel
         self._local_channel = local_channel
         self._local_unsub = await self._local_channel.subscribe(self._on_local_message)
-        _LOGGER.info("Successfully connected to local device %s", self._device_uid)
+        self._logger.info("Connected to local channel successfully")
 
     async def _background_reconnect(self) -> None:
         """Task to run in the background to manage the local connection."""