-
Notifications
You must be signed in to change notification settings - Fork 102
Add Atlas Cloud OpenAI-compatible runtime - #1174
Open
binyangzhu000-sudo wants to merge 2 commits into
Open
Conversation
Contributor
|
No actionable comments were generated in the recent review. 🎉 i️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughAtlas Cloud is registered as an OpenAI-compatible provider with API-key aliases, remote-provider status handling, curated models, runtime labels, and agent picker support across the backend and web application. ChangesAtlas Cloud provider integration
Estimated code review effort: 3 (Moderate) | ~20 minutes Sequence Diagram(s)sequenceDiagram
participant AgentRuntimePicker
participant NewOpenAICompatStreamFn
participant resolveOpenAICompatAPIKey
participant AtlasCloudEndpoint
AgentRuntimePicker->>NewOpenAICompatStreamFn: select atlascloud and model
NewOpenAICompatStreamFn->>resolveOpenAICompatAPIKey: resolve configured API key
resolveOpenAICompatAPIKey-->>NewOpenAICompatStreamFn: ATLASCLOUD_API_KEY or ATLAS_CLOUD_API_KEY
NewOpenAICompatStreamFn->>AtlasCloudEndpoint: send authenticated OpenAI-compatible request
Suggested reviewers: 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Contributor
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.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@web/src/components/agents/AgentProfilePanel.tsx`:
- Line 44: Synchronize the provider fallbacks with the registry by reusing
DEFAULT_LLM_KINDS where possible: update the fallback list in
AgentProfilePanel.tsx (Lines 530-537) and AgentWizard.tsx (Lines 198-205) to
include "atlascloud", preserving offline picker and suggestion behavior.
In `@web/src/components/apps/SettingsApp.tsx`:
- Around line 277-282: Remove the Atlas Cloud entry from the LOCAL_PROVIDERS
collection used by LocalProviderCard, and place it in the existing cloud/remote
provider collection and corresponding status-card rendering path. Preserve the
Atlas Cloud label and configuration guidance while ensuring it is not presented
as a local runtime or under no-cloud-key messaging.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
i️ Review info
⚙️ Run configuration
Configuration used: Path: .coderabbit.yaml
Review profile: ASSERTIVE
Plan: Pro
Run ID: a7f010a3-02d2-4ae2-b33c-86c404cda65f
📒 Files selected for processing (15)
internal/provider/atlascloud.gointernal/provider/binding.gointernal/provider/binding_test.gointernal/provider/openai_compat.gointernal/provider/openai_compat_test.gointernal/team/local_providers_status.gointernal/team/local_providers_status_test.goweb/src/api/client.tsweb/src/components/agents/AgentProfilePanel.tsxweb/src/components/agents/AgentWizard.tsxweb/src/components/apps/SettingsApp.tsxweb/src/components/onboarding/runtimeConstants.tsweb/src/lib/modelCatalog.tsweb/src/lib/providerBinding.tsweb/src/lib/runtimeProviders.ts
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.
Uh oh!
There was an error while loading. Please reload this page.
Summary
atlascloudruntime backed by the existing OpenAI-compatible streaming provider pathhttps://api.atlascloud.ai/v1,qwen/qwen3.5-flash) plusATLASCLOUD_API_KEY/ATLAS_CLOUD_API_KEYfallback authValidation
/v1/modelscatalog returned 118 models and confirmedqwen/qwen3.5-flashanddeepseek-ai/deepseek-v4-progit diff --no-index --checkpassed against a clean codeload copygoorgofmtnpm installis blocked by the existing@tanstack/ai/@tanstack/ai-openaipeer dependency conflict; retrying with--legacy-peer-deps --package-lock=falsemade no filesystem progress before being interruptedNo README, sponsor, logo, credits, or partner-promotion changes are included.
Summary by CodeRabbit