---
fe_id: FE-02
date: 2026-05-15
round: 2
reviewer: fe-code-reviewer
---

# Review: FE-02 — round 2

## 结论
approve

## Round 1 修复落地核对

| # | 项目 | 状态 |
|---|------|-----|
| H1 | UserFormPage edit prefill canEditDocument from backend（之前硬编码 false） | ✓ |
| M1 | UsersTable a11y 键盘可达（tabIndex/role/aria-label/onKeyDown）+ 操作列编辑链接 + 列头 sorter | ✓ |
| M2 | UserFormPage employeeId 三态映射（toCreateEmployeeId→null / toUpdateEmployeeId→保留） | ✓ |
| M3 | api/users.ts CreateUserReq/UpdateUserReq.employeeId 类型补 `| null` | ✓ |
| L1 | UserPermissionPanel 补 process/driver 两个 disabled Tab（与 prototype 对齐） | ✓ |
| L2 | UsersListPage 默认 sortField='tCreateDate'/sortOrder='desc' 显式发后端 | ✓ |
| L3 | usersConstants QUERY_FIELD_OPTIONS 补 lastLoginDate | ✓ |
| TEST | canEditDocument prefill 测试 + msw handler 返回该字段 | ✓ |

## Nice-to-have（保留延后）

- frontend/src/pages/users/UserFormPage.tsx:129 — 40004 仍 banner；后端协议补 detail.field 后切 field-level
- frontend/src/pages/users/UsersTable.tsx:73 — 列头 sorter UI 已加但 onChange 未连到 UsersListPage.setQuery；视觉 sorter 生效但点击列头不会实际改变排序
- frontend/src/pages/users/UsersListPage.test.tsx — 缺 error banner 渲染态测试
- frontend/src/pages/users/UserFormPage.test.tsx — 缺 40004 错误码测试
- frontend/src/pages/users/UsersToolbar.tsx — prototype 还有作废 / 删除 / 重置密码 等按钮；FE-02 spec § 一已声明推后

## 7 维 checklist

| # | 维度 | 状态 |
|---|------|-----|
| 1 | Prototype consistency | pass（UsersTable 操作列 + UserPermissionPanel 5 个 Tab） |
| 2 | Design tokens | pass（仅 App.tsx 受控 hex 镜像 ConfigProvider colorPrimary） |
| 3 | A11y | pass（tabIndex/role/aria-label/onKeyDown + Form.Item label） |
| 4 | Responsive | pass |
| 5 | 业务校验前端复刻 | pass（spec § 五 #3-7,11,12,16-17 全部实现；#13 banner 可接受） |
| 6 | API consistency | pass（统一 apiClient） |
| 7 | 状态机覆盖 | pass（5 基础态 + spec 列出的错误态） |

## 反例 / 测试覆盖缺口

44 vitest 通过；3 个 Playwright E2E `.fixme()` 留作手工验收。缺 list error 渲染态 + form 40004 路径的单测覆盖（已记 nice-to-have）。sortField/sortOrder UI ↔ query 通道未拉通——sorter 列头视觉已有但回调未绑，不影响默认排序但影响主动排序；可下个 FE 接前接 Table onChange。

## 总结

Round 1 全部 8 项 must-fix 已正确落地：编辑模式 prefill canEditDocument 走 detail VO 透传修复了静默数据回写 bug；UsersTable a11y / 操作列 / sorter 全套；employeeId 三态语义按 spec § 八 锁定；prototype Tab 对齐；默认排序显式发后端；QUERY_FIELD_OPTIONS 完整。所有 nice-to-have 均为用户标注的延后项，不阻断 approve。