fix: display actual API error message for 400+ errors instead of i18n strings

hannesrudolph · hannesrudolph · commit e3d75a08906f · 2025-12-09T13:32:53.000-07:00
Show the actual error text from the API response as the main error message
for HTTP 400+ errors, rather than using generic i18n translated strings.
This provides users with specific, actionable error information directly.
diff --git a/webview-ui/src/components/chat/ChatRow.tsx b/webview-ui/src/components/chat/ChatRow.tsx
@@ -162,7 +162,7 @@ export const ChatRowContent = ({
 	onBatchFileResponse,
 	isFollowUpAnswered,
 }: ChatRowContentProps) => {
-	const { t, i18n } = useTranslation()
+	const { t } = useTranslation()
 
 	const { mcpServers, alwaysAllowMcp, currentCheckpoint, mode, apiConfiguration, clineMessages } = useExtensionState()
 	const { info: model } = useSelectedModel(apiConfiguration)
@@ -1110,45 +1110,27 @@ export const ChatRowContent = ({
 					)
 				case "api_req_retry_delayed":
 					let body = t(`chat:apiRequest.failed`)
-					let retryInfo, code, docsURL
+					let code, docsURL
 					if (message.text !== undefined) {
-						// Try to show richer error message for that code, if available
+						// Check if error message starts with an HTTP status code (400+)
 						const potentialCode = parseInt(message.text.substring(0, 3))
 						if (potentialCode >= 400) {
 							code = potentialCode
-							const stringForError = `chat:apiRequest.errorMessage.${code}`
-							if (i18n.exists(stringForError)) {
-								body = t(stringForError)
-								// Fill this out in upcoming PRs
-								// Do not remove this
-								// switch(code) {
-								// 	case ERROR_CODE:
-								// 		docsURL = ???
-								// 		break;
-								// }
-							} else {
-								body = t("chat:apiRequest.errorMessage.unknown")
-								docsURL = "mailto:support@roocode.com?subject=Unknown API Error"
-							}
-							retryInfo = (
-								<p className="mt-1 font-light text-xs text-vscode-errorForeground/80 cursor-default">
-									{message.text.substring(4)}
-								</p>
-							)
+							// Display the actual error message (after the status code prefix)
+							body = message.text.substring(4)
+							// Fill this out in upcoming PRs
+							// Do not remove this
+							// switch(code) {
+							// 	case ERROR_CODE:
+							// 		docsURL = ???
+							// 		break;
+							// }
 						} else {
 							// Non-HTTP-status-code error message - display the actual error text
 							body = message.text
 						}
 					}
-					return (
-						<ErrorRow
-							type="api_req_retry_delayed"
-							code={code}
-							message={body}
-							docsURL={docsURL}
-							additionalContent={retryInfo}
-						/>
-					)
+					return <ErrorRow type="api_req_retry_delayed" code={code} message={body} docsURL={docsURL} />
 				case "api_req_finished":
 					return null // we should never see this message type
 				case "text":
