前端代码审查技能

技能用途

对前端代码进行系统化审查，涵盖代码质量、性能、安全、可访问性等多个维度。支持 React、Vue、Angular 和原生 JavaScript/TypeScript 项目。

审查工作流程

第一步：识别项目类型和范围

执行以下识别工作：

确定框架类型（React、Vue、Angular、原生 JS/TS）
评估代码规模（单组件、功能模块、完整应用）
检查明显的语法错误、格式问题或导入错误

第二步：多维度系统审查

按照以下维度逐一审查代码：

代码质量

检查命名是否语义化且一致
识别代码重复和可重构的机会
确认函数/组件是否遵循单一职责原则
评估文件组织和模块结构的合理性

功能实现

验证业务逻辑正确性
检查边界情况和错误处理
审查异步操作处理（Promise、async/await）
确认加载状态和错误状态的用户反馈机制

性能优化

识别不必要的组件重渲染
检查列表渲染的 key 使用是否正确
评估对打包体积的影响
审查图片优化策略（格式、懒加载、响应式）
检查网络请求效率（缓存、去重、防抖节流）

安全性

检测 XSS 漏洞（未转义输入、dangerouslySetInnerHTML 使用）
验证输入验证和数据清洗
审查敏感数据处理（localStorage、API 令牌）
检查 CSRF 保护机制
评估第三方依赖的安全风险

可访问性 (a11y)

检查语义化 HTML 的使用
验证 ARIA 属性的正确性
测试键盘导航支持
检查颜色对比度
确认图片 alt 文本和表单 label

框架特定模式

React:

检查 Hook 使用（useEffect 依赖数组、自定义 Hook 设计）
识别闭包陷阱
验证受控与非受控组件的正确使用
评估 Context 使用的适当性
审查状态管理设计（本地 vs 全局）

Vue 3 Composition API:

检查 ref/reactive 使用是否合理
验证 computed/watch 使用是否正确
评估组件通信方式（props/emit/provide/inject）
检查生命周期钩子使用
审查组合函数（Composables）抽取是否合理

TypeScript:

验证类型定义的准确性和完整性
识别 any 类型的过度使用
检查泛型使用的合理性
评估接口/类型的可复用性

浏览器兼容性

检查目标浏览器支持的特性
验证 CSS 供应商前缀
评估 polyfill 必要性
审查响应式设计实现

第三步：生成结构化报告

使用以下格式输出审查结果：

## 🔍 代码审查报告

### 文件：[文件路径]

#### ✅ 优秀实践

列出代码中做得好的地方：
- 优雅的解决方案
- 良好的设计模式
- 清晰的代码结构
- 周到的抽象设计

#### ⚠️ 需要改进的问题

| 类别 | 问题描述 | 改进建议 | 严重程度 |
|------|---------|---------|----------|
| 性能 | [具体问题] | [改进方案] | 🔴 高 / 🟡 中 / 🟢 低 |

#### 📊 代码评分

> ### 🎯 综合评分：X.X / 10
> **等级：[🏆S/🥇A/🥈B/🥉C/⚠️D/❌F]（[等级描述]）** | **状态：[✅可直接上线/⚠️需修复后上线/❌需要重构]**

| 维度 | 得分 | 评价 |
|------|------|------|
| 代码质量 | X/10 | [简要评价] |
| 功能实现 | X/10 | [简要评价] |
| 性能优化 | X/10 | [简要评价] |
| 安全性 | X/10 | [简要评价] |
| 可访问性 | X/10 | [简要评价] |

**主要扣分项**：[具体说明扣分原因]

#### 📝 改进建议

- **优先修复**：[关键问题列表]
- **后续优化**：[可选改进方向]

评分等级标准：

等级	分数范围	状态	含义
🏆 S	9.5-10	✅ 可直接上线	卓越，可作为最佳实践范本
🥇 A	8.5-9.4	✅ 可直接上线	优秀，生产级代码
🥈 B	7.0-8.4	⚠️ 修复后上线	良好，少量问题需修复
🥉 C	5.5-6.9	⚠️ 修复后上线	一般，需要认真修复优化
⚠️ D	4.0-5.4	❌ 需要重构	较差，建议较大范围重构
❌ F	< 4.0	❌ 需要重构	不合格，建议重写

严重程度定义：

🔴 高：安全漏洞、严重 bug、崩溃风险 → 必须修复
🟡 中：性能问题、代码规范、可维护性 → 建议修复
🟢 低：代码风格、优化建议 → 可选改进

第四步：积极认可优秀实践

在报告中突出强调：

优雅的解决方案和设计模式
良好的编码实践
结构清晰的代码组织
周到的抽象和封装

审查深度调整策略

根据以下因素调整审查深度和重点：

代码规模：小改动快速审查核心问题，大功能进行全面深度审查
代码上下文：生产环境代码采用严格标准，原型代码可适度宽容
时间限制：时间紧张时按优先级审查（安全 > 性能 > 规范 > 风格）

补充资源

详细的审查检查清单和工具推荐请参考 references/checklist.md。

frontend-code-review