关于 frontend-review-hygiene
这项技能检查前端仓库的 TypeScript、Lint、未使用代码和重复代码,并生成 KPI 对比及分组整改建议;不会修改源文件。
主要功能
- 01审计类型、Lint 和相似度
- 02与 KPI 基线进行比较
- 03汇报质量指标及变化
- 04按影响归类整改工作
- 05检查 Lint 与格式化的职责边界
适用场景
- 01建立前端质量基线
- 02追踪每周质量退化与改善
- 03规划适用于 Pull Request 的分组清理工作
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ mizchi/ skills --skill frontend-review-hygiene安装后在 Claude Code 中输入/frontend-review-hygiene确认已加载。
附带文件README.md
详细介绍
前端审查 — 代码卫生
评估前端项目的基线代码质量卫生状况:类型、Lint、死代码和重复代码。这些 KPI 将逐周提高标准。
流程
- 并行运行:
scripts/audit-typescript.sh --repo <client-repo>scripts/audit-lint.sh --repo <client-repo>scripts/audit-similarity.sh --repo <client-repo>
- 阅读各个
raw/*.json。 - 如果
<client-repo>/.frontend-review/kpi/baseline.json存在,则与上次运行结果比较。
输出
将 <client-repo>/.frontend-review/report/latest/md/hygiene-summary.md 写入以下内容:
- KPI 表格,涵盖:
any数量、@ts-ignore数量、Lint 错误/警告、knip 未使用的文件/导出/依赖、重复代码对数量 - 与基线的差异(回退项用粗体标记;改进项用 ✅ 标记)
- 修复批次,按影响分组——哪些项目适合在同一个 PR 中修复
- 不要包含逐文件发现(这些内容保留在原始 JSON 中)。报告用于决策,而非代码审查。
如果客户端尚无基线,则创建一个:将当前 JSON 复制到 <client-repo>/.frontend-review/kpi/baseline.json,并在报告中说明这一点。
工具链职责划分
评估 Lint / 格式化工具栈时,检查各工具的职责边界是否清晰:
| 工具 | 预期职责 |
|---|---|
| TypeScript | 仅负责类型正确性 |
| ESLint / Biome / oxlint | AST 级别的错误模式、导入规则 |
| Prettier / Biome / oxfmt | 仅负责格式化——不与 Lint 重叠 |
| ast-grep / 自定义规则 | 项目特定的结构规则 |
| vitest / jest | 行为 |
职责重叠(例如,ESLint 同时负责格式化)会导致冲突并拖慢 CI。请指出并建议划分职责。
自动生成的文件(锁文件、生成的 schema、工具产物)必须从格式化运行中排除。反复格式化并还原生成文件,表明这些文件可能未被加入 .prettierignore / .biomeignore / 等效配置。
边界
- 不要评估依赖的新旧程度或 CVE——这属于
frontend-review-deps。 - 不要提出代码级修复方案。这由 5 个视角技能负责。
- 不要修改客户端仓库中的源文件。
- 不要运行
audit-security.sh——这是安全技能的工作。
参考
- 检查清单:
03-typescript.md、04-lint-format.md、05-deadcode-knip.md、06-similarity.md、13-kpi-tracking.md - 相关内容:
frontend-review-deps(依赖健康状况)、frontend-review-security(CVE + 身份验证 + 环境变量) - 阶段:
week-1-ci-baseline.md、week-4-ai-review.md
评论
支持 Markdown