Menu
Review P4 pending changelist 的代码质量。Use when the user says 'review p4 CL' / 'review p4 changelist' / '审查 p4 CL' / 'review下 p4 CL'，或提供了一个 CL 编号要求审查时触发。
Install with Codex or Claude Copy this prompt, paste it into Codex, Claude, or another assistant, and let it review the skill page and install it for you.
Based on SOC occupation classification
探索代码与文档以回答用户的疑问，不产出文档。TRIGGER WHEN: 用户提问、想了解现有实现、排查问题、搞清某功能如何工作时使用。
需求工作流入口。根据用户描述自动路由到对应的子流程。TRIGGER WHEN: 用户描述需求时使用。
理解拆解需求并生成设计文档。TRIGGER WHEN: 分析需求、拆解功能、产出边界或书写设计文档时使用。
根据设计文档实现代码。TRIGGER WHEN 当用户已完成需求设计文档、开始编码实现时使用，或需要修改代码时使用。
初始化需求工作环境。TRIGGER: 当用户说'初始化'、'story init'、'开始需求'、'环境准备'、'准备工作区'时使用。
Review 需求设计或代码实现。TRIGGER: 当用户说'开始 Review'、'审查代码'、'检查实现'时使用。
| 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，则提示用户并退出。
p4 info 获取 P4 用户名。p4 clients -u <用户名> 获取所有客户端。并根据当前目录判断具体的workspacep4 changes -s pending -c <workspace> 获取所有 pending CL。p4 -c <workspace> opened -c default 检查 default CL 是否有文件。AskUserQuestion询问用户选择要审查的 CL。并行启用2-3个Agent工具
逐文件使用：
p4 -c <客户端名> diff -du "<depot路径>"
对收集到的 diff，按以下维度逐一审查：
如果是C++代码，加载 .agent/library/common/ue5-cpp-rule.md 获取规范
汇总所有的agent结论，生成 review 报告。
使用以下固定格式输出：
## CL <编号|default> Review — <一句话描述变更主题>
### 变更概述
<2-3 句话概述这次改动做了什么，为什么这样做>
| 文件 | 变更类型 |
|---|---|
| `文件名` | 新增/删减/修改：简要说明 |
| ... | ... |
---
### 必须修复的问题
#### 1. <问题标题>
```cpp
// 有问题的代码片段
问题：具体说明为什么有问题 建议：给出修复方案或代码示例
具体说明 + 建议方案
| 级别 | 数量 | 说明 |
|---|---|---|
| 必须修 | N | 简要列举 |
| 建议改 | N | 简要列举 |
<一句话总结评价>
## 注意事项
- **用中文**输出 review 报告
- 必须修复 vs 建议改进 的区分标准：
- **必须修复**：逻辑错误、会导致 bug 或 crash、行为意外变更
- **建议改进**：代码风格、可读性、潜在优化、非阻塞性问题
- 不要虚构问题，每个指出的问题都必须有 diff 中的代码依据
- 对于大型 CL（>15 个文件），先给出整体架构评价，再逐模块 review
- 如果 CL 中包含纯机械性改动（如批量 rename、格式化），简要提及即可，不必逐行 review