fix: address python browser routing ci follow-ups

Make the browser routing helpers type-check cleanly in CI, keep copy() signatures aligned with __init__, and avoid cache-warming errors on raw response wrappers.

Made-with: Cursor

Author: rgarcia

examples/browser_routing.py

@@ -17,7 +17,7 @@ def main() -> None:
             response = cast(httpx.Response, browsers.request(browser.session_id, "GET", "https://example.com"))
             print("status", response.status_code)
 
-            with cast(Any, browsers.stream(browser.session_id, "GET", "https://example.com")) as streamed:
+            with browsers.stream(browser.session_id, "GET", "https://example.com") as streamed:
                 print("streamed-bytes", len(streamed.read()))
         finally:
             browsers.delete_by_id(browser.session_id)

src/kernel/lib/browser_routing/raw_http.py

@@ -1,24 +1,26 @@
 from __future__ import annotations
 
-from typing import IO, Any, Mapping, cast
+from typing import IO, Any, Union, Mapping, cast
 from contextlib import contextmanager, asynccontextmanager
 from collections.abc import Iterable, Iterator, AsyncIterator
 
 import httpx
 
 from .util import sanitize_curl_raw_params
 from .routing import BrowserRoute
-from ..._types import Body, Timeout, NotGiven, BinaryTypes, not_given
+from ..._types import Body, Timeout, NotGiven, not_given
 from ..._models import FinalRequestOptions
 
+BrowserRawContent = Union[bytes, bytearray, memoryview, str, IO[bytes], Iterable[bytes]]
+
 
 def request_via_browser_route(
     parent: Any,
     route: BrowserRoute,
     method: str,
     url: str,
     *,
-    content: BinaryTypes | None = None,
+    content: BrowserRawContent | None = None,
     json: Body | None = None,
     headers: Mapping[str, str] | None = None,
     params: Mapping[str, object] | None = None,
@@ -34,7 +36,7 @@ def request_via_browser_route(
         headers=headers or {},
         content=_normalize_binary_content(content),
         json_data=json,
-        timeout=timeout,
+        timeout=_normalize_timeout(timeout),
     )
     return cast(httpx.Response, parent.request(httpx.Response, options))
 
@@ -46,7 +48,7 @@ def stream_via_browser_route(
     method: str,
     url: str,
     *,
-    content: BinaryTypes | None = None,
+    content: BrowserRawContent | None = None,
     headers: Mapping[str, str] | None = None,
     params: Mapping[str, object] | None = None,
     timeout: float | Timeout | None | NotGiven = not_given,
@@ -78,7 +80,7 @@ async def async_request_via_browser_route(
     method: str,
     url: str,
     *,
-    content: BinaryTypes | None = None,
+    content: BrowserRawContent | None = None,
     json: Body | None = None,
     headers: Mapping[str, str] | None = None,
     params: Mapping[str, object] | None = None,
@@ -94,7 +96,7 @@ async def async_request_via_browser_route(
         headers=headers or {},
         content=_normalize_binary_content(content),
         json_data=json,
-        timeout=timeout,
+        timeout=_normalize_timeout(timeout),
     )
     return cast(httpx.Response, await parent.request(httpx.Response, options))
 
@@ -106,7 +108,7 @@ async def async_stream_via_browser_route(
     method: str,
     url: str,
     *,
-    content: BinaryTypes | None = None,
+    content: BrowserRawContent | None = None,
     headers: Mapping[str, str] | None = None,
     params: Mapping[str, object] | None = None,
     timeout: float | Timeout | None | NotGiven = not_given,
@@ -136,7 +138,7 @@ def _normalize_timeout(timeout: float | Timeout | None | NotGiven) -> float | Ti
     return None if isinstance(timeout, NotGiven) else timeout
 
 
-def _normalize_binary_content(content: BinaryTypes | None) -> bytes | IO[bytes] | Iterable[bytes] | None:
+def _normalize_binary_content(content: BrowserRawContent | None) -> bytes | str | IO[bytes] | Iterable[bytes] | None:
     if content is None:
         return None
     if isinstance(content, bytearray):

src/kernel/lib/browser_routing/routing.py

@@ -1,6 +1,6 @@
 from __future__ import annotations
 
-from typing import Any, Mapping, cast
+from typing import Any
 from dataclasses import field, dataclass
 
 import httpx
@@ -95,8 +95,7 @@ def rewrite_direct_vm_options(
     rewritten.url = f"{route.base_url.rstrip('/')}/{subresource}{suffix}"
 
     params: dict[str, object] = {}
-    if isinstance(options.params, Mapping):
-        params.update(cast(Mapping[str, object], options.params))
+    params.update(options.params)
     params["jwt"] = route.jwt
     rewritten.params = params or options.params
     return rewritten

src/kernel/resources/browsers/browsers.py

@@ -999,7 +999,8 @@ def list(
             ),
             model=BrowserListResponse,
         )
-        for item in page.items:
+        typed_page = cast(AsyncOffsetPagination[BrowserListResponse], page)
+        for item in typed_page.items:
             route = browser_route_from_browser(item)
             if route is not None:
                 self._client.browser_route_cache.set(route)