Cleaup old exp (#1254)

Author: lramos15

src/extension/conversation/vscode-node/languageModelAccess.ts

@@ -12,7 +12,7 @@ import { IBlockedExtensionService } from '../../../platform/chat/common/blockedE
 import { ChatFetchResponseType, ChatLocation, getErrorDetailsFromChatFetchError } from '../../../platform/chat/common/commonTypes';
 import { getTextPart } from '../../../platform/chat/common/globalStringUtils';
 import { EmbeddingType, getWellKnownEmbeddingTypeInfo, IEmbeddingsComputer } from '../../../platform/embeddings/common/embeddingsComputer';
-import { AutoChatEndpoint, isAutoModelDefault } from '../../../platform/endpoint/common/autoChatEndpoint';
+import { AutoChatEndpoint } from '../../../platform/endpoint/common/autoChatEndpoint';
 import { IAutomodeService } from '../../../platform/endpoint/common/automodeService';
 import { IEndpointProvider } from '../../../platform/endpoint/common/endpointProvider';
 import { CustomDataPartMimeTypes } from '../../../platform/endpoint/common/endpointTypes';
@@ -58,7 +58,6 @@ export class LanguageModelAccess extends Disposable implements IExtensionContrib
 		@IEndpointProvider private readonly _endpointProvider: IEndpointProvider,
 		@IEmbeddingsComputer private readonly _embeddingsComputer: IEmbeddingsComputer,
 		@IVSCodeExtensionContext private readonly _vsCodeExtensionContext: IVSCodeExtensionContext,
-		@IExperimentationService private readonly _expService: IExperimentationService,
 		@IAutomodeService private readonly _automodeService: IAutomodeService
 	) {
 		super();
@@ -113,7 +112,8 @@ export class LanguageModelAccess extends Disposable implements IExtensionContrib
 		let defaultChatEndpoint = chatEndpoints.find(e => e.isDefault) ?? await this._endpointProvider.getChatEndpoint('gpt-4.1') ?? chatEndpoints[0];
 		const autoEndpoint = await this._automodeService.resolveAutoModeEndpoint(undefined, chatEndpoints);
 		chatEndpoints.push(autoEndpoint);
-		if (isAutoModelDefault(this._expService, this._authenticationService)) {
+		// No Auth users always get Auto as the default model
+		if (this._authenticationService.copilotToken?.isNoAuthUser) {
 			defaultChatEndpoint = autoEndpoint;
 		}
 		const seenFamilies = new Set<string>();

src/platform/endpoint/common/autoChatEndpoint.ts

@@ -8,15 +8,13 @@ import { Raw } from '@vscode/prompt-tsx';
 import type { CancellationToken } from 'vscode';
 import { ITokenizer, TokenizerType } from '../../../util/common/tokenizer';
 import { AsyncIterableObject } from '../../../util/vs/base/common/async';
-import { IAuthenticationService } from '../../authentication/common/authentication';
 import { IChatMLFetcher, Source } from '../../chat/common/chatMLFetcher';
 import { ChatLocation, ChatResponse } from '../../chat/common/commonTypes';
 import { ILogService } from '../../log/common/logService';
 import { FinishedCallback, OptionalChatRequestParams } from '../../networking/common/fetch';
 import { Response } from '../../networking/common/fetcherService';
 import { IChatEndpoint, ICreateEndpointBodyOptions, IEndpointBody, IMakeChatRequestOptions } from '../../networking/common/networking';
 import { ChatCompletion } from '../../networking/common/openai';
-import { IExperimentationService } from '../../telemetry/common/nullExperimentationService';
 import { ITelemetryService, TelemetryProperties } from '../../telemetry/common/telemetry';
 import { TelemetryData } from '../../telemetry/common/telemetryData';
 
@@ -116,17 +114,4 @@ export class AutoChatEndpoint implements IChatEndpoint {
 			telemetryProperties,
 		}, token);
 	}
-}
-
-/**
- * Checks if the auto chat model is the default model
- * @param expService The experimentation service to use to check if the auto model is the default
- * @param authService The authentication service to use to check if the auto model is the default
- * @returns True if the auto model is the default, false otherwise
- */
-export function isAutoModelDefault(expService: IExperimentationService, authService: IAuthenticationService) {
-	if (authService.copilotToken?.isNoAuthUser) {
-		return true;
-	}
-	return !!expService.getTreatmentVariable<boolean>('autoModelDefault');
-}
+}