feat: Add modelId support when exporting tasks #2142
Merged
Conversation
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
Signed-off-by: feifei <[email protected]>
|
dosubot
bot
added
size:S
monotykamary
pushed a commit
to monotykamary/Roo-Code
that referenced
this pull request
Apr 4, 2025
* Set up branch * Feat/add sambanova provider (RooCodeInc#2142) * feat(api): Add SambaNova provider type definitions - Add 'sambanova' to ApiProvider type - Add sambanovaApiKey to ApiHandlerOptions interface - Add SambaNova model definitions * feat(api): Implement SambaNova provider handler - Create SambanovaHandler class - Implement createMessage and getModel methods - Register the provider in the API factory * feat(ui): Add SambaNova provider UI components - Add SambaNova to provider dropdown - Add UI components for API key and model selection * feat(storage): Update secret storage for SambaNova API key - Update SecretKey type to include sambanovaApiKey - Update getState method to include the SambaNova API key - Update API configuration handler to store the SambaNova API key * feat(sambanova): format fix * feat(sambanova): adding previously untracked files * fix: will ensure that when no model ID is explicitly selected for the Sambanova provider, it will use the default model ID that's already defined in the codebase (Meta-Llama-3.3-70B-Instruct) * fix: changing how the dropdown is created by following existing pattern * fix: changing how the dropdown is created by following existing pattern * fix: added SamanovaHandler to buildApiHandler --------- Co-authored-by: saad.khan.hd <[email protected]> Co-authored-by: Dennis Bartlett <[email protected]> * support deepseek and qwen models * remove unused imports * Add missing API provider implementation --------- Co-authored-by: saad-noodleseed <[email protected]> Co-authored-by: saad.khan.hd <[email protected]> Co-authored-by: Dennis Bartlett <[email protected]> Co-authored-by: Saoud Rizwan <[email protected]>
Collaborator
|
@cte does this one look ok to you? |
cte
approved these changes
Apr 14, 2025
github-project-automation
bot
moved this from PR [Pre Approval Review]
to Done
in Roo Code Roadmap
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Context
Implementation
Screenshots
How to Test
Get in Touch
Important
Add
modelIdsupport to task export process, updatingCline,ClineProvider, andApiOptionsto handlemodelIdsynchronization.modelIdto task export process inClineandClineProvider.Clineto include<model>tag in task details.apiModelIdwithselectedModelIdinApiOptions.ApiOptionsto setapiModelIdwhenselectedModelIdchanges.apiModelIdto state inClineProvider.This description was created by
for 8aaec7c. It will automatically update as commits are pushed.