Agent skill

code-review

执行代码审查流程（提交PR→自查→同行审查→反馈修改→合并），覆盖功能正确性、代码质量、测试完整性、安全合规四大维度。当提交Pull Request、进行结对编程、执行代码走查、需要知识共享时使用。支持L1/L2/L3三级审查深度。

View SKILL.md on GitHub Repository
Stars 5
Forks 1

Install this agent skill to your Project

npx add-skill https://github.com/tikazyq/agentic-spec-forge/tree/main/AGENTIC_SPEC_FORGE/spec_stage_skill/execspec_fulfill/code-review

SKILL.md

Code Review Skill

Scope: EXECSPEC_FULFILL — Fulfill ExecSpec（落实 ExecSpec）

版本: 0.1.0（占位）| 创建日期: 2025-11-27

概述

Code Review 是保证代码质量的关键门禁：

┌─────────────────────────────────────────────────────┐
│              👀 Code Review Flow                    │
├─────────────────────────────────────────────────────┤
│  提交 PR → 自查 → 同行审查 → 反馈修改 → 合并       │
│  (Submit) (Self) (Peer)    (Fix)      (Merge)     │
└─────────────────────────────────────────────────────┘

核心目标：

  • 发现缺陷和潜在问题
  • 确保代码符合规范
  • 知识共享和团队学习
  • 保持代码库一致性

审查维度

1. 功能正确性

  • 代码是否实现了预期功能
  • 边界条件是否处理
  • 错误处理是否完善

2. 代码质量

  • 命名是否清晰有意义
  • 函数是否单一职责
  • 是否有重复代码
  • 复杂度是否可接受

3. 测试覆盖

  • 单元测试是否充分
  • 测试是否有意义（不是为测而测）
  • 边界用例是否覆盖

4. 安全性

  • 是否有注入风险
  • 敏感数据是否保护
  • 权限检查是否到位

5. 性能

  • 是否有明显性能问题
  • 数据库查询是否合理
  • 是否有 N+1 问题

L1-STREAMLINED 检查清单

  • 代码可读性良好
  • 测试通过
  • 无明显安全问题
  • GOAL 追溯完整

通过标准

  • 4 项全部通过（100%）

L2-BALANCED 增加项

  • 设计模式使用合理
  • 性能考量充分
  • 文档/注释适当

L3-RIGOROUS 增加项

  • 架构符合性审查
  • 安全深度审查
  • 性能基准测试

>> 命令

>>review_checklist_l1   # L1 审查清单
>>review_checklist_l2   # L2 审查清单
>>review_feedback       # 生成审查反馈

审查反馈模板

markdown
## Code Review Feedback

### 概要
- PR: #XXX
- 审查人: @reviewer
- 状态: ✅ Approved / 🔄 Changes Requested

### 发现
#### 🔴 必须修改
- [ ] 问题描述 (文件:行号)

#### 🟡 建议修改
- [ ] 改进建议

#### 🟢 良好实践
- 值得表扬的点

### 总结
整体代码质量良好/需要改进...

相关 Skills

  • 前置: tdd-cycle（代码已完成）
  • 并行: code-quality（质量门禁）
  • 原则: principle-solid, principle-dry, principle-kiss

TODO: 待细化不同语言的审查重点和自动化检查集成

Recommended Agent Skills

Expand your agent's capabilities with these related and highly-rated skills.

tikazyq/agentic-spec-forge

code-quality

执行代码质量门禁检查（测试通过→覆盖率→Lint→GOAL追溯→可提交），验证单元测试覆盖率≥80%、集成测试通过率≥95%、代码规范零错误。当完成代码实现、准备提交代码、需要质量验证、进行提交前自检时使用。确保代码可合并。

5 1
Explore
tikazyq/agentic-spec-forge

tdd-cycle

执行 TDD 红绿重构循环（RED→GREEN→REFACTOR→REPEAT），通过先写失败测试、最小代码实现、保持绿色重构的节奏驱动开发。当启动功能开发、需要测试驱动节奏、进行迭代编码、追求高测试覆盖率时使用。提供检查清单和节奏控制。

5 1
Explore
tikazyq/agentic-spec-forge

refactoring

执行代码重构（识别坏味道→选择手法→小步修改→运行测试），在保持外部行为不变前提下改进内部结构。当TDD进入REFACTOR阶段、发现代码坏味道、需要消除重复代码、优化代码结构时使用。支持提取方法、类、参数对象等重构手法。

5 1
Explore
tikazyq/agentic-spec-forge

arch-options-compare

比较多个架构方案（单体/微服务等），从SOLID原则、成本、复杂度等维度评分，生成对比表和推荐方案。当需要向客户解释架构选择或ADR决策时使用。

5 1
Explore
tikazyq/agentic-spec-forge

dev-tech-view-supplement

根据设计补充Dev视角的技术细节（数据库/API/通信协议），帮助Dev快速进入实施。当L1/L2设计缺乏技术细节、或准备向Dev交付前使用。

5 1
Explore
tikazyq/agentic-spec-forge

design-consistency-check

检查VS↔SD/DM一致性（value_path是否与序列图/数据模型匹配），生成不一致项清单和修复建议。当Self-Reflection报告一致性问题或准备CONSTRAINT验收前使用。

5 1
Explore

Didn't find tool you were looking for?

Be as detailed as possible for better results