Skip to content

Correct schema $comment for OAD content definition#5289

Open
notEthan wants to merge 1 commit intoOAI:v3.2-devfrom
notEthan:content-object-schema-comment
Open

Correct schema $comment for OAD content definition#5289
notEthan wants to merge 1 commit intoOAI:v3.2-devfrom
notEthan:content-object-schema-comment

Conversation

@notEthan
Copy link
Copy Markdown
Contributor

@notEthan notEthan commented Apr 14, 2026

The content object is a property of Parameter, Request Body, Response, and Header. Currently its schema $comment just links to Request Body.

I'm targeting this at 3.2 instead of 3.3 because, in the linked anchors "fixed-fields-{n}", the numbers seem likely to end up different by publication. Not sure what to do about that. All of the other $comment spec links use more unique anchors.

  • schema changes are included in this pull request

@notEthan notEthan requested review from a team as code owners April 14, 2026 21:40
Comment thread src/schemas/validation/schema.yaml Outdated
Comment on lines +569 to +573
$comment: |
Parameter Object: https://spec.openapis.org/oas/v3.2#fixed-fields-9
Request Body Object: https://spec.openapis.org/oas/v3.2#fixed-fields-10
Response Object: https://spec.openapis.org/oas/v3.2#fixed-fields-14
Header Object: https://spec.openapis.org/oas/v3.2#fixed-fields-17
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.

Why not link to the corresponding "Xxx Object" sections instead of the "Fixed Fields" subsections? Those are "close enough", especially for Parameter/Header Object where content is way down in the "Fixed Fields for use with content" subsection.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That does seem better, thanks for the suggestion. Pushed.

…, Request Body, Response, Header (not just Request Body)
@notEthan notEthan force-pushed the content-object-schema-comment branch from 277abba to d5c847d Compare April 17, 2026 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants