---
name: review
description: 审查当前PR，检查安全问题、代码风格和潜在bug
---

请审查当前的pull request。检查以下方面：
1. 安全漏洞（SQL注入、XSS、认证绕过等）
2. 代码风格违规（与项目现有风格不一致的地方）
3. 潜在bug（空指针、竞态条件、资源泄漏等）
4. 性能问题（N+1查询、不必要的循环等）
5. 改进建议（可读性、可维护性、测试覆盖）

输出格式：先给总体评分（1-10），再逐条列出发现的问题，最后给出改进建议。

---
name: review
description: 全面审查代码，检查安全、风格、bug和性能
args:
  - name: scope
    description: 审查范围（files/diff/pr），默认diff
    required: false
---

请审查$scope的代码变更。检查清单：
1. 🔒 **安全**：认证/授权绕过、注入漏洞、敏感数据泄露、不安全的依赖
2. 📐 **风格**：命名规范、代码组织结构、注释质量、类型使用
3. 🐛 **Bug**：空值处理、边界条件、错误处理、异步逻辑正确性
4. ⚡ **性能**：不必要的计算、内存泄漏风险、数据库查询效率
5. 💡 **改进**：可读性、可测试性、可维护性建议

输出格式：
## 审查报告
**总体评分**：X/10

### 🔴 严重问题（必须修复）
- [ ] 问题描述 + 位置 + 修复建议

### 🟡 改进建议
- [ ] 建议描述 + 理由

### 🟢 亮点
- 做得好的地方

---
name: test
description: 为指定文件/模块生成单元测试
args:
  - name: target
    description: 要测试的文件或模块路径
    required: true
---

为 $target 生成全面的单元测试。

测试框架：根据项目现有测试推断（Jest/Vitest/Pytest/Go test等）

覆盖率要求：
- 正常路径：100%
- 边界条件：至少3个用例
- 错误处理：至少2个用例
- 异步/并发场景（如适用）

每个测试用例格式：
```语言
// 测试：[一句话描述测试目的]
// 输入：[具体输入值]
// 期望：[期望输出或行为]

### 模板3：`/commit` — 自动生成commit信息

```markdown
---
name: commit
description: 分析变更并生成规范的commit message
---

分析当前git diff，生成符合 Conventional Commits 规范的commit message。

规则：
- type(scope): description
- Type: feat/fix/docs/refactor/test/chore/perf
- Description: 中文，简洁描述"做了什么、为什么"
- Body（可选）：详细说明、breaking changes、关联issue

输出格式：

如果检测到breaking change，在body第一行加 "BREAKING CHANGE: 说明"。

---
name: refactor
description: 重构指定代码，保证行为不变
args:
  - name: target
    description: 要重构的代码范围
    required: true
---

重构 $target，遵循以下原则：

**铁律**：
1. ❌ 不改变任何外部行为（输入输出、API签名、副作用）
2. ❌ 不删除任何现有测试
3. ✅ 保持向后兼容

**目标**（按优先级）：
1. 消除重复代码（DRY）
2. 降低圈复杂度（拆分过长函数，目标 <15）
3. 改善命名（让代码自解释）
4. 提取魔法数字为常量
5. 简化条件逻辑（早返回、卫语句、策略模式）

**输出**：
1. 重构前后对比（关键改动部分）
2. 重构理由（每条改动说明"为什么更好"）
3. 风险评估（哪些改动风险较高，如何验证）

---
name: explain
description: 用中文解释代码逻辑，适合onboarding和学习
args:
  - name: target
    description: 要解释的代码范围
    required: true
---

用中文解释 $target 的逻辑，假设读者是有2年经验的开发者。

输出结构：

## 概述
用1-2句话总结这段代码做什么

## 执行流程
用编号列表描述代码的执行流程

## 关键设计决策
- **为什么用这个数据结构**：
- **为什么用这个算法**：
- **为什么这样处理错误**：

## 潜在陷阱
- 哪里容易出错
- 什么场景下行为可能不符合预期

## 改进方向
- 短期：可以立即优化的点
- 长期：架构层面的改进建议

args:
  - name: scope
    description: 审查范围
    required: false

---
name: ship
description: 完整的发布前检查流程
---

请依次执行以下检查，全部通过后再给出"可以发布"的结论：

1. 运行 `/test target=src/`
2. 运行 `/review scope=diff`
3. 检查CHANGELOG是否更新
4. 检查版本号是否递增
5. 运行项目构建命令确认无编译错误

输出格式：每步的检查结果 + 最终结论（✅ 可发布 / ❌ 需修复X项）

{
  "hooks": {
    "BeforeCommit": [
      {
        "command": "/review scope=diff",
        "description": "提交前自动代码审查"
      }
    ]
  }
}