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

# Review: REQ-MOD-003 — round 1

## 结论
approve

## Must-fix
（无）

## Nice-to-have

- backend/src/test/java/com/xly/erp/module/mod/controller/ModuleControllerIT.java:291 — `deleteValidId_with_jwt_returns200_andSoftDeletes` 仅断言 `sProcedureName` / `sCreatedBy` 不变，spec § 验收 第 95 行明确要求同时断 `sBrandsId`。建议把 SELECT 列表追加 `sBrandsId, sSubsidiaryId, tCreateDate, iSortOrder` 并加断言（service 单测已 captureArgument 验证 entity 上字段为 null + 依赖 NOT_NULL 策略，IT 端到端双保险更稳）。
- backend/src/test/java/com/xly/erp/module/mod/controller/ModuleControllerIT.java:362 — `deleteWithoutJwt_permitAllStub_returns200_andDeletedByIsSTUB` 沿袭 MOD-001/002 stub 用例缺 `// REQ-MOD-001 stub: see USR-004 follow-up` 锚点注释；USR-004 上线时一次性 grep 替换为 401 会漏这条。
- backend/src/main/java/com/xly/erp/module/mod/service/impl/ModuleServiceImpl.java:112 — `moduleMapper.updateById(entity)` 返回值（受影响行数）丢弃；并发场景下 selectById 与 updateById 之间另一线程已软删则 affected=0 而方法仍返回成功，覆盖之前的 `tDeletedDate` / `sDeletedBy`。可断言 affected==1 否则抛 BizException(40400)。同 MOD-002 update review 同源问题，stub 期接受。
- backend/src/main/java/com/xly/erp/module/mod/service/impl/ModuleServiceImpl.java:97 — `delete()` 方法缺行内 `// REQ-MOD-003: 模块删除` 锚点（CLAUDE.md § 编码行为约束 #4 要求 REQ-XXX-NNN 行内可追溯）；MOD-001/002 的 create/update 也缺，建议三处统一补。
- backend/src/main/java/com/xly/erp/module/mod/service/impl/ModuleServiceImpl.java:97 — 删除路径未加悲观锁（无 SELECT … FOR UPDATE）；spec 声明非幂等单实例顺序满足，多副本并发可能重复进入。建议未来用乐观条件 UPDATE WHERE iIncrement=? AND bDeleted=0 收口，本期接受。

## 反例 / 测试覆盖缺口

Spec 验收清单（service 5 + mapperIT 1 + controllerIT 6 + 工程 3）端到端实现 53/53 全绿。校验顺序（目标存在 → 子模块 → 写库）与 spec 一致；软删除字段策略与 MOD-001/002 风格统一；mapper `findActiveChildFlag` 用 SELECT 1 LIMIT 1 高效；JWT 伪造路径 IT 通过 filter 短路确认 service 未触发（`bDeleted` 仍 0）；40902 外部引用拦截在 spec 双处显式声明本期不实现并指明 USR/后续模块入口；SecurityContextHelper 处理 anonymous 的方式与 MOD-001/002 一致；测试隔离用 sProcedureName LIKE 'sp_test_%' + Before/AfterEach 双清健壮。

非阻塞缺口：
1. `deleteValidId` IT 未断言 `sBrandsId` 等保留列（spec 明言）。
2. `// REQ-MOD-001 stub: see USR-004 follow-up` 锚点未在 MOD-003 stub 用例上贴。
3. `updateById` 返回值未校验，并发覆盖窗口存在。
4. service 方法缺行内 REQ 锚点（MOD-001/002/003 同源）。
5. 40902 外部引用待 USR/后续模块落地后回填。
