smart-review 1.0.1 → 1.0.2

package/bin/review.js

@@ -6,6 +6,7 @@ import { CodeReviewer } from '../lib/reviewer.js';
 import { ConfigLoader } from '../lib/config-loader.js';
 import { logger } from '../lib/utils/logger.js';
 import { BATCH_CONSTANTS } from '../lib/utils/constants.js';
+import { t, displayRisk } from '../lib/utils/i18n.js';
 
 class ReviewCLI {
   constructor() {
@@ -33,30 +34,31 @@ class ReviewCLI {
     const args = process.argv.slice(2);
     
     try {
-      logger.progress('代码审查启动中，请等待...');
       // 加载配置
       const configLoader = new ConfigLoader(this.projectRoot);
       const config = await configLoader.loadConfig();
+      this.config = config;
+      logger.progress(t(config, 'cli_start'));
       
       // 调试日志开关（命令行）
       if (args.includes('--debug')) {
         logger.debugMode = true;
-        logger.info('通过命令行参数启用调试日志模式');
+        logger.info(t(config, 'debug_enabled'));
       }
 
       // 处理AI相关的命令行参数
       if (args.includes('--no-ai')) {
         config.ai = { ...config.ai, enabled: false };
-        logger.info('通过命令行参数禁用AI分析');
+        logger.info(t(config, 'ai_disabled'));
       } else if (args.includes('--ai')) {
         config.ai = { ...config.ai, enabled: true };
-        logger.info('通过命令行参数启用AI分析');
+        logger.info(t(config, 'ai_enabled'));
       }
       
       // 处理Git Diff审查相关参数
       if (args.includes('--diff-only')) {
         config.ai = { ...config.ai, reviewOnlyChanges: true };
-        logger.info('通过命令行参数启用Git Diff增量审查模式');
+        logger.info(t(config, 'diff_only_enabled'));
       }
       
       const rules = await configLoader.loadRules(config);
@@ -71,33 +73,33 @@ class ReviewCLI {
         const fileList = args[filesIndex + 1]?.split(',').map(f => f.trim()) || [];
         result = await reviewer.reviewSpecificFiles(fileList);
       } else {
-        logger.info('使用方法:');
-        logger.info('  npx smart-code-reviewer --staged    # 审查暂存区文件');
-        logger.info('  npx smart-code-reviewer --staged --diff-only  # 仅审查暂存区变动内容(git diff)');
-        logger.info('  npx smart-code-reviewer --files file1,file2  # 审查指定文件');
+        logger.info(t(config, 'usage_header'));
+        logger.info(t(config, 'usage_staged'));
+        logger.info(t(config, 'usage_diffonly'));
+        logger.info(t(config, 'usage_files'));
         process.exit(1);
       }
       
-      this.printResults(result);
+      this.printResults(result, config);
       process.exit(result.blockSubmission ? 1 : 0);
       
     } catch (error) {
-      logger.error('审查执行失败:', error);
+      logger.error(t(this.config || process.env.SMART_REVIEW_LOCALE || 'zh-CN', 'review_error', { error: error.message }));
       process.exit(1);
     }
   }
 
-  printResults(result) {
+  printResults(result, config) {
     const staticIssues = result.issues.filter(i => i.source === 'static');
     const aiIssues = result.issues.filter(i => i.source === 'ai');
     // 本地规则审查结果
-    logger.info('\n本地规则审查结果');
+    logger.info('\n' + t(config, 'local_analysis_header'));
     const staticByFile = this.groupIssuesByFile(staticIssues);
       if (staticIssues.length === 0) {
-        logger.info('无');
+        logger.info(t(config, 'no_issues'));
       } else {
         Object.entries(staticByFile).forEach(([file, issues]) => {
-          logger.info(`\n文件: ${file}`);
+          logger.info('\n' + t(config, 'file_label', { file }));
           // 根据行号排序，保证位置从上到下
           const getLineKey = (i) => {
             const s = Number(i.lineStart);
@@ -110,36 +112,36 @@ class ReviewCLI {
           };
           const sorted = [...issues].sort((a, b) => getLineKey(a) - getLineKey(b));
           sorted.forEach((issue, index) => {
-            logger.info(`\n问题${index + 1}:`);
-            const locationLabel = this.formatLocationLabel(issue);
+            logger.info('\n' + t(config, 'issue_label', { index: index + 1 }));
+            const locationLabel = this.formatLocationLabel(issue, config);
             if (locationLabel) logger.info(locationLabel);
             // 美化代码片段输出：去除行号前缀并统一缩进
             if (issue.snippet && issue.snippet.trim().length > 0) {
-              logger.info('代码片段：');
+              logger.info(t(config, 'snippet_label'));
               logger.info(this.formatSnippet(issue.snippet));
             } else {
-              logger.info('代码片段：(全局性问题)');
+              logger.info(t(config, 'snippet_global_label'));
             }
-            logger.info(`风险等级：${this.getRiskLevelText(issue.risk)}`);
-            logger.info(`风险原因：${issue.message}`);
-            if (issue.suggestion) logger.info(`修改建议：${issue.suggestion}`);
+            logger.info(t(config, 'risk_level_label') + displayRisk(issue.risk, config));
+            logger.info(t(config, 'risk_reason_label') + issue.message);
+            if (issue.suggestion) logger.info(t(config, 'suggestions_label') + issue.suggestion);
           });
         });
       }
 
     // AI代码分析结果（若有）
     if (result.aiRan) {
-      logger.info('\nAI代码分析结果');
+      logger.info('\n' + t(config, 'ai_analysis_header'));
       // 说明：行号可能不连续是预处理所致（剥离注释/无需审查片段），请忽略行号跳跃
-      logger.info('提示：片段行号为源文件绝对行号，因清洗注释/无需审查片段可能出现跳跃，请忽略行号不连续。');
+      logger.info(t(config, 'tip_line_numbers'));
       // 去重：按 file+line+message 进行去重，避免重复输出
       // 打印时不再对 AI 结果做粗略去重（聚合逻辑已在 reviewer.generateResult 中完成），仅分文件展示
       const aiByFile = this.groupIssuesByFile(aiIssues);
       if (aiIssues.length === 0) {
-        logger.info('无');
+        logger.info(t(config, 'no_issues'));
       } else {
         Object.entries(aiByFile).forEach(([file, issues]) => {
-          logger.info(`\n文件: ${file}`);
+          logger.info('\n' + t(config, 'file_label', { file }));
           // 根据行号排序：起始行号优先，其次单行号；无行号的排后
           const getLineKey = (i) => {
             const s = Number(i.lineStart);
@@ -152,19 +154,19 @@ class ReviewCLI {
           };
           const sorted = [...issues].sort((a, b) => getLineKey(a) - getLineKey(b));
           sorted.forEach((issue, index) => {
-            logger.info(`\n问题${index + 1}:`);
-            const locationLabel = this.formatLocationLabel(issue);
+            logger.info('\n' + t(config, 'issue_label', { index: index + 1 }));
+            const locationLabel = this.formatLocationLabel(issue, config);
             if (locationLabel) logger.info(locationLabel);
             // 美化代码片段输出：去除行号前缀并统一缩进
             if (issue.snippet && issue.snippet.trim().length > 0) {
-              logger.info('代码片段：');
+              logger.info(t(config, 'snippet_label'));
               logger.info(this.formatSnippet(issue.snippet));
             } else {
-              logger.info('代码片段：(全局性问题)');
+              logger.info(t(config, 'snippet_global_label'));
             }
-            logger.info(`风险等级：${this.getRiskLevelText(issue.risk)}`);
-            logger.info(`风险原因：${issue.message}`);
-            if (issue.suggestion) logger.info(`修改建议：${issue.suggestion}`);
+            logger.info(t(config, 'risk_level_label') + displayRisk(issue.risk, config));
+            logger.info(t(config, 'risk_reason_label') + issue.message);
+            if (issue.suggestion) logger.info(t(config, 'suggestions_label') + issue.suggestion);
           });
         });
       }
@@ -188,28 +190,21 @@ class ReviewCLI {
     }, {});
   }
 
-  getRiskLevelText(risk) {
-    const levels = {
-      'critical': '致命',
-      'high': '高危', 
-      'medium': '中危',
-      'low': '低危',
-      'suggestion': '建议'
-    };
-    return levels[risk] || risk;
+  getRiskLevelText(risk, config) {
+    return displayRisk(risk, config);
   }
 
   // 位置标签：范围为“行号范围：start-end”，单行为“行号：n”
-  formatLocationLabel(issue) {
+  formatLocationLabel(issue, config) {
     const start = Number(issue.lineStart);
     const end = Number(issue.lineEnd);
     const single = Number(issue.line);
     if (Number.isFinite(start) && Number.isFinite(end) && start > 0 && end >= start) {
-      if (start === end) return `行号：${start}`;
-      return `行号范围：${start}-${end}`;
+      if (start === end) return t(config, 'line_label', { line: start });
+      return t(config, 'line_range_label', { start, end });
     }
     if (Number.isFinite(single) && single > 0) {
-      return `行号：${single}`;
+      return t(config, 'line_label', { line: single });
     }
     return '';
   }
@@ -253,4 +248,4 @@ class ReviewCLI {
 
 // 运行审查
 const cli = new ReviewCLI();
-cli.run().catch(error => logger.error('CLI运行失败:', error));
+cli.run().catch(error => logger.error(t(process.env.SMART_REVIEW_LOCALE || 'zh-CN', 'review_error', { error: error.message })));

package/index.js

@@ -2,4 +2,3 @@ export { CodeReviewer } from './lib/reviewer.js';
 export { AIClient } from './lib/ai-client.js';
 export { ConfigLoader } from './lib/config-loader.js';
 export { defaultConfig, defaultRules } from './lib/default-config.js';
-// https://ci.das-security.cn/repository/ah_npm

package/lib/ai-client-pool.js

@@ -3,9 +3,9 @@
  * 管理多个AI客户端实例，支持并发请求处理
  */
 
-import path from 'path';
 import { AIClient } from './ai-client.js';
 import { logger } from './utils/logger.js';
+import { t } from './utils/i18n.js';
 
 export class AIClientPool {
   constructor(config, rules, poolSize = 3, concurrencyLimiter = null) {
@@ -30,10 +30,15 @@ export class AIClientPool {
    * 初始化客户端池
    */
   initializePool() {
-    logger.debug(`初始化AI客户端池，大小: ${this.poolSize}`);
+    logger.debug(t(this.config, 'init_pool_dbg', { size: this.poolSize }));
     
     for (let i = 0; i < this.poolSize; i++) {
-      const client = new AIClient(this.config.ai);
+      // 将 reviewDir 与 locale 一并传递，确保池中客户端的国际化与自定义提示词目录正确
+      const client = new AIClient({
+        ...this.config.ai,
+        reviewDir: this.config.reviewDir,
+        locale: this.config.locale
+      });
       client.poolId = i;
       // 注入全局并发限速器，确保批次与分段共享并发资源
       if (this.concurrencyLimiter) {
@@ -42,7 +47,7 @@ export class AIClientPool {
       this.clients.push(client);
     }
     
-    logger.debug(`AI客户端池初始化完成，共${this.clients.length}个客户端`);
+    logger.debug(t(this.config, 'pool_init_done_dbg', { count: this.clients.length }));
   }
 
   /**
@@ -142,7 +147,7 @@ export class AIClientPool {
          successCount++;
        } catch (error) {
          failureCount++;
-         logger.error(`批次 ${batch.originalIndex + 1} 处理失败: ${error.message}`);
+         logger.error(t(this.config, 'batch_process_failed', { i: batch.originalIndex + 1, error: error.message }));
        }
      }
     
@@ -165,7 +170,7 @@ export class AIClientPool {
       return [];
     }
 
-    logger.debug(`开始并发处理 ${batches.length} 个批次`);
+    logger.debug(t(this.config, 'start_concurrent_dbg', { count: batches.length }));
     
     // 设置总请求数用于进度显示
     this.stats.totalRequests = batches.length;
@@ -198,15 +203,15 @@ export class AIClientPool {
           }
         } else {
           failureCount++;
-          logger.error(`文件组 ${index + 1} 处理失败: ${result.reason.message}`);
+          logger.error(t(this.config, 'concurrent_group_failed', { index: index + 1, error: result.reason?.message || String(result.reason) }));
         }
       });
 
-      logger.debug(`并发处理完成: 成功 ${successCount}, 失败 ${failureCount}`);
+      logger.debug(t(this.config, 'concurrent_done_dbg', { succ: successCount, fail: failureCount }));
       return { issues: allIssues, totalDurationMs };
       
     } catch (error) {
-      logger.error(`并发处理过程中发生错误: ${error.message}`);
+      logger.error(t(this.config, 'concurrent_processing_error', { error: error?.message || String(error) }));
       throw error;
     }
   }
@@ -254,7 +259,13 @@ export class AIClientPool {
                 if (Array.isArray(parsed)) issueCount = parsed.length;
                 else if (parsed && Array.isArray(parsed.issues)) issueCount = parsed.issues.length;
               } catch (e) {}
-              logger.success(`批次 ${batchIndex + 1}/${this.stats.totalRequests}（${fileNames}）分析完成，发现 ${issueCount} 个问题，耗时 ${(duration/1000).toFixed(1)}秒`);
+      logger.success(t(this.config, 'batch_complete', {
+        i: batchIndex + 1,
+        total: this.stats.totalRequests,
+        context: fileNames,
+        issues: issueCount,
+        secs: (duration/1000).toFixed(1)
+      }));
               if (typeof progressCallback === 'function') {
                 try { progressCallback(batchIndex, batch, 'completed', null); } catch (e) {}
               }
@@ -268,13 +279,24 @@ export class AIClientPool {
           const item = batch.items[0];
           const fullPath = item.originalFilePath || item.filePath;
           const totalSeg = item.totalChunks || batch.totalSegments || 1;
-          logger.info(`开始分析第 ${batchIndex + 1}/${this.stats.totalRequests} 批次（分段整体），文件: ${fullPath}，共 ${totalSeg} 段`);
+      logger.info(t(this.config, 'batch_start_segmented', {
+        i: batchIndex + 1,
+        total: this.stats.totalRequests,
+        path: fullPath,
+        segments: totalSeg
+      }));
         } else {
           // 小文件批次：保持一致的“批次 i/x”格式
-          logger.info(`开始分析第 ${batchIndex + 1}/${this.stats.totalRequests} 批次，文件: ${fileNames}，预估${batch.totalTokens} tokens, 共${batch.items.length}个文件`);
+      logger.info(t(this.config, 'batch_start_regular', {
+        i: batchIndex + 1,
+        total: this.stats.totalRequests,
+        files: fileNames,
+        tokens: batch.totalTokens,
+        count: batch.items.length
+      }));
         }
         const startTime = Date.now();
-        logger.debug(`🔍 开始AI分析请求，批次 ${batchIndex + 1}，使用客户端: ${client.constructor.name}`);
+        logger.debug(t(this.config, 'request_batch_start_dbg', { index: batchIndex + 1, client: client.constructor.name }));
         
         const result = await client.analyzeSmartBatch(formattedBatch, batch, requestMeta);
         
@@ -286,9 +308,21 @@ export class AIClientPool {
           if (batch.isLargeFileSegment) {
             const item = batch.items[0];
             const fullPath = item.originalFilePath || item.filePath;
-            logger.success(`批次 ${batchIndex + 1}/${this.stats.totalRequests}（${fullPath}）分析完成，发现 ${issueCount} 个问题，耗时 ${(duration/1000).toFixed(1)}秒`);
+      logger.success(t(this.config, 'batch_complete', {
+        i: batchIndex + 1,
+        total: this.stats.totalRequests,
+        context: fullPath,
+        issues: issueCount,
+        secs: (duration/1000).toFixed(1)
+      }));
           } else {
-            logger.success(`批次 ${batchIndex + 1}/${this.stats.totalRequests}（${fileNames}）分析完成，发现 ${issueCount} 个问题，耗时 ${(duration/1000).toFixed(1)}秒`);
+      logger.success(t(this.config, 'batch_complete', {
+        i: batchIndex + 1,
+        total: this.stats.totalRequests,
+        context: fileNames,
+        issues: issueCount,
+        secs: (duration/1000).toFixed(1)
+      }));
           }
         }
         
@@ -296,19 +330,19 @@ export class AIClientPool {
           if (batch.isLargeFileSegment) {
             const item = batch.items[0];
             const fullPath = item.originalFilePath || item.filePath;
-            logger.debug(`⚠️  ${fullPath} 第 ${item.chunkIndex + 1}/${item.totalChunks} 段未发现问题，AI响应内容: ${JSON.stringify(result).substring(0, 200)}...`);
+            logger.debug(t(this.config, 'seg_chunk_no_issues_dbg', { file: fullPath, chunk: item.chunkIndex + 1, total: item.totalChunks, preview: JSON.stringify(result).substring(0, 200) + '...' }));
           } else {
             const fileNamesAbs = batch.items.map(item => (item.originalFilePath || item.filePath)).join(', ');
-            logger.debug(`⚠️  ${fileNamesAbs} 未发现问题，AI响应内容: ${JSON.stringify(result).substring(0, 200)}...`);
+            logger.debug(t(this.config, 'files_no_issues_dbg', { files: fileNamesAbs, preview: JSON.stringify(result).substring(0, 200) + '...' }));
           }
         } else {
             if (batch.isLargeFileSegment) {
               const item = batch.items[0];
               const fullPath = item.originalFilePath || item.filePath;
-              logger.debug(`📋 ${fullPath} 第 ${item.chunkIndex + 1}/${item.totalChunks} 段发现的问题风险等级: ${result.issues.map(i => i.risk || 'unknown').join(', ')}`);
+              logger.debug(t(this.config, 'issues_risk_levels_dbg', { file: fullPath, levels: result.issues.map(i => i.risk || 'unknown').join(', ') }));
             } else {
               const fileNamesAbs = batch.items.map(item => (item.originalFilePath || item.filePath)).join(', ');
-              logger.debug(`📋 ${fileNamesAbs} 发现的问题风险等级: ${result.issues.map(i => i.risk || 'unknown').join(', ')}`);
+              logger.debug(t(this.config, 'issues_risk_levels_dbg', { file: fileNamesAbs, levels: result.issues.map(i => i.risk || 'unknown').join(', ') }));
             }
           }
         
@@ -317,13 +351,13 @@ export class AIClientPool {
           if (batch.isLargeFileSegment) {
             const item = batch.items[0];
             const fullPath = item.originalFilePath || item.filePath;
-            logger.debug(`${fullPath} 第 ${item.chunkIndex + 1}/${item.totalChunks} 段发现的问题详情:`);
+            logger.debug(t(this.config, 'issues_details_dbg', { file: fullPath }));
           } else {
             const fileNamesAbs = batch.items.map(item => (item.originalFilePath || item.filePath)).join(', ');
-            logger.debug(`${fileNamesAbs} 发现的问题详情:`);
+            logger.debug(t(this.config, 'issues_details_dbg', { file: fileNamesAbs }));
           }
           result.issues.forEach((issue, idx) => {
-            logger.debug(`  问题 ${idx + 1}: ${issue.risk} - ${issue.message?.slice(0, 100)}...`);
+            logger.debug(t(this.config, 'issue_item_dbg', { index: idx + 1, risk: issue.risk, message: (issue.message || '').slice(0, 100) + '...' }));
           });
         }
         
@@ -343,7 +377,11 @@ export class AIClientPool {
       this.stats.retryCount++;
       
       if (retryCount < maxRetries) {
-        logger.warn(`批次 ${batchIndex + 1} 处理失败，进行第 ${retryCount + 1} 次重试: ${error.message}`);
+      logger.warn(t(this.config, 'batch_retry_warn', {
+        i: batchIndex + 1,
+        retry: retryCount + 1,
+        error: error.message
+      }));
         
         // 指数退避延迟
         const delay = Math.pow(2, retryCount) * 1000;
@@ -351,7 +389,7 @@ export class AIClientPool {
         
         return this.processBatchWithRetry(batch, batchIndex, progressCallback, retryCount + 1);
       } else {
-        logger.error(`批次 ${batchIndex + 1} 重试 ${maxRetries} 次后仍然失败: ${error.message}`);
+        logger.error(t(this.config, 'batch_retry_error', { i: batchIndex + 1, max: maxRetries, error: error.message }));
         
         if (progressCallback) {
           progressCallback(batchIndex, batch, 'failed', error);
@@ -405,7 +443,7 @@ export class AIClientPool {
    * 清理资源
    */
   cleanup() {
-    logger.debug('清理AI客户端池资源');
+    logger.debug(t(this.config, 'cleanup_pool_dbg'));
     
     // 清理等待队列
     this.waitingQueue.forEach(resolve => {