Skip to content

feat(generated)!: regenerate from spec (6 changes)#661

Open
workos-sdk-automation[bot] wants to merge 1 commit into
mainfrom
oagen/spec-update-eb5bcc44eddeb6ad299608224cfcfcaa74db966e
Open

feat(generated)!: regenerate from spec (6 changes)#661
workos-sdk-automation[bot] wants to merge 1 commit into
mainfrom
oagen/spec-update-eb5bcc44eddeb6ad299608224cfcfcaa74db966e

Conversation

@workos-sdk-automation
Copy link
Copy Markdown
Contributor

Summary

feat(authorization)!: Remove search parameter from listResources

  • Breaking change: Removed search optional parameter from listResources() method
  • Updated both sync and async implementations to remove the search filter capability
  • Updated tests to remove search parameter validation

feat(authorization): Add resource filtering to listRoleAssignments

  • Added resource_id, resource_external_id, and resource_type_slug optional parameters to listRoleAssignments() method
  • Enables filtering role assignments by resource attributes
  • Updated both sync and async implementations

feat(authorization): Add role_slug filter to listRoleAssignmentsForResourceByExternalId

  • Added role_slug optional parameter to listRoleAssignmentsForResourceByExternalId() method
  • Enables filtering assignments by role slug for resource lookups by external ID
  • Updated both sync and async implementations

feat(authorization): Add role_slug filter to listRoleAssignmentsForResource

  • Added role_slug optional parameter to listRoleAssignmentsForResource() method
  • Enables filtering assignments by role slug for resource-specific queries
  • Updated both sync and async implementations

fix(groups): Add BadRequestError to addMember error documentation

  • Updated addMember() method docstring to document BadRequestError (400) exception
  • Indicates that the operation can now return HTTP 400 for malformed requests
  • Applied to both sync and async implementations

chore(sso): Update login_hint documentation

  • Updated docstring for login_hint parameter in SSO methods to include custom SAML connections
  • Changed from: "...OAuth, OpenID Connect, Okta, and Entra ID connections"
  • Changed to: "...OAuth, OpenID Connect, Okta, Entra ID, and custom SAML connections"
  • Documentation-only change, no API signature modification

Triggered by workos/openapi-spec@eb5bcc4

BEGIN_COMMIT_OVERRIDE
feat(authorization)!: Remove search parameter from listResources
feat(authorization): Add resource filtering to listRoleAssignments
feat(authorization): Add role_slug filter to listRoleAssignmentsForResourceByExternalId
feat(authorization): Add role_slug filter to listRoleAssignmentsForResource
fix(groups): Add BadRequestError to addMember error documentation
chore(sso): Update login_hint documentation
END_COMMIT_OVERRIDE

@workos-sdk-automation workos-sdk-automation Bot added the autogenerated Autogenerated code or content label May 18, 2026
@workos-sdk-automation workos-sdk-automation Bot requested review from a team as code owners May 18, 2026 17:54
@workos-sdk-automation workos-sdk-automation Bot requested a review from alisherry May 18, 2026 17:54
@workos-sdk-automation workos-sdk-automation Bot added the autogenerated Autogenerated code or content label May 18, 2026
@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented May 18, 2026

Greptile Summary

This is a generated SDK update regenerated from the OpenAPI spec, introducing new optional filter parameters across authorization methods and a breaking removal of the search parameter from listResources.

  • Breaking change: search parameter removed from listResources (sync + async); callers relying on it will need to update.
  • New filters: resource_id, resource_external_id, and resource_type_slug added to listRoleAssignments; role_slug added to both listRoleAssignmentsForResource and listRoleAssignmentsForResourceByExternalId — all optional and keyword-only, consistent between sync and async classes.
  • Docs-only: BadRequestError added to the addMember docstring in groups; login_hint docstring in SSO updated to include custom SAML connections.

Confidence Score: 5/5

Safe to merge. All new parameters are optional and keyword-only, parameter forwarding to query strings is consistent across sync/async pairs, and tests cover every new param as well as the removal of search.

All changes are generated from the OpenAPI spec. New optional filter params are correctly threaded through to the query-param dicts behind if-v-is-not-None guards, are consistent between sync and async classes, and have explicit test coverage. The only behavioral change — removing search from listResources — is intentional and reflected in both implementation and tests.

No files require special attention.

Important Files Changed

Filename Overview
src/workos/authorization/_resource.py Adds optional resource filter params to listRoleAssignments, adds role_slug to both listRoleAssignmentsForResource variants, and removes the search param from listResources in both sync and async classes.
src/workos/groups/_resource.py Documentation-only change: adds BadRequestError (HTTP 400) to the addMember docstring in both sync and async classes.
src/workos/sso/_resource.py Documentation-only change: updates login_hint docstring to include 'custom SAML connections' in both sync and async classes.
tests/test_authorization.py Tests updated to cover all new parameters and remove search param assertions from listResources tests, for both sync and async classes.

Sequence Diagram

sequenceDiagram
    participant Caller
    participant Authorization SDK
    participant WorkOS API

    Note over Caller, WorkOS API: listRoleAssignments (new filters)
    Caller->>Authorization SDK: list_role_assignments(org_membership_id, resource_id?, resource_external_id?, resource_type_slug?)
    Authorization SDK->>WorkOS API: GET /authorization/role-assignments?resource_id=...&resource_type_slug=...
    WorkOS API-->>Authorization SDK: SyncPage[UserRoleAssignment]
    Authorization SDK-->>Caller: SyncPage[UserRoleAssignment]

    Note over Caller, WorkOS API: listRoleAssignmentsForResource (new role_slug filter)
    Caller->>Authorization SDK: list_role_assignments_for_resource(resource_id, role_slug?)
    Authorization SDK->>WorkOS API: GET /authorization/resources/{id}/role-assignments?role_slug=...
    WorkOS API-->>Authorization SDK: SyncPage[UserRoleAssignment]
    Authorization SDK-->>Caller: SyncPage[UserRoleAssignment]

    Note over Caller, WorkOS API: listResources (search removed — breaking)
    Caller->>Authorization SDK: list_resources(organization_id?, resource_type_slug?, resource_external_id?)
    Authorization SDK->>WorkOS API: GET /authorization/resources (no search param)
    WorkOS API-->>Authorization SDK: SyncPage[AuthorizationResource]
    Authorization SDK-->>Caller: SyncPage[AuthorizationResource]
Loading

Reviews (1): Last reviewed commit: "feat(generated)!: regenerate from spec (..." | Re-trigger Greptile

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

autogenerated Autogenerated code or content

Development

Successfully merging this pull request may close these issues.

0 participants