From c5608869c00c433a468fe5e88bcbafd83f6599a1 Mon Sep 17 00:00:00 2001 From: Taylor Mullen Date: Thu, 29 May 2025 21:57:25 -0700 Subject: [PATCH] Change ReadFile to not have a result display. - It's verbose on its own; however, if file content is truncated we'll indicate that in the result display. --- packages/server/src/utils/fileUtils.test.ts | 8 +++----- packages/server/src/utils/fileUtils.ts | 2 +- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/packages/server/src/utils/fileUtils.test.ts b/packages/server/src/utils/fileUtils.test.ts index df912b0c..4f4c7c1e 100644 --- a/packages/server/src/utils/fileUtils.test.ts +++ b/packages/server/src/utils/fileUtils.test.ts @@ -258,7 +258,7 @@ describe('fileUtils', () => { tempRootDir, ); expect(result.llmContent).toBe(content); - expect(result.returnDisplay).toContain('Read text file: test.txt'); + expect(result.returnDisplay).toBe(''); expect(result.error).toBeUndefined(); }); @@ -380,9 +380,7 @@ describe('fileUtils', () => { expect(result.llmContent).toContain( '[File content truncated: showing lines 6-10 of 20 total lines. Use offset/limit parameters to view more.]', ); - expect(result.returnDisplay).toContain( - 'Read text file: test.txt (truncated)', - ); + expect(result.returnDisplay).toBe('(truncated)'); expect(result.isTruncated).toBe(true); expect(result.originalLineCount).toBe(20); expect(result.linesShown).toEqual([6, 10]); @@ -401,7 +399,7 @@ describe('fileUtils', () => { const expectedContent = lines.join('\n'); expect(result.llmContent).toBe(expectedContent); - expect(result.returnDisplay).toContain('Read text file: test.txt'); + expect(result.returnDisplay).toBe(''); expect(result.isTruncated).toBe(false); expect(result.originalLineCount).toBe(2); expect(result.linesShown).toEqual([1, 2]); diff --git a/packages/server/src/utils/fileUtils.ts b/packages/server/src/utils/fileUtils.ts index 0e9eef4c..d726c053 100644 --- a/packages/server/src/utils/fileUtils.ts +++ b/packages/server/src/utils/fileUtils.ts @@ -236,7 +236,7 @@ export async function processSingleFileContent( return { llmContent: llmTextContent, - returnDisplay: `Read text file: ${relativePathForDisplay}${isTruncated ? ' (truncated)' : ''}`, + returnDisplay: isTruncated ? '(truncated)' : '', isTruncated, originalLineCount, linesShown: [actualStartLine + 1, endLine],