Skip to content

feat: add Atlas Cloud provider - #3962

Open
binyangzhu000-sudo wants to merge 1 commit into
docker:mainfrom
binyangzhu000-sudo:codex/add-atlas-cloud-provider
Open

feat: add Atlas Cloud provider#3962
binyangzhu000-sudo wants to merge 1 commit into
docker:mainfrom
binyangzhu000-sudo:codex/add-atlas-cloud-provider

Conversation

@binyangzhu000-sudo

Copy link
Copy Markdown

Summary

  • add Atlas Cloud as a built-in OpenAI-compatible provider alias
  • discover ATLASCLOUD_API_KEY for automatic model selection and credential forwarding
  • document inline and named-model configuration

Testing

  • go test ./pkg/model/provider (focused alias and end-to-end provider tests)
  • go test ./pkg/config ./pkg/runtime ./pkg/creator ./pkg/environment (focused provider/config tests)
  • go run ./lint .
  • golangci-lint v2.12.2 run
  • go mod tidy --diff
  • CI=true ./scripts/build.sh
  • live Atlas Cloud Chat Completions smoke test (HTTP 200)

Signed-off-by: binyangzhu000-sudo <224954946+binyangzhu000-sudo@users.noreply.github.com>
@binyangzhu000-sudo
binyangzhu000-sudo requested a review from a team as a code owner August 10, 2026 15:05
@aheritier aheritier added area/config For configuration parsing, YAML, environment variables area/docs Documentation changes area/providers For features/issues/fixes related to LLM providers (Bedrock, LiteLLM, Qwen, custom, etc.) area/providers/openai For features/issues/fixes related to the usage of OpenAI models kind/feat PR adds a new feature (maps to feat:). Use on PRs only. status/needs-signed-commits Some commits in the PR are signed with a valid SSH/GPG key labels Aug 10, 2026
@aheritier

Copy link
Copy Markdown
Collaborator

👋 Some commits in this PR are not signed and verified by GitHub. Please sign your commits with a GPG or SSH key registered in your GitHub account, then force-push.

Commits that are not verified: 24eb110

See GitHub's guide on signing commits for setup instructions. I've added status/needs-signed-commits; it will be removed automatically once every commit in this PR carries a valid GitHub-verified signature.

@Sayt-0 Sayt-0 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Adding Atlas Cloud as a built-in alias follows the established pattern and the Go changes are complete and well tested. Three integration points are missing and the default model needs verification before merge.

# Finding File Blocking
1 New docs page absent from the sidebar docs/data/nav.yml yes
2 Built-in alias table not updated docs/concepts/models/index.md yes
3 Provider list in schema description not updated agent-schema.json no
4 Default model qwen/qwen3.8-max unverifiable pkg/config/auto.go (see inline) yes
5 Front matter weight breaks alphabetical convention docs/providers/atlascloud/index.md (see inline) no

Ready-to-paste fixes for the files outside the diff:

1. docs/data/nav.yml, between the Anthropic and AWS Bedrock entries:

    - title: Atlas Cloud
      url: /providers/atlascloud/

2. docs/concepts/models/index.md, built-in alias table, after the OpenRouter row (matches the placement used in docs/providers/overview/index.md; adjust the description column if needed):

| Atlas Cloud         | `atlascloud`     | Qwen and other open models           | `ATLASCLOUD_API_KEY`                |

3. agent-schema.json, provider description, add atlascloud after openrouter in the alias enumeration:

... any built-in alias (requesty, openrouter, atlascloud, azure, xai, ...)

Process notes:

  • The status/needs-signed-commits label is set; the commit must be signed before merge.
  • The branch is 11 commits behind main. A rebase also picks up 95f3d1a, which fixes the pre-existing TestParseExamples gopher failures in pkg/config, unrelated to this PR but noisy in local runs.

title: "Atlas Cloud"
description: "Use Atlas Cloud models with Docker Agent."
keywords: docker agent, ai agents, model providers, llm, atlas cloud
weight: 235

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Provider pages use alphabetical-by-title weights in increments of 10 (Anthropic 20, AWS Bedrock 30, Baseten 40, ...). "Atlas Cloud" sorts between Anthropic and AWS Bedrock, so 235 places the page between OpenRouter and OVHcloud in weight-ordered listings.

Suggested change
weight: 235
weight: 25

Comment thread pkg/config/auto.go
"dmr": "ai/qwen3:latest",
"mistral": "mistral-small-latest",
"openrouter": "meta-llama/llama-3.3-70b-instruct",
"atlascloud": "qwen/qwen3.8-max",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can the existence of qwen/qwen3.8-max in the Atlas Cloud catalog be confirmed (catalog link or output of the live smoke test mentioned in the PR description)? Two concerns:

  • This ID is the auto-selection default: if it is wrong, every user with only ATLASCLOUD_API_KEY set gets a broken first run.
  • Neither an atlascloud provider nor any qwen3.8-max entry exists in pkg/modelsdev/snapshot.json, so capability lookup falls back to text-only with a warning. A "-max" model is also unusual for an open-model host (the -max line is API-only, not open weights), which is worth double-checking against the openModelHostProviders classification.

If the ID changes, the two examples in docs/providers/atlascloud/index.md (atlascloud/qwen/qwen3.8-max and model: qwen/qwen3.8-max) and the assertions in pkg/config/auto_test.go and pkg/model/provider/openai_alias_providers_test.go must be updated in sync.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/config For configuration parsing, YAML, environment variables area/docs Documentation changes area/providers/openai For features/issues/fixes related to the usage of OpenAI models area/providers For features/issues/fixes related to LLM providers (Bedrock, LiteLLM, Qwen, custom, etc.) kind/feat PR adds a new feature (maps to feat:). Use on PRs only. status/needs-signed-commits Some commits in the PR are signed with a valid SSH/GPG key

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants