Skip to content

feat: localize fixed SQL execution fail reasons on read path - #3360

Merged
Seechi-Yolo merged 1 commit into
mainfrom
dev-dms-i18n-en-completion
Aug 10, 2026
Merged

feat: localize fixed SQL execution fail reasons on read path#3360
Seechi-Yolo merged 1 commit into
mainfrom
dev-dms-i18n-en-completion

Conversation

@LordofAvernus

@LordofAvernus LordofAvernus commented Aug 7, 2026

Copy link
Copy Markdown
Collaborator

User description

关联的 issue

#3342

描述你的变更

  • 固定失败原因落库仍为中文常量;读路径 LocalizeFixedFailReason 按用户语言本地化已知串
  • 同步 message 与 active en/zh toml

确认项(pr提交后操作)

Tip

请在指定复审人之前,确认并完成以下事项,完成后✅


  • 我已完成自测
  • 我已记录完整日志方便进行诊断
  • 我已在关联的issue里补充了实现方案
  • 我已在关联的issue里补充了测试影响面
  • 我已确认了变更的兼容性,如果不兼容则在issue里标记 not_compatible
  • 我已确认了是否要更新文档,如果要更新则在issue里标记 need_update_doc


Description

  • 为 SQL 执行失败原因增加国际化支持

  • 修改任务转换函数以传递 context 参数

  • 新增本地化函数处理固定失败描述

  • 更新英文及中文国际化配置文件


Diagram Walkthrough

flowchart LR
  A["更新任务控制器"] --> B["传递 Context 参数"]
  B --> C["调用本地化函数"]
  D["新增本地化函数"] --> C
  E["更新国际化消息文件"] --> C
Loading

File Walkthrough

Relevant files
Enhancement
task.go
修改任务转换逻辑以支持国际化                                                                                     

sqle/api/controller/v1/task.go

  • 增加 context 参数至转换函数
  • 本地化 ExecFailReason 字段
  • 更新 CreateAndAuditTask 与 GetTask 调用
+4/-4     
task.go
修改任务 SQL 路径本地化处理                                                                                 

sqle/api/controller/v2/task.go

  • 本地化任务 SQL 执行结果
  • 改用 model.LocalizeFixedFailReason 处理 execResult
+1/-1     
message_zh.go
新增中文国际化消息项                                                                                             

sqle/locale/message_zh.go

  • 新增 SQLNotExecutedReason 提示
  • 新增 OnlineFailReasonFallback 提示
  • 添加 AIHub 系列国际化消息
+7/-0     
task.go
新增本地化函数处理失败原因                                                                                       

sqle/model/task.go

  • 添加 LocalizeFixedFailReason 函数
  • 根据 context 返回本地化失败原因
+13/-1   
active.en.toml
更新英文国际化配置文件                                                                                           

sqle/locale/active.en.toml

  • 增加 AIHub 相关英文消息
  • 调整部分字段翻译及顺序
+16/-11 
active.zh.toml
更新中文国际化配置文件                                                                                           

sqle/locale/active.zh.toml

  • 增加 AIHub 相关中文消息
  • 调整部分字段翻译及描述
+17/-11 

Keep Chinese constants in DB for historical matching; localize known fixed strings via goi18n when assembling API responses. refs #3342
@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

🎫 Ticket compliance analysis 🔶

3342 - Partially compliant

Compliant requirements:

  • 为 SQL 执行失败原因增加国际化支持
  • 修改任务转换函数以传递 context 参数
  • 新增本地化处理函数及更新国际化配置文件

Non-compliant requirements:

Requires further human verification:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
增加nil检查


建议在调用locale.Bundle.LocalizeMsgByCtx之前检查ctx是否为nil,以避免因nil上下文导致的panic问题。这可以提高代码的健壮性,防止潜在的运行时错误。

sqle/model/task.go [154-163]

 func LocalizeFixedFailReason(ctx context.Context, reason string) string {
+	if ctx == nil {
+		return reason
+	}
 	switch reason {
 	case SQLNotExecutedReason:
 		return locale.Bundle.LocalizeMsgByCtx(ctx, locale.SQLNotExecutedReason)
 	case OnlineFailReasonFallback:
 		return locale.Bundle.LocalizeMsgByCtx(ctx, locale.OnlineFailReasonFallback)
 	default:
 		return reason
 	}
 }
Suggestion importance[1-10]: 5

__

Why: The suggestion adds a defensive nil check for ctx before invoking locale.Bundle.LocalizeMsgByCtx, which can prevent a runtime panic if a nil context is passed. However, in typical Go usage, contexts are expected to be non-nil, so while this improves robustness, its overall impact is moderate.

Low

@Seechi-Yolo
Seechi-Yolo merged commit e584e45 into main Aug 10, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants