| name | p4-review-cl |
| description | Review P4 pending changelist 的代码质量。Use when the user says 'review p4 CL' / 'review p4 changelist' / '审查 p4 CL' / 'review下 p4 CL'，或提供了一个 CL 编号要求审查时触发。 |
必须 仅查看符合当前的workspace的 CL。若当前workspace没有CL，则提示用户并退出。
step 1
- 通过
p4 info 获取 P4 用户名。
- 通过
p4 clients -u <用户名> 获取所有客户端。并根据当前目录判断具体的workspace
- 通过
p4 changes -s pending -c <workspace> 获取所有 pending CL。
- 通过
p4 -c <workspace> opened -c default 检查 default CL 是否有文件。
step 2
- 将 所有 pending CL 和 default CL（如果有文件）一起列出供用户选择。
- 列出每一个 CL 的文件列表。
使用
AskUserQuestion询问用户选择要审查的 CL。
step 3
并行启用2-3个Agent工具
逐文件使用：
p4 -c <客户端名> diff -du "<depot路径>"
对收集到的 diff，按以下维度逐一审查：
如果是C++代码，加载 .agent/library/common/ue5-cpp-rule.md 获取规范
step 4
汇总所有的agent结论，生成 review 报告。
使用以下固定格式输出：
## CL <编号|default> Review — <一句话描述变更主题>
### 变更概述
<2-3 句话概述这次改动做了什么，为什么这样做>
| 文件 | 变更类型 |
|---|---|
| `文件名` | 新增/删减/修改：简要说明 |
| ... | ... |
---
### 必须修复的问题
#### 1. <问题标题>
```cpp
// 有问题的代码片段
问题：具体说明为什么有问题
建议：给出修复方案或代码示例
2. ...
建议改进
N. <建议标题>
具体说明 + 建议方案
总结
<一句话总结评价>
## 注意事项
- **用中文**输出 review 报告
- 必须修复 vs 建议改进 的区分标准：
- **必须修复**：逻辑错误、会导致 bug 或 crash、行为意外变更
- **建议改进**：代码风格、可读性、潜在优化、非阻塞性问题
- 不要虚构问题，每个指出的问题都必须有 diff 中的代码依据
- 对于大型 CL（>15 个文件），先给出整体架构评价，再逐模块 review
- 如果 CL 中包含纯机械性改动（如批量 rename、格式化），简要提及即可，不必逐行 review
