shumengya/sproutclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Display return values and fix duplicate error messages in REPL

Browse Source 
- javascript-repl now displays return values with => prefix
- Objects are JSON.stringify'd with formatting
- Removed console.error from wrapper to prevent duplicate error messages
- Error is already captured and sent via execution-error message
- Fixes missing return value display and duplicate error output
This commit is contained in:
Mario Zechner
2025-10-09 20:51:25 +02:00
parent 5fc3857666
commit 8747e01c31
2 changed files with 7 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
packages/web-ui/src/components/SandboxedIframe.ts
View File

@@ -347,8 +347,7 @@ export class SandboxIframe extends LitElement {
await window.complete(null, returnValue); await window.complete(null, returnValue);
} catch (error) { } catch (error) {
console.error(error?.stack || error?.message || String(error));
// Call completion callbacks before complete() (error path) // Call completion callbacks before complete() (error path)
if (window.__completionCallbacks && window.__completionCallbacks.length > 0) { if (window.__completionCallbacks && window.__completionCallbacks.length > 0) {
try { try {

6
packages/web-ui/src/tools/javascript-repl.ts
View File

@@ -63,6 +63,12 @@ export async function executeJavaScript(
} }
} }
// Add return value if present
if (result.returnValue !== undefined) {
if (output) output += "\n";
output += `=> ${typeof result.returnValue === "object" ? JSON.stringify(result.returnValue, null, 2) : result.returnValue}`;
}
// Add file notifications // Add file notifications
if (result.files && result.files.length > 0) { if (result.files && result.files.length > 0) {
output += `\n[Files returned: ${result.files.length}]\n`; output += `\n[Files returned: ${result.files.length}]\n`;