mirror of
https://github.com/stablyai/orca.git
synced 2026-09-23 00:02:29 +00:00
* native-chat: render one indicator row for the live desktop turn The turn-timing row and the spinner+activity line were two rows saying "Working" at once. A settled turn keeps its own row; the live turn now has only the spinner row, labelled provider activity -> Thinking -> Working for N through the shared resolver. Reasoning is the turn's content, so it no longer becomes the activity label, and "Thinking" now means the turn is reasoning right now rather than that it has produced no output yet. * mobile: give the live turn row a spinner and the shared indicator label Mobile's per-turn row is already the only live indicator on the structured lane, but it pulsed a bare word and never showed what the provider said it was doing. It now renders a spinner beside the same resolved label desktop uses, and reads reasoning from the journal instead of inferring it from missing output. The bridge lane's four prompt/interrupt write seams move to one module so the controller stays under its line cap. * codex: mark streamed reasoning as reasoning too, and pin the provider markers The settled reasoning item carried the marker but the streaming one did not, so a live Codex turn - the only time the indicator is on screen - never read as reasoning. Both paths now stamp it; a plan document keeps its own presentation and must never read as reasoning. * fix(native-chat): tighten live turn reasoning state --------- Co-authored-by: Merge Sim <sim@local>
114 lines
4.1 KiB
TypeScript
114 lines
4.1 KiB
TypeScript
import { describe, expect, it } from 'vitest'
|
|
import type { AgentJournalRenderItem } from './agent-session-journal-types'
|
|
import { isStructuredAgentSessionThinking } from './structured-agent-session-live-turn'
|
|
|
|
function item(
|
|
itemId: string,
|
|
sequence: number,
|
|
body: AgentJournalRenderItem['body']
|
|
): AgentJournalRenderItem {
|
|
return { itemId, sequence, revision: 1, observedAt: sequence, body }
|
|
}
|
|
|
|
describe('isStructuredAgentSessionThinking', () => {
|
|
const turnStart = item('turn-start', 1, {
|
|
kind: 'status',
|
|
text: 'Working',
|
|
turnLifecycle: { turnId: 'turn-1', state: 'running' }
|
|
})
|
|
const reasoning = (sequence: number): AgentJournalRenderItem =>
|
|
item(`reasoning-${sequence}`, sequence, {
|
|
kind: 'message',
|
|
role: 'reasoning',
|
|
blocks: [{ type: 'text', text: 'Weighing two approaches' }]
|
|
})
|
|
|
|
it('is true while reasoning is the newest thing the turn produced', () => {
|
|
expect(isStructuredAgentSessionThinking([turnStart, reasoning(2)])).toBe(true)
|
|
})
|
|
|
|
it('is false once a tool call, a message or a diff lands after the reasoning', () => {
|
|
const after = (body: AgentJournalRenderItem['body']): boolean =>
|
|
isStructuredAgentSessionThinking([turnStart, reasoning(2), item('after', 3, body)])
|
|
expect(after({ kind: 'tool-call', name: 'shell', input: null, state: 'running' })).toBe(false)
|
|
expect(
|
|
after({ kind: 'message', role: 'assistant', blocks: [{ type: 'text', text: 'Here you go' }] })
|
|
).toBe(false)
|
|
expect(
|
|
after({
|
|
kind: 'diff',
|
|
path: 'src/a.ts',
|
|
patch: { head: '@@', byteLength: 2, digest: 'd', truncated: false }
|
|
})
|
|
).toBe(false)
|
|
})
|
|
|
|
it('is false when a turn produced no reasoning at all', () => {
|
|
expect(isStructuredAgentSessionThinking([turnStart])).toBe(false)
|
|
expect(isStructuredAgentSessionThinking([])).toBe(false)
|
|
})
|
|
|
|
it('does not read an earlier turn as this one reasoning', () => {
|
|
// The scan stops at this turn's own record, so the previous turn's reasoning
|
|
// cannot leak forward into a turn that has produced nothing yet.
|
|
const newTurn = item('turn-2-start', 2, {
|
|
kind: 'status',
|
|
text: 'Working',
|
|
turnLifecycle: { turnId: 'turn-2', state: 'running' }
|
|
})
|
|
expect(isStructuredAgentSessionThinking([reasoning(1), newTurn])).toBe(false)
|
|
})
|
|
|
|
it('does not read a completed turn as reasoning during the next pending dispatch', () => {
|
|
const completedTurn = item('turn-1', 1, {
|
|
kind: 'turn',
|
|
turnId: 'turn-1',
|
|
state: 'completed'
|
|
})
|
|
expect(isStructuredAgentSessionThinking([completedTurn, reasoning(2)])).toBe(false)
|
|
})
|
|
|
|
it('stops at a typed turn item, the carrier this host writes', () => {
|
|
const typedTurn = (sequence: number, turnId: string): AgentJournalRenderItem =>
|
|
item(`turn-${turnId}`, sequence, { kind: 'turn', turnId, state: 'running' })
|
|
expect(isStructuredAgentSessionThinking([typedTurn(1, 'turn-1'), reasoning(2)])).toBe(true)
|
|
expect(isStructuredAgentSessionThinking([reasoning(1), typedTurn(2, 'turn-2')])).toBe(false)
|
|
})
|
|
|
|
it('lets an unmarked status stay transparent to the latest reasoning state', () => {
|
|
const plan = item('plan', 3, { kind: 'status', text: 'Step 1. Read the file' })
|
|
expect(isStructuredAgentSessionThinking([turnStart, plan])).toBe(false)
|
|
expect(isStructuredAgentSessionThinking([turnStart, reasoning(2), plan])).toBe(true)
|
|
})
|
|
|
|
it.each([
|
|
{
|
|
kind: 'approval' as const,
|
|
title: 'Run the command?',
|
|
detail: null,
|
|
options: [],
|
|
resolution: {
|
|
state: 'pending' as const,
|
|
selectedOptionId: null,
|
|
resolvedBy: null,
|
|
resolvedAt: null
|
|
}
|
|
},
|
|
{
|
|
kind: 'question' as const,
|
|
question: 'Which path?',
|
|
options: [],
|
|
resolution: {
|
|
state: 'pending' as const,
|
|
selectedOptionId: null,
|
|
resolvedBy: null,
|
|
resolvedAt: null
|
|
}
|
|
}
|
|
])('stops thinking when the turn is waiting on a $kind', (prompt) => {
|
|
expect(
|
|
isStructuredAgentSessionThinking([turnStart, reasoning(2), item('prompt', 3, prompt)])
|
|
).toBe(false)
|
|
})
|
|
})
|