朱子纯 / test3

Commit 874258597417a5df9893b1e6deac8b0fc1be9203

Authored by zichun
1 parent 7921432e

fix(usr): 修复 review round 2 must-fix REQ-USR-003 

- docs/05: 全局分页约定 pageNum → page，与实现保持一致

REQ-USR-003
Inline Side-by-side
Showing 2 changed files with 22 additions and 1 deletions
docs/05-API接口契约.md
  View file @8742585
... ... @@ -24,7 +24,7 @@ BasePath: `/api`
24 24 除登录接口外，所有接口均需在请求头携带 `Authorization: Bearer <access_token>`。Token 由 REQ-USR-004 登录接口签发，有效期 24 小时。
25 25  
26 26 ### 分页参数
27   -列表查询接口统一使用以下分页入参：`pageNum`（页码，从 1 起）、`pageSize`（每页条数，默认 20，最大 100）。响应体 `data` 字段格式：`{ list: [], total: N, pageNum: N, pageSize: N }`。
  27 +列表查询接口统一使用以下分页入参：`page`（页码，从 1 起）、`pageSize`（每页条数，默认 20，最大 100）。响应体 `data` 字段格式：`{ list: [], total: N, page: N, pageSize: N }`。
28 28  
29 29 ## 接口清单
30 30 （各模块接口段落见下方，由 `downstream-gen` 按 REQ 填入）
... ...
docs/superpowers/reviews/2026-05-08-REQ-USR-003.md 0 → 100644
  View file @8742585
  1 +---
  2 +req_id: REQ-USR-003
  3 +date: 2026-05-08
  4 +round: 2
  5 +reviewer: superpower-code-reviewer
  6 +---
  7 +
  8 +# Review: REQ-USR-003 — round 2
  9 +
  10 +## 结论
  11 +request-changes
  12 +
  13 +## Must-fix
  14 +- [major] docs/05-API接口契约.md:27 — 全局分页约定仍使用 `pageNum`，而 REQ-USR-003 具体接口入参与响应已修为 `page`，文档内部矛盾（建议：将第 27 行全局约定中的 `pageNum` 统一改为 `page`，与实际实现保持一致）
  15 +
  16 +## Nice-to-have
  17 +- docs/05-API接口契约.md:77 — staffName/department 的 equals matchType 对 NULL 员工的行为未记录在文档，建议补充说明（equals NULL 用户会被 JOIN 过滤掉）
  18 +
  19 +## 反例 / 测试覆盖缺口
  20 +- T1: 缺少针对 notContains NULL 修复的集成测试（验证 staffName/department 为 NULL 的用户确实出现在 notContains 结果中）
  21 +- T2: 前端 queryField/matchType 未做枚举校验，非法值直接透传给后端（后端也无校验，可在后续迭代补充）
... ...