fix: wrap aiohttp network errors in RoborockException (#835)

Network-class exceptions raised inside PreparedRequest.request
(aiohttp.ClientError, TimeoutError, OSError) were bubbling out
unwrapped, bypassing consumers that catch RoborockException. This
caused Home Assistant's roborock integration to mark config entries
as terminally failed after a power outage when DNS hadn't resolved
yet, requiring a manual restart to recover.

Catch the network-class exceptions at the single chokepoint
(PreparedRequest.request) and re-raise as RoborockException,
preserving the original cause via `from err`. Every method on
RoborockApiClient / UserWebApiClient benefits since they all route
through this method.

Co-authored-by: Claude Opus 4.7 <noreply@anthropic.com>

Author: TheMerinoWolf

roborock/web_api.py

@@ -692,6 +692,8 @@ async def request(self, method: str, url: str, params=None, data=None, headers=N
             _LOGGER.info("Resp raw: %s", resp_raw)
             # Still raise the err so that it's clear it failed.
             raise err
+        except (aiohttp.ClientError, TimeoutError, OSError) as err:
+            raise RoborockException(f"Network error contacting {_url}: {err}") from err
         finally:
             if close_session:
                 await session.close()

tests/test_web_api.py

@@ -7,8 +7,8 @@
 from aioresponses.compat import normalize_url
 
 from roborock import HomeData, HomeDataScene, UserData
-from roborock.exceptions import RoborockAccountDoesNotExist, RoborockInvalidCredentials
-from roborock.web_api import IotLoginInfo, RoborockApiClient, UserWebApiClient
+from roborock.exceptions import RoborockAccountDoesNotExist, RoborockException, RoborockInvalidCredentials
+from roborock.web_api import IotLoginInfo, PreparedRequest, RoborockApiClient, UserWebApiClient
 from tests.mock_data import HOME_DATA_RAW, USER_DATA
 
 pytest_plugins = [
@@ -398,3 +398,24 @@ async def test_user_web_api_client_unauthorized_hook() -> None:
     with pytest.raises(RoborockInvalidCredentials):
         await client.get_rooms()
     mock_hook.assert_called_once()
+
+
+@pytest.mark.parametrize(
+    "exception",
+    [
+        aiohttp.ClientError("connection failed"),
+        TimeoutError("timed out"),
+        OSError("dns failure"),
+    ],
+    ids=["client_error", "timeout", "os_error"],
+)
+async def test_prepared_request_wraps_network_errors(exception: Exception, mock_rest: Any) -> None:
+    """Network errors from aiohttp must be wrapped as RoborockException so
+    consumers can rely on the typed exception hierarchy."""
+    url_pattern = re.compile(r"https://example\.com/api/test.*")
+    mock_rest.post(url_pattern, exception=exception)
+
+    req = PreparedRequest("https://example.com")
+    with pytest.raises(RoborockException) as exc_info:
+        await req.request("post", "/api/test")
+    assert exc_info.value.__cause__ is exception