Guard status-code normalization strip failures in polling

Co-authored-by: Shri Sukhani <shrisukhani@users.noreply.github.com>

Author: cursoragent

hyperbrowser/client/polling.py

@@ -395,12 +395,15 @@ def _normalize_status_code_for_retry(status_code: object) -> Optional[int]:
         status_text = _decode_ascii_bytes_like(status_code)
 
     if status_text is not None:
-        normalized_status = status_text.strip()
-        if not normalized_status:
-            return None
-        if len(normalized_status) > _MAX_STATUS_CODE_TEXT_LENGTH:
-            return None
-        if not normalized_status.isascii() or not normalized_status.isdigit():
+        try:
+            normalized_status = status_text.strip()
+            if not normalized_status:
+                return None
+            if len(normalized_status) > _MAX_STATUS_CODE_TEXT_LENGTH:
+                return None
+            if not normalized_status.isascii() or not normalized_status.isdigit():
+                return None
+        except Exception:
             return None
         try:
             return int(normalized_status, 10)

tests/test_polling.py

@@ -1768,6 +1768,34 @@ def operation() -> str:
     assert attempts["count"] == 3
 
 
+def test_retry_operation_handles_status_code_strip_runtime_failures_as_retryable():
+    class _BrokenStatusCode(str):
+        def strip(self, chars=None):  # type: ignore[override]
+            _ = chars
+            raise RuntimeError("status code strip exploded")
+
+    attempts = {"count": 0}
+
+    def operation() -> str:
+        attempts["count"] += 1
+        if attempts["count"] < 3:
+            raise HyperbrowserError(
+                "status metadata strip failure",
+                status_code=_BrokenStatusCode("429"),  # type: ignore[arg-type]
+            )
+        return "ok"
+
+    result = retry_operation(
+        operation_name="sync retry status code strip failure",
+        operation=operation,
+        max_attempts=5,
+        retry_delay_seconds=0.0001,
+    )
+
+    assert result == "ok"
+    assert attempts["count"] == 3
+
+
 def test_retry_operation_rejects_awaitable_operation_result():
     async def async_operation() -> str:
         return "ok"
@@ -2544,6 +2572,37 @@ async def operation() -> str:
     asyncio.run(run())
 
 
+def test_retry_operation_async_handles_status_code_strip_runtime_failures_as_retryable():
+    class _BrokenStatusCode(str):
+        def strip(self, chars=None):  # type: ignore[override]
+            _ = chars
+            raise RuntimeError("status code strip exploded")
+
+    async def run() -> None:
+        attempts = {"count": 0}
+
+        async def operation() -> str:
+            attempts["count"] += 1
+            if attempts["count"] < 3:
+                raise HyperbrowserError(
+                    "status metadata strip failure",
+                    status_code=_BrokenStatusCode("429"),  # type: ignore[arg-type]
+                )
+            return "ok"
+
+        result = await retry_operation_async(
+            operation_name="async retry status code strip failure",
+            operation=operation,
+            max_attempts=5,
+            retry_delay_seconds=0.0001,
+        )
+
+        assert result == "ok"
+        assert attempts["count"] == 3
+
+    asyncio.run(run())
+
+
 def test_retry_operation_async_does_not_retry_stop_async_iteration_errors():
     async def run() -> None:
         attempts = {"count": 0}