test: cover the TUI search-card fallback and refresh the cordis-inspect golden
- Add a search result-view scenario to the TUI tool-card test so the card:'search' fallback branch (no view content -> raw result content) is covered; restores transcript.ts branch coverage to 100%. - Refresh the cordis-inspect-jsdoc golden for the kind -> shape rename in the ToolResultView JSDoc that cordis_inspect echoes.
This commit is contained in:
File diff suppressed because one or more lines are too long
@@ -4368,6 +4368,20 @@ describe('tool cards and surface replay', () => {
|
|||||||
presentCall: () => ({ card: 'generic', title: 'Becomes terminal' }),
|
presentCall: () => ({ card: 'generic', title: 'Becomes terminal' }),
|
||||||
presentResult: () => ({ card: 'terminal', output: 'converted terminal' }),
|
presentResult: () => ({ card: 'terminal', output: 'converted terminal' }),
|
||||||
},
|
},
|
||||||
|
// A search card carries no result text of its own; the TUI has no dedicated
|
||||||
|
// search arm and falls back to the raw result content, rendered as the same
|
||||||
|
// dim generic body a pre-search-card grep/glob result showed.
|
||||||
|
search: {
|
||||||
|
name: 'search', description: '', parameters: {}, output: UNUSED_TOOL_OUTPUT, execute: async () => [],
|
||||||
|
presentCall: () => ({ card: 'generic', title: 'Grep todo', kind: 'search' }),
|
||||||
|
presentResult: () => ({
|
||||||
|
card: 'search',
|
||||||
|
shape: 'matches',
|
||||||
|
files: [{ path: 'a.ts', matches: [{ lineNumber: 1, line: 'todo one' }] }],
|
||||||
|
truncated: false,
|
||||||
|
total: 1,
|
||||||
|
}),
|
||||||
|
},
|
||||||
symbolic: {
|
symbolic: {
|
||||||
name: 'symbolic', description: '', parameters: {}, output: UNUSED_TOOL_OUTPUT, execute: async () => [],
|
name: 'symbolic', description: '', parameters: {}, output: UNUSED_TOOL_OUTPUT, execute: async () => [],
|
||||||
presentCall: () => ({ card: 'generic', title: 'Symbol input', rawInput: Symbol('input') }),
|
presentCall: () => ({ card: 'generic', title: 'Symbol input', rawInput: Symbol('input') }),
|
||||||
@@ -4396,6 +4410,7 @@ describe('tool cards and surface replay', () => {
|
|||||||
['c11', 'terminalResult', '{}'],
|
['c11', 'terminalResult', '{}'],
|
||||||
['c12', 'symbolic', '{}'],
|
['c12', 'symbolic', '{}'],
|
||||||
['c13', 'knownXml', '{}'],
|
['c13', 'knownXml', '{}'],
|
||||||
|
['c16', 'search', '{"pattern":"todo"}'],
|
||||||
] as const
|
] as const
|
||||||
appendAssistant(result.session, [
|
appendAssistant(result.session, [
|
||||||
{ type: 'text', text: 'Calling tools' },
|
{ type: 'text', text: 'Calling tools' },
|
||||||
@@ -4489,6 +4504,14 @@ describe('tool cards and surface replay', () => {
|
|||||||
isError: false,
|
isError: false,
|
||||||
}),
|
}),
|
||||||
}, { surfaceOp: 'append' })
|
}, { surfaceOp: 'append' })
|
||||||
|
result.session.append('tool/result', {
|
||||||
|
turn: 1, step: 1,
|
||||||
|
message: createToolResultMessage({
|
||||||
|
callId: 'c16' as never,
|
||||||
|
content: [{ type: 'text', text: 'Found 1 match\n\na.ts\nLine 1: todo one' }],
|
||||||
|
isError: false,
|
||||||
|
}),
|
||||||
|
}, { surfaceOp: 'append' })
|
||||||
result.session.append('tool/result', {
|
result.session.append('tool/result', {
|
||||||
turn: 1,
|
turn: 1,
|
||||||
step: 1,
|
step: 1,
|
||||||
@@ -4520,6 +4543,11 @@ describe('tool cards and surface replay', () => {
|
|||||||
expect(output).toContain('$ blank desc command')
|
expect(output).toContain('$ blank desc command')
|
||||||
// A card whose title only repeats the name renders header-only (empty body).
|
// A card whose title only repeats the name renders header-only (empty body).
|
||||||
expect(output).toContain('Tool / emptyBody')
|
expect(output).toContain('Tool / emptyBody')
|
||||||
|
// A search result view carries no `content` of its own, so the card renders
|
||||||
|
// the raw model-facing result text through the same dim generic body — the
|
||||||
|
// TUI has no dedicated search arm.
|
||||||
|
expect(output).toContain('Tool / search')
|
||||||
|
expect(output).toContain('Line 1: todo one')
|
||||||
// A diff card drops its title (the paths + change footer carry the meaning).
|
// A diff card drops its title (the paths + change footer carry the meaning).
|
||||||
// The first file's path is head-visible; the second file and the change
|
// The first file's path is head-visible; the second file and the change
|
||||||
// footer sit past this card's 4-line budget and appear only when expanded.
|
// footer sit past this card's 4-line budget and appear only when expanded.
|
||||||
|
|||||||
Reference in New Issue
Block a user