# Code Review

> 以高级工程师视角审查项目代码，检测 SOLID 违规、安全风险、性能隐患

- Skill: `winhok/code-review` (Agent Skill)
- Install (CLI): `npx skillmds@latest add winhok/code-review`
- Raw SKILL.md: https://api.skillmd.com/api/skills/winhok/code-review/raw
- Safety review: pending
- Works with: Claude Code, Claude.ai, OpenAI Codex
- Category: Coding & Dev Tools
- Author: winhok (https://skillmd.com/u/winhok)
- Updated: 2026-09-17
- Page: https://skillmd.com/skills/winhok/code-review

---


# Code Review

## 审查流程

**1) 确定审查范围**

先问用户要审查哪部分代码。如果用户没指定，用 `list_directory` 看一下 `src/` 的目录结构，让用户选一个模块。不要一次审查整个项目——聚焦到具体的目录或文件。

**2) 逐文件阅读和审查**

用 `read_file` 读取目标模块的源码文件，关注以下问题：
- **SRP**：一个模块是不是干了太多不相关的事
- **OCP**：新功能是靠修改已有代码实现的，还是通过扩展点
- **DIP**：高层逻辑是不是直接依赖了低层实现

如果建议重构，说清楚为什么能改善内聚/耦合，并给出最小改动方案。

**3) 安全扫描**

检查常见风险：
- 注入漏洞（SQL/命令/XSS）
- 认证/授权缺口
- 密钥泄漏（硬编码在代码里的 API Key、Token）
- 竞态条件

**4) 代码质量**

- 错误处理：有没有吞异常、catch 太宽泛、async 错误没处理
- 性能：N+1 查询、热路径上的重计算、缺少缓存
- 边界条件：null/undefined、空集合、数字溢出

## 输出格式

按 P0（必须修复）、P1（建议修复）、P2（可以改进）分级输出。
每个发现标注文件:行号和具体建议。
默认只输出审查结果，不直接改代码——除非用户明确要求。