gitlab-ai-review 3.2.0 → 3.2.2

package/lib/prompt-tools.js

@@ -67,12 +67,24 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
     prompt += `\`\`\`\n`;
   });
 
-  prompt += `\n## 审查要求
+  prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
+
+**请务必注意以下标记：**
+- 标记为 **"类型: 删除"** 且代码以 **"-"** 开头的，是**被删除的代码**（不再存在）
+- 标记为 **"类型: 新增"** 且代码以 **"+"** 开头的，是**新增的代码**（新添加的）
+
+**特别注意：**
+- 如果看到 **"类型: 删除"** 和 **"-"**，这意味着该代码已被移除，不存在了
+- 不要把"删除"理解成"存在"或"新增"
+- 删除的代码需要重点关注是否仍被其他地方使用
+
+## 审查要求
 
 1. 仔细审查每一行变更，判断是否存在问题
-2. **只对有问题的行提出审查意见**，没有问题的行不需要评论
-3. **必须使用中文**返回审查意见
-4. 返回 JSON 格式的结果，格式如下：
+2. **优先检查"类型: 删除"的代码**，确认是否有其他地方仍在使用
+3. **只对有问题的行提出审查意见**，没有问题的行不需要评论
+4. **必须使用中文**返回审查意见
+5. 返回 JSON 格式的结果，格式如下：
 
 \`\`\`json
 {
@@ -80,7 +92,7 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
     {
       "lineNumber": 15,
       "hasIssue": true,
-      "comment": "这里用中文描述具体的问题和改进建议"
+      "comment": "请按照项目审查规则中定义的格式填写审查意见"
     },
     {
       "lineNumber": 20,
@@ -90,9 +102,19 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
 }
 \`\`\`
 
-5. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-6. 只返回 JSON，不要包含其他文字说明
-7. comment 字段必须使用中文，要简洁明了`;
+**JSON 格式要求：**
+- lineNumber 必须是数字
+- hasIssue 必须是布尔值
+- **comment 字段的内容格式：请严格遵循"项目特定的审查规则"中定义的格式**
+- 如果项目规则中没有定义特殊格式，使用中文简洁描述即可
+
+6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+7. 只返回 JSON，不要包含其他文字说明
+
+**再次强调：**
+- 看到 "类型: 删除" 和 "-" → 代码被移除了
+- 看到 "类型: 新增" 和 "+" → 代码是新加的
+- 不要混淆删除和新增！`;
 
   return prompt;
 }
@@ -255,18 +277,70 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     prompt += `\`\`\`\n`;
   });
 
-  prompt += `\n## 审查要求
+  prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
 
-1. 仔细审查每一行变更，判断是否存在问题
-2. **🚨 最高优先级**：检查是否有符号被删除但仍被使用（文件内部或其他文件）
+**请务必正确理解以下标记：**
+1. **"类型: 删除"** + 代码以 **"-"** 开头 = **代码已被移除**（不存在了）
+2. **"类型: 新增"** + 代码以 **"+"** 开头 = **代码是新加的**（新存在的）
+
+**🔥 关于代码修改的重要理解：**
+
+如果在**同一位置**看到：
+```
+- old code    // 删除旧代码
++ new code    // 添加新代码
+```
+
+这是**修改（替换）**，不是"重复定义"或"两次定义"！
+
+**正确理解示例：**
+
+❌ **错误理解**：
+```
+- const name = 'old';
++ const name = 'new';
+```
+错误：❌ "代码中定义了两次 name 变量，违反唯一性"
+
+✅ **正确理解**：
+```
+- const name = 'old';
++ const name = 'new';
+```
+正确：✅ "将 name 的值从 'old' 修改为 'new'，这是正常的代码变更"
+
+❌ **错误理解**：
+```
+- all: [...addedSymbols, ...deletedSymbols],
++ all: [...addedSymbols, ...allDeleted],
+```
+错误：❌ "返回对象中存在两个 all 属性"
+
+✅ **正确理解**：
+```
+- all: [...addedSymbols, ...deletedSymbols],
++ all: [...addedSymbols, ...allDeleted],
+```
+正确：✅ "修改了 all 属性的值，从 deletedSymbols 改为 allDeleted"
+
+**核心原则：**
+- **"-"** 的代码已经**不存在**了（被删除）
+- **"+"** 的代码是**新存在**的（被添加）
+- 如果同一个变量/属性有 "-" 又有 "+"，这是**修改**，不是重复定义
+- 在最终代码中，只有 "+" 的版本会保留
+
+## 审查要求
+
+1. **首先确认变更类型**：看清楚是"删除"还是"新增"
+2. **🚨 最高优先级**：检查"类型: 删除"的符号是否仍被使用（文件内部或其他文件）
 3. **特别关注**：
-   - 删除的函数/类/组件是否在其他地方被调用
+   - **删除的函数/类/组件**是否在其他地方被调用（这会导致错误）
    - 修改的函数签名是否会导致现有调用失效
-   - 新增的代码是否会影响其他文件的正常运行
+   - 新增的代码质量是否符合规范
 4. 检查函数签名是否改变，所有调用方式是否需要更新
 5. 检查是否有潜在的 API 不兼容问题
-6. 如果发现文件内部使用了被删除的符号，这是**严重错误**
-7. 如果发现其他文件使用了被删除/修改的符号，需要提醒可能的影响
+6. 如果发现文件内部使用了**被删除**的符号，这是**严重错误**
+7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
 8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 9. **必须使用中文**返回审查意见
 10. 返回 JSON 格式的结果，格式如下：
@@ -277,7 +351,7 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     {
       "lineNumber": 15,
       "hasIssue": true,
-      "comment": "这里用中文描述具体的问题和改进建议"
+      "comment": "请按照项目审查规则中定义的格式填写审查意见"
     },
     {
       "lineNumber": 20,
@@ -293,6 +367,7 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 - 如果发现其他文件受影响，在当前变更行的 comment 中说明影响范围
 - lineNumber 必须是数字，不能是 null、undefined 或其他值
 - 如果某个变更没有问题，可以不在 reviews 中包含它
+- **comment 字段的内容格式：请严格遵循"项目特定的审查规则"中定义的格式**（如果有定义）
 
 **示例：**
 如果删除了第 15 行的函数，且发现其他文件在使用：
@@ -306,7 +381,12 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 
 11. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
 12. 只返回 JSON，不要包含其他文字说明
-13. comment 字段必须使用中文，要简洁明了，重点指出影响范围`;
+13. comment 字段必须使用中文，要简洁明了，重点指出影响范围
+
+**最后再次确认：**
+- 看到 **"类型: 删除"** 和 **"-"** → 代码**被移除了**，需要检查影响
+- 看到 **"类型: 新增"** 和 **"+"** → 代码**是新的**，检查代码质量
+- **千万不要把删除理解成新增！**`;
 
   return prompt;
 }

package/package.json

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