gitlab-ai-review 3.6.2 → 3.7.0

package/index.js

@@ -406,6 +406,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,
@@ -413,7 +417,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

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

package/package.json

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