Croway commented on PR #25203: URL: https://github.com/apache/camel/pull/25203#issuecomment-5140595205
Hello @davsclaus @atiaomar1978-hub @jamesnetherton please hold before merging this PR, yesterday I opened https://issues.apache.org/jira/browse/CAMEL-24308 that will introduce new camel component that integrate ai-tools with an embedded MCP Server, it will provide a default implementation, and a specific implementation for quarkus (based on quarkus-mcp-server), the configuration of the MCP Server will be done via properties, like ``` camel.server.mcp-enabled=true camel.server.mcp-path=/mcp camel.server.mcp-server-name=my-integration-app camel.server.mcp-tags=crm,notify ``` that bridges the ai-tools with the MCP. I am a little bit concerned about this PR, in particular the configuration part `camel.mcp.*` I am afraid it can be misleading having multiple configurations. I do understand that this PR is for development/management only, but maybe we can improve/review the implementation given CAMEL-24308 -- 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]
