docs: update completion review to reflect P2 handler unification progress
- Mark P1 Service 层 DIP as fully resolved - Note P2 handler format work is partially complete (13/16 handlers fixed) - Remaining handlers to fix: device_handler.go, avatar_handler.go, auth_handler.go
This commit is contained in:
@@ -152,7 +152,7 @@ go run golang.org/x/vuln/cmd/govulncheck@latest ./... # PASS
|
||||
- ✅ "无 N+1 查询" — `GetUserRoles`/`ListAdmins` 改用批量查询
|
||||
- ✅ "行尾符无污染" — `.gitattributes` 已添加统一 LF
|
||||
- ✅ "Service 层无架构问题" — **已修复** — `UserService` 依赖抽象接口而非具体 Repository 类型,支持 Mock
|
||||
- ⚠️ "Handler 响应格式统一" — **未修复** — 部分 handler 返回 `code/message/data`,部分裸返回
|
||||
- ⚠️ "Handler 响应格式统一" — **部分修复** — 已统一 13 个 handler,剩余 device/avatar/auth_handler.go 待修复
|
||||
|
||||
## 经验总结(来自 PROJECT_EXPERIENCE_SUMMARY.md)
|
||||
|
||||
@@ -186,7 +186,7 @@ go run golang.org/x/vuln/cmd/govulncheck@latest ./... # PASS
|
||||
6. ~~P1: AssignRoles 非事务~~ ✅ 已修复
|
||||
7. ~~P1: N+1 查询~~ ✅ 已修复
|
||||
8. ~~P1: Service 层 DIP 违规~~ ✅ 已修复 — 提取 userRepository/roleRepository 等本地接口,`NewUserService` 接受接口类型
|
||||
9. **P2: 统一 Handler 响应格式**(全部 `code/message/data` 结构)— 近期待办
|
||||
9. **P2: 统一 Handler 响应格式**(部分完成)— 已修复 13 个 handler:stats/captcha/sms/theme/password_reset/totp/sso/settings/log/webhook/permission/custom_field/role_handler.go;剩余 device_handler.go、avatar_handler.go、auth_handler.go(近期待办)
|
||||
|
||||
## 状态
|
||||
|
||||
|
||||
Reference in New Issue
Block a user