gitlab-ai-review 3.8.3 → 3.9.1

package/lib/impact-analyzer.js

@@ -195,6 +195,48 @@ export function extractChangedSymbols(diff, fileName) {
   };
 }
 
+/**
+ * 检查一行代码中的符号使用是否可能有局部定义
+ * @param {string} line - 代码行
+ * @param {string} symbol - 符号名
+ * @returns {boolean} 是否可能有局部定义
+ */
+function hasLocalDefinition(line, symbol) {
+  const trimmedLine = line.trim();
+  
+  // 检查是否是局部变量定义
+  const varPatterns = [
+    new RegExp(`(const|let|var)\\s+${symbol}\\s*[=:]`),  // const symbol = ...
+    new RegExp(`(const|let|var)\\s*{[^}]*\\b${symbol}\\b[^}]*}`),  // const { symbol } = ...
+    new RegExp(`function\\s+\\w+\\([^)]*\\b${symbol}\\b[^)]*\\)`),  // function foo(symbol)
+    new RegExp(`\\([^)]*\\b${symbol}\\b[^)]*\\)\\s*=>`),  // (symbol) => ...
+    new RegExp(`\\b${symbol}\\s*:`),  // symbol: in object or parameter
+  ];
+  
+  return varPatterns.some(pattern => pattern.test(trimmedLine));
+}
+
+/**
+ * 检查一行代码中是否有导入语句
+ * @param {string} line - 代码行
+ * @param {string} symbol - 符号名
+ * @returns {boolean} 是否有导入
+ */
+function hasImportStatement(line, symbol) {
+  const trimmedLine = line.trim();
+  
+  // 检查各种导入模式
+  const importPatterns = [
+    new RegExp(`import\\s+${symbol}\\s+from`),  // import symbol from ...
+    new RegExp(`import\\s*{[^}]*\\b${symbol}\\b[^}]*}\\s+from`),  // import { symbol } from ...
+    new RegExp(`import\\s*\\*\\s+as\\s+${symbol}\\s+from`),  // import * as symbol from ...
+    new RegExp(`require\\([^)]*\\)\\.${symbol}`),  // require(...).symbol
+    new RegExp(`const\\s+${symbol}\\s*=\\s*require`),  // const symbol = require(...)
+  ];
+  
+  return importPatterns.some(pattern => pattern.test(trimmedLine));
+}
+
 /**
  * 检查文件内部是否使用了指定的符号
  * @param {string} content - 文件内容

package/lib/prompt-tools.js

@@ -210,18 +210,27 @@ export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, imp
     
     // 文件内部使用情况（重要！）
     if (impactAnalysis.internalUsage && impactAnalysis.internalUsage.length > 0) {
-      prompt += `**🚨 文件内部冲突** (${impactAnalysis.internalUsage.length} 处):\n`;
-      prompt += `当前文件内部仍在使用已删除的符号，将导致运行时错误：\n\n`;
+      prompt += `**🚨 文件内部冲突检测** (${impactAnalysis.internalUsage.length} 处):\n`;
+      prompt += `当前文件内部发现以下位置使用了被删除的符号：\n\n`;
       
       impactAnalysis.internalUsage.slice(0, 5).forEach((usage, index) => {
-        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`:\n`;
+        const localWarning = usage.maybeLocal ? ' ⚠️ **可能有局部定义/导入**' : '';
+        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`${localWarning}:\n`;
         prompt += `\`\`\`javascript\n${usage.line}\n\`\`\`\n`;
       });
       
       if (impactAnalysis.internalUsage.length > 5) {
         prompt += `... 还有 ${impactAnalysis.internalUsage.length - 5} 处\n`;
       }
-      prompt += `\n`;
+      
+      prompt += `\n**🔍 重要：作用域分析指南**\n`;
+      prompt += `标记了 ⚠️ **可能有局部定义/导入** 的行，表示该行可能：\n`;
+      prompt += `1. 定义了局部变量（const/let/var symbol = ...）\n`;
+      prompt += `2. 是函数参数（function(symbol) 或 (symbol) => ...）\n`;
+      prompt += `3. 从其他模块导入（import { symbol } from ...）\n`;
+      prompt += `4. 对象属性或解构（{ symbol } 或 symbol:）\n`;
+      prompt += `\n**对于这些标记的行，请仔细检查代码上下文，确认是否真的使用了被删除的符号。**\n`;
+      prompt += `**如果有局部定义或导入，则不是问题，不要报告！**\n\n`;
     }
     
     // 函数签名

package/package.json

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