gitlab-ai-review 3.3.0 → 3.4.1

package/lib/diff-parser.js

@@ -115,6 +115,7 @@ export function extractMeaningfulChanges(hunks) {
 
   hunks.forEach(hunk => {
     let currentNewLine = hunk.newStart;
+    let currentOldLine = hunk.oldStart;
 
     hunk.changes.forEach((change, index) => {
       if (change.type === 'addition') {
@@ -141,7 +142,7 @@ export function extractMeaningfulChanges(hunks) {
           meaningfulChanges.push({
             type: 'deletion',
             content: change.content,
-            oldLineNumber: currentNewLine,
+            lineNumber: currentOldLine,  // 使用旧文件的行号
             hunk: hunk.header,
             context: {
               before: hunk.changes.slice(Math.max(0, index - 2), index).map(c => c.content),
@@ -149,8 +150,10 @@ export function extractMeaningfulChanges(hunks) {
             },
           });
         }
+        currentOldLine++;
       } else if (change.type === 'context') {
         currentNewLine++;
+        currentOldLine++;
       }
     });
   });

package/lib/prompt-tools.js

@@ -92,7 +92,7 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
     {
       "lineNumber": 15,
       "hasIssue": true,
-      "comment": "这里填写审查意见内容"
+      "comment": "必须按照上面'项目特定的审查规则'中定义的格式填写"
     }
   ]
 }
@@ -106,16 +106,13 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
 - comment 必须是字符串
 
 **comment 字段的内容：**
-- 如果"项目特定的审查规则"中定义了 comment 格式（如包含 emoji、评分等），请在 comment 字符串内使用该格式
-- 如果没有定义特殊格式，使用中文简洁描述即可
+- **必须严格遵循"项目特定的审查规则"（reviewguard.md）中定义的格式**
+- 如果规则中定义了包含 emoji、评分等格式，**必须完整使用该格式**
+- comment 是一个字符串，可以包含换行符 \\n 来格式化内容
+- 不要自创格式，必须参考上面"项目特定的审查规则"中的示例
 
 6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-7. **只返回 JSON**，不要在 JSON 外面添加任何文字说明
-
-**再次强调：**
-- 看到 "类型: 删除" 和 "-" → 代码被移除了
-- 看到 "类型: 新增" 和 "+" → 代码是新加的
-- 不要混淆删除和新增！`;
+7. 只返回 JSON，不要包含其他文字说明`;
 
   return prompt;
 }
@@ -278,83 +275,14 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     prompt += `\`\`\`\n`;
   });
 
-  prompt += `\n## ⚠️ 🚨 最重要的规则：理解代码修改 vs 重复定义
-
-**🔥 核心概念（必读）：**
-
-在 diff 中，如果你看到：
-- 第一行：**"-"** 开头（删除旧值）
-- 第二行：**"+"** 开头（添加新值）
-- 并且是同一个变量名或属性名
-
-这是 **"修改/替换"**，NOT "重复定义"！
-
-**❌ 常见错误（请务必避免）：**
-
-**错误 1：对象属性修改**
-看到 diff：
-  - deleted: deletedSymbols,
-  + deleted: allDeleted,
-
-错误理解：❌ "返回对象中存在重复的 'deleted' 键，违反对象键唯一性"
-正确理解：✅ "将 deleted 属性的值从 deletedSymbols 修改为 allDeleted"
-
-解释：
-- 旧版本（-）的 "deleted: deletedSymbols" 已经被删除了，不存在了
-- 新版本（+）的 "deleted: allDeleted" 是现在的版本
-- 最终代码中只有一个 deleted 键，没有重复！
-
-**错误 2：变量赋值修改**
-看到 diff：
-  - const name = 'old';
-  + const name = 'new';
+  prompt += `\n## 审查要求
 
-错误理解：❌ "定义了两次 name 变量"
-正确理解：✅ "修改了 name 的初始值"
-
-**错误 3：属性值修改**
-看到 diff：
-  - all: [...addedSymbols, ...deletedSymbols],
-  + all: [...addedSymbols, ...allDeleted],
-
-错误理解：❌ "all 属性重复定义"
-正确理解：✅ "修改了 all 数组的第二个展开项"
-
-**🎯 判断标准：**
-
-1. **如何判断是修改而非重复？**
-   - 看是否在**相邻的行**
-   - 一行是 **"-"**（删除）
-   - 下一行是 **"+"**（新增）
-   - **同一个变量名/属性名**
-   → 这就是**修改**！
-
-2. **如何判断是真正的重复定义？**
-   - 两行都是 **"+"**（新增）
-   - 没有对应的 **"-"**（删除）
-   - 同一个变量名/属性名
-   → 这才是**重复定义**！
-
-**🚨 重要提醒：**
-- **"-"** 开头的代码已经**不存在**了，不要把它当作"存在"
-- **"+"** 开头的代码才是**最终保留**的版本
-- 不要说"存在两个XXX"，应该说"将XXX从A修改为B"
-
-## 审查要求
-
-1. **首先确认变更类型**：看清楚是"删除"还是"新增"
-2. **🚨 最高优先级**：检查"类型: 删除"的符号是否仍被使用（文件内部或其他文件）
-3. **特别关注**：
-   - **删除的函数/类/组件**是否在其他地方被调用（这会导致错误）
-   - 修改的函数签名是否会导致现有调用失效
-   - 新增的代码质量是否符合规范
-4. 检查函数签名是否改变，所有调用方式是否需要更新
-5. 检查是否有潜在的 API 不兼容问题
-6. 如果发现文件内部使用了**被删除**的符号，这是**严重错误**
-7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
-8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
-9. **必须使用中文**返回审查意见
-10. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
+1. 仔细审查每一行变更，判断是否存在问题
+2. 注意区分**修改**和**重复定义**：如果同一位置有删除（-）和新增（+），这是修改，不是重复
+3. 优先检查"类型: 删除"的代码，确认是否有其他地方仍在使用
+4. **只对有问题的行提出审查意见**，没有问题的行不需要评论
+5. **必须使用中文**返回审查意见
+6. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
 
 \`\`\`json
 {
@@ -377,22 +305,17 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 - 如果发现其他文件受影响，在当前变更行的 comment 字符串中说明
 
 **comment 字段的内容：**
-- 如果"项目特定的审查规则"中定义了 comment 格式（如包含【严重程度评分】、emoji、分点说明等），**请在 comment 字符串内**使用该格式
+- **必须严格遵循"项目特定的审查规则"（reviewguard.md）中定义的格式**
+- 如果规则中定义了包含【严重程度评分】、emoji、分点说明等格式，**必须完整使用该格式**
 - comment 是一个字符串，可以包含换行符 \\n 来格式化内容
-- 如果没有定义特殊格式，使用中文简洁描述即可
+- 不要自创格式，必须参考上面"项目特定的审查规则"中的示例
 
 **示例：**
-如果删除了第 15 行的函数，且发现其他文件在使用，comment 内容可以是：
+如果 reviewguard.md 中定义了格式，删除第 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**，不要在 JSON 外面添加任何文字说明
-
-**最后再次确认：**
-- 看到 **"类型: 删除"** 和 **"-"** → 代码**被移除了**，需要检查影响
-- 看到 **"类型: 新增"** 和 **"+"** → 代码**是新的**，检查代码质量
-- **千万不要把删除理解成新增！**
-- **返回的必须是标准 JSON，不要在最外层使用特殊字符！**`;
+7. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+8. **只返回 JSON**，不要在 JSON 外面添加任何文字说明`;
 
   return prompt;
 }

package/package.json

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