Skip to content

Commit 579a217

Browse files
committed
test(transcript-rewrite): clarify misleading test name
Greptile review on #66443: test name said "preserves messages with same content but distinct entry-identity inputs" but the three messages all have *different* content ("first"/"second"/"third"). The internal comment already describes the actual invariant ("Distinct content with the same role must NOT collapse"); rename the it() to match.
1 parent a2fd188 commit 579a217

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/agents/pi-embedded-runner/transcript-rewrite.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -362,7 +362,7 @@ describe("rewriteTranscriptEntriesInSessionManager — #66443 dedupe invariant",
362362
expect(branchMessages[0]).toMatchObject({ role: "user", content: heartbeatPrompt });
363363
});
364364

365-
it("preserves messages with same content but distinct entry-identity inputs", () => {
365+
it("preserves distinct user messages with the same role", () => {
366366
// Two user messages with same content but different timestamps and metadata
367367
// are separate logical entries; we dedupe on (role + content), so they
368368
// collapse. This documents the chosen invariant: identity = role + content.

0 commit comments

Comments
 (0)