From 0897ddb33abf16f9fb333343320ddc3805403797 Mon Sep 17 00:00:00 2001 From: murashit Date: Tue, 19 May 2026 16:16:18 +0900 Subject: [PATCH] Allow duplicate auto-review summaries --- src/display/blocks.ts | 2 +- styles.css | 4 ---- tests/display/display-model.test.ts | 12 +++++++++++- tests/ui/view-renderers.test.ts | 4 ++-- 4 files changed, 14 insertions(+), 8 deletions(-) diff --git a/src/display/blocks.ts b/src/display/blocks.ts index e2a55de7..b6bdbc75 100644 --- a/src/display/blocks.ts +++ b/src/display/blocks.ts @@ -128,7 +128,7 @@ function autoReviewSummariesForTurns(items: DisplayItem[]): Map block.type === "item" && block.item.role === "assistant"); - expect(assistantBlock).toMatchObject({ item: { autoReviewSummaries: ["Auto-review approved: npm test"] } }); + expect(assistantBlock).toMatchObject({ + item: { autoReviewSummaries: ["Auto-review approved: npm test", "Auto-review approved: npm test"] }, + }); }); }); diff --git a/tests/ui/view-renderers.test.ts b/tests/ui/view-renderers.test.ts index 225779fe..dfcfcfba 100644 --- a/tests/ui/view-renderers.test.ts +++ b/tests/ui/view-renderers.test.ts @@ -1636,7 +1636,7 @@ describe("pending request renderer decisions", () => { text: "Done", turnId: "turn", markdown: true, - autoReviewSummaries: ["Auto-review approved: npm test"], + autoReviewSummaries: ["Auto-review approved: npm test", "Auto-review approved: npm test"], }, ], openDetails: new Set(), @@ -1647,7 +1647,7 @@ describe("pending request renderer decisions", () => { const element = block.render(); - expect(element.querySelector(".codex-panel__auto-reviews summary")?.textContent).toBe("Auto-reviewed 1 request"); + expect(element.querySelector(".codex-panel__auto-reviews summary")?.textContent).toBe("Auto-reviewed 2 requests"); expect(element.querySelector(".codex-panel__auto-reviews")?.textContent).toContain("Auto-review approved: npm test"); });