| name | html-effectiveness-review |
| description | 子 skill — 代码评审与理解。当用户要 PR 评审、PR 写作（作者侧）、或要理解陌生代码模块的请求路径/调用栈时使用。包含 PR 评审（diff.html）、PR 写作（pr.html）、模块图三种范式。当父 skill 路由命中"评审/PR/审阅/这段代码怎么工作"类请求时加载。 |
02-review — 代码评审与理解
"Diffs and call-graphs are spatial information; markdown flattens them. Let the agent render the change as an annotated diff, draw the module as boxes and arrows, or write the PR description your reviewers actually want."
何时用
| 用户说 | 用哪个范式 | 模板 |
|---|
| "review this PR / 评审一下 #312" | PR 评审 (reviewer 视角) | ../../templates/diff.html |
| "帮我写 PR 描述 / write a PR writeup" | PR 写作 (author 视角) | ../../templates/pr.html |
| "X 在这个 repo 怎么工作 / 调用栈是什么" | 模块图 | 自建（见下） |
| "解释一下 auth flow / sync 是怎么转的" | 模块图 | 同上 |
范式 2.1 · PR 评审（reviewer 视角）
空间形状：
- 顶部：一句话总结 + 文件风险地图（每文件一个圆点，颜色=风险，大小=改动量）
- 文件列表：每文件折叠区块；标题含 risk badge + diff 行数；展开看带行号 diff
- 行号 diff 必须等宽对齐
风险标签 3 档（不要再细分，用户记不住）：
safe — 纯加法 / 类型扩展 / 测试 → 绿色worth a look — 行为微变 / 重构 → 黄色needs attention — 关键路径 / 新抽象 / 易错点 → 红色
HTML 骨架：
<header class="pr-summary">
<h1>PR #247 — Review Summary</h1>
<p>[一句话: what this PR does]</p>
</header>
<section class="risk-map">
<h2>Risk map</h2>
<svg viewBox="0 0 600 120">
<g><circle cx="60" cy="60" r="14" class="needs-attn" />
<text x="60" y="95" text-anchor="middle">useOptimisticTasks.ts</text></g>
<g><circle cx="160" cy="60" r="20" class="worth" />
<text x="160" y="95" text-anchor="middle">TaskList.tsx</text></g>
</svg>
</section>
<section class="files">
<details open>
<summary>
<code>src/hooks/useOptimisticTasks.ts</code>
<span class="badge danger">needs attention</span>
<span class="muted">+58 −0</span>
</summary>
<pre class="diff"><div class="line added"><span class="ln">12</span><span>+ ...</span></div></pre>
</details>
</section>
关键 CSS（diff 行号）：
.diff { font-family: var(--font-mono); font-size: 12.5px; line-height: 1.55; }
.diff .line { display: grid; grid-template-columns: 56px 1fr; gap: 0; }
.diff .ln { color: var(--gray-500); padding: 0 12px; text-align: right;
background: var(--gray-100); user-select: none; }
.diff .added { background: rgba(120, 140, 93, 0.12); }
.diff .removed { background: rgba(176, 74, 74, 0.10); }
shared/base.css 已含上述类，直接使用。
直接可用模板：../../templates/diff.html
范式 2.2 · PR 写作（author 视角）
空间形状：6 段，按 reviewer 阅读顺序排：
- TL;DR 框 — 一段话讲完动机和结果
- Why — 为什么之前的做法不行
- Before / After — 双列对比，4 条 vs 4 条
- File-by-file — 按建议阅读顺序，每文件折叠
- Where to focus — 编号 1/2/3，每条说"看这里因为什么"，含 "what I deliberately did NOT do"
- Test plan — unit / integration / staging / manual 四组
- Rollout — Day 0 / Day 2 / Day 4 阶梯
关键 CSS（rollout 阶梯）：
ol.ramp { list-style: none; padding: 0; display: grid;
grid-template-columns: repeat(3, 1fr); gap: 12px; }
ol.ramp li { padding: 16px 18px; background: var(--white);
border: 1px solid var(--gray-300); border-radius: 10px;
display: flex; flex-direction: column; gap: 4px; }
ol.ramp .pct { font-size: 22px; font-weight: 500; color: var(--clay); }
直接可用模板：../../templates/pr.html
范式 2.3 · 模块图 / Code Understanding
空间形状：
- 顶部：请求路径 SVG（盒子 + 箭头），高亮 hot path
- 下方：编号 1/2/3/4/5 的 callstack
- 每步：文件名 + 行号 + 一段中英说明 +
<details> 折叠源码
关键 CSS：
.callstack li { display: grid; grid-template-columns: 32px 1fr; gap: 16px;
margin-bottom: 24px; list-style: none; }
.callstack .num { width: 32px; height: 32px; border-radius: 50%;
background: var(--clay); color: white; display: grid;
place-items: center; font-weight: 600;
font-family: var(--font-mono); }
.callstack .file { font-family: var(--font-mono); font-size: 13px;
color: var(--gray-700); }
.callstack details summary { cursor: pointer; padding: 6px 0;
color: var(--info); font-size: 13px; }
无独立模板（按需生成，结构清晰，可由 plan.html 的 step 部分启发）。
共同原则
- 风险标签必须是 agent 真给出的判断，不要"All files look fine"
- 行号 diff 必须等宽——浏览器渲染才有"diff"的感觉
- Where to focus 永远要有 "what I did NOT do"——避免 reviewer 浪费时间提"为什么没加 X"
- PR 写作的 file-by-file 要按阅读顺序，不是字母序——告诉 reviewer 从哪开始
