From ea9a0daa82bd152a1e9fb2cddf642214e2571fd8 Mon Sep 17 00:00:00 2001 From: Jinwoo-H Date: Tue, 6 Oct 2026 15:23:53 -0400 Subject: [PATCH] fix(ci): restore main's typecheck and lint after #25751 and #24369 A test still called isStructuredAgentSessionThinking with a bare array after #25751 changed it to take { items }, and #24369 left the session host one line over its max-lines budget. Both fail CI on every PR. --- .../agent-session-wire/structured-agent-session-host.ts | 3 +-- src/shared/structured-agent-session-live-turn.test.ts | 8 ++++++-- 2 files changed, 7 insertions(+), 4 deletions(-) diff --git a/src/main/native-chat/agent-session-wire/structured-agent-session-host.ts b/src/main/native-chat/agent-session-wire/structured-agent-session-host.ts index d8daaa659538..e2f59123e863 100644 --- a/src/main/native-chat/agent-session-wire/structured-agent-session-host.ts +++ b/src/main/native-chat/agent-session-wire/structured-agent-session-host.ts @@ -260,8 +260,7 @@ export class StructuredAgentSessionHost { } /** Test barrier: every write has landed by its call's return, so no production path needs it. */ - flushStreamedEvents = (sessionId: string): Promise => - this.runtimeState.flushEventSink(sessionId) + flushStreamedEvents = (id: string): Promise => this.runtimeState.flushEventSink(id) // Trigger inlined rather than imported: `AgentSessionResumeTrigger` in shared is the canonical // type, and this file has no line budget left for the import. diff --git a/src/shared/structured-agent-session-live-turn.test.ts b/src/shared/structured-agent-session-live-turn.test.ts index 5a4d1f7d43c8..389e6b2aff38 100644 --- a/src/shared/structured-agent-session-live-turn.test.ts +++ b/src/shared/structured-agent-session-live-turn.test.ts @@ -39,9 +39,13 @@ describe('isStructuredAgentSessionThinking', () => { state, ...(state === 'completed' ? { completedAt: 3 } : {}) }) - expect(isStructuredAgentSessionThinking([turnStart, withState('running')])).toBe(true) + expect(isStructuredAgentSessionThinking({ items: [turnStart, withState('running')] })).toBe( + true + ) // Ended reasoning stays the newest row while Claude streams a tool's input after it. - expect(isStructuredAgentSessionThinking([turnStart, withState('completed')])).toBe(false) + expect(isStructuredAgentSessionThinking({ items: [turnStart, withState('completed')] })).toBe( + false + ) }) it('is false once a tool call, a message or a diff lands after the reasoning', () => {