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.
This commit is contained in:
Taylor Mullen 2025-05-29 21:57:25 -07:00 committed by N. Taylor Mullen
parent 9537ff4762
commit c5608869c0
2 changed files with 4 additions and 6 deletions

View File

@ -258,7 +258,7 @@ describe('fileUtils', () => {
tempRootDir, tempRootDir,
); );
expect(result.llmContent).toBe(content); expect(result.llmContent).toBe(content);
expect(result.returnDisplay).toContain('Read text file: test.txt'); expect(result.returnDisplay).toBe('');
expect(result.error).toBeUndefined(); expect(result.error).toBeUndefined();
}); });
@ -380,9 +380,7 @@ describe('fileUtils', () => {
expect(result.llmContent).toContain( expect(result.llmContent).toContain(
'[File content truncated: showing lines 6-10 of 20 total lines. Use offset/limit parameters to view more.]', '[File content truncated: showing lines 6-10 of 20 total lines. Use offset/limit parameters to view more.]',
); );
expect(result.returnDisplay).toContain( expect(result.returnDisplay).toBe('(truncated)');
'Read text file: test.txt (truncated)',
);
expect(result.isTruncated).toBe(true); expect(result.isTruncated).toBe(true);
expect(result.originalLineCount).toBe(20); expect(result.originalLineCount).toBe(20);
expect(result.linesShown).toEqual([6, 10]); expect(result.linesShown).toEqual([6, 10]);
@ -401,7 +399,7 @@ describe('fileUtils', () => {
const expectedContent = lines.join('\n'); const expectedContent = lines.join('\n');
expect(result.llmContent).toBe(expectedContent); 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.isTruncated).toBe(false);
expect(result.originalLineCount).toBe(2); expect(result.originalLineCount).toBe(2);
expect(result.linesShown).toEqual([1, 2]); expect(result.linesShown).toEqual([1, 2]);

View File

@ -236,7 +236,7 @@ export async function processSingleFileContent(
return { return {
llmContent: llmTextContent, llmContent: llmTextContent,
returnDisplay: `Read text file: ${relativePathForDisplay}${isTruncated ? ' (truncated)' : ''}`, returnDisplay: isTruncated ? '(truncated)' : '',
isTruncated, isTruncated,
originalLineCount, originalLineCount,
linesShown: [actualStartLine + 1, endLine], linesShown: [actualStartLine + 1, endLine],