From 874258597417a5df9893b1e6deac8b0fc1be9203 Mon Sep 17 00:00:00 2001 From: zichun Date: Fri, 8 May 2026 11:49:43 +0800 Subject: [PATCH] fix(usr): 修复 review round 2 must-fix REQ-USR-003 --- docs/05-API接口契约.md | 2 +- docs/superpowers/reviews/2026-05-08-REQ-USR-003.md | 21 +++++++++++++++++++++ 2 files changed, 22 insertions(+), 1 deletion(-) create mode 100644 docs/superpowers/reviews/2026-05-08-REQ-USR-003.md diff --git a/docs/05-API接口契约.md b/docs/05-API接口契约.md index fd400c1..551f1e0 100644 --- a/docs/05-API接口契约.md +++ b/docs/05-API接口契约.md @@ -24,7 +24,7 @@ BasePath: `/api` 除登录接口外，所有接口均需在请求头携带 `Authorization: Bearer `。Token 由 REQ-USR-004 登录接口签发，有效期 24 小时。 ### 分页参数 -列表查询接口统一使用以下分页入参：`pageNum`（页码，从 1 起）、`pageSize`（每页条数，默认 20，最大 100）。响应体 `data` 字段格式：`{ list: [], total: N, pageNum: N, pageSize: N }`。 +列表查询接口统一使用以下分页入参：`page`（页码，从 1 起）、`pageSize`（每页条数，默认 20，最大 100）。响应体 `data` 字段格式：`{ list: [], total: N, page: N, pageSize: N }`。 ## 接口清单 （各模块接口段落见下方，由 `downstream-gen` 按 REQ 填入） diff --git a/docs/superpowers/reviews/2026-05-08-REQ-USR-003.md b/docs/superpowers/reviews/2026-05-08-REQ-USR-003.md new file mode 100644 index 0000000..3731a1d --- /dev/null +++ b/docs/superpowers/reviews/2026-05-08-REQ-USR-003.md @@ -0,0 +1,21 @@ +--- +req_id: REQ-USR-003 +date: 2026-05-08 +round: 2 +reviewer: superpower-code-reviewer +--- + +# Review: REQ-USR-003 — round 2 + +## 结论 +request-changes + +## Must-fix +- [major] docs/05-API接口契约.md:27 — 全局分页约定仍使用 `pageNum`，而 REQ-USR-003 具体接口入参与响应已修为 `page`，文档内部矛盾（建议：将第 27 行全局约定中的 `pageNum` 统一改为 `page`，与实际实现保持一致） + +## Nice-to-have +- docs/05-API接口契约.md:77 — staffName/department 的 equals matchType 对 NULL 员工的行为未记录在文档，建议补充说明（equals NULL 用户会被 JOIN 过滤掉） + +## 反例 / 测试覆盖缺口 +- T1: 缺少针对 notContains NULL 修复的集成测试（验证 staffName/department 为 NULL 的用户确实出现在 notContains 结果中） +- T2: 前端 queryField/matchType 未做枚举校验，非法值直接透传给后端（后端也无校验，可在后续迭代补充） -- libgit2 0.22.2