refactor(common): toolbox config api

[renbaoshuo]

自查 PR 结构

• PR 标题符合这个格式: <type>(optional scope): <description>

• 此 PR 标题的描述以用户为导向，足够清晰，其他人可以理解。

• 我已经对所有 commit 提供了签名（GPG 密钥签名、SSH 密钥签名）

• 这个 PR 属于强制变更/破坏性更改

如果是，请在 PR 标题中添加 BREAKING CHANGE 前缀，并在 PR 描述中详细说明。

这个 PR 的类型是什么？

refactor

这个 PR 做了什么 / 我们为什么需要这个 PR？

重构工具箱管理接口。

(可选)这个 PR 解决了哪个/些 issue？

对 Reviewer 预留的一些提醒

[codecov]

Codecov Report

❌ Patch coverage is 82.55319% with 41 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
internal/common/service/manage_toolbox_config.go	70.37%	8 Missing and 8 partials ⚠️
api/handler/api/common_service.go	87.23%	7 Missing and 5 partials ⚠️
pkg/db/toolbox/get_configs.go	71.42%	5 Missing and 1 partial ⚠️
pkg/db/toolbox/update_config.go	82.60%	2 Missing and 2 partials ⚠️
pkg/db/toolbox/delete_config.go	81.81%	1 Missing and 1 partial ⚠️
pkg/db/toolbox/create_config.go	83.33%	1 Missing ⚠️

@@            Coverage Diff             @@
##             main     #509      +/-   ##
==========================================
+ Coverage   63.51%   64.07%   +0.56%     
==========================================
  Files         234      239       +5     
  Lines        6169     6450     +281     
==========================================
+ Hits         3918     4133     +215     
- Misses       2127     2162      +35     
- Partials      124      155      +31     

Flag	Coverage Δ
unittest	64.07% <82.55%> (+0.56%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
internal/common/service/get_toolbox_config.go	100.00% <100.00%> (ø)
internal/common/service/list_toolbox_configs.go	80.95% <100.00%> (ø)
pkg/db/toolbox/create_config.go	71.42% <83.33%> (ø)
pkg/db/toolbox/delete_config.go	81.81% <81.81%> (ø)
pkg/db/toolbox/update_config.go	82.60% <82.60%> (ø)
pkg/db/toolbox/get_configs.go	71.42% <71.42%> (+1.86%)	⬆️
api/handler/api/common_service.go	91.86% <87.23%> (-8.14%)	⬇️
internal/common/service/manage_toolbox_config.go	70.37% <70.37%> (ø)

... and 5 files with indirect coverage changes

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[renbaoshuo]

那得改一下表定义了

…

--- Best Regards, Baoshuo https://baoshuo.ren Cai ***@***.***> 于 2026年8月2日周日 17:14写道：

***@***.**** commented on this pull request. ------------------------------ On pkg/db/toolbox/manage_configs_test.go <#509 (comment)> : 应该和其他测试一样写成 testCase 子测试形式 ------------------------------ In pkg/db/toolbox/create_config.go <#509 (comment)> : > - {Name: "platform"}, - {Name: "version"}, - }, - DoUpdates: clause.AssignmentColumns([]string{ - "visible", - "name", - "icon", - "type", - "message", - "extra", - "updated_at", - }), - }).Create(config).Error +func (c *DBToolbox) CreateToolboxConfig(ctx context.Context, config *model.ToolboxConfig) error { + err := c.client.WithContext(ctx).Table(constants.ToolboxConfigTableName).Create(config).Error + if errors.Is(err, gorm.ErrDuplicatedKey) { 现在建表语句 UNIQUE KEY `uk_toolbox_config` (`tool_id`, `student_id`, `platform`, `version`) 因为MySQL里NULL!=NULL，所以student_id, platform, version只要有一个为NULL就不受这个约束影响，可以无限插入，可能不太符合预期，需要业务层判断一下重复 (? — Reply to this email directly, view it on GitHub <#509?email_source=notifications&email_token=ALHJ6YCJDEVGM4R24CVAZJT5H4A7JA5CNFSNUABKM5UWIORPF5TWS5BNNB2WEL2QOVWGYUTFOF2WK43UKJSXM2LFO4XTIOBTG44DEOBQGEY2M4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#pullrequestreview-4837828011>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ALHJ6YGY6VMSNRETJQFAIP35H4A7JAVCNFSNUABFKJSXA33TNF2G64TZHM3DQOJYG44DIOBRHNEXG43VMU5TKMBSGYYTONZTGIY2C5QC> . You are receiving this because you authored the thread.Message ID: ***@***.***>