-
-
Notifications
You must be signed in to change notification settings - Fork 279
feat: add dynamic model switching via /model command #237
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
Merged
Merged
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
5f36e23
feat: add dynamic model switching via /model command
vitali87 91bae6c
refactor: simplify model switching with dataclasses.replace and speci…
vitali87 ecc99b2
refactor: return canonical provider:model_id format from model creation
vitali87 2ed0807
refactor: remove redundant boolean return value from _handle_model_co…
vitali87 2d72509
fix: load provider API keys from environment variables when not expli…
vitali87 5a7ad72
fix: validate model_id is not empty to prevent invalid model configur…
vitali87 085c918
feat: add /model help to display usage instructions
vitali87 3d57132
refactor: rename MODEL_COMMAND_HELP to HELP_COMMAND for clarity
vitali87 6e8d4f7
fix: require provider in model string format (provider:model)
vitali87 a52bae0
refactor: consolidate command handling logic to reduce repetition
vitali87 6c6d019
fix: use exact command matching and rename misleading test
vitali87 5a67501
fix: use DEFAULT_API_KEY constant for Ollama api_key field
vitali87 8aa3e33
test: add console output assertion to trailing space test
vitali87 eae9259
fix: use CHAR_COLON constant and correct provider name in example
vitali87 2670bb6
refactor: simplify command handling with direct continue statements
vitali87 471153e
fix: catch specific exceptions (ValueError, AssertionError) in model …
vitali87 29dc51e
refactor: consolidate argument parsing in model command handler
vitali87 f01aca8
fix: strip both provider and model_id when parsing model string
vitali87 14ede30
test: add unit tests for model switching and fix empty provider valid…
vitali87 e380d74
refactor: inline redundant new_model_string_arg variable
vitali87 44a8eb9
fix: track config when switching models to preserve settings for same…
vitali87 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
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
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
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
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
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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.