@spaceflow/review 0.55.0 → 0.56.0

package/CHANGELOG.md

@@ -1,5 +1,16 @@
 # Changelog
 
+## [0.55.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.54.0...@spaceflow/review@0.55.0) (2026-03-02)
+
+### 新特性
+
+* **review:** 保留历史行级评论，为每轮 Review 生成独立评论并添加上轮回顾 ([de431a0](https://github.com/Lydanne/spaceflow/commit/de431a09b4e3b5e1ada9ee5f1ee65786d22b6ff9))
+* **review:** 支持用户手动 resolve 评论并在报告中区分 AI 修复与手动解决 ([c968b65](https://github.com/Lydanne/spaceflow/commit/c968b65c850bc68de3f4409aa3b5294e5a0311ff))
+
+### 修复BUG
+
+* **review:** 修复率计算仅统计 AI 修复的问题，排除手动解决的问题 ([12b3415](https://github.com/Lydanne/spaceflow/commit/12b3415749c9d8523e8b23365fbb39fc7657ff1d))
+
 ## [0.54.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.53.0...@spaceflow/review@0.54.0) (2026-03-02)
 
 ### 新特性

package/dist/index.js

@@ -143,9 +143,9 @@ __webpack_require__.d(__webpack_exports__, {
 ;// CONCATENATED MODULE: external "@spaceflow/core"
 
 ;// CONCATENATED MODULE: ./src/locales/zh-cn/review.json
-var review_namespaceObject = JSON.parse('{"description":"代码审查命令，使用 LLM 对 PR 代码进行自动审查","options.dryRun":"仅打印将要执行的操作，不实际提交评论","options.prNumber":"PR 编号，如果不指定则从环境变量获取","options.base":"基准分支/tag，用于比较差异","options.head":"目标分支/tag，用于比较差异","options.includes":"要审查的文件 glob 模式，如 *.ts *.js（可多次指定）","options.llmMode":"使用的 LLM 模式: claude-code, openai, gemini","options.files":"仅审查指定的文件（空格分隔）","options.commits":"仅审查指定的 commits（空格分隔）","options.verifyFixes":"是否验证历史问题是否已修复（默认从配置文件读取）","options.noVerifyFixes":"禁用历史问题验证","options.verifyConcurrency":"验证历史问题的并发数（默认 10）","options.analyzeDeletions":"分析删除代码可能带来的影响 (true, false, ci, pr, terminal)","options.deletionAnalysisMode":"删除代码分析模式: openai (标准模式) 或 claude-code (Agent 模式，可使用工具)","options.deletionOnly":"仅执行删除代码分析，跳过常规代码审查","options.outputFormat":"输出格式: markdown, terminal, json。不指定则智能选择（PR 用 markdown，终端用 terminal）","options.generateDescription":"使用 AI 生成 PR 功能描述","options.showAll":"显示所有发现的问题，不过滤非变更行的问题","options.flush":"仅刷新状态（同步 reactions、resolved conversations、replies），不执行 LLM 审查","options.eventAction":"PR 事件类型（opened, synchronize, closed 等），closed 时仅收集统计不进行 AI 审查","extensionDescription":"代码审查命令，使用 LLM 对 PR 代码进行自动审查","mcp.serverDescription":"代码审查规则查询服务","mcp.listRules":"获取当前项目的所有代码审查规则，返回规则列表包含 ID、标题、描述、适用的文件扩展名等信息","mcp.getRulesForFile":"获取某个文件应该使用的代码审查规则，根据文件扩展名和 includes 配置过滤","mcp.getRuleDetail":"获取某个规则的完整详情，包括描述、示例代码等","mcp.ruleNotFound":"规则 {{ruleId}} 不存在","mcp.dto.cwd":"项目根目录路径，默认为当前工作目录","mcp.dto.filePath":"文件路径，可以是相对路径或绝对路径","mcp.dto.includeExamples":"是否包含规则示例代码，默认 false","mcp.dto.ruleId":"规则 ID，如 JsTs.Naming.FileName"}')
+var review_namespaceObject = JSON.parse('{"description":"代码审查命令，使用 LLM 对 PR 代码进行自动审查","options.dryRun":"仅打印将要执行的操作，不实际提交评论","options.prNumber":"PR 编号，如果不指定则从环境变量获取","options.base":"基准分支/tag，用于比较差异","options.head":"目标分支/tag，用于比较差异","options.includes":"要审查的文件 glob 模式，如 *.ts *.js（可多次指定）","options.llmMode":"使用的 LLM 模式: claude-code, openai, gemini","options.files":"仅审查指定的文件（空格分隔）","options.commits":"仅审查指定的 commits（空格分隔）","options.verifyFixes":"是否验证历史问题是否已修复（默认从配置文件读取）","options.noVerifyFixes":"禁用历史问题验证","options.verifyConcurrency":"验证历史问题的并发数（默认 10）","options.analyzeDeletions":"分析删除代码可能带来的影响 (true, false, ci, pr, terminal)","options.deletionAnalysisMode":"删除代码分析模式: openai (标准模式) 或 claude-code (Agent 模式，可使用工具)","options.deletionOnly":"仅执行删除代码分析，跳过常规代码审查","options.outputFormat":"输出格式: markdown, terminal, json。不指定则智能选择（PR 用 markdown，终端用 terminal）","options.generateDescription":"使用 AI 生成 PR 功能描述","options.showAll":"显示所有发现的问题，不过滤非变更行的问题","options.flush":"仅刷新状态（同步 reactions、resolved conversations、replies），不执行 LLM 审查","options.eventAction":"PR 事件类型（opened, synchronize, closed 等），closed 时仅收集统计不进行 AI 审查","extensionDescription":"代码审查命令，使用 LLM 对 PR 代码进行自动审查","mcp.serverDescription":"代码审查规则查询服务","mcp.listRules":"获取当前项目的所有代码审查规则，返回规则列表包含 ID、标题、描述、适用的文件扩展名等信息","mcp.getRulesForFile":"获取某个文件应该使用的代码审查规则，根据文件扩展名和 includes 配置过滤","mcp.getRuleDetail":"获取某个规则的完整详情，包括描述、示例代码等","mcp.ruleNotFound":"规则 {{ruleId}} 不存在","mcp.dto.cwd":"项目根目录路径，默认为当前工作目录","mcp.dto.filePath":"文件路径，可以是相对路径或绝对路径","mcp.dto.includeExamples":"是否包含规则示例代码，默认 false","mcp.dto.ruleId":"规则 ID，如 JsTs.Naming.FileName","mcp.getRulesFromDir":"从指定目录加载代码审查规则。读取目录下所有 .md 文件，解析为规则并按规则 ID 去重（后加载的覆盖先加载的）","mcp.dto.dirPath":"包含规则 .md 文件的目录路径，可以是相对路径或绝对路径"}')
 ;// CONCATENATED MODULE: ./src/locales/en/review.json
-var en_review_namespaceObject = JSON.parse('{"description":"Code review command using LLM for automated PR review","options.dryRun":"Only print actions without posting comments","options.prNumber":"PR number, auto-detected from env if not specified","options.base":"Base branch/tag for diff comparison","options.head":"Head branch/tag for diff comparison","options.includes":"File glob patterns to review, e.g. *.ts *.js (can be specified multiple times)","options.llmMode":"LLM mode: claude-code, openai, gemini","options.files":"Only review specified files (space-separated)","options.commits":"Only review specified commits (space-separated)","options.verifyFixes":"Verify if historical issues are fixed (default from config)","options.noVerifyFixes":"Disable historical issue verification","options.verifyConcurrency":"Concurrency for verifying historical issues (default 10)","options.analyzeDeletions":"Analyze impact of deleted code (true, false, ci, pr, terminal)","options.deletionAnalysisMode":"Deletion analysis mode: openai (standard) or claude-code (Agent mode with tools)","options.deletionOnly":"Only run deletion analysis, skip regular code review","options.outputFormat":"Output format: markdown, terminal, json. Auto-selected if not specified (markdown for PR, terminal for CLI)","options.generateDescription":"Generate PR description using AI","options.showAll":"Show all issues found, including those on unchanged lines","options.flush":"Only sync status (reactions, resolved conversations, replies), skip LLM review","options.eventAction":"PR event type (opened, synchronize, closed, etc.), closed only collects stats without AI review","extensionDescription":"Code review command using LLM for automated PR review","mcp.serverDescription":"Code review rules query service","mcp.listRules":"List all code review rules for the current project, returning rule list with ID, title, description, applicable file extensions, etc.","mcp.getRulesForFile":"Get applicable code review rules for a specific file, filtered by file extension and includes configuration","mcp.getRuleDetail":"Get full details of a specific rule, including description, example code, etc.","mcp.ruleNotFound":"Rule {{ruleId}} not found","mcp.dto.cwd":"Project root directory path, defaults to current working directory","mcp.dto.filePath":"File path, can be relative or absolute","mcp.dto.includeExamples":"Whether to include rule example code, defaults to false","mcp.dto.ruleId":"Rule ID, e.g. JsTs.Naming.FileName"}')
+var en_review_namespaceObject = JSON.parse('{"description":"Code review command using LLM for automated PR review","options.dryRun":"Only print actions without posting comments","options.prNumber":"PR number, auto-detected from env if not specified","options.base":"Base branch/tag for diff comparison","options.head":"Head branch/tag for diff comparison","options.includes":"File glob patterns to review, e.g. *.ts *.js (can be specified multiple times)","options.llmMode":"LLM mode: claude-code, openai, gemini","options.files":"Only review specified files (space-separated)","options.commits":"Only review specified commits (space-separated)","options.verifyFixes":"Verify if historical issues are fixed (default from config)","options.noVerifyFixes":"Disable historical issue verification","options.verifyConcurrency":"Concurrency for verifying historical issues (default 10)","options.analyzeDeletions":"Analyze impact of deleted code (true, false, ci, pr, terminal)","options.deletionAnalysisMode":"Deletion analysis mode: openai (standard) or claude-code (Agent mode with tools)","options.deletionOnly":"Only run deletion analysis, skip regular code review","options.outputFormat":"Output format: markdown, terminal, json. Auto-selected if not specified (markdown for PR, terminal for CLI)","options.generateDescription":"Generate PR description using AI","options.showAll":"Show all issues found, including those on unchanged lines","options.flush":"Only sync status (reactions, resolved conversations, replies), skip LLM review","options.eventAction":"PR event type (opened, synchronize, closed, etc.), closed only collects stats without AI review","extensionDescription":"Code review command using LLM for automated PR review","mcp.serverDescription":"Code review rules query service","mcp.listRules":"List all code review rules for the current project, returning rule list with ID, title, description, applicable file extensions, etc.","mcp.getRulesForFile":"Get applicable code review rules for a specific file, filtered by file extension and includes configuration","mcp.getRuleDetail":"Get full details of a specific rule, including description, example code, etc.","mcp.ruleNotFound":"Rule {{ruleId}} not found","mcp.dto.cwd":"Project root directory path, defaults to current working directory","mcp.dto.filePath":"File path, can be relative or absolute","mcp.dto.includeExamples":"Whether to include rule example code, defaults to false","mcp.dto.ruleId":"Rule ID, e.g. JsTs.Naming.FileName","mcp.getRulesFromDir":"Load code review rules from a specific directory. Reads all .md files in the directory, parses them into rules, and deduplicates by rule ID (later rules override earlier ones)","mcp.dto.dirPath":"Directory path containing rule .md files, can be relative or absolute"}')
 ;// CONCATENATED MODULE: ./src/locales/index.ts
 
 
@@ -5038,6 +5038,10 @@ const getRulesForFileInputSchema = z.object({
 const getRuleDetailInputSchema = z.object({
     ruleId: z.string().describe(t("review:mcp.dto.ruleId"))
 });
+const getRulesFromDirInputSchema = z.object({
+    dirPath: z.string().describe(t("review:mcp.dto.dirPath")),
+    includeExamples: z.boolean().optional().describe(t("review:mcp.dto.includeExamples"))
+});
 /**
  * 获取 GitProviderService（可选）
  */ function getGitProvider(ctx) {
@@ -5190,6 +5194,49 @@ const tools = [
                     }))
             };
         }
+    },
+    {
+        name: "get_rules_from_dir",
+        description: t("review:mcp.getRulesFromDir"),
+        inputSchema: getRulesFromDirInputSchema,
+        handler: async (input, ctx)=>{
+            const { dirPath, includeExamples } = input;
+            const workDir = ctx.cwd;
+            const resolvedDir = dirPath.startsWith("/") ? dirPath : join(workDir, dirPath);
+            if (!existsSync(resolvedDir)) {
+                return {
+                    error: `Directory not found: ${resolvedDir}`
+                };
+            }
+            const gitProvider = getGitProvider(ctx);
+            const specService = new ReviewSpecService(gitProvider);
+            const specs = await specService.loadReviewSpecs(resolvedDir);
+            const dedupedSpecs = specService.deduplicateSpecs(specs);
+            const rules = dedupedSpecs.flatMap((spec)=>spec.rules.map((rule)=>({
+                        id: rule.id,
+                        title: rule.title,
+                        description: includeExamples ? rule.description : rule.description.slice(0, 200) + (rule.description.length > 200 ? "..." : ""),
+                        severity: rule.severity || spec.severity,
+                        extensions: spec.extensions,
+                        specFile: spec.filename,
+                        includes: spec.includes,
+                        ...includeExamples && rule.examples.length > 0 ? {
+                            examples: rule.examples.map((ex)=>({
+                                    type: ex.type,
+                                    lang: ex.lang,
+                                    code: ex.code
+                                }))
+                        } : {
+                            hasExamples: rule.examples.length > 0
+                        }
+                    })));
+            return {
+                dir: resolvedDir,
+                specFiles: dedupedSpecs.length,
+                total: rules.length,
+                rules
+            };
+        }
     }
 ];
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spaceflow/review",
-  "version": "0.55.0",
+  "version": "0.56.0",
   "description": "Spaceflow 代码审查插件，使用 LLM 对 PR 代码进行自动审查",
   "license": "MIT",
   "author": "Lydanne",

package/src/locales/en/review.json

@@ -28,5 +28,7 @@
   "mcp.dto.cwd": "Project root directory path, defaults to current working directory",
   "mcp.dto.filePath": "File path, can be relative or absolute",
   "mcp.dto.includeExamples": "Whether to include rule example code, defaults to false",
-  "mcp.dto.ruleId": "Rule ID, e.g. JsTs.Naming.FileName"
+  "mcp.dto.ruleId": "Rule ID, e.g. JsTs.Naming.FileName",
+  "mcp.getRulesFromDir": "Load code review rules from a specific directory. Reads all .md files in the directory, parses them into rules, and deduplicates by rule ID (later rules override earlier ones)",
+  "mcp.dto.dirPath": "Directory path containing rule .md files, can be relative or absolute"
 }

package/src/locales/zh-cn/review.json

@@ -28,5 +28,7 @@
   "mcp.dto.cwd": "项目根目录路径，默认为当前工作目录",
   "mcp.dto.filePath": "文件路径，可以是相对路径或绝对路径",
   "mcp.dto.includeExamples": "是否包含规则示例代码，默认 false",
-  "mcp.dto.ruleId": "规则 ID，如 JsTs.Naming.FileName"
+  "mcp.dto.ruleId": "规则 ID，如 JsTs.Naming.FileName",
+  "mcp.getRulesFromDir": "从指定目录加载代码审查规则。读取目录下所有 .md 文件，解析为规则并按规则 ID 去重（后加载的覆盖先加载的）",
+  "mcp.dto.dirPath": "包含规则 .md 文件的目录路径，可以是相对路径或绝对路径"
 }

package/src/mcp/index.ts

@@ -16,6 +16,11 @@ export const getRuleDetailInputSchema = z.object({
   ruleId: z.string().describe(t("review:mcp.dto.ruleId")),
 });
 
+export const getRulesFromDirInputSchema = z.object({
+  dirPath: z.string().describe(t("review:mcp.dto.dirPath")),
+  includeExamples: z.boolean().optional().describe(t("review:mcp.dto.includeExamples")),
+});
+
 /**
  * 获取 GitProviderService（可选）
  */
@@ -165,4 +170,53 @@ export const tools = [
       };
     },
   },
+  {
+    name: "get_rules_from_dir",
+    description: t("review:mcp.getRulesFromDir"),
+    inputSchema: getRulesFromDirInputSchema,
+    handler: async (input, ctx) => {
+      const { dirPath, includeExamples } = input as z.infer<typeof getRulesFromDirInputSchema>;
+      const workDir = ctx.cwd;
+      const resolvedDir = dirPath.startsWith("/") ? dirPath : join(workDir, dirPath);
+
+      if (!existsSync(resolvedDir)) {
+        return { error: `Directory not found: ${resolvedDir}` };
+      }
+
+      const gitProvider = getGitProvider(ctx);
+      const specService = new ReviewSpecService(gitProvider);
+      const specs = await specService.loadReviewSpecs(resolvedDir);
+      const dedupedSpecs = specService.deduplicateSpecs(specs);
+
+      const rules = dedupedSpecs.flatMap((spec) =>
+        spec.rules.map((rule) => ({
+          id: rule.id,
+          title: rule.title,
+          description: includeExamples
+            ? rule.description
+            : rule.description.slice(0, 200) + (rule.description.length > 200 ? "..." : ""),
+          severity: rule.severity || spec.severity,
+          extensions: spec.extensions,
+          specFile: spec.filename,
+          includes: spec.includes,
+          ...(includeExamples && rule.examples.length > 0
+            ? {
+                examples: rule.examples.map((ex) => ({
+                  type: ex.type,
+                  lang: ex.lang,
+                  code: ex.code,
+                })),
+              }
+            : { hasExamples: rule.examples.length > 0 }),
+        })),
+      );
+
+      return {
+        dir: resolvedDir,
+        specFiles: dedupedSpecs.length,
+        total: rules.length,
+        rules,
+      };
+    },
+  },
 ];