job51-gitlab-cr-node-jt-1 2.7.0 → 2.7.2

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

@@ -184,35 +184,26 @@
     > - **行号定义**：行号是**变更后文件中的绝对行号**（从 1 开始计数，即该行在完整文件中的实际行号）
     > - **⚠️ 关键规则：如何计算绝对行号**（强制执行，推荐方法 A）：
         >   - **方法 A：直接从 new_start 开始递增**（推荐，不易出错）
-            >     1. 找到 `@@ -old_start,old_count +new_start,new_count @@` 行
-            >     2. 从这一行**之后**的第一行开始，行号 = `new_start`
+            >     1. 从元数据中读取 `New Start` 值
+            >     2. 从第 1 行代码开始，行号 = `New Start`
             >     3. 每往下一行（`+` 开头或空格开头），行号 +1
             >     4. `-` 开头的删除行**不计数**，行号不增加
-            >     5. 示例：`@@ -0,0 +1,58 @@` 之后的第一行行号=1，第二行行号=2，以此类推
-        >   - **方法 B：先编号再转换**
-            >     1. 从 `@@` 之后开始编号：第 1 行、第 2 行、第 3 行...
-            >     2. 绝对行号 = 编号 + new_start - 1
-            >     3. 对于 `@@ -0,0 +1,N @@`（新增文件）：绝对行号 = 编号
-        >   - **⚠️ 重要：忽略所有元数据行**（临时文件开头的 `# File Information`、`# New Path:` 等以 `# ` 开头的行都**不编号**，直接跳过）
-        >   - **⚠️ 关键**：`@@ ... @@` 这一行本身**不编号**，从它**之后**的第一行才开始编号
+            >     5. 示例：`New Start: 1` 时，第 1 行代码行号=1，第 2 行行号=2，以此类推
+        >   - **⚠️ 重要：忽略所有元数据行**（临时文件末尾的 `# File Information` 等以 `# ` 开头的行都**不编号**，直接跳过）
     > - **⚠️ 行号计算示例**（必读）：
         >   - **示例 0：临时文件完整格式**（理解计数起点）
             ```
-            # File Information (AI 请忽略这些元数据行，从 @@ 之后开始计数)
-            # New Path: b/src/Test.java     <-- 元数据行，忽略，直接跳过
-            # Block Index: 0                <-- 元数据行，忽略，直接跳过
-            # New Start: 29                 <-- 元数据行，忽略，直接跳过
-            # Diff Content Below
-            @@ -28,8 +29,10 @@            <-- @@ 行，不编号！从下一行开始行号=29（new_start）
-            [代码行 1]                     <-- 空格开头，行号 = 29（new_start）
+            [代码行 1]                     <-- 空格开头，行号 = 29（New Start=29）
             + [代码行 2]                   <-- + 开头，行号 = 30（29+1）← 问题在这，new_line=30
             + [代码行 3]                   <-- + 开头，行号 = 31（30+1）← 问题在这，new_line=31
             [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
+            ...（diff 内容结束）
+            # File Information
+            # New Start: 29                <-- 元数据在末尾，提供行号计算起点，不参与计数
             ```
-        >   - **示例 1：新增文件** `@@ -0,0 +1,58 @@`（new_start=1）
+        >   - **示例 1：新增文件** `New Start: 1, New Count: 58`
             ```
-            @@ -0,0 +1,58 @@
-            + [代码行 1]    // 行号 = 1（new_start）
+            + [代码行 1]    // 行号 = 1（New Start）
             + [代码行 2]    // 行号 = 2
             + [代码行 3]    // 行号 = 3
             ...
@@ -222,32 +213,31 @@
             + [代码行 30]   // 行号 = 30 ← 问题在这，new_line = 30
             + [代码行 38]   // 行号 = 38 ← 问题在这，new_line = 38
             ```
-        >   - **示例 2：已有文件修改** `@@ -42,3 +43,5 @@`（new_start=43）
+        >   - **示例 2：已有文件修改** `New Start: 43`
             ```
-            @@ -42,3 +43,5 @@
-            [代码行 1]    // 行号 = 43（new_start）
+            [代码行 1]    // 行号 = 43（New Start）
             + [代码行 2]  // 行号 = 44（43+1）
             + [代码行 3]  // 行号 = 45（44+1）← 问题在这，new_line = 45
             [代码行 4]    // 行号 = 46（45+1）
             ```
     > - **⚠️ 验证方法**（强制执行，输出前必须检查）：
-        >   1. **查看 diff 头的 new_start 和 new_count 值**：
-            >      - `@@ -0,0 +1,N @@` → 行号范围 = [1, N]
-            >      - `@@ -old_start,old_count +new_start,new_count @@` → 行号范围 = [new_start, new_start + new_count - 1]
+        >   1. **从元数据提取 `New Start` 和 `New Count` 值**：
+            >      - `New Start: 1, New Count: N` → 行号范围 = [1, N]
+            >      - `New Start: X, New Count: Y` → 行号范围 = [X, X + Y - 1]
         >   2. **确保所有问题的行号在范围内**
-        >   3. **如果行号超出范围，说明计算错误，必须重新计数**
+        >   3. **如果任何行号超出范围，说明计算错误，必须重新计数**
         >   4. **⚠️ 特别场景：读取完整项目文件后的行号处理**
             >      - 如果你读取了完整的项目文件（通过 Read 工具）来追踪问题位置
             >      - **必须验证：该行是否在 diff 块范围内**
-            >      - 验证公式：`new_start <= 行号 <= new_start + new_count - 1`
+            >      - 验证公式：`New Start <= 行号 <= New Start + New Count - 1`
             >      - **如果行号超出范围，说明你定位的行不在当前 diff 块内**
             >      - **正确做法**：放弃从完整文件读取的行号，重新基于 diff 块内容逐行计数
-            >      - 示例：diff 块 `@@ -0,0 +1,56 @@` 范围是 [1, 56]，如果你在完整文件中定位到第 60 行 → **该行不在 diff 块内，不能用**
+            >      - 示例：`New Start: 1, New Count: 56` 范围是 [1, 56]，如果你在完整文件中定位到第 60 行 → **该行不在 diff 块内，不能用**
             >      - **原因**：GitLab API 只能评论 diff 块范围内的行
         >   5. **输出前最终验证**：
-            >      - 对于每个问题的行号 X，问自己："X - new_start + 1 是否等于我在 diff 块中从 `@@` 之后开始数的位置？"
-            >      - 如果不等，说明计算错误，必须重新从 `@@` 之后逐行计数
-            >      - 示例：`@@ -0,0 +1,56 @@` 的 diff 块，行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
+            >      - 对于每个问题的行号 X，问自己："X - New Start + 1 是否等于我从 diff 块第 1 行开始数的位置？"
+            >      - 如果不等，说明计算错误，必须重新从第 1 行代码逐行计数
+            >      - 示例：`New Start: 1, New Count: 56` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
             >      - 示例：`@@ -167,7 +167,6 @@` 的 diff 块，new_start=167, new_count=6，行号范围是 [167, 172]
     > - **关键**：绝对行号必须通过 diff 块内计数计算，且结果必须在 diff 块范围 [new_start, new_start + new_count - 1] 内
 

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

@@ -7,48 +7,39 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 1. **读取 diff 文件**：使用 Read 工具读取文件：$ARGUMENTS
 
-2. **解析文件信息**：从 `# File Information` 部分提取
-   - `Block Index`: 当前块索引号
-   - `New Start`: 当前 diff 块在变更后文件中的起始行号（仅供参考）
+2. **解析文件信息**：从文件末尾的 `# File Information` 部分提取
+   - `New Start`: 当前 diff 块在变更后文件中的起始行号
    - `New Count`: 当前 diff 块的行数
-   - `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
+   - `New Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
+   - **注意**：元数据在文件末尾，`@@` 行已被移除，不影响行号计数
 
 3. **解析行号**：**基于 diff 块内容逐行计数，计算绝对行号**
-   - **⚠️ 重要**：行号计算规则详见 @.claude/rules/code-review-rules.md 第 13 条
-   - **行号计算方法**（任选一种，推荐方法 A）：
-     - **方法 A：直接从 new_start 开始递增**（推荐，不易出错）
-       1. 找到 `@@ -old_start,old_count +new_start,new_count @@` 行
-       2. 从这一行**之后**的第一行开始，行号 = `new_start`
-       3. 每往下一行（`+` 开头或空格开头），行号 +1
-       4. `-` 开头的删除行**不计数**，行号不增加
-       5. 示例：`@@ -0,0 +1,58 @@` 之后的第一行行号=1，第二行行号=2，以此类推
-     - **方法 B：先编号再转换**
-       1. 从 `@@` 之后开始编号：第 1 行、第 2 行、第 3 行...
-       2. 绝对行号 = 编号 + new_start - 1
-       3. 对于 `@@ -0,0 +1,N @@`（新增文件）：绝对行号 = 编号
-   - **忽略元数据行**：临时文件开头的 `# File Information`、`# New Path:` 等以 `# ` 开头的行都**不编号**，直接跳过
-   - **⚠️ 关键**：`@@ ... @@` 这一行本身也不编号，从它**之后**的第一行才开始编号
+   - **行号计算方法**：
+     1. 从元数据中读取 `New Start` 值
+     2. 从第 1 行代码开始，行号 = `New Start`
+     3. 每往下一行（`+` 开头或空格开头），行号 +1
+     4. `-` 开头的删除行**不计数**，行号不增加
+     5. 示例：`New Start: 1` 时，第 1 行代码行号=1，第 2 行行号=2，以此类推
+   - **忽略元数据行**：文件末尾的 `# File Information` 等以 `# ` 开头的行都**不编号**，直接跳过
    - **⚠️ 临时文件格式示例**（理解计数起点）：
      ```
-     # File Information (AI 请忽略这些元数据行，从 @@ 之后开始计数)
-     # New Path: b/src/Test.java     <-- 元数据行，忽略，不编号
-     # Block Index: 0                <-- 元数据行，忽略，不编号
-     # New Start: 29                 <-- 元数据行，忽略，不编号
-     # Diff Content Below
-     @@ -28,8 +29,10 @@            <-- @@ 行，不编号！从下一行才开始计数
-     [代码行 1]                     <-- 空格开头，行号 = 29（new_start）
+     [代码行 1]                     <-- 空格开头，行号 = 29（New Start=29）
      + [代码行 2]                   <-- + 开头，行号 = 30（29+1）
      + [代码行 3]                   <-- + 开头，行号 = 31（30+1）
      [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
+     ...（diff 内容结束）
+     # File Information
+     # New Path: b/src/Test.java
+     # New Start: 29                <-- 元数据在末尾，提供行号计算起点
      ```
    - **⚠️ 强制验证步骤**（输出前必须执行）：
-     1. 从 diff 头提取 `new_start` 和 `new_count`，计算行号范围：`[new_start, new_start + new_count - 1]`
+     1. 从元数据提取 `New Start` 和 `New Count`，计算行号范围：`[New Start, New Start + New Count - 1]`
      2. **逐个检查**每个问题的行号是否在该范围内
      3. **如果任何行号超出范围，必须重新计数**
      4. **禁止**直接使用从完整文件读取的行号
    - **示例**：
-     - `@@ -0,0 +1,56 @@` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
-     - `@@ -167,7 +167,6 @@` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**
+     - `New Start: 1, New Count: 56` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
+     - `New Start: 167, New Count: 6` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**
 
 4. **读取上下文**：基于 New Path 读取变更后文件，涉及方法调用时追踪读取实现代码
    - **⚠️ 重要**：读取完整文件仅用于理解代码逻辑，**不得**直接使用完整文件中的行号

package/index.js

@@ -84,16 +84,19 @@ class GitLabCodeReviewer {
         debugLog(`diff 内容：${diffObject.diff}`);
         debugLog(`========== Diff Block ${blockIndex} 结束 ==========`);
 
-        // 构造包含元数据的 diff 内容
-        // 注意：元数据行以 '# ' 开头，AI 会忽略这些行，从 '@@' 之后开始计数
-        const diffContentWithMetadata = `# File Information
+        // 解析 diff 内容，移除 @@ 行，将其信息放入元数据
+        // 这样 AI 不会把 @@ 行误计数
+        const diffLines = diffObject.diff.split('\n');
+        const codeLines = diffLines.filter(line => !line.startsWith('@@')).join('\n');
+
+        // 构造临时文件内容：纯代码在前，元数据在后
+        const diffContentWithMetadata = `${codeLines}
+
+# File Information
 # New Path: ${diffObject.new_path || 'N/A'}
-# Old Path: ${diffObject.old_path || 'N/A'}
-# Block Index: ${blockIndex}
 # New Start: ${diffObject.line_info?.new_start || 1}
 # New Count: ${diffObject.line_info?.new_count || 1}
-# Diff Content Below (start counting line numbers from the first line after @@)
-${diffObject.diff}`;
+# 行号计算：从第 1 行代码开始计数，第 1 行行号=NewStart，每往下一行行号 +1，#开头的元数据行不计数`;
 
         // 将diff内容写入临时文件
         fs.writeFileSync(tmpFileName, diffContentWithMetadata);