atiaomar1978-hub commented on PR #1918: URL: https://github.com/apache/camel-spring-boot/pull/1918#issuecomment-5469888961
@davsclaus Thanks for the review — all feedback has been addressed in commit 9892b0c. **Summary of changes:** 1. **New `camel-ai-observability-starter` module** — GenAI observability auto-configuration moved out of `core/camel-spring-boot` into a dedicated starter under `components-starter/`, matching the `camel-opentelemetry2-starter` precedent. The mandatory `camel-ai-observability-api` dependency was removed from core. 2. **Removed redundant PropertiesComponent wiring** — no longer calls `setCamelContext()` / `setPropertiesComponent()` since `CamelAutoConfiguration` already handles this. 3. **Aligned with camel-main** — writes `camel.aiObservability.enabled` to PropertiesComponent **local properties** instead of override properties. 4. **Replaced marker bean** — uses a `CamelContextConfiguration` bean with `beforeApplicationStart()` instead of an empty marker class. Replies posted on each inline thread. Ready for another look when you have time. _AI-generated on behalf of atiaomar1978-hub_ -- 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]
