@spaceflow/review 0.81.0 → 0.83.0

package/src/review-issue-filter.ts

@@ -5,20 +5,9 @@ import {
   type VerboseLevel,
   shouldLog,
   GitSdkService,
-  parseChangedLinesFromPatch,
-  parseDiffText,
-  parseHunksFromPatch,
-  calculateNewLineNumber,
 } from "@spaceflow/core";
 import type { IConfigReader } from "@spaceflow/core";
-import { PullRequestModel } from "./pull-request-model";
-import {
-  ReviewSpecService,
-  ReviewSpec,
-  ReviewIssue,
-  FileContentsMap,
-  FileContentLine,
-} from "./review-spec";
+import { ReviewSpecService, ReviewSpec, ReviewIssue, FileContentsMap } from "./review-spec";
 import { IssueVerifyService } from "./issue-verify.service";
 import { generateIssueKey } from "./utils/review-pr-comment";
 import type { ReviewContext } from "./review-context";
@@ -65,16 +54,14 @@ export class ReviewIssueFilter {
 
   /**
    * LLM 验证历史问题是否已修复
-   * 如果传入 preloaded（specs/fileContents），直接使用；否则从 PR 获取
    */
   async verifyAndUpdateIssues(
     context: ReviewContext,
     issues: ReviewIssue[],
     commits: PullRequestCommit[],
-    preloaded?: { specs: ReviewSpec[]; fileContents: FileContentsMap },
-    pr?: PullRequestModel,
+    preloaded: { specs: ReviewSpec[]; fileContents: FileContentsMap },
   ): Promise<ReviewIssue[]> {
-    const { llmMode, specSources, verbose } = context;
+    const { llmMode, verbose } = context;
     const unfixedIssues = issues.filter((i) => i.valid !== "false" && !i.fixed);
 
     if (unfixedIssues.length === 0) {
@@ -88,37 +75,11 @@ export class ReviewIssueFilter {
       return issues;
     }
 
-    if (!preloaded && (!specSources?.length || !pr)) {
-      if (shouldLog(verbose, 1)) {
-        console.log(`   ⏭️  跳过 LLM 验证（缺少 specSources 或 pr）`);
-      }
-      return issues;
-    }
-
     if (shouldLog(verbose, 1)) {
       console.log(`\n🔍 开始 LLM 验证 ${unfixedIssues.length} 个未修复问题...`);
     }
 
-    let specs: ReviewSpec[];
-    let fileContents: FileContentsMap;
-
-    if (preloaded) {
-      specs = preloaded.specs;
-      fileContents = preloaded.fileContents;
-    } else {
-      const changedFiles = await pr!.getFiles();
-      const headSha = await pr!.getHeadSha();
-      specs = await this.loadSpecs(specSources, verbose);
-      fileContents = await this.getFileContents(
-        pr!.owner,
-        pr!.repo,
-        changedFiles,
-        commits,
-        headSha,
-        pr!.number,
-        verbose,
-      );
-    }
+    const { specs, fileContents } = preloaded;
 
     return await this.issueVerifyService.verifyIssueFixes(
       issues,
@@ -175,105 +136,6 @@ export class ReviewIssueFilter {
     }
   }
 
-  /**
-   * 获取文件内容并构建行号到 commit hash 的映射
-   * 返回 Map<filename, Array<[commitHash, lineCode]>>
-   */
-  async getFileContents(
-    owner: string,
-    repo: string,
-    changedFiles: ChangedFile[],
-    commits: PullRequestCommit[],
-    ref: string,
-    prNumber?: number,
-    verbose?: VerboseLevel,
-    isLocalMode?: boolean,
-  ): Promise<FileContentsMap> {
-    const contents: FileContentsMap = new Map();
-    const latestCommitHash = commits[commits.length - 1]?.sha?.slice(0, 7) || "+local+";
-
-    // 优先使用 changedFiles 中的 patch 字段（来自 PR 的整体 diff base...head）
-    // 这样行号是相对于最终文件的，而不是每个 commit 的父 commit
-    // buildLineCommitMap 遍历每个 commit 的 diff，行号可能与最终文件不一致
-    if (shouldLog(verbose, 1)) {
-      console.log(`📊 正在构建行号到变更的映射...`);
-    }
-
-    for (const file of changedFiles) {
-      if (file.filename && file.status !== "deleted") {
-        try {
-          let rawContent: string;
-          if (isLocalMode) {
-            // 本地模式：读取工作区文件的当前内容
-            rawContent = this.gitSdk.getWorkingFileContent(file.filename);
-          } else if (prNumber) {
-            rawContent = await this.gitProvider.getFileContent(owner, repo, file.filename, ref);
-          } else {
-            rawContent = await this.gitSdk.getFileContent(ref, file.filename);
-          }
-          const lines = rawContent.split("\n");
-
-          // 优先使用 file.patch（PR 整体 diff），这是相对于最终文件的行号
-          let changedLines = parseChangedLinesFromPatch(file.patch);
-
-          // 如果 changedLines 为空，需要判断是否应该将所有行标记为变更
-          // 情况1: 文件是新增的（status 为 added/A）
-          // 情况2: patch 为空但文件有 additions（部分 Git Provider API 可能不返回完整 patch）
-          const isNewFile =
-            file.status === "added" ||
-            file.status === "A" ||
-            (file.additions && file.additions > 0 && file.deletions === 0 && !file.patch);
-          if (changedLines.size === 0 && isNewFile) {
-            changedLines = new Set(lines.map((_, i) => i + 1));
-            if (shouldLog(verbose, 2)) {
-              console.log(
-                `   ℹ️ ${file.filename}: 新增文件无 patch，将所有 ${lines.length} 行标记为变更`,
-              );
-            }
-          }
-
-          if (shouldLog(verbose, 3)) {
-            console.log(`   📄 ${file.filename}: ${lines.length} 行, ${changedLines.size} 行变更`);
-            console.log(`      latestCommitHash: ${latestCommitHash}`);
-            if (changedLines.size > 0 && changedLines.size <= 20) {
-              console.log(
-                `      变更行号: ${Array.from(changedLines)
-                  .sort((a, b) => a - b)
-                  .join(", ")}`,
-              );
-            } else if (changedLines.size > 20) {
-              console.log(`      变更行号: (共 ${changedLines.size} 行，省略详情)`);
-            }
-            if (!file.patch) {
-              console.log(
-                `      ⚠️ 该文件没有 patch 信息 (status=${file.status}, additions=${file.additions}, deletions=${file.deletions})`,
-              );
-            } else {
-              console.log(
-                `      patch 前 200 字符: ${file.patch.slice(0, 200).replace(/\n/g, "\\n")}`,
-              );
-            }
-          }
-
-          const contentLines: FileContentLine[] = lines.map((line, index) => {
-            const lineNum = index + 1;
-            // 如果该行在 PR 的整体 diff 中被标记为变更，则使用最新 commit hash
-            const hash = changedLines.has(lineNum) ? latestCommitHash : "-------";
-            return [hash, line];
-          });
-          contents.set(file.filename, contentLines);
-        } catch (error) {
-          console.warn(`警告: 无法获取文件内容: ${file.filename}`, error);
-        }
-      }
-    }
-
-    if (shouldLog(verbose, 1)) {
-      console.log(`📊 映射构建完成，共 ${contents.size} 个文件`);
-    }
-    return contents;
-  }
-
   async fillIssueCode(
     issues: ReviewIssue[],
     fileContents: FileContentsMap,
@@ -300,86 +162,6 @@ export class ReviewIssueFilter {
     });
   }
 
-  /**
-   * 根据代码变更更新历史 issue 的行号
-   * 当代码发生变化时，之前发现的 issue 行号可能已经不准确
-   * 此方法通过分析 diff 来计算新的行号
-   */
-  updateIssueLineNumbers(
-    issues: ReviewIssue[],
-    filePatchMap: Map<string, string>,
-    verbose?: VerboseLevel,
-  ): ReviewIssue[] {
-    let updatedCount = 0;
-    let invalidatedCount = 0;
-    const updatedIssues = issues.map((issue) => {
-      // 如果 issue 已修复、已解决或无效，不需要更新行号
-      if (issue.fixed || issue.resolved || issue.valid === "false") {
-        return issue;
-      }
-
-      const patch = filePatchMap.get(issue.file);
-      if (!patch) {
-        // 文件没有变更，行号不变
-        return issue;
-      }
-
-      const lines = this.reviewSpecService.parseLineRange(issue.line);
-      if (lines.length === 0) {
-        return issue;
-      }
-
-      const startLine = lines[0];
-      const endLine = lines[lines.length - 1];
-      const hunks = parseHunksFromPatch(patch);
-
-      // 计算新的起始行号
-      const newStartLine = calculateNewLineNumber(startLine, hunks);
-      if (newStartLine === null) {
-        // 起始行被删除，直接标记为无效问题
-        invalidatedCount++;
-        if (shouldLog(verbose, 1)) {
-          console.log(`📍 Issue ${issue.file}:${issue.line} 对应的代码已被删除，标记为无效`);
-        }
-        return { ...issue, valid: "false", originalLine: issue.originalLine ?? issue.line };
-      }
-
-      // 如果是范围行号，计算新的结束行号
-      let newLine: string;
-      if (startLine === endLine) {
-        newLine = String(newStartLine);
-      } else {
-        const newEndLine = calculateNewLineNumber(endLine, hunks);
-        if (newEndLine === null || newEndLine === newStartLine) {
-          // 结束行被删除或范围缩小为单行，使用起始行
-          newLine = String(newStartLine);
-        } else {
-          newLine = `${newStartLine}-${newEndLine}`;
-        }
-      }
-
-      // 如果行号发生变化，更新 issue
-      if (newLine !== issue.line) {
-        updatedCount++;
-        if (shouldLog(verbose, 1)) {
-          console.log(`📍 Issue 行号更新: ${issue.file}:${issue.line} -> ${issue.file}:${newLine}`);
-        }
-        return { ...issue, line: newLine, originalLine: issue.originalLine ?? issue.line };
-      }
-
-      return issue;
-    });
-
-    if ((updatedCount > 0 || invalidatedCount > 0) && shouldLog(verbose, 1)) {
-      const parts: string[] = [];
-      if (updatedCount > 0) parts.push(`更新 ${updatedCount} 个行号`);
-      if (invalidatedCount > 0) parts.push(`标记 ${invalidatedCount} 个无效`);
-      console.log(`📊 Issue 行号处理: ${parts.join("，")}`);
-    }
-
-    return updatedIssues;
-  }
-
   /**
    * 过滤掉不属于本次 PR commits 的问题（排除 merge commit 引入的代码）
    * 根据 fileContents 中问题行的实际 commit hash 进行验证，而不是依赖 LLM 填写的 commit
@@ -391,9 +173,15 @@ export class ReviewIssueFilter {
     verbose?: VerboseLevel,
   ): ReviewIssue[] {
     const validCommitHashes = new Set(commits.map((c) => c.sha?.slice(0, 7)).filter(Boolean));
+    // commits 为空时（如分支比较模式本地无 commit 信息），退化为"行是否在 diff 变更范围内"模式
+    const useChangedLinesMode = validCommitHashes.size === 0;
 
     if (shouldLog(verbose, 3)) {
-      console.log(`   🔍 有效 commit hashes: ${Array.from(validCommitHashes).join(", ")}`);
+      if (useChangedLinesMode) {
+        console.log(`   🔍 commits 为空，使用变更行模式过滤`);
+      } else {
+        console.log(`   🔍 有效 commit hashes: ${Array.from(validCommitHashes).join(", ")}`);
+      }
     }
 
     const beforeCount = issues.length;
@@ -415,12 +203,16 @@ export class ReviewIssueFilter {
         return true;
       }
 
-      // 检查问题行范围内是否有任意一行属于本次 PR 的有效 commits
+      // 检查问题行范围内是否有任意一行属于本次变更（diff 范围）
       for (const lineNum of lineNums) {
         const lineData = contentLines[lineNum - 1];
         if (lineData) {
           const [actualHash] = lineData;
-          if (actualHash !== "-------" && validCommitHashes.has(actualHash)) {
+          const isChangedLine = actualHash !== "-------";
+          const isValid = useChangedLinesMode
+            ? isChangedLine
+            : isChangedLine && validCommitHashes.has(actualHash);
+          if (isValid) {
             if (shouldLog(verbose, 3)) {
               console.log(
                 `   ✅ Issue ${issue.file}:${issue.line} - 行 ${lineNum} hash=${actualHash} 匹配，保留`,
@@ -431,9 +223,9 @@ export class ReviewIssueFilter {
         }
       }
 
-      // 问题行都不属于本次 PR 的有效 commits
+      // 问题行都不属于本次变更范围
       if (shouldLog(verbose, 2)) {
-        console.log(`   Issue ${issue.file}:${issue.line} 不在本次 PR 变更行范围内，跳过`);
+        console.log(`   Issue ${issue.file}:${issue.line} 不在本次变更行范围内，跳过`);
       }
       if (shouldLog(verbose, 3)) {
         const hashes = lineNums.map((ln) => {
@@ -445,7 +237,7 @@ export class ReviewIssueFilter {
       return false;
     });
     if (beforeCount !== filtered.length && shouldLog(verbose, 1)) {
-      console.log(`   过滤非本次 PR commits 问题后: ${beforeCount} -> ${filtered.length} 个问题`);
+      console.log(`   变更行过滤后: ${beforeCount} -> ${filtered.length} 个问题`);
     }
     return filtered;
   }
@@ -469,55 +261,4 @@ export class ReviewIssueFilter {
   generateIssueKey(issue: ReviewIssue): string {
     return generateIssueKey(issue);
   }
-
-  /**
-   * 构建文件行号到 commit hash 的映射
-   * 遍历每个 commit，获取其修改的文件和行号
-   * 优先使用 API，失败时回退到 git 命令
-   */
-  async buildLineCommitMap(
-    owner: string,
-    repo: string,
-    commits: PullRequestCommit[],
-    verbose?: VerboseLevel,
-  ): Promise<Map<string, Map<number, string>>> {
-    // Map<filename, Map<lineNumber, commitHash>>
-    const fileLineMap = new Map<string, Map<number, string>>();
-
-    // 按时间顺序遍历 commits（早的在前），后面的 commit 会覆盖前面的
-    for (const commit of commits) {
-      if (!commit.sha) continue;
-
-      const shortHash = commit.sha.slice(0, 7);
-      let files: Array<{ filename: string; patch: string }> = [];
-
-      // 优先使用 getCommitDiff API 获取 diff 文本
-      try {
-        const diffText = await this.gitProvider.getCommitDiff(owner, repo, commit.sha);
-        files = parseDiffText(diffText);
-      } catch {
-        // API 失败，回退到 git 命令
-        files = this.gitSdk.getCommitDiff(commit.sha);
-      }
-      if (shouldLog(verbose, 2)) console.log(`   commit ${shortHash}: ${files.length} 个文件变更`);
-
-      for (const file of files) {
-        // 解析这个 commit 修改的行号
-        const changedLines = parseChangedLinesFromPatch(file.patch);
-
-        // 获取或创建文件的行号映射
-        if (!fileLineMap.has(file.filename)) {
-          fileLineMap.set(file.filename, new Map());
-        }
-        const lineMap = fileLineMap.get(file.filename)!;
-
-        // 记录每行对应的 commit hash
-        for (const lineNum of changedLines) {
-          lineMap.set(lineNum, shortHash);
-        }
-      }
-    }
-
-    return fileLineMap;
-  }
 }

package/src/review-llm.spec.ts

@@ -0,0 +1,287 @@
+import { vi } from "vitest";
+import { ReviewLlmProcessor } from "./review-llm";
+import type { ReviewPrompt } from "./review.service";
+import { ChangedFileCollection } from "./changed-file-collection";
+
+vi.mock("c12");
+vi.mock("@anthropic-ai/claude-agent-sdk", () => ({
+  query: vi.fn(),
+}));
+vi.mock("@opencode-ai/sdk", () => ({
+  createOpencodeClient: vi.fn().mockReturnValue({
+    session: {
+      create: vi.fn(),
+      prompt: vi.fn(),
+      delete: vi.fn(),
+    },
+  }),
+}));
+vi.mock("openai", () => {
+  const mCreate = vi.fn();
+  class MockOpenAI {
+    chat = {
+      completions: {
+        create: mCreate,
+      },
+    };
+    static APIError = class extends Error {
+      status: number;
+      constructor(status: number, message: string) {
+        super(message);
+        this.status = status;
+      }
+    };
+  }
+  return {
+    __esModule: true,
+    default: MockOpenAI,
+  };
+});
+
+describe("ReviewLlmProcessor", () => {
+  let processor: ReviewLlmProcessor;
+  let mockLlmProxyService: any;
+  let mockReviewSpecService: any;
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+
+    mockReviewSpecService = {
+      resolveSpecSources: vi.fn().mockResolvedValue(["/mock/spec/dir"]),
+      loadReviewSpecs: vi.fn().mockResolvedValue([
+        {
+          filename: "ts.base.md",
+          extensions: ["ts"],
+          rules: [{ id: "R1", title: "Rule 1", description: "D1", examples: [], overrides: [] }],
+          overrides: [],
+          includes: [],
+        },
+      ]),
+      applyOverrides: vi.fn().mockImplementation((specs) => specs),
+      filterApplicableSpecs: vi.fn().mockImplementation((specs) => specs),
+      filterIssuesByIncludes: vi.fn().mockImplementation((issues) => issues),
+      filterIssuesByOverrides: vi.fn().mockImplementation((issues) => issues),
+      filterIssuesByCommits: vi.fn().mockImplementation((issues) => issues),
+      formatIssues: vi.fn().mockImplementation((issues) => issues),
+      buildSpecsSection: vi.fn().mockReturnValue("mock specs section"),
+      filterIssuesByRuleExistence: vi.fn().mockImplementation((issues) => issues),
+      deduplicateSpecs: vi.fn().mockImplementation((specs) => specs),
+    };
+
+    mockLlmProxyService = {
+      chat: vi.fn(),
+      chatStream: vi.fn(),
+      createSession: vi.fn(),
+      getAvailableAdapters: vi.fn().mockReturnValue(["claude-code", "openai"]),
+    };
+
+    processor = new ReviewLlmProcessor(mockLlmProxyService as any, mockReviewSpecService as any);
+  });
+
+  afterEach(() => {
+    vi.clearAllMocks();
+  });
+
+  describe("runLLMReview", () => {
+    it("should call callLLM when llmMode is claude", async () => {
+      const callLLMSpy = vi
+        .spyOn(processor, "callLLM")
+        .mockResolvedValue({ issues: [], summary: [] } as any);
+
+      const mockPrompt: ReviewPrompt = {
+        filePrompts: [{ filename: "test.ts", systemPrompt: "system", userPrompt: "user" }],
+      };
+
+      await processor.runLLMReview("claude-code", mockPrompt);
+
+      expect(callLLMSpy).toHaveBeenCalledWith("claude-code", mockPrompt, {});
+    });
+
+    it("should call callLLM when llmMode is openai", async () => {
+      const callLLMSpy = vi
+        .spyOn(processor, "callLLM")
+        .mockResolvedValue({ issues: [], summary: [] } as any);
+
+      const mockPrompt: ReviewPrompt = {
+        filePrompts: [{ filename: "test.ts", systemPrompt: "system", userPrompt: "user" }],
+      };
+
+      await processor.runLLMReview("openai", mockPrompt);
+
+      expect(callLLMSpy).toHaveBeenCalledWith("openai", mockPrompt, {});
+    });
+  });
+
+  describe("normalizeIssues", () => {
+    it("should split comma separated lines", () => {
+      const issues = [
+        {
+          file: "test.ts",
+          line: "10, 12",
+          ruleId: "R1",
+          specFile: "s1.md",
+          reason: "r1",
+          suggestion: "fix",
+        } as any,
+      ];
+      const normalized = processor.normalizeIssues(issues);
+      expect(normalized).toHaveLength(2);
+      expect(normalized[0].line).toBe("10");
+      expect(normalized[1].line).toBe("12");
+      expect(normalized[1].suggestion).toContain("参考 test.ts:10");
+    });
+  });
+
+  describe("buildReviewPrompt", () => {
+    it("should build prompts for changed files", async () => {
+      const specs = [{ extensions: ["ts"], includes: [], rules: [{ id: "R1" }] }];
+      const changedFiles = [{ filename: "test.ts", status: "modified" }];
+      const fileContents = new Map([
+        [
+          "test.ts",
+          [
+            ["abc1234", "const x = 1;"],
+            ["-------", "const y = 2;"],
+          ],
+        ],
+      ]);
+      const commits = [{ sha: "abc1234567890", commit: { message: "fix" } }];
+      const result = await processor.buildReviewPrompt(
+        specs as any,
+        ChangedFileCollection.from(changedFiles),
+        fileContents as any,
+        commits,
+      );
+      expect(result.filePrompts).toHaveLength(1);
+      expect(result.filePrompts[0].filename).toBe("test.ts");
+      expect(result.filePrompts[0].userPrompt).toContain("test.ts");
+      expect(result.filePrompts[0].systemPrompt).toContain("代码审查专家");
+    });
+
+    it("should skip deleted files", async () => {
+      const specs = [{ extensions: ["ts"], includes: [], rules: [] }];
+      const changedFiles = [{ filename: "deleted.ts", status: "deleted" }];
+      const result = await processor.buildReviewPrompt(
+        specs as any,
+        ChangedFileCollection.from(changedFiles),
+        new Map(),
+        [],
+      );
+      expect(result.filePrompts).toHaveLength(0);
+    });
+
+    it("should handle missing file contents", async () => {
+      const specs = [{ extensions: ["ts"], includes: [], rules: [] }];
+      const changedFiles = [{ filename: "test.ts", status: "modified" }];
+      const result = await processor.buildReviewPrompt(
+        specs as any,
+        ChangedFileCollection.from(changedFiles),
+        new Map(),
+        [],
+      );
+      expect(result.filePrompts).toHaveLength(1);
+      expect(result.filePrompts[0].userPrompt).toContain("无法获取内容");
+    });
+
+    it("should include existing result in prompt", async () => {
+      const specs = [{ extensions: ["ts"], includes: [], rules: [] }];
+      const changedFiles = [{ filename: "test.ts", status: "modified" }];
+      const fileContents = new Map([["test.ts", [["-------", "code"]]]]);
+      const existingResult = {
+        issues: [{ file: "test.ts", line: "1", ruleId: "R1", reason: "bad code" }],
+        summary: [{ file: "test.ts", summary: "has issues" }],
+      };
+      const result = await processor.buildReviewPrompt(
+        specs as any,
+        ChangedFileCollection.from(changedFiles),
+        fileContents as any,
+        [],
+        existingResult as any,
+      );
+      expect(result.filePrompts[0].userPrompt).toContain("bad code");
+    });
+  });
+
+  describe("generatePrDescription", () => {
+    it("should generate description from LLM", async () => {
+      const mockStream = (async function* () {
+        yield { type: "text", content: "# Feat: 新功能\n\n详细描述" };
+      })();
+      mockLlmProxyService.chatStream.mockReturnValue(mockStream);
+      const commits = [{ sha: "abc123", commit: { message: "feat: add" } }];
+      const changedFiles = [{ filename: "a.ts", status: "modified" }];
+      const result = await processor.generatePrDescription(
+        commits,
+        ChangedFileCollection.from(changedFiles),
+        "openai",
+      );
+      expect(result.title).toBe("Feat: 新功能");
+      expect(result.description).toContain("详细描述");
+    });
+
+    it("should fallback on LLM error", async () => {
+      const mockStream = (async function* () {
+        yield { type: "error", message: "fail" };
+      })();
+      mockLlmProxyService.chatStream.mockReturnValue(mockStream);
+      const commits = [{ sha: "abc123", commit: { message: "feat: add" } }];
+      const changedFiles = [{ filename: "a.ts", status: "modified" }];
+      const result = await processor.generatePrDescription(
+        commits,
+        ChangedFileCollection.from(changedFiles),
+        "openai",
+      );
+      expect(result.title).toBeDefined();
+    });
+
+    it("should include code changes section when fileContents provided", async () => {
+      const mockStream = (async function* () {
+        yield { type: "text", content: "Feat: test\n\ndesc" };
+      })();
+      mockLlmProxyService.chatStream.mockReturnValue(mockStream);
+      const commits = [{ sha: "abc123", commit: { message: "feat" } }];
+      const changedFiles = [{ filename: "a.ts", status: "modified" }];
+      const fileContents = new Map([["a.ts", [["abc1234", "new code"]]]]) as any;
+      const result = await processor.generatePrDescription(
+        commits,
+        ChangedFileCollection.from(changedFiles),
+        "openai",
+        fileContents,
+      );
+      expect(result.title).toBeDefined();
+    });
+  });
+
+  describe("buildBasicDescription", () => {
+    it("should build description from commits and files", async () => {
+      const mockStream = (async function* () {
+        yield { type: "text", content: "Feat: test" };
+      })();
+      mockLlmProxyService.chatStream.mockReturnValue(mockStream);
+      const commits = [{ sha: "abc", commit: { message: "feat: add feature" } }];
+      const changedFiles = [
+        { filename: "a.ts", status: "added" },
+        { filename: "b.ts", status: "modified" },
+        { filename: "c.ts", status: "deleted" },
+      ];
+      const result = await processor.buildBasicDescription(
+        commits,
+        ChangedFileCollection.from(changedFiles),
+      );
+      expect(result.description).toContain("提交记录");
+      expect(result.description).toContain("文件变更");
+      expect(result.description).toContain("新增 1");
+      expect(result.description).toContain("修改 1");
+      expect(result.description).toContain("删除 1");
+    });
+
+    it("should handle empty commits", async () => {
+      const mockStream = (async function* () {
+        yield { type: "text", content: "Feat: empty" };
+      })();
+      mockLlmProxyService.chatStream.mockReturnValue(mockStream);
+      const result = await processor.buildBasicDescription([], ChangedFileCollection.empty());
+      expect(result.title).toBeDefined();
+    });
+  });
+});