@spaceflow/review 5.0.1 → 5.0.2

package/CHANGELOG.md

@@ -1,5 +1,23 @@
 # Changelog
 
+## [5.0.1](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@5.0.0...@spaceflow/review@5.0.1) (2026-04-14)
+
+### 修复BUG
+
+* **review:** 修正已解决问题统计逻辑,避免与已修复问题重复计数 ([6d191bb](https://github.com/Lydanne/spaceflow/commit/6d191bbd71b8ce6194ce56a8d008dd966173541b))
+
+### 代码重构
+
+* **shared:** 移除 Claude Code 相关配置和文档,统一使用 OpenAI/Gemini/OpenCode ([b0754d6](https://github.com/Lydanne/spaceflow/commit/b0754d64f8991a10708730c191cf0f335e8aa6ca))
+
+### 其他修改
+
+* **cli:** released version 5.0.1 [no ci] ([f3009ea](https://github.com/Lydanne/spaceflow/commit/f3009eaaf26759d708541f54e707882b4927ebbd))
+* **core:** released version 5.0.1 [no ci] ([7904b9b](https://github.com/Lydanne/spaceflow/commit/7904b9bbc619c91a3a4fa1c8ea04d9dc548495ad))
+* **core:** 移除 @anthropic-ai/claude-agent-sdk 依赖及相关 sharp 图像处理包 ([7b50e06](https://github.com/Lydanne/spaceflow/commit/7b50e066df97d408eb63707eb27273592a28d34d))
+* **publish:** released version 5.0.1 [no ci] ([b14849f](https://github.com/Lydanne/spaceflow/commit/b14849ffafd38f230a3e69c2d63c468cb1946388))
+* **shared:** released version 5.0.1 [no ci] ([f70cd93](https://github.com/Lydanne/spaceflow/commit/f70cd930b01ffa09187f66522c8d6b18f835ec53))
+
 ## [4.0.1](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@4.0.0...@spaceflow/review@4.0.1) (2026-04-13)
 
 ### 新特性

package/dist/index.js

@@ -2254,7 +2254,7 @@ function generateIssueKey(issue) {
     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 pending = validTotal - validIssue.filter((i)=>i.fixed || i.resolved).length;
     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 {
@@ -3131,10 +3131,7 @@ function generateIssueKey(issue) {
         if (round > 1) {
             const prevIssues = allIssues.filter((i)=>i.round === round - 1);
             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" && !i.fixed && !i.resolved).length;
-                const prevPending = prevIssues.length - prevFixed - prevResolved - prevInvalid;
+                const { fixed: prevFixed, resolved: prevResolved, invalid: prevInvalid, pending: prevPending } = calculateIssueStats(prevIssues);
                 parts.push("");
                 parts.push(`<details><summary>📊 Round ${round - 1} 回顾 (${prevIssues.length} 个问题)</summary>\n`);
                 parts.push(`| 状态 | 数量 |`);

package/package.json

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

package/src/review-result-model.ts

@@ -1069,12 +1069,12 @@ export class ReviewResultModel {
     if (round > 1) {
       const prevIssues = allIssues.filter((i) => i.round === round - 1);
       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" && !i.fixed && !i.resolved,
-        ).length;
-        const prevPending = prevIssues.length - prevFixed - prevResolved - prevInvalid;
+        const {
+          fixed: prevFixed,
+          resolved: prevResolved,
+          invalid: prevInvalid,
+          pending: prevPending,
+        } = calculateIssueStats(prevIssues);
         parts.push("");
         parts.push(
           `<details><summary>📊 Round ${round - 1} 回顾 (${prevIssues.length} 个问题)</summary>\n`,

package/src/utils/review-pr-comment.spec.ts

@@ -27,9 +27,7 @@ describe("utils/review-pr-comment", () => {
 
   describe("extractIssueKeyFromBody", () => {
     it("提取标准格式的 issue key", () => {
-      expect(extractIssueKeyFromBody("<!-- issue-key: src/a.ts:10:R1 -->")).toBe(
-        "src/a.ts:10:R1",
-      );
+      expect(extractIssueKeyFromBody("<!-- issue-key: src/a.ts:10:R1 -->")).toBe("src/a.ts:10:R1");
     });
 
     it("body 不含 issue-key 时返回 null", () => {
@@ -69,9 +67,9 @@ describe("utils/review-pr-comment", () => {
 
   describe("generateIssueKey", () => {
     it("拼接 file:line:ruleId", () => {
-      expect(
-        generateIssueKey({ file: "src/a.ts", line: "10", ruleId: "R1" } as any),
-      ).toBe("src/a.ts:10:R1");
+      expect(generateIssueKey({ file: "src/a.ts", line: "10", ruleId: "R1" } as any)).toBe(
+        "src/a.ts:10:R1",
+      );
     });
   });
 
@@ -129,21 +127,20 @@ describe("utils/review-pr-comment", () => {
     });
 
     it("resolved（非 fixed）计入 resolved，不计入 pending", () => {
-      const issues = [
-        { file: "a.ts", line: "1", ruleId: "R1", resolved: "2024-01-01" },
-      ] as any[];
+      const issues = [{ file: "a.ts", line: "1", ruleId: "R1", resolved: "2024-01-01" }] as any[];
       const stats = calculateIssueStats(issues);
       expect(stats.resolved).toBe(1);
       expect(stats.pending).toBe(0);
     });
 
-    it("fixed 同时有 resolved 时只计入 fixed", () => {
+    it("fixed 和 resolved 同时存在时各自独立计数，pending 不重复减", () => {
       const issues = [
         { file: "a.ts", line: "1", ruleId: "R1", fixed: "2024-01-01", resolved: "2024-01-01" },
       ] as any[];
       const stats = calculateIssueStats(issues);
       expect(stats.fixed).toBe(1);
-      expect(stats.resolved).toBe(0);
+      expect(stats.resolved).toBe(1);
+      expect(stats.pending).toBe(0);
     });
 
     it("fixRate 计算正确（2/4 = 50%）", () => {

package/src/utils/review-pr-comment.ts

@@ -179,7 +179,7 @@ export function calculateIssueStats(issues: ReviewIssue[]): ReviewStats {
   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 pending = validTotal - validIssue.filter((i) => i.fixed || i.resolved).length;
   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 };