python-roborock 7.4.2__tar.gz → 7.5.0__tar.gz

{python_roborock-7.4.2 → python_roborock-7.5.0}/PKG-INFO

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

{python_roborock-7.4.2 → python_roborock-7.5.0}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-roborock"
-version = "7.4.2"
+version = "7.5.0"
 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"
@@ -106,14 +106,40 @@ allowed_tags = [
 major_tags= ["refactor"]
 
 [tool.ruff]
-lint.ignore = ["F403", "E741"]
-lint.select=["E", "F", "UP", "I"]
+lint.ignore = []
+# TODO: Enable ASYNC (flake8-async) in a dedicated follow-up PR once async open() and timeout parameters are refactored.
+lint.select = [
+    "A",
+    "B",
+    "BLE",
+    "C4",
+    "E",
+    "F",
+    "FA",
+    "I",
+    "ICN",
+    "ISC",
+    "PGH",
+    "PIE",
+    "UP",
+    "W",
+    "YTT",
+]
 line-length = 120
 extend-exclude = ["roborock/map/proto/*_pb2.py"]
 
 [tool.ruff.lint.per-file-ignores]
 "*/__init__.py" = ["F401"]
+"roborock/__init__.py" = ["F403"]
+"roborock/data/__init__.py" = ["F403"]
+"roborock/data/b01_q7/__init__.py" = ["F403"]
+"roborock/data/b01_q10/__init__.py" = ["F403"]
+"roborock/data/dyad/__init__.py" = ["F403"]
+"roborock/data/mower/__init__.py" = ["F403"]
+"roborock/data/v1/__init__.py" = ["F403"]
+"roborock/data/zeo/__init__.py" = ["F403"]
 "roborock/map/proto/*_pb2.py" = ["E501", "I001", "UP009"]
+"tests/*" = ["B010"]
 
 [[tool.mypy.overrides]]
 module = ["roborock.map.proto.*"]

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/broadcast_protocol.py

@@ -4,7 +4,7 @@ import json
 import logging
 from asyncio import BaseTransport, Lock
 
-from construct import (  # type: ignore
+from construct import (  # type: ignore[import-untyped]
     Bytes,
     Checksum,
     GreedyBytes,
@@ -63,7 +63,7 @@ class RoborockProtocol(asyncio.DatagramProtocol):
                     parsed_message = BroadcastMessage(duid=json_payload["duid"], ip=json_payload["ip"], version=version)
                     _LOGGER.debug(f"Received broadcast: {parsed_message}")
                     self.devices_found.append(parsed_message)
-        except Exception as e:
+        except Exception as e:  # noqa: BLE001
             _LOGGER.warning(f"Failed to decode message: {data!r}. Error: {e}")
 
     async def discover(self) -> list[BroadcastMessage]:

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/cli.py

@@ -22,6 +22,8 @@ roborock> status --device_id <device_id>
 ```
 """
 
+# ruff: noqa: BLE001
+
 import asyncio
 import datetime
 import functools
@@ -38,9 +40,9 @@ try:
     import click
     import click_shell
     import yaml
-    from pyshark import FileCapture  # type: ignore
-    from pyshark.capture.live_capture import LiveCapture, UnknownInterfaceException  # type: ignore
-    from pyshark.packet.packet import Packet  # type: ignore
+    from pyshark import FileCapture  # type: ignore[import-untyped]
+    from pyshark.capture.live_capture import LiveCapture, UnknownInterfaceException  # type: ignore[import-untyped]
+    from pyshark.packet.packet import Packet  # type: ignore[import-untyped]
 except ImportError as err:
     raise SystemExit(
         f"The 'roborock' command line tool requires extra dependencies that are not installed ({err.name}).\n"
@@ -307,7 +309,7 @@ class RoborockContext(Cache):
 @click.pass_context
 def cli(ctx, debug: int):
     logging_config: dict[str, Any] = {"level": logging.DEBUG if debug > 0 else logging.INFO}
-    logging.basicConfig(**logging_config)  # type: ignore
+    logging.basicConfig(**logging_config)  # type: ignore[call-overload]
     ctx.obj = RoborockContext()
 
 
@@ -988,9 +990,8 @@ async def parser(_, local_key, device_ip, file):
                                     local_key,
                                 )
                                 print(f"Received request: {f}")
-                            except BaseException as e:
+                            except Exception as e:
                                 print(e)
-                                pass
                         elif packet.ip.src == device_ip:
                             try:
                                 f, buffer["data"] = MessageParser.parse(
@@ -998,16 +999,15 @@ async def parser(_, local_key, device_ip, file):
                                     local_key,
                                 )
                                 print(f"Received response: {f}")
-                            except BaseException as e:
+                            except Exception as e:
                                 print(e)
-                                pass
 
     try:
         await capture.packets_from_tshark(on_package, close_tshark=not file_provided)
-    except UnknownInterfaceException:
+    except UnknownInterfaceException as err:
         raise RoborockException(
             "You need to run 'rvictl -s XXXXXXXX-XXXXXXXXXXXXXXXX' first, with an iPhone connected to usb port"
-        )
+        ) from err
 
 
 def _parse_diagnostic_file(diagnostic_path: Path) -> dict[str, dict[str, Any]]:
@@ -1319,7 +1319,7 @@ def update_docs(data_file: str, output_file: str):
         ]
         # Regular features are the remaining keys, sorted alphabetically
         # We filter out the special rows to avoid duplicating them.
-        sorted_features = sorted(list(all_features - set(special_rows)))
+        sorted_features = sorted(all_features - set(special_rows))
 
         header = ["Feature"] + sorted_products
 

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/data/b01_q7/b01_q7_code_mappings.py

@@ -302,7 +302,11 @@ class B01Fault(RoborockModeEnum):
     # charging.
     F_2101 = ("fault_2101", 2101)
     F_2102 = ("cleaning_complete", 2102)  # Cleaning completed. Returning to the dock.
-    F_2103 = ("fault_2103", 2103)
+    F_2103 = (
+        "cleaning_complete_docked",
+        2103,
+    )  # Task finished and robot docked. Follows 2102 once docking and any auto-empty complete; persists
+    # while docked-idle. Not an error (hw-confirmed on a Q7 M5+).
     F_2104 = ("fault_2104", 2104)
     F_2105 = ("fault_2105", 2105)
     F_2108 = ("fault_2108", 2108)

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/data/b01_q7/b01_q7_containers.py

@@ -137,11 +137,11 @@ class B01Props(RoborockBase):
     custom_type: int | None = None
     sound: int | None = None
     work_mode: WorkModeMapping | None = None
-    station_act: int | None = None
+    station_act: int | None = None  # dock activity; 3 while collecting dust, 0 idle. hw-confirmed (Q7 M5+)
     charge_state: int | None = None
     current_map_id: int | None = None
     map_num: int | None = None
-    dust_action: int | None = None
+    dust_action: int | None = None  # 1 while the dock is collecting dust; read-only (prop.set rejected, code 1)
     quiet_is_open: int | None = None
     quiet_begin_time: int | None = None
     quiet_end_time: int | None = None

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/data/code_mappings.py

@@ -33,7 +33,7 @@ class RoborockEnum(IntEnum):
             if warning not in completed_warnings:
                 completed_warnings.add(warning)
                 _LOGGER.warning(warning)
-            return cls.unknown  # type: ignore
+            return cls.unknown  # type: ignore[attr-defined]
         default_value = next(item for item in cls)
         warning = f"Missing {cls.__name__} code: {key} - defaulting to {default_value}"
         if warning not in completed_warnings:

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/data/containers.py

@@ -52,7 +52,7 @@ def _attr_repr(obj: Any) -> str:
             continue
         try:
             v = getattr(obj, k)
-        except (RuntimeError, Exception):
+        except Exception:  # noqa: BLE001
             continue
         if callable(v):
             continue
@@ -210,7 +210,7 @@ class Reference(RoborockBase):
     r: str | None = None
     a: str | None = None
     m: str | None = None
-    l: str | None = None
+    l: str | None = None  # noqa: E741
 
 
 @dataclass
@@ -379,9 +379,9 @@ class HomeDataSchedule(RoborockBase):
 class HomeData(RoborockBase):
     id: int
     name: str
-    products: list[HomeDataProduct] = field(default_factory=lambda: [])
-    devices: list[HomeDataDevice] = field(default_factory=lambda: [])
-    received_devices: list[HomeDataDevice] = field(default_factory=lambda: [])
+    products: list[HomeDataProduct] = field(default_factory=list)
+    devices: list[HomeDataDevice] = field(default_factory=list)
+    received_devices: list[HomeDataDevice] = field(default_factory=list)
     lon: Any | None = None
     lat: Any | None = None
     geo_name: Any | None = None

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/data/v1/v1_code_mappings.py

@@ -151,7 +151,7 @@ class RoborockDssCodes(RoborockEnum):
     def _missing_(cls: type[Self], key) -> Self:
         # If the calculated value is not provided, then it should be viewed as okay.
         # As the math will sometimes result in you getting numbers that don't matter.
-        return cls.okay  # type: ignore
+        return cls.okay  # type: ignore[attr-defined]
 
 
 class ClearWaterBoxStatus(RoborockDssCodes):

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/device_features.py

@@ -643,7 +643,7 @@ class DeviceFeatures(RoborockBase):
             elif (product_features := f.metadata.get("product_features")) is not None:
                 if product_nickname is not None:
                     available_features = PRODUCT_FEATURE_MAP.get(product_nickname, [])
-                    if any(feat in available_features for feat in product_features):  # type: ignore
+                    if any(feat in available_features for feat in product_features):
                         kwargs[f.name] = True
 
         # The app combines runtime shake-mop, model shake/spin, and roller-mop

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/rpc/b01_q7_channel.py

@@ -84,7 +84,7 @@ async def _send_command(
             return
         try:
             response = response_matcher(response_message)
-        except Exception as ex:
+        except Exception as ex:  # noqa: BLE001
             future.set_exception(ex)
             return
         if response is not None:

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/traits/__init__.py

@@ -24,5 +24,5 @@ __all__ = [
 ]
 
 
-class Trait(ABC):
+class Trait(ABC):  # noqa: B024
     """Base class for all traits."""

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/traits/b01/q7/map_content.py

@@ -6,6 +6,10 @@ This intentionally mirrors the v1 `MapContentTrait` contract:
 - fields `image_content`, `map_data`, and `raw_api_response` are then readable
 
 For B01/Q7 devices, the underlying raw map payload is retrieved via `MapTrait`.
+
+Q7 devices additionally stream map frames unprompted while cleaning; those arrive
+through `update_from_push()`, which keeps the same cached fields current and
+notifies registered update listeners.
 """
 
 import asyncio

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/traits/v1/clean_summary.py

@@ -56,7 +56,7 @@ class CleanRecordConverter(common.V1TraitDataConverter):
                             rec.square_meter_area or 0
                         )
                     return final_record
-                except Exception:
+                except Exception:  # noqa: BLE001
                     # Return final record when an exception occurred
                     return final_record
             # There are still a few unknown variables in this.

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/traits/v1/rooms.py

@@ -144,6 +144,6 @@ class RoomsTrait(Rooms, common.V1TraitMixin):
                 rooms_by_id.update({room.iot_id: room for room in shared_rooms})
                 return list(rooms_by_id.values())
             return await self._web_api.get_rooms()
-        except Exception:
+        except Exception:  # noqa: BLE001
             _LOGGER.debug("Failed to fetch rooms from web API", exc_info=True)
             return []

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/devices/transport/local_channel.py

@@ -158,7 +158,7 @@ class LocalChannel(Channel):
                     await self._ping()
             except asyncio.CancelledError:
                 break
-            except Exception:
+            except Exception:  # noqa: BLE001
                 self._logger.debug("Keep-alive ping failed", exc_info=True)
                 # Retry next interval
 

{python_roborock-7.4.2 → python_roborock-7.5.0}/roborock/map/b01_grid_layers.py

@@ -198,7 +198,7 @@ def solve_calibration(
             min_sx, max_sx, min_sy, max_sy = min(sx), max(sx), min(sy), max(sy)
             if (max_sx - min_sx) >= w or (max_sy - min_sy) >= h:
                 continue  # path wider/taller than the map at this resolution
-            pts = list(zip(sx, sy))
+            pts = list(zip(sx, sy, strict=True))
             # Slide so every point stays in-bounds: px = px_f + ox in [0, w), py = oy - py_f in [0, h).
             for ox in range(ceil(-min_sx), ceil(w - max_sx)):
                 for oy in range(ceil(max_sy), ceil(h + min_sy)):