-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Upgrade Supernova #8330
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
Upgrade Supernova #8330
Conversation
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.
I found some issues that need attention. See inline comments for details.
| [K in ProviderName]?: Record<string, string> | ||
| } | ||
|
|
||
| const MODEL_MIGRATIONS: ModelMigrations = { |
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.
P2: The migration map is typed as Record<string, string>. Narrow to RooModelId keys/values to catch typos at compile time.
| const MODEL_MIGRATIONS: ModelMigrations = { | |
| type ModelMigrations = { | |
| [K in ProviderName]?: Partial<Record<RooModelId, RooModelId>> | |
| } | |
| const MODEL_MIGRATIONS: ModelMigrations = { | |
| roo: { | |
| 'roo/code-supernova': 'roo/code-supernova-1-million', | |
| }, | |
| } as const |
| const newModelId = providerMigrations[apiConfig.apiModelId] | ||
| if (newModelId && newModelId !== apiConfig.apiModelId) { | ||
| console.log( | ||
| `[ModelMigration] Migrating ${apiConfig.apiProvider} model from ${apiConfig.apiModelId} to ${newModelId}`, |
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.
P2: Prefer project logger/telemetry over console.log/console.error to reduce noise and centralize diagnostics. Consider emitting a dedicated event for model migrations.
| Object.values(providerProfiles.apiConfigs)[0]?.id ?? | ||
| this.defaultConfigId | ||
| providerProfiles.modeApiConfigs = Object.fromEntries(modes.map((m) => [m.slug, seedId])) | ||
| isDirty = true |
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.
P3: Migrations run on initialize(). Consider applying the same normalization step wherever providerProfiles are mutated (e.g., save/import) to keep configs normalized after startup.
| "roo/code-supernova": { | ||
| maxTokens: 16_384, | ||
| contextWindow: 200_000, | ||
| "roo/code-supernova-1-million": { |
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.
P3: New ID and limits look good. Ensure any user-facing docs/help listing model IDs are updated to 'roo/code-supernova-1-million' to avoid confusion.
* feat: Add DeepInfra as a model provider in Roo Code (RooCodeInc#7677) * fix: validate MCP tool exists before execution (RooCodeInc#7632) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Daniel Riccio <[email protected]> * fix: add error transform to cryptic openAI SDK errors when API key is invalid (RooCodeInc#7586) Co-authored-by: Roo Code <[email protected]> Co-authored-by: Daniel Riccio <[email protected]> * feat: OpenAI Responses API service tiers (flex/priority) — UI selector, pricing, and tests (RooCodeInc#7646) Co-authored-by: Daniel Riccio <[email protected]> * fix: prevent countdown timer from showing in history for answered follow-up questions (RooCodeInc#7686) * fix: Fix the issue of Moonshot's maximum return token count being limited to 1024 (RooCodeInc#7673) Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> Co-authored-by: roomote[bot] <219738659+roomote[bot]@users.noreply.github.com> * feat: update kimi-k2-0905-preview and kimi-k2-turbo-preview (RooCodeInc#7663) * Fix test * Release v3.26.7 (RooCodeInc#7691) * Delete .changeset/petite-rats-admire.md * feat: Add Kimi K2 0905 model to Groq, Moonshot, and Fireworks providers (RooCodeInc#7693) * Use an allowlist to keep the prompt default shell sane (RooCodeInc#7681) * Changeset version bump (RooCodeInc#7690) Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Matt Rubens <[email protected]> * Fix: Improve WelcomeView styling and readability (RooCodeInc#7682) * fix: improve WelcomeView styling and readability - Reduced excessive padding from p-16 to p-6 - Removed excessive bold text throughout - Made provider cards more compact with smaller icons (32x32px) - Increased introduction text size from text-sm to text-base for better visibility - Centered the main greeting title - Improved visual hierarchy with proper text sizing and spacing - Added subtle borders and improved color contrast * fix: remove undefined color class from incentive text Removed the text-vscode-warningForeground class as it wasn't defined in index.css. The incentive text now uses default color which looks good. * feat: add Kimi K2-0905 model to Chutes provider (RooCodeInc#7701) Co-authored-by: Roo Code <[email protected]> * fix: handle array paths from VSCode terminal profiles (RooCodeInc#7697) * fix: handle array paths from VSCode terminal profiles - Updated terminal profile interfaces to support string | string[] for path property - Added normalizeShellPath helper to safely extract first element from array paths - Modified isShellAllowed to handle both string and array inputs - Updated getWindowsShellFromVSCode, getMacShellFromVSCode, and getLinuxShellFromVSCode to use normalizeShellPath - Added comprehensive tests for array path handling Fixes RooCodeInc#7695 * feat: add validateShellPath export for robust shell validation - Created validateShellPath as a public API for shell path validation - Refactored internal validation logic into isShellAllowedInternal - Added comprehensive test coverage for all edge cases - Maintains backward compatibility with deprecated isShellAllowed - Handles arrays, strings, null, undefined, and nested arrays gracefully * Simplify roomote's work a little --------- Co-authored-by: Roo Code <[email protected]> Co-authored-by: John Richmond <[email protected]> * Edit/Delete User Message (RooCodeInc#7447) * fix: prevent stack overflow in codebase indexing for large projects (RooCodeInc#7712) * fix: identify mcp and slash command config path in multiple folder workspace (RooCodeInc#6904) * fix: resolve CI e2e test ETIMEDOUT errors when downloading VS Code (RooCodeInc#7583) * fix: Tackling Race/State condition issue by Changing the Code Design for Gemini Grounding Sources (RooCodeInc#7434) Co-authored-by: daniel-lxs <[email protected]> Co-authored-by: Matt Rubens <[email protected]> * fix: preserve context by retrying with full conversation on invalid previous_response_id (
Important
Upgrade
roo/code-supernovatoroo/code-supernova-1-millionwith model migration and update UI elements and translations accordingly.roo/code-supernovatoroo/code-supernova-1-millioninroo.tswith increasedmaxTokensandcontextWindow.ProviderSettingsManager.tsto automatically update old model IDs to new ones.ProviderSettingsManager.spec.tsto verify model migration logic, including repeated migrations and handling of different providers.chat.jsoninca,de,en) to reflect the new model upgrade.latestAnnouncementIdinClineProvider.tstosep-2025-code-supernova-1m.This description was created by
for 17d8470. You can customize this summary. It will automatically update as commits are pushed.