---
req_id: REQ-MOD-002
date: 2026-04-29
round: 1
reviewer: superpower-code-reviewer
---

# Review: REQ-MOD-002 — round 1

## 结论
approve

## Must-fix
（无）

## Nice-to-have

- backend/src/main/java/com/xly/erp/module/mod/service/impl/ModuleServiceImpl.java:111 — 环检测 `depth >= MAX_PARENT_DEPTH` 守卫位于循环体中段，最大执行 51 次 mapper 调用而非 50（`cur.equals(id)` 判断先于守卫）。建议把守卫挪到 for 条件 `depth < MAX_PARENT_DEPTH` 或先判 depth 再判 equals。无功能影响，仅语义更精确。
- backend/src/test/java/com/xly/erp/module/mod/controller/ModuleControllerIT.java:263 — `putWithoutJwt_permitAllStub_returns200_andDoesNotChangeCreatedBy` 与 MOD-001 同名 POST 用例同样硬绑 stub 行为；建议加 javadoc 锚点 `// REQ-MOD-001 stub: see USR-004 follow-up`，便于 USR-004 上线时一次性 grep 替换为 401 期望。
- backend/src/test/java/com/xly/erp/module/mod/controller/ModuleControllerIT.java:188 — happy-path IT 仅断言 `sProcedureName` / `sCreatedBy` 保留，未端到端断言 `tCreateDate` / `sBrandsId` / `sSubsidiaryId`。Service 单测已 captureArgument 验证这些字段在 entity 上为 null（依赖 NOT_NULL 跳过），如要 IT 双重保险可加列断言。
- backend/src/test/java/com/xly/erp/module/mod/service/ModuleServiceImplTest.java:215 — update 用例直接 stub interface default 方法 `existsActiveById` / `selectParentIdById`，与 MOD-001 create 用例 stub 抽象方法 `findActiveFlagById` 风格不一致。功能上 Mockito 支持；建议统一风格——推荐 MOD-001 改为 stub 默认方法（更直观）。
- backend/src/main/java/com/xly/erp/module/mod/service/impl/ModuleServiceImpl.java:92 — `moduleMapper.updateById(entity)` 返回值（受影响行数）被丢弃。`@Transactional` + 先做 selectById 校验后并发 DELETE 概率极低，但若命中可能 0 行受影响而方法仍返回成功。可断言 affected==1 否则抛 BizException(40400)。
- backend/src/main/java/com/xly/erp/module/mod/dto/UpdateModuleDTO.java:32 — `iParentId` / `iSortOrder` 缺 `@PositiveOrZero` / `@Min` 约束。spec 未要求；负数等异常值需要在 service 层走父链失败才抛 40021。

## 反例 / 测试覆盖缺口

Spec 验收清单（单元 7 + 集成 7 + 工程 3）端到端实现并 41/41 全绿；分层 / 命名 / 统一响应 / 异常处理 / 事务边界 / 安全 stub 均符合 docs/04 规范。

非阻塞缺口：
1. `tCreateDate` / `sBrandsId` / `sSubsidiaryId` 在 IT 端未断言保持原值（service 单测已断言 entity 字段 null + NOT_NULL 策略保证不覆盖）。
2. 父模块指向 `bDeleted=1` 旧记录是否回 40021 未单独验证（与 MOD-001 review gap #2 同源）。
3. `updateById` 返回 0 行（并发删除）的场景未覆盖。
4. DTO `@Size` 长度溢出（如 sModuleNameZh > 100）路径未单独 IT。
5. 环检测 `MAX_PARENT_DEPTH` 超深路径仅有间接覆盖（脏数据 50 层链未构造测试）。
