代码审查指南

在审查本仓库中的后端代码变更时，请使用本指南。

审查目标

• 发现正确性问题、回归缺陷、缺失的校验逻辑、不安全的数据访问、松散的模块依赖关系，以及缺失的测试。
• 审查应依据 AGENTS.md 中对应的实现技能（skill），而非仅基于个人风格偏好。
• 优先报告具体、可定位的问题（需注明文件路径及影响），而非泛泛而谈的意见。

强制审查规则

• 首先根据 AGENTS.md 确认本次变更应遵循哪项技能（skill）。
• 拒绝所有未遵循对应技能结构或检查清单（checklist）的后端变更。
• 拒绝任何缺失以下注释的生成或修改过的函数/方法：

// Generated by skills.

• 将缺失校验、缺失鉴权检查、DI 依赖注入（DI wiring）异常、仓库（repository）行为不安全、以及 schema/数据一致性问题，视为高置信度（high-signal）问题。

审查工作流

1. 识别变更涉及的模块区域：controller、service、module、repo、DTO、schema、queue、MCP、deployment 或 tests。
2. 打开 skills/skill-enforcement.skill.md。
3. 打开 AGENTS.md 中与该任务匹配的特定技能文档。
4. 将代码变更与要求的结构、命名规范、目录位置及实现检查清单进行比对。
5. 验证每个生成或修改的函数/方法签名正上方是否包含 // Generated by skills. 注释。
6. 检查行为变更处是否新增或更新了相应测试。
7. 按严重程度降序列出问题，每项需附带文件路径及简要说明其影响。

需要检查的内容

架构

• Controller 应保持轻量，仅负责请求分发，不包含业务逻辑。
• Service 应仅封装编排逻辑与核心业务逻辑。
• Repository 应仅处理数据访问相关职责。
• Module 应正确配置 providers、controllers，并准确声明 imports/exports。

正确性

• DTO 校验规则需与实际请求需求严格一致。
• Service 逻辑需覆盖预期的成功路径与各类失败场景。
• Repository 方法应查询正确的字段，并安全处理“未找到”（not-found）等边界情况。
• Schema 变更需在必要时保留必填字段、默认值、索引及时间戳字段。

安全性与可靠性

• Controller 响应中不得暴露敏感字段。
• 不得绕过身份认证（authentication）与权限控制（authorization）检查。
• 输入数据应在抵达 service 或 repository 层之前完成校验。
• 日志记录与错误信息不得泄露密钥、凭证或内部实现细节。

测试

• 新增行为应在适当时机补充单元测试或端到端（e2e）测试。
• 现有测试用例仍需准确反映当前预期行为。
• 关键逻辑需覆盖边缘情况与各类失败路径。

审查输出格式

• 问题列表置于最前，按严重程度从高到低排序。
• 每个问题须包含以下要素：
• 文件路径
• 简明的问题描述
• 该问题为何重要（即潜在影响）
• 若无任何问题，须明确声明，并指出尚存的测试缺口或待验证事项。

拒绝条件

• 后端工作未使用必需的 skill。
• 任意生成或修改的函数/方法签名上方缺失 // Generated by skills. 标记。
• 代码被放置在错误的架构层级（如在 controller 中写业务逻辑）。
• 缺失输入校验，或存在不安全的持久化逻辑。
• 行为发生变更但未提供相应的测试覆盖。