---

req_id: REQ-USR-002 date: 2026-05-08 round: 2

reviewer: superpower-code-reviewer

Review: REQ-USR-002 — round 2

结论

approve

Must-fix

（无）

Round 1 全部 must-fix 已正确修复：

• M1 ✓ updateUser_selfAdminKeepType_doesNotThrow 测试已添加（含 userMapper.update stub）
• M2 ✓ UserUpdateReqDTO 已添加 @NotBlank/@Pattern/@NotNull 校验注解
• M3 ✓ UserFormDrawer.InitialData 已添加 permGroupIds 字段，useEffect 用 ?? [] 初始化
• M4 ✓ UserListItemVO 已添加 bCanEditDoc，mapper SELECT 已加列，UserListPage 传真实值

Nice-to-have

• backend/.../vo/UserUpdateRespVO.java — updatedAt 建议添加 @JsonFormat(pattern="yyyy-MM-dd HHss") 符合 docs/04 § 3.3 规范
• frontend/src/test/UserListPage.test.tsx:106 — editMode_submit_callsUpdateUser 可补 toHaveBeenCalledWith('u1', ...) 断言 userId 参数正确
• frontend/src/pages/usr/UserListPage.tsx:121-128 — initialData 未传 permGroupIds（因列表 API 无此字段），与规格 line 103 说明一致，属有意为之，建议加注释说明

反例 / 测试覆盖缺口

• controller 层 BizException 40300/40400/40301 → 错误码 JSON 映射缺测试（低风险，handler 已在 createUser 路径覆盖）
• happy-path 未用 ArgumentCaptor 断言 bCanEditDoc/bIsDisabled 映射为正确 int 值（低风险）