@spaceflow/review 0.71.0 → 0.73.0

package/CHANGELOG.md

@@ -1,5 +1,35 @@
 # Changelog
 
+## [0.72.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.71.0...@spaceflow/review@0.72.0) (2026-04-03)
+
+### 修复BUG
+
+* **review:** 为 skipDuplicateWorkflow 添加错误处理以应对权限不足问题 ([6d0292d](https://github.com/Lydanne/spaceflow/commit/6d0292d20b6eba790f594c555ceb18b75ebfe253))
+
+### 其他修改
+
+* **core:** released version 0.28.0 [no ci] ([6baebaa](https://github.com/Lydanne/spaceflow/commit/6baebaa4cd777b6f8babcb5d8a2f25c426f8da05))
+* **publish:** released version 0.52.0 [no ci] ([bcbc2a9](https://github.com/Lydanne/spaceflow/commit/bcbc2a97e5bdedfd728330a33f19ec9172d80f40))
+* **review-summary:** released version 0.40.0 [no ci] ([9c5c262](https://github.com/Lydanne/spaceflow/commit/9c5c26256984c9349def2801af6e77b7013e703f))
+
+## [0.71.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.70.0...@spaceflow/review@0.71.0) (2026-04-02)
+
+### 新特性
+
+* **review:** 添加 skipDuplicateWorkflow 和 autoApprove 配置选项支持 ([3ede6e3](https://github.com/Lydanne/spaceflow/commit/3ede6e30d89ced5ef57e154e42f723da5e3c5d92))
+
+### 修复BUG
+
+* **review-summary:** 在用户名前添加 @ 符号以支持用户提及 ([a4aa03f](https://github.com/Lydanne/spaceflow/commit/a4aa03f377bb26e1fa7adc90568d89f2bb9936ce))
+* **review-summary:** 在用户名前添加 @ 符号以支持用户提及 ([fd83df4](https://github.com/Lydanne/spaceflow/commit/fd83df4a647e6c4d42617b44fdaf0ebbec7892e4))
+
+### 其他修改
+
+* **review-summary:** released version 0.38.0 [no ci] ([119c2a8](https://github.com/Lydanne/spaceflow/commit/119c2a8861a02e68a4e31135aa88799fe63d29ff))
+* **review-summary:** released version 0.39.0 [no ci] ([40fbd43](https://github.com/Lydanne/spaceflow/commit/40fbd435b3895b22b04dce7cce4c487f0edbdd2a))
+* **scripts:** released version 0.29.0 [no ci] ([496843a](https://github.com/Lydanne/spaceflow/commit/496843add224941b646837d8fc8ec03f790530fc))
+* **shell:** released version 0.29.0 [no ci] ([368f8c8](https://github.com/Lydanne/spaceflow/commit/368f8c83ae3e485df5bfd4128e46aef4a6689eb0))
+
 ## [0.70.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.69.0...@spaceflow/review@0.70.0) (2026-04-02)
 
 ### 新特性

package/dist/index.js

@@ -2209,6 +2209,7 @@ class ReviewService {
             }
             // 检查是否有其他同名 review workflow 正在运行中（防止同一 PR 重复审查）
             // 需要显式启用 skipDuplicateWorkflow 配置
+            // 注意：Gitea Actions API 要求用户是 repo owner，否则会返回 403
             if (skipDuplicateWorkflow && ci && pr?.head?.sha) {
                 const headSha = pr.head.sha;
                 // 获取当前 PR 编号（从 CI 环境变量）
@@ -2217,25 +2218,33 @@ class ReviewService {
                 const ref = process.env.GITHUB_REF || process.env.GITEA_REF || "";
                 const prMatch = ref.match(/refs\/pull\/(\d+)/);
                 const currentPrNumber = prMatch ? parseInt(prMatch[1], 10) : prNumber;
-                const runningWorkflows = await this.gitProvider.listWorkflowRuns(owner, repo, {
-                    status: "in_progress"
-                });
-                // 获取当前 workflow 名称和 run ID
-                const currentWorkflowName = process.env.GITHUB_WORKFLOW || process.env.GITEA_WORKFLOW;
-                const currentRunId = process.env.GITHUB_RUN_ID || process.env.GITEA_RUN_ID;
-                // 只检查同 PR 同名的其他 workflow run（排除当前 run）
-                const duplicateReviewRuns = runningWorkflows.filter((w)=>w.sha === headSha && w.name === currentWorkflowName && (!currentRunId || String(w.id) !== currentRunId));
-                if (duplicateReviewRuns.length > 0) {
+                try {
+                    const runningWorkflows = await this.gitProvider.listWorkflowRuns(owner, repo, {
+                        status: "in_progress"
+                    });
+                    // 获取当前 workflow 名称和 run ID
+                    const currentWorkflowName = process.env.GITHUB_WORKFLOW || process.env.GITEA_WORKFLOW;
+                    const currentRunId = process.env.GITHUB_RUN_ID || process.env.GITEA_RUN_ID;
+                    // 只检查同 PR 同名的其他 workflow run（排除当前 run）
+                    const duplicateReviewRuns = runningWorkflows.filter((w)=>w.sha === headSha && w.name === currentWorkflowName && (!currentRunId || String(w.id) !== currentRunId));
+                    if (duplicateReviewRuns.length > 0) {
+                        if (shouldLog(verbose, 1)) {
+                            console.log(`⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`);
+                        }
+                        return {
+                            success: true,
+                            description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
+                            issues: [],
+                            summary: [],
+                            round: 1
+                        };
+                    }
+                } catch (error) {
+                    // Gitea Actions API 可能返回 403（需要 repo owner 权限）
+                    // 捕获错误后跳过重复检查，继续执行审查
                     if (shouldLog(verbose, 1)) {
-                        console.log(`⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`);
+                        console.warn(`⚠️ 无法检查重复 workflow（可能缺少 repo owner 权限），跳过此检查:`, error instanceof Error ? error.message : error);
                     }
-                    return {
-                        success: true,
-                        description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
-                        issues: [],
-                        summary: [],
-                        round: 1
-                    };
                 }
             }
         } else if (effectiveBaseRef && effectiveHeadRef) {
@@ -2348,7 +2357,7 @@ class ReviewService {
                     summary: applicableSpecs.length === 0 ? "无适用的审查规则" : "已跳过"
                 }));
             const prInfo = context.generateDescription && llmMode ? await this.generatePrDescription(commits, changedFiles, llmMode, undefined, verbose) : await this.buildFallbackDescription(commits, changedFiles);
-            return {
+            const result = {
                 success: true,
                 title: prInfo.title,
                 description: prInfo.description,
@@ -2356,6 +2365,17 @@ class ReviewService {
                 summary,
                 round: 1
             };
+            // CI 模式下也需要发送 review 评论
+            if (ci && prNumber && !dryRun) {
+                if (shouldLog(verbose, 1)) {
+                    console.log(`💬 提交 PR 评论...`);
+                }
+                await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose, autoApprove);
+                if (shouldLog(verbose, 1)) {
+                    console.log(`✅ 评论已提交`);
+                }
+            }
+            return result;
         }
         const headSha = pr?.head?.sha || headRef || "HEAD";
         const fileContents = await this.getFileContents(owner, repo, changedFiles, commits, headSha, prNumber, verbose, isLocalMode);

package/package.json

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

package/src/review.service.ts

@@ -540,6 +540,7 @@ export class ReviewService {
 
       // 检查是否有其他同名 review workflow 正在运行中（防止同一 PR 重复审查）
       // 需要显式启用 skipDuplicateWorkflow 配置
+      // 注意：Gitea Actions API 要求用户是 repo owner，否则会返回 403
       if (skipDuplicateWorkflow && ci && pr?.head?.sha) {
         const headSha = pr.head.sha;
         // 获取当前 PR 编号（从 CI 环境变量）
@@ -549,32 +550,43 @@ export class ReviewService {
         const prMatch = ref.match(/refs\/pull\/(\d+)/);
         const currentPrNumber = prMatch ? parseInt(prMatch[1], 10) : prNumber;
 
-        const runningWorkflows = await this.gitProvider.listWorkflowRuns(owner, repo, {
-          status: "in_progress",
-        });
-        // 获取当前 workflow 名称和 run ID
-        const currentWorkflowName = process.env.GITHUB_WORKFLOW || process.env.GITEA_WORKFLOW;
-        const currentRunId = process.env.GITHUB_RUN_ID || process.env.GITEA_RUN_ID;
-        // 只检查同 PR 同名的其他 workflow run（排除当前 run）
-        const duplicateReviewRuns = runningWorkflows.filter(
-          (w) =>
-            w.sha === headSha &&
-            w.name === currentWorkflowName &&
-            (!currentRunId || String(w.id) !== currentRunId),
-        );
-        if (duplicateReviewRuns.length > 0) {
+        try {
+          const runningWorkflows = await this.gitProvider.listWorkflowRuns(owner, repo, {
+            status: "in_progress",
+          });
+          // 获取当前 workflow 名称和 run ID
+          const currentWorkflowName = process.env.GITHUB_WORKFLOW || process.env.GITEA_WORKFLOW;
+          const currentRunId = process.env.GITHUB_RUN_ID || process.env.GITEA_RUN_ID;
+          // 只检查同 PR 同名的其他 workflow run（排除当前 run）
+          const duplicateReviewRuns = runningWorkflows.filter(
+            (w) =>
+              w.sha === headSha &&
+              w.name === currentWorkflowName &&
+              (!currentRunId || String(w.id) !== currentRunId),
+          );
+          if (duplicateReviewRuns.length > 0) {
+            if (shouldLog(verbose, 1)) {
+              console.log(
+                `⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`,
+              );
+            }
+            return {
+              success: true,
+              description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
+              issues: [],
+              summary: [],
+              round: 1,
+            };
+          }
+        } catch (error) {
+          // Gitea Actions API 可能返回 403（需要 repo owner 权限）
+          // 捕获错误后跳过重复检查，继续执行审查
           if (shouldLog(verbose, 1)) {
-            console.log(
-              `⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`,
+            console.warn(
+              `⚠️ 无法检查重复 workflow（可能缺少 repo owner 权限），跳过此检查:`,
+              error instanceof Error ? error.message : error,
             );
           }
-          return {
-            success: true,
-            description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
-            issues: [],
-            summary: [],
-            round: 1,
-          };
         }
       }
     } else if (effectiveBaseRef && effectiveHeadRef) {
@@ -700,7 +712,7 @@ export class ReviewService {
         context.generateDescription && llmMode
           ? await this.generatePrDescription(commits, changedFiles, llmMode, undefined, verbose)
           : await this.buildFallbackDescription(commits, changedFiles);
-      return {
+      const result: ReviewResult = {
         success: true,
         title: prInfo.title,
         description: prInfo.description,
@@ -708,6 +720,19 @@ export class ReviewService {
         summary,
         round: 1,
       };
+
+      // CI 模式下也需要发送 review 评论
+      if (ci && prNumber && !dryRun) {
+        if (shouldLog(verbose, 1)) {
+          console.log(`💬 提交 PR 评论...`);
+        }
+        await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose, autoApprove);
+        if (shouldLog(verbose, 1)) {
+          console.log(`✅ 评论已提交`);
+        }
+      }
+
+      return result;
     }
 
     const headSha = pr?.head?.sha || headRef || "HEAD";