gitlab-ai-review 3.2.3 → 3.2.5

package/lib/prompt-tools.js

@@ -84,7 +84,7 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
 2. **优先检查"类型: 删除"的代码**，确认是否有其他地方仍在使用
 3. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 4. **必须使用中文**返回审查意见
-5. 返回 JSON 格式的结果，格式如下：
+5. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
 
 \`\`\`json
 {
@@ -92,24 +92,25 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
     {
       "lineNumber": 15,
       "hasIssue": true,
-      "comment": "请按照项目审查规则中定义的格式填写审查意见"
-    },
-    {
-      "lineNumber": 20,
-      "hasIssue": false
+      "comment": "这里填写审查意见内容"
     }
   ]
 }
 \`\`\`
 
-**JSON 格式要求：**
+**🚨 重要：外层必须是标准 JSON**
+- 整个响应必须是合法的 JSON 对象
+- 不要在最外层使用【】、emoji 或其他特殊格式
 - lineNumber 必须是数字
-- hasIssue 必须是布尔值
-- **comment 字段的内容格式：请严格遵循"项目特定的审查规则"中定义的格式**
-- 如果项目规则中没有定义特殊格式，使用中文简洁描述即可
+- hasIssue 必须是布尔值（true/false）
+- comment 必须是字符串
+
+**comment 字段的内容：**
+- 如果"项目特定的审查规则"中定义了 comment 格式（如包含 emoji、评分等），请在 comment 字符串内使用该格式
+- 如果没有定义特殊格式，使用中文简洁描述即可
 
 6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-7. 只返回 JSON，不要包含其他文字说明
+7. **只返回 JSON**，不要在 JSON 外面添加任何文字说明
 
 **再次强调：**
 - 看到 "类型: 删除" 和 "-" → 代码被移除了
@@ -277,42 +278,67 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     prompt += `\`\`\`\n`;
   });
 
-  prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
+  prompt += `\n## ⚠️ 🚨 最重要的规则：理解代码修改 vs 重复定义
+
+**🔥 核心概念（必读）：**
 
-**请务必正确理解以下标记：**
-1. **"类型: 删除"** + 代码以 **"-"** 开头 = **代码已被移除**（不存在了）
-2. **"类型: 新增"** + 代码以 **"+"** 开头 = **代码是新加的**（新存在的）
+在 diff 中，如果你看到：
+- 第一行：**"-"** 开头（删除旧值）
+- 第二行：**"+"** 开头（添加新值）
+- 并且是同一个变量名或属性名
 
-**🔥 关于代码修改的重要理解：**
+这是 **"修改/替换"**，NOT "重复定义"！
 
-如果在**同一位置**看到以下情况（删除旧代码，添加新代码）：
-- 第一行以 "-" 开头（删除）
-- 第二行以 "+" 开头（新增）
-- 修改的是同一个变量/属性/函数
+**❌ 常见错误（请务必避免）：**
 
-这是**修改（替换）**，不是"重复定义"或"两次定义"！
+**错误 1：对象属性修改**
+看到 diff：
+  - deleted: deletedSymbols,
+  + deleted: allDeleted,
 
-**常见错误理解：**
+错误理解：❌ "返回对象中存在重复的 'deleted' 键，违反对象键唯一性"
+正确理解：✅ "将 deleted 属性的值从 deletedSymbols 修改为 allDeleted"
 
-❌ 错误示例 1：
-看到：
+解释：
+- 旧版本（-）的 "deleted: deletedSymbols" 已经被删除了，不存在了
+- 新版本（+）的 "deleted: allDeleted" 是现在的版本
+- 最终代码中只有一个 deleted 键，没有重复！
+
+**错误 2：变量赋值修改**
+看到 diff：
   - const name = 'old';
   + const name = 'new';
-错误理解：❌ "代码中定义了两次 name 变量，违反唯一性"
-正确理解：✅ "将 name 的值从 'old' 修改为 'new'，这是正常的代码变更"
 
-❌ 错误示例 2：
-看到：
+错误理解：❌ "定义了两次 name 变量"
+正确理解：✅ "修改了 name 的初始值"
+
+**错误 3：属性值修改**
+看到 diff：
   - all: [...addedSymbols, ...deletedSymbols],
   + all: [...addedSymbols, ...allDeleted],
-错误理解：❌ "返回对象中存在两个 all 属性"
-正确理解：✅ "修改了 all 属性的值，从 deletedSymbols 改为 allDeleted"
 
-**核心原则：**
-- **"-"** 的代码已经**不存在**了（被删除）
-- **"+"** 的代码是**新存在**的（被添加）
-- 如果同一个变量/属性有 "-" 又有 "+"，这是**修改**，不是重复定义
-- 在最终代码中，只有 "+" 的版本会保留
+错误理解：❌ "all 属性重复定义"
+正确理解：✅ "修改了 all 数组的第二个展开项"
+
+**🎯 判断标准：**
+
+1. **如何判断是修改而非重复？**
+   - 看是否在**相邻的行**
+   - 一行是 **"-"**（删除）
+   - 下一行是 **"+"**（新增）
+   - **同一个变量名/属性名**
+   → 这就是**修改**！
+
+2. **如何判断是真正的重复定义？**
+   - 两行都是 **"+"**（新增）
+   - 没有对应的 **"-"**（删除）
+   - 同一个变量名/属性名
+   → 这才是**重复定义**！
+
+**🚨 重要提醒：**
+- **"-"** 开头的代码已经**不存在**了，不要把它当作"存在"
+- **"+"** 开头的代码才是**最终保留**的版本
+- 不要说"存在两个XXX"，应该说"将XXX从A修改为B"
 
 ## 审查要求
 
@@ -328,7 +354,7 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
 8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 9. **必须使用中文**返回审查意见
-10. 返回 JSON 格式的结果，格式如下：
+10. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
 
 \`\`\`json
 {
@@ -336,42 +362,37 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     {
       "lineNumber": 15,
       "hasIssue": true,
-      "comment": "请按照项目审查规则中定义的格式填写审查意见"
-    },
-    {
-      "lineNumber": 20,
-      "hasIssue": false
+      "comment": "这里填写审查意见内容"
     }
   ]
 }
 \`\`\`
 
-**🚨 关键规则：**
-- lineNumber 必须是上面"代码变更"中列出的行号（如第 15 行、第 20 行等）
+**🚨 JSON 格式的关键规则：**
+- **整个响应必须是合法的 JSON 对象**
+- **不要在最外层使用【】、emoji 或其他特殊格式**
+- lineNumber 必须是上面"代码变更"中列出的行号
+- lineNumber 必须是数字，不能是 null、undefined
 - **不要使用其他文件的行号**，即使发现其他文件受影响
-- 如果发现其他文件受影响，在当前变更行的 comment 中说明影响范围
-- lineNumber 必须是数字，不能是 null、undefined 或其他值
-- 如果某个变更没有问题，可以不在 reviews 中包含它
-- **comment 字段的内容格式：请严格遵循"项目特定的审查规则"中定义的格式**（如果有定义）
+- 如果发现其他文件受影响，在当前变更行的 comment 字符串中说明
+
+**comment 字段的内容：**
+- 如果"项目特定的审查规则"中定义了 comment 格式（如包含【严重程度评分】、emoji、分点说明等），**请在 comment 字符串内**使用该格式
+- comment 是一个字符串，可以包含换行符 \\n 来格式化内容
+- 如果没有定义特殊格式，使用中文简洁描述即可
 
 **示例：**
-如果删除了第 15 行的函数，且发现其他文件在使用：
-\`\`\`json
-{
-  "lineNumber": 15,
-  "hasIssue": true,
-  "comment": "删除函数 fetchData 会导致以下文件调用失败：components/UserList.vue (第 45 行)、pages/Dashboard.vue (第 78 行)。建议保留该函数或提供替代方案。"
-}
-\`\`\`
+如果删除了第 15 行的函数，且发现其他文件在使用，comment 内容可以是：
+"【严重程度评分：8/10】\\n\\n📋 问题描述：\\n删除了函数 fetchData 但该函数仍被多处使用\\n\\n🔍 影响分析：\\n• 当前文件：函数定义被完全移除\\n• 受影响的其他文件：components/UserList.vue (第 45 行)\\n\\n💡 改进建议：\\n1. 恢复此函数或提供替代方案\\n2. 先标记为 @deprecated"
 
 11. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-12. 只返回 JSON，不要包含其他文字说明
-13. comment 字段必须使用中文，要简洁明了，重点指出影响范围
+12. **只返回 JSON**，不要在 JSON 外面添加任何文字说明
 
 **最后再次确认：**
 - 看到 **"类型: 删除"** 和 **"-"** → 代码**被移除了**，需要检查影响
 - 看到 **"类型: 新增"** 和 **"+"** → 代码**是新的**，检查代码质量
-- **千万不要把删除理解成新增！**`;
+- **千万不要把删除理解成新增！**
+- **返回的必须是标准 JSON，不要在最外层使用特殊字符！**`;
 
   return prompt;
 }

package/package.json

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