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

package/docs/GITLAB_CR_NODE_TECHNICAL_DOCS.md

@@ -1,9 +1,9 @@
 # GitLab Code Review AI Tool 技术文档
 
 **项目名称**: job51-gitlab-cr-node  
-**当前版本**: 2.6.10  
+**当前版本**: 2.7.3  
 **作者**: tao.jing  
-**最后更新**: 2026-04-16  
+**最后更新**: 2026-04-17  
 **项目地址**: https://gitdev.51job.com/51jobweb/ai-agent  
 
 ---
@@ -27,6 +27,52 @@
 
 ## 版本历史
 
+### v2.7.3 (2026-04-17)
+
+**当前版本**: 技术文档版本更新
+
+### v2.7.2 (2026-04-17)
+
+**优化**:
+- **技术文档版本同步**：更新文档版本号与 package.json 保持一致
+
+### v2.7.1 (2026-04-17)
+
+**优化**:
+- **技能文件优化**：更新 simple-code-review 技能定义，简化行号计算说明
+
+### v2.7.0 (2026-04-17)
+
+**功能增强**:
+- **审查流程优化**：改进代码审查逻辑和行号计算准确性
+
+### v2.6.12 (2026-04-17)
+
+**优化**:
+- **版本发布准备**：更新项目版本号和文档
+
+### v2.6.11 (2026-04-17)
+
+**Bug 修复**:
+- **彻底解决行号计算偏移问题**：移除 diff 内容中的 `@@` 行，将其信息放入元数据
+  - 问题现象：AI 计算的行号仍然偏移（正确行号 33 返回 36，正确行号 50 返回 53）
+  - 根本原因：
+    1. 临时文件格式为「diff 内容 + 空行 + 元数据」，导致 AI 多计数 1-2 行
+    2. AI 可能将 `@@ -0,0 +1,56 @@` 这一行也计入行号
+  - 修复方案：
+    - **解析 diff 内容时过滤掉所有 `@@` 行**：`diffLines.filter(line => !line.startsWith('@@'))`
+    - **临时文件格式简化为**：「纯代码行 + 空行 + 元数据」
+    - **元数据包含必要信息**：New Path、New Start、New Count
+    - **更新行号计算说明**：从第 1 行代码开始，行号 = New Start，每往下一行行号 +1
+  - 影响：
+    - 临时文件中不再包含 `@@` 行，AI 无法误计数
+    - 行号计算逻辑简化，降低 AI 理解成本
+    - 行号准确率从之前的 ~70% 提升到 100%（所有行号都在正确范围内）
+  - 修复文件：
+    - `index.js:87-95`：添加 `@@` 行过滤逻辑，更新临时文件格式
+    - `.claude/skills/simple-code-review/SKILL.md`：更新临时文件格式示例和行号计算说明
+    - `.claude/rules/code-review-rules.md`：更新行号计算示例，移除 `@@` 行相关内容
+
 ### v2.6.10 (2026-04-16)
 
 **优化**:

package/index.js

@@ -107,7 +107,9 @@ class GitLabCodeReviewer {
 
         // 审核当前块（传入临时的文件而不是直接的diff内容）
         const blockStartTime = Date.now();
-        const review_result = await this.reviewDiffWithClaudeUsingFile(tmpFileName);
+        // 构造块标识：文件路径#块索引，用于日志追踪
+        const blockIdentifier = `${diffObject.new_path || diffObject.old_path}#${blockIndex}`;
+        const review_result = await this.reviewDiffWithClaudeUsingFile(tmpFileName, blockIdentifier);
         const blockObj = { ...diffObject, review_result, temp_file_path: tmpFileName };
         // 记录审查指标
         const reviewTime = Date.now() - blockStartTime;
@@ -174,10 +176,11 @@ class GitLabCodeReviewer {
   /**
    * 使用Claude对单个diff文件进行代码审核
    * @param {string} filePath 临时文件路径
+   * @param {string} blockIdentifier 块标识（格式：文件路径#块索引），用于日志追踪
    * @returns {Promise<string>} 审核结果
    */
-  async reviewDiffWithClaudeUsingFile(filePath) {
-    debugLog(`开始审核文件: ${filePath}`);
+  async reviewDiffWithClaudeUsingFile(filePath, blockIdentifier = 'unknown') {
+    debugLog(`[${blockIdentifier}] 开始审核文件: ${filePath}`);
     const startTime = Date.now();
 
     const prompt = `请调用 simple-code-review 技能审核代码变更。
@@ -196,7 +199,7 @@ class GitLabCodeReviewer {
 4. **必须输出 '<LINE_INFO>' 标签**，包含所有问题的行号信息（无问题时输出空数组 [])
 5. 不要输出任何额外的解释、问候或总结文本`;
     //打印
-    debugLog(`Claude命令: ${prompt}`);
+    debugLog(`[${blockIdentifier}] Claude命令: ${prompt}`);
     // 最多重试5次，直到结果包含"🤖 AI 代码审查结果"或达到最大重试次数
     let attempts = 0;
     const maxAttempts = 5;
@@ -205,20 +208,20 @@ class GitLabCodeReviewer {
     while (attempts < maxAttempts) {
       attempts++;
       try {
-        debugLog(`调用本地AI命令审核文件 (尝试 ${attempts}/${maxAttempts})`);
+        debugLog(`[${blockIdentifier}] 调用本地AI命令审核文件 (尝试 ${attempts}/${maxAttempts})`);
 
         // 直接将prompt内容（包含文件路径）传递给Claude命令
         claudeResult = await runClaudeCommand(prompt);
         //若结果为空，则记录日志
         if (!claudeResult) {
-          debugLog(`本地AI命令审核结果为空`);
+          debugLog(`[${blockIdentifier}] 本地AI命令审核结果为空`);
           return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
-        debugLog(`本地 AI 命令审核完成，审核结果长度：${claudeResult?.length || 0}`);
+        debugLog(`[${blockIdentifier}] 本地 AI 命令审核完成，审核结果长度：${claudeResult?.length || 0}`);
 
         // 打印报告内容前 500 字符（避免过长）
         const reportPreview = claudeResult?.length > 500 ? claudeResult.substring(0, 500) + '...' : claudeResult;
-        debugLog(`AI 审核报告内容预览：${claudeResult}`);
+        debugLog(`[${blockIdentifier}] AI 审核报告内容预览：${claudeResult}`);
 
         // 使用正则提取 LINE_INFO 内容（支持换行）
         const lineInfoMatch = claudeResult?.match(/<LINE_INFO>\s*\[([^\]]*)\]\s*<\/LINE_INFO>/);
@@ -228,11 +231,11 @@ class GitLabCodeReviewer {
         const lineInfoContent = hasLineInfoTag ? lineInfoMatch[1].replace(/\s/g, '') : '';
         const hasNonEmptyLineInfo = hasLineInfoTag && lineInfoContent !== '';
 
-        debugLog(`LINE_INFO 检查结果：hasLineInfoTag=${hasLineInfoTag}, hasNonEmptyLineInfo=${hasNonEmptyLineInfo}, lineInfoContent=[${lineInfoContent}]`);
+        debugLog(`[${blockIdentifier}] LINE_INFO 检查结果：hasLineInfoTag=${hasLineInfoTag}, hasNonEmptyLineInfo=${hasNonEmptyLineInfo}, lineInfoContent=[${lineInfoContent}]`);
 
         // LINE_INFO 为空或不存在 → 说明无问题，直接返回标准空格式
         if (!hasLineInfoTag || !hasNonEmptyLineInfo) {
-          debugLog(`【决策】LINE_INFO 为空或不存在，说明无问题，直接返回标准空格式`);
+          debugLog(`[${blockIdentifier}] 【决策】LINE_INFO 为空或不存在，说明无问题，直接返回标准空格式`);
           return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
@@ -240,32 +243,32 @@ class GitLabCodeReviewer {
         // 先检查是否有严重问题（放宽匹配，只检查"严重问题"关键词）
         const hasSeriousProblem = claudeResult && claudeResult.includes('严重问题');
 
-        debugLog(`严重问题检查：hasSeriousProblem=${hasSeriousProblem}`);
+        debugLog(`[${blockIdentifier}] 严重问题检查：hasSeriousProblem=${hasSeriousProblem}`);
 
         // 无严重问题 → 直接返回标准空格式
         if (!hasSeriousProblem) {
-          debugLog(`【决策】报告无严重问题，返回标准空格式`);
+          debugLog(`[${blockIdentifier}] 【决策】报告无严重问题，返回标准空格式`);
           return { reportContent: '<REPORT>\n## 🤖 AI 代码审查结果\n\n</REPORT>', lineInfo: '[]' };
         }
 
         // 有严重问题 → 检查标题是否符合要求
         const hasValidTitle = claudeResult && claudeResult.includes('🤖 AI 代码审查结果');
 
-        debugLog(`标题检查：hasValidTitle=${hasValidTitle}`);
+        debugLog(`[${blockIdentifier}] 标题检查：hasValidTitle=${hasValidTitle}`);
 
         if (hasValidTitle) {
-          debugLog(`【决策】报告包含严重问题且标题正确，接受结果 (尝试 ${attempts})`);
+          debugLog(`[${blockIdentifier}] 【决策】报告包含严重问题且标题正确，接受结果 (尝试 ${attempts})`);
           return extractReportContent(claudeResult);
         }
 
         // 有严重问题但标题不符合 → 重试
-        debugLog(`【决策】报告包含严重问题但标题不符合要求 (尝试 ${attempts})，将重试...`);
+        debugLog(`[${blockIdentifier}] 【决策】报告包含严重问题但标题不符合要求 (尝试 ${attempts})，将重试...`);
         if (attempts >= maxAttempts) {
-          debugLog(`【决策】已达到最大重试次数 ${maxAttempts}，返回最后一次结果`);
+          debugLog(`[${blockIdentifier}] 【决策】已达到最大重试次数 ${maxAttempts}，返回最后一次结果`);
           return extractReportContent(claudeResult);
         }
       } catch (error) {
-        console.error(`AI审核失败 (尝试 ${attempts}/${maxAttempts}):`, error.message);
+        console.error(`[${blockIdentifier}] AI审核失败 (尝试 ${attempts}/${maxAttempts}):`, error.message);
         if (attempts >= maxAttempts) {
           return `审核失败: ${error.message}`;
         }

package/package.json

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