Skip to content

fix: support x-oapi-codegen-extra-tags on parameter schemas (#2232)#2235

Merged
mromaszewicz merged 1 commit intooapi-codegen:mainfrom
mromaszewicz:fix/issue-2232
Feb 24, 2026
Merged

fix: support x-oapi-codegen-extra-tags on parameter schemas (#2232)#2235
mromaszewicz merged 1 commit intooapi-codegen:mainfrom
mromaszewicz:fix/issue-2232

Conversation

@mromaszewicz
Copy link
Member

When x-oapi-codegen-extra-tags was placed on the schema inside a parameter definition (rather than on the parameter itself), the extension was silently ignored. This happened because GenerateParamsTypes only read parameter-level extensions, not schema-level ones.

Merge extensions from both the parameter and its schema when building the Params struct, with parameter-level extensions taking precedence.

…egen#2232)

When x-oapi-codegen-extra-tags was placed on the schema inside a
parameter definition (rather than on the parameter itself), the
extension was silently ignored. This happened because
GenerateParamsTypes only read parameter-level extensions, not
schema-level ones.

Merge extensions from both the parameter and its schema when building
the Params struct, with parameter-level extensions taking precedence.

Co-Authored-By: Claude Opus 4.6 <[email protected]>
@mromaszewicz mromaszewicz requested a review from a team as a code owner February 24, 2026 01:00
@greptile-apps
Copy link

greptile-apps bot commented Feb 24, 2026

Required label not found on this PR.

@mromaszewicz mromaszewicz merged commit 12a9b20 into oapi-codegen:main Feb 24, 2026
26 checks passed
@jamietanna jamietanna added the bug Something isn't working label Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants