refactor: use shared resolveAutoModelSelection utility across all features

- Replace manual 'auto' model resolution in autocompleteService with shared utility
- Replace manual 'auto' model resolution in quickEditActions with shared utility
- Replace manual 'auto' model resolution in codeReviewService with shared utility
- Improve error message in sendLLMMessage for auto model selection failures
- Fix duration calculation bug in sendLLMMessage (use Date.now() instead of getMilliseconds)

This ensures consistent auto model selection behavior across all features and reduces code duplication.

Author: Tajudeen

src/vs/workbench/contrib/cortexide/browser/autocompleteService.ts

@@ -791,10 +791,16 @@ export class AutocompleteService extends Disposable implements IAutocompleteServ
 
 		// Detect if using local provider for prefix/suffix optimization
 		const featureName: FeatureName = 'Autocomplete'
-		const modelSelection = this._settingsService.state.modelSelectionOfFeature[featureName]
-		const isLocal = modelSelection && modelSelection.providerName !== 'auto'
-			? isLocalProvider(modelSelection.providerName, this._settingsService.state.settingsOfProvider)
-			: false
+		const modelSelection = this._settingsService.resolveAutoModelSelection(
+			this._settingsService.state.modelSelectionOfFeature[featureName]
+		)
+
+		if (!modelSelection || modelSelection.providerName === 'auto') {
+			// No model available - skip autocomplete
+			return []
+		}
+
+		const isLocal = isLocalProvider(modelSelection.providerName, this._settingsService.state.settingsOfProvider)
 
 		const { shouldGenerate, predictionType, llmPrefix, llmSuffix, stopTokens } = getCompletionOptions(prefixAndSuffix, relevantContext, justAcceptedAutocompletion, isLocal)
 
@@ -822,15 +828,11 @@ export class AutocompleteService extends Disposable implements IAutocompleteServ
 		console.log('starting autocomplete...', predictionType)
 
 		const overridesOfModel = this._settingsService.state.overridesOfModel
-		// Skip "auto" - it's not a real provider
-		const modelSelectionOptions = modelSelection && !(modelSelection.providerName === 'auto' && modelSelection.modelName === 'auto')
-			? this._settingsService.state.optionsOfModelSelection[featureName][modelSelection.providerName]?.[modelSelection.modelName]
-			: undefined
+		// Model selection is already resolved above, so we can safely access options
+		const modelSelectionOptions = this._settingsService.state.optionsOfModelSelection[featureName]?.[modelSelection.providerName]?.[modelSelection.modelName]
 
 		// Warm up local model in background (fire-and-forget, doesn't block)
-		if (modelSelection && modelSelection.providerName !== 'auto' && modelSelection.modelName !== 'auto') {
-			this._modelWarmupService.warmupModelIfNeeded(modelSelection.providerName, modelSelection.modelName, featureName)
-		}
+		this._modelWarmupService.warmupModelIfNeeded(modelSelection.providerName, modelSelection.modelName, featureName)
 
 		// set parameters of `newAutocompletion` appropriately
 		newAutocompletion.llmPromise = new Promise((resolve, reject) => {

src/vs/workbench/contrib/cortexide/browser/quickEditActions.ts

@@ -164,10 +164,12 @@ registerAction2(class extends Action2 {
 
 		if (!instruction) return
 
-		// Check for model selection
-		const modelSelection = settingsService.state.modelSelectionOfFeature['Chat']
+		// Check for model selection and resolve "auto" if needed
+		const modelSelection = settingsService.resolveAutoModelSelection(
+			settingsService.state.modelSelectionOfFeature['Chat']
+		)
 		if (!modelSelection) {
-			notificationService.warn('Please select a model in CortexIDE Settings to use Inline Edit.')
+			notificationService.error('No model provider configured. Please configure a model provider in CortexIDE Settings.')
 			return
 		}
 
@@ -232,11 +234,15 @@ ${contextCode}
 
 				const userMessage = `Edit instruction: ${instruction}\n\nGenerate a SEARCH/REPLACE block for the selected code.`
 
+				// Ensure modelSelection is resolved and not null
+				if (!modelSelection || modelSelection.providerName === 'auto') {
+					notificationService.error('Failed to resolve model selection. Please configure a model provider in CortexIDE Settings.')
+					return
+				}
+
 				const chatOptions = settingsService.state.optionsOfModelSelection['Chat']
-				// Skip "auto" - it's not a real provider
-				const modelOptions = modelSelection && !(modelSelection.providerName === 'auto' && modelSelection.modelName === 'auto')
-					? chatOptions[modelSelection.providerName]?.[modelSelection.modelName]
-					: undefined
+				// Model selection is already resolved above, so we can safely access options
+				const modelOptions = chatOptions[modelSelection.providerName]?.[modelSelection.modelName]
 				const overrides = settingsService.state.overridesOfModel
 
 				requestId = llmMessageService.sendLLMMessage({

src/vs/workbench/contrib/cortexide/common/codeReviewService.ts

@@ -176,10 +176,21 @@ Provide your review annotations as a JSON array:`;
 
 			// Get model selection from settings (use Chat feature model selection)
 			const settings = this.settingsService.state;
-			const modelSelection = settings.modelSelectionOfFeature['Chat'] || { providerName: 'auto', modelName: 'auto' };
-			const modelOptions = modelSelection && !(modelSelection.providerName === 'auto' && modelSelection.modelName === 'auto')
-				? settings.optionsOfModelSelection['Chat']?.[modelSelection.providerName]?.[modelSelection.modelName]
-				: undefined;
+			const modelSelection = this.settingsService.resolveAutoModelSelection(
+				settings.modelSelectionOfFeature['Chat'] || { providerName: 'auto', modelName: 'auto' }
+			);
+
+			if (!modelSelection) {
+				return {
+					uri,
+					annotations: [],
+					summary: 'No model provider configured. Please configure a model provider in CortexIDE Settings.',
+					success: false,
+					error: 'No models available',
+				};
+			}
+
+			const modelOptions = settings.optionsOfModelSelection['Chat']?.[modelSelection.providerName]?.[modelSelection.modelName];
 			const overrides = settings.overridesOfModel;
 
 			// Call LLM directly

src/vs/workbench/contrib/cortexide/electron-main/llmMessage/sendLLMMessage.ts

@@ -68,7 +68,7 @@ export const sendLLMMessage = async ({
 	const onFinalMessage: OnFinalMessage = (params) => {
 		const { fullText, fullReasoning, toolCall } = params
 		if (_didAbort) return
-		captureLLMEvent(`${loggingName} - Received Full Message`, { messageLength: fullText.length, reasoningLength: fullReasoning?.length, duration: new Date().getMilliseconds() - submit_time.getMilliseconds(), toolCallName: toolCall?.name })
+		captureLLMEvent(`${loggingName} - Received Full Message`, { messageLength: fullText.length, reasoningLength: fullReasoning?.length, duration: Date.now() - submit_time.getTime(), toolCallName: toolCall?.name })
 		onFinalMessage_(params)
 	}
 
@@ -109,7 +109,7 @@ export const sendLLMMessage = async ({
 	try {
 		// Skip "auto" - it's not a real provider
 		if (providerName === 'auto') {
-			onError({ message: `Error: Cannot use "auto" provider - must resolve to a real model first.`, fullError: null })
+			onError({ message: `Error: Cannot use "auto" provider - must resolve to a real model first. This usually means auto model selection failed. Please check your model provider settings or select a specific model.`, fullError: null })
 			return
 		}
 		const implementation = sendLLMMessageToProviderImplementation[providerName]