mcpField
R
review-design

review-design

github.com/gtrabanco设计与 UI2285 次安装今天更新
代码仓库

关于 review-design

供 Agent 使用的内部设计审查技能,只检查变更,不编辑代码。它依据项目设计规则评估问题,并以固定格式返回附有依据的发现。

主要功能

  1. 01检查设计系统与组件复用
  2. 02审查加载、空、错误和成功状态
  3. 03评估响应式行为与本地化
  4. 04只报告问题,不修改代码

适用场景

  1. 01按设计规则审查界面变更
  2. 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。次要问题绝不阻止通过——它们会转交给调用方的分诊步骤。

完成条件

  • 每项检查清单内容均有证据(文件:行号或命令输出),或明确标记为不适用并说明原因。
  • 返回上述固定格式的内容——不多不少——且未更改任何代码。

评论

支持 Markdown