gitlab-ai-review 3.7.0 → 3.8.0

package/index.js

@@ -209,10 +209,6 @@ export class GitLabAIReview {
         for (const review of fileReview.reviews) {
           if (review.hasIssue) {
             try {
-              // 找到对应的变更，判断是删除还是新增
-              const relatedChange = meaningfulChanges.find(c => c.lineNumber === review.lineNumber);
-              const isDeletion = relatedChange && relatedChange.type === 'deletion';
-              
               const commentResult = await this.gitlabClient.createLineComment(
                 this.config.project.projectId,
                 this.config.project.mergeRequestIid,
@@ -220,8 +216,7 @@ export class GitLabAIReview {
                 {
                   filePath: fileName,
                   oldPath: change.old_path,
-                  // 删除的行使用 oldLine，新增的行使用 newLine
-                  ...(isDeletion ? { oldLine: review.lineNumber } : { newLine: review.lineNumber }),
+                  newLine: review.lineNumber,
                 }
               );
 
@@ -266,6 +261,66 @@ export class GitLabAIReview {
     return results;
   }
 
+  /**
+   * 智能合并修改块的评论
+   * 如果相邻的删除和新增是同一个修改，合并为一个评论
+   * @param {Array} reviews - AI 返回的审查结果
+   * @param {Array} meaningfulChanges - 有意义的变更列表
+   * @returns {Array} 合并后的审查结果
+   */
+  mergeModificationReviews(reviews, meaningfulChanges) {
+    if (!reviews || reviews.length === 0) return reviews;
+    
+    const merged = [];
+    const processed = new Set();
+    
+    reviews.forEach((review, index) => {
+      if (processed.has(index)) return;
+      
+      const currentChange = meaningfulChanges.find(c => c.lineNumber === review.lineNumber);
+      if (!currentChange) {
+        merged.push(review);
+        return;
+      }
+      
+      // 如果是删除，查找是否有对应的新增（可能是修改）
+      if (currentChange.type === 'deletion') {
+        // 查找相近行号的新增
+        const nextReviewIndex = reviews.findIndex((r, i) => {
+          if (i <= index || processed.has(i)) return false;
+          const nextChange = meaningfulChanges.find(c => c.lineNumber === r.lineNumber);
+          // 相差不超过 5 行，且是新增类型
+          return nextChange && 
+                 nextChange.type === 'addition' && 
+                 Math.abs(nextChange.lineNumber - currentChange.lineNumber) <= 5;
+        });
+        
+        if (nextReviewIndex !== -1) {
+          const nextReview = reviews[nextReviewIndex];
+          const nextChange = meaningfulChanges.find(c => c.lineNumber === nextReview.lineNumber);
+          
+          // 合并为一个修改块的评论
+          merged.push({
+            hasIssue: review.hasIssue || nextReview.hasIssue,
+            oldLine: currentChange.lineNumber,  // 删除行
+            newLine: nextChange.lineNumber,     // 新增行
+            comment: review.hasIssue ? review.comment : nextReview.comment,  // 使用有问题的那个评论
+          });
+          
+          processed.add(index);
+          processed.add(nextReviewIndex);
+          return;
+        }
+      }
+      
+      // 如果不是修改块，保持原样
+      merged.push(review);
+      processed.add(index);
+    });
+    
+    return merged;
+  }
+
   /**
    * 审查单个文件的所有变更（一次 API 调用）
    * @param {Object} change - 代码变更对象
@@ -406,10 +461,6 @@ export class GitLabAIReview {
         for (const review of fileReview.reviews) {
           if (review.hasIssue) {
             try {
-              // 找到对应的变更，判断是删除还是新增
-              const relatedChange = meaningfulChanges.find(c => c.lineNumber === review.lineNumber);
-              const isDeletion = relatedChange && relatedChange.type === 'deletion';
-              
               const commentResult = await this.gitlabClient.createLineComment(
                 this.config.project.projectId,
                 this.config.project.mergeRequestIid,
@@ -417,8 +468,7 @@ export class GitLabAIReview {
                 {
                   filePath: fileName,
                   oldPath: change.old_path,
-                  // 删除的行使用 oldLine，新增的行使用 newLine
-                  ...(isDeletion ? { oldLine: review.lineNumber } : { newLine: review.lineNumber }),
+                  newLine: review.lineNumber,
                 }
               );
 

package/lib/prompt-tools.js

@@ -312,17 +312,12 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 
 1. 仔细审查每一行变更，判断是否存在问题
 2. 注意区分**修改**和**重复定义**：如果同一位置有删除（-）和新增（+），这是修改，不是重复
-3. **🚨 作用域检查的最高优先级：**
-   - SDK 会标记"⚠️ **可能有局部定义/导入**"的行，这些行**大概率不是问题**
-   - 对于标记的行，**默认假设不是问题**，除非有明确证据表明使用的是被删除的符号
-   - 对于未标记的行，也要检查上下文（前后 3-5 行）是否有局部定义
-   - **宁可漏报也不要误报**：不确定时不要报告为问题
-4. **判断标准（按优先级）：**
-   - ✅ 有 "⚠️ 可能有局部定义" 标记 → **跳过，不报告**
-   - ✅ 同一行或前几行有 \`const/let/var symbol\` → **跳过，不报告**
-   - ✅ 同一行或前几行有 \`import { symbol }\` → **跳过，不报告**  
-   - ✅ 在函数参数中 \`function(symbol)\` → **跳过，不报告**
-   - ❌ 确认是全局使用，且无局部定义 → **报告为问题**
+3. **🚨 检查删除的符号时，必须考虑作用域：**
+   - 检查使用处是否有**局部定义**（函数参数、局部变量、解构等）
+   - 检查使用处是否**重新导入**了同名符号
+   - 只有确认使用的是**被删除的这个符号**时，才报告为错误
+   - 示例：如果删除了全局函数 \`getData()\`，但使用处有 \`const getData = () => {...}\` 或 \`import { getData } from 'other'\`，则**不是问题**
+4. 优先检查"类型: 删除"的代码，确认是否有其他地方仍在使用
 5. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 6. **必须使用中文**返回审查意见
 7. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：

package/package.json

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