Skip to content

Commit 0ba4d14

Browse files
committed
test: update condense.spec.ts to expect preserved first message
- Updated test expectations to match new behavior where original first message is preserved - Tests now verify that the first user message content is maintained, not replaced with generic text
1 parent 76cf136 commit 0ba4d14

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/core/condense/__tests__/condense.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -229,9 +229,9 @@ describe("Condense", () => {
229229

230230
const result = getMessagesSinceLastSummary(messages)
231231

232-
// Should include a user message prefix for Bedrock compatibility, the summary, and messages after
232+
// Should include the original first user message for context preservation, the summary, and messages after
233233
expect(result[0].role).toBe("user")
234-
expect(result[0].content).toBe("Please continue from the following summary:")
234+
expect(result[0].content).toBe("First message") // Preserves original first message
235235
expect(result[1]).toEqual(messages[2]) // The summary
236236
expect(result[2]).toEqual(messages[3])
237237
expect(result[3]).toEqual(messages[4])
@@ -249,9 +249,9 @@ describe("Condense", () => {
249249

250250
const result = getMessagesSinceLastSummary(messages)
251251

252-
// Should only include from the last summary
252+
// Should only include from the last summary with original first message preserved
253253
expect(result[0].role).toBe("user")
254-
expect(result[0].content).toBe("Please continue from the following summary:")
254+
expect(result[0].content).toBe("First message") // Preserves original first message
255255
expect(result[1]).toEqual(messages[3]) // Second summary
256256
expect(result[2]).toEqual(messages[4])
257257
expect(result[3]).toEqual(messages[5])

0 commit comments

Comments
 (0)