fix: migrate deprecated code-supernova model ID to fix 404 errors #8881
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
…illion - Add model ID migration logic in RooHandler.getModel() to automatically convert the deprecated 'roo/code-supernova' model ID to 'roo/code-supernova-1-million' - Add comprehensive test coverage for the migration logic - Fixes #8880 where users get 404 errors when using the old model ID
dosubot
bot
added
size:M
Review CompleteNo issues found. The implementation looks good:
|
hannesrudolph
added
the
Issue/PR - Triage
![roomote[bot]](https://avatars.githubusercontent.com/in/1546624?s=60&v=4){kind=small}
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.
Summary
This PR attempts to address Issue #8880 where users encounter a "404 Not Found" error when using the deprecated
code-supernovamodel.Problem
Users who have
roo/code-supernovaconfigured as their model are experiencing 404 errors because this model ID has been deprecated and replaced withroo/code-supernova-1-million.Solution
Added automatic model ID migration in the
RooHandler.getModel()method that transparently converts the deprecated model ID to the new one. This ensures backward compatibility while users transition to the new model ID.Changes
getModel()methodTesting
Impact
This is a non-breaking change that:
Fixes #8880
Feedback and guidance are welcome!
Important
Migrates deprecated
roo/code-supernovamodel ID toroo/code-supernova-1-millioninRooHandler.getModel()to fix 404 errors.RooHandler.getModel()now migrates deprecatedroo/code-supernovamodel ID toroo/code-supernova-1-million.roo.spec.tsto verify model ID migration and ensure non-deprecated models remain unaffected.This description was created by
for 008432a. You can customize this summary. It will automatically update as commits are pushed.