@spaceflow/review 0.83.0 → 1.0.0

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

@@ -12,9 +12,9 @@ import { readdir, readFile, mkdir, access, writeFile, unlink } from "fs/promises
 import { join, basename } from "path";
 import { homedir } from "os";
 import { execSync, execFileSync } from "child_process";
-import micromatch from "micromatch";
-import { ReviewSpec, ReviewRule, RuleExample, Severity } from "./types";
-import { extractGlobsFromIncludes } from "../review-includes-filter";
+import { ReviewSpec, ReviewRule, RuleExample, RuleContent, Severity } from "./types";
+import { matchIncludes } from "../review-includes-filter";
+import { buildSpecsSection as buildSpecsSectionPrompt } from "../prompt/specs-section";
 
 export class ReviewSpecService {
   constructor(protected readonly gitProvider?: GitProviderService) {}
@@ -606,7 +606,9 @@ export class ReviewSpecService {
 
       // 提取描述：在第一个例子之前的文本
       let description = ruleContent;
-      const firstExampleIndex = ruleContent.search(/(?:^|\n)###\s+(?:good|bad)/i);
+      const firstExampleIndex = ruleContent.search(
+        /(?:^|\n)(?:####\s+(?:good|bad)|###\s+Example:)/i,
+      );
       if (firstExampleIndex !== -1) {
         description = ruleContent.slice(0, firstExampleIndex).trim();
       } else {
@@ -684,27 +686,64 @@ export class ReviewSpecService {
 
   protected extractExamples(content: string): RuleExample[] {
     const examples: RuleExample[] = [];
-    const sections = content.split(/(?:^|\n)###\s+/);
 
-    for (const section of sections) {
-      const trimmedSection = section.trim();
-      if (!trimmedSection) continue;
+    // 按 ### Example: 分组
+    const groupSections = content.split(/(?:^|\n)(?=###\s+Example:)/);
+
+    for (const groupSection of groupSections) {
+      const trimmedGroup = groupSection.trim();
+      if (!trimmedGroup) continue;
 
-      let type: "good" | "bad" | null = null;
-      if (/^good\b/i.test(trimmedSection)) {
-        type = "good";
-      } else if (/^bad\b/i.test(trimmedSection)) {
-        type = "bad";
+      // 提取分组描述，如 "### Example: 命名规则说明"
+      let exampleTitle = "";
+      let exampleDescription = "";
+      const groupMatch = trimmedGroup.match(/^###\s+Example\s*[:：]\s*(.+)/i);
+      if (groupMatch) {
+        exampleTitle = "Example";
+        exampleDescription = groupMatch[1].trim();
       }
 
-      if (!type) continue;
+      // 在分组内按 #### 提取 Good/Bad
+      const ruleContents: RuleContent[] = [];
+      const sections = trimmedGroup.split(/(?:^|\n)####\s+/);
+
+      for (const section of sections) {
+        const trimmedSection = section.trim();
+        if (!trimmedSection) continue;
+
+        let type: "good" | "bad" | null = null;
+        let contentTitle = "";
+        if (/^good:/i.test(trimmedSection)) {
+          type = "good";
+          contentTitle = trimmedSection.match(/^good:\s*(.+)/i)?.[1]?.trim() ?? "";
+        } else if (/^bad:/i.test(trimmedSection)) {
+          type = "bad";
+          contentTitle = trimmedSection.match(/^bad:\s*(.+)/i)?.[1]?.trim() ?? "";
+        }
+
+        if (!type) continue;
+
+        // 提取代码块作为 description
+        const codeBlockRegex = /```(\w*)\n([\s\S]*?)```/g;
+        let codeMatch;
+        const codeParts: string[] = [];
+        while ((codeMatch = codeBlockRegex.exec(trimmedSection)) !== null) {
+          codeParts.push(codeMatch[2].trim());
+        }
+
+        ruleContents.push({
+          title: contentTitle,
+          type,
+          description: codeParts.join("\n\n"),
+        });
+      }
 
-      const codeBlockRegex = /```(\w*)\n([\s\S]*?)```/g;
-      let codeMatch;
-      while ((codeMatch = codeBlockRegex.exec(trimmedSection)) !== null) {
-        const lang = codeMatch[1] || "text";
-        const code = codeMatch[2].trim();
-        examples.push({ lang, code, type });
+      if (ruleContents.length > 0) {
+        examples.push({
+          title: exampleTitle,
+          description: exampleDescription,
+          content: ruleContents,
+        });
       }
     }
 
@@ -748,10 +787,12 @@ export class ReviewSpecService {
    * 根据 spec 的 includes 配置过滤 issues
    * 只保留文件名匹配对应 spec includes 模式的 issues
    * 如果 spec 没有 includes 配置，则保留该 spec 的所有 issues
+   * 支持 `added|`/`modified|`/`deleted|` 前缀语法
    */
   filterIssuesByIncludes<T extends { file: string; ruleId: string }>(
     issues: T[],
     specs: ReviewSpec[],
+    changedFiles?: ChangedFileCollection,
   ): T[] {
     // 构建 spec filename -> includes 的映射
     const specIncludesMap = new Map<string, string[]>();
@@ -771,14 +812,9 @@ export class ReviewSpecService {
         return 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 模式，跳过`);
-      }
-      return matches;
+      // 使用 matchIncludes 检查文件是否匹配 includes 模式（支持 status|glob 前缀）
+      const fileStatus = changedFiles?.getStatus(issue.file);
+      return matchIncludes(includes, issue.file, fileStatus);
     });
   }
 
@@ -816,6 +852,7 @@ export class ReviewSpecService {
   filterIssuesByOverrides<T extends { ruleId: string; file?: string }>(
     issues: T[],
     specs: ReviewSpec[],
+    changedFiles?: ChangedFileCollection,
     verbose?: VerboseLevel,
   ): T[] {
     // ========== 阶段1: 收集 spec -> overrides 的映射（保留作用域信息） ==========
@@ -887,10 +924,9 @@ export class ReviewSpecService {
         if (scoped.includes.length === 0) {
           return true;
         }
-        // 使用 micromatch 检查文件是否匹配 includes 模式（转换为纯 glob）
-        const globs = extractGlobsFromIncludes(scoped.includes);
-        if (globs.length === 0) return true;
-        return issueFile && micromatch.isMatch(issueFile, globs, { matchBase: true });
+        // 使用 matchIncludes 检查文件是否匹配 includes 模式（支持 status|glob 前缀）
+        const fileStatus = changedFiles?.getStatus(issueFile);
+        return matchIncludes(scoped.includes, issueFile, fileStatus);
       });
 
       if (matched) {
@@ -1011,29 +1047,7 @@ export class ReviewSpecService {
    * 构建 specs 的 prompt 部分
    */
   buildSpecsSection(specs: ReviewSpec[]): string {
-    return specs
-      .map((spec) => {
-        const firstRule = spec.rules[0];
-        const rulesText = spec.rules
-          .slice(1)
-          .map((rule) => {
-            let text = `#### [${rule.id}] ${rule.title}\n`;
-            if (rule.description) {
-              text += `${rule.description}\n`;
-            }
-            if (rule.examples.length > 0) {
-              for (const example of rule.examples) {
-                text += `##### ${example.type === "good" ? "推荐做法 (Good)" : "不推荐做法 (Bad)"}\n`;
-                text += `\`\`\`${example.lang}\n${example.code}\n\`\`\`\n`;
-              }
-            }
-            return text;
-          })
-          .join("\n");
-
-        return `### ${firstRule.title}\n- 规范文件: ${spec.filename}\n- 适用扩展名: ${spec.extensions.join(", ")}\n\n${rulesText}`;
-      })
-      .join("\n\n-------------------\n\n");
+    return buildSpecsSectionPrompt(specs);
   }
 
   /**

package/src/review-spec/types.ts

@@ -24,9 +24,15 @@ export interface ReviewSpec {
 }
 
 export interface RuleExample {
-  lang: string;
-  code: string;
+  title: string;
+  description: string;
+  content: RuleContent[];
+}
+
+export interface RuleContent {
+  title: string;
   type: "good" | "bad";
+  description: string;
 }
 
 export interface ReviewRule {

package/src/review.service.ts

@@ -3,7 +3,6 @@ import {
   PullRequestCommit,
   type LLMMode,
   LlmProxyService,
-  type VerboseLevel,
   shouldLog,
   GitSdkService,
 } from "@spaceflow/core";
@@ -281,7 +280,7 @@ export class ReviewService {
     const { commits, fileContents, changedFiles, isDirectFileMode, context } = opts;
     const { verbose } = context;
 
-    let filtered = this.reviewSpecService.filterIssuesByIncludes(issues, specs);
+    let filtered = this.reviewSpecService.filterIssuesByIncludes(issues, specs, changedFiles);
     if (shouldLog(verbose, 1)) {
       console.log(`   应用 includes 过滤后: ${filtered.length} 个问题`);
     }
@@ -291,7 +290,12 @@ export class ReviewService {
       console.log(`   应用规则存在性过滤后: ${filtered.length} 个问题`);
     }
 
-    filtered = this.reviewSpecService.filterIssuesByOverrides(filtered, specs, verbose);
+    filtered = this.reviewSpecService.filterIssuesByOverrides(
+      filtered,
+      specs,
+      changedFiles,
+      verbose,
+    );
 
     // 变更行过滤
     if (shouldLog(verbose, 3)) {

package/dist/551.js

@@ -1,9 +0,0 @@
-export const __rspack_esm_id = "551";
-export const __rspack_esm_ids = ["551"];
-export const __webpack_modules__ = {
-946(module) {
-module.exports = import("micromatch").then(function(module) { return module; });
-
-},
-
-};