-
Notifications
You must be signed in to change notification settings - Fork 1.1k
docs: add client configuration section and support matrix for AI Bridge #20640
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…r additional providers Revised the AI Bridge documentation to clarify provider configuration steps, including new sections for OpenAI, Anthropic, and Amazon Bedrock. Added details on pre-configuring templates and using AI Bridge with Coder Tasks. Introduced a new image illustrating the AI Bridge implementation details. Updated compatibility notes for various AI clients. Addresses coder/aibridge#4
c9c35ac to
52f51bc
Compare
Revised client support table for enhanced clarity.
52f51bc to
458890f
Compare
david-fraley
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.
If you've got time to address this high level feedback before release, let's do it. If not, let's make it happen quickly after. It's not a hard requirement for merge, but I don't want to let it hang out too long...
This page is getting MaSSiVe 😂. Can we break it up into a top-level "AI Bridge" w/ the summary, then sub pages covering (this is just a first stab, feel free to riff on the sub page breakdown)
- Setup
- Client Config
- MCP
- Monitoring
- Reference (super technical documentation)
Splits the AI Bridge documentation into multiple pages to improve readability and navigation.
|
I have addressed the feedback and pushed the changes. |
3e24b0d to
33c54ea
Compare
Use "AI Bridge" consistently across the documentation, replacing instances of "Bridge." Adjust headings and content throughout to ensure clarity and alignment with the AI Bridge branding.
68daf40 to
82aa5dc
Compare
- Capitalize section headings for consistency. - Use the singular form for "provider" for clarity. - Ensure document syntax matches Coder documentation style guidelines.
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.
Really liking the reorganisation.
Pausing here for the moment; ran out of time to review.
I've only reviewed up to the start of Client Config page, in the order of how it appears on the website.
Thanks for providing a docs preview link, that was very helpful.
| Instead of distributing provider-specific API keys (OpenAI/Anthropic keys) to users, they authenticate to AI Bridge using their **Coder session token** or **API key**: | ||
|
|
||
| - **OpenAI clients**: Users set `OPENAI_API_KEY` to their Coder session token or API key | ||
| - **Anthropic clients**: Users set `ANTHROPIC_API_KEY` to their Coder session token or API key |
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.
Have you tested this? I've only tested ANTHROPIC_AUTH_TOKEN in Claude Code.
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'm gonna merge as-is for this sentence, but let @matifali fast follow if we need to change
|
|
||
| ## Tested Clients | ||
|
|
||
| The table below shows tested AI clients and their compatibility with AI Bridge. Click each client name for vendor-specific configuration instructions. Report issues or share compatibility updates in the [aibridge](https://github.com/coder/aibridge) issue tracker. |
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.
Click each client name for vendor-specific configuration instructions
Not all clients have links set?
Also, weren't we going to provide setup details for each client?
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'll leave this one for @matifali to quick follow
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.
Having details for each client can be overly extensive. Also most of them are just setting the correct URL and token which we already explain.
Also couldn't find a very good organization for clients.
For URLs I linked to the setup docs for each I could find. Will add any missing ones.
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
Co-authored-by: Danny Kopping <danny@coder.com>
|
@dannykopping @david-fraley follow up here: #20714 |
Revised the AI Bridge documentation to clarify provider configuration steps, including new sections for OpenAI, Anthropic, and Amazon Bedrock. Added details on pre-configuring templates and using AI Bridge with Coder Tasks. Introduced a new image illustrating the AI Bridge implementation details. Updated compatibility notes for various AI clients.
Addresses coder/aibridge#4
Preview: https://coder.com/docs/@aibridge-client-instructions/ai-coder/ai-bridge