@spaceflow/review 0.59.0 → 0.60.0

package/CHANGELOG.md

@@ -1,5 +1,37 @@
 # Changelog
 
+## [0.59.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.58.0...@spaceflow/review@0.59.0) (2026-03-02)
+
+### 新特性
+
+* **review-summary:** 实现 review-summary 命令核心功能 ([5964eec](https://github.com/Lydanne/spaceflow/commit/5964eec0e2ee4ac46f74808466429b0edc0cbaa1))
+* **review-summary:** 新增文件过滤功能并优化缺陷率计算 ([9655f82](https://github.com/Lydanne/spaceflow/commit/9655f828cec77c4a7db29ae1cda59fd3ea10ceab))
+
+### 修复BUG
+
+* **actions:** 修正 GitHub Actions 路径并优化命令行选项 ([50a9946](https://github.com/Lydanne/spaceflow/commit/50a99464633afe234ef56d4a081c6d04686b3e57))
+
+### 代码重构
+
+* **core:** 统一插件配置中的 'extensions' 为 'skills' ([50646a0](https://github.com/Lydanne/spaceflow/commit/50646a061f66ec6935c66199d78915b9d7896bd3))
+
+### 文档更新
+
+* **docs:** 统一扩展类型命名：extension 改为 skill ([96db479](https://github.com/Lydanne/spaceflow/commit/96db47948981c0c2b5860c2835f86dec77736cf0))
+* 更新 MCP 架构为 Meta-tool 代理模式并统一导出类型命名 ([f60c53b](https://github.com/Lydanne/spaceflow/commit/f60c53b103f24e8a4123e4c4ec850885cb40f7d7))
+* 简化 MCP 工具声明方式并完善扩展系统文档 ([9cdcab9](https://github.com/Lydanne/spaceflow/commit/9cdcab986559bd34c75e48b6c380c74516adb05c))
+
+### 其他修改
+
+* **core:** released version 0.22.0 [no ci] ([68aa47d](https://github.com/Lydanne/spaceflow/commit/68aa47df425eb9d1ceac1237fee3cc1b29de668f))
+* **publish:** released version 0.46.0 [no ci] ([68ebfc3](https://github.com/Lydanne/spaceflow/commit/68ebfc32bfb5b39354947f08b73306b5f8512fdf))
+* **review-summary:** released version 0.23.0 [no ci] ([f6681ec](https://github.com/Lydanne/spaceflow/commit/f6681ecdcd8f8036da3c4ac7778e5cc75af59c9f))
+* **review-summary:** released version 0.24.0 [no ci] ([690e9ed](https://github.com/Lydanne/spaceflow/commit/690e9ed64b197f50e201afe73dc55e866867a7fd))
+* **review-summary:** released version 0.25.0 [no ci] ([4eaca03](https://github.com/Lydanne/spaceflow/commit/4eaca0397411585112697e6800b66484bd73ffde))
+* **review-summary:** released version 0.26.0 [no ci] ([5e8dedf](https://github.com/Lydanne/spaceflow/commit/5e8dedf10cb7104dcc15550d7f5ba05f10cae7d3))
+* **scripts:** released version 0.23.0 [no ci] ([2f18d22](https://github.com/Lydanne/spaceflow/commit/2f18d2274e83b65ce006dceed47a985942c8dd1d))
+* **shell:** released version 0.23.0 [no ci] ([0668aa9](https://github.com/Lydanne/spaceflow/commit/0668aa97671ca235509bef547503c301237324f9))
+
 ## [0.58.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.57.0...@spaceflow/review@0.58.0) (2026-03-02)
 
 ### 新特性

package/dist/index.js

@@ -3822,8 +3822,8 @@ ${fileChanges || "无"}`;
             // 将变更文件的历史 issue 标记为无效
             let invalidatedCount = 0;
             const updatedIssues = issues.map((issue)=>{
-                // 如果 issue 已修复或已无效，不需要处理
-                if (issue.fixed || issue.valid === "false") {
+                // 如果 issue 已修复、已解决或已无效，不需要处理
+                if (issue.fixed || issue.resolved || issue.valid === "false") {
                     return issue;
                 }
                 // 如果 issue 所在文件有变更，标记为无效
@@ -3863,8 +3863,8 @@ ${fileChanges || "无"}`;
         let updatedCount = 0;
         let invalidatedCount = 0;
         const updatedIssues = issues.map((issue)=>{
-            // 如果 issue 已修复或无效，不需要更新行号
-            if (issue.fixed || issue.valid === "false") {
+            // 如果 issue 已修复、已解决或无效，不需要更新行号
+            if (issue.fixed || issue.resolved || issue.valid === "false") {
                 return issue;
             }
             const patch = filePatchMap.get(issue.file);
@@ -4123,7 +4123,7 @@ class IssueVerifyService {
                 continue;
             }
             // valid === 'false' 的问题跳过复查（已确认无效的问题无需再次验证）
-            if (issue.valid === "false") {
+            if (issue.valid === FALSE) {
                 if (shouldLog(verbose, 1)) {
                     console.log(`   ⏭️  跳过无效问题: ${issue.file}:${issue.line} (${issue.ruleId})`);
                 }
@@ -4213,7 +4213,7 @@ class IssueVerifyService {
             if (result) {
                 const updatedIssue = {
                     ...issue,
-                    valid: result.valid ? TRUE : FALSE
+                    valid: issue.fixed || issue.resolved ? TRUE : result.valid ? TRUE : FALSE
                 };
                 if (result.fixed) {
                     if (shouldLog(verbose, 1)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@spaceflow/review",
-  "version": "0.59.0",
+  "version": "0.60.0",
   "description": "Spaceflow 代码审查插件，使用 LLM 对 PR 代码进行自动审查",
   "license": "MIT",
   "author": "Lydanne",
@@ -28,7 +28,7 @@
     "@spaceflow/cli": "0.38.0"
   },
   "peerDependencies": {
-    "@spaceflow/core": "0.22.0"
+    "@spaceflow/core": "0.23.0"
   },
   "spaceflow": {
     "type": "flow",

package/src/issue-verify.service.ts

@@ -92,7 +92,7 @@ export class IssueVerifyService {
       }
 
       // valid === 'false' 的问题跳过复查（已确认无效的问题无需再次验证）
-      if (issue.valid === "false") {
+      if (issue.valid === FALSE) {
         if (shouldLog(verbose, 1)) {
           console.log(`   ⏭️  跳过无效问题: ${issue.file}:${issue.line} (${issue.ruleId})`);
         }
@@ -201,7 +201,7 @@ export class IssueVerifyService {
       if (result) {
         const updatedIssue: ReviewIssue = {
           ...issue,
-          valid: result.valid ? TRUE : FALSE,
+          valid: issue.fixed || issue.resolved ? TRUE : result.valid ? TRUE : FALSE,
         };
 
         if (result.fixed) {

package/src/review.service.ts

@@ -696,12 +696,10 @@ export class ReviewService {
 
         // 验证历史问题是否已修复
         if (context.verifyFixes) {
-          existingIssues = await this.verifyAndUpdateIssues(
-            context,
-            existingIssues,
-            commits,
-            { specs, fileContents },
-          );
+          existingIssues = await this.verifyAndUpdateIssues(context, existingIssues, commits, {
+            specs,
+            fileContents,
+          });
         } else {
           if (shouldLog(verbose, 1)) {
             console.log(`   ⏭️  跳过历史问题验证 (verifyFixes=false)`);
@@ -917,9 +915,7 @@ export class ReviewService {
     preloaded?: { specs: ReviewSpec[]; fileContents: FileContentsMap },
   ): Promise<ReviewIssue[]> {
     const { owner, repo, prNumber, llmMode, specSources, verbose } = context;
-    const unfixedIssues = issues.filter(
-      (i) => i.valid !== "false" && !i.fixed,
-    );
+    const unfixedIssues = issues.filter((i) => i.valid !== "false" && !i.fixed);
 
     if (unfixedIssues.length === 0) {
       return issues;
@@ -2520,7 +2516,9 @@ ${fileChanges || "无"}`;
     if (issues.length === 0) {
       parts.push(`> ✅ 未发现新问题`);
     } else {
-      parts.push(`> **${issues.length}** 个新问题 · **${fileCount}** 个文件${badges.length > 0 ? " · " + badges.join(" ") : ""}`);
+      parts.push(
+        `> **${issues.length}** 个新问题 · **${fileCount}** 个文件${badges.length > 0 ? " · " + badges.join(" ") : ""}`,
+      );
     }
 
     // 上轮回顾
@@ -2532,7 +2530,9 @@ ${fileChanges || "无"}`;
         const prevInvalid = prevIssues.filter((i) => i.valid === "false").length;
         const prevPending = prevIssues.length - prevFixed - prevResolved - prevInvalid;
         parts.push("");
-        parts.push(`<details><summary>📊 Round ${round - 1} 回顾 (${prevIssues.length} 个问题)</summary>\n`);
+        parts.push(
+          `<details><summary>📊 Round ${round - 1} 回顾 (${prevIssues.length} 个问题)</summary>\n`,
+        );
         parts.push(`| 状态 | 数量 |`);
         parts.push(`|------|------|`);
         if (prevFixed > 0) parts.push(`| ✅ 已修复 | ${prevFixed} |`);
@@ -2644,8 +2644,8 @@ ${fileChanges || "无"}`;
       // 将变更文件的历史 issue 标记为无效
       let invalidatedCount = 0;
       const updatedIssues = issues.map((issue) => {
-        // 如果 issue 已修复或已无效，不需要处理
-        if (issue.fixed || issue.valid === "false") {
+        // 如果 issue 已修复、已解决或已无效，不需要处理
+        if (issue.fixed || issue.resolved || issue.valid === "false") {
           return issue;
         }
 
@@ -2691,8 +2691,8 @@ ${fileChanges || "无"}`;
     let updatedCount = 0;
     let invalidatedCount = 0;
     const updatedIssues = issues.map((issue) => {
-      // 如果 issue 已修复或无效，不需要更新行号
-      if (issue.fixed || issue.valid === "false") {
+      // 如果 issue 已修复、已解决或无效，不需要更新行号
+      if (issue.fixed || issue.resolved || issue.valid === "false") {
         return issue;
       }