job51-gitlab-cr-node-jt-1 2.7.6 → 2.7.9

package/.claude/rules/code-review-rules.md

@@ -45,6 +45,17 @@
        >   - 如果项目现有接口多数没有显式参数判空逻辑，**不得建议添加**
        >   - 参数校验形式应参考同文件/同模块其他接口的写法，保持一致性
        >   - **仅在校验风格与同一文件中其他接口明显不一致时**，才需要提示
+   > - **⚠️ 老代码修改审查原则**（关键规则）：
+       >   - **审查范围限制**：对于已有代码的修改（不是新增），**只针对修改的具体内容进行评审**
+       >   - **禁止建议周边逻辑改动**：**不得报告整体逻辑或相关逻辑的修改优化建议**（如"建议重构整个方法"、"建议优化周边调用链"等）
+       >   - **原因**：
+           >     - 贸然修改原有逻辑会引入风险，可能破坏已验证的稳定功能
+           >     - 周边逻辑的改动不在当前需求点范围内，可能不会经过有效测试
+           >     - 原有代码已经过开发自测和 QA 测试验证，应保持最小改动原则
+       >   - **正确做法**：
+           >     - 只审查变更行本身是否有运行时异常风险、安全问题等严重缺陷
+           >     - 不建议对方法的整体逻辑、调用链、数据流等进行优化
+           >     - 即使周边逻辑看起来可以优化，也**保持沉默**，不在本次 CR 中报告
 
 0. **Diff 数据结构与上下文读取规则**：
 > - **临时文件格式说明**：
@@ -288,6 +299,33 @@
 
 18. **输出格式**：严格参照 SKILL.md 模板，无某类问题时省略对应部分；
     > - **必须输出 `<LINE_INFO>` 标签**：包含所有问题的行号信息
+    > - **⚠️ 标签位置要求（最高优先级）**：
+        >   - **`<LINE_INFO>` 标签必须放置在 `</REPORT>` 标签之后**
+        >   - **禁止**将 `<LINE_INFO>` 包含在 `<REPORT>...</REPORT>` 标签内部
+        >   - **禁止**将 `<LINE_INFO>` 穿插在报告内容中
+        >   - **正确格式示例**：
+            ```markdown
+            <REPORT>
+            ## 🤖 AI 代码审查结果
+            
+            ### 🔴 严重问题
+            
+            **问题 1**：...
+            </REPORT>
+            
+            <LINE_INFO>[{"new_path":"...","new_line":10}]
+            </LINE_INFO>
+            ```
+        >   - **错误格式示例**（禁止）：
+            ```markdown
+            <REPORT>
+            ## 🤖 AI 代码审查结果
+            
+            **问题 1**：...
+            <LINE_INFO>[{"new_path":"...","new_line":10}]
+            </LINE_INFO>
+            </REPORT>
+            ```
     > - **LINE_INFO 输出约束**（关键规则）：
         >   - **仅有存在严重问题时，才允许在 LINE_INFO 中输出行号数据**
         >   - **无严重问题时，LINE_INFO 必须输出空数组 `[]`**
@@ -322,6 +360,7 @@
         >   - **禁止**在报告中输出错误代码示例（已经有行号定位了，不需要重复）
 
 19. **误报防控检查清单**（报告前必须确认）：
+    > - [ ] **⚠️ 再次查阅代码确认（最高优先级）**：报告任何问题前，必须再次使用 Read 工具查阅相关代码文件。查阅范围包括：**代码所在方法、方法调用的上下游链路、相关依赖类/接口**。仔细确认代码逻辑，验证问题是否真实存在。禁止凭记忆或第一次分析就报告问题。
     > - [ ] **已安全处理验证**：代码是否已有判空/异常处理/边界检查？如是，省略该问题
     > - [ ] **逻辑一致性验证**：问题描述是否自相矛盾？（如"有判空但说未判空"、"逻辑正确但报告为问题"）
     > - [ ] 已追踪读取相关方法实现，确认可能返回危险值

package/.claude/skills/simple-code-review/SKILL.md

@@ -54,6 +54,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
    - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
    - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 18 条
    - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
+   - **⚠️ 关键格式要求**：`<LINE_INFO>` 标签必须放置在 `</REPORT>` 标签之后，**禁止**出现在 `</REPORT>` 之前或穿插在报告内容中
 
 ## 输出模板
 

package/index.js

@@ -170,9 +170,277 @@ class GitLabCodeReviewer {
     // 结束审查统计
     this.metrics.endReview();
 
+    // ========== 新增：生成并发布汇总报告 ==========
+    try {
+      debugLog('开始生成汇总报告...');
+
+      // 收集所有审查报告内容
+      const allReportsText = this.collectAllReviewReports(results);
+
+      // 调用 AI 生成汇总报告
+      const summaryReport = await this.generateSummaryReportWithAI(allReportsText);
+
+      debugLog(`汇总报告生成完成，长度：${summaryReport.length} 字符`);
+
+      // 发布汇总报告到 GitLab
+      await this.postSummaryCommentToGitLab(projectId, mergeRequestIid, summaryReport);
+      debugLog('汇总报告已成功发布到 GitLab MR');
+    } catch (error) {
+      console.error('汇总报告生成或发布失败:', error.message);
+      // 不抛出错误，确保主流程评论已发布的成功
+    }
+
     return results;
   }
 
+  /**
+   * 收集并拼接所有审查报告内容
+   * @param {Array} results 所有审查结果数组
+   * @returns {string} 拼接后的完整报告文本，包含文件和 diff 块标识
+   */
+  collectAllReviewReports(results) {
+    let allReportsText = '';
+
+    // 遍历所有审查结果
+    for (let i = 0; i < results.length; i++) {
+      const result = results[i];
+      const filePath = result.diff_info.new_path || result.diff_info.old_path;
+      const blockIndex = result.block_index;
+      const reportContent = result.review_result?.reportContent || '';
+
+      // 添加文件和 diff 块标识
+      allReportsText += `\n\n========== 审查报告 #${i + 1} ==========`;
+      allReportsText += `\n文件：${filePath}`;
+      allReportsText += `\nDiff 块索引：#${blockIndex}`;
+      allReportsText += `\n\n${reportContent}`;
+    }
+
+    return allReportsText;
+  }
+
+  /**
+   * 调用 Claude AI 生成汇总报告
+   * @param {string} allReportsText 所有审查报告拼接后的文本
+   * @returns {Promise<string>} AI 生成的汇总报告（Markdown 格式）
+   */
+  async generateSummaryReportWithAI(allReportsText) {
+    debugLog('开始调用 Claude 生成汇总报告');
+
+    // 构造 Prompt
+    const prompt = this.buildSummaryPrompt(allReportsText);
+
+    // 创建临时文件存储 prompt 内容
+    const tmpFileName = `summary_prompt_${Date.now()}.txt`;
+    fs.writeFileSync(tmpFileName, prompt, 'utf-8');
+
+    try {
+      // 调用 Claude CLI 命令
+      const summaryReport = await this.callClaudeForSummary(tmpFileName);
+
+      // 提取 <SUMMARY_REPORT> 标签内容（如果有）
+      const reportMatch = summaryReport.match(/<SUMMARY_REPORT>(.*?)<\/SUMMARY_REPORT>/s);
+      if (reportMatch) {
+        return reportMatch[1].trim();
+      }
+
+      // 如果没有标签，返回完整内容
+      return summaryReport.trim();
+
+    } finally {
+      // 删除临时文件
+      try {
+        if (fs.existsSync(tmpFileName)) {
+          fs.unlinkSync(tmpFileName);
+        }
+      } catch (cleanupError) {
+        console.error('清理临时文件失败:', cleanupError.message);
+      }
+    }
+  }
+
+  /**
+   * 构造汇总报告生成的 Prompt
+   * @param {string} allReportsText 所有审查报告文本
+   * @returns {string} 完整的 Prompt 内容
+   */
+  buildSummaryPrompt(allReportsText) {
+    const timestamp = new Date().toISOString().replace('T', ' ').substring(0, 19);
+
+    const prompt = `你是一个专业的代码审查汇总助手。请根据以下所有审查报告内容，生成一份汇总报告。
+
+**要求**：
+1. **仅输出一份汇总报告**，必须以 \`<SUMMARY_REPORT>\` 开始，以 \`</SUMMARY_REPORT>\` 结束
+2. **不得输出任何解释、问候或额外文本**
+3. **汇总报告必须包含**：
+   - 审查概览：文件数量、diff 块数量、问题总数、严重问题数、警告数
+   - 无问题文件列表：列出所有没有问题的文件路径（完整路径）
+   - 严重问题汇总：按文件分组，完整展示每个问题的描述、位置、修改建议
+   - 警告汇总：按文件分组，完整展示每个问题的描述、位置、修改建议
+   - 审查详情：使用 \`<details>\` 折叠标签隐藏所有完整报告内容
+   - 建议：针对问题的处理建议
+4. **统计数据准确**：根据提供的报告内容准确统计文件数、问题数
+5. **完整展示问题**：每个问题都要完整展示描述、位置、修改建议，不要简化
+6. **即使无问题也要生成报告**：如果所有文件都没有问题，生成一个简短的汇总报告说明审查完成且无问题
+
+**输出格式模板**：
+
+<SUMMARY_REPORT>
+## 🤖 AI 代码审查汇总报告
+
+**生成时间**: ${timestamp}
+**审查范围**: [文件数量]个文件，[diff块数量]个变更块
+**发现问题**: [总数]个，其中严重问题[数量]个
+
+---
+
+### 📊 审查概览
+
+- ✅ 无问题文件：[数量]个（详见下方列表）
+- ⚠️ 有问题文件：[数量]个
+- 🔴 严重问题：[数量]个
+- 🟡 一般警告：[数量]个
+
+---
+
+### ✅ 无问题文件列表
+
+以下文件审查完成，未发现问题：
+
+1. \`[文件路径1]\`
+2. \`[文件路径2]\`
+...
+
+---
+
+### 🔴 严重问题汇总
+
+以下文件包含严重问题，需要优先处理：
+
+#### 1. [文件路径] ([问题数量]个问题)
+
+**问题 1**：[完整问题描述]
+- **位置**：Diff 块 #[块索引]，[行号]
+- **修改建议**：[完整修改建议，包含示例代码]
+
+**问题 2**：...
+
+---
+
+### 🟡 一般警告汇总
+
+以下文件包含警告级别问题：
+
+#### 1. [文件路径] ([警告数量]个警告)
+
+**问题 1**：[完整问题描述]
+- **位置**：Diff 块 #[块索引]，[行号]
+- **修改建议**：[完整修改建议]
+
+...
+
+---
+
+### 📝 审查详情
+
+<details>
+<summary>点击展开完整审查报告</summary>
+
+#### 文件：[文件路径1]
+
+**Diff 块 #0**
+
+[完整报告内容]
+
+...
+
+</details>
+
+---
+
+### 💡 建议
+
+1. **优先处理严重问题**：这些问题可能导致运行时异常或数据错误
+2. **测试验证**：建议对修改后的代码进行充分的单元测试和集成测试
+3. **代码规范**：部分警告涉及代码风格，建议参考项目规范进行统一
+
+---
+
+*本报告由 AI 自动生成，建议结合人工审查确认问题准确性*
+</SUMMARY_REPORT>
+
+**提供的审查报告内容**：
+
+${allReportsText}
+
+**最后强调**：必须严格按照上述模板格式输出，以 \`<SUMMARY_REPORT>\` 开始，以 \`</SUMMARY_REPORT>\` 结束，不得输出其他任何文本！`;
+
+    return prompt;
+  }
+
+  /**
+   * 调用 Claude CLI 命令生成汇总报告
+   * @param {string} promptFile Prompt 临时文件路径
+   * @returns {Promise<string>} Claude 返回的汇总报告内容
+   */
+  async callClaudeForSummary(promptFile) {
+    const maxRetries = 5;
+    let attempt = 0;
+
+    while (attempt < maxRetries) {
+      attempt++;
+      debugLog(`调用 Claude 生成汇总报告，尝试 ${attempt}/${maxRetries}`);
+
+      try {
+        // 使用 spawn 执行 Claude CLI 命令
+        const claudeProcess = spawn('claude', ['--tools', 'default', '-p', '--', promptFile], {
+          cwd: process.cwd(),
+          env: process.env,
+          stdio: ['ignore', 'pipe', 'pipe']
+        });
+
+        let stdout = '';
+        let stderr = '';
+
+        claudeProcess.stdout.on('data', (data) => {
+          stdout += data.toString();
+        });
+
+        claudeProcess.stderr.on('data', (data) => {
+          stderr += data.toString();
+        });
+
+        // 等待进程完成
+        await new Promise((resolve, reject) => {
+          claudeProcess.on('close', (code) => {
+            if (code === 0) {
+              resolve();
+            } else {
+              reject(new Error(`Claude process exited with code ${code}: ${stderr}`));
+            }
+          });
+
+          claudeProcess.on('error', (error) => {
+            reject(error);
+          });
+        });
+
+        // 验证返回内容是否包含 SUMMARY_REPORT 标签
+        if (stdout.includes('<SUMMARY_REPORT>')) {
+          debugLog('汇总报告生成成功');
+          return stdout;
+        } else {
+          debugLog(`返回内容不符合预期，缺少 <SUMMARY_REPORT> 标签，重试...`);
+        }
+
+      } catch (error) {
+        debugLog(`调用 Claude 失败：${error.message}，重试...`);
+      }
+    }
+
+    // 如果所有重试都失败，抛出错误
+    throw new Error('生成汇总报告失败，已达到最大重试次数');
+  }
+
   /**
    * 使用Claude对单个diff文件进行代码审核
    * @param {string} filePath 临时文件路径
@@ -722,6 +990,38 @@ class GitLabCodeReviewer {
       { method: 'POST', data: payload }
     );
   }
+
+  /**
+   * 发布汇总报告到 GitLab MR
+   * @param {number} projectId GitLab 项目 ID
+   * @param {number} mergeRequestIid 合并请求 IID
+   * @param {string} summaryReport Markdown 格式的汇总报告
+   * @returns {Promise<void>}
+   */
+  async postSummaryCommentToGitLab(projectId, mergeRequestIid, summaryReport) {
+    try {
+      debugLog(`开始发布汇总报告到 MR ${mergeRequestIid}`);
+
+      // 构造评论 Payload
+      const discussionData = {
+        body: summaryReport
+      };
+
+      // 调用 GitLab API 发布一般讨论评论
+      // 使用已有的 gitlabClient 实例
+      await this.gitlabClient.callGitLabAPI(
+        `/projects/${projectId}/merge_requests/${mergeRequestIid}/discussions`,
+        { method: 'POST', data: discussionData }
+      );
+
+      debugLog('汇总报告已成功发布');
+      this.metrics.recordCommentPublished();
+
+    } catch (error) {
+      console.error('发布汇总报告失败:', error.message);
+      // 不抛出错误，允许主流程继续（确保已发布的单个评论不受影响）
+    }
+  }
 }
 
 // 主函数

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "2.7.6",
+  "version": "2.7.9",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {