feat(api): manual updates (#2422)

Author: stainless-app[bot]

api.md

@@ -63,15 +63,9 @@ Methods:
 
 ## Roles
 
-Types:
-
-```python
-from cloudflare.types.accounts import RoleListResponse
-```
-
 Methods:
 
-- <code title="get /accounts/{account_id}/roles">client.accounts.roles.<a href="./src/cloudflare/resources/accounts/roles.py">list</a>(\*, account_id) -> <a href="./src/cloudflare/types/accounts/role_list_response.py">Optional[RoleListResponse]</a></code>
+- <code title="get /accounts/{account_id}/roles">client.accounts.roles.<a href="./src/cloudflare/resources/accounts/roles.py">list</a>(\*, account_id) -> <a href="./src/cloudflare/types/shared/role.py">SyncSinglePage[Role]</a></code>
 - <code title="get /accounts/{account_id}/roles/{role_id}">client.accounts.roles.<a href="./src/cloudflare/resources/accounts/roles.py">get</a>(role_id, \*, account_id) -> <a href="./src/cloudflare/types/shared/role.py">Optional[Role]</a></code>
 
 ## Subscriptions

src/cloudflare/resources/accounts/roles.py

@@ -16,9 +16,9 @@
     async_to_streamed_response_wrapper,
 )
 from ..._wrappers import ResultWrapper
-from ..._base_client import make_request_options
+from ...pagination import SyncSinglePage, AsyncSinglePage
+from ..._base_client import AsyncPaginator, make_request_options
 from ...types.shared.role import Role
-from ...types.accounts.role_list_response import RoleListResponse
 
 __all__ = ["RolesResource", "AsyncRolesResource"]
 
@@ -53,7 +53,7 @@ def list(
         extra_query: Query | None = None,
         extra_body: Body | None = None,
         timeout: float | httpx.Timeout | None | NotGiven = NOT_GIVEN,
-    ) -> Optional[RoleListResponse]:
+    ) -> SyncSinglePage[Role]:
         """
         Get all available roles for an account.
 
@@ -70,16 +70,13 @@ def list(
         """
         if not account_id:
             raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
-        return self._get(
+        return self._get_api_list(
             f"/accounts/{account_id}/roles",
+            page=SyncSinglePage[Role],
             options=make_request_options(
-                extra_headers=extra_headers,
-                extra_query=extra_query,
-                extra_body=extra_body,
-                timeout=timeout,
-                post_parser=ResultWrapper[Optional[RoleListResponse]]._unwrapper,
+                extra_headers=extra_headers, extra_query=extra_query, extra_body=extra_body, timeout=timeout
             ),
-            cast_to=cast(Type[Optional[RoleListResponse]], ResultWrapper[RoleListResponse]),
+            model=Role,
         )
 
     def get(
@@ -147,7 +144,7 @@ def with_streaming_response(self) -> AsyncRolesResourceWithStreamingResponse:
         """
         return AsyncRolesResourceWithStreamingResponse(self)
 
-    async def list(
+    def list(
         self,
         *,
         account_id: str,
@@ -157,7 +154,7 @@ async def list(
         extra_query: Query | None = None,
         extra_body: Body | None = None,
         timeout: float | httpx.Timeout | None | NotGiven = NOT_GIVEN,
-    ) -> Optional[RoleListResponse]:
+    ) -> AsyncPaginator[Role, AsyncSinglePage[Role]]:
         """
         Get all available roles for an account.
 
@@ -174,16 +171,13 @@ async def list(
         """
         if not account_id:
             raise ValueError(f"Expected a non-empty value for `account_id` but received {account_id!r}")
-        return await self._get(
+        return self._get_api_list(
             f"/accounts/{account_id}/roles",
+            page=AsyncSinglePage[Role],
             options=make_request_options(
-                extra_headers=extra_headers,
-                extra_query=extra_query,
-                extra_body=extra_body,
-                timeout=timeout,
-                post_parser=ResultWrapper[Optional[RoleListResponse]]._unwrapper,
+                extra_headers=extra_headers, extra_query=extra_query, extra_body=extra_body, timeout=timeout
             ),
-            cast_to=cast(Type[Optional[RoleListResponse]], ResultWrapper[RoleListResponse]),
+            model=Role,
         )
 
     async def get(

src/cloudflare/types/accounts/__init__.py

@@ -6,7 +6,6 @@
 from .account import Account as Account
 from .token_list_params import TokenListParams as TokenListParams
 from .member_list_params import MemberListParams as MemberListParams
-from .role_list_response import RoleListResponse as RoleListResponse
 from .account_list_params import AccountListParams as AccountListParams
 from .token_create_params import TokenCreateParams as TokenCreateParams
 from .token_update_params import TokenUpdateParams as TokenUpdateParams

src/cloudflare/types/accounts/role_list_response.py

@@ -9,8 +9,8 @@
 
 from cloudflare import Cloudflare, AsyncCloudflare
 from tests.utils import assert_matches_type
+from cloudflare.pagination import SyncSinglePage, AsyncSinglePage
 from cloudflare.types.shared import Role
-from cloudflare.types.accounts import RoleListResponse
 
 base_url = os.environ.get("TEST_API_BASE_URL", "http://127.0.0.1:4010")
 
@@ -23,7 +23,7 @@ def test_method_list(self, client: Cloudflare) -> None:
         role = client.accounts.roles.list(
             account_id="eb78d65290b24279ba6f44721b3ea3c4",
         )
-        assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+        assert_matches_type(SyncSinglePage[Role], role, path=["response"])
 
     @parametrize
     def test_raw_response_list(self, client: Cloudflare) -> None:
@@ -34,7 +34,7 @@ def test_raw_response_list(self, client: Cloudflare) -> None:
         assert response.is_closed is True
         assert response.http_request.headers.get("X-Stainless-Lang") == "python"
         role = response.parse()
-        assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+        assert_matches_type(SyncSinglePage[Role], role, path=["response"])
 
     @parametrize
     def test_streaming_response_list(self, client: Cloudflare) -> None:
@@ -45,7 +45,7 @@ def test_streaming_response_list(self, client: Cloudflare) -> None:
             assert response.http_request.headers.get("X-Stainless-Lang") == "python"
 
             role = response.parse()
-            assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+            assert_matches_type(SyncSinglePage[Role], role, path=["response"])
 
         assert cast(Any, response.is_closed) is True
 
@@ -113,7 +113,7 @@ async def test_method_list(self, async_client: AsyncCloudflare) -> None:
         role = await async_client.accounts.roles.list(
             account_id="eb78d65290b24279ba6f44721b3ea3c4",
         )
-        assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+        assert_matches_type(AsyncSinglePage[Role], role, path=["response"])
 
     @parametrize
     async def test_raw_response_list(self, async_client: AsyncCloudflare) -> None:
@@ -124,7 +124,7 @@ async def test_raw_response_list(self, async_client: AsyncCloudflare) -> None:
         assert response.is_closed is True
         assert response.http_request.headers.get("X-Stainless-Lang") == "python"
         role = await response.parse()
-        assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+        assert_matches_type(AsyncSinglePage[Role], role, path=["response"])
 
     @parametrize
     async def test_streaming_response_list(self, async_client: AsyncCloudflare) -> None:
@@ -135,7 +135,7 @@ async def test_streaming_response_list(self, async_client: AsyncCloudflare) -> N
             assert response.http_request.headers.get("X-Stainless-Lang") == "python"
 
             role = await response.parse()
-            assert_matches_type(Optional[RoleListResponse], role, path=["response"])
+            assert_matches_type(AsyncSinglePage[Role], role, path=["response"])
 
         assert cast(Any, response.is_closed) is True