- Notifications
You must be signed in to change notification settings - Fork1.1k
chore(docs): update links and minor improvements to AI Bridge#20714
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
Conversation
| "title":"MCP Tools Injection", | ||
| "description":"How to configure MCP servers for tools injection through AI Bridge", | ||
| "path":"./ai-coder/ai-bridge/mcp.md", | ||
| "state": ["early access"] |
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.
Marking the MCP Tools injection section explicitly asEarly Access. We would like to keep this feature opt-in and EA even after AI Bridge itself graduates to Beta or GA.
| Template admins can pre-configure workspaces to route all AI tool requests through AI Bridge, providing a seamless and secure experience for users. This can be donefor bothCoder Tasks and otherAItools running in the workspace. | ||
| Template admins can pre-configure workspaces to route all AI tool requests through AI Bridge, providing a seamless and secure experience for users. This can be donewith either[Coder Tasks](../tasks.md) orAIagents running in the workspace. | ||
| ###Using Coder Tasks |
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 still think mentioning Coder Tasks here is distracting and unnecessary.
We're still talking about the basic setup here. Tasks is an optional integration, therefore it should have lower hierarchical priority.
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 will try to come up with a different version. Or we can drop Taks. Thoughts@david-fraley?
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.
We should absolutely keep the Tasks section, but not make it sound like Tasks arerequired for Bridge to work.
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.
What do you think about it now?a106652
Uh oh!
There was an error while loading.Please reload this page.
2479f38 toa106652Compare
dannykopping 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.
Thanks for reorganising the doc 👍
One suggestion but LGTM
Uh oh!
There was an error while loading.Please reload this page.
Co-authored-by: Danny Kopping <danny@coder.com>
086d6b8 intomainUh oh!
There was an error while loading.Please reload this page.
Uh oh!
There was an error while loading.Please reload this page.
Addresses some feedback from#20640