@spaceflow/review 0.76.0 → 0.78.0

package/src/review-spec/review-spec.service.ts

@@ -13,6 +13,7 @@ import { homedir } from "os";
 import { execSync } from "child_process";
 import micromatch from "micromatch";
 import { ReviewSpec, ReviewRule, RuleExample, Severity } from "./types";
+import { extractGlobsFromIncludes } from "../review-includes-filter";
 
 /** 远程规则缓存 TTL（毫秒），默认 5 分钟 */
 const REMOTE_SPEC_CACHE_TTL = 5 * 60 * 1000;
@@ -69,7 +70,23 @@ export class ReviewSpecService {
       }
     }
 
-    return specs.filter((spec) => spec.extensions.some((ext) => changedExtensions.has(ext)));
+    console.log(
+      `[filterApplicableSpecs] changedExtensions=${JSON.stringify([...changedExtensions])}, specs count=${specs.length}`,
+    );
+    const result = specs.filter((spec) => {
+      const matches = spec.extensions.some((ext) => changedExtensions.has(ext));
+      if (!matches) {
+        console.log(
+          `[filterApplicableSpecs] spec ${spec.filename} (ext: ${JSON.stringify(spec.extensions)}) NOT matched`,
+        );
+      } else {
+        console.log(
+          `[filterApplicableSpecs] spec ${spec.filename} (ext: ${JSON.stringify(spec.extensions)}) MATCHED`,
+        );
+      }
+      return matches;
+    });
+    return result;
   }
 
   async loadReviewSpecs(specDir: string): Promise<ReviewSpec[]> {
@@ -524,8 +541,10 @@ export class ReviewSpecService {
         return true;
       }
 
-      // 检查文件是否匹配 includes 模式
-      const matches = micromatch.isMatch(issue.file, includes, { matchBase: true });
+      // 检查文件是否匹配 includes 模式（转换为纯 glob，避免 status| 前缀和 code-* 空串传入 micromatch）
+      const globs = extractGlobsFromIncludes(includes);
+      if (globs.length === 0) return true;
+      const matches = micromatch.isMatch(issue.file, globs, { matchBase: true });
       if (!matches) {
         // console.log(`   Issue [${issue.ruleId}] 在文件 ${issue.file} 不匹配 includes 模式，跳过`);
       }
@@ -638,8 +657,10 @@ export class ReviewSpecService {
         if (scoped.includes.length === 0) {
           return true;
         }
-        // 使用 micromatch 检查文件是否匹配 includes 模式
-        return issueFile && micromatch.isMatch(issueFile, scoped.includes, { matchBase: true });
+        // 使用 micromatch 检查文件是否匹配 includes 模式（转换为纯 glob）
+        const globs = extractGlobsFromIncludes(scoped.includes);
+        if (globs.length === 0) return true;
+        return issueFile && micromatch.isMatch(issueFile, globs, { matchBase: true });
       });
 
       if (matched) {

package/src/review-spec/types.ts

@@ -147,6 +147,8 @@ export interface ReviewResult {
   summary: FileSummary[];
   deletionImpact?: DeletionImpactResult;
   round: number; // 当前 review 的轮次
+  /** 最后一次审查时的 head commit SHA */
+  headSha?: string;
   /** 问题统计信息 */
   stats?: ReviewStats;
 }

package/src/review.config.ts

@@ -27,6 +27,15 @@ export type AnalyzeDeletionsMode = z.infer<typeof analyzeDeletionsModeSchema>;
 /** 审查规则严重级别 */
 export type Severity = z.infer<typeof severitySchema>;
 
+/**
+ * 系统规则配置，不依赖 LLM，在构建 prompt 前直接检查并生成系统问题。
+ * 格式为 [阈值, severity]
+ */
+export interface SystemRules {
+  /** 单文件最大审查行数，超过时跳过 LLM 并生成系统问题。格式: [maxLine, severity] */
+  maxLinesPerFile?: [number, Severity];
+}
+
 /**
  * 变更文件处理策略
  * - 'invalidate': 将变更文件的历史问题标记为无效（默认）
@@ -47,6 +56,11 @@ export interface ReviewOptions {
   references?: string[];
   verbose?: VerboseLevel;
   includes?: string[];
+  /**
+   * 代码结构过滤配置，指定在代码审查时要关注的代码结构类型
+   * 支持格式："function"、"class"、"interface"、"type"、"method"
+   */
+  whenModifiedCode?: string[];
   llmMode?: LLMMode;
   files?: string[];
   commits?: string[];
@@ -80,17 +94,28 @@ export interface ReviewOptions {
    */
   local?: LocalReviewMode;
   /**
-   * 跳过重复的 review workflow 检查
-   * - true: 启用检查，当检测到同名 workflow 正在运行时跳过审查
-   * - false: 禁用检查（默认）
+   * 处理重复 workflow 的策略
+   * - 'off': 禁用检查
+   * - 'skip': 检测到同名 workflow 正在运行时跳过审查
+   * - 'delete': 检测到同名 workflow 时删除旧的 AI Review 评论和 PR Review（默认）
    */
-  skipDuplicateWorkflow?: boolean;
+  duplicateWorkflowResolved?: "off" | "skip" | "delete";
   /**
    * 自动批准合并
    * - true: 当所有问题都已解决时，自动提交 APPROVE review
    * - false: 不自动批准（默认）
    */
   autoApprove?: boolean;
+  /**
+   * 存在未解决问题时以非零退出码退出（工作流抛出异常）
+   * - 'off': 禁用（默认），即使有问题也正常退出
+   * - 'warn': 有未解决的 warn 级别问题时抛出异常
+   * - 'error': 有未解决的 error 级别问题时抛出异常
+   * - 'warn+error': 有未解决的 warn 或 error 级别问题时抛出异常
+   */
+  failOnIssues?: "off" | "warn" | "error" | "warn+error";
+  /** 系统规则配置，不依赖 LLM，直接在检查阶段生成系统问题 */
+  systemRules?: SystemRules;
 }
 
 /** review 命令配置 schema（LLM 敏感配置由系统 llm.config.ts 管理） */
@@ -99,6 +124,7 @@ export const reviewSchema = () =>
     references: z.array(z.string()).optional(),
     llmMode: llmModeSchema.default("openai").optional(),
     includes: z.array(z.string()).optional(),
+    whenModifiedCode: z.array(z.string()).optional(),
     rules: z.record(z.string(), severitySchema).optional(),
     verifyFixes: z.boolean().default(false),
     verifyFixesConcurrency: z.number().default(10).optional(),
@@ -112,8 +138,17 @@ export const reviewSchema = () =>
     retries: z.number().default(0).optional(),
     retryDelay: z.number().default(1000).optional(),
     invalidateChangedFiles: invalidateChangedFilesSchema.default("invalidate").optional(),
-    skipDuplicateWorkflow: z.boolean().default(false).optional(),
+    duplicateWorkflowResolved: z.enum(["off", "skip", "delete"]).default("delete").optional(),
     autoApprove: z.boolean().default(false).optional(),
+    failOnIssues: z.enum(["off", "warn", "error", "warn+error"]).default("off").optional(),
+    systemRules: z
+      .object({
+        maxLinesPerFile: z
+          .tuple([z.number(), severitySchema])
+          .transform((v): [number, Severity] => [v[0], v[1]])
+          .optional(),
+      })
+      .optional(),
   });
 
 /** review 配置类型（从 schema 推导） */