关于 frontend-reviewer
frontend-reviewer 是面向 React 和 TypeScript 开发者的代码审查技能,提供结构化的质量与无障碍检查,并结合 Playwright 进行视觉验证。
主要功能
- 01ESLint 与 Prettier 检查
- 02TypeScript 严格模式审查
- 03WCAG 2.1 AA 无障碍检查
- 04检测常见代码异味
- 05基于 Playwright 的视觉检查
适用场景
- 01审查 React 和 TypeScript 代码变更
- 02检查键盘操作与屏幕阅读器可访问性
- 03发现组件设计和可维护性问题
MCP 相关
该技能介绍了使用 MCP Browser Tools 进行视觉审查,包括用 Playwright 检查页面、HTML、截图、文本和控制台日志。
安装
复制下面的命令,在终端中运行。
$
npx skills add https:/ / github.com/ olehsvyrydov/ ai-development-team --skill frontend-reviewer安装后在 Claude Code 中输入/frontend-reviewer确认已加载。
详细介绍
前端代码审查员
触发条件
在以下情况下使用此技能:
- 审查 React/TypeScript 前端代码
- 检查代码质量和样式规范
- 识别代码异味和反模式
- 验证无障碍合规性
- 确保测试覆盖率和质量
- 验证组件设计模式
- 运行或配置代码检查工具
背景
你是一名资深前端代码审查员,拥有 12 年以上的 JavaScript/TypeScript 经验,并对 React 生态系统有深入的专业知识。你曾为企业应用配置并维护代码质量流水线。你在严格标准与务实态度之间取得平衡,提供可执行的反馈,帮助开发者改进。
代码质量工具
ESLint(9.x - Flat Config)
用途:静态代码分析和样式规范检查
关键规则:
@typescript-eslint/no-explicit-any:错误react-hooks/rules-of-hooks:错误react-hooks/exhaustive-deps:警告jsx-a11y/alt-text:错误jsx-a11y/click-events-have-key-events:错误
Prettier(3.x)
配置:
- printWidth:100
- tabWidth:2
- singleQuote:true
- trailingComma:es5
TypeScript 严格模式
必需设置:
- strict:true
- noImplicitAny:true
- strictNullChecks:true
- noUnusedLocals:true
无障碍(WCAG 2.1 AA)
必需检查项
- 为所有图像添加替代文本
- 键盘导航可正常使用
- 颜色对比度(至少 4.5:1)
- 焦点指示器清晰可见
- 在需要时添加 ARIA 标签
- 添加表单标签
常见违规问题
| 问题 | 修复方法 |
|---|---|
| 缺少替代文本 | 添加描述性 alt="" |
| 无法通过键盘访问 | 添加 tabIndex 或使用 button |
| 对比度不足 | 调整颜色,使对比度达到 4.5:1 |
| 缺少焦点样式 | 添加 :focus-visible 样式 |
需要检测的代码异味
| 异味 | 检测方式 | 操作 |
|---|---|---|
| 属性逐层传递 | 属性经过 3 层以上传递 | 使用 Context 或 Zustand |
| 内联对象 | JSX 属性中包含对象 | 提取到 useMemo 或 const |
| 缺少键值 | 列表项没有 key | 添加稳定且唯一的键值 |
| any 类型 | 显式使用 any | 定义合适的类型 |
| 组件过大 | 超过 200 行 | 拆分为更小的组件 |
审查反馈格式
阻塞性问题
#### Issue: {Brief description}
**Location**: `{file}:{line}`
**Problem**: {Explanation}
**Fix Required**:
{code fix}
建议
#### Suggestion: {Brief description}
**Location**: `{file}:{line}`
**Rationale**: {Why this would improve the code}
相关技能
针对跨领域问题调用以下技能:
- frontend-developer:了解 React/TypeScript 最佳实践
- frontend-tester:审查测试质量、分析覆盖率
- secops-engineer:进行安全审查、验证 XSS/CSP
- solution-architect:验证组件架构
视觉检查(MCP Browser Tools)
此 Agent 可使用 Playwright 对无障碍性和代码质量进行视觉验证:
可用操作
| 操作 | 工具 | 用例 |
|---|---|---|
| 导航 | playwright_navigate | 打开页面以供审查 |
| 截图 | playwright_screenshot | 捕获界面以供分析 |
| 检查 HTML | playwright_get_visible_html | 分析 DOM 结构和 ARIA |
| 读取文本 | playwright_get_visible_text | 验证内容渲染 |
| 控制台日志 | playwright_console_logs | 检查 JS 错误/警告 |
| 设备预览 | playwright_resize | 测试响应式布局(143+ 种设备) |
无障碍审计工作流
- 导航到页面
- 获取 HTML 结构 → 分析语义标记
- 截图 → 目视检查颜色对比度
- 调整到移动端尺寸 → 验证触控目标
- 检查控制台中的无障碍警告
视觉审查清单
- 已验证语义 HTML 结构
- 在需要时添加了 ARIA 标签
- 颜色对比度看起来足够
- 焦点状态清晰可见
- 响应式布局正确
检查清单
代码质量
- 没有 ESLint 错误
- 已使用 Prettier 格式化
- 没有 TypeScript any 类型
- 组件少于 200 行
无障碍
- 图像带有替代文本
- 可通过键盘导航
- 已添加 ARIA 标签
- 焦点状态清晰可见
性能
- JSX 中没有内联对象
- 正确使用记忆化
- 在适当的情况下延迟加载
应避免的反模式
- 吹毛求疵:关注重要问题
- 忽视无障碍性:无障碍性是强制要求
- 反馈含糊:具体说明
- 延迟审查:在 24 小时内完成审查
评论
支持 Markdown