From 2582669d6290a1a5148b5a7d3df540ae752cd3f7 Mon Sep 17 00:00:00 2001 From: Adam Weidman Date: Mon, 23 Mar 2026 10:58:41 -0400 Subject: [PATCH] fix(core): align legacy session tests with loop warnings --- packages/core/src/agent/legacy-agent-session.test.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/packages/core/src/agent/legacy-agent-session.test.ts b/packages/core/src/agent/legacy-agent-session.test.ts index 4ce59edd63..e008e33d94 100644 --- a/packages/core/src/agent/legacy-agent-session.test.ts +++ b/packages/core/src/agent/legacy-agent-session.test.ts @@ -673,7 +673,7 @@ describe('LegacyAgentSession', () => { expect(events.some((e) => e.type === 'agent_end')).toBe(true); }); - it('handles LoopDetected as non-terminal custom event', async () => { + it('handles LoopDetected as non-terminal warning event', async () => { const sendMock = deps.client.sendMessageStream as ReturnType< typeof vi.fn >; @@ -693,12 +693,12 @@ describe('LegacyAgentSession', () => { await session.send({ message: [{ type: 'text', text: 'hi' }] }); const events = await collectEvents(session); - // Should have a custom loop_detected event - const custom = events.find( - (e): e is AgentEvent<'custom'> => - e.type === 'custom' && e.kind === 'loop_detected', + const warning = events.find( + (e): e is AgentEvent<'error'> => + e.type === 'error' && e._meta?.['code'] === 'LOOP_DETECTED', ); - expect(custom).toBeDefined(); + expect(warning).toBeDefined(); + expect(warning?.fatal).toBe(false); // Stream should have continued — content after loop detected const messages = events.filter(