job51-gitlab-cr-node-jt-1 2.4.5 → 2.4.7

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

@@ -181,18 +181,15 @@
 12. 最终输出必须以 `<REPORT>` 开始，以 `</REPORT>` 结束；
 
 13. **行号计算规范**：
-    > - **Diff 头信息格式**：`@@ -84,9 +84,11 @@`
-        >   - `-84,9`：旧文件从第 84 行开始，共 9 行
-        >   - `+84,11`：新文件从第 84 行开始，共 11 行
-    > - **行号计算**（绝对行号，非相对偏移）：
-        >   - 初始值 = `new_start`（从 diff 头解析）
-        >   - 空格开头（上下文）：当前行号 = 计数器值，计数器 +1
-        >   - `-` 开头（删除）：跳过，计数器不变
-        >   - `+` 开头（新增）：当前行号 = 计数器值，计数器 +1
-    > - **示例**：`@@ -0,0 +1,49 @@`（新增文件）
-        >   - 第 1 行 `+`：行号 = 1
-        >   - 第 2 行 `+`：行号 = 2
-    > - **关键**：问题行号必须准确指向变更后代码的实际行号（绝对行号）
+    > - **行号定义**：行号是变更后文件中的**绝对行号**（从 1 开始计数）
+    > - **如何确定行号**：
+        >   - 读取变更后文件（基于 `New Path`）
+        >   - 找到新增代码在文件中的实际行号
+        >   - 例如：如果新增代码在变更后文件的第 43 行，则 `new_line = 43`
+    > - **示例**：
+        >   - 新增文件 `@@ -0,0 +1,49 @@`：第 1 行 `+` 的行号 = 1，第 2 行 `+` 的行号 = 2
+        >   - 已有文件 `@@ -42,3 +43,5 @@`：新增代码在文件第 43 行，则 `new_line = 43`
+    > - **关键**：问题行号必须准确指向变更后代码在文件中的实际行号（绝对行号）
 
 14. `<REPORT>` 标签示例中的问题块数量仅用于展示格式，实际数量由 review 结果决定；
 

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

@@ -1,4 +1,4 @@
----
+---
 name: simple-code-review
 description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_INFO
 ---
@@ -9,18 +9,14 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 2. **解析文件信息**：从 `=== File Information ===` 部分提取
    - `Block Index`: 当前块索引号
-   - `New Start`: 当前 diff 块在变更后文件中的起始行号（**关键：用于计算绝对行号**）
+   - `New Start`: 当前 diff 块在变更后文件中的起始行号（仅供参考）
    - `New Count`: 当前 diff 块的行数
    - `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
 
-3. **解析行号**：使用 `New Start` 作为基准，遍历 diff 内容计算每行的绝对行号
-   - 行号计算规则：
-     - **基准行号** = `New Start`（从 File Information 中获取）
-     - 遍历 diff 内容，只计数 `+` 开头（新增）和空格开头（上下文）的行
-     - 第 1 行 `+` 或空格：行号 = `New Start`
-     - 第 2 行 `+` 或空格：行号 = `New Start + 1`
-     - 以此类推，`-` 开头的删除行不计入行号
-   - **重要**：行号是变更后文件中的绝对行号，必须使用 `New Start` 作为基准计算
+3. **解析行号**：返回文件中变更代码的**绝对行号**（即变更后文件中的行号）
+   - 行号定义：行号是变更后文件中的绝对行号（从 1 开始计数）
+   - 例如：如果新增代码在变更后文件的第 43 行，则 `new_line = 43`
+   - **如何确定行号**：读取变更后文件（基于 `New Path`），找到新增代码在文件中的实际行号
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
 
@@ -29,7 +25,7 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 6. **输出结果**：
    - 审查结果放入 `<REPORT>` 标签
    - 行号信息放入 `<LINE_INFO>` 标签（必须输出）
-   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 19 条
+   - LINE_INFO 格式参考 @.claude/rules/code-review-rules.md 第 18 条
    - **问题和行号对应**：`<LINE_INFO>` 中的第 N 个元素对应第 N 个问题的行号
 
 ## 输出模板
@@ -39,16 +35,13 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 ### 🔴 严重问题
 
----
 **问题 1**：[问题描述]<br/>
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
----
 
 **问题 2**：[问题描述]<br/>
 **文件及行号**：[文件路径：行号]<br/>
 **修改建议**：[正确示例代码或说明]
----
 
 </REPORT>
 
@@ -57,7 +50,8 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 </LINE_INFO>
 
 **说明**：
-- 无问题时省略对应标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
-- 每个问题用 `---` 分割线分隔，方便切分
+- 无问题时省略 `### 🔴 严重问题` 标题，但 `<LINE_INFO>` 必须输出（空数组：`[]`）
+- 每个问题以 `**问题 N**：` 开头，方便切分
 - **`<LINE_INFO>` 数组中的元素顺序必须与问题顺序一致**：第 1 个元素对应问题 1 的行号，第 2 个元素对应问题 2 的行号，以此类推
-- 行号计算：使用 `New Start` 作为基准，第 1 行 `+` 或空格的行号 = `New Start`，第 2 行 = `New Start + 1`，依此类推
+- 行号：返回变更后文件中的绝对行号（从 1 开始计数），例如文件第 43 行则 `new_line = 43`
+- **无严重问题时，LINE_INFO 必须输出空数组 `[]`**

package/index.js

@@ -102,7 +102,7 @@ class GitLabCodeReviewer {
         // 无严重问题 → 直接返回标准空格式
         if (!hasSeriousProblem) {
           debugLog(`【决策】报告无严重问题，返回标准空格式`);
-          return { reportContent: claudeResult, lineInfo: lineInfoContent };
+          return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
         // 有严重问题 → 检查标题是否符合要求
@@ -297,7 +297,7 @@ ${diffObject.diff}`;
         // 无严重问题 → 直接返回标准空格式
         if (!hasSeriousProblem) {
           debugLog(`【决策】报告无严重问题，返回标准空格式`);
-          return { reportContent: claudeResult, lineInfo: lineInfoContent };
+          return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
         // 有严重问题 → 检查标题是否符合要求
@@ -383,7 +383,7 @@ ${diffObject.diff}`;
     const regex = /(?=@@\s-\d+(?:,\d+)?\s\+\d+(?:,\d+)?\s@@)/g;
     const diffBlocks = diffObj.diff.split(regex);
     // 过滤掉空块并提取行号信息
-    return diffBlocks
+    const result = diffBlocks
       .filter(block => block.trim() !== '')
       .map(block => {
         // 解析diff头信息 @@ -old_start,old_count +new_start,new_count @@
@@ -407,6 +407,8 @@ ${diffObject.diff}`;
           new_count = headerMatch[4] ? parseInt(headerMatch[4], 10) : 1;
         }
 
+        debugLog(`解析 diff 块：file=${diffObj.new_path || diffObj.old_path}, headerMatch=${headerMatch ? headerMatch[0] : 'N/A'}, new_start=${new_start}, new_count=${new_count}`);
+
         return {
           diff: block,
           new_path: diffObj.new_path,
@@ -427,6 +429,9 @@ ${diffObject.diff}`;
           }
         };
       });
+
+    debugLog(`文件 ${diffObj.new_path || diffObj.old_path} 拆分为 ${result.length} 个 diff 块`);
+    return result;
   }
 
   /**
@@ -514,29 +519,32 @@ ${diffObject.diff}`;
    */
   extractSingleProblemReport(fullReport, problemIndex) {
     const lines = fullReport.split('\n');
-    // 使用 --- 分割线来切分问题
+    // 使用 **问题 N**： 模式来切分问题
     const problemBlocks = [];
     let currentBlock = [];
-    let inBlock = false;
+    let currentBlockIndex = 0;
 
     for (const line of lines) {
-      if (line.trim() === '---') {
-        if (inBlock) {
-          // 块结束
-          if (currentBlock.length > 0) {
-            problemBlocks.push(currentBlock.join('\n'));
-            currentBlock = [];
-          }
-          inBlock = false;
-        } else {
-          // 块开始
-          inBlock = true;
+      // 检查是否是新的问题开始（如 **问题 1**：, **问题 2**：等）
+      const problemMatch = line.match(/\*\*问题\s*(\d+)\*\*/);
+      if (problemMatch) {
+        // 保存上一个问题块
+        if (currentBlock.length > 0 && currentBlockIndex > 0) {
+          problemBlocks.push(currentBlock.join('\n'));
+          currentBlock = [];
         }
-      } else if (inBlock) {
+        currentBlockIndex = parseInt(problemMatch[1]);
+        currentBlock.push(line);
+      } else {
         currentBlock.push(line);
       }
     }
 
+    // 保存最后一个问题块
+    if (currentBlock.length > 0 && currentBlockIndex > 0) {
+      problemBlocks.push(currentBlock.join('\n'));
+    }
+
     // 返回第 problemIndex 个问题（索引从 1 开始）
     if (problemIndex >= 1 && problemIndex <= problemBlocks.length) {
       return problemBlocks[problemIndex - 1];
@@ -588,11 +596,10 @@ ${diffObject.diff}`;
       const startSha = versionInfo.start_commit_sha;
       debugLog(`获取到版本信息 - base: ${baseSha}, head: ${headSha}, start: ${startSha}`);
 
-      // 解析 diff 头信息，计算有效行号范围
-      const newStart = diff_info.new_start || 1;
-      const newCount = diff_info.new_count || 1;
-      const maxValidLine = newStart + newCount - 1;
-      debugLog(`Diff 块行号范围：[${newStart}, ${maxValidLine}]，共 ${newCount} 行`);
+      // 解析 diff 头信息（用于日志记录）
+      const newStart = diff_info.line_info?.new_start || 1;
+      const newCount = diff_info.line_info?.new_count || 1;
+      debugLog(`Diff 块信息：new_start=${newStart}, new_count=${newCount}`);
 
       const allLineInfo = this.parseAllLineInfoFromReviewResult(lineInfoTag);
       debugLog(`解析到 ${allLineInfo.length} 个问题的行号信息`);
@@ -606,10 +613,8 @@ ${diffObject.diff}`;
         debugLog(`处理第 ${i + 1}/${allLineInfo.length} 个问题：文件=${problemInfo.new_path}, 行号=${problemInfo.new_line}`);
         const singleProblemContent = this.extractSingleProblemReport(fullReportContent, i + 1);
 
-        // 行号有效性验证和修正
+        // 构建目标行号
         let targetLine;
-        let originalLine = problemInfo.new_line;
-        let needsGeneralDiscussion = false;
 
         if (problemInfo.old_line && !problemInfo.new_line) {
           // 删除代码，使用 old_line
@@ -618,23 +623,18 @@ ${diffObject.diff}`;
             old_line: problemInfo.old_line
           };
         } else if (problemInfo.new_line) {
-          // 验证新行号是否在有效范围内
-          if (problemInfo.new_line < newStart || problemInfo.new_line > maxValidLine) {
-            debugLog(`⚠️ 行号 ${problemInfo.new_line} 超出有效范围 [${newStart}, ${maxValidLine}]，改用一般讨论`);
-            needsGeneralDiscussion = true;
-          } else {
-            targetLine = {
-              new_path: problemInfo.new_path,
-              new_line: problemInfo.new_line
-            };
-          }
+          // AI 返回的是文件中的绝对行号，直接使用
+          targetLine = {
+            new_path: problemInfo.new_path,
+            new_line: problemInfo.new_line
+          };
         }
 
-        if (needsGeneralDiscussion) {
-          // 行号无效，改用一般讨论
-          debugLog(`第 ${i + 1} 个问题的行号 ${originalLine} 无效，发布为一般讨论`);
+        if (!targetLine) {
+          // 无法解析行号，使用一般讨论
+          debugLog(`第 ${i + 1} 个问题无法解析行号，发布为一般讨论`);
           await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, singleProblemContent);
-          debugLog(`第 ${i + 1} 个问题的评论已发布 (作为一般讨论，行号无效)`);
+          debugLog(`第 ${i + 1} 个问题的评论已发布 (作为一般讨论)`);
           continue;
         }