mirror of
https://github.com/RooVetGit/Roo-Code.git
synced 2026-08-28 05:27:24 +00:00
- Add error handling for VS Code/Cursor 50MB file size limit - Catch and handle "Files above X MB cannot be synchronized" errors - Provide clearer user messages when files exceed display limits - Ensure file operations complete successfully even if display fails Fixes #9659
378 lines
13 KiB
TypeScript
378 lines
13 KiB
TypeScript
import path from "path"
|
|
import delay from "delay"
|
|
import * as vscode from "vscode"
|
|
import fs from "fs/promises"
|
|
|
|
import { Task } from "../task/Task"
|
|
import { ClineSayTool } from "../../shared/ExtensionMessage"
|
|
import { formatResponse } from "../prompts/responses"
|
|
import { RecordSource } from "../context-tracking/FileContextTrackerTypes"
|
|
import { fileExistsAtPath, createDirectoriesForFile } from "../../utils/fs"
|
|
import { stripLineNumbers, everyLineHasLineNumbers } from "../../integrations/misc/extract-text"
|
|
import { getReadablePath } from "../../utils/path"
|
|
import { isPathOutsideWorkspace } from "../../utils/pathUtils"
|
|
import { detectCodeOmission } from "../../integrations/editor/detect-omission"
|
|
import { unescapeHtmlEntities } from "../../utils/text-normalization"
|
|
import { DEFAULT_WRITE_DELAY_MS } from "@roo-code/types"
|
|
import { EXPERIMENT_IDS, experiments } from "../../shared/experiments"
|
|
import { convertNewFileToUnifiedDiff, computeDiffStats, sanitizeUnifiedDiff } from "../diff/stats"
|
|
import { BaseTool, ToolCallbacks } from "./BaseTool"
|
|
import type { ToolUse } from "../../shared/tools"
|
|
import { resolveToolProtocol } from "../../utils/resolveToolProtocol"
|
|
|
|
interface WriteToFileParams {
|
|
path: string
|
|
content: string
|
|
line_count: number
|
|
}
|
|
|
|
export class WriteToFileTool extends BaseTool<"write_to_file"> {
|
|
readonly name = "write_to_file" as const
|
|
|
|
parseLegacy(params: Partial<Record<string, string>>): WriteToFileParams {
|
|
return {
|
|
path: params.path || "",
|
|
content: params.content || "",
|
|
line_count: parseInt(params.line_count ?? "0", 10),
|
|
}
|
|
}
|
|
|
|
async execute(params: WriteToFileParams, task: Task, callbacks: ToolCallbacks): Promise<void> {
|
|
const { pushToolResult, handleError, askApproval, removeClosingTag, toolProtocol } = callbacks
|
|
const relPath = params.path
|
|
let newContent = params.content
|
|
const predictedLineCount = params.line_count
|
|
|
|
if (!relPath) {
|
|
task.consecutiveMistakeCount++
|
|
task.recordToolError("write_to_file")
|
|
pushToolResult(await task.sayAndCreateMissingParamError("write_to_file", "path"))
|
|
await task.diffViewProvider.reset()
|
|
return
|
|
}
|
|
|
|
if (newContent === undefined) {
|
|
task.consecutiveMistakeCount++
|
|
task.recordToolError("write_to_file")
|
|
pushToolResult(await task.sayAndCreateMissingParamError("write_to_file", "content"))
|
|
await task.diffViewProvider.reset()
|
|
return
|
|
}
|
|
|
|
const accessAllowed = task.rooIgnoreController?.validateAccess(relPath)
|
|
|
|
if (!accessAllowed) {
|
|
await task.say("rooignore_error", relPath)
|
|
pushToolResult(formatResponse.rooIgnoreError(relPath, toolProtocol))
|
|
return
|
|
}
|
|
|
|
const isWriteProtected = task.rooProtectedController?.isWriteProtected(relPath) || false
|
|
|
|
let fileExists: boolean
|
|
const absolutePath = path.resolve(task.cwd, relPath)
|
|
|
|
if (task.diffViewProvider.editType !== undefined) {
|
|
fileExists = task.diffViewProvider.editType === "modify"
|
|
} else {
|
|
fileExists = await fileExistsAtPath(absolutePath)
|
|
task.diffViewProvider.editType = fileExists ? "modify" : "create"
|
|
}
|
|
|
|
// Create parent directories early for new files to prevent ENOENT errors
|
|
// in subsequent operations (e.g., diffViewProvider.open, fs.readFile)
|
|
if (!fileExists) {
|
|
await createDirectoriesForFile(absolutePath)
|
|
}
|
|
|
|
if (newContent.startsWith("```")) {
|
|
newContent = newContent.split("\n").slice(1).join("\n")
|
|
}
|
|
|
|
if (newContent.endsWith("```")) {
|
|
newContent = newContent.split("\n").slice(0, -1).join("\n")
|
|
}
|
|
|
|
if (!task.api.getModel().id.includes("claude")) {
|
|
newContent = unescapeHtmlEntities(newContent)
|
|
}
|
|
|
|
const fullPath = relPath ? path.resolve(task.cwd, removeClosingTag("path", relPath)) : ""
|
|
const isOutsideWorkspace = isPathOutsideWorkspace(fullPath)
|
|
|
|
const sharedMessageProps: ClineSayTool = {
|
|
tool: fileExists ? "editedExistingFile" : "newFileCreated",
|
|
path: getReadablePath(task.cwd, removeClosingTag("path", relPath)),
|
|
content: newContent,
|
|
isOutsideWorkspace,
|
|
isProtected: isWriteProtected,
|
|
}
|
|
|
|
try {
|
|
if (predictedLineCount === undefined || predictedLineCount === 0) {
|
|
task.consecutiveMistakeCount++
|
|
task.recordToolError("write_to_file")
|
|
task.didToolFailInCurrentTurn = true
|
|
|
|
const actualLineCount = newContent.split("\n").length
|
|
const isNewFile = !fileExists
|
|
const diffStrategyEnabled = !!task.diffStrategy
|
|
const modelInfo = task.api.getModel().info
|
|
const toolProtocol = resolveToolProtocol(task.apiConfiguration, modelInfo)
|
|
|
|
await task.say(
|
|
"error",
|
|
`Roo tried to use write_to_file${
|
|
relPath ? ` for '${relPath.toPosix()}'` : ""
|
|
} but the required parameter 'line_count' was missing or truncated after ${actualLineCount} lines of content were written. Retrying...`,
|
|
)
|
|
|
|
pushToolResult(
|
|
formatResponse.toolError(
|
|
formatResponse.lineCountTruncationError(
|
|
actualLineCount,
|
|
isNewFile,
|
|
diffStrategyEnabled,
|
|
toolProtocol,
|
|
),
|
|
),
|
|
)
|
|
await task.diffViewProvider.revertChanges()
|
|
return
|
|
}
|
|
|
|
task.consecutiveMistakeCount = 0
|
|
|
|
const provider = task.providerRef.deref()
|
|
const state = await provider?.getState()
|
|
const diagnosticsEnabled = state?.diagnosticsEnabled ?? true
|
|
const writeDelayMs = state?.writeDelayMs ?? DEFAULT_WRITE_DELAY_MS
|
|
const isPreventFocusDisruptionEnabled = experiments.isEnabled(
|
|
state?.experiments ?? {},
|
|
EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION,
|
|
)
|
|
|
|
if (isPreventFocusDisruptionEnabled) {
|
|
task.diffViewProvider.editType = fileExists ? "modify" : "create"
|
|
if (fileExists) {
|
|
const absolutePath = path.resolve(task.cwd, relPath)
|
|
task.diffViewProvider.originalContent = await fs.readFile(absolutePath, "utf-8")
|
|
} else {
|
|
task.diffViewProvider.originalContent = ""
|
|
}
|
|
|
|
if (detectCodeOmission(task.diffViewProvider.originalContent || "", newContent, predictedLineCount)) {
|
|
if (task.diffStrategy) {
|
|
pushToolResult(
|
|
formatResponse.toolError(
|
|
`Content appears to be truncated (file has ${
|
|
newContent.split("\n").length
|
|
} lines but was predicted to have ${predictedLineCount} lines), and found comments indicating omitted code (e.g., '// rest of code unchanged', '/* previous code */'). Please provide the complete file content without any omissions if possible, or otherwise use the 'apply_diff' tool to apply the diff to the original file.`,
|
|
),
|
|
)
|
|
return
|
|
} else {
|
|
vscode.window
|
|
.showWarningMessage(
|
|
"Potential code truncation detected. cline happens when the AI reaches its max output limit.",
|
|
"Follow cline guide to fix the issue",
|
|
)
|
|
.then((selection) => {
|
|
if (selection === "Follow cline guide to fix the issue") {
|
|
vscode.env.openExternal(
|
|
vscode.Uri.parse(
|
|
"https://github.com/cline/cline/wiki/Troubleshooting-%E2%80%90-Cline-Deleting-Code-with-%22Rest-of-Code-Here%22-Comments",
|
|
),
|
|
)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
let unified = fileExists
|
|
? formatResponse.createPrettyPatch(relPath, task.diffViewProvider.originalContent, newContent)
|
|
: convertNewFileToUnifiedDiff(newContent, relPath)
|
|
unified = sanitizeUnifiedDiff(unified)
|
|
const completeMessage = JSON.stringify({
|
|
...sharedMessageProps,
|
|
content: unified,
|
|
diffStats: computeDiffStats(unified) || undefined,
|
|
} satisfies ClineSayTool)
|
|
|
|
const didApprove = await askApproval("tool", completeMessage, undefined, isWriteProtected)
|
|
|
|
if (!didApprove) {
|
|
return
|
|
}
|
|
|
|
await task.diffViewProvider.saveDirectly(relPath, newContent, false, diagnosticsEnabled, writeDelayMs)
|
|
} else {
|
|
if (!task.diffViewProvider.isEditing) {
|
|
const partialMessage = JSON.stringify(sharedMessageProps)
|
|
await task.ask("tool", partialMessage, true).catch(() => {})
|
|
await task.diffViewProvider.open(relPath)
|
|
}
|
|
|
|
await task.diffViewProvider.update(
|
|
everyLineHasLineNumbers(newContent) ? stripLineNumbers(newContent) : newContent,
|
|
true,
|
|
)
|
|
|
|
await delay(300)
|
|
task.diffViewProvider.scrollToFirstDiff()
|
|
|
|
if (detectCodeOmission(task.diffViewProvider.originalContent || "", newContent, predictedLineCount)) {
|
|
if (task.diffStrategy) {
|
|
await task.diffViewProvider.revertChanges()
|
|
|
|
pushToolResult(
|
|
formatResponse.toolError(
|
|
`Content appears to be truncated (file has ${
|
|
newContent.split("\n").length
|
|
} lines but was predicted to have ${predictedLineCount} lines), and found comments indicating omitted code (e.g., '// rest of code unchanged', '/* previous code */'). Please provide the complete file content without any omissions if possible, or otherwise use the 'apply_diff' tool to apply the diff to the original file.`,
|
|
),
|
|
)
|
|
return
|
|
} else {
|
|
vscode.window
|
|
.showWarningMessage(
|
|
"Potential code truncation detected. cline happens when the AI reaches its max output limit.",
|
|
"Follow cline guide to fix the issue",
|
|
)
|
|
.then((selection) => {
|
|
if (selection === "Follow cline guide to fix the issue") {
|
|
vscode.env.openExternal(
|
|
vscode.Uri.parse(
|
|
"https://github.com/cline/cline/wiki/Troubleshooting-%E2%80%90-Cline-Deleting-Code-with-%22Rest-of-Code-Here%22-Comments",
|
|
),
|
|
)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
let unified = fileExists
|
|
? formatResponse.createPrettyPatch(relPath, task.diffViewProvider.originalContent, newContent)
|
|
: convertNewFileToUnifiedDiff(newContent, relPath)
|
|
unified = sanitizeUnifiedDiff(unified)
|
|
const completeMessage = JSON.stringify({
|
|
...sharedMessageProps,
|
|
content: unified,
|
|
diffStats: computeDiffStats(unified) || undefined,
|
|
} satisfies ClineSayTool)
|
|
|
|
const didApprove = await askApproval("tool", completeMessage, undefined, isWriteProtected)
|
|
|
|
if (!didApprove) {
|
|
await task.diffViewProvider.revertChanges()
|
|
return
|
|
}
|
|
|
|
await task.diffViewProvider.saveChanges(diagnosticsEnabled, writeDelayMs)
|
|
}
|
|
|
|
if (relPath) {
|
|
await task.fileContextTracker.trackFileContext(relPath, "roo_edited" as RecordSource)
|
|
}
|
|
|
|
task.didEditFile = true
|
|
|
|
const message = await task.diffViewProvider.pushToolWriteResult(task, task.cwd, !fileExists)
|
|
|
|
pushToolResult(message)
|
|
|
|
await task.diffViewProvider.reset()
|
|
|
|
task.processQueuedMessages()
|
|
|
|
return
|
|
} catch (error) {
|
|
const err = error as Error
|
|
// Check if this is a VS Code file size limit error
|
|
if (err?.message?.includes("Files above") && err?.message?.includes("MB")) {
|
|
// The file was likely written successfully, but VS Code can't display it
|
|
// Provide a more helpful message to the user
|
|
await task.say(
|
|
"text",
|
|
`The file was saved successfully, but it exceeds VS Code's display size limit for extensions. The file operation completed successfully.`,
|
|
)
|
|
pushToolResult(
|
|
formatResponse.toolResult(
|
|
`File saved successfully. Note: File exceeds VS Code's display size limit but the operation completed.`,
|
|
[],
|
|
),
|
|
)
|
|
await task.diffViewProvider.reset()
|
|
return
|
|
}
|
|
// For other errors, use the standard error handling
|
|
await handleError("writing file", err)
|
|
await task.diffViewProvider.reset()
|
|
return
|
|
}
|
|
}
|
|
|
|
override async handlePartial(task: Task, block: ToolUse<"write_to_file">): Promise<void> {
|
|
const relPath: string | undefined = block.params.path
|
|
let newContent: string | undefined = block.params.content
|
|
|
|
if (!relPath || newContent === undefined) {
|
|
return
|
|
}
|
|
|
|
const provider = task.providerRef.deref()
|
|
const state = await provider?.getState()
|
|
const isPreventFocusDisruptionEnabled = experiments.isEnabled(
|
|
state?.experiments ?? {},
|
|
EXPERIMENT_IDS.PREVENT_FOCUS_DISRUPTION,
|
|
)
|
|
|
|
if (isPreventFocusDisruptionEnabled) {
|
|
return
|
|
}
|
|
|
|
let fileExists: boolean
|
|
const absolutePath = path.resolve(task.cwd, relPath)
|
|
|
|
if (task.diffViewProvider.editType !== undefined) {
|
|
fileExists = task.diffViewProvider.editType === "modify"
|
|
} else {
|
|
fileExists = await fileExistsAtPath(absolutePath)
|
|
task.diffViewProvider.editType = fileExists ? "modify" : "create"
|
|
}
|
|
|
|
// Create parent directories early for new files to prevent ENOENT errors
|
|
// in subsequent operations (e.g., diffViewProvider.open)
|
|
if (!fileExists) {
|
|
await createDirectoriesForFile(absolutePath)
|
|
}
|
|
|
|
const isWriteProtected = task.rooProtectedController?.isWriteProtected(relPath) || false
|
|
const fullPath = absolutePath
|
|
const isOutsideWorkspace = isPathOutsideWorkspace(fullPath)
|
|
|
|
const sharedMessageProps: ClineSayTool = {
|
|
tool: fileExists ? "editedExistingFile" : "newFileCreated",
|
|
path: getReadablePath(task.cwd, relPath),
|
|
content: newContent || "",
|
|
isOutsideWorkspace,
|
|
isProtected: isWriteProtected,
|
|
}
|
|
|
|
const partialMessage = JSON.stringify(sharedMessageProps)
|
|
await task.ask("tool", partialMessage, block.partial).catch(() => {})
|
|
|
|
if (newContent) {
|
|
if (!task.diffViewProvider.isEditing) {
|
|
await task.diffViewProvider.open(relPath)
|
|
}
|
|
|
|
await task.diffViewProvider.update(
|
|
everyLineHasLineNumbers(newContent) ? stripLineNumbers(newContent) : newContent,
|
|
false,
|
|
)
|
|
}
|
|
}
|
|
}
|
|
|
|
export const writeToFileTool = new WriteToFileTool()
|