SuperAPI AI 工具集

review-quality

测试与质量

查看项目

更新于 10-15 21:38

技能介绍

用途:将“这个改动是否合格”变成可重复的审查流程,并给出明确的合并/上线就绪结论。它把三种审查视角合并:合并就绪度(需求对齐、风险、验证证据)、代码可维护性(Clean Code 风格)、文档与实现一致性(README/docs 与代码对比)。同时内置自动分诊路由:始终运行统一审查;若检测到 React/Next.js 性能风险,追加运行 review-react-best-practices 并纳入其结论;若检测到 UI 界面变更,追加运行 Web Interface Guidelines 审计(可访问性/焦点/表单/动效/内容溢出),以 file:line 简明形式输出。

输入:推荐提供 BASE_SHA 与 HEAD_SHA 做基于 diff 的审查;可选提供需求/计划文件路径、文档范围(README.md、docs/**、API 契约)、run_dir(Ship Faster 运行目录)。

建议的范围命令:BASE_SHA=$(git merge-base HEAD main 2>/dev/null || git merge-base HEAD master);HEAD_SHA=$(git rev-parse HEAD);随后用 git diff --stat 与 git diff 查看改动。

流程:先做自动分诊,用 git diff --name-only、--stat 和完整 diff 收集信号,按顺序应用路由规则——仅文档变更走快速路径,聚焦文档与代码一致性及发布风险;匹配 /*.tsx、/.jsx、next.config.、app/、pages/、src/app/、src/pages/ 或 React/Next 入口文件,或 diff 含 use client、Suspense、dynamic(、next/dynamic、next/navigation、React.cache、revalidate、fetch(、headers()、cookies() 等关键词时,追加 React/Next 性能审查;匹配 UI 代码或常见 UI 目录时追加界面规范审计(可从指定 URL 用 WebFetch 或 curl 获取最新规范)。随后依次进行合并就绪度审查(模板 references/code-reviewer.md,检查需求对齐、副作用是否受限、错误处理是否显式、是否存在验证证据)、Clean Code 可维护性扫描(命名、函数粒度、重复、过度设计、魔法值、结构清晰度、项目约定)、文档与代码一致性扫描(逐条核对文档中的声明、配置、示例与权威代码/契约,记录不一致及其位置)。

输出:生成结构化报告 quality-review.md 及可选的 quality-review.json;若在 Ship Faster run_dir 中运行,写入 run_dir/evidence/quality-review.md 与 .json,并按分诊结果追加 react-best-practices-review.md、ui-guidelines-review.md。报告格式含 Summary(结论:Ready / With fixes / Not ready)、Triage、Strengths、Issues(Critical/Important/Minor,含位置、原因、最小修复建议)、UI Guidelines 简明清单。

限制:具体效果、收费与平台支持等资料未提供;使用依赖 git 仓库与相应 diff 范围,部分追加审查需可访问外部规范或相应技能。