@umsai/ums-code 0.3.0-v2 → 0.5.0-v1

package/dist/src/ui/commands/reviewCommand.js

@@ -0,0 +1,1267 @@
+/**
+ * Review Command - 智能代码评审命令
+ *
+ * 提供多模式代码评审功能，支持智能分批处理和文件过滤
+ *
+ * @module reviewCommand
+ */
+import { CommandKind } from './types.js';
+import { simpleGit } from 'simple-git';
+import { resolve } from 'path';
+// ============================================================================
+// 常量配置
+// ============================================================================
+const REVIEW_THRESHOLDS = {
+    /** 超过此数量的文件被认为是大型变更 */
+    LARGE_FILE_THRESHOLD: 50,
+    /** 超过此数量的行变更被认为是大型变更 */
+    LARGE_CHANGE_THRESHOLD: 3000,
+    /** 每批最多处理的文件数 */
+    MAX_FILES_PER_BATCH: 20,
+    /** 每批最多处理的行变更数 */
+    MAX_CHANGES_PER_BATCH: 2000,
+};
+const COMMIT_HASH_PATTERN = /^[0-9a-f]{7,}$/i;
+// ============================================================================
+// 错误类型
+// ============================================================================
+class ReviewCommandError extends Error {
+    code;
+    constructor(message, code) {
+        super(message);
+        this.name = 'ReviewCommandError';
+        this.code = code;
+    }
+}
+// ============================================================================
+// 工具类
+// ============================================================================
+/**
+ * Git 操作工具类
+ */
+class GitUtils {
+    /**
+     * 检测字符串是否像 commit hash（至少7位十六进制字符）
+     */
+    static looksLikeCommitHash(str) {
+        return COMMIT_HASH_PATTERN.test(str);
+    }
+    /**
+     * 根据 review 模式构建 git comparison 字符串
+     */
+    static buildComparisonString(options, format = 'command') {
+        const { mode, commit1, commit2, branch1, branch2 } = options;
+        const separator = format === 'range' ? '..' : ' ';
+        switch (mode) {
+            case 'default':
+                return 'HEAD';
+            case 'commit':
+                if (commit2) {
+                    return `${commit1}${separator}${commit2}`;
+                }
+                return `${commit1}^${separator}${commit1}`;
+            case 'branch':
+                if (branch2) {
+                    return `${branch1}${separator}${branch2}`;
+                }
+                return `main${separator}${branch1}`;
+            default:
+                throw new ReviewCommandError(`Unknown review mode: ${mode}`);
+        }
+    }
+    /**
+     * 构建 git diff --stat 命令
+     */
+    static buildDiffStatCommand(options) {
+        const comparison = this.buildComparisonString(options, 'command');
+        return `git diff --stat ${comparison}`;
+    }
+    /**
+     * 构建 git diff 命令
+     */
+    static buildDiffCommand(options) {
+        const comparison = this.buildComparisonString(options, 'command');
+        return `git diff ${comparison}`;
+    }
+    /**
+     * 构建 git diff 参数数组
+     */
+    static buildDiffArgs(options, command) {
+        const baseArgs = command === 'diff-stat'
+            ? ['diff', '--stat']
+            : ['diff', '--name-only'];
+        const comparison = this.buildComparisonString(options, 'range');
+        return [...baseArgs, comparison];
+    }
+}
+// ============================================================================
+// 参数解析器
+// ============================================================================
+/**
+ * Review 命令参数解析器
+ */
+class ReviewArgumentParser {
+    /**
+     * 解析 review 命令的参数
+     */
+    static parse(args) {
+        const trimmedArgs = args.trim();
+        if (!trimmedArgs) {
+            return { mode: 'default' };
+        }
+        const parts = trimmedArgs.split(/\s+/);
+        // 提取 --prompt-enhance 和 --notice 参数
+        const enhanceOptions = this.extractEnhanceOption(parts);
+        const noticeOption = this.extractNoticeOption(parts);
+        const filteredParts = this.removeOptions(parts);
+        if (filteredParts.length === 0) {
+            return { mode: 'default', ...enhanceOptions, ...noticeOption };
+        }
+        const command = filteredParts[0];
+        switch (command) {
+            case '-commit':
+                return { ...this.parseCommitMode(filteredParts), ...enhanceOptions, ...noticeOption };
+            case '-branch':
+                return { ...this.parseBranchMode(filteredParts), ...enhanceOptions, ...noticeOption };
+            default:
+                return { ...this.handleUnknownCommand(filteredParts), ...enhanceOptions, ...noticeOption };
+        }
+    }
+    /**
+     * 提取 --prompt-enhance 参数
+     */
+    static extractEnhanceOption(parts) {
+        for (let i = 0; i < parts.length; i++) {
+            if (parts[i] === '--prompt-enhance' && i + 1 < parts.length) {
+                return { promptEnhancePath: parts[i + 1] };
+            }
+        }
+        return {};
+    }
+    /**
+     * 提取 --notice 参数
+     */
+    static extractNoticeOption(parts) {
+        return { notice: parts.includes('--notice') };
+    }
+    /**
+     * 移除所有选项参数（--prompt-enhance 和 --notice）
+     */
+    static removeOptions(parts) {
+        const filtered = [];
+        for (let i = 0; i < parts.length; i++) {
+            if (parts[i] === '--prompt-enhance') {
+                i++; // 跳过参数值
+            }
+            else if (parts[i] === '--notice') {
+                // 跳过该标志
+            }
+            else {
+                filtered.push(parts[i]);
+            }
+        }
+        return filtered;
+    }
+    /**
+     * 解析 commit 模式参数
+     */
+    static parseCommitMode(parts) {
+        if (parts.length < 2) {
+            throw new ReviewCommandError('Commit mode requires at least one commit hash', 'MISSING_COMMIT_HASH');
+        }
+        return {
+            mode: 'commit',
+            commit1: parts[1],
+            commit2: parts[2],
+        };
+    }
+    /**
+     * 解析 branch 模式参数
+     */
+    static parseBranchMode(parts) {
+        if (parts.length < 2) {
+            throw new ReviewCommandError('Branch mode requires at least one branch name', 'MISSING_BRANCH_NAME');
+        }
+        return {
+            mode: 'branch',
+            branch1: parts[1],
+            branch2: parts[2],
+        };
+    }
+    /**
+     * 处理未知命令或可能的语法错误
+     */
+    static handleUnknownCommand(parts) {
+        // 检测是否忘记添加 -commit 或 -branch 参数
+        if (parts.length >= 1 && GitUtils.looksLikeCommitHash(parts[0])) {
+            throw new ReviewCommandError(this.buildUsageErrorMessage(parts[0], parts[1]), 'INVALID_SYNTAX');
+        }
+        // 默认模式
+        return { mode: 'default' };
+    }
+    /**
+     * 构建使用错误提示信息
+     */
+    static buildUsageErrorMessage(commitHash, secondArg) {
+        const example = secondArg
+            ? `/review -commit ${commitHash} ${secondArg}`
+            : `/review -commit ${commitHash}`;
+        return `Invalid syntax. Did you forget '-commit'?
+
+Usage:
+  /review -commit <commit1> [commit2]
+  /review -branch <branch1> [branch2]
+  /review  (for uncommitted changes)
+
+Example:
+  ${example}`;
+    }
+}
+// ============================================================================
+// Git 统计分析器
+// ============================================================================
+/**
+ * Git 统计分析器
+ */
+class GitStatsAnalyzer {
+    /**
+     * 获取 git diff 统计信息
+     */
+    static async getDiffStats(projectRoot, options) {
+        const git = this.createGitInstance(projectRoot);
+        try {
+            const diffArgs = GitUtils.buildDiffArgs(options, 'diff-stat');
+            const statsOutput = await git.raw(diffArgs);
+            return this.parseStatsOutput(statsOutput);
+        }
+        catch (error) {
+            throw new ReviewCommandError(`Failed to get git diff stats: ${error instanceof Error ? error.message : String(error)}`, 'GIT_STATS_ERROR');
+        }
+    }
+    /**
+     * 创建 Git 实例
+     */
+    static createGitInstance(projectRoot) {
+        return simpleGit(projectRoot, {
+            binary: 'git',
+            maxConcurrentProcesses: 1,
+        });
+    }
+    /**
+     * 解析 git diff --stat 输出
+     */
+    static parseStatsOutput(statsOutput) {
+        const lines = statsOutput.trim().split('\n');
+        if (lines.length === 0 || !lines[0]) {
+            return this.createEmptyStats();
+        }
+        const fileCount = this.calculateFileCount(lines);
+        const { addedLines, deletedLines } = this.calculateLineChanges(lines);
+        return {
+            fileCount,
+            addedLines,
+            deletedLines,
+            totalChanges: addedLines + deletedLines,
+        };
+    }
+    /**
+     * 创建空的统计对象
+     */
+    static createEmptyStats() {
+        return {
+            fileCount: 0,
+            addedLines: 0,
+            deletedLines: 0,
+            totalChanges: 0,
+        };
+    }
+    /**
+     * 计算文件数量
+     */
+    static calculateFileCount(lines) {
+        return lines.length > 1 ? lines.length - 1 : 0;
+    }
+    /**
+     * 计算行变更统计
+     */
+    static calculateLineChanges(lines) {
+        let addedLines = 0;
+        let deletedLines = 0;
+        for (const line of lines) {
+            const lineStats = this.parseLineStats(line);
+            if (lineStats) {
+                addedLines += lineStats.addedLines;
+                deletedLines += lineStats.deletedLines;
+            }
+        }
+        return { addedLines, deletedLines };
+    }
+    /**
+     * 解析单行统计信息
+     * 匹配格式：'file.ts | 10 ++++------'
+     */
+    static parseLineStats(line) {
+        const match = line.match(/\|\s*(\d+)\s*([+-]+)?/);
+        if (!match)
+            return null;
+        const totalChanges = parseInt(match[1], 10);
+        const signs = match[2] || '';
+        const additions = (signs.match(/\+/g) || []).length;
+        const deletions = (signs.match(/-/g) || []).length;
+        if (additions === 0 && deletions === 0) {
+            // 如果没有符号，估算各占一半
+            return {
+                addedLines: Math.floor(totalChanges / 2),
+                deletedLines: Math.ceil(totalChanges / 2),
+            };
+        }
+        // 按比例计算实际行数
+        const total = additions + deletions;
+        return {
+            addedLines: Math.round((additions / total) * totalChanges),
+            deletedLines: Math.round((deletions / total) * totalChanges),
+        };
+    }
+}
+// ============================================================================
+// Prompt 构建器
+// ============================================================================
+/**
+ * 代码评审 Prompt 构建器
+ */
+class ReviewPromptBuilder {
+    /**
+     * 构建小型变更的评审 prompt
+     */
+    static buildSmallChangePrompt(stats, projectRoot, gitDiffCommand, options) {
+        return this.buildPromptTemplate([
+            this.buildHeader(stats),
+            this.buildExecutionFlow(options),
+            this.buildFileFilteringPhase(projectRoot, gitDiffCommand),
+            this.buildCodeReviewPhase(projectRoot, gitDiffCommand, options),
+            this.buildReportGenerationPhase(stats),
+            this.buildQualityCheckPhase(projectRoot, options),
+            this.buildCallToAction()
+        ]);
+    }
+    /**
+     * 构建大型变更的评审 prompt
+     */
+    static buildLargeChangePrompt(stats, projectRoot, gitDiffStatCommand, options) {
+        return this.buildPromptTemplate([
+            this.buildLargeChangeHeader(stats),
+            this.buildLargeChangeExecutionFlow(options),
+            this.buildAnalysisPhase(projectRoot, gitDiffStatCommand),
+            this.buildBatchReviewPhase(projectRoot, options),
+            this.buildSummaryPhase(stats, projectRoot, options),
+            this.buildLargeChangeCallToAction()
+        ]);
+    }
+    /**
+     * 构建 prompt 模板
+     */
+    static buildPromptTemplate(sections) {
+        return sections.join('\n\n---\n\n');
+    }
+    /**
+     * 构建变更摘要头部
+     */
+    static buildHeader(stats) {
+        return `# 代码评审任务
+
+## 变更摘要
+
+- **文件总数**: ${stats.fileCount} 个
+- **新增行数**: ${stats.addedLines} 行
+- **删除行数**: ${stats.deletedLines} 行
+- **总变更量**: ${stats.totalChanges} 行
+
+变更规模适中，将使用 code-reviewer 一次性完成评审。`;
+    }
+    /**
+     * 构建大型变更头部
+     */
+    static buildLargeChangeHeader(stats) {
+        return `# 代码评审任务 - 大型变更检测
+
+## 变更摘要
+
+- **文件总数**: ${stats.fileCount} 个
+- **新增行数**: ${stats.addedLines} 行
+- **删除行数**: ${stats.deletedLines} 行
+- **总变更量**: ${stats.totalChanges} 行
+
+由于这是一个**大型变更**（超过 ${REVIEW_THRESHOLDS.LARGE_FILE_THRESHOLD} 个文件或 ${REVIEW_THRESHOLDS.LARGE_CHANGE_THRESHOLD} 行代码），需要分批次进行评审。`;
+    }
+    /**
+     * 构建执行流程
+     */
+    static buildExecutionFlow(options) {
+        const tasks = [
+            '1. AI 智能文件过滤',
+            '2. AI 智能代码评审',
+            '3. 生成评审报告',
+            '4. 评审报告质量检查'
+        ];
+        if (options?.notice) {
+            tasks.push('5. 生成完成标志文件');
+        }
+        return `## 执行流程
+
+请使用 **TodoWrite** 工具创建并跟踪以下任务：
+
+\`\`\`
+${tasks.join('\n')}
+\`\`\``;
+    }
+    /**
+     * 构建大型变更执行流程
+     */
+    static buildLargeChangeExecutionFlow(options) {
+        const noticeTask = options?.notice ? '\n6. 生成完成标志文件' : '';
+        return `## 执行流程（分三个阶段）
+
+### 阶段一：智能分析与过滤
+
+请使用 **TodoWrite** 创建初始任务列表：
+
+\`\`\`
+1. 分析变更文件
+2. AI 智能文件过滤
+3. AI 智能制定分批次评审策略
+4. 复核分批次评审策略
+5. [后续任务待分批完成后添加]${noticeTask}
+\`\`\``;
+    }
+    /**
+     * 构建文件过滤阶段
+     */
+    static buildFileFilteringPhase(projectRoot, gitDiffCommand) {
+        return `## 第一阶段：AI 智能文件过滤
+
+### 步骤 1：获取变更文件列表并进行智能过滤
+
+执行以下 bash 命令获取变更文件统计：
+
+\`\`\`bash
+cd ${projectRoot}
+git diff --stat ${gitDiffCommand.replace('git diff', '').trim()}
+\`\`\`
+
+**然后对变更文件进行 AI 智能过滤分析**：
+
+${this.buildFileFilteringRules()}
+
+${this.buildFileFilteringOutputFormat()}
+
+### 步骤 2：检查过滤结果
+
+**重要**：如果所有文件都被过滤掉（需要评审的文件为 0），则：
+1. 输出消息："✅ 本次变更无需代码评审（全部为文档、配置或自动生成文件）"
+2. 创建简单的 CODE_REVIEW.md 文件说明情况
+3. 结束评审流程
+
+如果还有需要评审的文件，继续下一步。`;
+    }
+    /**
+     * 构建文件过滤规则
+     */
+    static buildFileFilteringRules() {
+        return `**过滤原则**：
+1. **需要评审的文件**：
+   - 所有源代码文件（.java, .ts, .js, .py, .go, .rs, etc.）
+   - 配置文件（特别是涉及权限、安全、路由的配置）
+   - SQL/数据库迁移文件
+   - API定义文件（OpenAPI, GraphQL schemas）
+   - 关键的构建脚本（如果包含复杂逻辑）
+
+2. **可以过滤掉的文件**：
+   - 纯文档文件（README.md, CHANGELOG.md等，除非是技术文档）
+   - 依赖锁文件（package-lock.json, yarn.lock, Cargo.lock等）
+   - 二进制文件和媒体文件（图片、字体、音视频等）
+   - 自动生成的代码（如果有明确标记）
+   - 构建产物和编译输出
+   - IDE配置文件（.vscode, .idea等）
+   - Git相关配置（.gitignore, .gitattributes）
+
+3. **不确定时的策略**：
+   - 当无法确定文件类型或用途时，**倾向于包含而不是排除**
+   - 对于混合内容（如包含代码示例的markdown），考虑保留
+   - 对于小文件（<50行），包含成本低，可以保留`;
+    }
+    /**
+     * 构建文件过滤输出格式
+     */
+    static buildFileFilteringOutputFormat() {
+        return `**请使用以下格式输出过滤结果**：
+
+\`\`\`markdown
+## 文件过滤分析
+
+### 需要评审的文件（共 X 个，约 Y 行变更）
+
+1. [文件路径] - [变更行数] 行变更
+2. [文件路径] - [变更行数] 行变更
+...
+
+### 已过滤的文件（共 Z 个）
+
+**[类别名称]（N个）**：
+- [文件路径] - [过滤原因]
+...
+
+### 统计
+- 原始变更文件总数：X 个
+- 需要评审：X 个（Y%）
+- 已过滤：Z 个（W%）
+- 预计需要评审的代码行数：约 N 行
+\`\`\``;
+    }
+    /**
+     * 构建代码评审阶段
+     */
+    static buildCodeReviewPhase(projectRoot, gitDiffCommand, options) {
+        return `## 第二阶段：执行代码评审
+
+### 步骤 3：AI 智能代码评审
+
+使用 **Task tool** 调用 code-reviewer subagent，**仅对需要评审的文件进行评审**：
+
+\`\`\`typescript
+{
+  subagent_type: 'code-reviewer',
+  description: '代码评审 - 完整变更',
+  prompt: \`${this.buildSubagentPrompt(projectRoot, gitDiffCommand, options)}\`
+}
+\`\`\`
+
+### 步骤 4：等待评审完成
+
+subagent 会自动：
+1. 执行 git diff 命令获取变更内容
+2. 使用 Read tool 读取相关文件（重点关注需要评审的文件）
+3. 按照六个维度进行深度代码评审
+4. 进行自我检查确保评审质量`;
+    }
+    /**
+     * 构建附加规则部分
+     */
+    static buildEnhancedRulesSection(projectRoot, options) {
+        // 构建附加规则文件列表
+        const enhancedRules = [];
+        enhancedRules.push(`${projectRoot}/UMS.md`);
+        if (options?.promptEnhancePath) {
+            enhancedRules.push(resolve(projectRoot, options.promptEnhancePath));
+        }
+        return enhancedRules.length > 0 ? `
+## 附加规则
+
+尝试读取以下文件（如果文件不存在则跳过）：
+
+${enhancedRules.map(file => `- ${file}`).join('\n')}
+
+从这些文件中提取开发规范、编码标准和评审规则，作为附加规则。
+
+如果代码违反了附加规则，将其报告为问题并给出适当的严重性和维度。
+
+` : '';
+    }
+    /**
+     * 构建 subagent prompt
+     */
+    static buildSubagentPrompt(projectRoot, gitDiffCommand, options) {
+        return `请按照你的标准评审流程进行全面代码评审。
+
+## 变更信息
+
+- 需要评审的文件：X 个
+- 变更行数：约 Y 行
+
+## 需要评审的文件列表
+
+- file1.ts
+- file2.ts
+- file3.ts
+...
+
+## 获取变更内容
+
+请执行以下命令获取代码变更：
+
+\\\`\\\`\\\`bash
+cd ${projectRoot}
+${gitDiffCommand} -- file1.ts file2.ts file3.ts
+\\\`\\\`\\\`
+
+（将 file1.ts file2.ts file3.ts 替换为实际的批次文件列表）
+
+${this.buildEnhancedRulesSection(projectRoot, options)}
+
+## 结果检查
+
+返回结果前必须自检，剔除误报的问题，修正不准确的问题：
+
+**检查 1：文件覆盖完整性**
+- [ ] 已评审所有指定文件
+- [ ] 无法评审的文件已说明原因
+
+**检查 2：输出格式完整性**
+- [ ] 每个问题都有 [严重性][维度] 标签
+- [ ] CRITICAL/HIGH 有：问题描述、详细分析、当前代码、建议修复
+- [ ] MEDIUM/LOW 有：问题描述、简要说明
+
+**检查 3：分类准确性**
+- [ ] 严重性符合 SEVERITY_GUIDELINES
+- [ ] 维度符合 REVIEW_DIMENSIONS
+
+**检查 4：严重性合理性**
+对每个 CRITICAL/HIGH 问题：
+- [ ] 有明确代码证据证明是 bug？
+- [ ] "会出错"还是"可能出错"？
+- [ ] 代码是否已有防御/错误处理？
+- [ ] bug 修复还是改进建议？
+- [ ] 能写测试用例重现？
+
+**不确定或"否"→ 必须降低严重性！**
+
+**检查 5：Diff 覆盖验证(重要)**
+对每个报告的问题：
+- [ ] 问题所在代码行确实是在 diff 中
+- [ ] 问题确实是本次变更引入的，而非之前就存在
+
+**如果是未变更的代码，必须删除该问题！**
+
+**检查 6：误报风险(重要)**
+- [ ] 问题是真实存在，而非基于假设
+
+**如果是误报的问题，必须剔除！**`;
+    }
+    /**
+     * 构建报告生成阶段
+     */
+    static buildReportGenerationPhase(stats) {
+        return `## 第三阶段：生成最终报告
+
+### 步骤 5：整理评审结果并生成报告
+
+获取 subagent 的评审结果后，按照以下结构生成最终的评审报告，写入 \`CODE_REVIEW.md\`：
+
+${this.buildReportTemplate(stats)}
+
+**重要提示**
+- 请**逐条提取**所有评审问题，不要总结或遗漏任何细节
+- **每个问题必须包含文件路径**（从 subagent 输出中提取）
+- 保持原始的代码示例、分析和修复建议的完整性
+- 如果某个严重性级别下某个维度没有问题，则跳过该小节
+- 确保统计表中的数字与实际问题数量一致`;
+    }
+    /**
+     * 构建报告模板
+     */
+    static buildReportTemplate(stats) {
+        return `\`\`\`markdown
+# 代码评审报告
+
+## 变更概览
+
+- **原始文件总数**: ${stats.fileCount} 个
+- **评审文件数**: X 个（Y% 需要评审）
+- **过滤文件数**: Z 个（W% 无需评审）
+- **新增行数**: ${stats.addedLines} 行
+- **删除行数**: ${stats.deletedLines} 行
+- **总变更量**: ${stats.totalChanges} 行
+
+本次变更主要涉及 [简要概括主要变更内容]。
+
+---
+
+## 📋 问题统计
+
+| 严重性 | 正确性 | 安全性 | 性能 | 可维护性 | 架构设计 | 最佳实践 | 合计 |
+|--------|--------|--------|------|----------|----------|----------|------|
+| 🚨 CRITICAL | X | X | X | X | X | X | **X** |
+| ⚠️ HIGH | X | X | X | X | X | X | **X** |
+| 🔶 MEDIUM | X | X | X | X | X | X | **X** |
+| 🔷 LOW | X | X | X | X | X | X | **X** |
+| **合计** | **X** | **X** | **X** | **X** | **X** | **X** | **X** |
+
+---
+
+## 🚨 严重问题 (CRITICAL)
+
+[如果有CRITICAL级别问题，按维度组织]
+
+### 🐛 正确性
+
+#### [问题编号]. [问题标题]
+**文件**: [文件路径]
+
+**问题描述**：[简要说明]
+
+**影响范围**：[如果影响调用方，说明影响]
+
+**详细分析**：
+- [详细分析内容]
+
+**当前代码**：
+\\\`\\\`\\\`language
+[有问题的代码]
+\\\`\\\`\\\`
+
+**建议修复**：
+\\\`\\\`\\\`language
+[修复后的代码]
+\\\`\\\`\\\`
+
+---
+
+[其他维度的CRITICAL问题，格式相同]
+
+---
+
+## ⚠️ 高风险问题 (HIGH)
+
+[按维度组织HIGH级别问题， 格式与CRITICAL相同]
+
+---
+
+## 🔶 中等风险问题 (MEDIUM)
+
+[按维度组织MEDIUM级别问题， 格式与CRITICAL相同]
+
+---
+
+## 🔷 低风险问题 (LOW)
+
+[按维度组织LOW级别问题， 格式与CRITICAL相同]
+
+---
+
+## 📈 整体代码质量评估
+
+### 优点
+[列出代码的优点]
+
+### 需要改进的方面
+[列出需要改进的方面]
+\`\`\``;
+    }
+    /**
+     * 构建质量检查阶段
+     */
+    static buildQualityCheckPhase(projectRoot, options) {
+        let content = `## 第四阶段：质量检查
+
+### 步骤 6：评审结果完整性检查
+
+生成报告后，请进行以下检查：
+
+#### 1. 数量一致性检查
+- 统计表中的总问题数 = 报告中实际列出的问题数
+- 每个维度的计数 = 该维度下实际列出的问题数
+- 每个严重性级别的计数 = 该级别下实际列出的问题数
+
+#### 2. 结构完整性检查
+- 确认报告按 **严重性 > 维度** 两级结构组织
+- 确认每个问题都有完整的字段：
+  - 问题描述
+  - 影响范围（如适用）
+  - 详细分析
+  - 当前代码
+  - 建议修复
+
+#### 3. 内容完整性检查
+- 随机抽查 2-3 个问题，对比原始 subagent 输出
+- 确认代码示例、分析细节没有被简化或遗漏
+- 确认技术术语、文件路径等关键信息准确无误
+
+**如果发现问题**：
+- 如果数量不一致：重新检查 subagent 输出，补充遗漏的问题
+- 如果结构不正确：按照模板重新组织
+- 如果内容被简化：恢复完整的原始内容
+
+**检查通过后**：
+- 输出消息："✅ 评审报告已生成并通过质量检查"
+- 提供报告的关键统计信息`;
+        if (options?.notice) {
+            content += `
+
+---
+
+### 步骤 7：生成完成标志文件
+
+**重要**：为了方便 CI/CD 流水线集成，请在项目根目录创建标志文件：
+
+使用 **Write** 工具创建文件 \`${projectRoot}/.review-completed\`，内容如下：
+
+\`\`\`json
+{
+  "status": "completed",
+  "timestamp": "<当前时间戳>",
+  "report": "CODE_REVIEW.md"
+}
+\`\`\`
+
+**注意**：
+- 只有在评审报告 \`CODE_REVIEW.md\` 成功创建且质量检查通过后才创建此标志文件
+- 时间戳格式：ISO 8601（例如：2025-01-15T10:30:00Z）
+- 此文件用于 CI/CD 流水线检测评审是否完成`;
+        }
+        return content;
+    }
+    /**
+     * 构建调用行动
+     */
+    static buildCallToAction() {
+        return `## 开始执行
+
+现在请按照上述流程开始执行，首先进行文件过滤分析。`;
+    }
+    /**
+     * 构建分析阶段（大型变更）
+     */
+    static buildAnalysisPhase(projectRoot, gitDiffStatCommand) {
+        return `#### 步骤 1：执行 git diff --stat
+
+请执行以下命令获取所有变更文件：
+
+\`\`\`bash
+cd ${projectRoot}
+${gitDiffStatCommand}
+\`\`\`
+
+**重要提示：不要获取完整 diff！**
+- 只需要分析 \`git diff --stat\` 的输出
+- 这将列出所有变更的文件及其变更行数
+
+---
+
+#### 步骤 2：智能过滤文件
+
+**目标**：从变更文件中识别出真正需要代码评审的文件。
+
+${this.buildFileFilteringRules()}
+
+${this.buildFileFilteringOutputFormat()}
+
+**重要**：
+- 必须列出所有变更文件，要么在"需要评审"列表，要么在"已过滤"列表
+- 每个被过滤的文件都要简要说明原因
+- 统计数据要准确
+
+---
+
+#### 步骤 3：判断评审规模
+
+基于**过滤后的文件**（即"需要评审的文件"列表），判断是否需要分批：
+
+- 如果需要评审的文件 ≤ ${REVIEW_THRESHOLDS.LARGE_FILE_THRESHOLD} 个 且 变更量 ≤ ${REVIEW_THRESHOLDS.LARGE_CHANGE_THRESHOLD} 行
+  → 记录"无需分批，可以一次性评审"，跳转到阶段二（单批次评审）
+- 如果需要评审的文件≤ ${REVIEW_THRESHOLDS.LARGE_FILE_THRESHOLD} 个 且 变更量 ≤ ${REVIEW_THRESHOLDS.LARGE_CHANGE_THRESHOLD} 行， 但是存在较大的单文件变更(比如超过500行)，可视情况分批次
+
+- 否则 → 继续步骤 4 制定分批策略
+
+---
+
+#### 步骤 4：制定分批策略（如需分批）
+
+基于"需要评审的文件"列表，制定分批方案：
+
+**分批原则**：
+- 按功能模块、文件类型、业务关联性分组
+- 每批不超过 ${REVIEW_THRESHOLDS.MAX_FILES_PER_BATCH} 个文件
+- 每批不超过 ${REVIEW_THRESHOLDS.MAX_CHANGES_PER_BATCH} 行变更
+- 相关文件放在同一批（如 Service 和其对应的 Test）
+- 在每批次文件数和变更函数不超过限制的前提下，尽量减少批次
+
+**输出格式**：
+
+\`\`\`markdown
+## 分批方案
+
+### 批次 1：[批次描述，如 "用户认证模块"]
+- 文件数：8 个
+- 预计变更行数：650 行
+- 文件列表：
+  - src/services/UserService.java
+  - src/services/AuthService.java
+  - src/utils/jwt.ts
+...
+
+### 批次 2：[批次描述]
+...
+
+### 批次 N：[批次描述]
+...
+\`\`\`
+
+---
+
+#### 步骤 5：确认完整性
+
+检查分批方案（如果分批）或单批次方案：
+- [ ] "需要评审的文件"列表中的每个文件都已分配到某个批次
+- [ ] 没有文件被遗漏
+- [ ] 没有文件被重复分配
+
+如果检查通过，更新 TodoWrite，标记步骤1-5为完成，添加后续评审任务，继续阶段二。
+
+**重要**：此阶段只做分析和规划，不开始评审代码。`;
+    }
+    /**
+     * 构建分批评审阶段
+     */
+    static buildBatchReviewPhase(projectRoot, options) {
+        return `### 阶段二：批次评审（完成分批后执行）
+
+分批方案确定后，更新 **TodoWrite** 任务列表：
+
+\`\`\`
+1. [completed] 分析变更文件
+2. [completed] AI 智能文件过滤
+3. [completed] AI 智能制定分批次评审策略
+4. [completed] 复核分批次评审策略
+5. 评审批次 1/N
+6. 评审批次 2/N
+...
+N+6. 汇总所有批次评审结果
+N+7. 生成最终评审报告
+\`\`\`
+
+对于每个批次：
+
+1. 标记任务为 **in_progress**
+2. 使用 **Task tool** 调用 **code-reviewer** subagent
+3. **验证返回结果**：
+   - 如果返回"未发现需要关注的问题"，记录该批次无问题
+   - 如果返回了问题列表，快速检查格式是否正确
+4. 标记为 **completed**，继续下一批次
+
+**Task tool 调用示例**：
+
+\`\`\`typescript
+{
+  subagent_type: 'code-reviewer',
+  description: '代码评审 - 批次 1/N: [批次描述]',
+  prompt: \`请按照你的标准评审流程进行全面代码评审（批次 1/N）。
+
+## 批次信息
+
+- 批次描述：[描述，如 "用户认证模块"]
+- 文件数：X 个
+- 预计变更行数：Y 行
+
+## 需要评审的文件列表
+
+- file1.ts
+- file2.ts
+- file3.ts
+...
+
+## 获取变更内容
+
+请执行以下命令获取本批次的代码变更：
+
+\\\`\\\`\\\`bash
+cd ${projectRoot}
+git diff ${GitUtils.buildComparisonString(options, 'command')} -- file1.ts file2.ts file3.ts
+\\\`\\\`\\\`
+
+（将 file1.ts file2.ts file3.ts 替换为实际的批次文件列表）
+
+${this.buildEnhancedRulesSection(projectRoot, options)}
+
+## 结果检查
+
+在返回结果开头请注明：**批次 1/N - [批次描述]**
+
+返回结果前必须自检，剔除误报的问题，修正不准确的问题：
+
+**检查 1：文件覆盖完整性**
+- [ ] 已评审所有指定文件
+- [ ] 无法评审的文件已说明原因
+
+**检查 2：输出格式完整性**
+- [ ] 每个问题都有 [严重性][维度] 标签
+- [ ] CRITICAL/HIGH 有：问题描述、详细分析、当前代码、建议修复
+- [ ] MEDIUM/LOW 有：问题描述、简要说明
+
+**检查 3：分类准确性**
+- [ ] 严重性符合 SEVERITY_GUIDELINES
+- [ ] 维度符合 REVIEW_DIMENSIONS
+
+**检查 4：严重性合理性**
+对每个 CRITICAL/HIGH 问题：
+- [ ] 有明确代码证据证明是 bug？
+- [ ] "会出错"还是"可能出错"？
+- [ ] 代码是否已有防御/错误处理？
+- [ ] bug 修复还是改进建议？
+- [ ] 能写测试用例重现？
+
+**不确定或"否"→ 必须降低严重性！**
+
+**检查 5：Diff 覆盖验证(重要)**
+对每个报告的问题：
+- [ ] 问题所在代码行确实是在 diff 中
+- [ ] 问题确实是本次变更引入的，而非之前就存在
+
+**如果是未变更的代码，必须删除该问题！**
+
+**检查 6：误报风险(重要)**
+- [ ] 问题是真实存在，而非基于假设
+
+**如果是误报的问题，必须剔除！**\`
+}
+\`\`\`
+
+**关键点**：
+- 不要在主 task 中获取完整 diff，只提供 git diff 命令
+- code-reviewer subagent 会自己执行命令并获取 diff
+- code-reviewer subagent 会自己读取所有相关文件
+- code-reviewer subagent 会自己进行深度代码评审并自检
+- code-reviewer subagent 会返回评审结果`;
+    }
+    /**
+     * 构建汇总阶段
+     */
+    static buildSummaryPhase(stats, projectRoot, options) {
+        let content = `### 阶段三：汇总报告（所有批次完成后）
+
+所有批次评审完成后，执行以下步骤生成最终报告：
+
+#### 步骤 1：收集所有评审结果
+
+1. 标记汇总任务为 **in_progress**
+2. 从所有 code-reviewer subagent 的返回结果中，**逐条提取**所有问题
+3. 每个问题保留完整信息（不修改、不删减）：
+   - 文件路径
+   - [严重性][维度] 标签
+   - 问题标题
+   - 问题描述
+   - 影响范围
+   - 详细分析
+   - 当前代码（完整上下文）
+   - 建议修复（完整代码）
+
+**关键要求**：
+- 必须收集**所有问题**，不得遗漏任何一个
+- 必须保留**所有细节**，包括代码块、修复建议等
+- 如果某个subagent返回"未发现需要关注的问题"，则该批次无问题
+- 不要对问题进行总结或简化
+
+---
+
+#### 步骤 2：复核评审问题
+
+**要求**：逐一复核评审问题，核对检查项
+**目标**：移除误报的问题，降低误报率
+
+**检查 1：Diff 覆盖验证**
+对每个报告的问题：
+- 问题所在代码行确实是在 diff 中
+- 问题确实是本次变更引入的，而非之前就存在
+
+**检查 2：误报风险**
+对每个报告的问题：
+- [ ] 问题是真实存在，而非基于假设
+
+#### 步骤 3：按维度和严重性重新组织
+
+将收集到的所有问题按照以下结构重新组织：
+
+\`\`\`
+CRITICAL 问题列表
+├── 正确性
+│   ├── 问题1（完整信息）
+│   └── 问题2（完整信息）
+├── 安全性
+│   └── 问题3（完整信息）
+└── 性能
+    └── 问题4（完整信息）
+
+HIGH 问题列表
+├── 正确性
+├── 安全性
+├── 性能
+├── 可维护性
+├── 架构设计
+└── 最佳实践
+
+MEDIUM 问题列表
+...
+
+LOW 问题列表
+...
+\`\`\`
+
+**组织要求**：
+- 第一层：按严重性分类（CRITICAL → HIGH → MEDIUM → LOW）
+- 第二层：按维度分类（正确性、安全性、性能、可维护性、架构设计、最佳实践）
+- 如果某个严重性级别下某个维度没有问题，则不显示该维度
+- 每个问题保留完整的原始信息，不进行任何删减
+
+---
+
+#### 步骤 4：生成最终报告
+
+按照以下格式生成 \`CODE_REVIEW.md\`：
+
+${this.buildReportTemplate(stats)}
+
+**注意**：评审报告中不要体现批次信息，需要整体看上去是一次性完成的评审。
+
+---
+
+**所有步骤完成后**：
+1. 将最终报告写入 \`CODE_REVIEW.md\`
+2. 标记任务为 **completed**
+3. 输出简要总结：
+   \`\`\`
+   ✅ 代码评审完成
+
+   📊 评审统计：
+   - 批次数：X
+   - 原始文件数：Y
+   - 评审文件数：Z
+   - 问题总数：W (CRITICAL: a, HIGH: b, MEDIUM: c, LOW: d)
+
+   📁 报告已生成：CODE_REVIEW.md
+   \`\`\``;
+        if (options?.notice) {
+            content += `
+
+---
+
+#### 步骤 5：生成完成标志文件
+
+**重要**：为了方便 CI/CD 流水线集成，请在项目根目录创建标志文件：
+
+使用 **Write** 工具创建文件 \`${projectRoot}/.review-completed\`，内容如下：
+
+\`\`\`json
+{
+  "status": "completed",
+  "timestamp": "<当前时间戳>",
+  "report": "CODE_REVIEW.md"
+}
+\`\`\`
+
+**注意**：
+- 只有在评审报告 \`CODE_REVIEW.md\` 成功创建后才创建此标志文件
+- 时间戳格式：ISO 8601（例如：2025-01-15T10:30:00Z）
+- 此文件用于 CI/CD 流水线检测评审是否完成`;
+        }
+        return content;
+    }
+    /**
+     * 构建大型变更调用行动
+     */
+    static buildLargeChangeCallToAction() {
+        return `## 立即开始
+
+**第一步**：使用 TodoWrite 创建任务列表
+
+**第二步**：执行 git diff --stat 命令获取所有变更文件
+
+**第三步**：智能过滤文件，识别需要评审的文件
+
+**第四步**：基于过滤后的文件制定分批策略（如需要）
+
+**重要**：此阶段只做分析、过滤和规划，不开始评审代码！
+**重要**：此阶段后请继续执行阶段二和阶段三，不要中断，不要询问是否继续！
+
+请开始执行！`;
+    }
+}
+// ============================================================================
+// 评审策略选择器
+// ============================================================================
+/**
+ * 评审策略选择器
+ */
+class ReviewStrategySelector {
+    /**
+     * 根据统计信息选择评审策略
+     */
+    static selectStrategy(stats, projectRoot, options) {
+        const isLargeChange = this.isLargeChange(stats);
+        if (isLargeChange) {
+            const gitDiffStatCommand = GitUtils.buildDiffStatCommand(options);
+            return ReviewPromptBuilder.buildLargeChangePrompt(stats, projectRoot, gitDiffStatCommand, options);
+        }
+        else {
+            const gitDiffCommand = GitUtils.buildDiffCommand(options);
+            return ReviewPromptBuilder.buildSmallChangePrompt(stats, projectRoot, gitDiffCommand, options);
+        }
+    }
+    /**
+     * 判断是否为大型变更
+     */
+    static isLargeChange(stats) {
+        return stats.fileCount > REVIEW_THRESHOLDS.LARGE_FILE_THRESHOLD ||
+            stats.totalChanges > REVIEW_THRESHOLDS.LARGE_CHANGE_THRESHOLD;
+    }
+}
+// ============================================================================
+// 命令导出
+// ============================================================================
+export const reviewCommand = {
+    name: 'review',
+    description: 'Performs code review with multiple modes: default (uncommitted changes), commit comparison, or branch comparison.',
+    kind: CommandKind.BUILT_IN,
+    action: async (context, args) => {
+        // 验证服务可用性
+        if (!context.services.config) {
+            return {
+                type: 'message',
+                messageType: 'error',
+                content: 'Configuration not available.',
+            };
+        }
+        if (!context.services.git) {
+            return {
+                type: 'message',
+                messageType: 'error',
+                content: 'Git service not available. This command requires a git repository.',
+            };
+        }
+        const projectRoot = context.services.config.getProjectRoot();
+        if (!projectRoot) {
+            return {
+                type: 'message',
+                messageType: 'error',
+                content: 'Project root not available.',
+            };
+        }
+        try {
+            // 解析参数并获取统计信息
+            const options = ReviewArgumentParser.parse(args);
+            const stats = await GitStatsAnalyzer.getDiffStats(projectRoot, options);
+            // 检查是否有变更
+            if (stats.fileCount === 0) {
+                return {
+                    type: 'message',
+                    messageType: 'info',
+                    content: 'No differences found to review.',
+                };
+            }
+            // 根据变更规模选择评审策略
+            const reviewPrompt = ReviewStrategySelector.selectStrategy(stats, projectRoot, options);
+            return {
+                type: 'submit_prompt',
+                content: reviewPrompt,
+            };
+        }
+        catch (error) {
+            const errorMessage = error instanceof ReviewCommandError
+                ? error.message
+                : `Error during review: ${error instanceof Error ? error.message : String(error)}`;
+            return {
+                type: 'message',
+                messageType: 'error',
+                content: errorMessage,
+            };
+        }
+    },
+};
+//# sourceMappingURL=reviewCommand.js.map