job51-gitlab-cr-node-jt-1 3.1.0 → 3.1.2

package/README.md

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

package/index.js

@@ -98,12 +98,14 @@ class GitLabCodeReviewer {
     // 提取文件名（不含扩展名）
     const fileName = normalizedPath.split('/').pop() || '';
 
-    // DTO/VO 目录模式（整个目录都是数据模型）
+    // DTO/VO/Query 目录模式（整个目录都是数据模型）
     const dtoVoDirPatterns = [
       '/dto/',            // Java: com/example/dto/
       '/DTO/',            // 大写形式
       '/vo/',             // Java: com/example/vo/
       '/VO/',             // 大写形式
+      '/query/',          // Query 对象目录
+      '/Query/',          // 大写形式
       '/model/',          // 通用模型目录
       '/models/',         // 通用模型目录
       '/entity/',         // 实体类目录（有时也是纯数据）
@@ -119,13 +121,14 @@ class GitLabCodeReviewer {
       }
     }
 
-    // DTO/VO 文件命名模式
+    // DTO/VO/Query 文件命名模式
     // 匹配文件名中包含或以这些后缀结尾的情况
     const dtoVoPatterns = [
       'Dto',              // Java: XxxDto.java
       'DTO',              // Java: XxxDTO.java
       'Vo',               // Java: XxxVo.java
       'VO',               // Java: XxxVO.java
+      'Query',            // Query 对象（纯数据查询条件）
       'Request',          // Request 对象通常也是纯数据
       'Response',         // Response 对象通常也是纯数据
       'Form',             // Form 对象
@@ -143,12 +146,13 @@ class GitLabCodeReviewer {
       }
     }
 
-    // Java 特殊处理：文件名以常见 DTO/VO 后缀结尾
+    // Java 特殊处理：文件名以常见 DTO/VO/Query 后缀结尾
     const dtoVoEndPatterns = [
       'Dto.java',
       'DTO.java',
       'Vo.java',
       'VO.java',
+      'Query.java',
       'Request.java',
       'Response.java',
       'Form.java',
@@ -159,6 +163,7 @@ class GitLabCodeReviewer {
       'DTO.kt',
       'Vo.kt',
       'VO.kt',
+      'Query.kt',
       'Request.kt',
       'Response.kt',
       // TypeScript/JavaScript
@@ -170,6 +175,8 @@ class GitLabCodeReviewer {
       'Vo.js',
       'VO.ts',
       'VO.js',
+      'Query.ts',
+      'Query.js',
     ];
 
     for (const pattern of dtoVoEndPatterns) {
@@ -181,6 +188,130 @@ class GitLabCodeReviewer {
     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
@@ -445,6 +576,7 @@ 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;
 
@@ -462,6 +594,13 @@ class GitLabCodeReviewer {
         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++) {
         // 更新块索引
@@ -480,6 +619,11 @@ class GitLabCodeReviewer {
       infoLog(`已跳过 ${skippedDtoVoFiles.length} 个 DTO/VO 文件: ${skippedDtoVoFiles.join(', ')}`);
     }
 
+    // 输出跳过的非代码文件统计
+    if (skippedNonCodeFiles.length > 0) {
+      infoLog(`已跳过 ${skippedNonCodeFiles.length} 个非代码文件: ${skippedNonCodeFiles.join(', ')}`);
+    }
+
     // 使用线程池控制并发数量
     const results = await this.processWithThreadPool(allBlocks, processBlock, maxConcurrency);
 

package/package.json

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