Preserve explicit session upload state errors

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

Author: cursoragent

hyperbrowser/client/managers/async_manager/session.py

@@ -136,6 +136,8 @@ async def upload_file(
         else:
             try:
                 read_method = getattr(file_input, "read", None)
+            except HyperbrowserError:
+                raise
             except Exception as exc:
                 raise HyperbrowserError(
                     "file_input file-like object state is invalid",
@@ -144,6 +146,8 @@ async def upload_file(
             if callable(read_method):
                 try:
                     is_closed = bool(getattr(file_input, "closed", False))
+                except HyperbrowserError:
+                    raise
                 except Exception as exc:
                     raise HyperbrowserError(
                         "file_input file-like object state is invalid",

hyperbrowser/client/managers/sync_manager/session.py

@@ -128,6 +128,8 @@ def upload_file(
         else:
             try:
                 read_method = getattr(file_input, "read", None)
+            except HyperbrowserError:
+                raise
             except Exception as exc:
                 raise HyperbrowserError(
                     "file_input file-like object state is invalid",
@@ -136,6 +138,8 @@ def upload_file(
             if callable(read_method):
                 try:
                     is_closed = bool(getattr(file_input, "closed", False))
+                except HyperbrowserError:
+                    raise
                 except Exception as exc:
                     raise HyperbrowserError(
                         "file_input file-like object state is invalid",

tests/test_session_upload_file.py

@@ -169,6 +169,39 @@ def read(self):
         manager.upload_file("session_123", _BrokenFileLike())
 
 
+def test_sync_session_upload_file_preserves_hyperbrowser_read_state_errors():
+    manager = SyncSessionManager(_FakeClient(_SyncTransport()))
+
+    class _BrokenFileLike:
+        @property
+        def read(self):
+            raise HyperbrowserError("custom read state error")
+
+    with pytest.raises(HyperbrowserError, match="custom read state error") as exc_info:
+        manager.upload_file("session_123", _BrokenFileLike())
+
+    assert exc_info.value.original_error is None
+
+
+def test_sync_session_upload_file_preserves_hyperbrowser_closed_state_errors():
+    manager = SyncSessionManager(_FakeClient(_SyncTransport()))
+
+    class _BrokenFileLike:
+        def read(self):
+            return b"content"
+
+        @property
+        def closed(self):
+            raise HyperbrowserError("custom closed-state error")
+
+    with pytest.raises(
+        HyperbrowserError, match="custom closed-state error"
+    ) as exc_info:
+        manager.upload_file("session_123", _BrokenFileLike())
+
+    assert exc_info.value.original_error is None
+
+
 def test_async_session_upload_file_rejects_non_callable_read_attribute():
     manager = AsyncSessionManager(_FakeClient(_AsyncTransport()))
     fake_file = type("FakeFile", (), {"read": "not-callable"})()
@@ -209,6 +242,45 @@ async def run():
     asyncio.run(run())
 
 
+def test_async_session_upload_file_preserves_hyperbrowser_read_state_errors():
+    manager = AsyncSessionManager(_FakeClient(_AsyncTransport()))
+
+    class _BrokenFileLike:
+        @property
+        def read(self):
+            raise HyperbrowserError("custom read state error")
+
+    async def run():
+        with pytest.raises(
+            HyperbrowserError, match="custom read state error"
+        ) as exc_info:
+            await manager.upload_file("session_123", _BrokenFileLike())
+        assert exc_info.value.original_error is None
+
+    asyncio.run(run())
+
+
+def test_async_session_upload_file_preserves_hyperbrowser_closed_state_errors():
+    manager = AsyncSessionManager(_FakeClient(_AsyncTransport()))
+
+    class _BrokenFileLike:
+        def read(self):
+            return b"content"
+
+        @property
+        def closed(self):
+            raise HyperbrowserError("custom closed-state error")
+
+    async def run():
+        with pytest.raises(
+            HyperbrowserError, match="custom closed-state error"
+        ) as exc_info:
+            await manager.upload_file("session_123", _BrokenFileLike())
+        assert exc_info.value.original_error is None
+
+    asyncio.run(run())
+
+
 def test_sync_session_upload_file_raises_hyperbrowser_error_for_missing_path(tmp_path):
     manager = SyncSessionManager(_FakeClient(_SyncTransport()))
     missing_path = tmp_path / "missing-file.txt"