gnodet commented on PR #25274: URL: https://github.com/apache/camel/pull/25274#issuecomment-5171671465
@zbendhiba Thank you for the thoughtful review — these are valid architectural concerns. You're right that adding configuration fields one-by-one to the SPI scales into integration tests across sub-projects (`camel-quarkus`, `camel-spring-boot`), and that users with deep LangChain4j knowledge could create the `AiServices` builder themselves via `withAiServicesCustomizer`. The broader design question — whether an `AiService` component approach would be better than continuing to mirror the builder API in `AgentConfiguration` — is worth discussing before proceeding with this PR. I'll put this on hold pending that discussion. Happy to adapt or close this PR based on the outcome. _Claude Code on behalf of @gnodet_ -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
