job51-gitlab-cr-node-jt-1 2.4.6 → 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

@@ -9,19 +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` 作为基准计算
-   - **只报告当前 diff 块内的问题**：问题的行号必须在 `[New Start, New Start + New Count - 1]` 范围内
+3. **解析行号**：返回文件中变更代码的**绝对行号**（即变更后文件中的行号）
+   - 行号定义：行号是变更后文件中的绝对行号（从 1 开始计数）
+   - 例如：如果新增代码在变更后文件的第 43 行，则 `new_line = 43`
+   - **如何确定行号**：读取变更后文件（基于 `New Path`），找到新增代码在文件中的实际行号
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
 
@@ -58,5 +53,5 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 - 无问题时省略 `### 🔴 严重问题` 标题，但 `<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

@@ -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;
   }
 
   /**
@@ -591,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} 个问题的行号信息`);
@@ -609,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
@@ -621,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;
         }