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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: python-roborock
-Version: 7.4.2
+Version: 7.4.3
 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.4.3}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "python-roborock"
-version = "7.4.2"
+version = "7.4.3"
 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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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.4.3}/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)):

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/map/b01_q10_map_parser.py

@@ -341,7 +341,7 @@ def _drop_stray_leading_point(points: list[Q10Point]) -> list[Q10Point]:
     """
     if len(points) < 3:
         return points
-    steps = [math.hypot(b.x - a.x, b.y - a.y) for a, b in zip(points, points[1:])]
+    steps = [math.hypot(b.x - a.x, b.y - a.y) for a, b in zip(points, points[1:], strict=False)]
     median_rest = statistics.median(steps[1:])
     if median_rest > 0 and steps[0] > _STRAY_POINT_STEP_RATIO * median_rest:
         return points[1:]
@@ -425,7 +425,7 @@ def _infer_layout(decoded: bytes, width: int) -> tuple[int, bytes, bytes]:
     up with the marker. Used as a fallback when the header carries no usable
     height.
     """
-    for room_count in range(0, _MAX_ROOMS + 1):
+    for room_count in range(_MAX_ROOMS + 1):
         room_data_length = 2 + room_count * _ROOM_RECORD_LENGTH
         area = len(decoded) - room_data_length
         if area <= 0 or area % width:

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/mqtt/roborock_session.py

@@ -165,7 +165,7 @@ class RoborockMqttSession(MqttSession):
                 await self._connection_task
             except asyncio.CancelledError:
                 _LOGGER.debug("MQTT connection task cancelled")
-            except Exception:
+            except Exception:  # noqa: BLE001
                 # Exceptions are logged and handled in _run_connection.
                 # There is a special case for exceptions on startup where we return
                 # immediately. Otherwise, we let the reconnect loop retry with

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/protocol.py

@@ -5,7 +5,7 @@ import logging
 from collections.abc import Callable
 from urllib.parse import urlparse
 
-from construct import (  # type: ignore
+from construct import (  # type: ignore[import-untyped]
     Bytes,
     Checksum,
     ChecksumError,
@@ -65,7 +65,7 @@ class Utils:
     @staticmethod
     def encode_timestamp(_timestamp: int) -> bytes:
         hex_value = f"{_timestamp:x}".zfill(8)
-        return "".join(list(map(lambda idx: hex_value[idx], [5, 6, 3, 7, 1, 2, 0, 4]))).encode()
+        return "".join([hex_value[idx] for idx in [5, 6, 3, 7, 1, 2, 0, 4]]).encode()
 
     @staticmethod
     def md5(data: bytes) -> bytes:
@@ -470,7 +470,7 @@ class _Parser:
                 }
             )
         return self.con.build(
-            {"messages": [message for message in messages], "remaining": b""},
+            {"messages": list(messages), "remaining": b""},
             local_key=local_key,
             prefixed=prefixed,
             connect_nonce=connect_nonce,

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/protocols/a01_protocol.py

@@ -62,7 +62,7 @@ def decode_rpc_response(message: RoborockMessage) -> dict[int, Any]:
     try:
         unpadded = unpad(message.payload, AES.block_size)
     except ValueError as err:
-        raise RoborockException(f"Unable to unpad A01 payload: {err}")
+        raise RoborockException(f"Unable to unpad A01 payload: {err}") from err
 
     try:
         payload = json.loads(unpadded.decode())
@@ -74,5 +74,7 @@ def decode_rpc_response(message: RoborockMessage) -> dict[int, Any]:
         raise RoborockException(f"Invalid A01 message format: 'dps' should be a dictionary for {message.payload!r}")
     try:
         return {int(key): value for key, value in datapoints.items()}
-    except ValueError:
-        raise RoborockException(f"Invalid A01 message format: 'dps' key should be an integer for {message.payload!r}")
+    except ValueError as err:
+        raise RoborockException(
+            f"Invalid A01 message format: 'dps' key should be an integer for {message.payload!r}"
+        ) from err

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/protocols/b01_q7_protocol.py

@@ -83,8 +83,10 @@ def decode_rpc_response(message: RoborockMessage) -> dict[int, Any]:
         raise RoborockException(f"Invalid B01 message format: 'dps' should be a dictionary for {message.payload!r}")
     try:
         return {int(key): value for key, value in datapoints.items()}
-    except ValueError:
-        raise RoborockException(f"Invalid B01 message format: 'dps' key should be an integer for {message.payload!r}")
+    except ValueError as err:
+        raise RoborockException(
+            f"Invalid B01 message format: 'dps' key should be an integer for {message.payload!r}"
+        ) from err
 
 
 @dataclass

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/testing/v1_simulator.py

@@ -346,7 +346,7 @@ class V1VacuumSimulator(RoborockDeviceSimulator):
             msg_id = inner["id"]
             method = inner["method"]
             params = inner.get("params", [])
-        except Exception as e:
+        except Exception as e:  # noqa: BLE001
             _LOGGER.debug("Failed to parse plaintext JSON RPC payload: %s", e, exc_info=True)
             return
 
@@ -358,7 +358,7 @@ class V1VacuumSimulator(RoborockDeviceSimulator):
         if handler:
             try:
                 result = handler(params)
-            except Exception as e:
+            except Exception as e:  # noqa: BLE001
                 error = str(e)
                 _LOGGER.debug("Error executing command handler for %s: %s", method, e, exc_info=True)
         else:

{python_roborock-7.4.2 → python_roborock-7.4.3}/roborock/util.py

@@ -10,7 +10,7 @@ T = TypeVar("T")
 
 
 def unpack_list(value: list[T], size: int) -> list[T | None]:
-    return (value + [None] * size)[:size]  # type: ignore
+    return (value + [None] * size)[:size]
 
 
 class RoborockLoggerAdapter(logging.LoggerAdapter):