CAMEL-24330: Improve AI component documentation based on user feedback - #25314
CAMEL-24330: Improve AI component documentation based on user feedback#25314davsclaus wants to merge 1 commit into
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: Claus Ibsen <claus.ibsen@gmail.com>
gnodet
left a comment
There was a problem hiding this comment.
🤖 AI-assisted review — Claude Code on behalf of @gnodet
Review: CAMEL-24330 — Improve AI component documentation
Nice additions — the decision guide comparing camel-openai vs camel-langchain4j-chat, the production tips, and the cross-references between AI docs are all useful. One issue to fix:
[MINOR] Broken xref — wrong Antora module prefix
In langchain4j-chat-component.adoc, the xref uses the others: module prefix:
xref:others:ai-summary.adoc[Choosing the Right AI Component]
But ai-summary.adoc lives in the ROOT module (docs/components/modules/ROOT/pages/ai-summary.adoc) — it matches the *-summary.adoc pattern routed to ROOT by PrepareDocSymlinksMojo. Since langchain4j-chat-component.adoc is also in ROOT, this should be:
xref:ai-summary.adoc[Choosing the Right AI Component]
(Note: the others:openai-mcp.adoc xrefs in openai-component.adoc are correct since that file genuinely lives in the others module.)
Everything else looks good — the code examples are accurate and the comparison table is helpful. 👍
|
🌟 Thank you for your contribution to the Apache Camel project! 🌟 🐫 Apache Camel Committers, please review the following items:
|
Merge useful content from apache#25314: OpenAI production tips (including model output error handling and MCP streaming note), ai-summary quick comparison table, and langchain4j cross-links to OpenAI features. Co-authored-by: Cursor <cursoragent@cursor.com>
|
AI-generated comment on behalf of atiaomar1978-hub @davsclaus — per your suggestion on #25313, the useful content from this PR (OpenAI production tips, ai-summary comparison table, langchain4j cross-links) has been incorporated into #25313 in commit Happy to close this PR in favour of #25313 if that works for you. |
|
duplicate closing |
Summary
Improves the AI component documentation based on real-world user feedback from a blog post by a developer building LLM-powered pipelines with Camel. The author documented several documentation gaps they hit in practice.
Changes:
camel-openaiandcamel-langchain4j-chat, helping users pick the right component for their needsoutputClass/jsonSchema) as the recommended approach for JSON extractionTest plan
xref:links resolve to valid targetsClaude Code on behalf of davsclaus
🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.6 noreply@anthropic.com