@spaceflow/review 0.59.0 → 0.61.0

package/CHANGELOG.md

@@ -1,5 +1,55 @@
 # Changelog
 
+## [0.60.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.59.0...@spaceflow/review@0.60.0) (2026-03-02)
+
+### 代码重构
+
+* **review:** 优化问题验证逻辑，将 resolved 状态纳入有效性判断 ([1e2302d](https://github.com/Lydanne/spaceflow/commit/1e2302d81cd0f653d606483ef6c9138143ca6d60))
+
+### 代码格式
+
+* 格式化代码并更新 Prettier 忽略规则 ([baed10e](https://github.com/Lydanne/spaceflow/commit/baed10e7cd91fda1285d7e2e0019d291cb563055))
+
+### 其他修改
+
+* **core:** released version 0.23.0 [no ci] ([07a2d7d](https://github.com/Lydanne/spaceflow/commit/07a2d7d51223aeb98161f91fa931b4cb63b03cda))
+* **publish:** released version 0.47.0 [no ci] ([3f59345](https://github.com/Lydanne/spaceflow/commit/3f593450066c9138adac5b101edb8057a5de1ff6))
+* **review-summary:** released version 0.27.0 [no ci] ([90ac2a4](https://github.com/Lydanne/spaceflow/commit/90ac2a44706ddb2dd231ea57d3734c7445565ee9))
+* **scripts:** released version 0.24.0 [no ci] ([717de65](https://github.com/Lydanne/spaceflow/commit/717de6571faa2cb24f04b7493e7fd6d8404f2bd5))
+* **shell:** released version 0.24.0 [no ci] ([5694d19](https://github.com/Lydanne/spaceflow/commit/5694d193f9207e41e840d9ebaa5a43e3527e6af8))
+
+## [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

@@ -2242,6 +2242,8 @@ class ReviewService {
                 if (shouldLog(verbose, 1)) {
                     console.log(`📋 已有评论中存在 ${existingIssues.length} 个问题`);
                 }
+                // 先同步最新的 resolved 状态，确保后续 invalidate/verify 能正确跳过已解决的问题
+                await this.syncResolvedComments(owner, repo, prNumber, existingResult);
                 // 如果文件有变更，将该文件的历史问题标记为无效
                 // 简化策略：避免复杂的行号更新逻辑
                 const reviewConf = this.config.getPluginConfig("review");
@@ -2451,7 +2453,7 @@ class ReviewService {
         const total = issues.length;
         const fixed = issues.filter((i)=>i.fixed).length;
         const resolved = issues.filter((i)=>i.resolved && !i.fixed).length;
-        const invalid = issues.filter((i)=>i.valid === "false").length;
+        const invalid = issues.filter((i)=>i.valid === "false" && !i.fixed && !i.resolved).length;
         const pending = total - fixed - resolved - invalid;
         const fixRate = total > 0 ? Math.round(fixed / total * 100 * 10) / 10 : 0;
         const resolveRate = total > 0 ? Math.round((fixed + resolved) / total * 100 * 10) / 10 : 0;
@@ -3267,7 +3269,7 @@ ${fileChanges || "无"}`;
                 console.warn("⚠️ 更新 PR 标题失败:", error);
             }
         }
-        // 获取已解决的评论，同步 fixed 状态（在更新 review 之前）
+        // 获取已解决的评论，同步 resolve 状态（在更新 review 之前）
         await this.syncResolvedComments(owner, repo, prNumber, result);
         // 获取评论的 reactions，同步 valid 状态（👎 标记为无效）
         await this.syncReactionsToIssues(owner, repo, prNumber, result, verbose);
@@ -3724,7 +3726,7 @@ ${fileChanges || "无"}`;
             if (prevIssues.length > 0) {
                 const prevFixed = prevIssues.filter((i)=>i.fixed).length;
                 const prevResolved = prevIssues.filter((i)=>i.resolved && !i.fixed).length;
-                const prevInvalid = prevIssues.filter((i)=>i.valid === "false").length;
+                const prevInvalid = prevIssues.filter((i)=>i.valid === "false" && !i.fixed && !i.resolved).length;
                 const prevPending = prevIssues.length - prevFixed - prevResolved - prevInvalid;
                 parts.push("");
                 parts.push(`<details><summary>📊 Round ${round - 1} 回顾 (${prevIssues.length} 个问题)</summary>\n`);
@@ -3822,8 +3824,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 +3865,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 +4125,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 +4215,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.61.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

@@ -678,6 +678,9 @@ export class ReviewService {
           console.log(`📋 已有评论中存在 ${existingIssues.length} 个问题`);
         }
 
+        // 先同步最新的 resolved 状态，确保后续 invalidate/verify 能正确跳过已解决的问题
+        await this.syncResolvedComments(owner, repo, prNumber, existingResult);
+
         // 如果文件有变更，将该文件的历史问题标记为无效
         // 简化策略：避免复杂的行号更新逻辑
         const reviewConf = this.config.getPluginConfig<ReviewConfig>("review");
@@ -696,12 +699,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 +918,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;
@@ -982,7 +981,7 @@ export class ReviewService {
     const total = issues.length;
     const fixed = issues.filter((i) => i.fixed).length;
     const resolved = issues.filter((i) => i.resolved && !i.fixed).length;
-    const invalid = issues.filter((i) => i.valid === "false").length;
+    const invalid = issues.filter((i) => i.valid === "false" && !i.fixed && !i.resolved).length;
     const pending = total - fixed - resolved - invalid;
     const fixRate = total > 0 ? Math.round((fixed / total) * 100 * 10) / 10 : 0;
     const resolveRate = total > 0 ? Math.round(((fixed + resolved) / total) * 100 * 10) / 10 : 0;
@@ -1957,7 +1956,7 @@ ${fileChanges || "无"}`;
       }
     }
 
-    // 获取已解决的评论，同步 fixed 状态（在更新 review 之前）
+    // 获取已解决的评论，同步 resolve 状态（在更新 review 之前）
     await this.syncResolvedComments(owner, repo, prNumber, result);
 
     // 获取评论的 reactions，同步 valid 状态（👎 标记为无效）
@@ -2520,7 +2519,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(" ") : ""}`,
+      );
     }
 
     // 上轮回顾
@@ -2529,10 +2530,14 @@ ${fileChanges || "无"}`;
       if (prevIssues.length > 0) {
         const prevFixed = prevIssues.filter((i) => i.fixed).length;
         const prevResolved = prevIssues.filter((i) => i.resolved && !i.fixed).length;
-        const prevInvalid = prevIssues.filter((i) => i.valid === "false").length;
+        const prevInvalid = prevIssues.filter(
+          (i) => i.valid === "false" && !i.fixed && !i.resolved,
+        ).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 +2649,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 +2696,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;
       }