先完整 review，再分派原子修复，最后由主 agent 验收整个有效 diff。
SuperReview 是一个面向 Codex、AI coding agent 和多 agent 开发工作流的代码审查 skill。
强模型已经会主动调用 subagent，但 review 的判断权不应该一起外包。SuperReview 固定了一条简单的所有权边界：
- 主 agent 亲自读取完整改动、上下文和测试，形成有证据的 findings。
- 每个 confirmed finding 才能变成一个边界明确的原子修复任务。
- subagent 只负责修复，不负责定义 review 范围、严重级别或最终结论。
- 主 agent 检查每份补丁、独立复验，并重新 review 包含未提交修复的最终有效 diff。
flowchart LR
Change["PR / Branch / Local Diff"] --> Review["Main Agent<br/>完整 Review"]
Review --> Findings["Evidence-backed<br/>Findings"]
Findings --> RepairA["Repair SR-001"]
Findings --> RepairB["Repair SR-002"]
RepairA --> WorkerA["Subagent A"]
RepairB --> WorkerB["Subagent B"]
WorkerA --> Accept["Main Agent<br/>Accept / Reject"]
WorkerB --> Accept
Accept --> Rereview["最终有效 Diff<br/>重新 Review"]
Rereview --> Verdict["Verdict + Evidence"]
Rereview -. 新 finding .-> Findings
关键门槛只有一个：主 agent 完成首轮 review 并冻结 findings 之前，不允许把 review 分派出去。
| 角色 | 模型 | Intelligence / Reasoning | 职责 |
|---|---|---|---|
| Main reviewer | gpt-5.6-sol |
Ultra | 分析任务、完整首审、确定 findings、验收补丁、最终 verdict |
| Repair subagent | gpt-5.6-sol |
Extra High（xhigh） |
只修一个已确认的 finding |
Ultra 会让主 agent 更主动地调用 subagent，但不能越过 SuperReview 的 ownership gate：首审和 findings 冻结之前仍然禁止 spawn。
修复使用仓库自带的 superreview-repair custom agent。若运行时无法选择这个 agent profile，SuperReview 保持 findings 为 report-only 并明确报告 blocker，不会悄悄换成普通 worker。
| 常见问题 | SuperReview 的约束 |
|---|---|
| subagent 自己找问题、自己定范围 | findings 只能由主 agent 建立 |
| worker 说“修好了”就直接相信 | 主 agent 必须检查补丁并独立复验 |
| 多个 agent 同时改同一批文件 | 只有写集不重叠时才允许并行 |
只看 base..HEAD，漏掉未提交修复 |
review 最终有效 diff，包含 worktree repairs |
| 普通“帮我 review”意外改代码 | 隐式触发默认 report-only |
| review 顺手 commit、push 或发评论 | 外部动作必须由用户明确授权 |
- GitHub pull request
- branch 对 base / merge base
- commit range
- staged changes
- working-tree changes
- 上述边界的显式组合
这让 SuperReview 不只适合 PR，也可以验收 subagent 刚完成、还没有提交的本地改动。
显式调用 $superreview，或明确要求 review and fix。主 agent 先审，再把 confirmed findings 分派成原子修复。
普通的“review this PR / 检查这组改动”默认只输出 findings，不修改代码、不启动 repair worker。
用户提供已接受的 finding 列表时，主 agent 先验证 finding 仍然成立，再分派修复。
把仓库安装到 Codex skills 目录：
git clone https://github.com/fightheyyy/SuperReview.git ~/.codex/skills/superreview
mkdir -p ~/.codex/agents
cp ~/.codex/skills/superreview/agents/superreview-repair.toml ~/.codex/agents/superreview-repair.toml开始新 task 前，在 Codex composer 里选择 GPT-5.6 Sol + Ultra。Extra High repair agent 已由上面的 TOML 固定为 gpt-5.6-sol + xhigh。
然后在 Codex 中调用：
$superreview review and repair PR #123
只审不修：
$superreview review PR #123 in report-only mode
审查本地未提交改动：
$superreview review and repair the staged and working-tree changes
每个 finding 都要包含稳定 ID、优先级、精确位置、可观察失败、证据、最小修复边界和关闭它所需的验证。
每个 subagent 只能拿到一个有边界的 repair contract：
Repair: SR-001 <one-sentence objective>
Finding Evidence: <failure, cause, location>
Acceptance: <observable stop condition>
Allowed Scope: <exact files/modules>
Forbidden Work: <unrelated cleanup and broad refactors>
Required Verification: <focused checks>
worker 可以用证据证明 finding 是 false positive，但不能自行扩大范围，也不能宣称整个 PR 已通过。
SuperReview 可以独立使用，也可以和另外两个 skill 组合：
SuperGoal → 定义目标、拆分任务、控制停止条件
SuperDev → 对齐 Current / Target Architecture
SuperReview → 主审 findings、分派原子修复、最终验收
大型开发任务可以用 SuperGoal + SuperDev 完成实现，再用 SuperReview 验收最终 change set。
SuperReview 默认不会：
- 发布 GitHub review 或评论；
- approve、request changes 或 resolve thread；
- commit、push、merge 或创建新 PR；
- 把无关的本地脏改动吸收到 review 范围。
这些动作只有在用户明确授权后才执行。
SKILL.md：SuperReview 的完整工作流和 ownership invariant。agents/openai.yaml：Codex 展示与默认调用元数据。agents/superreview-repair.toml：固定为 GPT-5.6 Sol + Extra High 的原子修复 agent。assets/superreview-icon.svg：主标志，review lens + final verdict。docs/social-preview.png：GitHub 仓库社交预览图。README.en.md：English documentation。
如果你也认为 review judgment 应该留在主 agent，subagent 应该只做原子修复，欢迎点一个 Star，让更多 AI-assisted development 项目看到这套工作方式。