diff options
| author | Seth Troisi <[email protected]> | 2025-05-29 14:03:24 -0700 |
|---|---|---|
| committer | GitHub <[email protected]> | 2025-05-29 14:03:24 -0700 |
| commit | 4b4ba85313bb7fc276595124dabfe078a9015ba1 (patch) | |
| tree | 8a37b4bbd08340bb681d9a4cb369f71bc7bd8706 /packages/server/src | |
| parent | dc94a03f39c712a0bb9b4ab0de7bac0e29c2b12e (diff) | |
improve read-many-files output (#596)
Diffstat (limited to 'packages/server/src')
| -rw-r--r-- | packages/server/src/tools/read-many-files.test.ts | 4 | ||||
| -rw-r--r-- | packages/server/src/tools/read-many-files.ts | 6 |
2 files changed, 6 insertions, 4 deletions
diff --git a/packages/server/src/tools/read-many-files.test.ts b/packages/server/src/tools/read-many-files.test.ts index 50156b55..ea801ddb 100644 --- a/packages/server/src/tools/read-many-files.test.ts +++ b/packages/server/src/tools/read-many-files.test.ts @@ -293,9 +293,7 @@ describe('ReadManyFilesTool', () => { (c) => typeof c === 'string' && c.includes('--- notes.txt ---'), ), ).toBe(true); - expect(result.returnDisplay).toContain( - '**Skipped 1 item(s) (up to 5 shown):**', - ); + expect(result.returnDisplay).toContain('**Skipped 1 item(s):**'); expect(result.returnDisplay).toContain( '- `document.pdf` (Reason: asset file (image/pdf) was not explicitly requested by name or extension)', ); diff --git a/packages/server/src/tools/read-many-files.ts b/packages/server/src/tools/read-many-files.ts index 1f9ab86e..b825de04 100644 --- a/packages/server/src/tools/read-many-files.ts +++ b/packages/server/src/tools/read-many-files.ts @@ -402,7 +402,11 @@ Use this tool when the user's query implies needing the content of several files if (processedFilesRelativePaths.length === 0) { 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 .slice(0, 5) .forEach( |