openmatrix 0.1.99 → 0.2.2

package/dist/orchestrator/ai-reviewer.js

@@ -16,162 +16,162 @@ class AIReviewer {
      * 生成 AI Review 提示词
      */
     buildReviewPrompt(task) {
-        return `# AI Code Review - 验收阶段
-
-## 任务信息
-- Task ID: ${task.id}
-- 标题: ${task.title}
-- 描述: ${task.description}
-
-## 审查范围
-
-请对本次任务的所有代码变更进行完整审查。
-
-## 审查清单
-
-### 1. 代码质量 (Code Quality)
-- [ ] 代码可读性: 命名清晰、结构合理
-- [ ] 函数复杂度: 单一职责、适当长度
-- [ ] 代码复用: 避免重复、抽取公共逻辑
-- [ ] 注释质量: 关键逻辑有注释、复杂代码有说明
-- [ ] 错误处理: 异常情况有处理、错误信息清晰
-
-### 2. 安全性 (Security)
-- [ ] 输入验证: 用户输入有校验
-- [ ] SQL 注入: 数据库查询使用参数化
-- [ ] XSS 攻击: 输出有转义
-- [ ] 敏感数据: 密码/密钥不明文存储
-- [ ] 权限控制: 敏感操作有权限检查
-
-### 3. 性能 (Performance)
-- [ ] 算法效率: 时间复杂度合理
-- [ ] 数据库查询: 避免 N+1、使用索引
-- [ ] 内存使用: 无内存泄漏、大对象及时释放
-- [ ] 缓存策略: 适当使用缓存
-- [ ] 异步处理: 耗时操作使用异步
-
-### 4. 最佳实践 (Best Practices)
-- [ ] 设计模式: 合理使用设计模式
-- [ ] 依赖注入: 避免硬编码依赖
-- [ ] 配置管理: 配置与代码分离
-- [ ] 日志记录: 关键操作有日志
-- [ ] 测试友好: 代码易于测试
-
-### 5. 测试覆盖 (Test Coverage)
-- [ ] 单元测试: 核心逻辑有测试
-- [ ] 边界情况: 边界值有测试
-- [ ] 异常路径: 错误情况有测试
-- [ ] 测试质量: 测试有意义、不是为测试而测试
-
-## 输出格式
-
-在 \`.openmatrix/tasks/${task.id}/artifacts/\` 目录下创建:
-
-### ai-review-report.md
-
-\`\`\`markdown
-# AI Review 报告
-
-## 任务信息
-- Task ID: ${task.id}
-- 审查时间: [当前时间]
-
-## 总体评价
-[✅ 通过 / ⚠️ 需要修改 / ❌ 拒绝]
-
-## 分类评估
-
-### 1. 代码质量
-- 评分: [0-100]
-- 状态: ✅ / ⚠️ / ❌
-- 详情:
-  - [具体评价1]
-  - [具体评价2]
-
-### 2. 安全性
-- 评分: [0-100]
-- 状态: ✅ / ⚠️ / ❌
-- 详情:
-  - [具体评价1]
-  - [具体评价2]
-
-### 3. 性能
-- 评分: [0-100]
-- 状态: ✅ / ⚠️ / ❌
-- 详情:
-  - [具体评价1]
-  - [具体评价2]
-
-### 4. 最佳实践
-- 评分: [0-100]
-- 状态: ✅ / ⚠️ / ❌
-- 详情:
-  - [具体评价1]
-  - [具体评价2]
-
-### 5. 测试覆盖
-- 评分: [0-100]
-- 状态: ✅ / ⚠️ / ❌
-- 详情:
-  - [具体评价1]
-  - [具体评价2]
-
-## 问题列表
-
-### 严重 (Critical) - 必须修复
-1. [文件:行号] 问题描述
-   - 建议: 修复建议
-
-### 重要 (Major) - 强烈建议修复
-1. [文件:行号] 问题描述
-   - 建议: 修复建议
-
-### 次要 (Minor) - 可选优化
-1. [文件:行号] 问题描述
-
-### 建议 (Suggestion) - 改进建议
-1. [改进建议]
-
-## 总结
-[审查总结和最终建议]
-\`\`\`
-
-## 最终输出
-
-根据审查结果，输出以下之一:
-
-**通过:**
-\`\`\`
-AI_REVIEW_APPROVED
-\`\`\`
-
-**需要修改:**
-\`\`\`
-AI_REVIEW_NEEDS_CHANGES
-需要修复的问题:
-1. [Critical/Major 问题描述]
-\`\`\`
-
-**拒绝:**
-\`\`\`
-AI_REVIEW_REJECTED
-拒绝原因:
-1. [严重问题描述]
-\`\`\`
-
-## 审查原则
-
-1. **平衡** - 不要过于严苛，也不要过于宽松
-2. **实用** - 关注真正重要的问题
-3. **建设性** - 提供具体的改进建议
-4. **上下文** - 考虑项目的实际情况和约束
-5. **渐进** - 允许代码渐进式改进，不要求完美
+        return `# AI Code Review - 验收阶段
+
+## 任务信息
+- Task ID: ${task.id}
+- 标题: ${task.title}
+- 描述: ${task.description}
+
+## 审查范围
+
+请对本次任务的所有代码变更进行完整审查。
+
+## 审查清单
+
+### 1. 代码质量 (Code Quality)
+- [ ] 代码可读性: 命名清晰、结构合理
+- [ ] 函数复杂度: 单一职责、适当长度
+- [ ] 代码复用: 避免重复、抽取公共逻辑
+- [ ] 注释质量: 关键逻辑有注释、复杂代码有说明
+- [ ] 错误处理: 异常情况有处理、错误信息清晰
+
+### 2. 安全性 (Security)
+- [ ] 输入验证: 用户输入有校验
+- [ ] SQL 注入: 数据库查询使用参数化
+- [ ] XSS 攻击: 输出有转义
+- [ ] 敏感数据: 密码/密钥不明文存储
+- [ ] 权限控制: 敏感操作有权限检查
+
+### 3. 性能 (Performance)
+- [ ] 算法效率: 时间复杂度合理
+- [ ] 数据库查询: 避免 N+1、使用索引
+- [ ] 内存使用: 无内存泄漏、大对象及时释放
+- [ ] 缓存策略: 适当使用缓存
+- [ ] 异步处理: 耗时操作使用异步
+
+### 4. 最佳实践 (Best Practices)
+- [ ] 设计模式: 合理使用设计模式
+- [ ] 依赖注入: 避免硬编码依赖
+- [ ] 配置管理: 配置与代码分离
+- [ ] 日志记录: 关键操作有日志
+- [ ] 测试友好: 代码易于测试
+
+### 5. 测试覆盖 (Test Coverage)
+- [ ] 单元测试: 核心逻辑有测试
+- [ ] 边界情况: 边界值有测试
+- [ ] 异常路径: 错误情况有测试
+- [ ] 测试质量: 测试有意义、不是为测试而测试
+
+## 输出格式
+
+在 \`.openmatrix/tasks/${task.id}/artifacts/\` 目录下创建:
+
+### ai-review-report.md
+
+\`\`\`markdown
+# AI Review 报告
+
+## 任务信息
+- Task ID: ${task.id}
+- 审查时间: [当前时间]
+
+## 总体评价
+[✅ 通过 / ⚠️ 需要修改 / ❌ 拒绝]
+
+## 分类评估
+
+### 1. 代码质量
+- 评分: [0-100]
+- 状态: ✅ / ⚠️ / ❌
+- 详情:
+  - [具体评价1]
+  - [具体评价2]
+
+### 2. 安全性
+- 评分: [0-100]
+- 状态: ✅ / ⚠️ / ❌
+- 详情:
+  - [具体评价1]
+  - [具体评价2]
+
+### 3. 性能
+- 评分: [0-100]
+- 状态: ✅ / ⚠️ / ❌
+- 详情:
+  - [具体评价1]
+  - [具体评价2]
+
+### 4. 最佳实践
+- 评分: [0-100]
+- 状态: ✅ / ⚠️ / ❌
+- 详情:
+  - [具体评价1]
+  - [具体评价2]
+
+### 5. 测试覆盖
+- 评分: [0-100]
+- 状态: ✅ / ⚠️ / ❌
+- 详情:
+  - [具体评价1]
+  - [具体评价2]
+
+## 问题列表
+
+### 严重 (Critical) - 必须修复
+1. [文件:行号] 问题描述
+   - 建议: 修复建议
+
+### 重要 (Major) - 强烈建议修复
+1. [文件:行号] 问题描述
+   - 建议: 修复建议
+
+### 次要 (Minor) - 可选优化
+1. [文件:行号] 问题描述
+
+### 建议 (Suggestion) - 改进建议
+1. [改进建议]
+
+## 总结
+[审查总结和最终建议]
+\`\`\`
+
+## 最终输出
+
+根据审查结果，输出以下之一:
+
+**通过:**
+\`\`\`
+AI_REVIEW_APPROVED
+\`\`\`
+
+**需要修改:**
+\`\`\`
+AI_REVIEW_NEEDS_CHANGES
+需要修复的问题:
+1. [Critical/Major 问题描述]
+\`\`\`
+
+**拒绝:**
+\`\`\`
+AI_REVIEW_REJECTED
+拒绝原因:
+1. [严重问题描述]
+\`\`\`
+
+## 审查原则
+
+1. **平衡** - 不要过于严苛，也不要过于宽松
+2. **实用** - 关注真正重要的问题
+3. **建设性** - 提供具体的改进建议
+4. **上下文** - 考虑项目的实际情况和约束
+5. **渐进** - 允许代码渐进式改进，不要求完美
 `;
     }
     /**
      * 解析 AI Review 结果
      */
-    parseReviewResult(output) {
+    parseReviewResult(taskId, output) {
         const lines = output.split('\n');
         const issues = [];
         const categories = [];
@@ -249,7 +249,7 @@ AI_REVIEW_REJECTED
         const summaryMatch = output.match(/## 总结\s*([\s\S]*?)(?=```|$)/);
         const summary = summaryMatch ? summaryMatch[1].trim() : '';
         return {
-            taskId: '',
+            taskId,
             overallStatus,
             categories,
             issues,
@@ -263,64 +263,169 @@ AI_REVIEW_REJECTED
      */
     buildAcceptPrompt(task) {
         const reviewPrompt = this.buildReviewPrompt(task);
-        return `# 验收阶段 (Accept Phase)
-
-## 任务信息
-- ID: ${task.id}
-- 标题: ${task.title}
-- 描述: ${task.description}
-
-## 验收流程
-
-### 1. AI Code Review
-执行完整的代码审查:
-
-${reviewPrompt}
-
-### 2. 功能验证
-- 确认功能按需求实现
-- 检查边界情况处理
-- 验证错误处理
-
-### 3. 文档检查
-- README 是否更新
-- API 文档是否完整
-- 注释是否充分
-
-### 4. 最终确认
-- 所有测试通过
-- AI Review 通过
-- 文档已更新
-- 可以合并
-
-## 输出要求
-
-在 \`.openmatrix/tasks/${task.id}/artifacts/\` 目录下创建:
-- \`accept-report.md\` - 验收报告
-- \`ai-review-report.md\` - AI 审查报告
-
-## 结果格式
-
-如果验收通过，输出:
-\`\`\`
-ACCEPT_PASSED
-\`\`\`
-
-如果需要修改，输出:
-\`\`\`
-ACCEPT_NEEDS_MODIFICATION
-修改建议:
-1. [建议]
-2. [建议]
-\`\`\`
-
-如果验收失败，输出:
-\`\`\`
-ACCEPT_FAILED
-失败原因:
-1. [原因]
-\`\`\`
+        return `# 验收阶段 (Accept Phase)
+
+## 任务信息
+- ID: ${task.id}
+- 标题: ${task.title}
+- 描述: ${task.description}
+
+## 验收流程
+
+### 1. AI Code Review
+执行完整的代码审查:
+
+${reviewPrompt}
+
+### 2. 功能验证
+- 确认功能按需求实现
+- 检查边界情况处理
+- 验证错误处理
+
+### 3. 文档检查
+- README 是否更新
+- API 文档是否完整
+- 注释是否充分
+
+### 4. 最终确认
+- 所有测试通过
+- AI Review 通过
+- 文档已更新
+- 可以合并
+
+## 输出要求
+
+在 \`.openmatrix/tasks/${task.id}/artifacts/\` 目录下创建:
+- \`accept-report.md\` - 验收报告
+- \`ai-review-report.md\` - AI 审查报告
+
+## 结果格式
+
+如果验收通过，输出:
+\`\`\`
+ACCEPT_PASSED
+\`\`\`
+
+如果需要修改，输出:
+\`\`\`
+ACCEPT_NEEDS_MODIFICATION
+修改建议:
+1. [建议]
+2. [建议]
+\`\`\`
+
+如果验收失败，输出:
+\`\`\`
+ACCEPT_FAILED
+失败原因:
+1. [原因]
+\`\`\`
 `;
     }
+    /**
+     * 根据 Review 报告生成修复任务
+     *
+     * 对每个 critical/major 问题生成独立的修复任务，
+     * 这些任务会被加入执行队列并自动执行。
+     *
+     * @param originalTask 被审查的原始任务
+     * @param report Review 报告
+     * @param reviewTaskId 审查任务本身的 ID（用于依赖）
+     * @returns 修复任务列表
+     */
+    generateFixTasks(originalTask, report, reviewTaskId) {
+        const fixTasks = [];
+        let counter = 0;
+        // 只对 critical 和 major 级别的问题生成修复任务
+        const actionableIssues = report.issues.filter(issue => issue.severity === 'critical' || issue.severity === 'major');
+        // 按文件分组，减少上下文切换
+        const issuesByFile = new Map();
+        for (const issue of actionableIssues) {
+            const fileKey = issue.file || 'general';
+            if (!issuesByFile.has(fileKey)) {
+                issuesByFile.set(fileKey, []);
+            }
+            issuesByFile.get(fileKey).push(issue);
+        }
+        for (const [file, issues] of issuesByFile) {
+            counter++;
+            const isCritical = issues.some(i => i.severity === 'critical');
+            const severityLabel = isCritical ? '紧急修复' : '修复';
+            // 构建修复描述
+            const issueDetails = issues.map((issue, i) => `${i + 1}. [${issue.severity.toUpperCase()}] ${issue.description}${issue.suggestion ? ` → 建议: ${issue.suggestion}` : ''}`).join('\n');
+            fixTasks.push({
+                taskId: `${originalTask.id}-FIX-${String(counter).padStart(2, '0')}`,
+                title: `${severityLabel}: ${file === 'general' ? '通用问题' : file}`,
+                description: `# 代码修复任务
+
+## 来源
+由 AI Review 自动生成 — 审查任务: ${reviewTaskId}
+
+## 原始任务
+- ID: ${originalTask.id}
+- 标题: ${originalTask.title}
+
+## 需要修复的问题 (${issues.length} 个)
+
+${issueDetails}
+
+## 输出要求
+1. 修复上述所有问题
+2. 确保不引入新的问题
+3. 保持代码风格一致
+4. 完成后输出修复说明
+
+${file !== 'general' ? `\n## 目标文件\n\`${file}\`` : ''}`,
+                priority: isCritical ? 'P0' : 'P1',
+                assignedAgent: 'coder',
+                dependencies: [reviewTaskId],
+                timeout: 120000
+            });
+        }
+        // 修复完成后，生成一个重新验证任务
+        if (fixTasks.length > 0) {
+            fixTasks.push({
+                taskId: `${originalTask.id}-REVERIFY`,
+                title: `重新验证: ${originalTask.title}`,
+                description: `# 重新验证任务
+
+## 背景
+原始任务 ${originalTask.id} 经过 Review 后发现 ${actionableIssues.length} 个问题，已创建 ${fixTasks.length} 个修复任务。
+现在需要验证所有修复是否正确应用。
+
+## 验证清单
+- [ ] 所有修复任务已执行
+- [ ] 原 Review 报告中的 critical/major 问题已解决
+- [ ] 修复没有引入新的问题
+- [ ] 代码可以编译/运行
+- [ ] 测试仍然通过
+
+## 输出
+如果验证通过，输出:
+\`\`\`
+REVERIFY_PASSED
+所有 ${actionableIssues.length} 个问题已修复
+\`\`\`
+
+如果验证失败，输出:
+\`\`\`
+REVERIFY_FAILED
+遗留问题:
+1. [问题描述]
+\`\`\``,
+                priority: 'P1',
+                assignedAgent: 'reviewer',
+                dependencies: [reviewTaskId, ...fixTasks.map(f => f.taskId)],
+                timeout: 60000
+            });
+        }
+        return fixTasks;
+    }
+    /**
+     * 判断 Review 报告是否需要自动修复
+     */
+    needsAutoFix(report) {
+        return report.issues.some(issue => issue.severity === 'critical' || issue.severity === 'major');
+    }
 }
 exports.AIReviewer = AIReviewer;

package/dist/orchestrator/answer-mapper.d.ts

@@ -21,4 +21,5 @@ export declare function extractTasksInputFields(answers: Record<string, string |
     quality?: string;
     mode?: string;
     e2eTests?: boolean;
+    e2eType?: string;
 };

package/dist/orchestrator/answer-mapper.js

@@ -98,11 +98,14 @@ function extractTasksInputFields(answers) {
     if (typeof mode === 'string') {
         result.mode = mode;
     }
-    // e2e_tests → e2eTests 布尔字段
+    // e2e_tests → e2eTests 布尔字段 + e2eType
     const e2e = answers['e2e_tests'] || answers['e2eTests'] || answers['E2E测试'];
     if (e2e !== undefined) {
         const e2eStr = Array.isArray(e2e) ? e2e[0] : e2e;
-        result.e2eTests = e2eStr === 'true' || e2eStr === '启用 E2E 测试' || e2eStr === '是';
+        result.e2eTests = e2eStr === 'functional' || e2eStr === 'visual' || e2eStr === 'true' || e2eStr === '启用 E2E 测试' || e2eStr === '是';
+        if (result.e2eTests && e2eStr === 'visual') {
+            result.e2eType = 'visual';
+        }
     }
     return result;
 }

package/dist/orchestrator/approval-manager.js

@@ -62,9 +62,10 @@ class ApprovalManager {
         };
         await this.stateManager.updateApproval(updatedApproval);
         // 如果是 meeting 类型且被批准，解除任务阻塞
+        // 恢复到 pending 让调度器重新调度，而非直接 in_progress
         if (approval.type === 'meeting' && decision.decision === 'approve') {
             await this.stateManager.updateTask(approval.taskId, {
-                status: 'in_progress',
+                status: 'pending',
                 error: null
             });
         }
@@ -79,18 +80,18 @@ class ApprovalManager {
             taskId,
             title: `🔴 阻塞问题需要解决`,
             description: `任务 ${taskId} 遇到阻塞问题`,
-            content: `
-## 阻塞原因
-
-${blockingReason}
-
-## 影响范围
-
-${impactScope.map(t => `- ${t}`).join('\n')}
-
-## 需要决策
-
-请选择解决方案或提供自定义方案：
+            content: `
+## 阻塞原因
+
+${blockingReason}
+
+## 影响范围
+
+${impactScope.map(t => `- ${t}`).join('\n')}
+
+## 需要决策
+
+请选择解决方案或提供自定义方案：
       `.trim(),
             options: [
                 { key: 'approve', label: '✅ 问题已解决，继续执行' },

package/dist/orchestrator/executor.d.ts

@@ -40,6 +40,7 @@ export declare class OrchestratorExecutor {
     private stateMachine;
     private phaseExecutor;
     private retryManager;
+    private aiReviewer;
     private config;
     private taskTimers;
     constructor(stateManager: StateManager, approvalManager: ApprovalManager, config?: Partial<ExecutorConfig>);
@@ -92,12 +93,20 @@ export declare class OrchestratorExecutor {
     private handleBlockedTasks;
     /**
      * 标记任务完成
+     *
+     * 增强：对于 reviewer 任务，自动解析 Review 报告并生成修复任务
      */
     completeTask(taskId: string, result: {
         success: boolean;
         output?: string;
         error?: string;
-    }): Promise<void>;
+    }): Promise<{
+        createdFixTasks?: string[];
+    }>;
+    /**
+     * 处理 Review 结果：解析报告，如有 critical/major 问题则自动创建修复任务
+     */
+    private processReviewResult;
     /**
      * 获取当前阶段
      */