@spaceflow/review 0.68.0 → 0.70.0

package/CHANGELOG.md

@@ -1,5 +1,27 @@
 # Changelog
 
+## [0.69.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.68.0...@spaceflow/review@0.69.0) (2026-03-04)
+
+### 代码重构
+
+* **review:** 优化问题统计逻辑，区分 fixed 和 resolved 状态 ([c4dda30](https://github.com/Lydanne/spaceflow/commit/c4dda30fed17ce020fcce9af8874dfa89ccca20b))
+
+### 其他修改
+
+* **review-summary:** released version 0.36.0 [no ci] ([95c3d5c](https://github.com/Lydanne/spaceflow/commit/95c3d5cbac67c1ffa4c821faaffb476c502fe2c7))
+
+## [0.68.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.67.0...@spaceflow/review@0.68.0) (2026-03-04)
+
+### 代码重构
+
+* **review:** 区分 ☹️ 和 👎 reaction 的语义，☹️ 标记无效，👎 标记未解决 ([f1419fe](https://github.com/Lydanne/spaceflow/commit/f1419fe47448a80f373ffac082ac3a2e9320d200))
+
+### 其他修改
+
+* **review-summary:** released version 0.35.0 [no ci] ([4f2607d](https://github.com/Lydanne/spaceflow/commit/4f2607def2725946f32eccc4aa4e687a3cdd9bab))
+* **scripts:** released version 0.28.0 [no ci] ([55db5cf](https://github.com/Lydanne/spaceflow/commit/55db5cfa1dc0a1e318085caa0cfd9f91b06dcb21))
+* **shell:** released version 0.28.0 [no ci] ([01f180f](https://github.com/Lydanne/spaceflow/commit/01f180f2508e75524a33e66fea580a738adc689f))
+
 ## [0.67.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.66.0...@spaceflow/review@0.67.0) (2026-03-03)
 
 ### 新特性

package/dist/index.js

@@ -1169,24 +1169,28 @@ class MarkdownFormatter {
         if (summaries.length === 0) {
             return "没有需要审查的文件";
         }
-        // 🟢 已修复 | 🔴 待处理error | 🟡 待处理warn | ⚪ 已解决(非代码修复)
+        // 🟢 已修复 | 🔴 error数量 | 🟡 warn数量 | ⚪ 已解决(非代码修复)
         const issuesByFile = new Map();
         for (const issue of issues){
             if (issue.valid === "false") continue;
             const stats = issuesByFile.get(issue.file) || {
+                total: 0,
                 fixed: 0,
-                pendingErrors: 0,
-                pendingWarns: 0,
+                errorCount: 0,
+                warnCount: 0,
                 resolved: 0
             };
+            stats.total++;
             if (issue.fixed) {
                 stats.fixed++;
-            } else if (issue.resolved) {
+            }
+            if (issue.resolved) {
                 stats.resolved++;
-            } else if (issue.severity === "error") {
-                stats.pendingErrors++;
+            }
+            if (issue.severity === "error") {
+                stats.errorCount++;
             } else {
-                stats.pendingWarns++;
+                stats.warnCount++;
             }
             issuesByFile.set(issue.file, stats);
         }
@@ -1202,18 +1206,18 @@ class MarkdownFormatter {
         const fileSummaryLines = [];
         for (const fileSummary of summaries){
             const stats = issuesByFile.get(fileSummary.file) || {
+                total: 0,
                 fixed: 0,
-                pendingErrors: 0,
-                pendingWarns: 0,
+                errorCount: 0,
+                warnCount: 0,
                 resolved: 0
             };
-            const fileTotal = stats.fixed + stats.pendingErrors + stats.pendingWarns + stats.resolved;
-            totalAll += fileTotal;
+            totalAll += stats.total;
             totalFixed += stats.fixed;
-            totalPendingErrors += stats.pendingErrors;
-            totalPendingWarns += stats.pendingWarns;
+            totalPendingErrors += stats.errorCount;
+            totalPendingWarns += stats.warnCount;
             totalResolved += stats.resolved;
-            lines.push(`| \`${fileSummary.file}\` | ${fileTotal} | ${stats.fixed} | ${stats.pendingErrors} | ${stats.pendingWarns} | ${stats.resolved} |`);
+            lines.push(`| \`${fileSummary.file}\` | ${stats.total} | ${stats.fixed} | ${stats.errorCount} | ${stats.warnCount} | ${stats.resolved} |`);
             // 收集问题总结用于折叠块展示
             if (fileSummary.summary.trim()) {
                 fileSummaryLines.push(`### 💡 \`${fileSummary.file}\``);
@@ -1386,19 +1390,23 @@ class TerminalFormatter {
         for (const issue of issues){
             if (issue.valid === "false") continue;
             const stats = issuesByFile.get(issue.file) || {
+                total: 0,
                 fixed: 0,
-                pendingErrors: 0,
-                pendingWarns: 0,
+                errorCount: 0,
+                warnCount: 0,
                 resolved: 0
             };
+            stats.total++;
             if (issue.fixed) {
                 stats.fixed++;
-            } else if (issue.resolved) {
+            }
+            if (issue.resolved) {
                 stats.resolved++;
-            } else if (issue.severity === "error") {
-                stats.pendingErrors++;
+            }
+            if (issue.severity === "error") {
+                stats.errorCount++;
             } else {
-                stats.pendingWarns++;
+                stats.warnCount++;
             }
             issuesByFile.set(issue.file, stats);
         }
@@ -1411,21 +1419,21 @@ class TerminalFormatter {
         const lines = [];
         for (const fileSummary of summaries){
             const stats = issuesByFile.get(fileSummary.file) || {
+                total: 0,
                 fixed: 0,
-                pendingErrors: 0,
-                pendingWarns: 0,
+                errorCount: 0,
+                warnCount: 0,
                 resolved: 0
             };
-            const fileTotal = stats.fixed + stats.pendingErrors + stats.pendingWarns + stats.resolved;
-            totalAll += fileTotal;
+            totalAll += stats.total;
             totalFixed += stats.fixed;
-            totalPendingErrors += stats.pendingErrors;
-            totalPendingWarns += stats.pendingWarns;
+            totalPendingErrors += stats.errorCount;
+            totalPendingWarns += stats.warnCount;
             totalResolved += stats.resolved;
-            const totalText = fileTotal > 0 ? `${BOLD}${fileTotal} 问题${RESET}` : "";
+            const totalText = stats.total > 0 ? `${BOLD}${stats.total} 问题${RESET}` : "";
             const fixedText = stats.fixed > 0 ? `${GREEN}🟢 ${stats.fixed} 已修复${RESET}` : "";
-            const errorText = stats.pendingErrors > 0 ? `${RED}🔴 ${stats.pendingErrors} error${RESET}` : "";
-            const warnText = stats.pendingWarns > 0 ? `${YELLOW}🟡 ${stats.pendingWarns} warn${RESET}` : "";
+            const errorText = stats.errorCount > 0 ? `${RED}🔴 ${stats.errorCount} error${RESET}` : "";
+            const warnText = stats.warnCount > 0 ? `${YELLOW}🟡 ${stats.warnCount} warn${RESET}` : "";
             const resolvedText = stats.resolved > 0 ? `⚪ ${stats.resolved} 已解决` : "";
             const statsText = [
                 totalText,
@@ -1622,7 +1630,9 @@ class ReviewReportService {
         timeout: z.number().optional(),
         retries: z.number().default(0).optional(),
         retryDelay: z.number().default(1000).optional(),
-        invalidateChangedFiles: invalidateChangedFilesSchema.default("invalidate").optional()
+        invalidateChangedFiles: invalidateChangedFilesSchema.default("invalidate").optional(),
+        skipDuplicateWorkflow: z.boolean().default(false).optional(),
+        autoApprove: z.boolean().default(false).optional()
     });
 
 ;// CONCATENATED MODULE: ./src/parse-title-options.ts
@@ -2108,7 +2118,7 @@ class ReviewService {
    * @param context 审查上下文，包含 owner、repo、prNumber 等信息
    * @returns 审查结果，包含发现的问题列表和统计信息
    */ async execute(context) {
-        const { owner, repo, prNumber, baseRef, headRef, specSources, dryRun, ci, verbose, includes, llmMode, files, commits: filterCommits, deletionOnly, localMode } = context;
+        const { owner, repo, prNumber, baseRef, headRef, specSources, dryRun, ci, verbose, includes, llmMode, files, commits: filterCommits, deletionOnly, localMode, skipDuplicateWorkflow, autoApprove } = context;
         // 直接审查文件模式：指定了 -f 文件且 base=head
         const isDirectFileMode = files && files.length > 0 && baseRef === headRef;
         // 本地模式：审查未提交的代码（可能回退到分支比较）
@@ -2195,6 +2205,37 @@ class ReviewService {
                 console.log(`   Commits: ${commits.length}`);
                 console.log(`   Changed files: ${changedFiles.length}`);
             }
+            // 检查是否有其他同名 review workflow 正在运行中（防止同一 PR 重复审查）
+            // 需要显式启用 skipDuplicateWorkflow 配置
+            if (skipDuplicateWorkflow && ci && pr?.head?.sha) {
+                const headSha = pr.head.sha;
+                // 获取当前 PR 编号（从 CI 环境变量）
+                // GitHub: GITHUB_REF = refs/pull/123/merge
+                // Gitea: GITEA_REF = refs/pull/123/head
+                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) {
+                    if (shouldLog(verbose, 1)) {
+                        console.log(`⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`);
+                    }
+                    return {
+                        success: true,
+                        description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
+                        issues: [],
+                        summary: [],
+                        round: 1
+                    };
+                }
+            }
         } else if (effectiveBaseRef && effectiveHeadRef) {
             // 如果指定了 -f 文件且 base=head（无差异模式），直接审查指定文件
             if (files && files.length > 0 && effectiveBaseRef === effectiveHeadRef) {
@@ -2441,7 +2482,7 @@ class ReviewService {
             await this.postOrUpdateReviewComment(owner, repo, prNumber, {
                 ...result,
                 issues: allIssues
-            }, verbose);
+            }, verbose, autoApprove);
             if (shouldLog(verbose, 1)) {
                 console.log(`✅ 评论已提交`);
             }
@@ -2488,7 +2529,7 @@ class ReviewService {
    * 仅收集 review 状态模式（用于 PR 关闭或 --flush 指令）
    * 从现有的 AI review 评论中读取问题状态，同步已解决/无效状态，输出统计信息
    */ async executeCollectOnly(context) {
-        const { owner, repo, prNumber, verbose, ci, dryRun } = context;
+        const { owner, repo, prNumber, verbose, ci, dryRun, autoApprove } = context;
         if (shouldLog(verbose, 1)) {
             console.log(`📊 仅收集 review 状态模式`);
         }
@@ -2533,7 +2574,7 @@ class ReviewService {
             if (shouldLog(verbose, 1)) {
                 console.log(`💬 更新 PR 评论...`);
             }
-            await this.postOrUpdateReviewComment(owner, repo, prNumber, existingResult, verbose);
+            await this.postOrUpdateReviewComment(owner, repo, prNumber, existingResult, verbose, autoApprove);
             if (shouldLog(verbose, 1)) {
                 console.log(`✅ 评论已更新`);
             }
@@ -2602,20 +2643,23 @@ class ReviewService {
             specs = await this.loadSpecs(specSources, verbose);
             fileContents = await this.getFileContents(owner, repo, changedFiles, commits, headSha, prNumber, verbose);
         }
-        return this.issueVerifyService.verifyIssueFixes(issues, fileContents, specs, llmMode, verbose, context.verifyConcurrency);
+        return await this.issueVerifyService.verifyIssueFixes(issues, fileContents, specs, llmMode, verbose, context.verifyConcurrency);
     }
     /**
    * 计算问题状态统计
    */ calculateIssueStats(issues) {
         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" && !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;
+        const validIssue = issues.filter((i)=>i.valid !== "false");
+        const validTotal = validIssue.length;
+        const fixed = validIssue.filter((i)=>i.fixed).length;
+        const resolved = validIssue.filter((i)=>i.resolved).length;
+        const invalid = total - validTotal;
+        const pending = validTotal - fixed - resolved;
+        const fixRate = validTotal > 0 ? Math.round(fixed / validTotal * 100 * 10) / 10 : 0;
+        const resolveRate = validTotal > 0 ? Math.round(resolved / validTotal * 100 * 10) / 10 : 0;
         return {
             total,
+            validTotal,
             fixed,
             resolved,
             invalid,
@@ -2627,7 +2671,7 @@ class ReviewService {
     /**
    * 仅执行删除代码分析模式
    */ async executeDeletionOnly(context) {
-        const { owner, repo, prNumber, baseRef, headRef, dryRun, ci, verbose, llmMode } = context;
+        const { owner, repo, prNumber, baseRef, headRef, dryRun, ci, verbose, llmMode, autoApprove } = context;
         if (shouldLog(verbose, 1)) {
             console.log(`🗑️  仅执行删除代码分析模式`);
         }
@@ -2678,7 +2722,7 @@ class ReviewService {
             if (shouldLog(verbose, 1)) {
                 console.log(`💬 提交 PR 评论...`);
             }
-            await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose);
+            await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose, autoApprove);
             if (shouldLog(verbose, 1)) {
                 console.log(`✅ 评论已提交`);
             }
@@ -3415,7 +3459,7 @@ ${fileChanges || "无"}`;
         }
         return this.reviewReportService.format(result, format);
     }
-    async postOrUpdateReviewComment(owner, repo, prNumber, result, verbose) {
+    async postOrUpdateReviewComment(owner, repo, prNumber, result, verbose, autoApprove) {
         // 获取配置
         const reviewConf = this.config.getPluginConfig("review");
         // 如果配置启用且有 AI 生成的标题，只在第一轮审查时更新 PR 标题
@@ -3480,29 +3524,42 @@ ${fileChanges || "无"}`;
             console.warn("⚠️ 发布/更新 AI Review 评论失败:", error);
         }
         // 2. 发布本轮新发现的行级评论（使用 PR Review API，不删除旧的 review，保留历史）
+        // 如果启用 autoApprove 且所有问题已解决，使用 APPROVE event 合并发布
         let lineIssues = [];
         let comments = [];
         if (reviewConf.lineComments) {
             lineIssues = result.issues.filter((issue)=>issue.round === result.round && !issue.fixed && !issue.resolved && issue.valid !== "false");
             comments = lineIssues.map((issue)=>this.issueToReviewComment(issue)).filter((comment)=>comment !== null);
         }
+        // 计算是否需要自动批准
+        // 条件：启用 autoApprove 且没有待处理问题（包括从未发现问题的情况）
+        const stats = this.calculateIssueStats(result.issues);
+        const shouldAutoApprove = autoApprove && stats.pending === 0;
         if (reviewConf.lineComments) {
-            const reviewBody = this.buildLineReviewBody(lineIssues, result.round, result.issues);
+            const lineReviewBody = this.buildLineReviewBody(lineIssues, result.round, result.issues);
+            // 如果需要自动批准，追加批准信息到 body
+            const finalReviewBody = shouldAutoApprove ? lineReviewBody + `\n\n---\n\n✅ **自动批准合并**\n\n${stats.validTotal > 0 ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，` : "代码审查通过，未发现问题，"}自动批准此 PR。` : lineReviewBody;
+            const reviewEvent = shouldAutoApprove ? REVIEW_STATE.APPROVE : REVIEW_STATE.COMMENT;
             if (comments.length > 0) {
                 try {
                     await this.gitProvider.createPullReview(owner, repo, prNumber, {
-                        event: REVIEW_STATE.COMMENT,
-                        body: reviewBody,
+                        event: reviewEvent,
+                        body: finalReviewBody,
                         comments,
                         commit_id: commitId
                     });
-                    console.log(`✅ 已发布 ${comments.length} 条行级评论`);
+                    if (shouldAutoApprove) {
+                        console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+                    } else {
+                        console.log(`✅ 已发布 ${comments.length} 条行级评论`);
+                    }
                 } catch  {
                     // 批量失败时逐条发布，跳过无法定位的评论
                     console.warn("⚠️ 批量发布行级评论失败，尝试逐条发布...");
                     let successCount = 0;
                     for (const comment of comments){
                         try {
+                            // 逐条发布时只用 COMMENT event，避免重复 APPROVE
                             await this.gitProvider.createPullReview(owner, repo, prNumber, {
                                 event: REVIEW_STATE.COMMENT,
                                 body: successCount === 0 ? reviewBody : undefined,
@@ -3518,6 +3575,19 @@ ${fileChanges || "无"}`;
                     }
                     if (successCount > 0) {
                         console.log(`✅ 逐条发布成功 ${successCount}/${comments.length} 条行级评论`);
+                        // 如果需要自动批准，单独发一个 APPROVE review
+                        if (shouldAutoApprove) {
+                            try {
+                                await this.gitProvider.createPullReview(owner, repo, prNumber, {
+                                    event: REVIEW_STATE.APPROVE,
+                                    body: `✅ **自动批准合并**\n\n${stats.validTotal > 0 ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，` : "代码审查通过，未发现问题，"}自动批准此 PR。`,
+                                    commit_id: commitId
+                                });
+                                console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+                            } catch (error) {
+                                console.warn("⚠️ 自动批准失败:", error);
+                            }
+                        }
                     } else {
                         console.warn("⚠️ 所有行级评论均无法定位，已跳过");
                     }
@@ -3526,16 +3596,32 @@ ${fileChanges || "无"}`;
                 // 本轮无新问题，仍发布 Round 状态（含上轮回顾）
                 try {
                     await this.gitProvider.createPullReview(owner, repo, prNumber, {
-                        event: REVIEW_STATE.COMMENT,
-                        body: reviewBody,
+                        event: reviewEvent,
+                        body: finalReviewBody,
                         comments: [],
                         commit_id: commitId
                     });
-                    console.log(`✅ 已发布 Round ${result.round} 审查状态（无新问题）`);
+                    if (shouldAutoApprove) {
+                        console.log(`✅ 已自动批准 PR #${prNumber}（Round ${result.round}，所有问题已解决）`);
+                    } else {
+                        console.log(`✅ 已发布 Round ${result.round} 审查状态（无新问题）`);
+                    }
                 } catch (error) {
                     console.warn("⚠️ 发布审查状态失败:", error);
                 }
             }
+        } else if (shouldAutoApprove) {
+            // 未启用 lineComments 但需要自动批准
+            try {
+                await this.gitProvider.createPullReview(owner, repo, prNumber, {
+                    event: REVIEW_STATE.APPROVE,
+                    body: `✅ **自动批准合并**\n\n${stats.validTotal > 0 ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，` : "代码审查通过，未发现问题，"}自动批准此 PR。`,
+                    commit_id: commitId
+                });
+                console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+            } catch (error) {
+                console.warn("⚠️ 自动批准失败:", error);
+            }
         }
     }
     /**
@@ -4314,6 +4400,7 @@ class IssueVerifyService {
                 }
                 verifiedIssues.push({
                     ...issue,
+                    resolved: new Date().toISOString(),
                     fixed: new Date().toISOString(),
                     valid: FALSE,
                     reason: "文件已删除"
@@ -4397,6 +4484,7 @@ class IssueVerifyService {
                         console.log(`      ✅ 已修复: ${result.reason}`);
                     }
                     updatedIssue.fixed = new Date().toISOString();
+                    updatedIssue.resolved = new Date().toISOString();
                 } else if (!result.valid) {
                     if (shouldLog(verbose, 1)) {
                         console.log(`      ❌ 无效问题: ${result.reason}`);

package/package.json

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

package/src/issue-verify.service.ts

@@ -107,6 +107,7 @@ export class IssueVerifyService {
         }
         verifiedIssues.push({
           ...issue,
+          resolved: new Date().toISOString(),
           fixed: new Date().toISOString(),
           valid: FALSE,
           reason: "文件已删除",
@@ -209,6 +210,7 @@ export class IssueVerifyService {
             console.log(`      ✅ 已修复: ${result.reason}`);
           }
           updatedIssue.fixed = new Date().toISOString();
+          updatedIssue.resolved = new Date().toISOString();
         } else if (!result.valid) {
           if (shouldLog(verbose, 1)) {
             console.log(`      ❌ 无效问题: ${result.reason}`);

package/src/review-report/formatters/markdown.formatter.ts

@@ -130,27 +130,37 @@ export class MarkdownFormatter implements ReviewReportFormatter, ReviewReportPar
       return "没有需要审查的文件";
     }
 
-    // 🟢 已修复 | 🔴 待处理error | 🟡 待处理warn | ⚪ 已解决(非代码修复)
+    // 🟢 已修复 | 🔴 error数量 | 🟡 warn数量 | ⚪ 已解决(非代码修复)
     const issuesByFile = new Map<
       string,
-      { fixed: number; pendingErrors: number; pendingWarns: number; resolved: number }
+      {
+        fixed: number;
+        errorCount: number;
+        warnCount: number;
+        resolved: number;
+        total: number;
+      }
     >();
     for (const issue of issues) {
       if (issue.valid === "false") continue;
       const stats = issuesByFile.get(issue.file) || {
+        total: 0,
         fixed: 0,
-        pendingErrors: 0,
-        pendingWarns: 0,
+        errorCount: 0,
+        warnCount: 0,
         resolved: 0,
       };
+      stats.total++;
       if (issue.fixed) {
         stats.fixed++;
-      } else if (issue.resolved) {
+      }
+      if (issue.resolved) {
         stats.resolved++;
-      } else if (issue.severity === "error") {
-        stats.pendingErrors++;
+      }
+      if (issue.severity === "error") {
+        stats.errorCount++;
       } else {
-        stats.pendingWarns++;
+        stats.warnCount++;
       }
       issuesByFile.set(issue.file, stats);
     }
@@ -169,20 +179,20 @@ export class MarkdownFormatter implements ReviewReportFormatter, ReviewReportPar
     const fileSummaryLines: string[] = [];
     for (const fileSummary of summaries) {
       const stats = issuesByFile.get(fileSummary.file) || {
+        total: 0,
         fixed: 0,
-        pendingErrors: 0,
-        pendingWarns: 0,
+        errorCount: 0,
+        warnCount: 0,
         resolved: 0,
       };
-      const fileTotal = stats.fixed + stats.pendingErrors + stats.pendingWarns + stats.resolved;
-      totalAll += fileTotal;
+      totalAll += stats.total;
       totalFixed += stats.fixed;
-      totalPendingErrors += stats.pendingErrors;
-      totalPendingWarns += stats.pendingWarns;
+      totalPendingErrors += stats.errorCount;
+      totalPendingWarns += stats.warnCount;
       totalResolved += stats.resolved;
 
       lines.push(
-        `| \`${fileSummary.file}\` | ${fileTotal} | ${stats.fixed} | ${stats.pendingErrors} | ${stats.pendingWarns} | ${stats.resolved} |`,
+        `| \`${fileSummary.file}\` | ${stats.total} | ${stats.fixed} | ${stats.errorCount} | ${stats.warnCount} | ${stats.resolved} |`,
       );
 
       // 收集问题总结用于折叠块展示

package/src/review-report/formatters/terminal.formatter.ts

@@ -31,24 +31,34 @@ export class TerminalFormatter implements ReviewReportFormatter {
     // 🟢 已修复 | 🔴 待处理error | 🟡 待处理warn | ⚪ 已解决(非代码修复)
     const issuesByFile = new Map<
       string,
-      { fixed: number; pendingErrors: number; pendingWarns: number; resolved: number }
+      {
+        fixed: number;
+        errorCount: number;
+        warnCount: number;
+        resolved: number;
+        total: number;
+      }
     >();
     for (const issue of issues) {
       if (issue.valid === "false") continue;
       const stats = issuesByFile.get(issue.file) || {
+        total: 0,
         fixed: 0,
-        pendingErrors: 0,
-        pendingWarns: 0,
+        errorCount: 0,
+        warnCount: 0,
         resolved: 0,
       };
+      stats.total++;
       if (issue.fixed) {
         stats.fixed++;
-      } else if (issue.resolved) {
+      }
+      if (issue.resolved) {
         stats.resolved++;
-      } else if (issue.severity === "error") {
-        stats.pendingErrors++;
+      }
+      if (issue.severity === "error") {
+        stats.errorCount++;
       } else {
-        stats.pendingWarns++;
+        stats.warnCount++;
       }
       issuesByFile.set(issue.file, stats);
     }
@@ -63,24 +73,22 @@ export class TerminalFormatter implements ReviewReportFormatter {
     const lines: string[] = [];
     for (const fileSummary of summaries) {
       const stats = issuesByFile.get(fileSummary.file) || {
+        total: 0,
         fixed: 0,
-        pendingErrors: 0,
-        pendingWarns: 0,
+        errorCount: 0,
+        warnCount: 0,
         resolved: 0,
       };
-      const fileTotal = stats.fixed + stats.pendingErrors + stats.pendingWarns + stats.resolved;
-      totalAll += fileTotal;
+      totalAll += stats.total;
       totalFixed += stats.fixed;
-      totalPendingErrors += stats.pendingErrors;
-      totalPendingWarns += stats.pendingWarns;
+      totalPendingErrors += stats.errorCount;
+      totalPendingWarns += stats.warnCount;
       totalResolved += stats.resolved;
 
-      const totalText = fileTotal > 0 ? `${BOLD}${fileTotal} 问题${RESET}` : "";
+      const totalText = stats.total > 0 ? `${BOLD}${stats.total} 问题${RESET}` : "";
       const fixedText = stats.fixed > 0 ? `${GREEN}🟢 ${stats.fixed} 已修复${RESET}` : "";
-      const errorText =
-        stats.pendingErrors > 0 ? `${RED}🔴 ${stats.pendingErrors} error${RESET}` : "";
-      const warnText =
-        stats.pendingWarns > 0 ? `${YELLOW}🟡 ${stats.pendingWarns} warn${RESET}` : "";
+      const errorText = stats.errorCount > 0 ? `${RED}🔴 ${stats.errorCount} error${RESET}` : "";
+      const warnText = stats.warnCount > 0 ? `${YELLOW}🟡 ${stats.warnCount} warn${RESET}` : "";
       const resolvedText = stats.resolved > 0 ? `⚪ ${stats.resolved} 已解决` : "";
       const statsText = [totalText, fixedText, errorText, warnText, resolvedText]
         .filter(Boolean)

package/src/review-spec/types.ts

@@ -117,6 +117,8 @@ export interface DeletionImpactResult {
 export interface ReviewStats {
   /** 总问题数 */
   total: number;
+  /** 有效问题数（排除无效） */
+  validTotal: number;
   /** AI 验证已修复数 */
   fixed: number;
   /** 用户手动 resolve 数 */
@@ -125,9 +127,9 @@ export interface ReviewStats {
   invalid: number;
   /** 待处理数 */
   pending: number;
-  /** 修复率 (0-100)，仅计算代码修复：fixed / total */
+  /** 修复率 (0-100)，仅计算代码修复：fixed / validTotal */
   fixRate: number;
-  /** 解决率 (0-100)，计算修复+解决：(fixed + resolved) / total */
+  /** 解决率 (0-100)，计算已解决：resolved / validTotal */
   resolveRate: number;
 }
 

package/src/review.config.ts

@@ -79,6 +79,18 @@ export interface ReviewOptions {
    * 在非 CI 和非 PR 模式下默认为 'uncommitted'
    */
   local?: LocalReviewMode;
+  /**
+   * 跳过重复的 review workflow 检查
+   * - true: 启用检查，当检测到同名 workflow 正在运行时跳过审查
+   * - false: 禁用检查（默认）
+   */
+  skipDuplicateWorkflow?: boolean;
+  /**
+   * 自动批准合并
+   * - true: 当所有问题都已解决时，自动提交 APPROVE review
+   * - false: 不自动批准（默认）
+   */
+  autoApprove?: boolean;
 }
 
 /** review 命令配置 schema（LLM 敏感配置由系统 llm.config.ts 管理） */
@@ -100,6 +112,8 @@ export const reviewSchema = () =>
     retries: z.number().default(0).optional(),
     retryDelay: z.number().default(1000).optional(),
     invalidateChangedFiles: invalidateChangedFilesSchema.default("invalidate").optional(),
+    skipDuplicateWorkflow: z.boolean().default(false).optional(),
+    autoApprove: z.boolean().default(false).optional(),
   });
 
 /** review 配置类型（从 schema 推导） */

package/src/review.service.ts

@@ -433,6 +433,8 @@ export class ReviewService {
       commits: filterCommits,
       deletionOnly,
       localMode,
+      skipDuplicateWorkflow,
+      autoApprove,
     } = context;
 
     // 直接审查文件模式：指定了 -f 文件且 base=head
@@ -532,6 +534,46 @@ export class ReviewService {
         console.log(`   Commits: ${commits.length}`);
         console.log(`   Changed files: ${changedFiles.length}`);
       }
+
+      // 检查是否有其他同名 review workflow 正在运行中（防止同一 PR 重复审查）
+      // 需要显式启用 skipDuplicateWorkflow 配置
+      if (skipDuplicateWorkflow && ci && pr?.head?.sha) {
+        const headSha = pr.head.sha;
+        // 获取当前 PR 编号（从 CI 环境变量）
+        // GitHub: GITHUB_REF = refs/pull/123/merge
+        // Gitea: GITEA_REF = refs/pull/123/head
+        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) {
+          if (shouldLog(verbose, 1)) {
+            console.log(
+              `⏭️ 跳过审查: 当前 PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中`,
+            );
+          }
+          return {
+            success: true,
+            description: `跳过审查: PR #${currentPrNumber} 有 ${duplicateReviewRuns.length} 个同名 workflow 正在运行中，等待完成后重新审查`,
+            issues: [],
+            summary: [],
+            round: 1,
+          };
+        }
+      }
     } else if (effectiveBaseRef && effectiveHeadRef) {
       // 如果指定了 -f 文件且 base=head（无差异模式），直接审查指定文件
       if (files && files.length > 0 && effectiveBaseRef === effectiveHeadRef) {
@@ -856,6 +898,7 @@ export class ReviewService {
           issues: allIssues,
         },
         verbose,
+        autoApprove,
       );
       if (shouldLog(verbose, 1)) {
         console.log(`✅ 评论已提交`);
@@ -916,7 +959,7 @@ export class ReviewService {
    * 从现有的 AI review 评论中读取问题状态，同步已解决/无效状态，输出统计信息
    */
   protected async executeCollectOnly(context: ReviewContext): Promise<ReviewResult> {
-    const { owner, repo, prNumber, verbose, ci, dryRun } = context;
+    const { owner, repo, prNumber, verbose, ci, dryRun, autoApprove } = context;
 
     if (shouldLog(verbose, 1)) {
       console.log(`📊 仅收集 review 状态模式`);
@@ -982,7 +1025,14 @@ export class ReviewService {
       if (shouldLog(verbose, 1)) {
         console.log(`💬 更新 PR 评论...`);
       }
-      await this.postOrUpdateReviewComment(owner, repo, prNumber, existingResult, verbose);
+      await this.postOrUpdateReviewComment(
+        owner,
+        repo,
+        prNumber,
+        existingResult,
+        verbose,
+        autoApprove,
+      );
       if (shouldLog(verbose, 1)) {
         console.log(`✅ 评论已更新`);
       }
@@ -1079,7 +1129,7 @@ export class ReviewService {
       );
     }
 
-    return this.issueVerifyService.verifyIssueFixes(
+    return await this.issueVerifyService.verifyIssueFixes(
       issues,
       fileContents,
       specs,
@@ -1094,20 +1144,23 @@ export class ReviewService {
    */
   protected calculateIssueStats(issues: ReviewIssue[]): ReviewStats {
     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" && !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;
-    return { total, fixed, resolved, invalid, pending, fixRate, resolveRate };
+    const validIssue = issues.filter((i) => i.valid !== "false");
+    const validTotal = validIssue.length;
+    const fixed = validIssue.filter((i) => i.fixed).length;
+    const resolved = validIssue.filter((i) => i.resolved).length;
+    const invalid = total - validTotal;
+    const pending = validTotal - fixed - resolved;
+    const fixRate = validTotal > 0 ? Math.round((fixed / validTotal) * 100 * 10) / 10 : 0;
+    const resolveRate = validTotal > 0 ? Math.round((resolved / validTotal) * 100 * 10) / 10 : 0;
+    return { total, validTotal, fixed, resolved, invalid, pending, fixRate, resolveRate };
   }
 
   /**
    * 仅执行删除代码分析模式
    */
   protected async executeDeletionOnly(context: ReviewContext): Promise<ReviewResult> {
-    const { owner, repo, prNumber, baseRef, headRef, dryRun, ci, verbose, llmMode } = context;
+    const { owner, repo, prNumber, baseRef, headRef, dryRun, ci, verbose, llmMode, autoApprove } =
+      context;
 
     if (shouldLog(verbose, 1)) {
       console.log(`🗑️  仅执行删除代码分析模式`);
@@ -1172,7 +1225,7 @@ export class ReviewService {
       if (shouldLog(verbose, 1)) {
         console.log(`💬 提交 PR 评论...`);
       }
-      await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose);
+      await this.postOrUpdateReviewComment(owner, repo, prNumber, result, verbose, autoApprove);
       if (shouldLog(verbose, 1)) {
         console.log(`✅ 评论已提交`);
       }
@@ -2061,6 +2114,7 @@ ${fileChanges || "无"}`;
     prNumber: number,
     result: ReviewResult,
     verbose?: VerboseLevel,
+    autoApprove?: boolean,
   ): Promise<void> {
     // 获取配置
     const reviewConf = this.config.getPluginConfig<ReviewConfig>("review");
@@ -2135,6 +2189,7 @@ ${fileChanges || "无"}`;
     }
 
     // 2. 发布本轮新发现的行级评论（使用 PR Review API，不删除旧的 review，保留历史）
+    // 如果启用 autoApprove 且所有问题已解决，使用 APPROVE event 合并发布
     let lineIssues: ReviewIssue[] = [];
     let comments: CreatePullReviewComment[] = [];
     if (reviewConf.lineComments) {
@@ -2149,23 +2204,47 @@ ${fileChanges || "无"}`;
         .map((issue) => this.issueToReviewComment(issue))
         .filter((comment): comment is CreatePullReviewComment => comment !== null);
     }
+
+    // 计算是否需要自动批准
+    // 条件：启用 autoApprove 且没有待处理问题（包括从未发现问题的情况）
+    const stats = this.calculateIssueStats(result.issues);
+    const shouldAutoApprove = autoApprove && stats.pending === 0;
+
     if (reviewConf.lineComments) {
-      const reviewBody = this.buildLineReviewBody(lineIssues, result.round, result.issues);
+      const lineReviewBody = this.buildLineReviewBody(lineIssues, result.round, result.issues);
+
+      // 如果需要自动批准，追加批准信息到 body
+      const finalReviewBody = shouldAutoApprove
+        ? lineReviewBody +
+          `\n\n---\n\n✅ **自动批准合并**\n\n${
+            stats.validTotal > 0
+              ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，`
+              : "代码审查通过，未发现问题，"
+          }自动批准此 PR。`
+        : lineReviewBody;
+
+      const reviewEvent = shouldAutoApprove ? REVIEW_STATE.APPROVE : REVIEW_STATE.COMMENT;
+
       if (comments.length > 0) {
         try {
           await this.gitProvider.createPullReview(owner, repo, prNumber, {
-            event: REVIEW_STATE.COMMENT,
-            body: reviewBody,
+            event: reviewEvent,
+            body: finalReviewBody,
             comments,
             commit_id: commitId,
           });
-          console.log(`✅ 已发布 ${comments.length} 条行级评论`);
+          if (shouldAutoApprove) {
+            console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+          } else {
+            console.log(`✅ 已发布 ${comments.length} 条行级评论`);
+          }
         } catch {
           // 批量失败时逐条发布，跳过无法定位的评论
           console.warn("⚠️ 批量发布行级评论失败，尝试逐条发布...");
           let successCount = 0;
           for (const comment of comments) {
             try {
+              // 逐条发布时只用 COMMENT event，避免重复 APPROVE
               await this.gitProvider.createPullReview(owner, repo, prNumber, {
                 event: REVIEW_STATE.COMMENT,
                 body: successCount === 0 ? reviewBody : undefined,
@@ -2179,6 +2258,23 @@ ${fileChanges || "无"}`;
           }
           if (successCount > 0) {
             console.log(`✅ 逐条发布成功 ${successCount}/${comments.length} 条行级评论`);
+            // 如果需要自动批准，单独发一个 APPROVE review
+            if (shouldAutoApprove) {
+              try {
+                await this.gitProvider.createPullReview(owner, repo, prNumber, {
+                  event: REVIEW_STATE.APPROVE,
+                  body: `✅ **自动批准合并**\n\n${
+                    stats.validTotal > 0
+                      ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，`
+                      : "代码审查通过，未发现问题，"
+                  }自动批准此 PR。`,
+                  commit_id: commitId,
+                });
+                console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+              } catch (error) {
+                console.warn("⚠️ 自动批准失败:", error);
+              }
+            }
           } else {
             console.warn("⚠️ 所有行级评论均无法定位，已跳过");
           }
@@ -2187,16 +2283,36 @@ ${fileChanges || "无"}`;
         // 本轮无新问题，仍发布 Round 状态（含上轮回顾）
         try {
           await this.gitProvider.createPullReview(owner, repo, prNumber, {
-            event: REVIEW_STATE.COMMENT,
-            body: reviewBody,
+            event: reviewEvent,
+            body: finalReviewBody,
             comments: [],
             commit_id: commitId,
           });
-          console.log(`✅ 已发布 Round ${result.round} 审查状态（无新问题）`);
+          if (shouldAutoApprove) {
+            console.log(`✅ 已自动批准 PR #${prNumber}（Round ${result.round}，所有问题已解决）`);
+          } else {
+            console.log(`✅ 已发布 Round ${result.round} 审查状态（无新问题）`);
+          }
         } catch (error) {
           console.warn("⚠️ 发布审查状态失败:", error);
         }
       }
+    } else if (shouldAutoApprove) {
+      // 未启用 lineComments 但需要自动批准
+      try {
+        await this.gitProvider.createPullReview(owner, repo, prNumber, {
+          event: REVIEW_STATE.APPROVE,
+          body: `✅ **自动批准合并**\n\n${
+            stats.validTotal > 0
+              ? `所有问题都已解决 (${stats.fixed} 已修复, ${stats.resolved} 已解决)，`
+              : "代码审查通过，未发现问题，"
+          }自动批准此 PR。`,
+          commit_id: commitId,
+        });
+        console.log(`✅ 已自动批准 PR #${prNumber}（所有问题已解决）`);
+      } catch (error) {
+        console.warn("⚠️ 自动批准失败:", error);
+      }
     }
   }