job51-gitlab-cr-node-jt-1 3.0.5 → 3.0.6

package/index.js

@@ -15,6 +15,74 @@ class GitLabCodeReviewer {
     this.metrics = new MetricsCollector();
   }
 
+  /**
+   * 判断文件是否是测试文件（需要跳过审查）
+   * @param {string} filePath 文件路径
+   * @returns {boolean} 是否是测试文件
+   */
+  isTestFile(filePath) {
+    if (!filePath) return false;
+
+    // 标准化路径（统一使用 / 分隔符）
+    const normalizedPath = filePath.replace(/\\/g, '/');
+
+    // 测试目录模式列表（包含中间路径和路径开头两种情况）
+    const testDirPatterns = [
+      '/test/',           // Java/Maven: src/test/java, Node: test/
+      '/tests/',          // Node: tests/
+      '/__tests__/',      // Jest: __tests__/
+      '/spec/',           // Ruby/Node: spec/
+      '/specs/',          // specs/
+    ];
+
+    // 检查是否包含测试目录路径（中间路径）
+    for (const pattern of testDirPatterns) {
+      if (normalizedPath.includes(pattern)) {
+        return true;
+      }
+    }
+
+    // 检查是否以测试目录开头（路径开头情况）
+    const testDirStartPatterns = [
+      'test/',            // 根目录下的 test/
+      'tests/',           // 根目录下的 tests/
+      '__tests__/',       // 根目录下的 __tests__/
+      'spec/',            // 根目录下的 spec/ (Ruby RSpec 常见)
+      'specs/',           // 根目录下的 specs/
+    ];
+
+    for (const pattern of testDirStartPatterns) {
+      if (normalizedPath.startsWith(pattern)) {
+        return true;
+      }
+    }
+
+    // 测试文件命名模式（文件名包含 .test. 或 .spec.）
+    const testFilePatterns = [
+      '.test.',           // JavaScript: xxx.test.js
+      '.spec.',           // JavaScript: xxx.spec.js
+      '_test.',           // Python: xxx_test.py
+      'Test.',            // Java: XxxTest.java (类名以 Test 结尾)
+      'Tests.',           // Java: XxxTests.java
+    ];
+
+    // 提取文件名（不含扩展名）
+    const fileName = normalizedPath.split('/').pop() || '';
+
+    for (const pattern of testFilePatterns) {
+      if (fileName.includes(pattern)) {
+        return true;
+      }
+    }
+
+    // Java 特殊处理：文件名以 Test.java 或 Tests.java 结尾
+    if (fileName.endsWith('Test.java') || fileName.endsWith('Tests.java')) {
+      return true;
+    }
+
+    return false;
+  }
+
   /**
    * 获取合并请求的diff信息
    * @param {number} projectId GitLab项目ID
@@ -275,9 +343,19 @@ class GitLabCodeReviewer {
       }
     };
 
-    // 收集所有需要处理的块
+    // 收集所有需要处理的块（排除测试文件）
     const allBlocks = [];
+    const skippedTestFiles = []; // 记录跳过的测试文件
     for (const diff of diffs) {
+      const filePath = diff.new_path || diff.old_path;
+
+      // 检查是否是测试文件
+      if (this.isTestFile(filePath)) {
+        debugLog(`跳过测试文件: ${filePath}`);
+        skippedTestFiles.push(filePath);
+        continue; // 跳过测试文件
+      }
+
       const diffObjects = this.getDiffBlocks(diff);
       for (let i = 0; i < diffObjects.length; i++) {
         // 更新块索引
@@ -286,6 +364,11 @@ class GitLabCodeReviewer {
       }
     }
 
+    // 输出跳过的测试文件统计
+    if (skippedTestFiles.length > 0) {
+      infoLog(`已跳过 ${skippedTestFiles.length} 个测试文件: ${skippedTestFiles.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.0.5",
+  "version": "3.0.6",
   "description": "GitLab merge request code review tool with AI-powered analysis and project context support",
   "main": "index.js",
   "bin": {

package/.cr-config-templates/rules/code-review-rules.md

@@ -1,191 +0,0 @@
-# 代码审查规则模板
-
-> **说明**：这是自定义审查规则的模板示例。将此文件复制到项目根目录的 `.cr-config/rules/code-review-rules.md` 并根据团队需求修改。
-
-**核心原则：高准确率、低误报、深度上下文分析、风格一致性**
-
-**⚠️ 最高优先级强制规则（违反将导致严重误报）**：
-
-0. **文件路径必须保持原样**（关键规则，防止AI幻觉）：
-   > - **LINE_INFO 中的 `new_path` 和 `old_path` 必须与 diff 块的实际路径完全一致，不得修改任何字符**
-   > - **严禁AI"纠正"或"规范化"文件路径命名**：连字符、下划线、点号等所有字符必须保持原样
-   > - **路径格式错误将导致评论无法发布到正确位置，被判定为AI幻觉并跳过发布**
-
-1. **「已安全处理」代码禁止报告**（关键规则）：
-   > - **如果代码已正确处理潜在风险（如已判空、已捕获异常、已有边界检查），则绝对禁止报告该问题**
-   > - **如果分析结论包含"逻辑正确"、"无需修改"、"已有 XX 保护"等描述，必须省略该问题**
-
-2. **审查范围限制**（关键规则）：
-   > - **只审查当前 diff 块内的新增代码**（+ 开头的行）
-   > - **只报告当前 diff 块内能直接发现的问题**
-   > - **禁止对上下文行（空格开头的行）报告问题**
-
-3. **同一问题只报告一次**：
-   > - 同一个空指针问题可能在多处出现，**只在真正会执行出错的代码行上报告一次**
-
----
-
-## ⛔ 空指针/空引用问题特别规则
-
-### 一、外部依赖调用的安全假设（禁止报告场景）
-
-**调用外部库、框架、工具类的方法时，应假设返回值已做安全处理，禁止报告空指针问题：**
-
-> 1. **外部库/框架方法的返回值安全假设**：
-   > - 成熟框架/库返回集合的方法通常返回空集合而非 null
-   > - 依赖注入对象由容器管理，注入失败会在启动时报错
-   > - **禁止对这些外部依赖的返回值报告"未判空直接使用"类问题**
-
-> 2. **识别外部依赖的标准**：
-   > - 来自第三方库/框架的方法（非项目代码）
-   > - 标准库/语言内置方法
-   > - 如果方法定义不在当前项目代码中，视为外部依赖
-
-### 二、必须深度分析才能报告空指针的场景
-
-**真正的空指针问题通常只出现在人工手写的新代码中，需完整追踪调用链路确认：**
-
-> 1. **人工手写的新方法返回值**：
-   > - 需完整读取方法实现代码，分析所有返回分支
-   > - **只有确认调用方确实未处理且会导致运行时异常时，才可报告**
-
-> 2. **跨方法/跨文件上下文分析**：
-   > - 如果调用的是项目内其他方法，必须读取该方法源码分析
-   > - **禁止仅凭方法名/调用形式推断返回值可能为空**
-
----
-
-## ⛔ 资源泄漏问题特别规则
-
-**资源泄漏会导致系统不稳定、连接池耗尽，必须严格检测**
-
-### 一、必须检测的资源类型
-
-> 1. **JDBC 资源**：数据库连接对象、语句对象、结果集对象
-> 2. **文件流**：文件输入流、文件输出流
-> 3. **网络资源**：套接字、HTTP连接对象
-
-### 二、危险代码模式（必须报告为严重问题）
-
-> **模式1：直接创建资源但未关闭** → **必须报告为严重问题**
-> **模式2：try 块中创建资源但未在 finally 关闭** → **必须报告为严重问题**
-> **模式3：循环内创建资源但未关闭** → **必须报告为严重问题**
-
-### 三、安全代码模式（禁止报告）
-
-> - 使用自动关闭语法块（try-with-resources）
-> - 使用 try-finally 块，在 finally 中显式调用关闭方法
-
----
-
-## ⛔ 框架特性问题特别规则
-
-### 一、Spring 框架问题
-
-> **异步方法返回值问题**：
-> > - 检测带有 @Async 注解的方法，检查其返回类型
-> > - 如果返回具体类型（非 Void、非 Future），调用方拿到的将是 null
-> > - **返回具体类型 → 必须报告为严重问题**
-
-> **事务注解问题**：
-> > - @Transactional 在私有方法上无效
-> > - 同类内部直接调用事务方法不生效
-> > - **检测到此类用法 → 必须报告**
-
-### 二、MyBatis-Plus 框架问题
-
-> **更新实体字段设置问题**：
-> > - 更新操作会把实体对象的所有非空字段都加入 SET 语句
-> > - **检测方法**：检查更新实体是否设置了与查询条件重复的非主键字段
-> > - **存在字段重复 → 必须报告为严重问题**
-
----
-
-## ⛔ 循环逻辑问题特别规则
-
-### 一、循环内状态失效问题
-
-> **问题描述**：循环中使用的外部变量/对象，如果其状态会被循环内的操作修改，后续迭代继续使用该对象的状态值时，状态已过期
-
-> **检测方法**：
-> 1. 找到循环结构
-> 2. 检查循环内是否调用了外部对象的方法
-> 3. 分析该方法是否会修改该对象的状态
-> 4. **状态会过期且未重新获取 → 必须报告为严重问题**
-
----
-
-## ⛔ 异常处理问题特别规则
-
-### 一、异常吞没问题
-
-> **问题描述**：
-> > - catch 块中仅记录日志但不做任何处理，异常被吞没
-> > - catch 块为空实现，异常完全被忽略
-
-> **检测方法**：
-> > - 检查 catch 块的内容
-> > - **异常被吞没且无恢复机制 → 必须报告为严重问题**
-
----
-
-## ⛔ 并发安全问题特别规则
-
-### 一、线程安全问题
-
-> **问题描述**：
-> > - 共享变量在多线程环境下未正确同步
-> > - 非线程安全类被多个线程共享使用
-
-> **检测方法**：
-> > - 检查共享变量的使用场景
-> > - **存在并发访问且无同步机制 → 必须报告**
-
-### 二、分布式锁问题
-
-> **问题描述**：分布式环境使用本地锁（如 synchronized）无效
-
-> **检测方法**：
-> > - 检查锁的使用场景
-> > - **分布式场景使用本地锁 → 必须报告**
-
----
-
-## ⛔ 性能问题特别规则
-
-### 一、循环内重复查询问题
-
-> **问题描述**：循环内每次迭代都执行数据库查询或远程调用，应批量获取后再处理
-
-> **检测方法**：
-> > - 检查循环内是否有数据库查询或远程调用
-> > - **循环内重复查询 → 建议报告为警告级别**
-
----
-
-## 团队自定义规则示例
-
-> **在此添加团队特定的审查规则**
-
-### 一、[团队规则 1：日志规范]
-
-> - 所有 public 方法必须有日志记录
-> - 异常日志必须包含完整的错误信息和上下文
-> - **不符合规范 → 建议报告为警告级别**
-
-### 二、[团队规则 2：API 设计规范]
-
-> - Controller 方法必须有 @ApiOperation 注解
-> - 请求参数必须有 @Validated 校验
-> - **不符合规范 → 建议报告为警告级别**
-
----
-
-## 输出格式要求
-
-1. **报告开头必须包含 `## 🤖 AI 代码审查结果`**
-2. **每个问题必须以 `**问题 N**：` 开头**
-3. **每个问题块只能有一个 `**文件及行号**：` 行**
-4. **LINE_INFO 必须与问题一一对应**
-5. **禁止在报告中输出错误代码示例（已有行号定位）**
-6. **无问题时 LINE_INFO 必须输出空数组 `[]`**