job51-gitlab-cr-node-jt-1 3.0.8 → 3.1.1

package/README.md

@@ -46,6 +46,9 @@ gitlab-cr
 - 使用 Claude AI 进行代码审查
 - 生成结构化的审查报告
 - 将审查结果发布到 GitLab MR
+- 自动跳过测试文件（如 `*Test.java`、`*.test.js` 等）
+- 自动跳过 DTO/VO 文件（如 `*Dto.java`、`*VO.java`、`*Request.java` 等）
+- 自动跳过非代码文件（如配置文件、文档文件、资源文件等）
 
 ## 依赖要求
 

package/index.js

@@ -83,6 +83,228 @@ class GitLabCodeReviewer {
     return false;
   }
 
+  /**
+   * 判断文件是否是 DTO/VO 类文件（需要跳过审查）
+   * DTO（Data Transfer Object）和 VO（Value Object）通常是纯数据载体，不含业务逻辑
+   * @param {string} filePath 文件路径
+   * @returns {boolean} 是否是 DTO/VO 文件
+   */
+  isDtoVoFile(filePath) {
+    if (!filePath) return false;
+
+    // 标准化路径（统一使用 / 分隔符）
+    const normalizedPath = filePath.replace(/\\/g, '/');
+
+    // 提取文件名（不含扩展名）
+    const fileName = normalizedPath.split('/').pop() || '';
+
+    // DTO/VO 目录模式（整个目录都是数据模型）
+    const dtoVoDirPatterns = [
+      '/dto/',            // Java: com/example/dto/
+      '/DTO/',            // 大写形式
+      '/vo/',             // Java: com/example/vo/
+      '/VO/',             // 大写形式
+      '/model/',          // 通用模型目录
+      '/models/',         // 通用模型目录
+      '/entity/',         // 实体类目录（有时也是纯数据）
+      '/entities/',       // 实体类目录
+      '/pojo/',           // POJO 目录
+      '/beans/',          // Bean 目录
+    ];
+
+    // 检查是否包含 DTO/VO 目录路径
+    for (const pattern of dtoVoDirPatterns) {
+      if (normalizedPath.includes(pattern)) {
+        return true;
+      }
+    }
+
+    // DTO/VO 文件命名模式
+    // 匹配文件名中包含或以这些后缀结尾的情况
+    const dtoVoPatterns = [
+      'Dto',              // Java: XxxDto.java
+      'DTO',              // Java: XxxDTO.java
+      'Vo',               // Java: XxxVo.java
+      'VO',               // Java: XxxVO.java
+      'Request',          // Request 对象通常也是纯数据
+      'Response',         // Response 对象通常也是纯数据
+      'Form',             // Form 对象
+      'Bean',             // Bean 对象
+      'POJO',             // POJO 对象
+      'Pojo',             // Pojo 对象
+      'Entity',           // Entity 对象（如果是简单的数据实体）
+    ];
+
+    // 检查文件名是否包含 DTO/VO 模式
+    for (const pattern of dtoVoPatterns) {
+      // 文件名包含模式（如 CreateUserDto.java）
+      if (fileName.includes(pattern)) {
+        return true;
+      }
+    }
+
+    // Java 特殊处理：文件名以常见 DTO/VO 后缀结尾
+    const dtoVoEndPatterns = [
+      'Dto.java',
+      'DTO.java',
+      'Vo.java',
+      'VO.java',
+      'Request.java',
+      'Response.java',
+      'Form.java',
+      'Bean.java',
+      'Entity.java',
+      // Kotlin
+      'Dto.kt',
+      'DTO.kt',
+      'Vo.kt',
+      'VO.kt',
+      'Request.kt',
+      'Response.kt',
+      // TypeScript/JavaScript
+      'Dto.ts',
+      'Dto.js',
+      'DTO.ts',
+      'DTO.js',
+      'Vo.ts',
+      'Vo.js',
+      'VO.ts',
+      'VO.js',
+    ];
+
+    for (const pattern of dtoVoEndPatterns) {
+      if (fileName.endsWith(pattern)) {
+        return true;
+      }
+    }
+
+    return false;
+  }
+
+  /**
+   * 判断文件是否是非代码文件（需要跳过审查）
+   * 非代码文件包括：配置文件、文档文件、资源文件、数据文件等
+   * @param {string} filePath 文件路径
+   * @returns {boolean} 是否是非代码文件
+   */
+  isNonCodeFile(filePath) {
+    if (!filePath) return false;
+
+    // 标准化路径（统一使用 / 分隔符）
+    const normalizedPath = filePath.replace(/\\/g, '/');
+
+    // 提取文件名和扩展名
+    const fileName = normalizedPath.split('/').pop() || '';
+    const ext = fileName.split('.').pop()?.toLowerCase() || '';
+
+    // 非代码文件扩展名列表
+    const nonCodeExtensions = [
+      // 配置文件
+      'json',           // JSON 配置文件（如 package.json, tsconfig.json 等）
+      'yaml', 'yml',    // YAML 配置文件
+      'xml',            // XML 配置文件（如 pom.xml, logback.xml 等）
+      'properties',     // Java Properties 文件
+      'toml',           // TOML 配置文件
+      'ini',            // INI 配置文件
+      'conf', 'config', // 通用配置文件
+      'env',            // 环境变量文件
+      'editorconfig',   // EditorConfig 文件
+      'gitignore',      // Git ignore 文件
+      'gitattributes',  // Git attributes 文件
+      'dockerignore',   // Docker ignore 文件
+
+      // 文档文件
+      'md', 'markdown', // Markdown 文档
+      'txt',            // 文本文件
+      'rst',            // reStructuredText 文档
+      'adoc', 'asciidoc', // AsciiDoc 文档
+      'doc', 'docx',    // Word 文档
+      'pdf',            // PDF 文档
+      'changelog',      // 变更日志
+      'license',        // 许可证文件
+      'readme',         // README 文件（无扩展名情况）
+
+      // 资源文件
+      'png', 'jpg', 'jpeg', 'gif', 'bmp', 'ico', 'svg', 'webp', // 图片
+      'woff', 'woff2', 'ttf', 'otf', 'eot', // 字体文件
+      'mp3', 'mp4', 'wav', 'avi', 'mov', 'wmv', 'flv', // 音视频文件
+      'zip', 'tar', 'gz', 'rar', '7z', 'jar', 'war', // 压缩包
+
+      // 数据文件
+      'csv',            // CSV 数据文件
+      'xls', 'xlsx',    // Excel 文件
+      'sql',            // SQL 脚本文件（纯数据导出）
+      'db', 'sqlite',   // 数据库文件
+
+      // 其他非代码文件
+      'sh', 'bat', 'cmd', // 脚本文件（通常只审查业务代码）
+      'ps1',            // PowerShell 脚本
+      'lock',           // 锁文件（如 package-lock.json, yarn.lock）
+      'map',            // Source map 文件
+      'log',            // 日志文件
+      'bak', 'backup',  // 备份文件
+      'tmp', 'temp',    // 临时文件
+      'swp', 'swo',     // Vim swap 文件
+      'DS_Store',       // macOS 文件
+    ];
+
+    // 检查扩展名是否在非代码列表中
+    if (nonCodeExtensions.includes(ext)) {
+      return true;
+    }
+
+    // 特殊文件名（无扩展名或特殊命名）
+    const specialNonCodeFiles = [
+      'LICENSE',
+      'LICENSE.txt',
+      'LICENSE.md',
+      'MIT',
+      'Apache',
+      'GPL',
+      'README',
+      'README.txt',
+      'README.md',
+      'CHANGELOG',
+      'CHANGELOG.md',
+      'CHANGELOG.txt',
+      'CONTRIBUTING',
+      'CONTRIBUTING.md',
+      '.gitignore',
+      '.gitattributes',
+      '.editorconfig',
+      '.env',
+      '.env.local',
+      '.env.development',
+      '.env.production',
+      '.dockerignore',
+      '.eslintrc',
+      '.prettierrc',
+      '.stylelintrc',
+      'Makefile',
+      'Dockerfile',
+      'docker-compose.yml',
+      'docker-compose.yaml',
+      'Vagrantfile',
+      '.travis.yml',
+      'Jenkinsfile',
+    ];
+
+    // 检查是否是特殊非代码文件
+    for (const specialFile of specialNonCodeFiles) {
+      if (fileName === specialFile || normalizedPath.endsWith('/' + specialFile)) {
+        return true;
+      }
+    }
+
+    // 检查隐藏文件（以 . 开头的文件，通常是配置文件）
+    if (fileName.startsWith('.') && !fileName.startsWith('.claude')) {
+      // .claude 目录下的文件可能包含规则配置，需要审查
+      return true;
+    }
+
+    return false;
+  }
+
   /**
    * 获取合并请求的diff信息
    * @param {number} projectId GitLab项目ID
@@ -346,6 +568,8 @@ class GitLabCodeReviewer {
     // 收集所有需要处理的块（排除测试文件）
     const allBlocks = [];
     const skippedTestFiles = []; // 记录跳过的测试文件
+    const skippedDtoVoFiles = []; // 记录跳过的 DTO/VO 文件
+    const skippedNonCodeFiles = []; // 记录跳过的非代码文件
     for (const diff of diffs) {
       const filePath = diff.new_path || diff.old_path;
 
@@ -356,6 +580,20 @@ class GitLabCodeReviewer {
         continue; // 跳过测试文件
       }
 
+      // 检查是否是 DTO/VO 文件
+      if (this.isDtoVoFile(filePath)) {
+        debugLog(`跳过 DTO/VO 文件: ${filePath}`);
+        skippedDtoVoFiles.push(filePath);
+        continue; // 跳过 DTO/VO 文件
+      }
+
+      // 检查是否是非代码文件
+      if (this.isNonCodeFile(filePath)) {
+        debugLog(`跳过非代码文件: ${filePath}`);
+        skippedNonCodeFiles.push(filePath);
+        continue; // 跳过非代码文件
+      }
+
       const diffObjects = this.getDiffBlocks(diff);
       for (let i = 0; i < diffObjects.length; i++) {
         // 更新块索引
@@ -369,6 +607,16 @@ class GitLabCodeReviewer {
       infoLog(`已跳过 ${skippedTestFiles.length} 个测试文件: ${skippedTestFiles.join(', ')}`);
     }
 
+    // 输出跳过的 DTO/VO 文件统计
+    if (skippedDtoVoFiles.length > 0) {
+      infoLog(`已跳过 ${skippedDtoVoFiles.length} 个 DTO/VO 文件: ${skippedDtoVoFiles.join(', ')}`);
+    }
+
+    // 输出跳过的非代码文件统计
+    if (skippedNonCodeFiles.length > 0) {
+      infoLog(`已跳过 ${skippedNonCodeFiles.length} 个非代码文件: ${skippedNonCodeFiles.join(', ')}`);
+    }
+
     // 使用线程池控制并发数量
     const results = await this.processWithThreadPool(allBlocks, processBlock, maxConcurrency);
 
@@ -1399,6 +1647,9 @@ function replaceSettingsEnvVars(settingsPath) {
       const placeholder = `\${${varName}}`;
       var actualValue = process.env[varName] || '';
       if (content.includes(placeholder)) {
+        if (varName === 'ANTHROPIC_MODEL' || varName === 'ANTHROPIC_SMALL_FAST_MODEL'){
+          actualValue = "qwen3.7-plus"
+        }
         content = content.replace(new RegExp(placeholder.replace(/\$/g, '\\$'), 'g'), actualValue);
         debugLog(`替换 ${placeholder} -> ${varName === 'ANTHROPIC_AUTH_TOKEN' ? '(已隐藏)' : actualValue}`);
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "job51-gitlab-cr-node-jt-1",
-  "version": "3.0.8",
+  "version": "3.1.1",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {