@spaceflow/review 0.63.0 → 0.64.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## [0.63.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.62.0...@spaceflow/review@0.63.0) (2026-03-03)
+
+### 代码重构
+
+* **review:** 为文件总结标题添加 💡 图标,增强视觉识别度 ([69cecf0](https://github.com/Lydanne/spaceflow/commit/69cecf0a6deadf1935db060800f8f110ae4b9889))
+
+### 其他修改
+
+* **review-summary:** released version 0.30.0 [no ci] ([3902c7b](https://github.com/Lydanne/spaceflow/commit/3902c7be16ceb6ab7ff2abe52e434634de12a664))
+
 ## [0.62.0](https://github.com/Lydanne/spaceflow/compare/@spaceflow/review@0.61.0...@spaceflow/review@0.62.0) (2026-03-03)
 
 ### 代码重构

package/package.json

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

package/src/review.service.spec.ts

@@ -75,6 +75,7 @@ describe("ReviewService", () => {
       updateIssueComment: vi.fn().mockResolvedValue({}),
       deleteIssueComment: vi.fn().mockResolvedValue(undefined),
       updatePullReview: vi.fn().mockResolvedValue({}),
+      getTeamMembers: vi.fn().mockResolvedValue([]),
     };
 
     configService = {
@@ -1246,6 +1247,7 @@ describe("ReviewService", () => {
         invalid: 0,
         pending: 0,
         fixRate: 0,
+        resolveRate: 0,
       });
     });
 
@@ -1555,6 +1557,19 @@ describe("ReviewService", () => {
       expect(consoleSpy).toHaveBeenCalled();
       consoleSpy.mockRestore();
     });
+
+    it("should log error when deleting comment fails", async () => {
+      const consoleSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
+      gitProvider.listPullReviews.mockResolvedValue([] as any);
+      gitProvider.listIssueComments.mockResolvedValue([
+        { id: 10, body: "<!-- spaceflow-review --> old comment" },
+      ] as any);
+      gitProvider.deleteIssueComment.mockRejectedValue(new Error("delete failed"));
+
+      await (service as any).deleteExistingAiReviews("o", "r", 1);
+      expect(consoleSpy).toHaveBeenCalledWith("⚠️ 删除评论 10 失败:", expect.any(Error));
+      consoleSpy.mockRestore();
+    });
   });
 
   describe("ReviewService.invalidateIssuesForChangedFiles", () => {
@@ -2381,11 +2396,48 @@ describe("ReviewService", () => {
         { content: "-1", user: { login: "random-user" } },
       ] as any);
       const result = { issues: [{ file: "test.ts", line: "10", valid: "true", reactions: [] }] };
-      await (service as any).syncReactionsToIssues("o", "r", 1, result);
       expect(result.issues[0].valid).toBe("true");
     });
   });
 
+  describe("ReviewService.buildLineReviewBody", () => {
+    it("should include previous round summary when round > 1", () => {
+      const issues = [
+        { round: 2, fixed: "2024-01-01", resolved: undefined, valid: undefined },
+        { round: 2, resolved: "2024-01-02", fixed: undefined, valid: undefined },
+        { round: 2, valid: "false", fixed: undefined, resolved: undefined },
+        { round: 2, fixed: undefined, resolved: undefined, valid: undefined },
+      ];
+      const allIssues = [
+        ...issues,
+        { round: 1, fixed: "2024-01-01" },
+        { round: 1, resolved: "2024-01-02" },
+        { round: 1, valid: "false" },
+        { round: 1 },
+      ];
+      const result = (service as any).buildLineReviewBody(issues, 2, allIssues);
+      expect(result).toContain("Round 1 回顾");
+      expect(result).toContain("🟢 已修复 | 1");
+      expect(result).toContain("⚪ 已解决 | 1");
+      expect(result).toContain("❌ 无效 | 1");
+      expect(result).toContain("⚠️ 待处理 | 1");
+    });
+
+    it("should not include previous round summary when round <= 1", () => {
+      const issues = [{ round: 1 }];
+      const allIssues = [{ round: 1 }];
+      const result = (service as any).buildLineReviewBody(issues, 1, allIssues);
+      expect(result).not.toContain("Round 1 回顾");
+    });
+
+    it("should show no issues message when issues array is empty", () => {
+      const issues = [];
+      const allIssues = [];
+      const result = (service as any).buildLineReviewBody(issues, 1, allIssues);
+      expect(result).toContain("✅ 未发现新问题");
+    });
+  });
+
   describe("ReviewService.buildReviewPrompt", () => {
     it("should build prompts for changed files", async () => {
       const specs = [{ extensions: ["ts"], includes: [], rules: [{ id: "R1" }] }];
@@ -3226,4 +3278,450 @@ describe("ReviewService", () => {
       expect(result.size).toBe(0);
     });
   });
+
+  describe("ReviewService.invalidateIssuesForChangedFiles", () => {
+    it("should return issues unchanged when no headSha", async () => {
+      const issues = [{ file: "test.ts" }];
+      const result = await (service as any).invalidateIssuesForChangedFiles(
+        issues,
+        undefined,
+        "o",
+        "r",
+      );
+      expect(result).toBe(issues);
+    });
+
+    it("should log warning when no headSha", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const issues = [{ file: "test.ts" }];
+      await (service as any).invalidateIssuesForChangedFiles(issues, undefined, "o", "r", 1);
+      expect(consoleSpy).toHaveBeenCalledWith("   ⚠️ 无法获取 PR head SHA，跳过变更文件检查");
+      consoleSpy.mockRestore();
+    });
+
+    it("should invalidate issues for changed files", async () => {
+      gitProvider.getCommitDiff = vi
+        .fn()
+        .mockResolvedValue(
+          "diff --git a/changed.ts b/changed.ts\n--- a/changed.ts\n+++ b/changed.ts\n@@ -1,1 +1,2 @@\n line1\n+new",
+        ) as any;
+      const issues = [
+        { file: "changed.ts", line: "1", ruleId: "R1" },
+        { file: "unchanged.ts", line: "2", ruleId: "R2" },
+        { file: "changed.ts", line: "3", ruleId: "R3", fixed: "2024-01-01" },
+      ];
+      const result = await (service as any).invalidateIssuesForChangedFiles(
+        issues,
+        "abc123",
+        "o",
+        "r",
+        1,
+      );
+      expect(result).toHaveLength(3);
+      expect(result[0].valid).toBe("false");
+      expect(result[1].valid).toBeUndefined();
+      expect(result[2].fixed).toBe("2024-01-01");
+    });
+
+    it("should log when files are invalidated", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      gitProvider.getCommitDiff = vi
+        .fn()
+        .mockResolvedValue(
+          "diff --git a/changed.ts b/changed.ts\n--- a/changed.ts\n+++ b/changed.ts\n@@ -1,1 +1,2 @@\n line1\n+new",
+        ) as any;
+      const issues = [{ file: "changed.ts", line: "1", ruleId: "R1" }];
+      await (service as any).invalidateIssuesForChangedFiles(issues, "abc123", "o", "r", 1);
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "   🗑️ Issue changed.ts:1 所在文件有变更，标记为无效",
+      );
+      expect(consoleSpy).toHaveBeenCalledWith("   📊 共标记 1 个历史问题为无效（文件有变更）");
+      consoleSpy.mockRestore();
+    });
+
+    it("should return issues unchanged when no diff files", async () => {
+      gitProvider.getCommitDiff = vi.fn().mockResolvedValue("") as any;
+      const issues = [{ file: "test.ts", line: "1" }];
+      const result = await (service as any).invalidateIssuesForChangedFiles(
+        issues,
+        "abc123",
+        "o",
+        "r",
+        1,
+      );
+      expect(result).toBe(issues);
+    });
+
+    it("should log when no diff files", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      gitProvider.getCommitDiff = vi.fn().mockResolvedValue("") as any;
+      const issues = [{ file: "test.ts", line: "1" }];
+      await (service as any).invalidateIssuesForChangedFiles(issues, "abc123", "o", "r", 1);
+      expect(consoleSpy).toHaveBeenCalledWith("   ⏭️ 最新 commit 无文件变更");
+      consoleSpy.mockRestore();
+    });
+
+    it("should handle API error gracefully", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      gitProvider.getCommitDiff = vi.fn().mockRejectedValue(new Error("fail")) as any;
+      const issues = [{ file: "test.ts", line: "1" }];
+      const result = await (service as any).invalidateIssuesForChangedFiles(
+        issues,
+        "abc123",
+        "o",
+        "r",
+        1,
+      );
+      expect(result).toBe(issues);
+      expect(consoleSpy).toHaveBeenCalledWith("   ⚠️ 获取最新 commit 变更文件失败: Error: fail");
+      consoleSpy.mockRestore();
+    });
+  });
+
+  describe("ReviewService.updateIssueLineNumbers", () => {
+    beforeEach(() => {
+      mockReviewSpecService.parseLineRange = vi.fn().mockImplementation((lineStr: string) => {
+        const lines: number[] = [];
+        const rangeMatch = lineStr.match(/^(\d+)-(\d+)$/);
+        if (rangeMatch) {
+          const start = parseInt(rangeMatch[1], 10);
+          const end = parseInt(rangeMatch[2], 10);
+          for (let i = start; i <= end; i++) {
+            lines.push(i);
+          }
+        } else {
+          const line = parseInt(lineStr, 10);
+          if (!isNaN(line)) {
+            lines.push(line);
+          }
+        }
+        return lines;
+      });
+    });
+
+    it("should return issues unchanged when no patch for file", () => {
+      const issues = [{ file: "test.ts", line: "5", ruleId: "R1" }];
+      const filePatchMap = new Map([["other.ts", "@@ -1,1 +1,2 @@\n-old1\n+new1\n+new2"]]);
+      const result = (service as any).updateIssueLineNumbers(issues, filePatchMap);
+      expect(result).toEqual(issues);
+    });
+
+    it("should skip issues that are already fixed/resolved/invalid", () => {
+      const issues = [
+        { file: "test.ts", line: "5", ruleId: "R1", fixed: "2024-01-01" },
+        { file: "test.ts", line: "6", ruleId: "R2", resolved: "2024-01-02" },
+        { file: "test.ts", line: "7", ruleId: "R3", valid: "false" },
+      ];
+      const filePatchMap = new Map([["test.ts", "@@ -1,1 +1,2 @@\n-old1\n+new1\n+new2"]]);
+      const result = (service as any).updateIssueLineNumbers(issues, filePatchMap);
+      expect(result).toEqual(issues);
+    });
+
+    it("should mark issue as invalid when line is deleted", () => {
+      const filePatchMap = new Map([["test.ts", "@@ -1,1 +1,0 @@\n-old1"]]);
+      const issues = [{ file: "test.ts", line: "1", ruleId: "R1" }];
+      const result = (service as any).updateIssueLineNumbers(issues, filePatchMap);
+      expect(result[0].valid).toBe("false");
+      expect(result[0].originalLine).toBe("1");
+    });
+
+    it("should log when line is deleted and marked invalid", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const filePatchMap = new Map([["test.ts", "@@ -1,1 +1,0 @@\n-old1"]]);
+      const issues = [{ file: "test.ts", line: "1", ruleId: "R1" }];
+      (service as any).updateIssueLineNumbers(issues, filePatchMap, 1);
+      expect(consoleSpy).toHaveBeenCalledWith("📍 Issue test.ts:1 对应的代码已被删除，标记为无效");
+      consoleSpy.mockRestore();
+    });
+
+    it("should log when line range is collapsed to single line", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const filePatchMap = new Map([["test.ts", "@@ -1,2 +1,1 @@\n-old1\n-old2\n+new1"]]);
+      const issues = [{ file: "test.ts", line: "1-2", ruleId: "R1" }];
+      (service as any).updateIssueLineNumbers(issues, filePatchMap, 1);
+      expect(consoleSpy).toHaveBeenCalledWith("📍 Issue 行号更新: test.ts:1-2 -> test.ts:1");
+      consoleSpy.mockRestore();
+    });
+  });
+
+  describe("ReviewService.findExistingAiComments", () => {
+    it("should log comments when verbose level >= 2", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const mockComments = [
+        { id: 1, body: "test comment 1" },
+        { id: 2, body: "test comment 2<!-- spaceflow-review -->" },
+      ] as any;
+      gitProvider.listIssueComments.mockResolvedValue(mockComments);
+
+      await (service as any).findExistingAiComments("o", "r", 1, 2);
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "[findExistingAiComments] listIssueComments returned 2 comments",
+      );
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "[findExistingAiComments] comment id=1, body starts with: test comment 1",
+      );
+      consoleSpy.mockRestore();
+    });
+
+    it("should log error when API fails", async () => {
+      const consoleSpy = vi.spyOn(console, "warn").mockImplementation(() => {});
+      gitProvider.listIssueComments.mockRejectedValue(new Error("API error"));
+
+      const result = await (service as any).findExistingAiComments("o", "r", 1);
+      expect(result).toEqual([]);
+      expect(consoleSpy).toHaveBeenCalledWith("[findExistingAiComments] error:", expect.any(Error));
+      consoleSpy.mockRestore();
+    });
+  });
+
+  describe("ReviewService.syncReactionsToIssues", () => {
+    it("should log when no AI review found", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      gitProvider.listPullReviews.mockResolvedValue([] as any);
+
+      await (service as any).syncReactionsToIssues("o", "r", 1, { issues: [] }, 2);
+      expect(consoleSpy).toHaveBeenCalledWith("[syncReactionsToIssues] No AI review found");
+      consoleSpy.mockRestore();
+    });
+
+    it("should log reviewers from reviews", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const mockReviews = [
+        { user: { login: "user1" }, body: "normal review" },
+        { user: { login: "bot" }, body: "<!-- spaceflow-review-lines --> AI review", id: 123 },
+      ] as any;
+      gitProvider.listPullReviews.mockResolvedValue(mockReviews);
+      gitProvider.listPullReviewComments.mockResolvedValue([] as any);
+
+      await (service as any).syncReactionsToIssues("o", "r", 1, { issues: [] }, 2);
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "[syncReactionsToIssues] reviewers from reviews: user1",
+      );
+      consoleSpy.mockRestore();
+    });
+
+    it("should log requested reviewers and teams", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const mockReviews = [
+        { user: { login: "bot" }, body: "<!-- spaceflow-review-lines --> AI review", id: 123 },
+      ] as any;
+      const mockPr = {
+        requested_reviewers: [{ login: "reviewer1" }],
+        requested_reviewers_teams: [{ name: "team1", id: 123 }],
+      } as any;
+      gitProvider.listPullReviews.mockResolvedValue(mockReviews);
+      gitProvider.getPullRequest.mockResolvedValue(mockPr);
+      gitProvider.getTeamMembers.mockResolvedValue([{ login: "teamuser1" }]);
+      gitProvider.listPullReviewComments.mockResolvedValue([] as any);
+
+      await (service as any).syncReactionsToIssues("o", "r", 1, { issues: [] }, 2);
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "[syncReactionsToIssues] requested_reviewers: reviewer1",
+      );
+      expect(consoleSpy).toHaveBeenCalledWith(
+        '[syncReactionsToIssues] requested_reviewers_teams: [{"name":"team1","id":123}]',
+      );
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "[syncReactionsToIssues] team team1(123) members: teamuser1",
+      );
+      consoleSpy.mockRestore();
+    });
+
+    it("should log final reviewers", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const mockReviews = [
+        { user: { login: "bot" }, body: "<!-- spaceflow-review-lines --> AI review", id: 123 },
+      ] as any;
+      gitProvider.listPullReviews.mockResolvedValue(mockReviews);
+      gitProvider.getPullRequest.mockRejectedValue(new Error("PR not found"));
+      gitProvider.listPullReviewComments.mockResolvedValue([] as any);
+
+      await (service as any).syncReactionsToIssues("o", "r", 1, { issues: [] }, 2);
+      expect(consoleSpy).toHaveBeenCalledWith("[syncReactionsToIssues] final reviewers: ");
+      consoleSpy.mockRestore();
+    });
+  });
+
+  describe("ReviewService.deleteExistingAiReviews", () => {
+    beforeEach(() => {
+      mockReviewSpecService.parseLineRange = vi.fn().mockImplementation((lineStr: string) => {
+        const lines: number[] = [];
+        const rangeMatch = lineStr.match(/^(\d+)-(\d+)$/);
+        if (rangeMatch) {
+          const start = parseInt(rangeMatch[1], 10);
+          const end = parseInt(rangeMatch[2], 10);
+          for (let i = start; i <= end; i++) {
+            lines.push(i);
+          }
+        } else {
+          const line = parseInt(lineStr, 10);
+          if (!isNaN(line)) {
+            lines.push(line);
+          }
+        }
+        return lines;
+      });
+    });
+
+    it("should filter issues by valid commit hashes", () => {
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([
+        [
+          "test.ts",
+          [
+            ["-------", "line1"],
+            ["abc1234", "line2"],
+            ["-------", "line3"],
+          ],
+        ],
+      ]);
+      const issues = [
+        { file: "test.ts", line: "2", ruleId: "R1" }, // 应该保留，hash匹配
+        { file: "test.ts", line: "1", ruleId: "R2" }, // 应该过滤，hash不匹配
+        { file: "test.ts", line: "3", ruleId: "R3" }, // 应该过滤，hash不匹配
+      ];
+      const result = (service as any).filterIssuesByValidCommits(issues, commits, fileContents, 2);
+      expect(result).toHaveLength(1);
+      expect(result[0].ruleId).toBe("R1");
+    });
+
+    it("should log filtering summary", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([
+        [
+          "test.ts",
+          [
+            ["-------", "line1"],
+            ["abc1234", "line2"],
+          ],
+        ],
+      ]);
+      const issues = [
+        { file: "test.ts", line: "1", ruleId: "R1" },
+        { file: "test.ts", line: "2", ruleId: "R2" },
+      ];
+      (service as any).filterIssuesByValidCommits(issues, commits, fileContents, 1);
+      expect(consoleSpy).toHaveBeenCalledWith("   过滤非本次 PR commits 问题后: 2 -> 1 个问题");
+      consoleSpy.mockRestore();
+    });
+
+    it("should keep issues when file not in fileContents", () => {
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map();
+      const issues = [{ file: "missing.ts", line: "1", ruleId: "R1" }];
+      const result = (service as any).filterIssuesByValidCommits(issues, commits, fileContents);
+      expect(result).toEqual(issues);
+    });
+
+    it("should keep issues when line range cannot be parsed", () => {
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([["test.ts", [["-------", "line1"]]]]);
+      const issues = [{ file: "test.ts", line: "abc", ruleId: "R1" }];
+      const result = (service as any).filterIssuesByValidCommits(issues, commits, fileContents);
+      expect(result).toEqual(issues);
+    });
+
+    it("should handle range line numbers", () => {
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([
+        [
+          "test.ts",
+          [
+            ["-------", "line1"],
+            ["abc1234", "line2"],
+            ["-------", "line3"],
+          ],
+        ],
+      ]);
+      const issues = [{ file: "test.ts", line: "1-3", ruleId: "R1" }];
+      const result = (service as any).filterIssuesByValidCommits(issues, commits, fileContents);
+      expect(result).toHaveLength(1); // 只要范围内有一行匹配就保留
+    });
+
+    it("should log when file not in fileContents at verbose level 3", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map();
+      const issues = [{ file: "missing.ts", line: "1", ruleId: "R1" }];
+      (service as any).filterIssuesByValidCommits(issues, commits, fileContents, 3);
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "   ✅ Issue missing.ts:1 - 文件不在 fileContents 中，保留",
+      );
+      consoleSpy.mockRestore();
+    });
+
+    it("should log when line range cannot be parsed at verbose level 3", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([["test.ts", [["-------", "line1"]]]]);
+      const issues = [{ file: "test.ts", line: "abc", ruleId: "R1" }];
+      (service as any).filterIssuesByValidCommits(issues, commits, fileContents, 3);
+      expect(consoleSpy).toHaveBeenCalledWith("   ✅ Issue test.ts:abc - 无法解析行号，保留");
+      consoleSpy.mockRestore();
+    });
+
+    it("should log detailed hash matching at verbose level 3", () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      const commits = [{ sha: "abc1234567890" }];
+      const fileContents = new Map([
+        [
+          "test.ts",
+          [
+            ["-------", "line1"],
+            ["abc1234", "line2"],
+          ],
+        ],
+      ]);
+      const issues = [{ file: "test.ts", line: "2", ruleId: "R1" }];
+      (service as any).filterIssuesByValidCommits(issues, commits, fileContents, 3);
+      expect(consoleSpy).toHaveBeenCalledWith("   🔍 有效 commit hashes: abc1234");
+      expect(consoleSpy).toHaveBeenCalledWith(
+        "   ✅ Issue test.ts:2 - 行 2 hash=abc1234 匹配，保留",
+      );
+      consoleSpy.mockRestore();
+    });
+  });
+
+  describe("ReviewService.ensureClaudeCli", () => {
+    it("should do nothing when claude is already installed", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      // execSync is already mocked globally
+
+      await (service as any).ensureClaudeCli();
+      expect(consoleSpy).not.toHaveBeenCalledWith("🔧 Claude CLI 未安装，正在安装...");
+      consoleSpy.mockRestore();
+    });
+
+    it("should install claude when not found", async () => {
+      const consoleSpy = vi.spyOn(console, "log").mockImplementation(() => {});
+      // Mock execSync to throw then succeed
+      const execSyncMock = vi.mocked(await import("child_process"));
+      execSyncMock.execSync
+        .mockImplementationOnce(() => {
+          throw new Error("command not found");
+        })
+        .mockImplementationOnce(() => Buffer.from(""));
+
+      await (service as any).ensureClaudeCli();
+      expect(consoleSpy).toHaveBeenCalledWith("🔧 Claude CLI 未安装，正在安装...");
+      expect(consoleSpy).toHaveBeenCalledWith("✅ Claude CLI 安装完成");
+      consoleSpy.mockRestore();
+    });
+
+    it("should throw error when installation fails", async () => {
+      const execSyncMock = vi.mocked(await import("child_process"));
+      execSyncMock.execSync
+        .mockImplementationOnce(() => {
+          throw new Error("command not found");
+        })
+        .mockImplementationOnce(() => {
+          throw new Error("install failed");
+        });
+
+      await expect((service as any).ensureClaudeCli()).rejects.toThrow(
+        "Claude CLI 安装失败: install failed",
+      );
+    });
+  });
 });