k-krawczyk commented on code in PR #23614:
URL: https://github.com/apache/camel/pull/23614#discussion_r3329325873


##########
components/camel-ai/camel-openai/src/main/docs/openai-component.adoc:
##########
@@ -492,6 +492,135 @@ String-valued fields are set directly. Non-string fields 
(numbers, booleans, obj
 
 NOTE: This maps fields from the response message's additional properties 
(fields not part of the standard schema). Standard response fields like 
`content`, `role`, and `tool_calls` are not accessible through this option.
 
+== OpenAI-Compatible Providers
+
+Because the component speaks the OpenAI API, you do not need a separate 
component to use third-party

Review Comment:
   Thanks for catching the overlap. Pushed 2264fbd which:
   
   - Removes `=== Using Third-Party or Local OpenAI-Compatible Endpoint` (was 
line 259); its example is now covered by `=== LM Studio (local)` in the new 
section.
   - Removes the `== Compatibility` heading along with its intro paragraph, 
bullet list, and NOTE block (redundant with the new provider table).
   - Preserves the caveat about provider variations from the old `== 
Compatibility` NOTE as a `NOTE:` in the intro of `== OpenAI-Compatible 
Providers`.
   - Promotes `=== Embedding Models by Provider` to sit under `== 
OpenAI-Compatible Providers`, right before `== Embeddings Operation`, so all 
provider info stays grouped.
   
   Net: -24/+3 lines, single source of truth for OpenAI-compatible provider 
docs.
   
   _Reported by Claude Code on behalf of Karol Krawczyk_



-- 
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]

Reply via email to