| name | frontend-review |
| description | Use when reviewing implemented frontend iteration work against docs/vX.Y.Z/ requirements, design, plans, and test report. |
| disable-model-invocation | true |
前端审查
目标
对照需求、方案、计划、测试审代码，分级记问题。无未解决 🔴 才可发布
输入
| 路径 | 必需 | 说明 |
|---|
docs/technical-architecture.md | 是 | 架构、约定 |
docs/vX.Y.Z/prd/summarized/*.md | 是 | 验收 |
docs/vX.Y.Z/design/*.md | 是 | 方案 |
docs/vX.Y.Z/plans/*.md | 是 | 边界 |
docs/vX.Y.Z/test-report.md | 是 | 结果、风险 |
docs/vX.Y.Z/progress.md | 是 | task、TDD 证据 |
| 代码变更 | 是 | diff / 文件 |
缺必需项或 test-report「阻塞」则停止，回 implement/test 补 TDD、实现或测试
输出
调用契约与要求
- 调用契约见
frontend-iteration/references/orchestrated-invocation.md
- Agent 沟通风格见
frontend-iteration/references/agent-communication-style.md
- 只出审查记录，若要修复则回上游
工作流
- 读 summarized、design、plans；定预期与边界
- 读
progress.md、test-report.md；task、TDD、结论、风险
- 审 diff；对照 plan、design；审前重读风格锚点
- 按审查维度；核对 API 封装、占位、
TODO(vX.Y.Z): 接口联调待定
- 出
review/*.md；结论：通过 / 有条件通过 / 不通过
- 门禁结果写
progress.md
- 完成检查
- 摘要：结论、🔴/🟡/🟢、建议；等确认
规则
- 证据导向：问题指向文件/行/行为；不空泛
- 对照上游：基准 summarized + design + plan；非 Agent 偏好
- 分级：🔴 必修；🟡 建议；🟢 可选
- 不改代码：修复回 implement/test
- 🔴 阻断：未解决 🔴 → 不通过；不进 release
- 最小改动：标 plan 外改、多余抽象、重复逻辑
- 测审交叉：test-report 过但审查有缺口 → 🔴/🟡
- 进度交叉：task 未完/blocked/缺 VERIFY → 不得「通过」
- 状态门禁：见
frontend-iteration/references/orchestrated-invocation.md；上游不可用或步骤 5 blocked → 不得通过
审查维度
| 维度 | 检查 |
|---|
| 需求符合 | 行为 = summarized 验收 |
| 方案符合 | = design；偏离须解释 |
| 范围符合 | ⊆ plan；无隐藏重构 |
| 架构一致 | 目录、命名、状态、请求；docs/technical-architecture.md + 风格锚点 |
| 代码风格 | = 邻文件、风格锚点；lint 可过（若配置） |
| 接口联调 | 封装、占位、TODO = design；页面无硬编码假数据 |
| 正确性 | 逻辑、边界、错误、权限 |
| 可维护性 | 可读、重复、过度抽象 |
| 测试充分 | = test-report；关键路径有覆盖 |
| 安全 | XSS、敏感信息、不安全 DOM（适用时） |
| 性能 | 多余渲染、重复请求、大列表（适用时） |
| 回归风险 | 既有功能影响已评估 |
严重级别
| 级别 | 含义 | 例 |
|---|
| 🔴 | 发布前必修 | 功能错、越权、plan 外大改、假通过 |
| 🟡 | 建议，可协商 | 命名差、缺错误处理、覆盖弱但可接受 |
| 🟢 | 可选 | 风格、微重构、文档 |
结论判定
| 条件 | 结论 |
|---|
| 无 🔴，且无 🟡、无 🟢 | 通过 |
| 无 🔴，有 🟡 和/或 🟢 | 有条件通过 |
| 有未解决 🔴 | 不通过 |
常见场景
| 场景 | 处理 |
|---|
| 偏离 design 但更优 | 🟡；回 design 补记或回滚 |
| plan 外文件改 | 🔴；除非 plan 已同步 |
| test-report 有风险 | 引用；评是否阻塞发布 |
| progress 与 test-report 不一致 | 🔴 或待确认问题；回步骤 4/5 补证据 |
| 上游 STALE | 停；回上游 |
| 有 🔴 | 不通过；指明回步骤 4 或 5 |
| 需求歧义 | 待确认问题；不扩范围 |
完成检查
交接下游 → 步骤 7
（无 🔴 或用户接受有条件通过）
review/*.md、test-report.md- 变更摘要、已知 🟡
文件路径解析
读 skill / reference 时按序尝试（命中即用）：
| 资源 | 路径 1（npx skills add） | 路径 2（源码） |
|---|
本子 SKILL.md 目录 | .agents/skills/frontend-review/ | skills/frontend-review/ |
| 编排 references | .agents/skills/frontend-iteration/references/<file> | skills/frontend-iteration/references/<file> |
| 其他 references | .agents/skills/<name>/references/<file> | skills/<name>/references/<file> |
| 样例 | .agents/skills/frontend-iteration/examples/ | skills/frontend-iteration/examples/ |
参考
- 产物模板：review-template.md
- 代码风格强制：
frontend-iteration/references/code-style-enforcement.md
- 接口联调指南：
frontend-iteration/references/api-integration-guide.md
