From df2c07422b4ef91f910ca280a2667cdc7eaf0726 Mon Sep 17 00:00:00 2001 From: Sandeep Somavarapu Date: Sun, 18 Jan 2026 14:54:13 +0100 Subject: [PATCH] fix skipping models from other vendors (#288713) --- src/vs/workbench/contrib/chat/common/languageModels.ts | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/src/vs/workbench/contrib/chat/common/languageModels.ts b/src/vs/workbench/contrib/chat/common/languageModels.ts index c3d0baa61ad..6127745702a 100644 --- a/src/vs/workbench/contrib/chat/common/languageModels.ts +++ b/src/vs/workbench/contrib/chat/common/languageModels.ts @@ -553,8 +553,14 @@ export class LanguageModelsService implements ILanguageModelsService { allModels.push(...models); const modelIdentifiers = []; for (const m of models) { - // Special case for copilot models - they are all user selectable unless marked otherwise - if (vendorId === 'copilot' && (m.metadata.isUserSelectable || this._modelPickerUserPreferences[m.identifier] === true)) { + if (vendorId === 'copilot') { + // Special case for copilot models - they are all user selectable unless marked otherwise + if (m.metadata.isUserSelectable || this._modelPickerUserPreferences[m.identifier] === true) { + modelIdentifiers.push(m.identifier); + } else { + this._logService.trace(`[LM] Skipping model ${m.identifier} from model picker as it is not user selectable.`); + } + } else { modelIdentifiers.push(m.identifier); } }