Skip to content

Conversation

@daniel-lxs
Copy link
Member

@daniel-lxs daniel-lxs commented May 29, 2025

Important

Add Nebius as a new provider with model handling, API configuration, and UI integration.

  • Behavior:
    • Add Nebius as a new provider option in new-run.tsx and provider-settings.ts.
    • Handle Nebius API key and model ID in webviewMessageHandler.ts.
    • Add Nebius to provider validation in validate.ts.
  • Models:
    • Define Nebius models and default model info in api.ts.
    • Add Nebius model fetching logic in fetchers/nebius.ts.
    • Include Nebius in useSelectedModel.ts and modelCache.ts.
  • UI:
    • Add Nebius provider UI components in ApiOptions.tsx, ModelPicker.tsx, and providers/Nebius.tsx.
    • Update provider list and constants in constants.ts and index.ts.

This description was created by Ellipsis for 1f58d87. You can customize this summary. It will automatically update as commits are pushed.

arafatkatze and others added 6 commits May 29, 2025 13:48
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
- Fix typo in Nebius component (was using litellmApiKey)
- Fix inconsistent default base URL between UI and API handler
- Add Zod validation for Nebius API responses
- Add comprehensive unit tests for Nebius provider and fetcher
- Improve error handling with schema validation
@daniel-lxs daniel-lxs requested review from cte and mrubens as code owners May 29, 2025 19:04
@dosubot dosubot bot added size:XL This PR changes 500-999 lines, ignoring generated files. enhancement New feature or request labels May 29, 2025
@daniel-lxs daniel-lxs moved this from Triage to PR [Needs Review] in Roo Code Roadmap May 29, 2025
@daniel-lxs daniel-lxs mentioned this pull request May 29, 2025
23 tasks
@daniel-lxs daniel-lxs moved this from PR [Needs Review] to PR [Changes Requested] in Roo Code Roadmap May 29, 2025
- Added fetchModel() call before testing createMessage to ensure models are loaded
- This ensures the correct model ID is used when checking for R1 format
- Test now properly verifies that system prompt and user message are merged for DeepSeek-R1
@daniel-lxs daniel-lxs moved this from PR [Changes Requested] to PR [Needs Review] in Roo Code Roadmap May 29, 2025
daniel-lxs and others added 2 commits May 29, 2025 14:30
- Changed from LITELLM_COMPUTER_USE_MODELS to OPEN_ROUTER_COMPUTER_USE_MODELS
- This matches what the actual nebius fetcher implementation uses
- Fixes the failing test that was checking against the wrong constant
Thanks @akhial, saw the PR to Cline!
@mrubens
Copy link
Collaborator

mrubens commented Jun 2, 2025

Not sure if I messed something up, but I don't think this one is quite ready.

Screenshot 2025-06-01 at 10 12 14 PM

@mrubens mrubens moved this from PR [Needs Review] to PR [Needs Prelim Review] in Roo Code Roadmap Jun 4, 2025
@daniel-lxs
Copy link
Member Author

I'll be closing this since Nebius seems OpenAI compatible, I don't see a reason to add this as of now.

This can be reopened if necessary.

@daniel-lxs daniel-lxs closed this Jun 4, 2025
@github-project-automation github-project-automation bot moved this from PR [Needs Prelim Review] to Done in Roo Code Roadmap Jun 4, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request PR - Needs Review size:XL This PR changes 500-999 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants