Skip to content

fix(generated): Changes to multi_factor_auth - #732

Merged
workos-sdk-automation[bot] merged 3 commits into
mainfrom
oagen/batch-666ef9e0
Sep 16, 2026
Merged

workos-sdk-automation[bot] merged 3 commits into
mainfrom
oagen/batch-666ef9e0

Conversation

@workos-sdk-automation

@workos-sdk-automation workos-sdk-automation Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

Summary

fix(multi_factor_auth): Change errors for endpoint POST /auth/factors/{id}/challenge

  • Changed errors for endpoint POST /auth/factors/{id}/challenge.

Triggered by workos/openapi-spec@949c30c

BEGIN_COMMIT_OVERRIDE
fix(multi_factor_auth): Change errors for endpoint POST /auth/factors/{id}/challenge (#732)
END_COMMIT_OVERRIDE

…iKeys, AuditLogs, Authorization, ClientApi, Connect, DirectorySync, Events, FeatureFlags, Groups, MultiFactorAuth, OrganizationDomains, OrganizationMembership, Organizations, OrganizationsDataProviders, Pipes, PipesProvider, PlatformTeams, Radar, SSO, UserManagement, Vault, Webhooks, Widgets
@workos-sdk-automation workos-sdk-automation Bot added the autogenerated Autogenerated code or content label Sep 16, 2026
@workos-sdk-automation
workos-sdk-automation Bot requested review from a team as code owners September 16, 2026 19:24
@greptile-apps

greptile-apps Bot commented Sep 16, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

The PR is not safe to merge until the Pipes provider-list endpoint preserves compatibility with its existing public response model.

Findings

  1. P1 Pipes Response Type Changed
Fix with agent prompt
### Issue 1
src/workos/pipes/_resource.py:24-28
`list_user_data_providers` now deserializes responses into the common `DataIntegrationsListResponse`, while `workos.pipes.models.DataIntegrationsListResponse` remains a separate class. Existing consumers importing the Pipes model will receive an object of a different type, so class-identity checks and related annotations can break. Re-export or alias the common implementation from the old Pipes module so both imports resolve to the same class.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

Summary

This regeneration updates the MFA challenge documentation and incorporates broader specification changes for organization data providers and Pipes.

  • Adds synchronous and asynchronous organization data-provider resources.
  • Adds connected-account ownership metadata, plural-connection response models, and new Pipes webhook events.
  • Updates common exports, fixtures, and generated round-trip/request tests.
  • Introduces a compatibility regression by returning a common response class distinct from the retained public Pipes class.
Diagram
%%{init: {'theme': 'neutral'}}%%
flowchart LR
  C[WorkOSClient] --> O[OrganizationsDataProviders]
  C --> P[Pipes]
  O --> M[Common connected-account models]
  P --> M
  P --> R[Common DataIntegrationsListResponse]
  L[Legacy workos.pipes.models import] --> D[Separate response class]
Loading

Reviews (1) · Last reviewed commit: "chore(generated): add release notes frag..."

Comment on lines 24 to +28
DataIntegrationsGetUserTokenRequestConnectionOwner,
)
from workos.common.models.data_integrations_list_response import (
DataIntegrationsListResponse,
)

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

P1 Pipes Response Type Changed

list_user_data_providers now deserializes responses into the common DataIntegrationsListResponse, while workos.pipes.models.DataIntegrationsListResponse remains a separate class. Existing consumers importing the Pipes model will receive an object of a different type, so class-identity checks and related annotations can break. Re-export or alias the common implementation from the old Pipes module so both imports resolve to the same class.

Prompt To Fix With AI
This is a comment left during a code review.
Path: src/workos/pipes/_resource.py
Line: 24-28

Comment:
**Pipes Response Type Changed**

`list_user_data_providers` now deserializes responses into the common `DataIntegrationsListResponse`, while `workos.pipes.models.DataIntegrationsListResponse` remains a separate class. Existing consumers importing the Pipes model will receive an object of a different type, so class-identity checks and related annotations can break. Re-export or alias the common implementation from the old Pipes module so both imports resolve to the same class.

---

For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.

@workos-sdk-automation
workos-sdk-automation Bot merged commit 7d3a7f1 into main Sep 16, 2026
11 checks passed
@workos-sdk-automation
workos-sdk-automation Bot deleted the oagen/batch-666ef9e0 branch September 16, 2026 19:30
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