# Development Review

> 开发规范 - 开发前规范参考 + 开发后提交前检查，必要时调用子技能进行深度验证

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

---


# Development Review

开发规范技能，覆盖开发前规范参考和开发后提交前检查。

## 定位

| 阶段 | 内容 | 说明 |
|------|------|------|
| **开发前** | 规范参考 | 架构思维、接口设计、性能优化、代码质量、TDD |
| **开发后** | 提交前检查 | 自动化检查 + verification + debugging + code-review |

**核心理念**：开发时专注实现，开发后快速验证。

---

## 开发前：规范参考

### 1. 架构思维

- **开发前分析**：了解项目目录结构，确认新功能放置位置
- **遵循现有模式**：参考同类功能实现，保持一致性
- **长期维护**：避免单文件过大，采用模块化设计
- **避免"最简单方案"陷阱**：选择最合理的方案，而非最快的

### 2. 接口设计

- **不破坏协议**：不改变已有的接口协议标准
- **增强而非新建**：在原接口基础上增强功能
- **接口统一**：避免同一功能多套实现

### 3. 性能优化

- **先分析瓶颈**：用数据说话，而非猜测
- **实测对比**：优化前后性能对比，验证效果
- **资源限制**：设置合理的最大文件大小、超时等

### 4. 代码质量

- **及时清理**：移除调试代码、注释掉的代码
- **无冗余**：不保留备用方案代码
- **命名规范**：使用统一的命名风格

### 5. 文档规范

- **必须有 README**：项目根目录必须有 README 文件
- **变更记录**：代码变更应记录在 CHANGELOG

### 6. 备份规范

- **重构前备份**：代码重构前必须创建备份
- **大规模变更提醒**：变更 > 500 行时提醒备份

---

## 开发后：提交前检查

### 检查级别

#### 快速检查（< 10秒）

跳过：编译、Lint、类型检查、冗余代码检查

适用：日常开发后快速验证

#### 完整检查

适用：提交前、代码审查、CI/CD

#### 宽松模式

只警告不阻止

适用：探索性开发、实验性功能

---

### 检查清单

#### 1. 提交前检查（优先级最高）

| 检查项 | 说明 |
|--------|------|
| 编译检查 | 运行构建命令 |
| Lint 检查 | 运行 linter |
| 类型检查 | 运行类型检查 |
| 空提交 | 检查是否有实际代码变更 |

#### 2. 代码质量检查

| 检查项 | 说明 |
|--------|------|
| 调试代码 | console.log, print, fmt.Println 等 |
| 敏感信息 | password, token, secret 等 |
| 冗余代码 | TODO, FIXME, HACK 注释 |

#### 3. 文档规范检查

| 检查项 | 说明 |
|--------|------|
| 文档完整性 | 检测 README.md |
| CHANGELOG | 检测变更日志 |

#### 4. 备份检查

| 检查项 | 说明 |
|--------|------|
| 大规模变更 | git 变更 > 500 行提醒 |

---

## 整合子技能

### 子技能列表

| 文件 | 触发场景 |
|------|---------|
| `verification.md` | 声称完成前必须验证 |
| `debugging.md` | 检查失败需要修复 |
| `tdd.md` | 开发中需要写测试 |
| `code-review.md` | 提交前人工审查 |

### 调用流程

```
开发完成 → 运行 development-review
            │
            ├── 检查失败 → debugging.md (调试)
            │                 │
            │                 └── 修复后再次检查
            │
            ├── 检查通过 → verification.md (验证证据)
            │                 │
            │                 └── 验证通过
            │
            └── code-review.md (人工审查)
                      │
                      └── 提交
```

---

## 退出码

| 码 | 说明 |
|----|------|
| 0 | 所有检查通过 |
| 1 | 有失败项（严格模式） |
| 2 | 有警告项（宽松模式） |