-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Add a couple more free models to the Roo provider #8304
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
| @@ -1,6 +1,10 @@ | ||||||
| import type { ModelInfo } from "../model.js" | ||||||
|
|
||||||
| export type RooModelId = "xai/grok-code-fast-1" | "roo/code-supernova" | ||||||
| export type RooModelId = | ||||||
| | "xai/grok-code-fast-1" | ||||||
| | "roo/code-supernova" | ||||||
| | "xai/grok-4-fast" | ||||||
| | "deepseek/deepseek-chat-v3.1" | ||||||
|
|
||||||
| export const rooDefaultModelId: RooModelId = "xai/grok-code-fast-1" | ||||||
|
|
||||||
|
|
@@ -25,4 +29,24 @@ export const rooModels = { | |||||
| description: | ||||||
| "A versatile agentic coding stealth model that supports image inputs, accessible for free through Roo Code Cloud for a limited time. (Note: the free prompts and completions are logged by the model provider and used to improve the model.)", | ||||||
| }, | ||||||
| "xai/grok-4-fast": { | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: New model IDs were added but RooModelId (see line 3) still only covers two IDs. This can cause downstream typing friction. Suggest either:
|
||||||
| maxTokens: 30_000, | ||||||
| contextWindow: 2_000_000, | ||||||
| supportsImages: false, | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: Description below calls Grok 4 Fast “multimodal”, but supportsImages is false. If images aren’t supported through the Roo provider for this free tier, consider removing “multimodal” from the description to prevent confusion. |
||||||
| supportsPromptCache: false, | ||||||
| inputPrice: 0, | ||||||
| outputPrice: 0, | ||||||
| description: | ||||||
| "Grok 4 Fast is xAI's latest multimodal model with SOTA cost-efficiency and a 2M token context window. (Note: prompts and completions are logged by xAI and used to improve the model.)", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P2: The description calls Grok 4 Fast “multimodal” but images are disabled. If that’s intentional (no image support via Roo), consider adjusting the wording to avoid “multimodal”. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| }, | ||||||
| "deepseek/deepseek-chat-v3.1": { | ||||||
mrubens marked this conversation as resolved.
Show resolved
Hide resolved
|
||||||
| maxTokens: 16_384, | ||||||
| contextWindow: 163_840, | ||||||
| supportsImages: false, | ||||||
| supportsPromptCache: false, | ||||||
| inputPrice: 0, | ||||||
| outputPrice: 0, | ||||||
| description: | ||||||
| "DeepSeek-V3.1 is a large hybrid reasoning model (671B parameters, 37B active). It extends the DeepSeek-V3 base with a two-phase long-context training process, reaching up to 128K tokens, and uses FP8 microscaling for efficient inference.", | ||||||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. P3: Description says “up to 128K tokens” but contextWindow is 163,840. Consider aligning the description with the configured window (e.g., ~160K). There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
Suggested change
|
||||||
| }, | ||||||
| } as const satisfies Record<string, ModelInfo> | ||||||
Uh oh!
There was an error while loading. Please reload this page.