MediaMetz/gemini-cli
1
0
Fork 0
mirror of https://github.com/google-gemini/gemini-cli.git synced 2026-07-05 07:37:50 -07:00
Code Issues Packages Projects Releases Wiki Activity

fix(core): restore ReadFolder declarative display and add missing test snapshots

Browse Source
This commit is contained in:
Michael Bleigh
2026-04-12 16:44:05 -07:00
parent 8548c6675f
commit 46377d2133
2 changed files with 91 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
packages/cli/src/ui/components/messages/__snapshots__/ToolGroupDisplay.test.tsx.snap
+83
View File
@@ -0,0 +1,83 @@
// Vitest Snapshot v1, https://vitest.dev/guide/snapshot.html
exports[`<ToolGroupDisplay /> > Border & Margin Logic > applies bottom margin in compact mode when group is at boundary 1`] = `
" ✓ Compact Tool Test description
"
`;
exports[`<ToolGroupDisplay /> > Border & Margin Logic > forces top border on box when it follows a notice 1`] = `
" Notice:
Test description
╭──────────────────────────────────────────────────────────────────────────────────────────────────╮
│ ✓ Tool in Box Test description │
│ │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
"
`;
exports[`<ToolGroupDisplay /> > Golden Snapshots > renders error message when display info is missing 1`] = `
" ⊷ Error: Tool display missing
"
`;
exports[`<ToolGroupDisplay /> > Golden Snapshots > renders in boxed mode (full verbosity) 1`] = `
"
╭──────────────────────────────────────────────────────────────────────────────────────────────────╮
│ ✓ Tool A Test description │
│ │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
"
`;
exports[`<ToolGroupDisplay /> > Golden Snapshots > renders in compact mode (no box borders) 1`] = `
" ✓ Tool A Test description
✓ Tool B Test description
"
`;
exports[`<ToolGroupDisplay /> > Golden Snapshots > renders notices at the top (hoisting) 1`] = `
"
Update Topic Context:
New Topic
╭──────────────────────────────────────────────────────────────────────────────────────────────────╮
│ ✓ Tool A Test description │
│ │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
"
`;
exports[`<ToolGroupDisplay /> > Golden Snapshots > renders standalone notices without a box 1`] = `
"
Notice Only:
Test description
"
`;
exports[`<ToolGroupDisplay /> > Result Formatting > renders compact tools with summary on same line 1`] = `
" ✓ test-tool Test description → Success summary
"
`;
exports[`<ToolGroupDisplay /> > Result Formatting > renders placeholder for diff results 1`] = `
"
╭──────────────────────────────────────────────────────────────────────────────────────────────────╮
│ ✓ test-tool Test description │
│ [Diff Display: 3 -> 3 chars] │
│ │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
"
`;
exports[`<ToolGroupDisplay /> > Result Formatting > renders text results with summary below 1`] = `
"
╭──────────────────────────────────────────────────────────────────────────────────────────────────╮
│ ✓ test-tool Test description │
│ Detailed output │
│ │
│ Short summary │
│ │
╰──────────────────────────────────────────────────────────────────────────────────────────────────╯
"
`;
packages/core/src/tools/ls.ts
+8
View File
@@ -288,6 +288,14 @@ class LSToolInvocation extends BaseToolInvocation<LSToolParams, ToolResult> {
name: LS_DISPLAY_NAME,
description: this.getDescription(),
resultSummary: displayMessage,
result: {
type: 'text',
text: entries
.map(
(entry) => `${entry.isDirectory ? '[DIR] ' : ''}${entry.name}`,
)
.join('\n'),
},
},
returnDisplay: {
summary: displayMessage,