code-review-assistant

# Code Review Assistant

## Overview

提供结构化的代码审核工作流程，支持两种审核模式：

### 本地 Git 模式
- 审核当前最新 commit
- 审核暂存的修改 (`git add` 的文件)
- 审核未暂存的修改（工作区变更）
- 审核指定 commit 的 diff

### Gerrit Review 模式
- 通过 Gerrit Change ID 进行代码审核
- 自动获取 Change 详情和文件变更
- 支持自动发布评审到 Gerrit

**支持的语言**：
- Golang（完整支持）
- Qt/C++（完整支持）
- Ansible（完整支持）

## Review 评分规则

| 问题类型 | 建议 Score | 说明 |
|---------|-----------|------|
| 无问题 | **+1** | 代码质量优秀 |
| 仅有 Nice to Have | **-1** | 有改进空间但不影响功能 |
| 有 Should Fix | **-2** | 应该修复的问题 |
| 有 Must Fix | **-2** | 必须修复的严重问题 |

## 使用方式

### 本地 Git 审核用例

```
/code-review-assistant HEAD          # 审核最新 commit
/code-review-assistant --staged       # 审核暂存的修改
/code-review-assistant --unstaged     # 审核未暂存的修改
/code-review-assistant HEAD~1        # 审核上一个 commit
/code-review-assistant abc1234       # 审核指定 commit hash
```

### Gerrit Review 用例

```
/code-review-assistant <change_id>   # 审核 Gerrit Change
```

### 自动发布评论（仅 Gerrit 模式）

如果用户提示词中包含明确的发布请求（如"发布评论"、"提交评审"、"post review"、"发布到 Gerrit"等关键词），skill 会自动使用 Gerrit MCP 工具发布带评分的评论。

## 审核范围规范

### 支持的本地 Git 审核范围

| 用例 | Git 命令 | 说明 |
|------|----------|------|
| 最新 commit | `HEAD` | 审核最新的提交记录 |
| 上一个 commit | `HEAD~1` 或 `HEAD^` | 审核倒数第二次提交 |
| 指定 commit | `<commit_hash>` | 审核指定的某次提交 |
| 暂存的修改 | `--staged` 或 `--cached` | 审核已 `git add` 但未提交的变更 |
| 未暂存的修改 | `--unstaged` 或 `--working` | 审核工作区的变更 |
| 两次 commit 之间 | `<commit_a>...<commit_b>` | 审核两个 commit 之间的差异 |

### 不支持的审核范围

以下范围**不支持**，需要提醒用户提供更准确的范围：

- 多个 commit：如 `最近的5次提交`、`最近10次commit`、`所有提交`
- 指定范围的多次提交：如 `HEAD~5..HEAD` (除非用户明确表示可以接受)
- 整个仓库历史：如 `整个分支`、`所有变更`
- 大量的连续 commits

### 范围验证流程

当用户给出审核范围时：

1. **识别审核模式**：是本地 Git 还是 Gerrit Change ID
2. **验证范围有效性**：
   - 本地 Git：检查是否为 commit hash、`HEAD`、`--staged`、`--unstaged` 等
   - Gerrit：检查是否为有效的 Change ID 格式
3. **检查范围大小**：
   - 如果检测到多个 commit（如 `HEAD~10`），提醒用户建议审核单个 commit 或差异范围
   - 如果范围过大，提示用户缩小范围
4. **执行审核**：验证通过后开始审核流程

**范围不合理的提示模板**：
```
检测到您要求审核范围较大：{用户输入的范围}

建议使用以下更精确的范围：
- HEAD                    # 审核最新一次 commit
- HEAD~1