Skip to content

Commit 91cba93

Browse files
hannesrudolphdaniel-lxs
authored andcommitted
fix: address critical PR review feedback
- Fix error classification to include 'required' keyword for configuration errors - Add localization for error type headings in CodeIndexSettings - Update all language files with appropriate translations for error types
1 parent 432ddad commit 91cba93

File tree

20 files changed

+122
-24
lines changed

20 files changed

+122
-24
lines changed

src/services/code-index/manager.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,11 @@ export class CodeIndexManager {
264264
let errorMessage = error instanceof Error ? error.message : String(error)
265265
let suggestion: string | undefined
266266

267-
if (errorMessage.includes("configuration missing") || errorMessage.includes("missing for")) {
267+
if (
268+
errorMessage.includes("configuration missing") ||
269+
errorMessage.includes("missing for") ||
270+
errorMessage.includes("required")
271+
) {
268272
errorType = "configuration"
269273
suggestion = "Please check your embedder configuration in the settings."
270274
} else if (errorMessage.includes("authentication") || errorMessage.includes("API key")) {

webview-ui/src/components/settings/CodeIndexSettings.tsx

Lines changed: 9 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -272,12 +272,16 @@ export const CodeIndexSettings: React.FC<CodeIndexSettingsProps> = ({
272272
<span className="codicon codicon-error text-vscode-inputValidation-errorForeground flex-shrink-0 mt-0.5"></span>
273273
<div className="flex-1">
274274
<div className="text-sm font-medium text-vscode-inputValidation-errorForeground mb-1">
275-
{indexingStatus.errorDetails.type === "configuration" && "Configuration Error"}
275+
{indexingStatus.errorDetails.type === "configuration" &&
276+
t("settings:codeIndex.configurationError")}
276277
{indexingStatus.errorDetails.type === "authentication" &&
277-
"Authentication Error"}
278-
{indexingStatus.errorDetails.type === "network" && "Network Error"}
279-
{indexingStatus.errorDetails.type === "validation" && "Validation Error"}
280-
{indexingStatus.errorDetails.type === "unknown" && "Unknown Error"}
278+
t("settings:codeIndex.authenticationError")}
279+
{indexingStatus.errorDetails.type === "network" &&
280+
t("settings:codeIndex.networkError")}
281+
{indexingStatus.errorDetails.type === "validation" &&
282+
t("settings:codeIndex.validationError")}
283+
{indexingStatus.errorDetails.type === "unknown" &&
284+
t("settings:codeIndex.unknownError")}
281285
</div>
282286
<div className="text-sm text-vscode-foreground mb-2">
283287
{indexingStatus.errorDetails.message}

webview-ui/src/i18n/locales/ca/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Esborrar dades"
7474
},
7575
"testConfigButton": "Prova la configuració",
76-
"testingButton": "Provant..."
76+
"testingButton": "Provant...",
77+
"configurationError": "Error de configuració",
78+
"authenticationError": "Error d'autenticació",
79+
"networkError": "Error de xarxa",
80+
"validationError": "Error de validació",
81+
"unknownError": "Error desconegut"
7782
},
7883
"autoApprove": {
7984
"description": "Permet que Roo realitzi operacions automàticament sense requerir aprovació. Activeu aquesta configuració només si confieu plenament en la IA i enteneu els riscos de seguretat associats.",

webview-ui/src/i18n/locales/de/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Daten löschen"
7474
},
7575
"testConfigButton": "Konfiguration testen",
76-
"testingButton": "Testen..."
76+
"testingButton": "Testen...",
77+
"configurationError": "Konfigurationsfehler",
78+
"authenticationError": "Authentifizierungsfehler",
79+
"networkError": "Netzwerkfehler",
80+
"validationError": "Validierungsfehler",
81+
"unknownError": "Unbekannter Fehler"
7782
},
7883
"autoApprove": {
7984
"description": "Erlaubt Roo, Operationen automatisch ohne Genehmigung durchzuführen. Aktiviere diese Einstellungen nur, wenn du der KI vollständig vertraust und die damit verbundenen Sicherheitsrisiken verstehst.",

webview-ui/src/i18n/locales/en/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"description": "This action cannot be undone. This will permanently delete your codebase index data.",
7474
"cancelButton": "Cancel",
7575
"confirmButton": "Clear Data"
76-
}
76+
},
77+
"configurationError": "Configuration Error",
78+
"authenticationError": "Authentication Error",
79+
"networkError": "Network Error",
80+
"validationError": "Validation Error",
81+
"unknownError": "Unknown Error"
7782
},
7883
"autoApprove": {
7984
"description": "Allow Roo to automatically perform operations without requiring approval. Enable these settings only if you fully trust the AI and understand the associated security risks.",

webview-ui/src/i18n/locales/es/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Borrar datos"
7474
},
7575
"testConfigButton": "Probar configuración",
76-
"testingButton": "Probando..."
76+
"testingButton": "Probando...",
77+
"configurationError": "Error de configuración",
78+
"authenticationError": "Error de autenticación",
79+
"networkError": "Error de red",
80+
"validationError": "Error de validación",
81+
"unknownError": "Error desconocido"
7782
},
7883
"autoApprove": {
7984
"description": "Permitir que Roo realice operaciones automáticamente sin requerir aprobación. Habilite esta configuración solo si confía plenamente en la IA y comprende los riesgos de seguridad asociados.",

webview-ui/src/i18n/locales/fr/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Effacer les données"
7474
},
7575
"testConfigButton": "Tester la configuration",
76-
"testingButton": "Test en cours..."
76+
"testingButton": "Test en cours...",
77+
"configurationError": "Erreur de configuration",
78+
"authenticationError": "Erreur d'authentification",
79+
"networkError": "Erreur réseau",
80+
"validationError": "Erreur de validation",
81+
"unknownError": "Erreur inconnue"
7782
},
7883
"autoApprove": {
7984
"description": "Permettre à Roo d'effectuer automatiquement des opérations sans requérir d'approbation. Activez ces paramètres uniquement si vous faites entièrement confiance à l'IA et que vous comprenez les risques de sécurité associés.",

webview-ui/src/i18n/locales/hi/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "डेटा साफ़ करें"
7474
},
7575
"testConfigButton": "कॉन्फ़िगरेशन का परीक्षण करें",
76-
"testingButton": "परीक्षण हो रहा है..."
76+
"testingButton": "परीक्षण हो रहा है...",
77+
"configurationError": "कॉन्फ़िगरेशन त्रुटि",
78+
"authenticationError": "प्रमाणीकरण त्रुटि",
79+
"networkError": "नेटवर्क त्रुटि",
80+
"validationError": "सत्यापन त्रुटि",
81+
"unknownError": "अज्ञात त्रुटि"
7782
},
7883
"autoApprove": {
7984
"description": "Roo को अनुमोदन की आवश्यकता के बिना स्वचालित रूप से ऑपरेशन करने की अनुमति दें। इन सेटिंग्स को केवल तभी सक्षम करें जब आप AI पर पूरी तरह से भरोसा करते हों और संबंधित सुरक्षा जोखिमों को समझते हों।",

webview-ui/src/i18n/locales/id/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Hapus Data"
7474
},
7575
"testConfigButton": "Uji Konfigurasi",
76-
"testingButton": "Menguji..."
76+
"testingButton": "Menguji...",
77+
"configurationError": "Kesalahan Konfigurasi",
78+
"authenticationError": "Kesalahan Otentikasi",
79+
"networkError": "Kesalahan Jaringan",
80+
"validationError": "Kesalahan Validasi",
81+
"unknownError": "Kesalahan Tidak Dikenal"
7782
},
7883
"autoApprove": {
7984
"description": "Izinkan Roo untuk secara otomatis melakukan operasi tanpa memerlukan persetujuan. Aktifkan pengaturan ini hanya jika kamu sepenuhnya mempercayai AI dan memahami risiko keamanan yang terkait.",

webview-ui/src/i18n/locales/it/settings.json

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,12 @@
7373
"confirmButton": "Cancella dati"
7474
},
7575
"testConfigButton": "Testa configurazione",
76-
"testingButton": "Test in corso..."
76+
"testingButton": "Test in corso...",
77+
"configurationError": "Errore di configurazione",
78+
"authenticationError": "Errore di autenticazione",
79+
"networkError": "Errore di rete",
80+
"validationError": "Errore di validazione",
81+
"unknownError": "Errore sconosciuto"
7782
},
7883
"autoApprove": {
7984
"description": "Permetti a Roo di eseguire automaticamente operazioni senza richiedere approvazione. Abilita queste impostazioni solo se ti fidi completamente dell'IA e comprendi i rischi di sicurezza associati.",

0 commit comments

Comments
 (0)