关于 review-design
供 Agent 使用的内部设计审查技能,只检查变更,不编辑代码。它依据项目设计规则评估问题,并以固定格式返回附有依据的发现。
主要功能
- 01检查设计系统与组件复用
- 02审查加载、空、错误和成功状态
- 03评估响应式行为与本地化
- 04只报告问题,不修改代码
适用场景
- 01按设计规则审查界面变更
- 02合并前检查新视图
安装
复制下面的命令,在终端中运行。
npx skills add https:/ / github.com/ gtrabanco/ agentic-workflow --skill review-design安装后在 Claude Code 中输入/review-design确认已加载。
详细介绍
设计审查(内部)
由 review-change / product-audit 在其对话中编写——对于任何
Agent,都应按照路由步骤,在此文件中直接执行。仅报告问题;绝不编辑、
绝不重构。
范围
调用方传入的差异或路径/通配符;默认审查当前变更与默认分支之间的差异。 在返回表格顶部注明范围。
检查清单(评估每一项——无一例外;如不适用,必须注明)
✓ 先阅读项目的设计文档(例如 docs/frontend/DESIGN.md)——每个问题都要引用对应规则 ✓ 变更的组件复用设计系统(不得重复现有令牌或组件,创建一次性样式/颜色/间距) ✓ 每个新增/变更的视图均处理所有状态:加载、空、错误、成功(逐一引用) ✓ 说明并遵守项目断点处的响应式行为(平台没有断点时注明不适用) ✓ 项目声明采用 i18n 时,不得硬编码面向用户的字符串(引用 i18n 文档) ✓ 视觉层级:主操作清晰可辨,每个视图仅有一个主操作 ✓ 与同类屏幕保持一致(相同交互采用相同模式——引用不一致的同类屏幕) ✓ 危险操作必须要求确认,并在视觉上加以区分 ✓ 对每项用户操作(提交、保存、失败)都提供反馈——不得静默处理
重要性门槛
仅当问题会改变具备能力的用户的结果,或违反项目明确声明的规则时,才报告该行——须在证据旁引用所违反的规则。
不属于问题的情况:注释/标点错误、仅格式偏差、没有引用规则的风格偏好、超出 SPEC 所列场景的假设性健壮性要求。
对于规范完善的变更,预期结果是空表格并标注 Decision: PASS——绝不凑数。
严格按以下格式返回
REVIEW DESIGN — scope: <scope>
| # | Finding | Sev | Evidence | Suggested fix |
|---|---------|-----|----------|---------------|
| 1 | <what> | critical|major|minor | <file:line> | <smallest action> |
Checklist: <n> evaluated, <n> pass, <n> findings, <n> n/a (<which + why>)
Summary: <1-2 sentences>
Decision: PASS | FAIL
如果有任何未解决的严重或主要问题,则判定为 FAIL;否则判定为 PASS。次要问题绝不阻止通过——它们会转交给调用方的分诊步骤。
完成条件
- 每项检查清单内容均有证据(文件:行号或命令输出),或明确标记为不适用并说明原因。
- 返回上述固定格式的内容——不多不少——且未更改任何代码。