development-review

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

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	有警告项（宽松模式）