refactor: remove redundant map content parsing methods and update calls to use the converter directly.

Author: allenporter

roborock/devices/traits/v1/home.py

@@ -93,7 +93,7 @@ async def discover_home(self) -> None:
             self._discovery_completed = True
             try:
                 self._home_map_content = {
-                    k: self._map_content.parse_map_content(base64.b64decode(v))
+                    k: self._map_content.converter.parse_map_content(base64.b64decode(v))
                     for k, v in (device_cache_data.home_map_content_base64 or {}).items()
                 }
             except (ValueError, RoborockException) as ex:

roborock/devices/traits/v1/map_content.py

@@ -90,26 +90,3 @@ def __init__(self, map_parser_config: MapParserConfig | None = None) -> None:
         """Initialize MapContentTrait."""
         super().__init__()
         self.converter = MapContentConverter(MapParser(map_parser_config or MapParserConfig()))
-
-    def _convert(self, response: common.V1ResponseData) -> MapContent:
-        """Parse the response from the device into a MapContentTrait instance."""
-        if not isinstance(response, bytes):
-            raise ValueError(f"Unexpected MapContentTrait response format: {type(response)}")
-        return self.parse_map_content(response)
-
-    def parse_map_content(self, response: bytes) -> MapContent:
-        """Parse the map content from raw bytes.
-
-        This method is exposed so that cached map data can be parsed without
-        needing to go through the RPC channel.
-
-        Args:
-            response: The raw bytes of the map data from the API.
-
-        Returns:
-            MapContent: The parsed map content.
-
-        Raises:
-            RoborockException: If the map data cannot be parsed.
-        """
-        return self.converter.parse_map_content(response)