@spaceflow/review 0.68.0 → 0.69.0

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## [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,
@@ -2602,18 +2610,20 @@ 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;
+        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,
             fixed,
@@ -4314,6 +4324,7 @@ class IssueVerifyService {
                 }
                 verifiedIssues.push({
                     ...issue,
+                    resolved: new Date().toISOString(),
                     fixed: new Date().toISOString(),
                     valid: FALSE,
                     reason: "文件已删除"
@@ -4397,6 +4408,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.69.0",
   "description": "Spaceflow 代码审查插件，使用 LLM 对 PR 代码进行自动审查",
   "license": "MIT",
   "author": "Lydanne",

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

@@ -125,9 +125,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.service.ts

@@ -1079,7 +1079,7 @@ export class ReviewService {
       );
     }
 
-    return this.issueVerifyService.verifyIssueFixes(
+    return await this.issueVerifyService.verifyIssueFixes(
       issues,
       fileContents,
       specs,
@@ -1094,12 +1094,14 @@ 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;
+    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;
+    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, fixed, resolved, invalid, pending, fixRate, resolveRate };
   }