@spaceflow/review 5.0.2 → 5.0.3

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [5.0.2](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@5.0.1...@spaceflow/review@5.0.2) (2026-04-14)
+
+### 代码重构
+
+* **review:** 重构问题统计逻辑,修正 pending 计算并复用 calculateIssueStats ([23ddcac](https://github.com/Lydanne/spaceflow/commit/23ddcac7f4d8cf59b295b9b370e15d3a7058eecf))
+
+### 其他修改
+
+* **review-summary:** released version 5.0.1 [no ci] ([41d770e](https://github.com/Lydanne/spaceflow/commit/41d770e9800defbf1f37d15de6a889df598d45ae))
+* **scripts:** released version 5.0.1 [no ci] ([f527047](https://github.com/Lydanne/spaceflow/commit/f5270478d4a85ec19c2cc2ad6028972909fd7320))
+* **shell:** released version 5.0.1 [no ci] ([e04c907](https://github.com/Lydanne/spaceflow/commit/e04c907ae9bd38a73506f9f93ca21ee97a05c606))
+
 ## [5.0.1](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@5.0.0...@spaceflow/review@5.0.1) (2026-04-14)
 
 ### 修复BUG

package/dist/index.js

@@ -2544,7 +2544,7 @@ function generateIssueKey(issue) {
                 }
                 try {
                     const reactions = await this.pr.getReviewCommentReactions(comment.id);
-                    if (reactions.length === 0 || !matchedIssue) continue;
+                    if (!reactions || reactions.length === 0 || !matchedIssue) continue;
                     // 按 content 分组，收集每种 reaction 的用户列表
                     const reactionMap = new Map();
                     for (const r of reactions){
@@ -2720,8 +2720,10 @@ function generateIssueKey(issue) {
             outputFormat: "markdown",
             ci: true
         });
-        // 获取 PR 信息以获取 head commit SHA
-        const commitId = await this.pr.getHeadSha();
+        // 获取 PR 信息以获取 head commit SHA（同时缓存 PR 状态，供后续 closed 检查复用）
+        const prInfo = await this.pr.getInfo().catch(()=>null);
+        const prIsClosed = prInfo?.state === "closed" || !!prInfo?.merged;
+        const commitId = prInfo?.head?.sha || "HEAD";
         // 1. 发布或更新主评论（使用 Issue Comment API，支持删除和更新）
         try {
             if (existingComments.length > 0) {
@@ -2747,6 +2749,11 @@ function generateIssueKey(issue) {
             console.warn("⚠️ 发布/更新 AI Review 评论失败:", error);
         }
         // 2. 发布本轮新发现的行级评论（使用 PR Review API）
+        // PR 已关闭/合并时无法提交行级 Review，直接跳过
+        if (prIsClosed) {
+            console.log(`ℹ️  PR #${this.pr.number} 已关闭/合并，跳过行级 Review 提交`);
+            return;
+        }
         // 保留旧轮次的 review 历史，但清理同轮次的旧 AI review（重复触发场景）
         // 如果启用 autoApprove 且所有问题已解决，使用 APPROVE event 合并发布
         await this.cleanupDuplicateRoundReviews(this._result.round, verbose);

package/package.json

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

package/src/review-result-model.ts

@@ -368,7 +368,7 @@ export class ReviewResultModel {
         }
         try {
           const reactions = await this.pr.getReviewCommentReactions(comment.id);
-          if (reactions.length === 0 || !matchedIssue) continue;
+          if (!reactions || reactions.length === 0 || !matchedIssue) continue;
           // 按 content 分组，收集每种 reaction 的用户列表
           const reactionMap = new Map<string, string[]>();
           for (const r of reactions) {
@@ -585,8 +585,10 @@ export class ReviewResultModel {
       ci: true,
     });
 
-    // 获取 PR 信息以获取 head commit SHA
-    const commitId = await this.pr.getHeadSha();
+    // 获取 PR 信息以获取 head commit SHA（同时缓存 PR 状态，供后续 closed 检查复用）
+    const prInfo = await this.pr.getInfo().catch(() => null);
+    const prIsClosed = prInfo?.state === "closed" || !!prInfo?.merged;
+    const commitId = prInfo?.head?.sha || "HEAD";
 
     // 1. 发布或更新主评论（使用 Issue Comment API，支持删除和更新）
     try {
@@ -612,6 +614,12 @@ export class ReviewResultModel {
     }
 
     // 2. 发布本轮新发现的行级评论（使用 PR Review API）
+    // PR 已关闭/合并时无法提交行级 Review，直接跳过
+    if (prIsClosed) {
+      console.log(`ℹ️  PR #${this.pr.number} 已关闭/合并，跳过行级 Review 提交`);
+      return;
+    }
+
     // 保留旧轮次的 review 历史，但清理同轮次的旧 AI review（重复触发场景）
     // 如果启用 autoApprove 且所有问题已解决，使用 APPROVE event 合并发布
     await this.cleanupDuplicateRoundReviews(this._result.round, verbose);