smart-review 1.0.1 → 1.0.3

package/lib/segmented-analyzer.js

@@ -6,6 +6,7 @@
 import { stripCommentsForAI, stripNoReviewForAI } from './utils/strip.js';
 import { logger } from './utils/logger.js';
 import { SEGMENTATION_CONSTANTS, AI_CONSTANTS } from './utils/constants.js';
+import { t, getLocale, FIELD_LABELS, buildPrompts } from './utils/i18n.js';
 
 export class SegmentedAnalyzer {
   constructor(config = {}) {
@@ -37,11 +38,9 @@ export class SegmentedAnalyzer {
    */
   async analyzeFileSegmented(filePath, content, aiClient, customPrompt = '', staticIssues = []) {
     try {
-      logger.progress(`开始分段分析文件: ${filePath}`);
-      
       // 第一步：智能分段
       const segments = this.createIntelligentSegments(content, filePath);
-      logger.info(`文件分为 ${segments.length} 段`);
+      logger.progress(t(this.config, 'segment_overall_start', { file: filePath, total: segments.length, concurrency: '', totalNote: '' }));
       
       const allIssues = [];
       const segmentSummaries = []; // 存储每段的摘要
@@ -50,8 +49,8 @@ export class SegmentedAnalyzer {
       // 第二步：逐段分析
       for (let i = 0; i < segments.length; i++) {
         const segment = segments[i];
-        logger.info(`开始分析第 ${i + 1}/${segments.length} 段 (行 ${segment.startLine}-${segment.endLine})`);
-        logger.info(` 预估${segment.tokens} tokens, 共${segment.endLine - segment.startLine + 1} 行代码`);
+        logger.info(t(this.config, 'segment_start_label', { file: filePath, index: i + 1, total: segments.length, start: segment.startLine, end: segment.endLine }));
+        logger.debug(t(this.config, 'segment_size_estimate_dbg', { tokens: segment.tokens, lines: (segment.endLine - segment.startLine + 1) }));
         
         try {
           // 分析当前段
@@ -69,9 +68,9 @@ export class SegmentedAnalyzer {
           // 收集问题
           if (segmentResult.issues && segmentResult.issues.length > 0) {
             allIssues.push(...segmentResult.issues);
-            logger.success(`第 ${i + 1} 段分析完成，发现 ${segmentResult.issues.length} 个问题`);
+            logger.success(t(this.config, 'segment_analysis_done_n_issues', { batch: '', file: filePath, index: i + 1, count: segmentResult.issues.length }));
           } else {
-            logger.success(`第 ${i + 1} 段分析完成，未发现问题`);
+            logger.success(t(this.config, 'segment_analysis_done_zero', { batch: '', file: filePath, index: i + 1 }));
           }
           
           // 收集摘要（如果启用）
@@ -84,13 +83,14 @@ export class SegmentedAnalyzer {
             
             // 更新上下文（保留最近几段的摘要）
             const recentSummaries = segmentSummaries.slice(-3); // 保留最近3段摘要
-            previousContext = recentSummaries.map(s => 
-              `第${s.segmentIndex}段(行${s.lineRange}): ${s.summary}`
-            ).join('\n');
+            const loc = getLocale(this.config);
+            previousContext = [t(this.config, 'segment_prev_context_header'),
+              ...recentSummaries.map(s => t(this.config, 'segment_prev_context_item', { index: s.segmentIndex, range: s.lineRange, summary: s.summary }))
+            ].join('\n');
           }
           
         } catch (error) {
-          logger.error(`第 ${i + 1} 段分析失败: ${error.message}`);
+          logger.error(t(this.config, 'segment_analysis_failed', { index: i + 1, error: error?.message || String(error) }));
           // 继续分析下一段，不中断整个流程
         }
       }
@@ -106,7 +106,7 @@ export class SegmentedAnalyzer {
       };
       
     } catch (error) {
-      logger.error(`分段分析失败: ${error.message}`);
+      logger.error(t(this.config, 'segment_file_failed', { error: error?.message || String(error) }));
       throw error;
     }
   }
@@ -227,13 +227,13 @@ export class SegmentedAnalyzer {
    */
   async analyzeSegment(segment, filePath, aiClient, customPrompt, staticIssues, previousContext, currentSegmentNum, totalSegments) {
     try {
-      logger.debug(` 准备第 ${currentSegmentNum} 段代码内容...`);
+      logger.debug(t(this.config, 'segment_prepare_content_dbg', { index: currentSegmentNum }));
       
       // 准备代码内容
       const prepared = await stripNoReviewForAI(segment.content, filePath);
       const clean = await stripCommentsForAI(prepared, filePath);
       
-      logger.debug(` 构建第 ${currentSegmentNum} 段分析提示词...`);
+      logger.debug(t(this.config, 'segment_build_prompt_dbg', { index: currentSegmentNum }));
       
       // 构建分段分析提示词
       const segmentPrompt = this.buildSegmentPrompt(
@@ -267,12 +267,12 @@ export class SegmentedAnalyzer {
       // 解析AI响应
       const result = this.parseSegmentResponse(responseContent, filePath, segment);
       
-      logger.info(` 第 ${currentSegmentNum} 段响应解析完成，发现 ${result.issues ? result.issues.length : 0} 个问题`);
+      logger.info(t(this.config, 'segment_response_parsed_info', { index: currentSegmentNum, count: (result.issues ? result.issues.length : 0) }));
       
       return result;
       
     } catch (error) {
-      logger.error(`第 ${currentSegmentNum} 段分析失败: ${error.message}`);
+      logger.error(t(this.config, 'segment_analysis_failed', { index: currentSegmentNum, error: error?.message || String(error) }));
       throw error;
     }
   }
@@ -281,61 +281,48 @@ export class SegmentedAnalyzer {
    * 构建分段分析提示词
    */
   buildSegmentPrompt(segment, filePath, cleanContent, customPrompt, staticIssues, previousContext, currentSegmentNum, totalSegments) {
-    let prompt = `我正在对一个大文件进行分段代码审查。以下是第 ${currentSegmentNum}/${totalSegments} 段的内容：
-
-**文件路径**: ${filePath}
-**当前分段**: 第 ${currentSegmentNum}/${totalSegments} 段
-**代码行范围**: ${segment.startLine}-${segment.endLine}`;
-
-    // 添加上下文说明
-    if (segment.contextInfo.hasPreContext || segment.contextInfo.hasPostContext) {
-      prompt += `\n**上下文说明**: `;
-      if (segment.contextInfo.hasPreContext) {
-        prompt += `前 ${segment.contextInfo.preContextLines} 行为上文上下文`;
-      }
-      if (segment.contextInfo.hasPostContext) {
-        if (segment.contextInfo.hasPreContext) prompt += `，`;
-        prompt += `后 ${segment.contextInfo.postContextLines} 行为下文上下文`;
-      }
-      prompt += `，这些上下文行仅用于帮助理解代码逻辑，请避免对重叠部分重复报告问题。`;
-    }
+    const loc = getLocale(this.config);
+    const L = FIELD_LABELS[loc];
+    let prompt = t(this.config, 'segment_prompt_template', {
+      index: currentSegmentNum,
+      total: totalSegments,
+      Lfile: L.file,
+      Lcontent: L.content,
+      file: filePath,
+      content: cleanContent
+    });
 
-    // 添加前面段落的摘要上下文
+    // 追加前面段落摘要上下文
     if (previousContext && this.config.enableSummaryContext) {
-      prompt += `\n\n**前面段落摘要**:\n${previousContext}`;
+      prompt += `\n\n${previousContext}`;
     }
 
-    // 添加代码内容
-    prompt += `\n\n**代码内容**:\n\`\`\`\n${cleanContent}\n\`\`\``;
-
-    // 添加静态问题提示
+    // 添加静态问题提示（仅与当前段相关）
     if (staticIssues && staticIssues.length > 0) {
-      const relevantStaticIssues = staticIssues.filter(issue => {
-        // 简单的行号匹配，实际可以更精确
-        return issue.line >= segment.startLine && issue.line <= segment.endLine;
-      });
-      
+      const relevantStaticIssues = staticIssues.filter(issue => issue.line >= segment.startLine && issue.line <= segment.endLine);
       if (relevantStaticIssues.length > 0) {
-        prompt += `\n\n**本段相关的静态检测问题**:\n`;
+        prompt += `\n\n${t(this.config, 'segment_static_issues_header', { index: currentSegmentNum })}\n`;
         relevantStaticIssues.forEach((issue, idx) => {
-          prompt += `${idx + 1}. 第${issue.line}行 (${issue.risk}): ${issue.message}\n`;
+          const riskDisp = issue.risk;
+          const snippetLabel = L.snippet;
+          const suggest = issue.suggestion ? ` ${issue.suggestion}` : '';
+          prompt += t(this.config, 'segment_static_issue_line', {
+            index: idx + 1,
+            risk: riskDisp,
+            message: issue.message,
+            suggest,
+            snippetLabel,
+            snippet: issue.snippet || ''
+          }) + '\n';
         });
       }
     }
 
-    // 添加自定义提示词
+    // 追加自定义提示词
     if (customPrompt) {
-      prompt += `\n\n**自定义审查要求**:\n${customPrompt}`;
+      prompt += `\n\n${customPrompt}`;
     }
 
-    // 添加分段分析要求
-    prompt += `\n\n**分段分析要求**:
-1. 请仅分析当前分段的代码，不要分析上下文行中的问题
-2. 如果启用摘要功能，请在分析结果后提供一个简短的代码摘要
-3. 严格按照指定格式返回结果
-4. 对于跨段的问题，请在当前段中标注，后续段会参考前面的摘要进行综合判断
-5. 分段上下文限制：不要评估“导入是否被使用”，严格忽略关于“未使用的导入/模块/依赖”的任何提示或建议（包括建议删除未使用的导入）`;
-
     return prompt;
   }
 
@@ -343,52 +330,7 @@ export class SegmentedAnalyzer {
    * 获取分段分析的系统提示词
    */
   getSegmentSystemPrompt() {
-    let systemPrompt = `你是一个专业的代码审查专家，正在对大文件进行分段分析。
-
-**分段分析特点**:
-- 你收到的是文件的一个片段，可能包含上下文行
-- 上下文行仅用于理解代码逻辑，不应对其报告问题
-- 需要考虑代码的连续性和上下文关系
-- 避免对重叠部分重复报告问题
-
-**输出格式要求**:
-请严格按照以下格式返回分析结果：
-
-**-----代码分析结果-----**
-文件路径：{文件路径}
-代码片段：{具体的问题代码片段}
-风险等级：{致命/高危/中危/低危/建议}
-风险原因：{详细原因}
-修改建议：{具体建议}
-
-[如果有多个问题，用空行分隔]`;
-
-    // 如果启用摘要功能，添加摘要要求
-    if (this.config.enableSummaryContext) {
-      systemPrompt += `
-
-**摘要要求**:
-在所有问题分析完成后，请提供一个简短摘要：
-
-**-----段落摘要-----**
-{简要描述这段代码的主要功能、关键逻辑和重要特征，控制在${this.config.maxSummaryLength}字符以内}`;
-    }
-
-    systemPrompt += `
-
-**风险等级定义**:
-- 致命：可能导致系统崩溃、数据丢失、严重安全漏洞
-- 高危：可能导致安全漏洞、数据泄露、业务逻辑错误  
-- 中危：可能影响系统稳定性、性能问题、用户体验
-- 低危：代码质量问题、不符合最佳实践
-- 建议：改进建议，提升代码质量`;
-
-    systemPrompt += `
-
-**分段场景的忽略规则（必须遵守）**:
-- 由于其它段可能引用当前段的导入，请不要评估或报告“未使用的导入/模块/依赖”相关问题；即使当前片段内未见引用，也不要建议移除该导入。
-- 仅在本段内可以明确识别的实际错误或风险时再输出问题。`;
-
+    const { systemPrompt } = buildPrompts(this.config);
     return systemPrompt;
   }
 
@@ -400,19 +342,51 @@ export class SegmentedAnalyzer {
     let summary = '';
     
     try {
-      // 分离问题分析和摘要
-      const parts = responseContent.split('**-----段落摘要-----**');
-      const analysisContent = parts[0];
-      const summaryContent = parts[1];
+      // 分离问题分析和摘要（支持中英文摘要标记）
+      const summaryMarkerZh = t(this.config, 'segment_summary_marker');
+      const summaryMarkerEn = '**-----Segment Summary-----**';
+      let analysisContent = responseContent;
+      let summaryContent = '';
+      if (responseContent.includes(summaryMarkerZh)) {
+        const parts = responseContent.split(summaryMarkerZh);
+        analysisContent = parts[0];
+        summaryContent = parts[1];
+      } else if (responseContent.includes(summaryMarkerEn)) {
+        const parts = responseContent.split(summaryMarkerEn);
+        analysisContent = parts[0];
+        summaryContent = parts[1];
+      }
       
-      // 解析问题
-      if (analysisContent.includes('**-----代码分析结果-----**')) {
-        const problemSections = analysisContent.split('**-----代码分析结果-----**').slice(1);
-        
-        for (const section of problemSections) {
-          const issue = this.parseIssueSection(section.trim(), filePath, segment);
-          if (issue) {
-            issues.push(issue);
+      // 解析问题（支持多种标记：开始/结束 或 单标记 中英文）
+      const startEndMarkers = [
+        { start: '**-----代码分析结果开始-----**', end: '**-----代码分析结果结束-----**' },
+        { start: '**-----Code Analysis Result Start-----**', end: '**-----Code Analysis Result End-----**' },
+      ];
+      let parsedByStartEnd = false;
+      for (const m of startEndMarkers) {
+        if (analysisContent.includes(m.start) && analysisContent.includes(m.end)) {
+          parsedByStartEnd = true;
+          const regex = new RegExp(`${m.start}([\s\S]*?)${m.end}`, 'g');
+          const matches = Array.from(analysisContent.matchAll(regex));
+          for (const match of matches) {
+            const section = match[1].trim();
+            const issue = this.parseIssueSection(section, filePath, segment);
+            if (issue) issues.push(issue);
+          }
+          break;
+        }
+      }
+
+      if (!parsedByStartEnd) {
+        const singleMarkers = ['**-----代码分析结果-----**', '**-----Code Review Result-----**'];
+        for (const marker of singleMarkers) {
+          if (analysisContent.includes(marker)) {
+            const problemSections = analysisContent.split(marker).slice(1);
+            for (const section of problemSections) {
+              const issue = this.parseIssueSection(section.trim(), filePath, segment);
+              if (issue) issues.push(issue);
+            }
+            break;
           }
         }
       }
@@ -423,7 +397,7 @@ export class SegmentedAnalyzer {
       }
       
     } catch (error) {
-      logger.warn(`解析AI响应失败: ${error.message}`);
+      logger.warn(t(this.config, 'parse_seg_response_failed_warn', { error: error?.message || String(error) }));
     }
     
     return {
@@ -438,6 +412,8 @@ export class SegmentedAnalyzer {
   parseIssueSection(section, filePath, segment) {
     try {
       const lines = section.split('\n').filter(line => line.trim());
+      const loc = getLocale(this.config);
+      const L = FIELD_LABELS[loc];
       
       let issue = {
         file: filePath,
@@ -447,16 +423,16 @@ export class SegmentedAnalyzer {
       };
       
       for (const line of lines) {
-        if (line.startsWith('文件路径：')) {
+        if (line.startsWith(L.file)) {
           // 文件路径已知，跳过
-        } else if (line.startsWith('代码片段：')) {
-          issue.snippet = line.substring('代码片段：'.length).trim();
-        } else if (line.startsWith('风险等级：')) {
-          issue.risk = line.substring('风险等级：'.length).trim();
-        } else if (line.startsWith('风险原因：')) {
-          issue.message = line.substring('风险原因：'.length).trim();
-        } else if (line.startsWith('修改建议：')) {
-          issue.suggestion = line.substring('修改建议：'.length).trim();
+        } else if (line.startsWith(L.snippet)) {
+          issue.snippet = line.substring(L.snippet.length).trim();
+        } else if (line.startsWith(L.risk)) {
+          issue.risk = line.substring(L.risk.length).trim();
+        } else if (line.startsWith(L.reason)) {
+          issue.message = line.substring(L.reason.length).trim();
+        } else if (line.startsWith(L.suggestion)) {
+          issue.suggestion = line.substring(L.suggestion.length).trim();
         }
       }
       
@@ -466,7 +442,7 @@ export class SegmentedAnalyzer {
       }
       
     } catch (error) {
-      logger.warn(`解析问题段落失败: ${error.message}`);
+      logger.warn(t(this.config, 'parse_issue_section_failed_warn', { error: error?.message || String(error) }));
     }
     
     return null;

package/lib/utils/git-diff-parser.js

@@ -2,6 +2,7 @@ import { exec } from 'child_process';
 import { promisify } from 'util';
 import path from 'path';
 import { logger } from './logger.js';
+import { t } from './i18n.js';
 
 const execAsync = promisify(exec);
 
@@ -28,7 +29,7 @@ export class GitDiffParser {
       });
       return stdout;
     } catch (error) {
-      logger.error('获取git diff失败:', error);
+      logger.error(t(this.config, 'git_diff_parse_failed', { error: error?.message || String(error) }));
       return '';
     }
   }
@@ -81,7 +82,7 @@ export class GitDiffParser {
 
     // 检查文件是否应该被忽略
     if (!this.isReviewableFile(filePath)) {
-      logger.debug(`文件被忽略规则跳过: ${filePath}`);
+      logger.debug(t(this.config, 'file_skipped_by_ignore_rule_dbg', { file: filePath }));
       return null;
     }
 
@@ -259,7 +260,7 @@ export class GitDiffParser {
       smartSegments.push(this.finalizeSegment(currentSegment));
     }
 
-    logger.debug(`文件 ${filePath} 智能分段完成: ${sections.length} 个原始段 -> ${smartSegments.length} 个智能段`);
+    logger.debug(t(this.config, 'smart_segmentation_done_dbg', { file: filePath, raw: sections.length, smart: smartSegments.length }));
     return smartSegments;
   }
 
@@ -309,7 +310,7 @@ export class GitDiffParser {
     const reviewLineStrings = reviewItems.map(it => (it.type === '+' ? '+' : ' ') + it.content);
     const filteredLines = this.filterDisabledLines(reviewLineStrings, addedLineNumbers);
     if (filteredLines.addedLineNumbers.length === 0) {
-      logger.debug(`所有新增行都被代码内指令忽略: ${filePath}`);
+      logger.debug(t(this.config, 'all_added_lines_ignored_dbg', { file: filePath }));
       return null;
     }
 
@@ -363,7 +364,7 @@ export class GitDiffParser {
       });
       return stdout;
     } catch (error) {
-      logger.debug(`无法获取暂存区文件内容 ${filePath}:`, error.message);
+      logger.debug(t(this.config, 'get_staged_file_failed_dbg', { file: filePath, error: error?.message || String(error) }));
       return '';
     }
   }
@@ -373,11 +374,11 @@ export class GitDiffParser {
    * @returns {Promise<Array>} 审查数据数组
    */
   async getStagedDiffReviewData() {
-    logger.progress('正在分析暂存区变更...');
+    logger.progress(t(this.config, 'analyze_staged_changes_progress'));
     
     const diffOutput = await this.getStagedDiff();
     if (!diffOutput.trim()) {
-      logger.info('暂存区没有代码变更需要审查');
+      logger.info(t(this.config, 'no_changes_skip'));
       return [];
     }
 
@@ -398,7 +399,7 @@ export class GitDiffParser {
       }
     }
 
-    logger.info(`发现 ${reviewData.length} 个文件有代码变更需要审查`);
+    logger.info(t(this.config, 'found_changed_files_n', { count: reviewData.length }));
     return reviewData;
   }