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

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

@@ -204,31 +204,31 @@
             # New Start: 29                 <-- 元数据行，忽略，直接跳过
             # Diff Content Below
             @@ -28,8 +29,10 @@            <-- @@ 行，不编号！从下一行开始行号=29（new_start）
-             public void method() {       <-- 空格开头，行号 = 29（new_start）
-            +    Object obj = getXxx();   <-- + 开头，行号 = 30（29+1）← 问题在这，new_line=30
-            +    obj.toString();          <-- + 开头，行号 = 31（30+1）← 问题在这，new_line=31
-             }                            <-- 空格开头，行号 = 32（31+1）
+            [代码行 1]                     <-- 空格开头，行号 = 29（new_start）
+            + [代码行 2]                   <-- + 开头，行号 = 30（29+1）← 问题在这，new_line=30
+            + [代码行 3]                   <-- + 开头，行号 = 31（30+1）← 问题在这，new_line=31
+            [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
             ```
         >   - **示例 1：新增文件** `@@ -0,0 +1,58 @@`（new_start=1）
             ```
             @@ -0,0 +1,58 @@
-            +package com.job51.dev.bbs.service.impl;    // 行号 = 1（new_start）
-            +                                            // 行号 = 2
-            +import com.job51.dev.bbs.mapper.ForumMapper; // 行号 = 3
+            + [代码行 1]    // 行号 = 1（new_start）
+            + [代码行 2]    // 行号 = 2
+            + [代码行 3]    // 行号 = 3
             ...
-            +public class TestMultiIssuesService {       // 行号 = 17
-            +    public Map<String, Object> getTestData(String forumId, String userId) { // 行号 = 27
-            +        Object forum = forumMapper.selectById(forumId); // 行号 = 28
-            +        String forumName = forum.toString();            // 行号 = 30 ← 问题在这，new_line = 30
-            +        Object firstPost = posts.get(0);                // 行号 = 38 ← 问题在这，new_line = 38
+            + [代码行 17]   // 行号 = 17
+            + [代码行 27]   // 行号 = 27
+            + [代码行 28]   // 行号 = 28
+            + [代码行 30]   // 行号 = 30 ← 问题在这，new_line = 30
+            + [代码行 38]   // 行号 = 38 ← 问题在这，new_line = 38
             ```
         >   - **示例 2：已有文件修改** `@@ -42,3 +43,5 @@`（new_start=43）
             ```
             @@ -42,3 +43,5 @@
-             public void method() {  // 行号 = 43（new_start）
-            +    Object obj = service.getData();  // 行号 = 44（43+1）
-            +    obj.toString();                  // 行号 = 45（44+1）← 问题在这，new_line = 45
-             }                                   // 行号 = 46（45+1）
+            [代码行 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 值**：

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

@@ -36,10 +36,10 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      # New Start: 29                 <-- 元数据行，忽略，不编号
      # Diff Content Below
      @@ -28,8 +29,10 @@            <-- @@ 行，不编号！从下一行才开始计数
-      public void method() {       <-- 空格开头，行号 = 29（new_start）
-     +    Object obj = getXxx();   <-- + 开头，行号 = 30（29+1）
-     +    obj.toString();          <-- + 开头，行号 = 31（30+1）
-      }                            <-- 空格开头，行号 = 32（31+1）
+     [代码行 1]                     <-- 空格开头，行号 = 29（new_start）
+     + [代码行 2]                   <-- + 开头，行号 = 30（29+1）
+     + [代码行 3]                   <-- + 开头，行号 = 31（30+1）
+     [代码行 4]                     <-- 空格开头，行号 = 32（31+1）
      ```
    - **⚠️ 强制验证步骤**（输出前必须执行）：
      1. 从 diff 头提取 `new_start` 和 `new_count`，计算行号范围：`[new_start, new_start + new_count - 1]`
@@ -97,8 +97,8 @@ description: 代码审查技能，审查变更代码并输出 REPORT 和 LINE_IN
      \`\`\`
      **修改建议**：添加空值检查
      \`\`\`java
-     if (forum != null) {
-         String forumName = forum.toString();
+     if ([变量] != null) {
+         [方法调用];
      }
      \`\`\`
      \`\`\`

package/docs/GITLAB_CR_NODE_TECHNICAL_DOCS.md

@@ -1,7 +1,7 @@
 # GitLab Code Review AI Tool 技术文档
 
 **项目名称**: job51-gitlab-cr-node  
-**当前版本**: 2.6.8  
+**当前版本**: 2.6.10  
 **作者**: tao.jing  
 **最后更新**: 2026-04-16  
 **项目地址**: https://gitdev.51job.com/51jobweb/ai-agent  
@@ -27,16 +27,26 @@
 
 ## 版本历史
 
-### v2.6.8 (2026-04-16)
+### v2.6.10 (2026-04-16)
+
+**优化**:
+- **添加临时文件格式调试日志**：输出临时文件前 10 行预览，用于确认元数据格式是否正确
+  - 可以在日志中看到临时文件的实际格式，确认 AI 是否正确忽略元数据行
+  - 帮助诊断行号计算偏移问题
+
+### v2.6.9 (2026-04-16)
 
 **Bug 修复**:
-- **修复 SKILL.md 中临时文件格式描述不一致问题**：将 `=== File Information ===` 更新为 `# File Information`
-  - 问题现象：v2.6.7 修复行号偏移后，AI 仍不能正确计算行号
-  - 根本原因：SKILL.md 第 2 行仍描述从 `=== File Information ===` 提取信息，但临时文件格式已改为 `# File Information`
-  - 修复方案：更新 SKILL.md 中的格式描述，使其与临时文件实际格式一致
-  - 修复文件：`.claude/skills/simple-code-review/SKILL.md`
+- **优化行号计算规则说明**：改用更直观的「直接从 new_start 开始递增」方法
+  - 问题现象：AI 仍不能正确计算行号，偏移 8 行
+  - 根本原因：之前的规则使用「先编号再转换」方法（编号 + new_start - 1），容易出错
+  - 修复方案：
+    - 改用「直接从 new_start 开始递增」方法：从 @@ 之后第一行开始，行号=new_start，每往下一行行号 +1
+    - 更新示例中的行号标注，直接使用「行号 = X」而非「编号=X → 绝对行号=...」
+    - 移除容易引起误解的「编号」概念，直接用行号递增加示例
+  - 修复文件：`.claude/skills/simple-code-review/SKILL.md`、`.claude/rules/code-review-rules.md`
 
-### v2.6.7 (2026-04-16)
+### v2.6.8 (2026-04-16)
 
 **Bug 修复**:
 - **修复行号计算偏移问题**：修改临时文件格式，元数据行改为以 `# ` 开头，方便 AI 识别并忽略

package/index.js

@@ -98,6 +98,10 @@ ${diffObject.diff}`;
         // 将diff内容写入临时文件
         fs.writeFileSync(tmpFileName, diffContentWithMetadata);
 
+        // 输出临时文件前 10 行用于调试（确认元数据格式）
+        const tmpFileLines = diffContentWithMetadata.split('\n').slice(0, 10).join('\n');
+        debugLog(`临时文件前 10 行预览:\n${tmpFileLines}`);
+
         // 审核当前块（传入临时的文件而不是直接的diff内容）
         const blockStartTime = Date.now();
         const review_result = await this.reviewDiffWithClaudeUsingFile(tmpFileName);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "2.6.9",
+  "version": "2.7.0",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {