@spaceflow/review 0.60.0 → 0.61.0

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # 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)
 
 ### 新特性

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`);

package/package.json

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

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");
@@ -978,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;
@@ -1953,7 +1956,7 @@ ${fileChanges || "无"}`;
       }
     }
 
-    // 获取已解决的评论，同步 fixed 状态（在更新 review 之前）
+    // 获取已解决的评论，同步 resolve 状态（在更新 review 之前）
     await this.syncResolvedComments(owner, repo, prNumber, result);
 
     // 获取评论的 reactions，同步 valid 状态（👎 标记为无效）
@@ -2527,7 +2530,9 @@ ${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(