fix: list_client_secrets returns raw list, not paginated response

[blackmad-cradle]

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).

Apologies, got this one wrong in my last MR. Interactively tested that this works.

Description

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[x] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

[greptile-apps]

Greptile Summary

This PR corrects list_client_secrets in both Connect and AsyncConnect to match the actual API contract: the endpoint returns a plain JSON array, not a paginated list object. The fix replaces the WorkOSListResource wrapper (with its pagination parameters and cursor logic) with a simple list comprehension that validates each item against the ClientSecret model.

Key changes:

• list_client_secrets signature simplified — pagination kwargs (limit, before, after, order) removed from both the ConnectModule protocol and the two concrete implementations
• Return type changed from ClientSecretsListResource (a WorkOSListResource generic) to Sequence[ClientSecret]
• ClientSecretsListResource type alias and its ClientSecretListFilters import removed from connect.py
• Tests updated: mock fixture is now a plain list; auto-pagination tests are correctly removed
• ClientSecretListFilters class in src/workos/types/connect/list_filters.py is now orphaned dead code and should be deleted as cleanup

Confidence Score: 5/5

• Safe to merge — the change is a straightforward bug fix that aligns the SDK with the actual API response shape.
• The change is minimal, well-scoped, and consistent with how other raw-list endpoints are handled in the codebase. Both sync and async paths are updated symmetrically, and the tests correctly validate the new behavior. The only leftover is the now-unused ClientSecretListFilters class in list_filters.py, which is dead code but does not affect runtime correctness.
• src/workos/types/connect/list_filters.py — ClientSecretListFilters is unused dead code and should be removed.

Comments Outside Diff (1)

1. src/workos/types/connect/list_filters.py, line 10-11 (link)

   ClientSecretListFilters is no longer used. The ClientSecretsListResource type alias that previously depended on it was removed as part of this PR, and this class is not imported anywhere else in the codebase.

_{Last reviewed commit: 5a04c49}