gitlab-ai-review 3.1.1 → 3.2.1

package/lib/impact-analyzer.js

@@ -59,13 +59,14 @@ export function extractImportsExports(content, fileName) {
  * 从 diff 中提取变更的函数/类/组件名称
  * @param {string} diff - diff 内容
  * @param {string} fileName - 文件名
- * @returns {Object} 变更的符号列表 { added: [], deleted: [], modified: [] }
+ * @returns {Object} 变更的符号列表 { added: [], deleted: [], modified: [], commented: [] }
  */
 export function extractChangedSymbols(diff, fileName) {
-  if (!diff) return { added: [], deleted: [], modified: [] };
+  if (!diff) return { added: [], deleted: [], modified: [], commented: [] };
   
   const addedSymbols = [];
   const deletedSymbols = [];
+  const commentedSymbols = [];  // 新增：被注释掉的符号
   const lines = diff.split('\n');
   
   // 匹配函数、类、常量定义
@@ -84,55 +85,113 @@ export function extractChangedSymbols(diff, fileName) {
     if (isAddition || isDeletion) {
       const cleanLine = line.substring(1); // 移除 +/- 前缀
       
-      // 检查定义
-      definitionPatterns.forEach(pattern => {
-        const match = cleanLine.match(pattern);
-        if (match) {
-          const symbol = {
-            name: match[1],
-            type: 'definition',
-            line: index + 1,
-          };
+      // 检查是否是"注释掉代码"的情况
+      if (isAddition && isDeletion) {
+        // 这种情况不会发生在单行中
+      } else if (isAddition) {
+        // 检查新增的行是否是注释掉的代码
+        const trimmedLine = cleanLine.trim();
+        
+        // 判断是否是注释行
+        const isComment = trimmedLine.startsWith('//') || 
+                         trimmedLine.startsWith('/*') || 
+                         trimmedLine.startsWith('*');
+        
+        if (isComment) {
+          // 提取注释后的代码
+          const codeAfterComment = trimmedLine.replace(/^\/\/\s*/, '')
+                                              .replace(/^\/\*\s*/, '')
+                                              .replace(/^[\*\s]*/, '');
           
-          if (isAddition) {
-            addedSymbols.push(symbol);
-          } else {
-            deletedSymbols.push(symbol);
-          }
-        }
-      });
-      
-      // 匹配函数调用
-      if (isAddition) {
-        const callPattern = /(\w+)\s*\(/g;
-        let callMatch;
-        while ((callMatch = callPattern.exec(cleanLine)) !== null) {
-          addedSymbols.push({
-            name: callMatch[1],
-            type: 'usage',
-            line: index + 1,
+          // 检查注释后的内容是否包含定义
+          definitionPatterns.forEach(pattern => {
+            const match = codeAfterComment.match(pattern);
+            if (match) {
+              commentedSymbols.push({
+                name: match[1],
+                type: 'commented',
+                line: index + 1,
+                originalCode: codeAfterComment,
+              });
+            }
+          });
+        } else {
+          // 正常的新增代码
+          definitionPatterns.forEach(pattern => {
+            const match = cleanLine.match(pattern);
+            if (match) {
+              addedSymbols.push({
+                name: match[1],
+                type: 'definition',
+                line: index + 1,
+              });
+            }
           });
+          
+          // 匹配函数调用
+          const callPattern = /(\w+)\s*\(/g;
+          let callMatch;
+          while ((callMatch = callPattern.exec(cleanLine)) !== null) {
+            addedSymbols.push({
+              name: callMatch[1],
+              type: 'usage',
+              line: index + 1,
+            });
+          }
         }
+      } else if (isDeletion) {
+        // 删除的代码
+        definitionPatterns.forEach(pattern => {
+          const match = cleanLine.match(pattern);
+          if (match) {
+            deletedSymbols.push({
+              name: match[1],
+              type: 'definition',
+              line: index + 1,
+            });
+          }
+        });
       }
     }
   });
   
-  // 找出被修改的符号（既被删除又被添加）
+  // 检测"注释掉"的模式：删除了代码，然后添加了注释版本
+  // 对于每个被注释的符号，检查是否有对应的删除
+  const commentedAsDeleted = [];
+  commentedSymbols.forEach(commented => {
+    const wasDeleted = deletedSymbols.find(del => del.name === commented.name);
+    if (wasDeleted) {
+      // 这个符号被注释掉了（相当于删除）
+      commentedAsDeleted.push({
+        name: commented.name,
+        type: 'commented-out',
+        line: commented.line,
+        wasDeleted: true,
+      });
+    }
+  });
+  
+  // 将"被注释掉"的符号也算作删除
+  const allDeleted = [...deletedSymbols, ...commentedAsDeleted];
+  
+  // 找出被修改的符号（既被删除又被添加，但不是注释掉）
   const modifiedSymbols = [];
   const deletedNames = new Set(deletedSymbols.map(s => s.name));
   const addedNames = new Set(addedSymbols.map(s => s.name));
+  const commentedNames = new Set(commentedAsDeleted.map(s => s.name));
   
   deletedNames.forEach(name => {
-    if (addedNames.has(name)) {
+    if (addedNames.has(name) && !commentedNames.has(name)) {
       modifiedSymbols.push({ name, type: 'modified' });
     }
   });
   
   return {
     added: addedSymbols,
-    deleted: deletedSymbols,
+    deleted: allDeleted,  // 包含注释掉的符号
     modified: modifiedSymbols,
-    all: [...addedSymbols, ...deletedSymbols], // 所有变更的符号
+    commented: commentedAsDeleted,  // 单独记录被注释掉的符号
+    all: [...addedSymbols, ...allDeleted], // 所有变更的符号
   };
 }
 

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
 {
@@ -90,9 +102,14 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges) {
 }
 \`\`\`
 
-5. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-6. 只返回 JSON，不要包含其他文字说明
-7. comment 字段必须使用中文，要简洁明了`;
+6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+7. 只返回 JSON，不要包含其他文字说明
+8. comment 字段必须使用中文，要简洁明了
+
+**再次强调：**
+- 看到 "类型: 删除" 和 "-" → 代码被移除了
+- 看到 "类型: 新增" 和 "+" → 代码是新加的
+- 不要混淆删除和新增！`;
 
   return prompt;
 }
@@ -132,15 +149,18 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     
     // 变更的符号统计
     if (impactAnalysis.changedSymbols) {
-      const { added, deleted, modified } = impactAnalysis.changedSymbols;
+      const { added, deleted, modified, commented } = impactAnalysis.changedSymbols;
       
-      if (added.length > 0 || deleted.length > 0 || modified.length > 0) {
+      if (added.length > 0 || deleted.length > 0 || modified.length > 0 || (commented && commented.length > 0)) {
         prompt += `**符号变更统计**:\n`;
         if (added.length > 0) {
           prompt += `- 新增: ${added.length} 个\n`;
         }
         if (deleted.length > 0) {
-          prompt += `- 删除: ${deleted.length} 个 ⚠️\n`;
+          prompt += `- 删除/注释掉: ${deleted.length} 个 ⚠️\n`;
+        }
+        if (commented && commented.length > 0) {
+          prompt += `  (其中 ${commented.length} 个是被注释掉的代码)\n`;
         }
         if (modified.length > 0) {
           prompt += `- 修改: ${modified.length} 个 ⚠️\n`;
@@ -150,11 +170,12 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
       
       // 被删除的符号详情
       if (deleted.length > 0) {
-        const deletedDefs = deleted.filter(s => s.type === 'definition');
+        const deletedDefs = deleted.filter(s => s.type === 'definition' || s.type === 'commented-out');
         if (deletedDefs.length > 0) {
-          prompt += `**⚠️ 被删除的函数/类/组件** (${deletedDefs.length} 个):\n`;
+          prompt += `**⚠️ 被删除/注释掉的函数/类/组件** (${deletedDefs.length} 个):\n`;
           deletedDefs.slice(0, 10).forEach(symbol => {
-            prompt += `- \`${symbol.name}\`\n`;
+            const status = symbol.type === 'commented-out' ? ' (注释掉)' : '';
+            prompt += `- \`${symbol.name}\`${status}\n`;
           });
           prompt += `\n`;
         }
@@ -251,18 +272,33 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     prompt += `\`\`\`\n`;
   });
 
-  prompt += `\n## 审查要求
+  prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
 
-1. 仔细审查每一行变更，判断是否存在问题
-2. **🚨 最高优先级**：检查是否有符号被删除但仍被使用（文件内部或其他文件）
+**请务必正确理解以下标记：**
+1. **"类型: 删除"** + 代码以 **"-"** 开头 = **代码已被移除**（不存在了）
+2. **"类型: 新增"** + 代码以 **"+"** 开头 = **代码是新加的**（新存在的）
+
+**错误示例（请避免）：**
+❌ 看到 "类型: 删除" 却说"新增了XXX"
+❌ 看到 "-" 号却理解为"添加了"
+❌ 把删除当成存在
+
+**正确理解：**
+✅ "类型: 删除" → 这段代码被删掉了 → 需要检查是否还有地方在用
+✅ "类型: 新增" → 这段代码是新加的 → 检查新代码的质量
+
+## 审查要求
+
+1. **首先确认变更类型**：看清楚是"删除"还是"新增"
+2. **🚨 最高优先级**：检查"类型: 删除"的符号是否仍被使用（文件内部或其他文件）
 3. **特别关注**：
-   - 删除的函数/类/组件是否在其他地方被调用
+   - **删除的函数/类/组件**是否在其他地方被调用（这会导致错误）
    - 修改的函数签名是否会导致现有调用失效
-   - 新增的代码是否会影响其他文件的正常运行
+   - 新增的代码质量是否符合规范
 4. 检查函数签名是否改变，所有调用方式是否需要更新
 5. 检查是否有潜在的 API 不兼容问题
-6. 如果发现文件内部使用了被删除的符号，这是**严重错误**
-7. 如果发现其他文件使用了被删除/修改的符号，需要提醒可能的影响
+6. 如果发现文件内部使用了**被删除**的符号，这是**严重错误**
+7. 如果发现其他文件使用了**被删除/修改**的符号，需要提醒可能的影响
 8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
 9. **必须使用中文**返回审查意见
 10. 返回 JSON 格式的结果，格式如下：
@@ -302,7 +338,12 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
 
 11. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
 12. 只返回 JSON，不要包含其他文字说明
-13. comment 字段必须使用中文，要简洁明了，重点指出影响范围`;
+13. comment 字段必须使用中文，要简洁明了，重点指出影响范围
+
+**最后再次确认：**
+- 看到 **"类型: 删除"** 和 **"-"** → 代码**被移除了**，需要检查影响
+- 看到 **"类型: 新增"** 和 **"+"** → 代码**是新的**，检查代码质量
+- **千万不要把删除理解成新增！**`;
 
   return prompt;
 }

package/package.json

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