fix: refine default model selection to include vendor check for 'copilot' (#299064)

* fix: refine default model selection to include vendor check for 'copilot'

* fix tests

* try this

* fix these too

Author: TylerLeonhardt

extensions/vscode-api-tests/package.json

@@ -68,7 +68,7 @@
   "contributes": {
     "languageModelChatProviders": [
       {
-        "vendor": "test-lm-vendor",
+        "vendor": "copilot",
         "displayName": "Test LM Vendor"
       }
     ],

extensions/vscode-api-tests/src/singlefolder-tests/chat.test.ts

@@ -17,7 +17,7 @@ suite('chat', () => {
 		disposables = [];
 
 		// Register a dummy default model which is required for a participant request to go through
-		disposables.push(lm.registerLanguageModelChatProvider('test-lm-vendor', {
+		disposables.push(lm.registerLanguageModelChatProvider('copilot', {
 			async provideLanguageModelChatInformation(_options, _token) {
 				return [{
 					id: 'test-lm',

extensions/vscode-api-tests/src/singlefolder-tests/lm.test.ts

@@ -40,7 +40,7 @@ suite('lm', function () {
 		const defer = new DeferredPromise<void>();
 
 		try {
-			disposables.push(vscode.lm.registerLanguageModelChatProvider('test-lm-vendor', {
+			disposables.push(vscode.lm.registerLanguageModelChatProvider('copilot', {
 				async provideLanguageModelChatInformation(_options, _token) {
 					return [testProviderOptions];
 				},
@@ -92,7 +92,7 @@ suite('lm', function () {
 
 	test('lm request fail', async function () {
 
-		disposables.push(vscode.lm.registerLanguageModelChatProvider('test-lm-vendor', {
+		disposables.push(vscode.lm.registerLanguageModelChatProvider('copilot', {
 			async provideLanguageModelChatInformation(_options, _token) {
 				return [testProviderOptions];
 			},
@@ -119,7 +119,7 @@ suite('lm', function () {
 
 		const defer = new DeferredPromise<void>();
 
-		disposables.push(vscode.lm.registerLanguageModelChatProvider('test-lm-vendor', {
+		disposables.push(vscode.lm.registerLanguageModelChatProvider('copilot', {
 			async provideLanguageModelChatInformation(_options, _token) {
 				return [testProviderOptions];
 			},
@@ -157,7 +157,7 @@ suite('lm', function () {
 
 	test('LanguageModelError instance is not thrown to extensions#235322 (SYNC)', async function () {
 
-		disposables.push(vscode.lm.registerLanguageModelChatProvider('test-lm-vendor', {
+		disposables.push(vscode.lm.registerLanguageModelChatProvider('copilot', {
 			async provideLanguageModelChatInformation(_options, _token) {
 				return [testProviderOptions];
 			},
@@ -183,7 +183,7 @@ suite('lm', function () {
 
 	test('LanguageModelError instance is not thrown to extensions#235322 (ASYNC)', async function () {
 
-		disposables.push(vscode.lm.registerLanguageModelChatProvider('test-lm-vendor', {
+		disposables.push(vscode.lm.registerLanguageModelChatProvider('copilot', {
 			async provideLanguageModelChatInformation(_options, _token) {
 				return [testProviderOptions];
 			},

src/vs/workbench/api/common/extHostLanguageModels.ts

@@ -363,7 +363,8 @@ export class ExtHostLanguageModels implements ExtHostLanguageModelsShape {
 		}
 
 		for (const [modelIdentifier, modelData] of this._localModels) {
-			if (modelData.metadata.isDefaultForLocation[ChatAgentLocation.Chat]) {
+			// TODO: use product.json to determine default vendor
+			if (modelData.metadata.isDefaultForLocation[ChatAgentLocation.Chat] && modelData.metadata.vendor === 'copilot') {
 				defaultModelId = modelIdentifier;
 				break;
 			}