fix: list_client_secrets returns raw list, not paginated response

The API endpoint returns a plain JSON array for client secrets, not a
paginated response object. Updated to match the pattern used by other
raw-list endpoints (e.g. fga.batch_check, vault.list_object_versions).

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

Author: blackmad-cradle

src/workos/connect.py

@@ -1,12 +1,8 @@
-from functools import partial
 from typing import Optional, Protocol, Sequence
 
 from workos.types.connect import ClientSecret, ConnectApplication
 from workos.types.connect.connect_application import ApplicationType
-from workos.types.connect.list_filters import (
-    ClientSecretListFilters,
-    ConnectApplicationListFilters,
-)
+from workos.types.connect.list_filters import ConnectApplicationListFilters
 from workos.types.list_resource import ListMetadata, ListPage, WorkOSListResource
 from workos.typing.sync_or_async import SyncOrAsync
 from workos.utils.http_client import AsyncHTTPClient, SyncHTTPClient
@@ -26,10 +22,6 @@
     ConnectApplication, ConnectApplicationListFilters, ListMetadata
 ]
 
-ClientSecretsListResource = WorkOSListResource[
-    ClientSecret, ClientSecretListFilters, ListMetadata
-]
-
 
 class ConnectModule(Protocol):
     """Offers methods through the WorkOS Connect service."""
@@ -145,25 +137,14 @@ def create_client_secret(self, application_id: str) -> SyncOrAsync[ClientSecret]
     def list_client_secrets(
         self,
         application_id: str,
-        *,
-        limit: int = DEFAULT_LIST_RESPONSE_LIMIT,
-        before: Optional[str] = None,
-        after: Optional[str] = None,
-        order: PaginationOrder = "desc",
-    ) -> SyncOrAsync[ClientSecretsListResource]:
+    ) -> SyncOrAsync[Sequence[ClientSecret]]:
         """List client secrets for a connect application.
 
         Args:
             application_id (str): Application ID or client ID.
 
-        Kwargs:
-            limit (int): Maximum number of records to return. (Optional)
-            before (str): Pagination cursor to receive records before a provided ID. (Optional)
-            after (str): Pagination cursor to receive records after a provided ID. (Optional)
-            order (Literal["asc","desc"]): Sort records in either ascending or descending order. (Optional)
-
         Returns:
-            ClientSecretsListResource: Client secrets list response from WorkOS.
+            Sequence[ClientSecret]: Client secrets for the application.
         """
         ...
 
@@ -297,30 +278,13 @@ def create_client_secret(self, application_id: str) -> ClientSecret:
     def list_client_secrets(
         self,
         application_id: str,
-        *,
-        limit: int = DEFAULT_LIST_RESPONSE_LIMIT,
-        before: Optional[str] = None,
-        after: Optional[str] = None,
-        order: PaginationOrder = "desc",
-    ) -> ClientSecretsListResource:
-        list_params: ClientSecretListFilters = {
-            "limit": limit,
-            "before": before,
-            "after": after,
-            "order": order,
-        }
-
+    ) -> Sequence[ClientSecret]:
         response = self._http_client.request(
             f"{CONNECT_APPLICATIONS_PATH}/{application_id}/client_secrets",
             method=REQUEST_METHOD_GET,
-            params=list_params,
         )
 
-        return WorkOSListResource[ClientSecret, ClientSecretListFilters, ListMetadata](
-            list_method=partial(self.list_client_secrets, application_id),
-            list_args=list_params,
-            **ListPage[ClientSecret](**response).model_dump(),
-        )
+        return [ClientSecret.model_validate(secret) for secret in response]
 
     def delete_client_secret(self, client_secret_id: str) -> None:
         self._http_client.request(
@@ -447,30 +411,13 @@ async def create_client_secret(self, application_id: str) -> ClientSecret:
     async def list_client_secrets(
         self,
         application_id: str,
-        *,
-        limit: int = DEFAULT_LIST_RESPONSE_LIMIT,
-        before: Optional[str] = None,
-        after: Optional[str] = None,
-        order: PaginationOrder = "desc",
-    ) -> ClientSecretsListResource:
-        list_params: ClientSecretListFilters = {
-            "limit": limit,
-            "before": before,
-            "after": after,
-            "order": order,
-        }
-
+    ) -> Sequence[ClientSecret]:
         response = await self._http_client.request(
             f"{CONNECT_APPLICATIONS_PATH}/{application_id}/client_secrets",
             method=REQUEST_METHOD_GET,
-            params=list_params,
         )
 
-        return WorkOSListResource[ClientSecret, ClientSecretListFilters, ListMetadata](
-            list_method=partial(self.list_client_secrets, application_id),
-            list_args=list_params,
-            **ListPage[ClientSecret](**response).model_dump(),
-        )
+        return [ClientSecret.model_validate(secret) for secret in response]
 
     async def delete_client_secret(self, client_secret_id: str) -> None:
         await self._http_client.request(

tests/test_connect.py

@@ -45,17 +45,7 @@ def mock_client_secret(self):
 
     @pytest.fixture
     def mock_client_secrets(self):
-        secret_list = [MockClientSecret(id=f"cs_{i}").dict() for i in range(10)]
-        return {
-            "data": secret_list,
-            "list_metadata": {"before": None, "after": None},
-            "object": "list",
-        }
-
-    @pytest.fixture
-    def mock_client_secrets_multiple_data_pages(self):
-        secrets_list = [MockClientSecret(id=f"cs_{i + 1}").dict() for i in range(40)]
-        return list_response_of(data=secrets_list)
+        return [MockClientSecret(id=f"cs_{i}").dict() for i in range(10)]
 
     # --- Application Tests ---
 
@@ -250,9 +240,7 @@ def test_list_client_secrets(
         assert request_kwargs["url"].endswith(
             "/connect/applications/app_01ABC/client_secrets"
         )
-        assert (
-            list(map(lambda x: x.dict(), response.data)) == mock_client_secrets["data"]
-        )
+        assert [secret.dict() for secret in response] == mock_client_secrets
 
     def test_delete_client_secret(self, capture_and_mock_http_client_request):
         request_kwargs = capture_and_mock_http_client_request(
@@ -269,29 +257,3 @@ def test_delete_client_secret(self, capture_and_mock_http_client_request):
         assert request_kwargs["url"].endswith("/connect/client_secrets/cs_01ABC")
         assert request_kwargs["method"] == "delete"
         assert response is None
-
-    def test_list_client_secrets_auto_pagination_for_single_page(
-        self,
-        mock_client_secrets,
-        test_auto_pagination: TestAutoPaginationFunction,
-    ):
-        test_auto_pagination(
-            http_client=self.http_client,
-            list_function=self.connect.list_client_secrets,
-            expected_all_page_data=mock_client_secrets["data"],
-            list_function_params={"application_id": "app_01ABC"},
-            url_path_keys=["application_id"],
-        )
-
-    def test_list_client_secrets_auto_pagination_for_multiple_pages(
-        self,
-        mock_client_secrets_multiple_data_pages,
-        test_auto_pagination: TestAutoPaginationFunction,
-    ):
-        test_auto_pagination(
-            http_client=self.http_client,
-            list_function=self.connect.list_client_secrets,
-            expected_all_page_data=mock_client_secrets_multiple_data_pages["data"],
-            list_function_params={"application_id": "app_01ABC"},
-            url_path_keys=["application_id"],
-        )