docs: refresh outdated model examples and add Chat Server page#2656
Merged
dgageot merged 1 commit intodocker:mainfrom May 6, 2026
Merged
Conversation
|
|
||
| ## Endpoints | ||
|
|
||
| All endpoints live under the `/v1` prefix to match the OpenAI API surface. |
There was a problem hiding this comment.
[MEDIUM] Introductory claim contradicts the endpoint table
Line 47 states:
"All endpoints live under the
/v1prefix to match the OpenAI API surface."
But the table immediately below lists /openapi.json at the root level — not under /v1. This is confirmed by the actual server implementation in pkg/chatserver/server.go:
// Register /openapi.json *before* the bearer-auth middleware so the
// spec is publicly accessible.
e.GET("/openapi.json", s.handleOpenAPI)The introductory sentence is factually incorrect. Consider changing it to something like:
"The two core API endpoints follow the OpenAI API surface under the
/v1prefix. The OpenAPI spec is served separately at/openapi.json."
Or simply remove the "all endpoints" claim and let the table speak for itself.
d63fb7e to
234c39e
Compare
- Bump example Anthropic model from claude-sonnet-4-0 to claude-sonnet-4-5 (the recommended default per docs/providers/anthropic/index.md) across guides, configuration, tools, features, and the home page. - Replace the broken modelname.ai link in the OpenAI provider page with the correct modelnames.ai and the official OpenAI models doc. - Update the Bedrock IAM-role example from the long-deprecated anthropic.claude-3-sonnet-20240229-v1:0 model to the current global.anthropic.claude-sonnet-4-5-20250929-v1:0 used elsewhere on the page. - Add a dedicated docs/features/chat/ page for 'docker agent serve chat' (the OpenAI-compatible chat-completions server), wire it into the navigation, and cross-link it from the CLI reference and API server pages. Assisted-By: docker-agent
234c39e to
2b7fb57
Compare
derekmisler
approved these changes
May 6, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
This PR fixes a few pieces of the docs that were outdated, plain wrong, or missing.
Outdated / wrong
claude-sonnet-4-0(older) andclaude-sonnet-4-5(the current recommended default perdocs/providers/anthropic/index.md). All example configs are now normalized toclaude-sonnet-4-5. The Anthropic provider page still listsclaude-sonnet-4-0in the Available Models table as the previous generation.docs/providers/openai/index.md–modelname.aireturns a 404 ("This app isn't live yet"). Replaced with the correctmodelnames.aiand added a link to the official OpenAI models doc.anthropic.claude-3-sonnet-20240229-v1:0model id while every other example on the same page uses the currentglobal.anthropic.claude-sonnet-4-5-20250929-v1:0. Made the IAM section consistent.Missing
docker agent serve chat(the OpenAI-compatible chat-completions server) was documented as a CLI command in the CLI reference but had no dedicated feature page and no entry in the side navigation. Addeddocs/features/chat/index.mdcovering:/v1/models,/v1/chat/completions, and/openapi.jsonendpointscurland the OpenAI Python SDKX-Conversation-Id)docker agent serve chatdocs/_data/nav.ymland added cross-links from the API server page and CLI reference.How verified
Docs-only changes. Verified:
docs/_data/nav.ymlis still valid YAML.claude-sonnet-4-5is the recommended default indocs/providers/anthropic/index.md,global.anthropic.claude-sonnet-4-5-20250929-v1:0matches the rest of the Bedrock page).modelnames.airesolves;modelname.aidoes not.Files
docs/features/chat/index.mddocs/_data/nav.yml,docs/index.md, plus 14 other doc files updating model names and adding cross-links.