Skip to content

fix(apidocs): Correct schema declarations on unpublished endpoints - #129568

Merged
billyvg merged 2 commits into
masterfrom
billyvg/fix/unpublished-endpoint-schemas
Nov 3, 2026
Merged

billyvg merged 2 commits into
masterfrom
billyvg/fix/unpublished-endpoint-schemas

Conversation

@billyvg

@billyvg billyvg commented Nov 2, 2026

Copy link
Copy Markdown
Member

Why?

Five unpublished methods declare responses or request bodies that drf-spectacular can't describe. The published build skips unpublished methods, so nothing has caught them. A build that describes every method does: the csrf endpoint makes its spec invalid, and the other four come out wrong or untyped. This is split out of the internal-spec PR below, which needs it first, so each endpoint's owners can review their change on its own.

How?

Each declaration now keys responses by status code and describes TypedDicts through an inline response serializer, as published endpoints do. The SCIM replace body gets its own component name. The published spec is byte-identical.

Generated with Claude Code

These methods declare responses or request bodies that drf-spectacular can't describe
correctly. The published build skips them, being unpublished, but a build that describes
every method gets an invalid spec from the csrf endpoint, whose responses are keyed by field
names, and wrong or untyped shapes from the rest: a request body sharing another's component
name, a response type that isn't a TypedDict, a bare object response, and a TypedDict passed
where an inline response serializer belongs.
@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label Nov 2, 2026
billyvg added a commit that referenced this pull request Nov 2, 2026
They're in #129568, which lands first; the internal build needs its csrf fix to pass
--validate.
@billyvg
billyvg marked this pull request as ready for review November 2, 2026 22:20
@billyvg
billyvg requested review from a team as code owners November 2, 2026 22:20
@linear-code

linear-code Bot commented Nov 3, 2026

Copy link
Copy Markdown

ENG-8959

@billyvg
billyvg enabled auto-merge (squash) November 3, 2026 19:35
@billyvg
billyvg merged commit 40be943 into master Nov 3, 2026
76 checks passed
@billyvg
billyvg deleted the billyvg/fix/unpublished-endpoint-schemas branch November 3, 2026 19:41

This branch was successfully deployed

1 active deployment
Preview — da1f6886 Deployed Nov 3, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants