fix: drop displayName from getLanguageModelKey to fix MCP model selection
Closed
SA
4mo ago+3+3+0123456789-2-2-0123456789
Fixes #1137
getLanguageModelKey included displayName in the model key, but displayName is a cosmetic label, not an identity field. The MCP schema only exposes {provider, model}, so explicit model selection via the ask_codebase tool always produced a key mismatch.
Dropping displayName from the key fixes the MCP path at the root and prevents the same bug pattern from appearing in other callers.
Summary by CodeRabbit
- Bug Fixes
- Fixed a “Language model not configured” error when an explicit model is selected.
- Updated model matching to use only provider + model (removed the extra name component), ensuring the chat tool selects the correct configuration consistently.