From c0ad46fa012dda4dd8e7ba4e0f1e1f6b5b3b7ed4 Mon Sep 17 00:00:00 2001 From: Mario Zechner Date: Thu, 9 Oct 2025 23:30:40 +0200 Subject: [PATCH] Fix javascript-repl to throw errors on execution failure - Throw error instead of returning output when result.success is false - Ensures tool call is marked as failed when code execution fails or is aborted - Matches browser-javascript error handling pattern --- packages/web-ui/src/tools/javascript-repl.ts | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/packages/web-ui/src/tools/javascript-repl.ts b/packages/web-ui/src/tools/javascript-repl.ts index 7d1cbd81..9031f039 100644 --- a/packages/web-ui/src/tools/javascript-repl.ts +++ b/packages/web-ui/src/tools/javascript-repl.ts @@ -59,9 +59,8 @@ export async function executeJavaScript( if (output) output += "\n"; output += `Error: ${result.error?.message || "Unknown error"}\n${result.error?.stack || ""}`; - return { - output: output.trim(), - }; + // Throw error so tool call is marked as failed + throw new Error(output.trim()); } // Add return value if present