feat(models): add Atlas Cloud provider preset - #3777
Open
binyangzhu000-sudo wants to merge 3 commits into
Open
Conversation
Atlas Cloud exposes an OpenAI-compatible endpoint, but configuring its endpoint, credential environment variable, and model currently requires manual models.yml edits. The preset keeps credentials external and reuses the existing Chat Completions transport. Lore-id: 7f3a9c1d Constraint: reuse provider onboarding and the existing OpenAI-compatible transport Tested: provider onboarding, model registry, guidance, package check, and live tool call Scope-risk: narrow Reversibility: config-only Signed-off-by: binyangzhu000-sudo <224954946+binyangzhu000-sudo@users.noreply.github.com>
Yeachan-Heo
requested changes
Aug 5, 2026
Yeachan-Heo
left a comment
Owner
There was a problem hiding this comment.
GJC hostile-review verdict — exact head
Reviewed head: 787d061b5dd9e225d7510dbc825369e273ce89a2
Authoritative dev: 732856b3ccb3fade6e9fbc17908a4fbca5a7682f
Verdict: REQUEST_CHANGES
Blocking findings:
- Atlas output caps use the wrong documented field. The preset omits
compat.maxTokensField: "max_tokens". GJC consequently selectsmax_completion_tokens, while Atlas's Chat API and exact/v1/modelsentry advertisemax_tokens. - Same-ID metadata leaks from NVIDIA into Atlas. The preset writes only
deepseek-ai/deepseek-v4-pro;finalizeCustomModel()therefore inherits the bundled NVIDIA same-ID record. Context/output happen to match Atlas, but the effective model inherits NVIDIA reasoning/effort behavior and NVIDIA pricing. Atlas's live catalog does not advertise reasoning for this model and reports approximately $1.68/M input, $3.38/M output, and $0.13/M cache read rather than the inherited $0.435/M, $0.87/M, and $0.003625/M. - No exact-head CI exists. This SHA has zero check runs and zero legacy status contexts. Author-reported local tests and the package check are not required-CI evidence.
- The head is stale and conflicting. It is 9 commits behind current
dev; GitHub reportsDIRTY, unmergeable, and unrebaseable. Currentdevoverlaps the environment-variable documentation and generated embedded-doc artifact.
Atlas's provider identity, https://api.atlascloud.ai/v1, Bearer authentication, ATLASCLOUD_API_KEY, and exact model ID are validated by official Atlas documentation and the live catalog. These are ordinary preset/runtime defects, not a provider-existence or owner-confirmation blocker.
Required before re-review: rebase and regenerate; pin max_tokens; represent Atlas-specific effective model metadata rather than inheriting NVIDIA transport economics/capabilities; test the finalized registry model and emitted capped payload; obtain exact-head CI.
PR-3777 head=787d061b5dd9e225d7510dbc825369e273ce89a2 dev=732856b3ccb3fade6e9fbc17908a4fbca5a7682f evidence-sha256=3ba60e486c2f95842e36517c9eec831ab09667e167068a589eb1f297836c529e verdict=REQUEST_CHANGES
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
atlas-cloudprovider preset backed by the existing OpenAI Chat Completions transport.ATLASCLOUD_API_KEY,https://api.atlascloud.ai/v1, anddeepseek-ai/deepseek-v4-protogether without persisting credentials.models.ymlcontract.Why
Atlas Cloud is OpenAI-compatible, but configuring its endpoint, credential environment variable, model id, and compatibility flags currently requires error-prone manual YAML.
Testing
bun test packages/coding-agent/test/provider-onboarding.test.ts packages/coding-agent/test/model-registry.test.ts packages/coding-agent/test/model-onboarding-guidance.test.ts(248 passed)bun --cwd=packages/coding-agent run checkgjc setup provider --preset atlas-cloudgenerated a credential-freemodels.yml.deepseek-ai/deepseek-v4-pro.GJC verdict
devbun checkpasses (package-scoped coding-agent check passed; full root check was not run)