jcarr
/
gemini-cli
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

improve read-many-files output (#596)

This commit is contained in:
Seth Troisi 2025-05-29 14:03:24 -07:00 committed by GitHub
parent dc94a03f39
commit 4b4ba85313
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 6 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
packages/server/src/tools/read-many-files.test.ts
View File

@ -293,9 +293,7 @@ describe('ReadManyFilesTool', () => {
(c) => typeof c === 'string' && c.includes('--- notes.txt ---'), (c) => typeof c === 'string' && c.includes('--- notes.txt ---'),
), ),
).toBe(true); ).toBe(true);
expect(result.returnDisplay).toContain( expect(result.returnDisplay).toContain('**Skipped 1 item(s):**');
'**Skipped 1 item(s) (up to 5 shown):**',
);
expect(result.returnDisplay).toContain( expect(result.returnDisplay).toContain(
'- `document.pdf` (Reason: asset file (image/pdf) was not explicitly requested by name or extension)', '- `document.pdf` (Reason: asset file (image/pdf) was not explicitly requested by name or extension)',
); );

6
packages/server/src/tools/read-many-files.ts
View File

@ -402,7 +402,11 @@ Use this tool when the user's query implies needing the content of several files
if (processedFilesRelativePaths.length === 0) { if (processedFilesRelativePaths.length === 0) {
displayMessage += `No files were read and concatenated based on the criteria.\n`; displayMessage += `No files were read and concatenated based on the criteria.\n`;
} }
displayMessage += `\n**Skipped ${skippedFiles.length} item(s) (up to 5 shown):**\n`; if (skippedFiles.length <= 5) {
displayMessage += `\n**Skipped ${skippedFiles.length} item(s):**\n`;
} else {
displayMessage += `\n**Skipped ${skippedFiles.length} item(s) (first 5 shown):**\n`;
}
skippedFiles skippedFiles
.slice(0, 5) .slice(0, 5)
.forEach( .forEach(