gitlab-ai-review 3.2.2 → 3.2.4

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 外面添加任何文字说明
 
 **再次强调：**
 - 看到 "类型: 删除" 和 "-" → 代码被移除了
@@ -285,43 +286,28 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 
 **🔥 关于代码修改的重要理解：**
 
-如果在**同一位置**看到：
-```
-- 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：
+看到：
+  - const name = 'old';
+  + const name = 'new';
+错误理解：❌ "代码中定义了两次 name 变量，违反唯一性"
+正确理解：✅ "将 name 的值从 'old' 修改为 'new'，这是正常的代码变更"
+
+❌ 错误示例 2：
+看到：
+  - all: [...addedSymbols, ...deletedSymbols],
+  + all: [...addedSymbols, ...allDeleted],
+错误理解：❌ "返回对象中存在两个 all 属性"
+正确理解：✅ "修改了 all 属性的值，从 deletedSymbols 改为 allDeleted"
 
 **核心原则：**
 - **"-"** 的代码已经**不存在**了（被删除）
@@ -343,7 +329,7 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
 8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 9. **必须使用中文**返回审查意见
-10. 返回 JSON 格式的结果，格式如下：
+10. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
 
 \`\`\`json
 {
@@ -351,42 +337,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.2",
+  "version": "3.2.4",
   "description": "GitLab AI Review SDK with Impact Analysis - 支持影响分析、删除符号检测、注释代码识别、文件内部冲突检查、智能文件过滤的智能代码审查工具",
   "main": "index.js",
   "type": "module",