Ensures URL prompt confirmation if key is missing

Allows easy reset of the key and change URL
Refs: #4263

Author: eamodio

src/plus/ai/azureProvider.ts

@@ -27,13 +27,16 @@ export class AzureProvider extends OpenAICompatibleProviderBase<typeof provider.
 		return configuration.get('ai.azure.url') ?? undefined;
 	}
 
-	private async getOrPromptBaseUrl(silent: boolean): Promise<string | undefined> {
+	private async getOrPromptBaseUrl(silent: boolean, hasApiKey: boolean): Promise<string | undefined> {
 		let url: string | undefined = this.getUrl();
 
-		if (silent || url != null) return url;
+		if (silent || (url != null && hasApiKey)) return url;
 
 		const input = window.createInputBox();
 		input.ignoreFocusOut = true;
+		if (url) {
+			input.value = url;
+		}
 
 		const disposables: Disposable[] = [];
 
@@ -94,10 +97,12 @@ export class AzureProvider extends OpenAICompatibleProviderBase<typeof provider.
 	}
 
 	override async configured(silent: boolean): Promise<boolean> {
-		const url = await this.getOrPromptBaseUrl(silent);
-		if (url == null) return false;
+		const hasApiKey = await super.configured(true);
+
+		const url = await this.getOrPromptBaseUrl(silent, hasApiKey);
+		if (!url) return false;
 
-		return super.configured(silent);
+		return silent ? hasApiKey : super.configured(silent);
 	}
 
 	protected override getHeaders<TAction extends AIActionType>(

src/plus/ai/openAICompatibleProvider.ts

@@ -27,13 +27,16 @@ export class OpenAICompatibleProvider extends OpenAICompatibleProviderBase<typeo
 		return configuration.get('ai.openaicompatible.url') ?? undefined;
 	}
 
-	private async getOrPromptBaseUrl(silent: boolean): Promise<string | undefined> {
+	private async getOrPromptBaseUrl(silent: boolean, hasApiKey: boolean): Promise<string | undefined> {
 		let url: string | undefined = this.getUrl();
 
-		if (silent || url != null) return url;
+		if (silent || (url != null && hasApiKey)) return url;
 
 		const input = window.createInputBox();
 		input.ignoreFocusOut = true;
+		if (url) {
+			input.value = url;
+		}
 
 		const disposables: Disposable[] = [];
 
@@ -94,9 +97,11 @@ export class OpenAICompatibleProvider extends OpenAICompatibleProviderBase<typeo
 	}
 
 	override async configured(silent: boolean): Promise<boolean> {
-		const url = await this.getOrPromptBaseUrl(silent);
-		if (url == null || isAzureUrl(url)) return false;
+		const hasApiKey = await super.configured(true);
+
+		const url = await this.getOrPromptBaseUrl(silent, hasApiKey);
+		if (!url || isAzureUrl(url)) return false;
 
-		return super.configured(silent);
+		return silent ? hasApiKey : super.configured(silent);
 	}
 }