davsclaus commented on code in PR #27839:
URL: https://github.com/apache/camel/pull/27839#discussion_r4279111120
##########
dsl/camel-jbang/camel-jbang-core/src/main/java/org/apache/camel/dsl/jbang/core/commands/ai/AuthoringTools.java:
##########
@@ -103,11 +103,9 @@ static void register(Consumer<ToolDescriptor> registry) {
"common (default: no deprecated or advanced),
required, all or false", false)
.param("includeHeaders", "boolean", "Include the message
headers of a component (default false)", false)
.param("includeDoc", "boolean", "Include the full AsciiDoc
page (default false)", false)
- .param("docPage", "string", "simple doc sub-page to return as
text (functions, operators, ognl, advanced)",
- false)
- .param("optionsFilter", "string",
- "Keyword to match in option names or descriptions; for
an EIP whose own options do not match, the options of its elements are searched
(nestedOptions)",
- false)
+ .param("docPage", "string", CatalogDocs.DOC_PAGE_DESC, false)
+ .param("optionsFilter", "string",
CatalogDocs.OPTIONS_FILTER_DESC, false)
+ .param("limit", "integer", CatalogDocs.OPTIONS_LIMIT_DESC,
false)
Review Comment:
its okay to increate the budget (mcp descrptions must also be accurate and
somethging LLMs can understand and use)
--
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]
