fix: resolve pre-commit lint issues (black, flake8, pylint)

- Reformat with black (line-length 79)
- Fix extra blank line in transport.py
- Wrap docstrings within 79 chars
- Suppress pylint missing-kwoa in intentional negative test

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: foleydang

dashscope/agentstudio/resources/agents.py

@@ -6,15 +6,22 @@
 
 from typing import Any, Mapping, Optional, Sequence
 
-from dashscope.agentstudio.pagination import (AsyncCursorPage, CursorPage,
-                                              build_page)
-from dashscope.agentstudio.resources._helpers import (_coerce_agent,
-                                                      _coerce_agent_version)
+from dashscope.agentstudio.pagination import (
+    AsyncCursorPage,
+    CursorPage,
+    build_page,
+)
+from dashscope.agentstudio.resources._helpers import (
+    _coerce_agent,
+    _coerce_agent_version,
+)
 from dashscope.agentstudio.types import Agent, AgentVersion
-from dashscope.agentstudio.types.params import (AgentCreateParams,
-                                                AgentListParams,
-                                                AgentUpdateParams,
-                                                AgentVersionListParams)
+from dashscope.agentstudio.types.params import (
+    AgentCreateParams,
+    AgentListParams,
+    AgentUpdateParams,
+    AgentVersionListParams,
+)
 
 _PATH_AGENTS = "/agents"
 
@@ -86,9 +93,9 @@ def update(
     ) -> Agent:
         """Update the latest version of an agent.
 
-        ``version`` must equal the server's current version; the server rejects mismatches with
-        HTTP 409. Call ``retrieve(agent_id)`` first to obtain the current version — the SDK does
-        not auto-resolve it.
+        ``version`` must equal the server's current version;
+        the server rejects mismatches with HTTP 409.
+        Retrieve the agent first to obtain the current version.
         """
         body = AgentUpdateParams(
             version=version,
@@ -239,9 +246,9 @@ async def update(
     ) -> Agent:
         """Update the latest version of an agent.
 
-        ``version`` must equal the server's current version; the server rejects mismatches with
-        HTTP 409. Call ``retrieve(agent_id)`` first to obtain the current version — the SDK does
-        not auto-resolve it.
+        ``version`` must equal the server's current version;
+        the server rejects mismatches with HTTP 409.
+        Retrieve the agent first to obtain the current version.
         """
         body = AgentUpdateParams(
             version=version,

dashscope/agentstudio/transport.py

@@ -131,7 +131,6 @@ def is_error_payload(payload: Any) -> bool:
     return False
 
 
-
 def _resolve_timeout(
     timeout: Union[float, httpx.Timeout, Tuple[float, float], None],
 ) -> httpx.Timeout:
@@ -377,16 +376,16 @@ def request(  # pylint: disable=too-many-branches
                     else:
                         return resp
                 if _should_retry(
-                    resp.status_code, resp.headers,
+                    resp.status_code,
+                    resp.headers,
                 ):
                     last_exc = exceptions.APIStatusError(
                         f"HTTP {resp.status_code}",
                         status_code=resp.status_code,
                     )
                     if attempt < self.max_retries:
-                        wait = (
-                            _get_retry_after(resp.headers)
-                            or _backoff(attempt)
+                        wait = _get_retry_after(resp.headers) or _backoff(
+                            attempt
                         )
                         resp.close()
                         time.sleep(wait)
@@ -601,16 +600,16 @@ async def request(  # pylint: disable=too-many-branches
                     else:
                         return resp
                 if _should_retry(
-                    resp.status_code, resp.headers,
+                    resp.status_code,
+                    resp.headers,
                 ):
                     last_exc = exceptions.APIStatusError(
                         f"HTTP {resp.status_code}",
                         status_code=resp.status_code,
                     )
                     if attempt < self.max_retries:
-                        wait = (
-                            _get_retry_after(resp.headers)
-                            or _backoff(attempt)
+                        wait = _get_retry_after(resp.headers) or _backoff(
+                            attempt
                         )
                         await resp.aclose()
                         await asyncio.sleep(wait)

tests/unit/test_agentstudio_protocol.py

@@ -20,9 +20,13 @@
 
 from dashscope.agentstudio import exceptions
 from dashscope.agentstudio.transport import is_error_payload, unwrap
-from dashscope.agentstudio.types import (user_custom_tool_result,
-                                         user_define_outcome, user_interrupt,
-                                         user_message, user_tool_confirmation)
+from dashscope.agentstudio.types import (
+    user_custom_tool_result,
+    user_define_outcome,
+    user_interrupt,
+    user_message,
+    user_tool_confirmation,
+)
 
 # ---------------------------------------------------------------------------
 # 1. error.{code, message}
@@ -303,16 +307,24 @@ def _client_with_recording_transport():
 
 
 def test_agents_update_requires_version_kwarg():
-    """version is a required keyword-only argument — omitting it is a TypeError."""
+    """version is required — omitting it is a TypeError."""
     client = _client_with_recording_transport()
     with pytest.raises(TypeError):
-        client.agents.update("agent_1", name="new-name")  # type: ignore[call-arg]
+        # pylint: disable=missing-kwoa
+        client.agents.update(  # type: ignore[call-arg]
+            "agent_1",
+            name="new-name",
+        )
 
 
 def test_agents_update_with_version_sends_body():
-    """update(agent_id, version=...) sends POST /agents/{id} with version in body."""
+    """update() sends POST /agents/{id} with version."""
     client = _client_with_recording_transport()
-    client.agents.update("agent_1", version=3, name="new-name")
+    client.agents.update(
+        "agent_1",
+        version=3,
+        name="new-name",
+    )
     assert len(client.transport.calls) == 1
     call = client.transport.calls[0]
     assert call["method"] == "POST"
@@ -323,14 +335,12 @@ def test_agents_update_with_version_sends_body():
 
 
 def test_agents_update_does_not_auto_retrieve():
-    """SDK must NOT call retrieve() internally — caller manages version now.
-
-    Mirrors the Anthropic Managed Agents SDK: the server only accepts updates
-    against the latest version (409 on mismatch), so the caller is responsible
-    for retrieving the current version first.
-    """
+    """SDK must NOT call retrieve() internally."""
     client = _client_with_recording_transport()
-    client.agents.update("agent_1", version=3, name="new-name")
-    # Only the POST should be recorded, no preceding GET.
+    client.agents.update(
+        "agent_1",
+        version=3,
+        name="new-name",
+    )
     assert len(client.transport.calls) == 1
     assert client.transport.calls[0]["method"] == "POST"