Repository navigation
[kotlin] Add MicroProfile REST client support - #25374
Merged
wing328 merged 8 commits intoNov 2, 2026
Merged
Conversation
![cubic-dev-ai[bot]](https://avatars.githubusercontent.com/in/1082092?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 23 files
Reply to a comment to ask cubic a question or push back. It learns from your replies.
View guided diff | Re-trigger cubic
Generate Kotlin-native clients for MicroProfile runtimes with optional Mutiny APIs, preserving Kotlin nullability at service boundaries. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
rm127
force-pushed
the
feature/kotlin-microprofile-client
branch
from
October 23, 2026 06:25
9718147 to
b6ecb9b
Compare
Reject unsupported query formats instead of silently changing the contract, compile against matching MicroProfile API versions, and document Kotlin nullability accurately. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Make discriminated alternatives implement their union interfaces, reject invalid required JSON values, and include CDI discovery metadata for separate client JARs. Extend the generated sample and real-HTTP regression coverage. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 of 3 tasks
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 29 files (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Re-trigger cubic
Keep shared Jackson formatting unchanged outside MicroProfile, reject unsupported Android Room generation, and link inherited properties from concrete model documentation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Member
|
thanks for the PR i tried to build the project with your change locally (windows box) but got the following errors: can you please review the errors when you've time? |
Select generated sources with native Path components and reject empty source lists before compilation. Preserve exact generator whitespace in checked-in MicroProfile samples so CI regeneration has no diff. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Author
Of course! This should be resolved with the latest commit. |
wing328
reviewed
Oct 26, 2026
| @@ -0,0 +1,13 @@ | |||
| generatorName: kotlin | |||
| outputDir: samples/client/petstore/kotlin-jvm-microprofile-mutiny | |||
Member
There was a problem hiding this comment.
can you please add the new folder to the github workflow for testing kotlin clients?
Contributor
Author
There was a problem hiding this comment.
I've added it to the list. Is that what you meant?
Build the generated library in the Kotlin client matrix and run its real-HTTP integration suite. Use JDK 17 for this Quarkus sample while preserving JDK 11 for existing entries. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Re-trigger cubic
Keep the maintainer-requested sample list addition without a custom JDK or integration-test step. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Reply with feedback, questions, or to request a fix.
View guided diff | Re-trigger cubic
Use JDK 17 and cached Maven dependencies for the Quarkus HTTP tests instead of invoking a missing Gradle wrapper. Preserve existing Gradle jobs unchanged. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Related to #12065: adds Kotlin MicroProfile clients usable with Quarkus, not the server-generation portion of that request.
Adds
library=jvm-microprofileto the Kotlin client generator so MicroProfile applications can use Kotlin interfaces and models instead of Java platform types, while retaining their runtime's transport, authentication and header providers.Uni<T>, and explicit or class-derived configuration keys.Uni<Void?>for bodyless operations. Required nullable fields require JSON presence; required non-null fields reject explicit null, including primitives.allOfand discriminated objectoneOf/anyOfinterfaces, including alternatives shared between unions.META-INF/beans.xmlso separately built client JARs can be discovered by CDI/Quarkus.The hierarchy repair is related to #19928 and #22013, but is scoped to this new library; it does not fix those reports' Moshi/Spring configurations. Other Kotlin client libraries retain their existing behavior.
Undiscriminated unions, complete JSON Schema validation and missing-versus-null tracking are outside scope. Jackson's Kotlin module is required; strict collection-element null checking remains a runtime configuration.
Verification
130 targeted generator tests and Checkstyle pass, including a non-MicroProfile Jackson formatting regression. The generated sample has 17 passing real-HTTP Quarkus tests covering polymorphism, nullability, errors, bodyless responses and combined header-factory/header-parameter/filter decorations. Separate-JAR consumption also passes in a Quarkus acceptance harness. Generated sample changes are confined to this library.
PR checklist