davsclaus commented on PR #27422: URL: https://github.com/apache/camel/pull/27422#issuecomment-6013834576
Thanks for the review. The two points in the summary are addressed in 0f284acb0b: - **`camel mcp`**: `camel_validate_source` in camel-jbang-mcp now takes the `McpConnection` and goes through `callDeterministic`, so the third identical validation of the same content (without a directory) gets the note there too. `aThirdIdenticalValidationOfTheSameContentGetsAShortNote` tests it at the server level, and the listing test now expects the connection for tools that are deterministic for some arguments. The tool is still not marked deterministic in the listing, as it is not for every call. The AI panel keeps its own per-turn guard, so the description now names `camel mcp` and `camel tui --mcp` only. - **Docs**: the Repeated calls section of `camel-jbang-mcp.adoc` explains when `camel_validate_source` counts (content and no directory) and when it is always answered in full, and its table row mentions `sameAsFile` and the note. The failed `upload` check was the artifact-storage outage you mentioned; the new push runs CI again. _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]
