fix(ci): restore main's typecheck and lint after #25751 and #24369 (#25949)

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.
This commit is contained in:
Jinwoo Hong
2026-10-06 15:24:03 -04:00
committed by GitHub
parent b922342199
commit 4c35516150
2 changed files with 7 additions and 4 deletions
@@ -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<void> =>
this.runtimeState.flushEventSink(sessionId)
flushStreamedEvents = (id: string): Promise<void> => 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.
@@ -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', () => {