job51-gitlab-cr-node-jt-1 3.0.0 → 3.0.2

package/index.js

@@ -144,7 +144,22 @@ class GitLabCodeReviewer {
           const blockStartTime = Date.now();
           const blockIdentifier = `${diffObject.new_path || diffObject.old_path}#large-file`;
           const review_result = await this.reviewDiffWithClaudeUsingFile(tmpFileName, blockIdentifier, true);
-          const blockObj = { ...diffObject, review_result, temp_file_path: tmpFileName, is_large_file: true };
+
+          // 对于大文件，设置完整的行号范围信息（使用文件总行数）
+          const fileLineCount = fileContent.split('\n').length;
+          const blockObj = {
+            ...diffObject,
+            review_result,
+            temp_file_path: tmpFileName,
+            is_large_file: true,
+            // 设置行号范围信息，让行级评论验证能够通过
+            line_info: {
+              old_start: diffObject.new_file ? 0 : 1,
+              old_count: diffObject.new_file ? 0 : fileLineCount,
+              new_start: 1,
+              new_count: fileLineCount
+            }
+          };
 
           // 记录审查指标
           const reviewTime = Date.now() - blockStartTime;
@@ -156,11 +171,16 @@ class GitLabCodeReviewer {
 
           // 检查审查结果中是否包含严重问题
           if (blockObj.review_result && blockObj.review_result.reportContent && blockObj.review_result.reportContent.includes('严重问题')) {
-            // 对于大文件，发布为一般讨论（无法精确定位行号）
-            const file_path_with_line = `${diffObject.new_path || diffObject.old_path}#L1`;
-            await this.createGeneralDiscussion(projectId, mergeRequestIid, file_path_with_line, blockObj.review_result.reportContent);
-            debugLog(`大文件审查结果已发布为一般讨论: ${file_path_with_line}`);
-            this.metrics.recordCommentPublished();
+            // 尝试发布行级评论（大文件现在也有 line_info，可以验证行号）
+            const commentResult = await this.postSingleCommentToGitLab(projectId, mergeRequestIid, {
+              diff_info: blockObj,
+              block_index: blockIndex,
+              review_result: blockObj.review_result,
+            });
+            if (commentResult && commentResult.hallucination_detected) {
+              hallucinationDetected = true;
+            }
+            debugLog(`大文件审查结果已尝试发布行级评论: ${diffObject.new_path || diffObject.old_path}`);
           } else {
             debugLog(`大文件审查未发现严重问题: ${diffObject.new_path || diffObject.old_path}`);
           }
@@ -563,35 +583,7 @@ ${allReportsText}
 ${largeFilePrompt}
 **⚠️ 强制输出格式要求（违反会导致审查结果丢失）**：
 
-你必须直接输出以下格式，不要输出任何分析过程、思考过程或中间文本：
-
-<REPORT>
-## 🤖 AI 代码审查结果
-
-### 🔴 严重问题
-
-**问题 1**：[问题描述]<br/>
-**文件及行号**：[文件路径：实际行号]<br/>
-**修改建议**：[修复建议]
-
-**问题 2**：[问题描述]<br/>
-**文件及行号**：[文件路径：实际行号]<br/>
-**修改建议**：[修复建议]
-
-...（更多问题按同样格式）
-
-</REPORT>
-
-<LINE_INFO>
-[{"new_path":"文件路径","new_line":实际行号},{"new_path":"文件路径","new_line":实际行号}]
-</LINE_INFO>
-
-**审查重点**：
-1. 资源泄漏：JDBC/文件流未关闭 → 报告资源创建行号
-2. 框架问题：@Async返回具体类型 → 报告方法定义行号
-3. 循环逻辑：状态过期、累积不一致 → 报告循环内出错行号
-4. 异常吞没：catch块空实现 → 报告catch行号
-5. 空指针：包装类型拆箱 → 报告使用行号
+必须以规定格式输出，不要输出任何分析过程、思考过程或中间文本：
 
 **关键**：
 - 有严重问题时，LINE_INFO 必须包含对应行号，否则结果被丢弃

package/package.json

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