gitlab-ai-review 4.2.4 → 6.3.9

package/lib/prompt-tools.js

@@ -1,14 +1,18 @@
 /**
- * Prompt 工具 - 用于拼接行级代码审查的提示词
- * 简化版：只支持行级审查
+ * Prompt 工具 - 用于拼接代码审查的提示词
+ * 支持按块审查（删除+新增组合为一个变更块）
  */
 
 /**
- * 构建系统提示词（附带项目 prompt）
+ * 构建系统提示词（附带项目 prompt 和正负样本）
  * @param {string} projectPrompt - 项目配置的 prompt（来自 reviewguard.md）
+ * @param {Object} feedbackSamples - 历史反馈样本（可选）
+ * @param {Array} feedbackSamples.positive - 正样本（获得好评的审查意见）
+ * @param {Array} feedbackSamples.negative - 负样本（获得差评的审查意见）
+ * @param {string} documentsContext - 项目文档上下文（可选）
  * @returns {string} 系统提示词
  */
-export function buildSystemPrompt(projectPrompt = '') {
+export function buildSystemPrompt(projectPrompt = '', feedbackSamples = null, documentsContext = '') {
   let prompt = `你是一个专业的代码审查助手，负责审查 GitLab Merge Request 的代码变更。
 
 你的职责是：
@@ -19,437 +23,260 @@ export function buildSystemPrompt(projectPrompt = '') {
 
 请以专业、建设性的语气提供审查意见。`;
 
+  // 附带项目文档上下文（PRD 和 TD 的总结）
+  if (documentsContext && documentsContext.trim().length > 0) {
+    prompt += `\n\n${documentsContext}`;
+  }
+
   // 附带项目特定的审查规则
   if (projectPrompt) {
     prompt += `\n\n## 项目特定的审查规则\n\n${projectPrompt}`;
   }
 
-  return prompt;
-}
-
-/**
- * 构建整个文件所有变更行的批量审查提示词
- * @param {string} fileName - 文件名
- * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {string} projectPrompt - 项目配置的 prompt
- * @returns {string} 批量审查提示词
- */
-export function buildFileReviewPrompt(fileName, meaningfulChanges) {
-  let prompt = `请审查以下文件的代码变更，只对**有问题的行**提出审查意见，代码没有问题的行不需要评论。
+  // 附带历史反馈样本（正负样本学习）
+  if (feedbackSamples && (feedbackSamples.positive?.length > 0 || feedbackSamples.negative?.length > 0)) {
+    prompt += `\n\n## 📊 历史反馈学习（基于用户真实评价）\n`;
+    prompt += `\n以下是用户对历史审查意见的反馈，请学习这些示例来提升审查质量。\n`;
 
-**文件名**: ${fileName}
-**变更数量**: ${meaningfulChanges.length} 行
-
-`;
-
-  meaningfulChanges.forEach((change, index) => {
-    prompt += `\n### 变更 ${index + 1} - 第 ${change.lineNumber} 行\n`;
-    prompt += `**类型**: ${change.type === 'addition' ? '新增' : '删除'}\n`;
-    prompt += `**内容**:\n\`\`\`\n`;
-    
-    // 收集所有其他变更的内容（用于过滤）
-    const otherChangesContent = meaningfulChanges
-      .filter((_, i) => i !== index)
-      .map(c => c.content.trim());
-    
-    // 上下文（只显示不变的行，过滤掉其他变更的内容）
-    if (change.context?.before?.length > 0) {
-      change.context.before.forEach(line => {
-        const trimmedLine = line.trim();
-        // 跳过其他变更的内容
-        if (!otherChangesContent.includes(trimmedLine)) {
-          prompt += `  ${line}\n`;
-        }
-      });
-    }
-    
-    // 变更行
-    prompt += `${change.type === 'addition' ? '+' : '-'} ${change.content}\n`;
-    
-    // 下文（只显示不变的行，过滤掉其他变更的内容）
-    if (change.context?.after?.length > 0) {
-      change.context.after.forEach(line => {
-        const trimmedLine = line.trim();
-        // 跳过其他变更的内容
-        if (!otherChangesContent.includes(trimmedLine)) {
-          prompt += `  ${line}\n`;
-        }
+    // 正样本 - 获得好评的审查
+    if (feedbackSamples.positive?.length > 0) {
+      prompt += `\n### ✅ 优秀审查意见示例（获得用户好评 👍）\n`;
+      prompt += `\n学习这些示例的：语气、格式、问题描述方式、建议的具体性\n\n`;
+      
+      feedbackSamples.positive.forEach((sample, index) => {
+        prompt += `**示例 ${index + 1}** (👍${sample.up} 👎${sample.down}):\n`;
+        prompt += `\`\`\`\n${sample.content}\n\`\`\`\n\n`;
       });
     }
-    
-    prompt += `\`\`\`\n`;
-  });
-
-  prompt += `\n## ⚠️ 重要说明：如何区分删除和新增
-
-**请务必注意以下标记：**
-- 标记为 **"类型: 删除"** 且代码以 **"-"** 开头的，是**被删除的代码**（不再存在）
-- 标记为 **"类型: 新增"** 且代码以 **"+"** 开头的，是**新增的代码**（新添加的）
-
-**特别注意：**
-- 如果看到 **"类型: 删除"** 和 **"-"**，这意味着该代码已被移除，不存在了
-- 不要把"删除"理解成"存在"或"新增"
-- 删除的代码需要重点关注是否仍被其他地方使用
 
-## 审查要求
-
-1. 仔细审查每一行变更，判断是否存在问题
-2. **优先检查"类型: 删除"的代码**，确认是否有其他地方仍在使用
-3. **只对有问题的行提出审查意见**，没有问题的行不需要评论
-4. **必须使用中文**返回审查意见
-5. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
-
-\`\`\`json
-{
-  "reviews": [
-    {
-      "lineNumber": 15,
-      "hasIssue": true,
-      "comment": "必须按照上面'项目特定的审查规则'中定义的格式填写"
+    // 负样本 - 获得差评的审查
+    if (feedbackSamples.negative?.length > 0) {
+      prompt += `\n### ❌ 需要避免的审查意见示例（获得用户差评 👎）\n`;
+      prompt += `\n避免这些示例中的问题：可能过于严苛、建议不实用、或误报问题\n\n`;
+      
+      feedbackSamples.negative.forEach((sample, index) => {
+        prompt += `**反例 ${index + 1}** (👍${sample.up} 👎${sample.down}):\n`;
+        prompt += `\`\`\`\n${sample.content}\n\`\`\`\n\n`;
+      });
     }
-  ]
-}
-\`\`\`
 
-**🚨 重要：外层必须是标准 JSON**
-- 整个响应必须是合法的 JSON 对象
-- 不要在最外层使用【】、emoji 或其他特殊格式
-- lineNumber 必须是数字
-- hasIssue 必须是布尔值（true/false）
-- comment 必须是字符串
-
-**comment 字段的内容：**
-- **必须严格遵循"项目特定的审查规则"（reviewguard.md）中定义的格式**
-- 如果规则中定义了包含 emoji、评分等格式，**必须完整使用该格式**
-- comment 是一个字符串，可以包含换行符 \\n 来格式化内容
-- 不要自创格式，必须参考上面"项目特定的审查规则"中的示例
-
-6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-7. 只返回 JSON，不要包含其他文字说明`;
+    prompt += `---\n\n**重要提示**: 请参考正样本的风格，避免负样本中的问题，生成高质量的审查意见。\n`;
+  }
 
   return prompt;
 }
 
 /**
- * 构建整个文件批量审查的完整消息数组
- * @param {string} fileName - 文件名
- * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {string} projectPrompt - 项目配置的 prompt
- * @returns {Array} 消息数组
+ * 格式化单个变更块的内容
+ * @param {Object} block - 变更块
+ * @param {number} index - 变更块索引
+ * @returns {string} 格式化后的内容
  */
-export function buildFileReviewMessages(fileName, meaningfulChanges, projectPrompt = '') {
-  return [
-    { role: 'system', content: buildSystemPrompt(projectPrompt) },
-    { role: 'user', content: buildFileReviewPrompt(fileName, meaningfulChanges) },
-  ];
+function formatChangeBlock(block, index) {
+  // 计算评论应该使用的行号
+  // 使用 endLine（结束行号），这样 GitLab 显示评论时会包含整个代码块的上下文
+  let targetLine = null;
+  let isOldLine = false;
+  
+  if (block.additions) {
+    // 新增或修改：使用新增部分的最后一行
+    targetLine = block.additions.endLine;
+  } else if (block.deletions) {
+    // 纯删除：使用删除部分的最后一行
+    targetLine = block.deletions.endLine;
+    isOldLine = true;
+  }
+  
+  let content = `\n### 变更 ${index + 1}`;
+  
+  // 添加变更类型标识和目标行号
+  if (block.type === 'modification') {
+    content += ` [修改] → 评论行号: ${targetLine}\n`;
+  } else if (block.type === 'deletion') {
+    content += ` [删除] → 评论行号: ${targetLine} (旧文件, isOldLine: true)\n`;
+  } else {
+    content += ` [新增] → 评论行号: ${targetLine}\n`;
+  }
+
+  // 显示删除部分
+  if (block.deletions) {
+    const { startLine, endLine, lines } = block.deletions;
+    const lineRange = startLine === endLine ? `旧文件第 ${startLine} 行` : `旧文件第 ${startLine}-${endLine} 行`;
+    content += `\n**删除** ${lineRange}：\n\`\`\`diff\n`;
+    lines.forEach((line, i) => {
+      content += `${startLine + i}: - ${line}\n`;
+    });
+    content += `\`\`\`\n`;
+  }
+
+  // 显示新增部分
+  if (block.additions) {
+    const { startLine, endLine, lines } = block.additions;
+    const lineRange = startLine === endLine ? `新文件第 ${startLine} 行` : `新文件第 ${startLine}-${endLine} 行`;
+    content += `\n**新增** ${lineRange}：\n\`\`\`diff\n`;
+    lines.forEach((line, i) => {
+      content += `${startLine + i}: + ${line}\n`;
+    });
+    content += `\`\`\`\n`;
+  }
+
+  return content;
 }
 
 /**
- * 构建包含影响分析的审查提示词
+ * 构建整个文件所有变更的批量审查提示词
  * @param {string} fileName - 文件名
- * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {Object} impactAnalysis - 影响分析结果
- * @returns {string} 审查提示词
+ * @param {Array} meaningfulChanges - 有意义的变更数组（按块格式）
+ * @param {string} fullFileContent - 完整文件内容（可选）
+ * @param {Object} impactInfo - 调用链影响信息（可选）
+ * @returns {string} 批量审查提示词
  */
-export function buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, impactAnalysis) {
-  let prompt = `请审查以下文件的代码变更，只对**有问题的行**提出审查意见，代码没有问题的行不需要评论。
+export function buildFileReviewPrompt(fileName, meaningfulChanges, fullFileContent = null, impactInfo = null) {
+  let prompt = `请审查以下文件的代码变更，只对**有问题的变更块**提出审查意见，没有问题的不需要评论。
 
 **文件名**: ${fileName}
-**变更数量**: ${meaningfulChanges.length} 行
+**变更块数量**: ${meaningfulChanges.length} 个
 
 `;
 
-  // 添加影响分析信息
-  if (impactAnalysis && !impactAnalysis.error) {
-    prompt += `\n## 🔍 影响分析\n\n`;
+  // 🎯 添加完整文件内容（如果有）
+  if (fullFileContent) {
+    const lines = fullFileContent.split('\n');
+    const totalLines = lines.length;
+    const maxLines = 500; // 限制最多显示 500 行，避免 prompt 过长
     
-    // 变更的符号统计
-    if (impactAnalysis.changedSymbols) {
-      const { added, deleted, modified, commented } = impactAnalysis.changedSymbols;
-      
-      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`;
-        }
-        if (commented && commented.length > 0) {
-          prompt += `  (其中 ${commented.length} 个是被注释掉的代码)\n`;
-        }
-        if (modified.length > 0) {
-          prompt += `- 修改: ${modified.length} 个 ⚠️\n`;
-        }
-        prompt += `\n`;
-      }
-      
-      // 被删除的符号详情
-      if (deleted.length > 0) {
-        const deletedDefs = deleted.filter(s => s.type === 'definition' || s.type === 'commented-out');
-        if (deletedDefs.length > 0) {
-          prompt += `**⚠️ 被删除/注释掉的函数/类/组件** (${deletedDefs.length} 个):\n`;
-          deletedDefs.slice(0, 10).forEach(symbol => {
-            const status = symbol.type === 'commented-out' ? ' (注释掉)' : '';
-            prompt += `- \`${symbol.name}\`${status}\n`;
-          });
-          prompt += `\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`;
-      }
-    }
+    prompt += `## 📄 完整文件内容（共 ${totalLines} 行）\n\n`;
     
-    // 文件内部使用情况（重要！）
-    if (impactAnalysis.internalUsage && impactAnalysis.internalUsage.length > 0) {
-      prompt += `**🚨 文件内部冲突检测** (${impactAnalysis.internalUsage.length} 处):\n`;
-      prompt += `当前文件内部发现以下位置使用了被删除的符号：\n\n`;
-      
-      impactAnalysis.internalUsage.slice(0, 5).forEach((usage, index) => {
-        const localWarning = usage.maybeLocal ? ' ⚠️ **可能有局部定义/导入**' : '';
-        prompt += `${index + 1}. 第 ${usage.lineNumber} 行使用了被删除的 \`${usage.symbol}\`${localWarning}:\n`;
-        prompt += `\`\`\`javascript\n${usage.line}\n\`\`\`\n`;
+    if (totalLines <= maxLines) {
+      prompt += '```\n';
+      lines.forEach((line, index) => {
+        prompt += `${index + 1}| ${line}\n`;
       });
-      
-      if (impactAnalysis.internalUsage.length > 5) {
-        prompt += `... 还有 ${impactAnalysis.internalUsage.length - 5} 处\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`;
-    }
-    
-    // 函数签名
-    if (impactAnalysis.signatures && impactAnalysis.signatures.length > 0) {
-      prompt += `**函数/类签名**:\n`;
-      impactAnalysis.signatures.forEach(sig => {
-        prompt += `\`\`\`javascript\n${sig.signature}\n\`\`\`\n`;
+      prompt += '```\n\n';
+    } else {
+      prompt += `⚠️ 文件过长（${totalLines} 行），仅显示前 ${maxLines} 行\n\n`;
+      prompt += '```\n';
+      lines.slice(0, maxLines).forEach((line, index) => {
+        prompt += `${index + 1}| ${line}\n`;
       });
-      prompt += `\n`;
-    }
-    
-    // 🎯 优先显示 TypeScript 调用链信息
-    if (impactAnalysis.callChain && impactAnalysis.callChain.chains) {
-      prompt += `**🔗 TypeScript 调用链分析**:\n`;
-      prompt += `使用了 TypeScript Compiler API 进行精确的类型分析和调用链追踪\n\n`;
-      
-      impactAnalysis.callChain.chains.forEach((chain, chainIndex) => {
-        prompt += `### 调用链 ${chainIndex + 1}: \`${chain.symbol}\`\n`;
-        
-        // 显示问题（如果有）
-        if (chain.issues && chain.issues.length > 0) {
-          prompt += `**🚨 检测到的问题**:\n`;
-          chain.issues.forEach(issue => {
-            prompt += `- [${issue.severity}] ${issue.message}\n`;
-          });
-          prompt += `\n`;
-        }
-        
-        // 显示使用处
-        if (chain.usages && chain.usages.length > 0) {
-          prompt += `**使用位置** (${chain.usages.length} 处):\n\n`;
-          
-          chain.usages.slice(0, 5).forEach((usage, usageIndex) => {
-            prompt += `${usageIndex + 1}. **${usage.file}:${usage.line}**\n`;
-            
-            // 显示数据流（如果有）
-            if (usage.dataFlow && usage.dataFlow.length > 0) {
-              prompt += `   数据流: ${usage.dataFlow.join(' → ')}\n`;
-            }
-            
-            // 显示问题（如果有）
-            if (usage.issue) {
-              prompt += `   ⚠️  **问题**: ${usage.issue.message}\n`;
-            }
-            
-            // 显示代码上下文
-            if (usage.context) {
-              prompt += `\n\`\`\`typescript\n${usage.context}\n\`\`\`\n`;
-            }
-            prompt += `\n`;
-          });
-          
-          if (chain.usages.length > 5) {
-            prompt += `... 还有 ${chain.usages.length - 5} 处使用\n\n`;
-          }
-        }
-      });
-      
-      // 显示摘要
-      if (impactAnalysis.callChain.summary) {
-        const summary = impactAnalysis.callChain.summary;
-        prompt += `**📊 调用链分析摘要**:\n`;
-        prompt += `- 分析的符号: ${summary.totalSymbols || 0} 个\n`;
-        prompt += `- 找到的引用: ${summary.totalReferences || 0} 处\n`;
-        prompt += `- 检测到的问题: ${summary.totalIssues || 0} 个\n`;
-        if (summary.issuesByType && Object.keys(summary.issuesByType).length > 0) {
-          prompt += `- 问题类型分布:\n`;
-          Object.entries(summary.issuesByType).forEach(([type, count]) => {
-            prompt += `  - ${type}: ${count}\n`;
-          });
-        }
-        prompt += `\n`;
-      }
-      
-      prompt += `**✅ 这是精确的类型分析结果，可以直接信任！**\n\n`;
-      prompt += `---\n\n`;
-    }
-    // 如果没有调用链，显示传统的 GitLab Search 结果
-    else 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`;
-      prompt += `⚠️  使用 GitLab Search API 搜索（文本匹配，可能不够精确）\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 += `\n`;
-      });
-      
-      prompt += `---\n\n`;
-      
-      prompt += `**🔍 跨文件影响分析提醒：**\n`;
-      prompt += `上述"受影响的文件"是通过文本搜索找到的，可能存在以下情况：\n`;
-      prompt += `1. **局部定义**：受影响的文件中可能有同名的局部变量、函数参数、类方法等\n`;
-      prompt += `2. **重新导入**：受影响的文件可能从其他模块导入了同名符号\n`;
-      prompt += `3. **命名空间**：可能在不同的命名空间或作用域中\n`;
-      prompt += `\n**请仔细检查代码片段，只有确认是引用被删除的符号时，才报告为影响！**\n\n`;
+      prompt += '... (省略剩余 ' + (totalLines - maxLines) + ' 行)\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`;
-    
-    // 收集所有其他变更的内容（用于过滤）
-    const otherChangesContent = meaningfulChanges
-      .filter((_, i) => i !== index)
-      .map(c => c.content.trim());
+  // 🎯 添加调用链影响信息（如果有）
+  if (impactInfo && impactInfo.references && impactInfo.references.length > 0) {
+    prompt += `## 🔗 调用链影响分析\n\n`;
     
-    // 上下文（只显示不变的行，过滤掉其他变更的内容）
-    if (change.context?.before?.length > 0) {
-      change.context.before.forEach(line => {
-        const trimmedLine = line.trim();
-        // 跳过其他变更的内容
-        if (!otherChangesContent.includes(trimmedLine)) {
-          prompt += `  ${line}\n`;
-        }
-      });
+    // 显示导出变更
+    const { changedExports } = impactInfo;
+    if (changedExports) {
+      if (changedExports.removed?.length > 0) {
+        prompt += `**⚠️ 删除的导出:** \`${changedExports.removed.join('`, `')}\`\n`;
+      }
+      if (changedExports.modified?.length > 0) {
+        const modifiedList = changedExports.modified.map(m => 
+          `${m.name}(${m.changeType === 'signature' ? '签名变更' : '内容变更'})`
+        ).join(', ');
+        prompt += `**✏️ 修改的导出:** ${modifiedList}\n`;
+      }
     }
     
-    // 变更行
-    prompt += `${change.type === 'addition' ? '+' : '-'} ${change.content}\n`;
+    prompt += `\n以下文件使用了这些导出，请检查本次修改是否会导致它们出现问题：\n\n`;
     
-    // 下文（只显示不变的行，过滤掉其他变更的内容）
-    if (change.context?.after?.length > 0) {
-      change.context.after.forEach(line => {
-        const trimmedLine = line.trim();
-        // 跳过其他变更的内容
-        if (!otherChangesContent.includes(trimmedLine)) {
-          prompt += `  ${line}\n`;
-        }
-      });
-    }
+    impactInfo.references.forEach((ref, index) => {
+      prompt += `### ${index + 1}. ${ref.file}\n`;
+      prompt += `**使用的符号:** \`${ref.symbols.join('`, `')}\`\n\n`;
+      prompt += `**使用代码片段:**\n`;
+      prompt += `\`\`\`\n${ref.usageContext}\n\`\`\`\n\n`;
+    });
     
-    prompt += `\`\`\`\n`;
+    prompt += `---\n\n`;
+    prompt += `⚠️ **请特别关注以下风险：**\n`;
+    prompt += `- 函数签名变化（参数增减、类型变更）可能导致调用方参数不匹配\n`;
+    prompt += `- 删除的导出会导致依赖文件的 import 失败\n`;
+    prompt += `- 返回值类型或结构变化可能导致调用方逻辑错误\n`;
+    prompt += `- 行为变更可能导致依赖方出现非预期结果\n\n`;
+  }
+
+  prompt += `## 📝 代码变更详情\n`;
+
+  // 格式化每个变更块
+  meaningfulChanges.forEach((block, index) => {
+    prompt += formatChangeBlock(block, index);
   });
 
-  prompt += `\n## 审查要求
+  prompt += `
+## ⚠️ 重要说明
+
+**变更类型说明：**
+- **[修改]**：同时包含删除和新增，表示代码被修改
+- **[删除]**：只有删除，表示代码被移除
+- **[新增]**：只有新增，表示新添加的代码
+
+**审查重点：**
+- 对于 **[修改]** 类型，关注修改是否正确、是否有遗漏
+- 对于 **[删除]** 类型，关注被删除的代码是否还有其他地方在使用
+- 对于 **[新增]** 类型，关注新代码的质量和规范性
 
-1. 仔细审查每一行变更，判断是否存在问题
-2. 注意区分**修改**和**重复定义**：如果同一位置有删除（-）和新增（+），这是修改，不是重复
-3. **🚨 检查删除的符号时，必须考虑作用域：**
-   - 检查使用处是否有**局部定义**（函数参数、局部变量、解构等）
-   - 检查使用处是否**重新导入**了同名符号
-   - 只有确认使用的是**被删除的这个符号**时，才报告为错误
-   - 示例：如果删除了全局函数 \`getData()\`，但使用处有 \`const getData = () => {...}\` 或 \`import { getData } from 'other'\`，则**不是问题**
-4. 优先检查"类型: 删除"的代码，确认是否有其他地方仍在使用
-5. **只对有问题的行提出审查意见**，没有问题的行不需要评论
-6. **必须使用中文**返回审查意见
-7. **必须返回标准 JSON 格式**，不要用其他格式，结构如下：
+## 审查要求
+
+1. 仔细审查每个变更块，判断是否存在问题
+2. **优先检查删除的代码**，确认是否有其他地方仍在使用
+3. **只对有问题的变更块提出审查意见**，没有问题的不需要评论
+4. **必须使用中文**返回审查意见
+5. **必须返回标准 JSON 格式**，结构如下：
 
 \`\`\`json
 {
   "reviews": [
     {
       "lineNumber": 15,
+      "isOldLine": false,
       "hasIssue": true,
-      "comment": "这里填写审查意见内容"
+      "comment": "审查意见内容"
     }
   ]
 }
 \`\`\`
 
-**🚨 JSON 格式的关键规则：**
-- **整个响应必须是合法的 JSON 对象**
-- **不要在最外层使用【】、emoji 或其他特殊格式**
-- lineNumber 必须是上面"代码变更"中列出的行号
-- lineNumber 必须是数字，不能是 null、undefined
-- **不要使用其他文件的行号**，即使发现其他文件受影响
-- 如果发现其他文件受影响，在当前变更行的 comment 字符串中说明
+**🚨 JSON 格式规则：**
+- 整个响应必须是合法的 JSON 对象
+- **lineNumber**: 必须使用每个变更块标注的"评论行号"（这是代码块的最后一行，便于显示完整上下文）
+- **isOldLine**: 如果标注了"(旧文件, isOldLine: true)"，设为 true；否则设为 false
+- hasIssue 必须是布尔值（true/false）
+- comment 必须是字符串，可以包含换行符 \\n
 
 **comment 字段的内容：**
 - **必须严格遵循"项目特定的审查规则"（reviewguard.md）中定义的格式**
-- 如果规则中定义了包含【严重程度评分】、emoji、分点说明等格式，**必须完整使用该格式**
-- comment 是一个字符串，可以包含换行符 \\n 来格式化内容
+- 如果规则中定义了包含 emoji、评分等格式，**必须完整使用该格式**
 - 不要自创格式，必须参考上面"项目特定的审查规则"中的示例
 
-
-7. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
-8. **只返回 JSON**，不要在 JSON 外面添加任何文字说明`;
+6. 如果所有代码都没有问题，返回空的 reviews 数组：\`{"reviews": []}\`
+7. 只返回 JSON，不要包含其他文字说明`;
 
   return prompt;
 }
 
 /**
- * 构建包含影响分析的完整消息数组
+ * 构建整个文件批量审查的完整消息数组
  * @param {string} fileName - 文件名
  * @param {Array} meaningfulChanges - 有意义的变更数组
- * @param {Object} impactAnalysis - 影响分析结果
  * @param {string} projectPrompt - 项目配置的 prompt
+ * @param {string} fullFileContent - 完整文件内容（可选）
+ * @param {Object} impactInfo - 跨文件影响信息（可选）
+ * @param {Object} feedbackSamples - 历史反馈样本（可选）
  * @returns {Array} 消息数组
  */
-export function buildFileReviewWithImpactMessages(fileName, meaningfulChanges, impactAnalysis, projectPrompt = '') {
+export function buildFileReviewMessages(fileName, meaningfulChanges, projectPrompt = '', fullFileContent = null, impactInfo = null, feedbackSamples = null, documentsContext = '') {
   return [
-    { role: 'system', content: buildSystemPrompt(projectPrompt) },
-    { role: 'user', content: buildFileReviewWithImpactPrompt(fileName, meaningfulChanges, impactAnalysis) },
+    { role: 'system', content: buildSystemPrompt(projectPrompt, feedbackSamples, documentsContext) },
+    { role: 'user', content: buildFileReviewPrompt(fileName, meaningfulChanges, fullFileContent, impactInfo) },
   ];
 }
 
 /**
  * 构建生成审核提示词的消息（当找不到 reviewguard.md 时使用）
- * @param {Object} projectAnalysis - 项目分析结果
- * @returns {Array} 消息数组
  */
 export function buildGenerateGuardPromptMessages(projectAnalysis) {
   const { techStack, keyFiles, snippets } = projectAnalysis;
@@ -481,110 +308,72 @@ ${snippet.content}
 
 ---
 
-## 要求
-
-请根据以上项目信息，生成一份完整的 **reviewguard.md** 文档内容。文档应包含：
-
-### 1. 项目概述
-- 简要描述项目的技术栈和架构
-- 列出主要使用的框架和工具
-
-### 2. 开发原则
-- 根据项目技术栈，列出该项目应遵循的开发原则
-- 例如：React 项目应包含组件设计原则、Hooks 使用规范等
-- 例如：使用 TypeScript 的项目应强调类型安全
-
-### 3. AI 代码审查规则
-必须包含以下审查类别（按严重程度分类）：
+请生成一份完整的 reviewguard.md 文档，包含审查规则和回复格式规范。`;
 
-#### 破坏性变更检查（严重程度 8-10 分）
-- 删除/重命名导出的函数、组件、类型
-- API 签名变更
-- 必须检查是否有其他文件引用
-
-#### 安全漏洞（严重程度 7-10 分）
-- XSS、SQL 注入等
-- 敏感信息硬编码
-- 用户输入未校验
-
-#### 性能问题（严重程度 5-8 分）
-- 不必要的重渲染（React）
-- 内存泄漏
-- 大列表未优化
-
-#### 类型安全（严重程度 4-7 分，TypeScript 项目必须包含）
-- 滥用 any 类型
-- 缺少类型定义
-- 不安全的类型断言
-
-#### 代码规范（严重程度 1-4 分）
-- 命名不规范
-- 注释缺失
-- 函数过长
-
-### 4. 回复格式规范（非常重要！）
-
-必须包含统一的回复格式，要求 AI 按照以下结构返回审查意见：
-
-\`\`\`
-【严重程度评分：X/10】
+  return [
+    {
+      role: 'system',
+      content: `你是一个专业的代码审查规范设计专家。生成的文档必须使用中文。`
+    },
+    {
+      role: 'user',
+      content: userMessage
+    }
+  ];
+}
 
-📋 问题描述：
-[一句话概括问题]
+/**
+ * 构建 MR 总结的提示词
+ */
+export function buildMRSummaryPrompt(stats, commentsByFile) {
+  let prompt = `请为本次 Merge Request 生成一个专业的代码审查总结。
 
-🔍 影响分析：
-• 当前文件：[具体影响]
-• 受影响的其他文件：[文件名和行号]
-• 可能后果：[具体说明]
+## 📊 审查统计
 
-💡 改进建议：
-1. [具体操作步骤]
-2. [注意事项]
-\`\`\`
+- **审查文件数**: ${stats.totalFiles} 个
+- **成功评论数**: ${stats.successComments} 条
+- **失败评论数**: ${stats.failedComments} 条
 
-### 5. 严重程度评分标准
 
-定义 0-10 分的评分标准：
-- 1-3 分：低风险（代码风格、命名规范）
-- 4-6 分：中等风险（违反最佳实践、性能问题）
-- 7-8 分：高风险（不兼容的变更、安全漏洞）
-- 9-10 分：严重风险（破坏性变更、系统崩溃）
+## 📁 审查的文件列表
 
-### 6. 示例（必须包含 3-5 个实际示例）
+${stats.filesList.map((file, index) => `${index + 1}. ${file}`).join('\n')}
 
-提供具体的审查示例，展示不同严重程度的问题和回复格式
+## 🔍 发现的问题汇总
 
----
+`;
 
-**输出要求：**
-1. 直接输出 reviewguard.md 的完整 Markdown 内容
-2. 内容必须用**中文**编写
-3. 必须针对该项目的技术栈定制审查规则
-4. 必须包含清晰的回复格式规范和评分标准
-5. 必须包含 3-5 个实际示例
-6. 内容要详细、实用、可操作
+  const filesWithIssues = Object.keys(commentsByFile);
+  
+  if (filesWithIssues.length === 0) {
+    prompt += `✅ **代码质量良好，未发现需要改进的问题。**\n\n`;
+  } else {
+    prompt += `共在 ${filesWithIssues.length} 个文件中发现 ${stats.successComments} 个问题：\n\n`;
+    
+    filesWithIssues.forEach((fileName, index) => {
+      const comments = commentsByFile[fileName];
+      prompt += `### ${index + 1}. ${fileName}\n\n`;
+      prompt += `发现 ${comments.length} 个问题\n\n`;
+    });
+  }
 
-请开始生成 reviewguard.md 的内容：`;
+  prompt += `---
 
-  return [
-    {
-      role: 'system',
-      content: `你是一个专业的代码审查规范设计专家。你的任务是根据项目的技术栈和代码结构，生成一份详细的 AI 代码审查规则文档（reviewguard.md）。
+请生成一个简洁、专业的 MR 总结评论。使用中文，内容控制在 500 字以内。`;
 
-这份文档将用于指导 AI 进行代码审查，必须：
-1. 针对具体技术栈定制（不要泛泛而谈）
-2. 包含清晰的严重程度评分标准（0-10分）
-3. 定义统一的回复格式（必须包含 emoji、评分、问题描述、影响分析、改进建议）
-4. 提供实际的审查示例
-5. 全部使用中文
+  return prompt;
+}
 
-生成的文档应该像一份专业的代码审查规范手册，让 AI 能够准确理解如何审查这个项目的代码。`
-    },
-    {
-      role: 'user',
-      content: userMessage
-    }
-  ];
+/**
+ * 格式化反馈样本信息（用于日志输出）
+ * @param {Object} samples - 反馈样本
+ * @returns {string} 格式化的信息
+ */
+export function formatFeedbackSamplesInfo(samples) {
+  if (!samples || (!samples.positive?.length && !samples.negative?.length)) {
+    return '无历史反馈样本';
+  }
+  return `正样本: ${samples.positive?.length || 0}, 负样本: ${samples.negative?.length || 0}`;
 }
 
 // 导出函数
@@ -592,8 +381,7 @@ export default {
   buildSystemPrompt,
   buildFileReviewPrompt,
   buildFileReviewMessages,
-  buildFileReviewWithImpactPrompt,
-  buildFileReviewWithImpactMessages,
   buildGenerateGuardPromptMessages,
+  buildMRSummaryPrompt,
+  formatFeedbackSamplesInfo,
 };
-