Code Review1

---
name: core-code-review
description: 在代码提交前或任务完成后使用，进行代码审查
---

# Code Review - 代码审查

## 概述

系统性地审查代码变更，确保质量、可维护性和正确性。

## 审查维度

### 1. 正确性

- 逻辑是否正确？
- 边界情况是否处理？
- 错误处理是否完整？

### 2. 可读性

- 命名是否清晰？
- 代码结构是否清晰？
- 注释是否有帮助（不是噪音）？

### 3. 可维护性

- 是否遵循 DRY 原则？
- 是否容易修改和扩展？
- 依赖是否合理？

### 4. 测试

- 测试覆盖是否充分？
- 测试是否有意义？
- 测试是否可读？

## 问题分级

| 级别 | 含义 | 处理 |
|------|------|------|
| **Critical** | 必须修复才能合并 | 阻塞 |
| **Important** | 应该修复 | 建议修复 |
| **Minor** | 可以改进 | 可选 |
| **Nitpick** | 个人偏好 | 仅供参考 |

## 审查输出格式

```markdown
## 代码审查报告

### 概览
- 变更: [简述变更内容]
- 审查者: AI
- 日期: YYYY-MM-DD

### 问题

#### Critical
- [ ] **文件:行号** - 问题描述

#### Important  
- [ ] **文件:行号** - 问题描述

#### Minor
- [ ] **文件:行号** - 问题描述

### 优点
- [列出代码的优点]

### 结论
[通过/需要修改后通过/不通过]
```

## 嵌入式领域扩展

**额外检查项:**
- 内存分配是否安全？
- 是否有资源泄漏风险？
- 中断上下文是否正确使用？
- 时序要求是否满足？
- 功耗影响是否考虑？