oscerd commented on PR #25050: URL: https://github.com/apache/camel/pull/25050#issuecomment-5058188002
Thank you for the review — all findings addressed: **High Severity (fixed):** 1. ✅ Fixed OpenAI header prefix casing: `CamelOpenAi` → `CamelOpenAI` (matching `OpenAIConstants.java`) 2. ✅ Removed non-existent `torchserve` scheme (no such Camel component) **Medium Severity (fixed):** 3. ✅ Added missing AI component schemes: `langchain4j-web-search`, `langchain4j-embeddingstore`, `ai-tool` 4. ✅ Added missing header prefixes: `CamelAwsTextract`, `CamelKServe`, `CamelDjl`, `CamelTensorFlowServing`, `CamelHuggingFace` 5. Main `camel_runtime_ai_trace` method requires a running Camel process to test — covered by manual verification. Unit tests cover the extraction/categorization logic. **Low Severity (fixed):** 6. ✅ Added `LOG.debugf()` on registry tool execution failure instead of silent swallow Added 3 new tests: OpenAI header casing, web-search/embeddingstore schemes, Textract headers. _Claude Code on behalf of oscerd_ -- 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]
