gitlab-ai-review 3.9.0 → 4.0.0

package/index.js

@@ -464,19 +464,42 @@ export class GitLabAIReview {
         for (const review of mergedReviews) {
           if (review.hasIssue) {
             try {
-              // 构建评论位置参数
+              // 找到对应的变更，获取 hunk 范围
+              const relatedChange = meaningfulChanges.find(c => 
+                c.lineNumber === review.lineNumber || 
+                c.lineNumber === review.oldLine ||
+                c.lineNumber === review.newLine
+              );
+              
+              // 构建评论位置参数，包含整个 hunk 的范围
               const positionParams = {
                 filePath: fileName,
                 oldPath: change.old_path,
               };
               
-              // 如果是修改块（同时有 oldLine 和 newLine），同时指定两个行号
-              if (review.oldLine && review.newLine) {
+              // 使用 hunk 的完整范围而不是单行
+              if (relatedChange && relatedChange.hunkRange) {
+                positionParams.oldLine = relatedChange.hunkRange.oldStart;
+                positionParams.newLine = relatedChange.hunkRange.newStart;
+                // 添加范围信息（GitLab 支持 line_range）
+                positionParams.lineRange = {
+                  start: {
+                    line_code: `${relatedChange.hunkRange.oldStart}_${relatedChange.hunkRange.oldStart}`,
+                    type: 'old',
+                    old_line: relatedChange.hunkRange.oldStart,
+                  },
+                  end: {
+                    line_code: `${relatedChange.hunkRange.newEnd}_${relatedChange.hunkRange.newEnd}`,
+                    type: 'new',
+                    new_line: relatedChange.hunkRange.newEnd,
+                  },
+                };
+              } else if (review.oldLine && review.newLine) {
+                // 修改块：同时指定两个行号
                 positionParams.oldLine = review.oldLine;
                 positionParams.newLine = review.newLine;
               } else if (review.lineNumber) {
-                // 单独的删除或新增
-                const relatedChange = meaningfulChanges.find(c => c.lineNumber === review.lineNumber);
+                // 单独的删除或新增（fallback）
                 const isDeletion = relatedChange && relatedChange.type === 'deletion';
                 if (isDeletion) {
                   positionParams.oldLine = review.lineNumber;

package/lib/diff-parser.js

@@ -128,6 +128,12 @@ export function extractMeaningfulChanges(hunks) {
             content: change.content,
             lineNumber: currentNewLine,
             hunk: hunk.header,
+            hunkRange: {  // 添加 hunk 的范围信息
+              oldStart: hunk.oldStart,
+              oldEnd: hunk.oldStart + hunk.oldLines - 1,
+              newStart: hunk.newStart,
+              newEnd: hunk.newStart + hunk.newLines - 1,
+            },
             context: {
               before: hunk.changes.slice(Math.max(0, index - 2), index).map(c => c.content),
               after: hunk.changes.slice(index + 1, index + 3).map(c => c.content),
@@ -144,6 +150,12 @@ export function extractMeaningfulChanges(hunks) {
             content: change.content,
             lineNumber: currentOldLine,  // 使用旧文件的行号
             hunk: hunk.header,
+            hunkRange: {  // 添加 hunk 的范围信息
+              oldStart: hunk.oldStart,
+              oldEnd: hunk.oldStart + hunk.oldLines - 1,
+              newStart: hunk.newStart,
+              newEnd: hunk.newStart + hunk.newLines - 1,
+            },
             context: {
               before: hunk.changes.slice(Math.max(0, index - 2), index).map(c => c.content),
               after: hunk.changes.slice(index + 1, index + 3).map(c => c.content),

package/lib/gitlab-client.js

@@ -92,13 +92,18 @@ export class GitLabClient {
       old_path: lineInfo.oldPath || lineInfo.filePath,
     };
 
-    // 设置行号
+    // 设置行号或行范围
     if (lineInfo.newLine !== undefined) {
       position.new_line = lineInfo.newLine;
     }
     if (lineInfo.oldLine !== undefined) {
       position.old_line = lineInfo.oldLine;
     }
+    
+    // 如果提供了范围信息，使用范围评论
+    if (lineInfo.lineRange) {
+      position.line_range = lineInfo.lineRange;
+    }
 
     return this.request(
       `/projects/${encodeURIComponent(projectId)}/merge_requests/${mergeRequestIid}/discussions`,

package/lib/prompt-tools.js

@@ -210,18 +210,27 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     
     // 文件内部使用情况（重要！）
     if (impactAnalysis.internalUsage && impactAnalysis.internalUsage.length > 0) {
-      prompt += `**🚨 文件内部冲突** (${impactAnalysis.internalUsage.length} 处):\n`;
-      prompt += `当前文件内部仍在使用已删除的符号，将导致运行时错误：\n\n`;
+      prompt += `**🚨 文件内部冲突检测** (${impactAnalysis.internalUsage.length} 处):\n`;
+      prompt += `当前文件内部发现以下位置使用了被删除的符号：\n\n`;
       
       impactAnalysis.internalUsage.slice(0, 5).forEach((usage, index) => {
-        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`:\n`;
+        const localWarning = usage.maybeLocal ? ' ⚠️ **可能有局部定义/导入**' : '';
+        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`${localWarning}:\n`;
         prompt += `\`\`\`javascript\n${usage.line}\n\`\`\`\n`;
       });
       
       if (impactAnalysis.internalUsage.length > 5) {
         prompt += `... 还有 ${impactAnalysis.internalUsage.length - 5} 处\n`;
       }
-      prompt += `\n`;
+      
+      prompt += `\n**🔍 重要：作用域分析指南**\n`;
+      prompt += `标记了 ⚠️ **可能有局部定义/导入** 的行，表示该行可能：\n`;
+      prompt += `1. 定义了局部变量（const/let/var symbol = ...）\n`;
+      prompt += `2. 是函数参数（function(symbol) 或 (symbol) => ...）\n`;
+      prompt += `3. 从其他模块导入（import { symbol } from ...）\n`;
+      prompt += `4. 对象属性或解构（{ symbol } 或 symbol:）\n`;
+      prompt += `\n**对于这些标记的行，请仔细检查代码上下文，确认是否真的使用了被删除的符号。**\n`;
+      prompt += `**如果有局部定义或导入，则不是问题，不要报告！**\n\n`;
     }
     
     // 函数签名

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitlab-ai-review",
-  "version": "3.9.0",
+  "version": "4.0.0",
   "description": "GitLab AI Review SDK with Impact Analysis - 支持影响分析、删除符号检测、注释代码识别、文件内部冲突检查、智能文件过滤的智能代码审查工具",
   "main": "index.js",
   "type": "module",