claude-code-workflow 6.3.22 → 6.3.24

package/.claude/skills/review-code/phases/state-schema.md

@@ -0,0 +1,174 @@
+# State Schema
+
+Code Review 状态结构定义。
+
+## Schema Definition
+
+```typescript
+interface ReviewState {
+  // === 元数据 ===
+  status: 'pending' | 'running' | 'completed' | 'failed' | 'user_exit';
+  started_at: string;      // ISO timestamp
+  updated_at: string;      // ISO timestamp
+  completed_at?: string;   // ISO timestamp
+  
+  // === 审查目标 ===
+  context: {
+    target_path: string;       // 目标路径（文件或目录）
+    files: string[];           // 待审查文件列表
+    language: string;          // 主要编程语言
+    framework?: string;        // 框架（如有）
+    total_lines: number;       // 总代码行数
+    file_count: number;        // 文件数量
+  };
+  
+  // === 扫描结果 ===
+  scan_completed: boolean;
+  scan_summary: {
+    risk_areas: RiskArea[];    // 高风险区域
+    complexity_score: number;  // 复杂度评分
+    quick_issues: QuickIssue[]; // 快速发现的问题
+  };
+  
+  // === 审查进度 ===
+  reviewed_dimensions: string[];  // 已完成的审查维度
+  current_dimension?: string;     // 当前审查维度
+  
+  // === 发现的问题 ===
+  findings: {
+    correctness: Finding[];
+    readability: Finding[];
+    performance: Finding[];
+    security: Finding[];
+    testing: Finding[];
+    architecture: Finding[];
+  };
+  
+  // === 报告状态 ===
+  report_generated: boolean;
+  report_path?: string;
+  
+  // === 执行跟踪 ===
+  current_action?: string;
+  completed_actions: string[];
+  errors: ExecutionError[];
+  error_count: number;
+  
+  // === 统计信息 ===
+  summary?: {
+    total_issues: number;
+    critical: number;
+    high: number;
+    medium: number;
+    low: number;
+    info: number;
+    review_duration_ms: number;
+  };
+}
+
+interface RiskArea {
+  file: string;
+  reason: string;
+  priority: 'high' | 'medium' | 'low';
+}
+
+interface QuickIssue {
+  type: string;
+  file: string;
+  line?: number;
+  message: string;
+}
+
+interface Finding {
+  id: string;                    // 唯一标识 e.g., "CORR-001"
+  severity: 'critical' | 'high' | 'medium' | 'low' | 'info';
+  dimension: string;             // 所属维度
+  category: string;              // 问题类别
+  file: string;                  // 文件路径
+  line?: number;                 // 行号
+  column?: number;               // 列号
+  code_snippet?: string;         // 问题代码片段
+  description: string;           // 问题描述
+  recommendation: string;        // 修复建议
+  fix_example?: string;          // 修复示例代码
+  references?: string[];         // 参考资料链接
+}
+
+interface ExecutionError {
+  action: string;
+  message: string;
+  timestamp: string;
+}
+```
+
+## Initial State
+
+```json
+{
+  "status": "pending",
+  "started_at": "2024-01-01T00:00:00.000Z",
+  "updated_at": "2024-01-01T00:00:00.000Z",
+  "context": null,
+  "scan_completed": false,
+  "scan_summary": null,
+  "reviewed_dimensions": [],
+  "current_dimension": null,
+  "findings": {
+    "correctness": [],
+    "readability": [],
+    "performance": [],
+    "security": [],
+    "testing": [],
+    "architecture": []
+  },
+  "report_generated": false,
+  "report_path": null,
+  "current_action": null,
+  "completed_actions": [],
+  "errors": [],
+  "error_count": 0,
+  "summary": null
+}
+```
+
+## State Transitions
+
+```mermaid
+stateDiagram-v2
+    [*] --> pending: Initialize
+    pending --> running: collect-context
+    running --> running: quick-scan
+    running --> running: deep-review (6x)
+    running --> running: generate-report
+    running --> completed: complete
+    running --> failed: error_count >= 3
+    running --> user_exit: User abort
+    completed --> [*]
+    failed --> [*]
+    user_exit --> [*]
+```
+
+## Dimension Review Order
+
+1. **correctness** - 正确性（最高优先级）
+2. **security** - 安全性（关键）
+3. **performance** - 性能
+4. **readability** - 可读性
+5. **testing** - 测试覆盖
+6. **architecture** - 架构一致性
+
+## Finding ID Format
+
+```
+{DIMENSION_PREFIX}-{SEQUENCE}
+
+Prefixes:
+- CORR: Correctness
+- READ: Readability
+- PERF: Performance
+- SEC:  Security
+- TEST: Testing
+- ARCH: Architecture
+
+Example: SEC-003 = Security issue #3
+```

package/.claude/skills/review-code/specs/issue-classification.md

@@ -0,0 +1,228 @@
+# Issue Classification
+
+问题分类和严重程度标准。
+
+## When to Use
+
+| Phase | Usage | Section |
+|-------|-------|---------|
+| action-deep-review | 确定问题严重程度 | Severity Levels |
+| action-generate-report | 问题分类展示 | Category Mapping |
+
+---
+
+## Severity Levels
+
+### Critical (严重) 🔴
+
+**定义**: 必须在合并前修复的阻塞性问题
+
+**标准**:
+- 安全漏洞 (可被利用)
+- 数据损坏或丢失风险
+- 系统崩溃风险
+- 生产环境重大故障
+
+**示例**:
+- SQL/XSS/命令注入
+- 硬编码密钥泄露
+- 未捕获的异常导致崩溃
+- 数据库事务未正确处理
+
+**响应**: 必须立即修复，阻塞合并
+
+---
+
+### High (高) 🟠
+
+**定义**: 应在合并前修复的重要问题
+
+**标准**:
+- 功能缺陷
+- 重要边界条件未处理
+- 性能严重退化
+- 资源泄漏
+
+**示例**:
+- 核心业务逻辑错误
+- 内存泄漏
+- N+1 查询问题
+- 缺少必要的错误处理
+
+**响应**: 强烈建议修复
+
+---
+
+### Medium (中) 🟡
+
+**定义**: 建议修复的代码质量问题
+
+**标准**:
+- 代码可维护性问题
+- 轻微性能问题
+- 测试覆盖不足
+- 不符合团队规范
+
+**示例**:
+- 函数过长
+- 命名不清晰
+- 缺少注释
+- 代码重复
+
+**响应**: 建议在后续迭代修复
+
+---
+
+### Low (低) 🔵
+
+**定义**: 可选优化的问题
+
+**标准**:
+- 风格问题
+- 微小优化
+- 可读性改进
+
+**示例**:
+- 变量声明顺序
+- 额外的空行
+- 可以更简洁的写法
+
+**响应**: 可根据团队偏好处理
+
+---
+
+### Info (信息) ⚪
+
+**定义**: 信息性建议，非问题
+
+**标准**:
+- 学习机会
+- 替代方案建议
+- 文档完善建议
+
+**示例**:
+- "这里可以考虑使用新的 API"
+- "建议添加 JSDoc 注释"
+- "可以参考 xxx 模式"
+
+**响应**: 仅供参考
+
+---
+
+## Category Mapping
+
+### By Dimension
+
+| Dimension | Common Categories |
+|-----------|-------------------|
+| Correctness | `null-check`, `boundary`, `error-handling`, `type-safety`, `logic-error` |
+| Security | `injection`, `xss`, `hardcoded-secret`, `auth`, `sensitive-data` |
+| Performance | `complexity`, `n+1-query`, `memory-leak`, `blocking-io`, `inefficient-algorithm` |
+| Readability | `naming`, `function-length`, `complexity`, `comments`, `duplication` |
+| Testing | `coverage`, `boundary-test`, `mock-abuse`, `test-isolation` |
+| Architecture | `layer-violation`, `circular-dependency`, `coupling`, `srp-violation` |
+
+### Category Details
+
+#### Correctness Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `null-check` | 缺少空值检查 | High |
+| `boundary` | 边界条件未处理 | High |
+| `error-handling` | 错误处理不当 | High |
+| `type-safety` | 类型安全问题 | Medium |
+| `logic-error` | 逻辑错误 | Critical/High |
+| `resource-leak` | 资源泄漏 | High |
+
+#### Security Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `injection` | 注入风险 (SQL/Command) | Critical |
+| `xss` | 跨站脚本风险 | Critical |
+| `hardcoded-secret` | 硬编码密钥 | Critical |
+| `auth` | 认证授权问题 | High |
+| `sensitive-data` | 敏感数据暴露 | High |
+| `insecure-dependency` | 不安全依赖 | Medium |
+
+#### Performance Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `complexity` | 高算法复杂度 | Medium |
+| `n+1-query` | N+1 查询问题 | High |
+| `memory-leak` | 内存泄漏 | High |
+| `blocking-io` | 阻塞 I/O | Medium |
+| `inefficient-algorithm` | 低效算法 | Medium |
+| `missing-cache` | 缺少缓存 | Low |
+
+#### Readability Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `naming` | 命名问题 | Medium |
+| `function-length` | 函数过长 | Medium |
+| `nesting-depth` | 嵌套过深 | Medium |
+| `comments` | 注释问题 | Low |
+| `duplication` | 代码重复 | Medium |
+| `magic-number` | 魔法数字 | Low |
+
+#### Testing Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `coverage` | 测试覆盖不足 | Medium |
+| `boundary-test` | 缺少边界测试 | Medium |
+| `mock-abuse` | Mock 过度使用 | Low |
+| `test-isolation` | 测试不独立 | Medium |
+| `flaky-test` | 不稳定测试 | High |
+
+#### Architecture Categories
+
+| Category | Description | Default Severity |
+|----------|-------------|------------------|
+| `layer-violation` | 层次违规 | Medium |
+| `circular-dependency` | 循环依赖 | High |
+| `coupling` | 耦合过紧 | Medium |
+| `srp-violation` | 单一职责违规 | Medium |
+| `god-class` | 上帝类 | High |
+
+---
+
+## Finding ID Format
+
+```
+{PREFIX}-{NNN}
+
+Prefixes by Dimension:
+- CORR: Correctness
+- SEC:  Security
+- PERF: Performance
+- READ: Readability
+- TEST: Testing
+- ARCH: Architecture
+
+Examples:
+- SEC-001: First security finding
+- CORR-015: 15th correctness finding
+```
+
+---
+
+## Quality Gates
+
+| Gate | Condition | Action |
+|------|-----------|--------|
+| **Block** | Critical > 0 | 禁止合并 |
+| **Warn** | High > 0 | 需要审批 |
+| **Pass** | Critical = 0, High = 0 | 允许合并 |
+
+### Recommended Thresholds
+
+| Metric | Ideal | Acceptable | Needs Work |
+|--------|-------|------------|------------|
+| Critical | 0 | 0 | Any > 0 |
+| High | 0 | ≤ 2 | > 2 |
+| Medium | ≤ 5 | ≤ 10 | > 10 |
+| Total | ≤ 10 | ≤ 20 | > 20 |

package/.claude/skills/review-code/specs/quality-standards.md

@@ -0,0 +1,214 @@
+# Quality Standards
+
+代码审查质量标准。
+
+## When to Use
+
+| Phase | Usage | Section |
+|-------|-------|---------|
+| action-generate-report | 质量评估 | Quality Dimensions |
+| action-complete | 最终评分 | Quality Gates |
+
+---
+
+## Quality Dimensions
+
+### 1. Completeness (完整性) - 25%
+
+**评估审查覆盖的完整程度**
+
+| Score | Criteria |
+|-------|----------|
+| 100% | 所有维度审查完成，所有高风险文件检查 |
+| 80% | 核心维度完成，主要文件检查 |
+| 60% | 部分维度完成 |
+| < 60% | 审查不完整 |
+
+**检查点**:
+- [ ] 6 个维度全部审查
+- [ ] 高风险区域重点检查
+- [ ] 关键文件覆盖
+
+---
+
+### 2. Accuracy (准确性) - 25%
+
+**评估发现问题的准确程度**
+
+| Score | Criteria |
+|-------|----------|
+| 100% | 问题定位准确，分类正确，无误报 |
+| 80% | 偶有分类偏差，定位准确 |
+| 60% | 存在误报或漏报 |
+| < 60% | 准确性差 |
+
+**检查点**:
+- [ ] 问题行号准确
+- [ ] 严重程度合理
+- [ ] 分类正确
+
+---
+
+### 3. Actionability (可操作性) - 25%
+
+**评估建议的实用程度**
+
+| Score | Criteria |
+|-------|----------|
+| 100% | 每个问题都有具体可执行的修复建议 |
+| 80% | 大部分问题有清晰建议 |
+| 60% | 建议较笼统 |
+| < 60% | 缺乏可操作建议 |
+
+**检查点**:
+- [ ] 提供具体修复建议
+- [ ] 包含代码示例
+- [ ] 说明修复优先级
+
+---
+
+### 4. Consistency (一致性) - 25%
+
+**评估审查标准的一致程度**
+
+| Score | Criteria |
+|-------|----------|
+| 100% | 相同问题相同处理，标准统一 |
+| 80% | 基本一致，偶有差异 |
+| 60% | 标准不太统一 |
+| < 60% | 标准混乱 |
+
+**检查点**:
+- [ ] ID 格式统一
+- [ ] 严重程度标准一致
+- [ ] 描述风格统一
+
+---
+
+## Quality Gates
+
+### Review Quality Gate
+
+| Gate | Overall Score | Action |
+|------|---------------|--------|
+| **Excellent** | ≥ 90% | 高质量审查 |
+| **Good** | ≥ 80% | 合格审查 |
+| **Acceptable** | ≥ 70% | 基本可接受 |
+| **Needs Improvement** | < 70% | 需要改进 |
+
+### Code Quality Gate (Based on Findings)
+
+| Gate | Condition | Recommendation |
+|------|-----------|----------------|
+| **Block** | Critical > 0 | 禁止合并，必须修复 |
+| **Warn** | High > 3 | 需要团队讨论 |
+| **Caution** | Medium > 10 | 建议改进 |
+| **Pass** | 其他 | 可以合并 |
+
+---
+
+## Report Quality Checklist
+
+### Structure
+
+- [ ] 包含审查概览
+- [ ] 包含问题统计
+- [ ] 包含高风险区域
+- [ ] 包含问题详情
+- [ ] 包含修复建议
+
+### Content
+
+- [ ] 问题描述清晰
+- [ ] 文件位置准确
+- [ ] 代码片段有效
+- [ ] 修复建议具体
+- [ ] 优先级明确
+
+### Format
+
+- [ ] Markdown 格式正确
+- [ ] 表格对齐
+- [ ] 代码块语法正确
+- [ ] 链接有效
+- [ ] 无拼写错误
+
+---
+
+## Validation Function
+
+```javascript
+function validateReviewQuality(state) {
+  const scores = {
+    completeness: 0,
+    accuracy: 0,
+    actionability: 0,
+    consistency: 0
+  };
+  
+  // 1. Completeness
+  const dimensionsReviewed = state.reviewed_dimensions?.length || 0;
+  scores.completeness = (dimensionsReviewed / 6) * 100;
+  
+  // 2. Accuracy (需要人工验证或后续反馈)
+  // 暂时基于有无错误来估算
+  scores.accuracy = state.error_count === 0 ? 100 : Math.max(0, 100 - state.error_count * 20);
+  
+  // 3. Actionability
+  const findings = Object.values(state.findings).flat();
+  const withRecommendations = findings.filter(f => f.recommendation).length;
+  scores.actionability = findings.length > 0 
+    ? (withRecommendations / findings.length) * 100 
+    : 100;
+  
+  // 4. Consistency (检查 ID 格式等)
+  const validIds = findings.filter(f => /^(CORR|SEC|PERF|READ|TEST|ARCH)-\d{3}$/.test(f.id)).length;
+  scores.consistency = findings.length > 0 
+    ? (validIds / findings.length) * 100 
+    : 100;
+  
+  // Overall
+  const overall = (
+    scores.completeness * 0.25 +
+    scores.accuracy * 0.25 +
+    scores.actionability * 0.25 +
+    scores.consistency * 0.25
+  );
+  
+  return {
+    scores,
+    overall,
+    gate: overall >= 90 ? 'excellent' :
+          overall >= 80 ? 'good' :
+          overall >= 70 ? 'acceptable' : 'needs_improvement'
+  };
+}
+```
+
+---
+
+## Improvement Recommendations
+
+### If Completeness is Low
+
+- 增加扫描的文件范围
+- 确保所有维度都被审查
+- 重点关注高风险区域
+
+### If Accuracy is Low
+
+- 提高规则精度
+- 减少误报
+- 验证行号准确性
+
+### If Actionability is Low
+
+- 为每个问题添加修复建议
+- 提供代码示例
+- 说明修复步骤
+
+### If Consistency is Low
+
+- 统一 ID 格式
+- 标准化严重程度判定
+- 使用模板化描述