atiaomar1978-hub commented on PR #25203:
URL: https://github.com/apache/camel/pull/25203#issuecomment-5148563873

   _AI-generated reply on behalf of 
[atiaomar1978-hub](https://github.com/atiaomar1978-hub) — no code changes in 
this update._
   
   Thanks @Croway — noted on 
[CAMEL-24308](https://issues.apache.org/jira/browse/CAMEL-24308). I share the 
concern about **two configuration surfaces** (`camel.mcp.*` here vs the 
`camel.server.mcp-*` direction you outlined).
   
   This PR was scoped as **dev/management-only** (`insecure:dev`, management 
HTTP server, localhost binding via JBang), but I agree that without 
coordination it could confuse users once CAMEL-24308 lands with ai-tools 
integration and Quarkus-specific wiring.
   
   **Proposal:** hold merge on this PR until we align with CAMEL-24308 — either 
unify property names/prefixes up front, or clearly document that this path is 
JBang-local-dev-only and defer production embedded MCP to the component work 
you’re doing.
   
   Happy to join a design thread on CAMEL-24308 / CAMEL-23853 and adjust scope 
accordingly. No push from my side to merge before that.


-- 
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]

Reply via email to