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

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

@@ -193,21 +193,20 @@
             >     1. 从 `@@` 之后开始编号：第 1 行、第 2 行、第 3 行...
             >     2. 绝对行号 = 编号 + new_start - 1
             >     3. 对于 `@@ -0,0 +1,N @@`（新增文件）：绝对行号 = 编号
-        >   - **⚠️ 重要：忽略所有元数据行**（临时文件开头的 `# File Information`、`# New Path:` 等以 `# ` 开头的行都**不编号**，直接跳过）
+        >   - **⚠️ 重要：忽略所有元数据行**（临时文件末尾的 `# 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）
             + [代码行 2]                   <-- + 开头，行号 = 30（29+1）← 问题在这，new_line=30
             + [代码行 3]                   <-- + 开头，行号 = 31（30+1）← 问题在这，new_line=31
             [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
+            ...（diff 内容结束）
+            # File Information (AI 请忽略：这些是元数据，不计入行号)
+            # New Path: b/src/Test.java    <-- 元数据在末尾，不影响计数
+            # New Start: 29
             ```
         >   - **示例 1：新增文件** `@@ -0,0 +1,58 @@`（new_start=1）
             ```

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

@@ -7,11 +7,11 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 1. **读取 diff 文件**：使用 Read 工具读取文件：$ARGUMENTS
 
-2. **解析文件信息**：从 `# File Information` 部分提取
+2. **解析文件信息**：从文件末尾的 `# File Information` 部分提取
    - `Block Index`: 当前块索引号
    - `New Start`: 当前 diff 块在变更后文件中的起始行号（仅供参考）
-   - `New Count`: 当前 diff 块的行数
    - `New Path` / `Old Path`: 文件路径（去掉 `a/` 或 `b/` 前缀）
+   - **注意**：元数据在文件末尾，不影响行号计数
 
 3. **解析行号**：**基于 diff 块内容逐行计数，计算绝对行号**
    - **⚠️ 重要**：行号计算规则详见 @.claude/rules/code-review-rules.md 第 13 条
@@ -26,20 +26,19 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
        1. 从 `@@` 之后开始编号：第 1 行、第 2 行、第 3 行...
        2. 绝对行号 = 编号 + new_start - 1
        3. 对于 `@@ -0,0 +1,N @@`（新增文件）：绝对行号 = 编号
-   - **忽略元数据行**：临时文件开头的 `# File Information`、`# New Path:` 等以 `# ` 开头的行都**不编号**，直接跳过
+   - **忽略元数据行**：文件末尾的 `# 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）
      + [代码行 2]                   <-- + 开头，行号 = 30（29+1）
      + [代码行 3]                   <-- + 开头，行号 = 31（30+1）
      [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
+     ...（diff 内容结束）
+     # File Information (AI 请忽略：这些是元数据，不计入行号)   <-- 元数据在末尾，不影响计数
+     # New Path: b/src/Test.java
+     # New Start: 29
      ```
    - **⚠️ 强制验证步骤**（输出前必须执行）：
      1. 从 diff 头提取 `new_start` 和 `new_count`，计算行号范围：`[new_start, new_start + new_count - 1]`

package/index.js

@@ -84,16 +84,15 @@ class GitLabCodeReviewer {
         debugLog(`diff 内容：${diffObject.diff}`);
         debugLog(`========== Diff Block ${blockIndex} 结束 ==========`);
 
-        // 构造包含元数据的 diff 内容
-        // 注意：元数据行以 '# ' 开头，AI 会忽略这些行，从 '@@' 之后开始计数
-        const diffContentWithMetadata = `# File Information
+        // 构造临时文件内容：diff 内容在前，元数据在后
+        // 这样 AI 无法将元数据行误计数到行号中
+        const diffContentWithMetadata = `${diffObject.diff}
+
+# File Information (AI 请忽略：这些是元数据，不计入行号)
 # 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}`;
+# 行号计算：从 @@ 之后开始计数，@@ 行和所有#开头的行都不计数`;
 
         // 将diff内容写入临时文件
         fs.writeFileSync(tmpFileName, diffContentWithMetadata);