gitlab-ai-review 3.6.0 → 3.6.2

package/index.js

@@ -209,6 +209,10 @@ 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,
@@ -216,7 +220,8 @@ export class GitLabAIReview {
                 {
                   filePath: fileName,
                   oldPath: change.old_path,
-                  newLine: review.lineNumber,
+                  // 删除的行使用 oldLine，新增的行使用 newLine
+                  ...(isDeletion ? { oldLine: review.lineNumber } : { newLine: review.lineNumber }),
                 }
               );
 

package/lib/prompt-tools.js

@@ -259,6 +259,13 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
       
       prompt += `---\n\n`;
     }
+    
+    prompt += `**🔍 跨文件影响分析提醒：**\n`;
+    prompt += `上述"受影响的文件"是通过文本搜索找到的，可能存在以下情况：\n`;
+    prompt += `1. **局部定义**：受影响的文件中可能有同名的局部变量、函数参数、类方法等\n`;
+    prompt += `2. **重新导入**：受影响的文件可能从其他模块导入了同名符号\n`;
+    prompt += `3. **命名空间**：可能在不同的命名空间或作用域中\n`;
+    prompt += `\n**请仔细检查代码片段，只有确认是引用被删除的符号时，才报告为影响！**\n\n`;
   }
 
   // 添加代码变更
@@ -305,10 +312,15 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 
 1. 仔细审查每一行变更，判断是否存在问题
 2. 注意区分**修改**和**重复定义**：如果同一位置有删除（-）和新增（+），这是修改，不是重复
-3. 优先检查"类型: 删除"的代码，确认是否有其他地方仍在使用
-4. **只对有问题的行提出审查意见**，没有问题的行不需要评论
-5. **必须使用中文**返回审查意见
-6. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
+3. **🚨 检查删除的符号时，必须考虑作用域：**
+   - 检查使用处是否有**局部定义**（函数参数、局部变量、解构等）
+   - 检查使用处是否**重新导入**了同名符号
+   - 只有确认使用的是**被删除的这个符号**时，才报告为错误
+   - 示例：如果删除了全局函数 \`getData()\`，但使用处有 \`const getData = () => {...}\` 或 \`import { getData } from 'other'\`，则**不是问题**
+4. 优先检查"类型: 删除"的代码，确认是否有其他地方仍在使用
+5. **只对有问题的行提出审查意见**，没有问题的行不需要评论
+6. **必须使用中文**返回审查意见
+7. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
 
 \`\`\`json
 {
@@ -336,9 +348,6 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 - comment 是一个字符串，可以包含换行符 \\n 来格式化内容
 - 不要自创格式，必须参考上面"项目特定的审查规则"中的示例
 
-**示例：**
-如果 reviewguard.md 中定义了格式，删除第 15 行的函数时，comment 内容必须是：
-"【严重程度评分：8/10】\\n\\n📋 问题描述：\\n删除了函数 fetchData 但该函数仍被多处使用\\n\\n🔍 影响分析：\\n• 当前文件：函数定义被完全移除\\n• 受影响的其他文件：components/UserList.vue (第 45 行)\\n\\n💡 改进建议：\\n1. 恢复此函数或提供替代方案\\n2. 先标记为 @deprecated"
 
 7. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
 8. **只返回 JSON**，不要在 JSON 外面添加任何文字说明`;

package/package.json

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