scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-08-28 05:27:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: add optional chaining to router model accesses to prevent TypeError when switching providers

Browse source 
Fixes #9047 - When switching providers after selecting a model, the router models
for the new provider may not be loaded yet, causing a TypeError when trying to
access properties on undefined. Added optional chaining to safely handle these cases.
This commit is contained in:
Roo Code 2025-11-05 09:23:42 +00:00
parent 6965e5c791
commit fd6c987ef1
1 changed files with 7 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
webview-ui/src/components/ui/hooks/useSelectedModel.ts
View file

@ -146,7 +146,7 @@ function getSelectedModel({
switch (provider) {
case "openrouter": {
const id = apiConfiguration.openRouterModelId ?? openRouterDefaultModelId
let info = routerModels.openrouter[id]
let info = routerModels.openrouter?.[id]
const specificProvider = apiConfiguration.openRouterSpecificProvider
if (specificProvider && openRouterModelProviders[specificProvider]) {
@ -162,22 +162,22 @@ function getSelectedModel({
}
case "requesty": {
const id = apiConfiguration.requestyModelId ?? requestyDefaultModelId
const info = routerModels.requesty[id]
const info = routerModels.requesty?.[id]
return { id, info }
}
case "glama": {
const id = apiConfiguration.glamaModelId ?? glamaDefaultModelId
const info = routerModels.glama[id]
const info = routerModels.glama?.[id]
return { id, info }
}
case "unbound": {
const id = apiConfiguration.unboundModelId ?? unboundDefaultModelId
const info = routerModels.unbound[id]
const info = routerModels.unbound?.[id]
return { id, info }
}
case "litellm": {
const id = apiConfiguration.litellmModelId ?? litellmDefaultModelId
const info = routerModels.litellm[id]
const info = routerModels.litellm?.[id]
return { id, info }
}
case "xai": {
@ -202,7 +202,7 @@ function getSelectedModel({
}
case "chutes": {
const id = apiConfiguration.apiModelId ?? chutesDefaultModelId
const info = routerModels.chutes[id]
const info = routerModels.chutes?.[id]
return { id, info }
}
case "bedrock": {
@ -354,7 +354,7 @@ function getSelectedModel({
case "roo": {
// Roo is a dynamic provider - models are loaded from API
const id = apiConfiguration.apiModelId ?? rooDefaultModelId
const info = routerModels.roo[id]
const info = routerModels.roo?.[id]
return { id, info }
}
case "qwen-code": {