Release V2.2.1 with bugfixes and better http provider support

Author: NikolaiRadke

aiduino/extension/out/utils/apiKeyManager.js

@@ -63,29 +63,55 @@ class ApiKeyManager {
                 }
             })
 
-            if (input) {
+             if (input) {
+                let finalValue = input;
+        
+               // For HTTP-based local providers: Run model detection
+                if (provider.type === 'local' && provider.httpConfig) {
+                    let normalizedUrl = input;
+    
+                    // Normalize URL: Add default port if missing
+                    try {
+                        const testUrl = new URL(input);
+                        if (!testUrl.port && provider.defaultPort) {
+                            normalizedUrl = `${testUrl.protocol}//${testUrl.hostname}:${provider.defaultPort}${testUrl.pathname}`;
+                        }
+                    } catch (e) {
+                        // Keep original if URL parsing fails
+                    }
+    
+                    // Use existing autoDetectLocalProvider with manual URL
+                    const apiManager = require('./apiManager');
+                    const detected = await apiManager.autoDetectLocalProvider(currentModel, providers, normalizedUrl);
+    
+                    if (detected) {
+                        finalValue = detected;
+                    } else {
+                        vscode.window.showErrorMessage(`${providerName} ${t('errors.localProviderNotRunning')} (${normalizedUrl})`);
+                        return false;
+                    }
+                }
+    
                 // Save API key using fileManager
-                if (fileManager.saveApiKey(currentModel, input, providers)) {
+                if (fileManager.saveApiKey(currentModel, finalValue, providers)) {
                     // Update in-memory API keys
-                    apiKeys[currentModel] = input;
-                    
+                    apiKeys[currentModel] = finalValue;
+        
                     // Update status bar
                     updateStatusBar();
-                    
+        
                     // Show success message
-                    const successMessage = t('messages.apiKeySaved', providerName) || 
-                                         `${providerName} API key saved!`;
+                    const successMessage = t('messages.apiKeySaved', providerName);
                     vscode.window.showInformationMessage(successMessage);
-                    
+        
                     return true;
                 } else {
-                    const errorMessage = t('errors.saveFailed', 'File save failed') || 
-                                        `Failed to save API key: File save failed`;
+                    const errorMessage = t('errors.saveFailed');
                     vscode.window.showErrorMessage(errorMessage);
                     return false;
                 }
             }
-            
+
             return false;
 
         } finally {

aiduino/extension/out/utils/apiManager.js

@@ -91,28 +91,31 @@ async function switchModel(context) {
 
             const provider = minimalModelManager.providers[selected.value];
 
-            // Force auto-detection for local providers
+            // Auto-detection for local HTTP providers (only if not configured)
             if (provider.type === 'local' && provider.httpConfig) {
-                await vscode.window.withProgress({
-                    location: vscode.ProgressLocation.Notification,
-                    title: `Detecting ${provider.name}...`,
-                    cancellable: false
-                }, async () => {
-                    // Clear existing config to force re-detection
-                    delete updatedContext.apiKeys[selected.value];
-                    
-                    // Run auto-detection
-                    const detected = await autoDetectLocalProvider(selected.value, minimalModelManager.providers);
-                    if (detected) {
-                        updatedContext.apiKeys[selected.value] = detected;
-                        fileManager.saveApiKey(selected.value, detected, minimalModelManager.providers);
-                        updateStatusBar();
-                            vscode.window.showInformationMessage(`${provider.name} detected: ${detected.split('|')[0]}`);
-                    } else {
-                        updateStatusBar();
-                            vscode.window.showWarningMessage(`$(warning) ${provider.name} not found...`);
-                    }
-                });
+                // Skip auto-detection if already configured
+                if (updatedContext.apiKeys[selected.value]) {
+                    updateStatusBar();
+                    vscode.window.showInformationMessage(t('messages.modelSwitched', provider.name));
+                } else {
+                    // Run auto-detection only if not configured
+                    await vscode.window.withProgress({
+                        location: vscode.ProgressLocation.Notification,
+                        title: t('messages.operationAlreadyRunning'),
+                        cancellable: false
+                    }, async () => {
+                        const detected = await autoDetectLocalProvider(selected.value, minimalModelManager.providers);
+                        if (detected) {
+                            updatedContext.apiKeys[selected.value] = detected;
+                            fileManager.saveApiKey(selected.value, detected, minimalModelManager.providers);
+                            updateStatusBar();
+                            vscode.window.showInformationMessage(t('messages.apiKeySaved', provider.name));
+                        } else {
+                            updateStatusBar();
+                            vscode.window.showWarningMessage(t('messages.noPath', provider.name));
+                        }
+                    });
+                }
 
             } else {
                 // All non-HTTP local providers: Process providers + Remote providers
@@ -197,19 +200,22 @@ async function validateApiConnection(context) {
  * Auto-detect local HTTP provider
  * @param {string} modelId - Model identifier
  * @param {Object} providers - Provider configurations
+ * @param {string|null} manualUrl - Optional manual URL to test (instead of autoDetectUrls)
  * @returns {Promise<string|null>} Detected URL or null
  */
-async function autoDetectLocalProvider(modelId, providers) {
+async function autoDetectLocalProvider(modelId, providers, manualUrl = null) {
     const provider = providers[modelId];
-    if (!provider?.autoDetectUrls) {
+    if (!provider?.autoDetectUrls && !manualUrl) {
         return null;
     }
 
-    // Hole den passenden HTTP Provider Handler
     const localProviders = require('../localProviders');
     const providerHandler = localProviders.getHttpProvider(provider.name);
 
-    for (const url of provider.autoDetectUrls) {
+    // If manual URL provided, test only that one
+    const urlsToTest = manualUrl ? [manualUrl] : provider.autoDetectUrls;
+    
+    for (const url of urlsToTest) {
         if (await testHttpProvider(url, provider)) {
             if (providerHandler && providerHandler.detectBestModel) {
                 const bestModel = await providerHandler.detectBestModel(
@@ -219,7 +225,6 @@ async function autoDetectLocalProvider(modelId, providers) {
                 );
                 return `${url}|${bestModel || provider.fallback}`;
             }
-            // Andere Provider (Process-basierte): nur URL zurückgeben
             return url;
         }
     }
@@ -241,5 +246,6 @@ module.exports = {
     callAI,
     switchModel,
     setApiKey,
-    validateApiConnection
+    validateApiConnection,
+    autoDetectLocalProvider
 };