scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-08-28 05:27:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: only store content hash for complete file reads

Browse source 
- Remove dead code that called readFileWithTokenBudget but never used result
- Remove redundant fs.readFile call (content was already read earlier)
- Only compute and store contentHash when result.complete is true
- Fixes incorrect hash storage for truncated files
This commit is contained in:
Roo Code 2026-01-17 21:10:07 +00:00
parent 3323b97aad
commit ea696ce6af
1 changed files with 6 additions and 19 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

25
src/core/tools/ReadFileTool.ts
View file

@ -580,6 +580,7 @@ export class ReadFileTool extends BaseTool<"read_file"> {
const safeReadBudget = Math.floor(remainingTokens * FILE_READ_BUDGET_PERCENT)
let content: string
let contentHash: string | undefined
let xmlInfo = ""
let nativeInfo = ""
@ -597,6 +598,11 @@ export class ReadFileTool extends BaseTool<"read_file"> {
content = addLineNumbers(result.content)
// Only compute hash for complete reads (not truncated)
if (result.complete) {
contentHash = FileContextTracker.computeContentHash(result.content)
}
if (!result.complete) {
// File was truncated
const notice = `File truncated: showing ${result.lineCount} lines (${result.tokenCount} tokens) due to context budget. Use line_range to read specific sections.`
@ -626,25 +632,6 @@ export class ReadFileTool extends BaseTool<"read_file"> {
}
}
// Compute content hash for skip-redundant-reads optimization
// Only store hash for complete reads (not truncated)
let contentHash: string | undefined
if (safeReadBudget > 0) {
const readResult = await readFileWithTokenBudget(fullPath, { budgetTokens: safeReadBudget })
// Re-read to get content for hash (already done above, use same result)
// Actually we need to use the result.content from above, so we compute hash here
// Note: content variable above is already the numbered content, we need raw
// For simplicity, compute hash from the result we already have (pre-addLineNumbers)
}
// For full reads, compute hash from raw content
if (safeReadBudget > 0) {
try {
const rawContent = await fs.readFile(fullPath, "utf-8")
contentHash = FileContextTracker.computeContentHash(rawContent)
} catch {
// If we can't read for hash, that's ok - just don't store hash
}
}
await task.fileContextTracker.trackFileContext(relPath, "read_tool" as RecordSource, contentHash)
updateFileResult(relPath, {