Keyboard shortcuts

Press ← or → to navigate between chapters

Press S or / to search in the book

Press ? to show this help

Press Esc to hide this help

输出模板说明

请使用此模板作为拉取请求审核总结评论的结构,复制后填写相应内容即可。

用于拉取请求总结评论

## Code Review Summary

**Verdict: [Approved ✅ | Changes Requested 🔴 | Reviewed 💬]** ([N] issues, [N] suggestions)

**PR:** #[number] — [title]
**Author:** @[username]
**Files changed:** [N] (+[additions] -[deletions])

### 🔴 Critical
<!-- Issues that MUST be fixed before merge -->
- **file.py:line** — [description]. Suggestion: [fix].

### ⚠️ Warnings
<!-- Issues that SHOULD be fixed, but not strictly blocking -->
- **file.py:line** — [description].

### 💡 Suggestions
<!-- Non-blocking improvements, style preferences, future considerations -->
- **file.py:line** — [description].

### ✅ Looks Good
<!-- Call out things done well — positive reinforcement -->
- [aspect that was done well]

---
*Reviewed by Hermes Agent*

严重性等级指南

等级图标适用场景是否阻止合并?
极严重🔴安全漏洞、数据丢失风险、程序崩溃、核心功能故障是
警告⚠️非关键路径中的缺陷、缺少错误处理机制、新代码缺乏测试通常会阻止
建议💡代码风格优化建议、重构思路、性能提升提示、文档缺失问题不会阻止
表现良好✅代码结构清晰、测试覆盖率达标、命名规范、设计合理无

审核结果判定

  • 已通过 ✅ — 不存在任何极严重或警告级别的问题,仅有建议项或所有问题均已解决。
  • 需要修改 🔴 — 存在任何极严重或警告级别的问题。
  • 正在审查 💬 — 仅包含观察意见(如草稿 Pull Request、不确定的发现或仅供参考的信息)。

内联注释说明

请在内联注释前加上对应严重性等级的图标,以便于快速识别:

🔴 **Critical:** User input passed directly to SQL query — use parameterized queries to prevent injection.
⚠️ **Warning:** This error is silently swallowed. At minimum, log it.
💡 **Suggestion:** This could be simplified with a dict comprehension:
`{k: v for k, v in items if v is not None}`
✅ **Nice:** Good use of context manager here — ensures cleanup on exceptions.

用于本地(推送前)审查

在推送之前进行本地审查时,可采用相同的结构,但需以消息形式呈现给用户,而非作为 PR 评论。无需包含 PR 元数据标题,直接从严重程度分类部分开始即可。