job51-gitlab-cr-node-jt-1 2.6.2 → 2.6.4

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

@@ -247,6 +247,15 @@
         >      - **正确做法**：放弃从完整文件读取的行号，重新基于 diff 块内容逐行计数
         >      - 示例：diff 块 `@@ -0,0 +1,56 @@` 范围是 [1, 56]，如果你在完整文件中定位到第 60 行 → **该行不在 diff 块内，不能用**
         >      - **原因**：GitLab API 只能评论 diff 块范围内的行
+        >   5. **⚠️ 临时文件元数据行处理**（关键）：
+        >      - 临时文件格式包含 8 行元数据（7 行信息 + 1 行`@@`），这些都**不编号**
+        >      - **必须从 `@@` 之后的第一行才开始编号为 1**
+        >      - 验证方法：检查你的第一个行号是否等于 `new_start`。如果不是，说明你把元数据行算进去了
+        >      - 示例：`@@ -0,0 +1,56 @@` 的第一个新增行（如 `+package`）的行号必须是 **1**
+        >      - 错误示例：如果你返回的行号是 9、10、43 等，说明你把 8 行元数据算进去了，**必须重新计数**
+        >   6. **输出前最终验证**：
+        >      - 对于每个问题的行号 X，问自己："X - new_start + 1 是否等于我在 diff 块中从 `@@` 之后开始数的位置？"
+        >      - 如果不等，说明计算错误，必须重新从 `@@` 之后逐行计数
         >   - 示例：`@@ -0,0 +1,56 @@` 的 diff 块，行号范围是 [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

@@ -46,6 +46,10 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      2. **逐个检查**每个问题的行号是否在该范围内
      3. **如果任何行号超出范围，必须重新基于 diff 块计数**
      4. **禁止**直接使用从完整文件读取的行号
+     5. **验证计算过程**：在输出前，必须在心中默念验证：
+        - "这个行号 X 是否在 [new_start, new_start + new_count - 1] 范围内？"
+        - "如果 X 超出范围，说明我计数错误，必须从 `@@` 行之后重新逐行编号"
+        - "临时文件的前 8 行（7 行元数据 + 1 行`@@`）都不能编号，从第 9 行（`@@`之后的第一行）才开始编号为 1"
    - **示例**：
      - `@@ -0,0 +1,56 @@` 的行号范围是 [1, 56]，**不能返回 57、60 等超出范围的值**
      - `@@ -167,7 +167,6 @@` 的行号范围是 [167, 172]，**不能返回 <167 或 >172 的值**
@@ -67,17 +71,28 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
 
 ## 输出模板
 
-**⚠️ 强制格式要求**：
+**⚠️ 强制格式要求（违反会导致解析失败）**：
 
-1. **问题格式**：每个问题必须以 `**问题 N**：` 开头（N 为阿拉伯数字 1,2,3...）
+1. **报告开头必须包含 `## 🤖 AI 代码审查结果`**：
+   - **正确**：`<REPORT>\n## 🤖 AI 代码审查结果\n\n### 🔴 严重问题`
+   - **错误**：`<REPORT>\n### 🔴 严重问题`（缺少主标题）
+   - **错误**：`<REPORT>\n## 代码审查结果`（缺少 emoji）
+
+2. **问题格式**：每个问题必须以 `**问题 N**：` 开头（N 为阿拉伯数字 1,2,3...）
    - **正确**：`**问题 1**：`、`**问题 2**：`
    - **错误**：`1. **xxx**`、`### 问题 1`、`**问题一**：`、`**问题 1**: xxx（行 43）`
 
-2. **行号标注**：**禁止**在问题描述中写"（行 X）"，行号只能通过 `<LINE_INFO>` 标签提供
+3. **每个问题块只能有一个 `**文件及行号**：` 行**：
+   - **正确**：每个问题块中 `**文件及行号**：` 只出现一次
+   - **错误**：同一个问题块中出现两次 `**文件及行号**：` 行
+
+4. **行号标注**：**禁止**在问题描述中写"（行 X）"，行号只能通过 `<LINE_INFO>` 标签提供
+
+5. **LINE_INFO**：必须与问题一一对应，第 N 个元素对应问题 N 的行号
 
-3. **LINE_INFO**：必须与问题一一对应，第 N 个元素对应问题 N 的行号
+6. **禁止输出错误代码示例**：已经有行号定位了，不需要重复输出错误代码
 
-4. **禁止输出错误代码示例**：已经有行号定位了，不需要重复输出错误代码
+7. **必须输出 `<LINE_INFO>` 标签**：即使无问题也要输出空数组 `<LINE_INFO>[]</LINE_INFO>`
 
 <REPORT>
 ## 🤖 AI 代码审查结果

package/docs/GITLAB_CR_NODE_TECHNICAL_DOCS.md

@@ -1,7 +1,7 @@
 # GitLab Code Review AI Tool 技术文档
 
 **项目名称**: job51-gitlab-cr-node  
-**当前版本**: 2.5.8  
+**当前版本**: 2.6.4  
 **作者**: tao.jing  
 **最后更新**: 2026-04-16  
 **项目地址**: https://gitdev.51job.com/51jobweb/ai-agent  
@@ -25,6 +25,44 @@
 
 ---
 
+## 版本历史
+
+### v2.6.4 (2026-04-16)
+
+**Bug 修复**:
+- **修复 GitLab API line_code 错误**：在构建 diff 评论 payload 时，添加 `position_type` 字段（`new` 或 `old`），确保 GitLab API 能够正确识别行号类型
+  - 问题现象：`400 Bad request - Note {:line_code=>["不能为空字符", "must be a valid line code"]}`
+  - 修复方案：`targetLine` 对象中显式指定 `position_type: 'new'`（新增行）或 `position_type: 'old'`（删除行）
+  - 修复文件：`index.js:590-629`
+
+### v2.6.3 (2026-04-16)
+
+**输出格式修复**:
+- **修复 AI 输出格式不标准问题**：在 SKILL.md 中增加强制格式要求，确保 AI 输出符合解析期望
+  - 增加 `## 🤖 AI 代码审查结果` 主标题的强制要求
+  - 禁止同一个问题块中出现多次 `**文件及行号**：` 行
+  - 修复文件：`.claude/skills/simple-code-review/SKILL.md`
+
+### v2.6.2 (2026-04-16)
+
+**行号计算修复**:
+- **修复临时文件元数据行计数问题**：明确临时文件中 7 行元数据 + 1 行 `@@` 头都必须忽略，从 `@@` 之后的第一行才开始编号
+  - 修复文件：`.claude/skills/simple-code-review/SKILL.md`、`.claude/rules/code-review-rules.md`
+
+### v2.6.1 (2026-04-16)
+
+**行号验证增强**:
+- **增加行号范围验证**：在 AI 输出前强制验证行号是否在 `[new_start, new_start + new_count - 1]` 范围内
+  - 修复文件：`.claude/skills/simple-code-review/SKILL.md`、`.claude/rules/code-review-rules.md`
+
+### v2.6.0 (2026-04-16)
+
+**初始版本**:
+- 新增行号计算规范，要求基于 diff 块内容逐行计数计算绝对行号
+- 新增跨 diff 块上下文分析，避免误报已删除代码的问题
+
+---
+
 ## 1. 项目概述
 
 ### 1.1 项目定位

package/index.js

@@ -600,7 +600,8 @@ ${diffObject.diff}`;
           } else {
             targetLine = {
               old_path: problemInfo.old_path || diff_info.old_path,
-              old_line: problemInfo.old_line
+              old_line: problemInfo.old_line,
+              position_type: 'old'
             };
           }
         } else if (problemInfo.new_line) {
@@ -613,7 +614,8 @@ ${diffObject.diff}`;
             } else {
               targetLine = {
                 old_path: problemInfo.new_path || diff_info.old_path,
-                old_line: problemInfo.new_line
+                old_line: problemInfo.new_line,
+                position_type: 'old'
               };
             }
           } else if (newCount === 0) {
@@ -621,9 +623,11 @@ ${diffObject.diff}`;
           } else if (problemInfo.new_line < newStart || problemInfo.new_line > newEnd) {
             skipReason = `new_line=${problemInfo.new_line} 超出 diff 块范围 [${newStart}, ${newEnd}]`;
           } else {
+            // 新增文件/新增行：使用 position_type: 'new'，GitLab API 需要这个字段
             targetLine = {
               new_path: problemInfo.new_path || diff_info.new_path,
-              new_line: problemInfo.new_line
+              new_line: problemInfo.new_line,
+              position_type: 'new'
             };
           }
         }