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