chore!: allow coder MCP tools to not be injected #20713
Conversation
Signed-off-by: Danny Kopping <danny@coder.com>
matifali
left a comment
There was a problem hiding this comment.
I suggest we make this opt in i.e. make the default value false. I know it's a breaking change but it's fine given not every client needs the coder MCP tools.
We will plan on a more granular per user/template config for MCP servers/tools for future roadmap.
OK. We will need to call it out in the next release's notes in that case. |
Signed-off-by: Danny Kopping <danny@coder.com>
78c7b26 to
9cd5ece
Compare
Signed-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
matifali
left a comment
There was a problem hiding this comment.
The flow looks good to me. Thanks.
Signed-off-by: Danny Kopping <danny@coder.com>
|
Can we update https://coder.com/docs/ai-coder/ai-bridge/mcp with this change? Thanks |
Good call, #20735 |
Currently, when AI Bridge is enabled AND the
oauth2andmcp-server-httpexperiments are enabled we inject Coder's MCP tools into all intercepted AI Bridge requests.This PR introduces a config to control this behaviour.
NOTE: this is a backwards-incompatible change; previously these tools would be injected automatically, now this setting will need to be explicitly enabled.