davsclaus commented on code in PR #29720:
URL: https://github.com/apache/camel/pull/29720#discussion_r4552255251
##########
components/camel-ai/camel-docling/src/main/java/org/apache/camel/component/docling/DoclingConfiguration.java:
##########
@@ -52,7 +52,8 @@ public class DoclingConfiguration implements Cloneable {
private String ocrLanguage = "en";
@UriParam
- @Metadata(description = "Output format for document conversion",
defaultValue = "markdown",
+ @Metadata(description = "Output format for document conversion. The
html_split_page value is rejected when useDoclingServe is true, as
docling-serve returns no split pages, and the CLI operations pass their own
format; use html instead",
+ defaultValue = "markdown",
Review Comment:
Covered: the generated JSON does not advertise an enum list for this
`String` option, so the catalog does not offer `html_split_page` as a choice,
and #29594 rejects it with an exception rather than falling back to markdown.
The new description says so. Resolving.
_Claude Code on behalf of davsclaus_
--
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]
