davsclaus commented on code in PR #27837:
URL: https://github.com/apache/camel/pull/27837#discussion_r4275138882
##########
dsl/camel-jbang/camel-jbang-mcp/src/main/java/org/apache/camel/dsl/jbang/core/commands/mcp/McpSecurityInterceptor.java:
##########
@@ -135,7 +135,8 @@ private void sanitizeParameters(InvocationContext ctx) {
throw new ToolCallException(
"Argument '" + parameterName(ctx.getMethod(), i) +
"' is " + clean.length()
+ " characters long, over the
limit of " + maxLen
- + "
(camel.mcp.security.max-argument-length); nothing was done");
+ + "; nothing was done. Change
a large file in parts with camel_edit_file,"
+ + " or raise
camel.mcp.security.max-argument-length");
Review Comment:
Optional: this message is shown for every String argument of every secured
tool, for example `camel_validate_source(content)` or a long `query`. Saying
when the hint applies keeps it right for those tools too:
```suggestion
+ "; nothing was done. To
write a large file, change it in parts with camel_edit_file,"
+ " or raise
camel.mcp.security.max-argument-length");
```
--
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]
