docs: document AI providers seeding mechanism & support for new types - #25855
Conversation
Docs preview📖 View docs preview for |
This stack of pull requests is managed by Graphite. Learn more about stacking. |
07ff199 to
36f2a0e
Compare
dannykopping
changed the title
|
/coder-agents-review |
|
Chat: Review posted | View chat Review history
deep-review v0.6.0 | Round 1 | Last posted: Round 1, 3 findings (2 P2, 1 P3), COMMENT. Review Finding inventoryFindings
Round logRound 1Netero-only. 2 P2, 1 P3. Reviewed against 23b60ef6dfaa..36f2a0eaaf59. Panel gated on P2 findings; author should fix broken links and missing ChatGPT documentation before full panel review. About deep-reviewCRF = Coder Review Finding (P0-P4, Nit, Note)
|
![coder-agents-review[bot]](https://avatars.githubusercontent.com/in/3454270?s=60&v=4){kind=small}
There was a problem hiding this comment.
First-pass review (Netero). 2 P2, 1 P3.
This is a first-pass review only: these are mechanical findings, the full review panel has not yet reviewed this PR. The panel will review after these findings are addressed.
The new providers.md page is well structured, the provider-type table and lifecycle status table are clear, and the monitoring metrics section with PromQL alert examples is a useful addition. The migration from env-var docs to dashboard-driven setup is well-motivated.
The three findings are all broken cross-references caused by removing headings from setup.md without updating the docs that linked to them, plus one content gap (ChatGPT provider type dropped entirely).
docs/ai-coder/ai-gateway/clients/copilot.md:20
P2 [CRF-1] This line links to ../setup.md#github-copilot, but the ### GitHub Copilot heading was removed from setup.md by this PR. The anchor resolves to nothing.
The target should be ../providers.md#github-copilot, which has the replacement content.
(Netero)
🤖
docs/ai-coder/ai-gateway/clients/coder-agents.md:47
P2 [CRF-2] This line links to ../setup.md#multiple-instances-of-the-same-provider, but that heading was removed from setup.md by this PR. The anchor is now dead.
Unlike the Copilot heading, there is no equivalent anchor in providers.md to redirect to. The concept of named provider instances with distinct routes is now implicit in the dashboard flow but needs an explicit section or anchor that this file can link to.
(Netero)
🤖
🤖 This review was automatically generated with Coder Agents.
ibetitsmike
left a comment
ibetitsmike
left a comment
There was a problem hiding this comment.
approved with a question
| [Provider Configuration](./providers.md) for the lifecycle these | ||
| metrics describe. | ||
|
|
||
| | Metric | Type | Labels | Purpose | |
There was a problem hiding this comment.
IIRC nothing in the stack actually adds those metrics, we just didn't have the docs before?
There was a problem hiding this comment.
"a Anthropic provider" just hurts to look at, but unrelated to this PR
There was a problem hiding this comment.
Fixed downstack 👍
36f2a0e to
e5df169
Compare
23b60ef to
8bea045
Compare
e5df169 to
cc99a93
Compare
8bea045 to
2e2fe02
Compare
Documentation CheckUpdates Needed
New Documentation Needed
Automated review via Coder Agents |
Merge activity
|
nickvigilante
left a comment
nickvigilante
left a comment
There was a problem hiding this comment.
LGTM overall! I have some other nits that I think I'll need to change more holistically, but I need to establish a style guide and clearly defined rules first and distribute that across the wider team, but this PR is informative and has no glaring issues that need to be fixed in the next 24 hours. 
🚀
There was a problem hiding this comment.
Nit: can you edit the web page for this prior to taking the screenshot and change the header label from a to an?
Signed-off-by: Danny Kopping <danny@coder.com>
cc99a93 to
eeff9a4
Compare
|
Cherry-pick PR created: #25906 |
3 participants
Adds a new Provider Configuration reference page (
providers.md) covering:CODER_AI_GATEWAY_PROVIDER_<N>_*and related flagsopenai,anthropic,bedrock,copilot,azure,google,openrouter,vercel,openai-compat) with setup notes for eachenabled,disabled,error) and their effect on request handlingcoderdUpdates Setup (
setup.md) to replace the environment-variable-based provider configuration instructions with dashboard-driven steps (Add provider form, provider list, edit/disable flow), referencing the newproviders.mdpage for deeper detail. Screenshots of the provider list, add, and edit forms are included.Adds a Provider metrics section to Monitoring (
monitoring.md) documenting thecoder_aibridged_*andcoder_aibridgeproxyd_*Prometheus metrics for provider status and reload timestamps, along with two suggested PromQL alert queries.