fix: honor the retries option

SeamHttpClient assigned self.retries after calling
niquests.Session.__init__, by which point the session had already mounted
its adapters with the default max_retries. The retries option was
therefore silently ignored: a caller passing Retry(total=5,
status_forcelist=[503]) still got exactly one attempt.

Pass retries through to niquests.Session so the mounted adapters are
built with it. Seam and SeamMultiWorkspace default the option to None,
which now falls back to DEFAULT_RETRIES rather than being dropped.

Drops the xfail markers from the two retry tests that covered this.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_011DzapiU8A9NMdyoTybL9xB

Author: claude

seam/client.py

@@ -47,16 +47,18 @@ def __init__(
         retries: Optional[Retry] = DEFAULT_RETRIES,
         **kwargs
     ):
-        super().__init__(**kwargs)
+        # niquests.Session mounts its adapters while initializing, so retries
+        # must be passed through here. Assigning self.retries afterwards leaves
+        # the mounted adapters on their default and the option has no effect.
+        super().__init__(
+            retries=DEFAULT_RETRIES if retries is None else retries, **kwargs
+        )
 
         self.base_url = base_url
 
         headers = {**auth_headers, **kwargs.get("headers", {}), **SDK_HEADERS}
         self.headers.update(headers)
 
-        if retries:
-            self.retries = retries
-
     def request(self, method, url, *args, **kwargs):
         url = urljoin(self.base_url, url)
         response = super().request(method, url, *args, **kwargs)

test/retry_test.py

@@ -7,17 +7,7 @@
 SERVICE_UNAVAILABLE = (503, "Service Unavailable")
 DEVICES = (200, {"devices": [{"device_id": "august_device_1"}]})
 
-# The retries option is currently ignored: SeamHttpClient sets self.retries after
-# calling niquests.Session.__init__ without forwarding it, so the mounted
-# HTTPAdapter keeps its default max_retries. Tests that depend on the option
-# being honored are marked xfail until seam/client.py passes retries through.
-retries_are_ignored = pytest.mark.xfail(
-    strict=True,
-    reason="SeamHttpClient does not forward retries to niquests.Session.",
-)
 
-
-@retries_are_ignored
 def test_seam_retries_service_unavailable_responses(recording_server):
     expected_retry_count = 2
     responses = [SERVICE_UNAVAILABLE, SERVICE_UNAVAILABLE, DEVICES]
@@ -34,7 +24,6 @@ def test_seam_retries_service_unavailable_responses(recording_server):
     assert len(requests) == expected_retry_count + 1
 
 
-@retries_are_ignored
 def test_seam_stops_retrying_once_retries_are_exhausted(recording_server):
     expected_retry_count = 1