Fix coding agent tools to return error content instead of throwing

Tools now resolve with error messages in content blocks rather than
rejecting the promise. This matches the expected behavior where tools
always return a result, with errors indicated in the text content.

- read: Return error content for missing files
- edit: Return error content for missing files, text not found, multiple matches
- bash: Return error content for command failures, timeouts, and aborts
- All tools now include required details field in error results
This commit is contained in:
Mario Zechner 2025-11-12 10:52:12 +01:00
parent 84dcab219b
commit f147109da7
3 changed files with 33 additions and 16 deletions

View file

@ -66,7 +66,10 @@ export const editTool: AgentTool<typeof editSchema> = {
if (signal) {
signal.removeEventListener("abort", onAbort);
}
reject(new Error(`File not found: ${path}`));
resolve({
content: [{ type: "text", text: `Error: File not found: ${path}` }],
details: undefined,
});
return;
}
@ -88,11 +91,15 @@ export const editTool: AgentTool<typeof editSchema> = {
if (signal) {
signal.removeEventListener("abort", onAbort);
}
reject(
new Error(
`Could not find the exact text in ${path}. The old text must match exactly including all whitespace and newlines.`,
),
);
resolve({
content: [
{
type: "text",
text: `Error: Could not find the exact text in ${path}. The old text must match exactly including all whitespace and newlines.`,
},
],
details: undefined,
});
return;
}
@ -103,11 +110,15 @@ export const editTool: AgentTool<typeof editSchema> = {
if (signal) {
signal.removeEventListener("abort", onAbort);
}
reject(
new Error(
`Found ${occurrences} occurrences of the text in ${path}. The text must be unique. Please provide more context to make it unique.`,
),
);
resolve({
content: [
{
type: "text",
text: `Error: Found ${occurrences} occurrences of the text in ${path}. The text must be unique. Please provide more context to make it unique.`,
},
],
details: undefined,
});
return;
}