gitlab-ai-review 2.5.3 → 3.0.0

package/lib/prompt-tools.js

@@ -28,90 +28,244 @@ export function buildSystemPrompt(projectPrompt = '') {
 }
 
 /**
- * 构建整个文件所有变更行的批量审查提示词（按 hunk 组织）
+ * 构建整个文件所有变更行的批量审查提示词
  * @param {string} fileName - 文件名
  * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {Array} hunkGroups - 按 hunk 分组的变更数组
  * @param {string} projectPrompt - 项目配置的 prompt
  * @returns {string} 批量审查提示词
  */
-export function buildFileReviewPrompt(fileName, meaningfulChanges, hunkGroups) {
-  const totalAdditions = meaningfulChanges.filter(c => c.type === 'addition').length;
-  const totalDeletions = meaningfulChanges.filter(c => c.type === 'deletion').length;
-  
-  let prompt = `请审查以下文件的代码变更，只对**有问题的新增行**提出审查意见。
+export function buildFileReviewPrompt(fileName, meaningfulChanges) {
+  let prompt = `请审查以下文件的代码变更，只对**有问题的行**提出审查意见，代码没有问题的行不需要评论。
 
 **文件名**: ${fileName}
-**变更统计**: 
-- 共 ${hunkGroups.length} 个代码块（hunk）
-- 删除了 ${totalDeletions} 行（旧代码）
-- 新增了 ${totalAdditions} 行（新代码）
+**变更数量**: ${meaningfulChanges.length} 行
 
-## 重要说明
-- **每个代码块都明确标注了删除内容和新增内容**
-- **删除的行**：标注为【删除】，是被移除的旧代码，**无需审查**
-- **新增的行**：标注为【新增】，是新加入的代码，**只审查这些行**
-- 行号是指新增行在新文件中的位置
+`;
+
+  meaningfulChanges.forEach((change, index) => {
+    prompt += `\n### 变更 ${index + 1} - 第 ${change.lineNumber} 行\n`;
+    prompt += `**类型**: ${change.type === 'addition' ? '新增' : '删除'}\n`;
+    prompt += `**内容**:\n\`\`\`\n`;
+    
+    // 上下文
+    if (change.context?.before?.length > 0) {
+      change.context.before.forEach(line => {
+        prompt += `  ${line}\n`;
+      });
+    }
+    
+    // 变更行
+    prompt += `${change.type === 'addition' ? '+' : '-'} ${change.content}\n`;
+    
+    // 下文
+    if (change.context?.after?.length > 0) {
+      change.context.after.forEach(line => {
+        prompt += `  ${line}\n`;
+      });
+    }
+    
+    prompt += `\`\`\`\n`;
+  });
+
+  prompt += `\n## 审查要求
+
+1. 仔细审查每一行变更，判断是否存在问题
+2. **只对有问题的行提出审查意见**，没有问题的行不需要评论
+3. **必须使用中文**返回审查意见
+4. 返回 JSON 格式的结果，格式如下：
+
+\`\`\`json
+{
+  "reviews": [
+    {
+      "lineNumber": 15,
+      "hasIssue": true,
+      "comment": "这里用中文描述具体的问题和改进建议"
+    },
+    {
+      "lineNumber": 20,
+      "hasIssue": false
+    }
+  ]
+}
+\`\`\`
+
+5. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+6. 只返回 JSON，不要包含其他文字说明
+7. comment 字段必须使用中文，要简洁明了`;
 
----
+  return prompt;
+}
+
+/**
+ * 构建整个文件批量审查的完整消息数组
+ * @param {string} fileName - 文件名
+ * @param {Array} meaningfulChanges - 有意义的变更数组
+ * @param {string} projectPrompt - 项目配置的 prompt
+ * @returns {Array} 消息数组
+ */
+export function buildFileReviewMessages(fileName, meaningfulChanges, projectPrompt = '') {
+  return [
+    { role: 'system', content: buildSystemPrompt(projectPrompt) },
+    { role: 'user', content: buildFileReviewPrompt(fileName, meaningfulChanges) },
+  ];
+}
+
+/**
+ * 构建包含影响分析的审查提示词
+ * @param {string} fileName - 文件名
+ * @param {Array} meaningfulChanges - 有意义的变更数组
+ * @param {Object} impactAnalysis - 影响分析结果
+ * @returns {string} 审查提示词
+ */
+export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, impactAnalysis) {
+  let prompt = `请审查以下文件的代码变更，只对**有问题的行**提出审查意见，代码没有问题的行不需要评论。
+
+**文件名**: ${fileName}
+**变更数量**: ${meaningfulChanges.length} 行
 
 `;
 
-  // 按 hunk 展示变更
-  hunkGroups.forEach((hunk, hunkIndex) => {
-    prompt += `## 代码块 ${hunkIndex + 1}：${hunk.header}\n\n`;
+  // 添加影响分析信息
+  if (impactAnalysis && !impactAnalysis.error) {
+    prompt += `\n## 🔍 影响分析\n\n`;
     
-    // 展示删除的内容
-    if (hunk.deletions.length > 0) {
-      prompt += `### 📝 该代码块中删除的内容（无需审查）\n\n`;
-      hunk.deletions.forEach((change, index) => {
-        prompt += `\`\`\`\n`;
-        // 上下文
-        if (change.context?.before?.length > 0 && index === 0) {
-          change.context.before.forEach(line => {
-            prompt += `  ${line}\n`;
+    // 变更的符号统计
+    if (impactAnalysis.changedSymbols) {
+      const { added, deleted, modified } = impactAnalysis.changedSymbols;
+      
+      if (added.length > 0 || deleted.length > 0 || modified.length > 0) {
+        prompt += `**符号变更统计**:\n`;
+        if (added.length > 0) {
+          prompt += `- 新增: ${added.length} 个\n`;
+        }
+        if (deleted.length > 0) {
+          prompt += `- 删除: ${deleted.length} 个 ⚠️\n`;
+        }
+        if (modified.length > 0) {
+          prompt += `- 修改: ${modified.length} 个 ⚠️\n`;
+        }
+        prompt += `\n`;
+      }
+      
+      // 被删除的符号详情
+      if (deleted.length > 0) {
+        const deletedDefs = deleted.filter(s => s.type === 'definition');
+        if (deletedDefs.length > 0) {
+          prompt += `**⚠️ 被删除的函数/类/组件** (${deletedDefs.length} 个):\n`;
+          deletedDefs.slice(0, 10).forEach(symbol => {
+            prompt += `- \`${symbol.name}\`\n`;
           });
+          prompt += `\n`;
         }
-        // 删除的行
-        prompt += `【删除】${change.content}\n`;
-        prompt += `\`\`\`\n\n`;
+      }
+      
+      // 新增或修改的符号
+      const changedDefs = [...added, ...modified].filter(s => s.type === 'definition');
+      if (changedDefs.length > 0) {
+        prompt += `**新增/修改的函数/类/组件** (${changedDefs.length} 个):\n`;
+        changedDefs.slice(0, 10).forEach(symbol => {
+          prompt += `- \`${symbol.name}\`\n`;
+        });
+        prompt += `\n`;
+      }
+    }
+    
+    // 文件内部使用情况（重要！）
+    if (impactAnalysis.internalUsage && impactAnalysis.internalUsage.length > 0) {
+      prompt += `**🚨 文件内部冲突** (${impactAnalysis.internalUsage.length} 处):\n`;
+      prompt += `当前文件内部仍在使用已删除的符号，将导致运行时错误：\n\n`;
+      
+      impactAnalysis.internalUsage.slice(0, 5).forEach((usage, index) => {
+        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`:\n`;
+        prompt += `\`\`\`javascript\n${usage.line}\n\`\`\`\n`;
+      });
+      
+      if (impactAnalysis.internalUsage.length > 5) {
+        prompt += `... 还有 ${impactAnalysis.internalUsage.length - 5} 处\n`;
+      }
+      prompt += `\n`;
+    }
+    
+    // 函数签名
+    if (impactAnalysis.signatures && impactAnalysis.signatures.length > 0) {
+      prompt += `**函数/类签名**:\n`;
+      impactAnalysis.signatures.forEach(sig => {
+        prompt += `\`\`\`javascript\n${sig.signature}\n\`\`\`\n`;
       });
+      prompt += `\n`;
     }
     
-    // 展示新增的内容
-    if (hunk.additions.length > 0) {
-      prompt += `### ✨ 该代码块中新增的内容（**请审查**）\n\n`;
-      hunk.additions.forEach((change, index) => {
-        prompt += `\`\`\`\n`;
-        // 上下文
-        if (change.context?.before?.length > 0 && index === 0) {
-          change.context.before.forEach(line => {
-            prompt += `  ${line}\n`;
+    // 受影响的其他文件
+    if (impactAnalysis.affectedFiles && impactAnalysis.affectedFiles.length > 0) {
+      prompt += `**其他受影响的文件** (${impactAnalysis.totalAffectedFiles || impactAnalysis.affectedFiles.length} 个`;
+      if (impactAnalysis.totalAffectedFiles > impactAnalysis.affectedFiles.length) {
+        prompt += `，显示前 ${impactAnalysis.affectedFiles.length} 个`;
+      }
+      prompt += `):\n\n`;
+      
+      impactAnalysis.affectedFiles.forEach((file, index) => {
+        prompt += `### ${index + 1}. ${file.path}\n`;
+        prompt += `使用的符号: ${file.symbols.join(', ')}\n`;
+        
+        // 添加代码片段
+        if (file.snippets && file.snippets.length > 0) {
+          prompt += `\n**使用示例**:\n`;
+          file.snippets.forEach(snippet => {
+            prompt += `\`\`\`javascript (第 ${snippet.lineNumber} 行附近)\n`;
+            prompt += snippet.snippet;
+            prompt += `\n\`\`\`\n`;
           });
         }
-        // 新增的行
-        prompt += `【新增 - 第 ${change.lineNumber} 行】${change.content}\n`;
-        prompt += `\`\`\`\n\n`;
+        prompt += `\n`;
       });
+      
+      prompt += `---\n\n`;
     }
+  }
+
+  // 添加代码变更
+  prompt += `## 📝 代码变更\n\n`;
+  meaningfulChanges.forEach((change, index) => {
+    prompt += `### 变更 ${index + 1} - 第 ${change.lineNumber} 行\n`;
+    prompt += `**类型**: ${change.type === 'addition' ? '新增' : '删除'}\n`;
+    prompt += `**内容**:\n\`\`\`\n`;
+    
+    // 上下文
+    if (change.context?.before?.length > 0) {
+      change.context.before.forEach(line => {
+        prompt += `  ${line}\n`;
+      });
+    }
+    
+    // 变更行
+    prompt += `${change.type === 'addition' ? '+' : '-'} ${change.content}\n`;
     
-    // 如果该 hunk 没有删除也没有新增
-    if (hunk.deletions.length === 0 && hunk.additions.length === 0) {
-      prompt += `*该代码块无实质性变更*\n\n`;
+    // 下文
+    if (change.context?.after?.length > 0) {
+      change.context.after.forEach(line => {
+        prompt += `  ${line}\n`;
+      });
     }
     
-    prompt += `---\n\n`;
+    prompt += `\`\`\`\n`;
   });
 
-  prompt += `## 审查要求
+  prompt += `\n## 审查要求
 
-1. **只审查上面标注为【新增】的代码行**
-2. **不要审查标注为【删除】的代码行**
-3. 每个代码块的删除和新增内容已明确标注，请关注新增内容的质量
-4. 判断新增的代码是否存在问题（安全、性能、逻辑、最佳实践等）
-5. **只对有问题的新增行提出审查意见**，没有问题的行不需要评论
-6. **必须使用中文**返回审查意见
-7. 返回 JSON 格式的结果，格式如下：
+1. 仔细审查每一行变更，判断是否存在问题
+2. **🚨 最高优先级**：检查是否有符号被删除但仍被使用（文件内部或其他文件）
+3. **特别关注**：
+   - 删除的函数/类/组件是否在其他地方被调用
+   - 修改的函数签名是否会导致现有调用失效
+   - 新增的代码是否会影响其他文件的正常运行
+4. 检查函数签名是否改变，所有调用方式是否需要更新
+5. 检查是否有潜在的 API 不兼容问题
+6. 如果发现文件内部使用了被删除的符号，这是**严重错误**
+7. 如果发现其他文件使用了被删除/修改的符号，需要提醒可能的影响
+8. **只对有问题的行提出审查意见**，没有问题的行不需要评论
+9. **必须使用中文**返回审查意见
+10. 返回 JSON 格式的结果，格式如下：
 
 \`\`\`json
 {
@@ -129,26 +283,25 @@ export function buildFileReviewPrompt(fileName, meaningfulChanges, hunkGroups) {
 }
 \`\`\`
 
-8. lineNumber 必须是新增行的行号（在【新增 - 第 X 行】中标注的行号）
-9. 如果所有新增代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-10. 只返回 JSON，不要包含其他文字说明
-11. comment 字段必须使用中文，要简洁明了`;
+11. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+12. 只返回 JSON，不要包含其他文字说明
+13. comment 字段必须使用中文，要简洁明了，重点指出影响范围`;
 
   return prompt;
 }
 
 /**
- * 构建整个文件批量审查的完整消息数组
+ * 构建包含影响分析的完整消息数组
  * @param {string} fileName - 文件名
  * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {Array} hunkGroups - 按 hunk 分组的变更数组
+ * @param {Object} impactAnalysis - 影响分析结果
  * @param {string} projectPrompt - 项目配置的 prompt
  * @returns {Array} 消息数组
  */
-export function buildFileReviewMessages(fileName, meaningfulChanges, hunkGroups, projectPrompt = '') {
+export function buildFileReviewWithImpactMessages(fileName, meaningfulChanges, impactAnalysis, projectPrompt = '') {
   return [
     { role: 'system', content: buildSystemPrompt(projectPrompt) },
-    { role: 'user', content: buildFileReviewPrompt(fileName, meaningfulChanges, hunkGroups) },
+    { role: 'user', content: buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, impactAnalysis) },
   ];
 }
 
@@ -157,5 +310,7 @@ export default {
   buildSystemPrompt,
   buildFileReviewPrompt,
   buildFileReviewMessages,
+  buildFileReviewWithImpactPrompt,
+  buildFileReviewWithImpactMessages,
 };
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "gitlab-ai-review",
-  "version": "2.5.3",
-  "description": "GitLab AI Review SDK - 支持 SDK 调用和 CLI 执行",
+  "version": "3.0.0",
+  "description": "GitLab AI Review SDK with Impact Analysis - 支持影响分析、删除符号检测、文件内部冲突检查的智能代码审查工具",
   "main": "index.js",
   "type": "module",
   "bin": {
@@ -22,11 +22,24 @@
     "gitlab",
     "ai",
     "review",
+    "code-review",
     "sdk",
-    "ci-cd"
+    "ci-cd",
+    "impact-analysis",
+    "static-analysis",
+    "merge-request",
+    "automation"
   ],
-  "author": "",
+  "author": "Your Name <your.email@example.com>",
   "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "https://github.com/yourusername/gitlab-ai-review.git"
+  },
+  "bugs": {
+    "url": "https://github.com/yourusername/gitlab-ai-review/issues"
+  },
+  "homepage": "https://github.com/yourusername/gitlab-ai-review#readme",
   "engines": {
     "node": ">=18.0.0"
   },