From e4b53f71340cf33cbf828a4c9aec55c1fc3f719b Mon Sep 17 00:00:00 2001
From: Allen Porter <allen.porter@gmail.com>
Date: Sat, 27 Sep 2025 18:25:55 -0700
Subject: [PATCH 1/2] feat: add module for parsing map content

---
 roborock/map/map_parser.py   | 106 +++++++++++++++++++++++++++++++++++
 tests/map/test_map_parser.py |  21 +++++++
 2 files changed, 127 insertions(+)
 create mode 100644 roborock/map/map_parser.py
 create mode 100644 tests/map/test_map_parser.py

diff --git a/roborock/map/map_parser.py b/roborock/map/map_parser.py
new file mode 100644
index 00000000..5bab061f
--- /dev/null
+++ b/roborock/map/map_parser.py
@@ -0,0 +1,106 @@
+"""Module for parsing v1 Roborock map content."""
+
+import io
+import logging
+from dataclasses import dataclass, field
+
+from vacuum_map_parser_base.config.color import ColorsPalette, SupportedColor
+from vacuum_map_parser_base.config.drawable import Drawable
+from vacuum_map_parser_base.config.image_config import ImageConfig
+from vacuum_map_parser_base.config.size import Size, Sizes
+from vacuum_map_parser_base.map_data import MapData
+from vacuum_map_parser_roborock.map_data_parser import RoborockMapDataParser
+
+from roborock.exceptions import RoborockException
+
+_LOGGER = logging.getLogger(__name__)
+
+DEFAULT_DRAWABLES = {
+    Drawable.CHARGER: True,
+    Drawable.CLEANED_AREA: False,
+    Drawable.GOTO_PATH: False,
+    Drawable.IGNORED_OBSTACLES: False,
+    Drawable.IGNORED_OBSTACLES_WITH_PHOTO: False,
+    Drawable.MOP_PATH: False,
+    Drawable.NO_CARPET_AREAS: False,
+    Drawable.NO_GO_AREAS: False,
+    Drawable.NO_MOPPING_AREAS: False,
+    Drawable.OBSTACLES: False,
+    Drawable.OBSTACLES_WITH_PHOTO: False,
+    Drawable.PATH: True,
+    Drawable.PREDICTED_PATH: False,
+    Drawable.VACUUM_POSITION: True,
+    Drawable.VIRTUAL_WALLS: False,
+    Drawable.ZONES: False,
+}
+DEFAULT_MAP_SCALE = 4
+MAP_FILE_FORMAT = "PNG"
+
+
+def _default_drawable_factory() -> list[Drawable]:
+    return [drawable for drawable, default_value in DEFAULT_DRAWABLES.items() if default_value]
+
+
+@dataclass
+class MapParserConfig:
+    """Configuration for the Roborock map parser."""
+
+    drawables: list[Drawable] = field(default_factory=_default_drawable_factory)
+    """List of drawables to include in the map rendering."""
+
+    show_background: bool = True
+    """Whether to show the background of the map."""
+
+    map_scale: int = DEFAULT_MAP_SCALE
+    """Scale factor for the map."""
+
+
+@dataclass
+class ParsedMapData:
+    """Roborock Map Data.
+
+    This class holds the parsed map data and the rendered image.
+    """
+
+    image_content: bytes | None
+    """The rendered image of the map in PNG format."""
+
+    map_data: MapData | None
+    """The parsed map data which contains metadata for points on the map."""
+
+
+class MapParser:
+    """Roborock Map Parser.
+
+    This class is used to parse the map data from the device and render it into an image.
+    """
+
+    def __init__(self, config: MapParserConfig) -> None:
+        """Initialize the MapParser."""
+        self._map_parser = _create_map_data_parser(config)
+
+    def parse(self, map_bytes: bytes) -> ParsedMapData | None:
+        """Parse map_bytes and return MapData and the image."""
+        try:
+            parsed_map = self._map_parser.parse(map_bytes)
+        except (IndexError, ValueError) as err:
+            raise RoborockException("Failed to parse map data") from err
+        if parsed_map.image is None:
+            raise RoborockException("Failed to render map image")
+        img_byte_arr = io.BytesIO()
+        parsed_map.image.data.save(img_byte_arr, format=MAP_FILE_FORMAT)
+        return ParsedMapData(image_content=img_byte_arr.getvalue(), map_data=parsed_map)
+
+
+def _create_map_data_parser(config: MapParserConfig) -> RoborockMapDataParser:
+    """Create a RoborockMapDataParser based on the config entry."""
+    colors = ColorsPalette()
+    if not config.show_background:
+        colors = ColorsPalette({SupportedColor.MAP_OUTSIDE: (0, 0, 0, 0)})
+    return RoborockMapDataParser(
+        colors,
+        Sizes({k: v * config.map_scale for k, v in Sizes.SIZES.items() if k != Size.MOP_PATH_WIDTH}),
+        config.drawables,
+        ImageConfig(scale=config.map_scale),
+        [],
+    )
diff --git a/tests/map/test_map_parser.py b/tests/map/test_map_parser.py
new file mode 100644
index 00000000..4860afa1
--- /dev/null
+++ b/tests/map/test_map_parser.py
@@ -0,0 +1,21 @@
+"""Tests for the map parser."""
+
+from pathlib import Path
+
+import pytest
+from roborock.exceptions import RoborockException
+from roborock.map.map_parser import MapParser, MapParserConfig
+
+MAP_DATA_FILE = Path(__file__).parent / "raw_map_data"
+DEFAULT_MAP_CONFIG = MapParserConfig()
+
+
+@pytest.mark.parametrize("map_content", [b"", b"12345"])
+def test_invalid_map_content(map_content: bytes):
+    """Test that parsing map data returns the expected image and data."""
+    parser = MapParser(DEFAULT_MAP_CONFIG)
+    with pytest.raises(RoborockException, match="Failed to parse map data"):
+        parser.parse(map_content)
+
+
+# We can add additional tests here in the future that actually parse valid map data

From c32f0d53d7d7009139019ae7d0c2e7872300453c Mon Sep 17 00:00:00 2001
From: Allen Porter <allen.porter@gmail.com>
Date: Sat, 27 Sep 2025 18:49:40 -0700
Subject: [PATCH 2/2] chore: fix lint

---
 tests/map/test_map_parser.py | 1 +
 1 file changed, 1 insertion(+)

diff --git a/tests/map/test_map_parser.py b/tests/map/test_map_parser.py
index 4860afa1..4c648972 100644
--- a/tests/map/test_map_parser.py
+++ b/tests/map/test_map_parser.py
@@ -3,6 +3,7 @@
 from pathlib import Path
 
 import pytest
+
 from roborock.exceptions import RoborockException
 from roborock.map.map_parser import MapParser, MapParserConfig