gitlab-ai-review 3.8.2 → 3.9.0

package/index.js

@@ -457,30 +457,53 @@ export class GitLabAIReview {
           impactAnalysis
         );
         
+        // 智能合并修改块的评论
+        const mergedReviews = this.mergeModificationReviews(fileReview.reviews, meaningfulChanges);
+        
         // 根据 AI 返回的结果，只对有问题的行添加评论
-        for (const review of fileReview.reviews) {
+        for (const review of mergedReviews) {
           if (review.hasIssue) {
             try {
+              // 构建评论位置参数
+              const positionParams = {
+                filePath: fileName,
+                oldPath: change.old_path,
+              };
+              
+              // 如果是修改块（同时有 oldLine 和 newLine），同时指定两个行号
+              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);
+                const isDeletion = relatedChange && relatedChange.type === 'deletion';
+                if (isDeletion) {
+                  positionParams.oldLine = review.lineNumber;
+                } else {
+                  positionParams.newLine = review.lineNumber;
+                }
+              }
+              
               const commentResult = await this.gitlabClient.createLineComment(
                 this.config.project.projectId,
                 this.config.project.mergeRequestIid,
                 `🤖 **AI 代码审查**\n\n${review.comment}`,
-                {
-                  filePath: fileName,
-                  oldPath: change.old_path,
-                  newLine: review.lineNumber,
-                }
+                positionParams
               );
 
               results.push({
                 status: 'success',
                 fileName,
-                lineNumber: review.lineNumber,
+                lineNumber: review.lineNumber || review.newLine || review.oldLine,
                 comment: review.comment,
                 commentResult,
               });
 
-              console.log(`  ✓ 第 ${review.lineNumber} 行：已添加评论`);
+              const lineDesc = review.oldLine && review.newLine 
+                ? `第 ${review.oldLine}-${review.newLine} 行` 
+                : `第 ${review.lineNumber} 行`;
+              console.log(`  ✓ ${lineDesc}：已添加评论`);
             } catch (error) {
               results.push({
                 status: 'error',

package/lib/impact-analyzer.js

@@ -195,6 +195,48 @@ export function extractChangedSymbols(diff, fileName) {
   };
 }
 
+/**
+ * 检查一行代码中的符号使用是否可能有局部定义
+ * @param {string} line - 代码行
+ * @param {string} symbol - 符号名
+ * @returns {boolean} 是否可能有局部定义
+ */
+function hasLocalDefinition(line, symbol) {
+  const trimmedLine = line.trim();
+  
+  // 检查是否是局部变量定义
+  const varPatterns = [
+    new RegExp(`(const|let|var)\\s+${symbol}\\s*[=:]`),  // const symbol = ...
+    new RegExp(`(const|let|var)\\s*{[^}]*\\b${symbol}\\b[^}]*}`),  // const { symbol } = ...
+    new RegExp(`function\\s+\\w+\\([^)]*\\b${symbol}\\b[^)]*\\)`),  // function foo(symbol)
+    new RegExp(`\\([^)]*\\b${symbol}\\b[^)]*\\)\\s*=>`),  // (symbol) => ...
+    new RegExp(`\\b${symbol}\\s*:`),  // symbol: in object or parameter
+  ];
+  
+  return varPatterns.some(pattern => pattern.test(trimmedLine));
+}
+
+/**
+ * 检查一行代码中是否有导入语句
+ * @param {string} line - 代码行
+ * @param {string} symbol - 符号名
+ * @returns {boolean} 是否有导入
+ */
+function hasImportStatement(line, symbol) {
+  const trimmedLine = line.trim();
+  
+  // 检查各种导入模式
+  const importPatterns = [
+    new RegExp(`import\\s+${symbol}\\s+from`),  // import symbol from ...
+    new RegExp(`import\\s*{[^}]*\\b${symbol}\\b[^}]*}\\s+from`),  // import { symbol } from ...
+    new RegExp(`import\\s*\\*\\s+as\\s+${symbol}\\s+from`),  // import * as symbol from ...
+    new RegExp(`require\\([^)]*\\)\\.${symbol}`),  // require(...).symbol
+    new RegExp(`const\\s+${symbol}\\s*=\\s*require`),  // const symbol = require(...)
+  ];
+  
+  return importPatterns.some(pattern => pattern.test(trimmedLine));
+}
+
 /**
  * 检查文件内部是否使用了指定的符号
  * @param {string} content - 文件内容

package/package.json

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