fix: show errorText for failed tools, bump eval max_iterations to 300

eval/hf_agent_connector.py

@@ -29,7 +29,7 @@ class AgentResponseGenerator:
     returns the assistant's final message.
     """
 
-    def __init__(self, config_path: str | Path, max_iterations: int = 10) -> None:
+    def __init__(self, config_path: str | Path, max_iterations: int = 300) -> None:
         self.config_path = _resolve_project_path(config_path)
         self.config: Config = load_config(str(self.config_path))
         self.max_iterations = max_iterations

frontend/src/components/Chat/ToolCallGroup.tsx

@@ -371,13 +371,14 @@ export default function ToolCallGroup({ tools, approveTools }: ToolCallGroupProp
       const displayName = toolDisplayMap[tool.toolCallId] || tool.toolName;
 
       if (tool.toolName === 'hf_jobs' && args?.script) {
-        const hasOutput = (tool.state === 'output-available' || tool.state === 'output-error') && tool.output;
+        const jobOutput = tool.output ?? (tool.state === 'output-error' ? (tool as Record<string, unknown>).errorText : undefined);
+        const hasOutput = (tool.state === 'output-available' || tool.state === 'output-error') && jobOutput;
         const scriptContent = getEditedScript(tool.toolCallId) || String(args.script);
         setPanel(
           {
             title: displayName,
             script: { content: scriptContent, language: 'python' },
-            ...(hasOutput ? { output: { content: String(tool.output), language: 'markdown' } } : {}),
+            ...(hasOutput ? { output: { content: String(jobOutput), language: 'markdown' } } : {}),
             parameters: { tool_call_id: tool.toolCallId },
           },
           hasOutput ? 'output' : 'script',
@@ -389,9 +390,10 @@ export default function ToolCallGroup({ tools, approveTools }: ToolCallGroupProp
 
       const inputSection = args ? { content: JSON.stringify(args, null, 2), language: 'json' } : undefined;
 
-      if ((tool.state === 'output-available' || tool.state === 'output-error') && tool.output) {
+      const outputText = tool.output ?? (tool.state === 'output-error' ? (tool as Record<string, unknown>).errorText : undefined);
+      if ((tool.state === 'output-available' || tool.state === 'output-error') && outputText) {
         let language = 'text';
-        const content = String(tool.output);
+        const content = String(outputText);
         if (content.trim().startsWith('{') || content.trim().startsWith('[')) language = 'json';
         else if (content.includes('```')) language = 'markdown';