hermes教程-教程：GitHub PR 审查代理

┌───────────────────────────────────────────────────────────────────┐
│                                                                   │
│   Cron 定时器 ──▶  Hermes 代理 ──▶  GitHub API ──▶ 审查交付      │
│   (每 2 小时)      + gh CLI           (PR diff)      (Telegram,   │
│                    + 技能                             Discord,     │
│                    + 记忆                             本地)       │
│                                                                   │
└───────────────────────────────────────────────────────────────────┘

- **已安装并认证 GitHub CLI (`gh`)**：
```bash
## 安装
brew install gh        # macOS
sudo apt install gh    # Ubuntu/Debian
## 认证
gh auth login

hermes

审查这个拉取请求。阅读 diff，检查 bug、安全问题和代码质量。
要具体指出行号并引用有问题的代码。

运行：gh pr diff 3888 --repo NousResearch/hermes-agent

mkdir -p ~/.hermes/skills/code-review

---
name: code-review
description: 审查拉取请求中的 bug、安全问题和代码质量
---
## 代码审查指南

审查拉取请求时：

## 检查内容
1. **Bug** —— 逻辑错误、差一错误、null/undefined 处理
2. **安全** —— 注入、认证绕过、代码中的密钥、SSRF
3. **性能** —— N+1 查询、无界循环、内存泄漏
4. **风格** —— 命名约定、死代码、缺少错误处理
5. **测试** —— 变更是否经过测试？测试是否覆盖了边界情况？

## 输出格式
对于每个发现：
- **文件:行号** —— 确切位置
- **严重性** —— 严重 / 警告 / 建议
- **问题** —— 一句话描述
- **修复** —— 如何修复

## 规则
- 要具体。引用有问题的代码。
- 不要标记风格上的吹毛求疵，除非它们影响可读性。
- 如果 PR 看起来不错，就直说。不要编造问题。
- 以以下之一结束：APPROVE / REQUEST_CHANGES / COMMENT

记住：在我们的后端仓库中，我们使用 Python 和 FastAPI。
所有端点必须有类型注解和 Pydantic 模型。
我们不允许使用原始 SQL —— 只使用 SQLAlchemy ORM。
测试文件放在 tests/ 中，并且必须使用 pytest fixtures。

记住：在我们的前端仓库中，我们使用 TypeScript 和 React。
不允许使用 `any` 类型。所有组件必须有 props 接口。
我们使用 React Query 进行数据获取，永远不要使用 useEffect 进行 API 调用。

hermes cron create "0 */2 * * *" \
  "检查新的打开 PR 并进行审查。

要监控的仓库：
- myorg/backend-api
- myorg/frontend-app

步骤：
1. 运行：gh pr list --repo REPO --state open --limit 5 --json number,title,author,createdAt
2. 对于每个在过去 4 小时内创建或更新的 PR：
   - 运行：gh pr diff NUMBER --repo REPO
   - 使用 code-review 指南审查 diff
3. 格式化输出为：

## PR 审查 — 今天

### [repo] #[number]: [title]
**作者：** [name] | **裁决：** APPROVE/REQUEST_CHANGES/COMMENT
[发现]

如果没有找到新 PR，则输出：没有新的 PR 需要审查。" \
  --name "pr-review" \
  --deliver telegram \
  --skill code-review

hermes cron list

hermes cron run pr-review

审查后，发布你的审查：
- 对于问题：gh pr review NUMBER --repo REPO --comment --body "YOUR_REVIEW"
- 对于严重问题：gh pr review NUMBER --repo REPO --request-changes --body "YOUR_REVIEW"
- 对于干净的 PR：gh pr review NUMBER --repo REPO --approve --body "Looks good"

hermes cron create "0 9 * * 1" \
  "生成每周 PR 仪表板：
- myorg/backend-api
- myorg/frontend-app
- myorg/infra

对于每个仓库显示：
1. 打开的 PR 数量和最旧 PR 的年龄
2. 本周合并的 PR
3. 过时的 PR（超过 5 天）
4. 没有分配审查者的 PR

格式化为清晰的摘要。" \
  --name "weekly-dashboard" \
  --deliver telegram

hermes gateway status    # 网关是否在运行？
hermes cron list         # 任务是否已启用？