ai-assistant: Add support for latest AI models across providers#496
ai-assistant: Add support for latest AI models across providers#496Tusharjamdade wants to merge 1 commit intoheadlamp-k8s:mainfrom
Conversation
Signed-off-by: Tusharjamdade <tusharnjamdade@gmail.com>
|
Hi @illume and @ashu8912, I also tested 
|
There was a problem hiding this comment.
Pull request overview
This pull request attempts to add support for newer AI models across multiple providers (OpenAI, Azure OpenAI, Anthropic, Mistral, and Google Gemini) in response to issue #365 about missing models. However, the PR adds many models that do not actually exist in the respective provider APIs, which will cause runtime failures when users attempt to use them.
Changes:
- Added GPT-5 series models and o4-mini to OpenAI and Azure OpenAI configurations
- Added Claude 4.1 and 4.5 series models to Anthropic configuration
- Added new Mistral model variants including ministral-14b, magistral (misspelled), and devstral
- Added Gemini 3.x models to Google Gemini configuration
- Changed OpenAI default model from o3-mini to o4-mini
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 'magistral-medium-latest', | ||
| 'magistral-small-latest', |
There was a problem hiding this comment.
The model names 'magistral-medium-latest' and 'magistral-small-latest' appear to contain a spelling error. Mistral AI's models use 'mistral' not 'magistral'. This typo will cause API calls to fail. Please correct these to 'mistral-medium-latest' and 'mistral-small-latest' if these are intended to be Mistral models, or remove them if they don't exist.
| 'magistral-medium-latest', | |
| 'magistral-small-latest', |
| 'gpt-5.2', | ||
| 'gpt-5', | ||
| 'gpt-5-pro', | ||
| 'gpt-5-mini', | ||
| 'gpt-5-nano', | ||
| 'gpt-5-turbo', |
There was a problem hiding this comment.
The PR description states this adds "support for latest models" but many of the added models do not exist (GPT-5 series, o4-mini, gpt-4.1, Claude 4.1/4.5, Gemini 3.x, etc.). The referenced issue #365 mentions "gpt-5 / 4.1 /etc missing", but these models are missing from the supported list because they don't actually exist in the respective provider APIs. Adding non-existent models will cause runtime failures when users try to use them. Please verify all model names against the official API documentation for each provider before adding them to the configuration.
There was a problem hiding this comment.
Where can we check the correct names?
illume
left a comment
illume
left a comment
There was a problem hiding this comment.
Thanks for this!
Please check the open review notes?
Please see the git commit guidelines and look at other git log messages.
Something like:
ai-assistant: modelConfig: Add support for latest AI models across providers
illume
changed the title
3 participants
Fixes #365
Adds support for latest models across all AI providers, including OpenAI, Azure OpenAI, Google Gemini, Anthropic, and Mistral.
Updates the provider configuration to include all core and commonly used models.
Example: