- Notifications
You must be signed in to change notification settings - Fork1.1k
chore!: allow coder MCP tools to not be injected#20713
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
Signed-off-by: Danny Kopping <danny@coder.com>
matifali left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
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.
dannykopping commentedNov 11, 2025
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 to9cd5eceCompareSigned-off-by: Danny Kopping <danny@coder.com>
Signed-off-by: Danny Kopping <danny@coder.com>
| --aibridge-inject-coder-mcp-tools bool, $CODER_AIBRIDGE_INJECT_CODER_MCP_TOOLS (default: false) | ||
| Whether to inject Coder's MCP tools into intercepted AI Bridge | ||
| requests (requires the "oauth2" and "mcp-server-http" experiments to | ||
| be enabled). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I assume this will not require the other experiments flags (http-mcp-server,oauth2) when they graduate from the experiment.
matifali left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
The flow looks good to me. Thanks.
| YAML:"bedrock_small_fast_model", | ||
| }, | ||
| { | ||
| Name:"AIBridge Inject Coder MCP tools", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
| Name:"AIBridge Inject Coder MCP tools", | |
| Name:"AI Bridge Inject Coder MCP tools", |
I think we useAI Bridge more thanAIBridge
Uh oh!
There was an error while loading.Please reload this page.
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.