Support for x-field-extra-annotation

[albertopneto]

Hi folks, I am trying to add a custom annotation for a property that exists in a DTO class, however I'm no able to make the x-field-extra-annotation to work.

that x-field-extra-annotation works fine when using open api generator with maven plugin.

Seems that the quarkus-generator-api ignores it...

is that a known issue ?

[ricardozanini]

It's not an issue, but a feature we haven't implemented yet. Feel free to send a PR!

[lordvlad]

Is there any way to easily hook into the generation process? Or will the PR require a generic Hook mechanism to process extensions?

[lordvlad]

Looking at these, it feels like in pojo.qute at least, it should have been x-field-extra-annotations, or am I wrong?

https://github.com/search?q=repo%3Aquarkiverse%2Fquarkus-openapi-generator%20x-extra-annotation&type=code

[lordvlad]

I can confirm it works with x-extra-annotation instead of x-extra-field-annotation

[ricardozanini]

@lordvlad, thanks for confirming it. I'm closing this discussion as answered.

[lordvlad]

But is it though? Shouldn't it be changed to accept `x-field-extra-annotations` instead? Ricardo Zanini ***@***.***> schrieb am Mo., 14. Apr. 2025, 18:01:

…

Closed #838 <#838> as resolved. — Reply to this email directly, view it on GitHub <#838>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAJJJ2OGSJEFZTPVV4HDRRD2ZPLVHAVCNFSM6AAAAABRAUWLP6VHI2DSMVQWIX3LMV45UABFIRUXGY3VONZWS33OIV3GK3TUHI5E433UNFTGSY3BORUW63R3GE4DSMBQGY4Q> . You are receiving this because you were mentioned.Message ID: <quarkiverse/quarkus-openapi-generator/repo-discussions/838/discussion_event/1890069 @github.com>

[ricardozanini]

@lordvlad Not sure. What tells the openapi-generator docs? If it's x-field-extra-annotations, so yes you may send a PR changing it. 👍

[lordvlad]

If I read this right, x-extra-annotation goes on the getter, while x-field-extra-annotation goes on the field itself, so technically, both extensions could be applied separately. For my use-case, I actually need it on the getter, so current implementation is sufficient. Sorry for the stir-up

[ricardozanini]

No worries, thanks for bringing this in.