菜单
Review P4 pending changelist 的代码质量。Use when the user says 'review p4 CL' / 'review p4 changelist' / '审查 p4 CL' / 'review下 p4 CL'，或提供了一个 CL 编号要求审查时触发。
用 Codex 或 Claude 帮你安装 复制这段 Prompt，粘贴到 Codex、Claude 或其他助手里，让它检查 Skill 页面并帮你完成安装。
基于 SOC 职业分类
| 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
探索代码与文档以回答用户的疑问，不产出文档。TRIGGER WHEN: 用户提问、想了解现有实现、排查问题、搞清某功能如何工作时使用。
需求工作流入口。根据用户描述自动路由到对应的子流程。TRIGGER WHEN: 用户描述需求时使用。
理解拆解需求并生成设计文档。TRIGGER WHEN: 分析需求、拆解功能、产出边界或书写设计文档时使用。
根据设计文档实现代码。TRIGGER WHEN 当用户已完成需求设计文档、开始编码实现时使用，或需要修改代码时使用。
初始化需求工作环境。TRIGGER: 当用户说'初始化'、'story init'、'开始需求'、'环境准备'、'准备工作区'时使用。
Review 需求设计或代码实现。TRIGGER: 当用户说'开始 Review'、'审查代码'、'检查实现'时使用。