Update js/iaConfigModule.js

js/iaConfigModule.js

@@ -18,14 +18,18 @@ export const llmProviders = [
     name: "OpenAI", 
     value: "openai", 
     models: [
-      "gpt-5-2025-05-01",         // ChatGPT-5 estándar
-      "gpt-5-mini-2025-05-01",    // ChatGPT-5 mini
+      // GPT-5 (IDs reales de la API)
+      "gpt-5",
+      "gpt-5-mini",
+      "gpt-5-nano",
+      "gpt-5-chat-latest",   // versión no-razonadora usada en ChatGPT
+      // Otros modelos aún comunes/compatibles en muchas cuentas
       "gpt-4o-mini-2024-07-18",
       "chatgpt-4o-latest",
       "o1-mini-2024-09-12",
       "o4-mini-2025-04-16"
-    ], 
-    url: "https://api.openai.com" 
+    ],
+    url: "https://api.openai.com"
   },
   { 
     name: "DeepSeek", 
@@ -34,6 +38,7 @@ export const llmProviders = [
     url: "https://api.deepseek.com" 
   }
 ];
+
 export const transcriptionProviders = [
   { name: "OpenAI Whisper", value: "openai", models: ["whisper-1"], url: "https://api.openai.com" },
   { name: "Deepgram", value: "deepgram", models: ["nova-2", "whisper-large"], url: "https://api.deepgram.com" }
@@ -42,8 +47,10 @@ export const transcriptionProviders = [
 function saveConfig(config) {
   localStorage.setItem("iaConfig", JSON.stringify(config));
 }
+
 function loadConfig() {
-  const config = JSON.parse(localStorage.getItem("iaConfig")) || defaultConfig;
+  const config = JSON.parse(localStorage.getItem("iaConfig")) || structuredClone(defaultConfig);
+
   // Migrar configuración antigua de transcription
   if (config.transcription.apiKey !== undefined) {
     const oldKey = config.transcription.apiKey;
@@ -54,6 +61,7 @@ function loadConfig() {
     delete config.transcription.model;
     saveConfig(config);
   }
+
   // Migrar configuración antigua de LLM apiKey a apiKeys
   if (config.llm.apiKey !== undefined) {
     const old = config.llm.apiKey;
@@ -61,14 +69,24 @@ function loadConfig() {
     delete config.llm.apiKey;
     saveConfig(config);
   }
+
   // Migrar modelos obsoletos de DeepSeek a 'deepseek-chat'
-  if (config.llm.provider === 'deepseek' && (config.llm.model === 'deepseek-v3' || config.llm.model === 'deepseek-llm')) {
-    config.llm.model = 'deepseek-chat';
-    console.log('[iaConfigModule] Migrado modelo DeepSeek a deepseek-chat');
+  if (config.llm.provider === "deepseek" && (config.llm.model === "deepseek-v3" || config.llm.model === "deepseek-llm")) {
+    config.llm.model = "deepseek-chat";
+    console.log("[iaConfigModule] Migrado modelo DeepSeek a deepseek-chat");
     saveConfig(config);
   }
+
+  // ✅ Migrar IDs inventados de GPT-5 a los oficiales
+  if (config.llm.provider === "openai") {
+    if (config.llm.model === "gpt-5-2025-05-01") config.llm.model = "gpt-5";
+    if (config.llm.model === "gpt-5-mini-2025-05-01") config.llm.model = "gpt-5-mini";
+    saveConfig(config);
+  }
+
   return config;
 }
+
 export function getIaConfig() {
   return loadConfig();
 }
@@ -78,14 +96,17 @@ export function renderIaConfigForm(containerId) {
   const container = document.getElementById(containerId);
   if (!container) {
     console.error(`[iaConfigModule] No se encontró el contenedor '${containerId}'`);
-    document.body.insertAdjacentHTML('beforeend', `<div style='color:red'>[Error] No se encontró el contenedor '${containerId}' para la configuración IA.</div>`);
+    document.body.insertAdjacentHTML(
+      "beforeend",
+      `<div style='color:red'>[Error] No se encontró el contenedor '${containerId}' para la configuración IA.</div>`
+    );
     return;
   }
 
   function maskApiKey(key) {
-    if (!key) return '';
-    if (key.length <= 8) return '*'.repeat(key.length);
-    return key.substring(0, 3) + '-****' + key.slice(-4);
+    if (!key) return "";
+    if (key.length <= 8) return "*".repeat(key.length);
+    return key.substring(0, 3) + "-****" + key.slice(-4);
   }
 
   container.innerHTML = `
@@ -141,9 +162,9 @@ export function renderIaConfigForm(containerId) {
 
   // Set initial values
   document.getElementById("llmProvider").value = config.llm.provider;
-  document.getElementById("llmApiKey").value = maskApiKey(config.llm.apiKeys[config.llm.provider] || '');
+  document.getElementById("llmApiKey").value = maskApiKey(config.llm.apiKeys[config.llm.provider] || "");
   document.getElementById("transProvider").value = config.transcription.provider;
-  document.getElementById("transApiKey").value = maskApiKey(config.transcription.apiKeys[config.transcription.provider] || '');
+  document.getElementById("transApiKey").value = maskApiKey(config.transcription.apiKeys[config.transcription.provider] || "");
 
   // API Key toggle (ver/ocultar)
   document.getElementById("toggleLlmApiKey").addEventListener("click", () => {
@@ -162,8 +183,15 @@ export function renderIaConfigForm(containerId) {
     const models = providerObj.models;
     const sel = document.getElementById("llmModel");
     sel.innerHTML = models.map(m => `<option value="${m}">${m}</option>`).join("");
+
+    // Si el modelo guardado no está en la lista del proveedor, usar el primero y persistir
+    if (!models.includes(config.llm.model)) {
+      config.llm.model = models[0];
+      saveConfig(config);
+    }
     sel.value = config.llm.model;
   }
+
   function updateTransModels() {
     const prov = document.getElementById("transProvider").value;
     const providerObj = transcriptionProviders.find(p => p.value === prov);
@@ -171,56 +199,78 @@ export function renderIaConfigForm(containerId) {
     const sel = document.getElementById("transModel");
     sel.innerHTML = models.map(m => `<option value="${m}">${m}</option>`).join("");
     sel.value = config.transcription.models[prov] || models[0];
-    document.getElementById("transApiKey").value = maskApiKey(config.transcription.apiKeys[prov] || '');
+    // Actualizar API Key input al cambiar proveedor
+    document.getElementById("transApiKey").value = maskApiKey(config.transcription.apiKeys[prov] || "");
   }
+
   document.getElementById("llmProvider").addEventListener("change", () => {
     const p = document.getElementById("llmProvider").value;
     updateLlmModels();
+    // Refrescar config con todas las llaves y mostrar la del proveedor seleccionado
     const fresh = loadConfig();
     const keyEl = document.getElementById("llmApiKey");
-    if (keyEl) keyEl.value = maskApiKey(fresh.llm.apiKeys[p] || '');
+    if (keyEl) keyEl.value = maskApiKey(fresh.llm.apiKeys[p] || "");
   });
+
   document.getElementById("transProvider").addEventListener("change", updateTransModels);
+
   updateLlmModels();
   updateTransModels();
 
   // Save on submit
   document.getElementById("iaConfigForm").addEventListener("submit", e => {
     e.preventDefault();
+    // Persistir configuración por proveedor
     const prev = config;
     const newConfig = { ...prev };
+
+    // LLM: provider, apiKeys y model
     const llProv = document.getElementById("llmProvider").value;
     const rawKey = document.getElementById("llmApiKey").value;
-    const oldKey = prev.llm.apiKeys[llProv] || '';
+    const oldKey = prev.llm.apiKeys[llProv] || "";
     const newKey = rawKey === maskApiKey(oldKey) ? oldKey : rawKey;
+
     newConfig.llm = { ...prev.llm, provider: llProv, model: document.getElementById("llmModel").value };
     newConfig.llm.apiKeys = { ...prev.llm.apiKeys, [llProv]: newKey };
+
+    // Transcription: provider, apiKeys y models por proveedor
     const tp = document.getElementById("transProvider").value;
     const rawKeyTrans = document.getElementById("transApiKey").value;
-    const existingKeyTrans = prev.transcription.apiKeys[tp] || '';
+    const existingKeyTrans = prev.transcription.apiKeys[tp] || "";
     const actualKeyTrans = rawKeyTrans === maskApiKey(existingKeyTrans) ? existingKeyTrans : rawKeyTrans;
+
     newConfig.transcription.provider = tp;
     newConfig.transcription.apiKeys = { ...prev.transcription.apiKeys, [tp]: actualKeyTrans };
     newConfig.transcription.models = { ...prev.transcription.models, [tp]: document.getElementById("transModel").value };
-    console.log('[iaConfigModule] Configuración guardada:', newConfig);
+
+    console.log("[iaConfigModule] Configuración guardada:", newConfig);
     saveConfig(newConfig);
+    // Actualizar config local después de guardar
     config = newConfig;
-    document.dispatchEvent(new CustomEvent('iaConfigChanged'));
-    document.getElementById("llmApiKey").value = maskApiKey(newConfig.llm.apiKeys[newConfig.llm.provider] || '');
-    document.getElementById("transApiKey").value = maskApiKey(newConfig.transcription.apiKeys[tp] || '');
+
+    // Notificar cambio
+    document.dispatchEvent(new CustomEvent("iaConfigChanged"));
+
+    // Ofuscar de nuevo las API keys en UI
+    document.getElementById("llmApiKey").value = maskApiKey(newConfig.llm.apiKeys[newConfig.llm.provider] || "");
+    document.getElementById("transApiKey").value = maskApiKey(newConfig.transcription.apiKeys[tp] || "");
     document.getElementById("llmApiKey").type = "password";
     document.getElementById("transApiKey").type = "password";
-    let msg = document.getElementById('iaConfigSavedMsg');
+
+    // Mensaje de éxito discreto
+    let msg = document.getElementById("iaConfigSavedMsg");
     if (!msg) {
-      msg = document.createElement('div');
-      msg.id = 'iaConfigSavedMsg';
-      msg.className = 'fixed left-1/2 top-6 -translate-x-1/2 bg-green-500 text-white px-6 py-3 rounded shadow text-lg z-50';
+      msg = document.createElement("div");
+      msg.id = "iaConfigSavedMsg";
+      msg.className = "fixed left-1/2 top-6 -translate-x-1/2 bg-green-500 text-white px-6 py-3 rounded shadow text-lg z-50";
       msg.innerHTML = '<i class="fas fa-check-circle mr-2"></i>¡Configuración guardada!';
       document.body.appendChild(msg);
     } else {
-      msg.style.display = 'block';
+      msg.style.display = "block";
     }
-    setTimeout(() => { msg.style.display = 'none'; }, 2000);
+    setTimeout(() => { msg.style.display = "none"; }, 2000);
+
+    // Cierra el modal
     const modal = document.getElementById("configModal");
     if (modal) modal.classList.remove("active");
   });