fix(error-code): optimize error handling and localization (#398)

Author: mini2s

src/core/costrict/error-code/ErrorCodeManager.ts

@@ -17,6 +17,7 @@ export interface IErrorMap {
 export class ErrorCodeManager {
 	private static instance: ErrorCodeManager
 	private errorMap: IErrorMap = {}
+	private unknownError = { message: t("apiErrors:status.unknown"), solution: t("apiErrors:solution.unknown") }
 	private provider!: ClineProvider
 
 	private constructor() {}
@@ -116,7 +117,6 @@ export class ErrorCodeManager {
 		const isHtml = error?.headers && error.headers["content-type"] && error.headers["content-type"] === "text/html"
 		let rawError = error.error?.metadata?.raw ? JSON.stringify(error.error.metadata.raw, null, 2) : error.message
 		let status = error.status as number
-		const unknownError = { message: t("apiErrors:status.unknown"), solution: t("apiErrors:solution.unknown") }
 		const { code, headers } = error
 		const requestId = headers?.get("x-request-id") ?? null
 		const { apiConfiguration } = await this.provider.getState()
@@ -154,7 +154,7 @@ export class ErrorCodeManager {
 			"quota-manager.voucher_expired",
 		]
 		if (code) {
-			let { message, solution } = this.errorMap[code] || unknownError
+			let { message, solution } = this.errorMap[code] || this.unknownError
 			if (authRequiredCodes.includes(code)) {
 				rawError = message
 				message = defaultError["401"].message
@@ -186,7 +186,7 @@ ${checkRemainingQuotaStr}
 			this.provider.log(`[Costrict#apiErrors] task ${taskId}.${instanceId} Raw Error: ${rawError}`)
 			return `${t("apiErrors:request.error_details")}\n\n${message}\n\n${requestId ? `RequestID: ${requestId}\n\n` : ""}${t("apiErrors:request.solution")}\n${solution}`
 		}
-		const { message, solution } = defaultError[status] || unknownError
+		const { message, solution } = defaultError[status] || this.unknownError
 		if (defaultError[status]) {
 			TelemetryService.instance.captureError(
 				status === undefined ? `ApiError_unknown` : `ApiError_status_${status}`,
@@ -205,4 +205,8 @@ ${checkRemainingQuotaStr}
 			.map((b) => b.toString(16).padStart(2, "0"))
 			.join("")
 	}
+
+	getErrorMessageByCode(code: string) {
+		return this.errorMap[code] || this.unknownError
+	}
 }

src/core/webview/webviewMessageHandler.ts

@@ -59,7 +59,7 @@ import { MarketplaceManager, MarketplaceItemType } from "../../services/marketpl
 import { setPendingTodoList } from "../tools/updateTodoListTool"
 import { ZgsmAuthConfig } from "../costrict/auth"
 import { CodeReviewService, startReview } from "../costrict/code-review"
-import { ZgsmCodebaseIndexManager, IndexSwitchRequest } from "../costrict/codebase-index"
+import { ZgsmCodebaseIndexManager, IndexSwitchRequest, IndexStatusInfo } from "../costrict/codebase-index"
 import { ErrorCodeManager } from "../costrict/error-code"
 import { writeCostrictAccessToken } from "../costrict/codebase-index/utils"
 
@@ -2460,8 +2460,21 @@ export const webviewMessageHandler = async (
 				// Call ZgsmCodebaseIndexManager.getIndexStatus()
 				const zgsmCodebaseIndexManager = ZgsmCodebaseIndexManager.getInstance()
 				const response = await zgsmCodebaseIndexManager.getIndexStatus(workspacePath)
+				const errorCodeManager = ErrorCodeManager.getInstance()
+
+				const updateFailedReason = (item: IndexStatusInfo) => {
+					if (item.status === "failed") {
+						item.failedReason = errorCodeManager.getErrorMessageByCode(item.failedReason).message
+					}
+				}
+
+				if (response?.data?.codegraph) {
+					updateFailedReason(response.data.codegraph)
+				}
+				if (response?.data?.embedding) {
+					updateFailedReason(response.data.embedding)
+				}
 
-				// Return response message
 				await provider.postMessageToWebview({
 					type: "codebaseIndexStatusResponse",
 					payload: {

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

@@ -148,10 +148,10 @@
 			"pendingSync": "Pending Sync",
 			"syncSuccess": "Sync Success",
 			"syncFailed": "Sync Failed",
-			"syncFailedFiles": "Sync Failed Files",
+			"syncFailedFiles": "Sync Failed Details",
 			"viewDetails": "View Details",
-			"syncFailedFilesTitle": "Sync Failed Files",
-			"failedFileList": "Failed Files:",
+			"syncFailedFilesTitle": "Number of Sync Failed Files",
+			"failedFileList": "Failed Files List:",
 			"copy": "Copy",
 			"noFailedFiles": "No failed files information",
 			"rebuild": "Rebuild",

webview-ui/src/i18n/costrict-i18n/locales/zh-CN/settings.json

@@ -148,9 +148,9 @@
 			"pendingSync": "待同步",
 			"syncSuccess": "同步成功",
 			"syncFailed": "同步失败",
-			"syncFailedFiles": "同步失败文件",
+			"syncFailedFiles": "同步失败详情",
 			"viewDetails": "查看详情",
-			"syncFailedFilesTitle": "同步失败文件",
+			"syncFailedFilesTitle": "同步失败文件数量",
 			"failedFileList": "失败文件列表:",
 			"copy": "复制",
 			"noFailedFiles": "暂无失败文件信息",

webview-ui/src/i18n/costrict-i18n/locales/zh-TW/settings.json

@@ -145,9 +145,9 @@
 			"pendingSync": "待同步",
 			"syncSuccess": "同步成功",
 			"syncFailed": "同步失敗",
-			"syncFailedFiles": "同步失敗檔案",
+			"syncFailedFiles": "同步失敗詳情",
 			"viewDetails": "查看詳情",
-			"syncFailedFilesTitle": "同步失敗檔案",
+			"syncFailedFilesTitle": "同步失敗檔案數量",
 			"failedFileList": "失敗檔案列表:",
 			"copy": "複製",
 			"noFailedFiles": "暫無失敗檔案資訊",