docs: add missing OpenAPI annotations#552
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
📝 WalkthroughWalkthroughAdds missing OpenAPI annotations to three REST interfaces: registers an ChangesOpenAPI Annotation Additions
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem
🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR improves generated OpenAPI/Swagger documentation coverage by adding missing MicroProfile OpenAPI annotations (@Operation, @Tag) to a few REST interfaces and registering a matching tag in the global OpenAPI definition.
Changes:
- Added
@Operationsummaries/descriptions to UI and conversation-store REST interfaces. - Added class-level
@Tag(name = "Authentication")to the logout/authentication REST interface. - Registered the new
Authenticationtag inOpenApiConfigso Swagger UI shows it with the intended metadata/order.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| src/main/java/ai/labs/eddi/ui/IRestHtmlChatResource.java | Adds @Operation metadata for the embedded chat UI endpoints. |
| src/main/java/ai/labs/eddi/engine/triggermanagement/IRestUserConversationStore.java | Adds @Operation metadata for read/create/delete user conversation endpoints. |
| src/main/java/ai/labs/eddi/engine/api/ILogoutEndpoint.java | Adds an Authentication tag to group auth endpoints in OpenAPI. |
| src/main/java/ai/labs/eddi/configs/OpenApiConfig.java | Registers the Authentication tag in the canonical tag taxonomy. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @GET | ||
| @Cache(noCache = true, mustRevalidate = true) | ||
| @Path("{path:.*}") | ||
| @Operation(summary = "View chat resource", description = "Serve a requested chat UI resource by path.") |
| @POST | ||
| @Path("/{intent}/{userId}") | ||
| @Produces(MediaType.APPLICATION_JSON) | ||
| @Operation(summary = "Create user conversation", description = "Create a new user conversation for intent and user id.") | ||
| Response createUserConversation(@PathParam("intent") String intent, @PathParam("userId") String userId, UserConversation userConversation); |
Summary
Added the missing OpenAPI annotations from issue #551.
IRestUserConversationStore.IRestHtmlChatResource.ILogoutEndpoint.Authenticationtag metadata inOpenApiConfig.Type of Change
Related Issue
Closes #551
Changes Made
How to Test
./mvnw compileonded-furby/EDDI.Checklist
./mvnw compile).Summary by CodeRabbit