format results

Signed-off-by: Yuval Turgeman <[email protected]>

Author: yuvalturg

backend/app/api/v1/chat_sessions.py

@@ -321,6 +321,7 @@ async def get_conversation_messages(
 
                 elif item_type in ("mcp_call", "file_search_call", "web_search_call"):
                     # Tool call field mappings: (name_field, server_label_field, args_field, output_field)
+                    # Note: web_search_call uses None for output_field because results aren't exposed in API
                     tool_map = {
                         "mcp_call": ("name", "server_label", "arguments", "output"),
                         "file_search_call": (
@@ -333,7 +334,7 @@ async def get_conversation_messages(
                             "web_search",
                             "llamastack",
                             "query",
-                            "results",
+                            None,
                         ),
                     }
 
@@ -354,15 +355,22 @@ async def get_conversation_messages(
                     )
 
                     args_val = item_dict.get(args_field)
-                    output_val = item_dict.get(output_field)
+                    output_val = item_dict.get(output_field) if output_field else None
+
+                    # Format output
+                    output = (
+                        f"Tool execution {item_dict.get('status', 'completed')}"
+                        if output_field is None
+                        else (str(output_val) if output_val else "No results found")
+                    )
 
                     tool_entry = {
                         "type": "tool_call",
                         "id": item_dict.get("id"),
                         "name": name,
                         "server_label": server_label,
                         "arguments": str(args_val) if args_val else None,
-                        "output": str(output_val) if output_val else "No results found",
+                        "output": output,
                         "error": item_dict.get("error"),
                         "status": "failed" if item_dict.get("error") else "completed",
                     }

backend/app/services/chat.py

@@ -332,6 +332,7 @@ def _handle_output_item_done(self, chunk: Dict[str, Any]):
             return
 
         # Tool call field mappings: (name_field, server_label_field, args_field, output_field)
+        # Note: web_search_call uses None for output_field because results aren't exposed in API
         tool_map = {
             "mcp_call": ("name", "server_label", "arguments", "output"),
             "function_call": ("name", "server_label", "arguments", "output"),
@@ -341,7 +342,7 @@ def _handle_output_item_done(self, chunk: Dict[str, Any]):
                 "queries",
                 "results",
             ),
-            "web_search_call": ("web_search", "llamastack", "query", "results"),
+            "web_search_call": ("web_search", "llamastack", "query", None),
         }
 
         name_field, server_field, args_field, output_field = tool_map[item_type]
@@ -359,15 +360,27 @@ def _handle_output_item_done(self, chunk: Dict[str, Any]):
         # Set final result
         is_standard = item_type in ("mcp_call", "function_call")
         args_val = item.get(args_field)
-        output_val = item.get(output_field)
+
+        # Get output value (None if output_field is None)
+        output_val = item.get(output_field) if output_field else None
+
+        # Format output based on tool type
+        if output_field is None:
+            # For tools like web_search that don't expose results
+            status = item.get("status", "completed")
+            output = f"Tool execution {status}"
+        elif output_val is not None:
+            # For tools that return results
+            if item_type == "file_search_call" and isinstance(output_val, list):
+                output = str(output_val) if output_val else "No results found"
+            else:
+                output = str(output_val)
+        else:
+            output = "No results found" if not is_standard else None
 
         tool_call.set_result(
             arguments=str(args_val) if args_val and not is_standard else args_val,
-            output=(
-                str(output_val)
-                if output_val
-                else ("No results found" if not is_standard else None)
-            ),
+            output=output,
             error=item.get("error"),
         )
 

deploy/cluster/helm/Chart.lock

@@ -10,7 +10,7 @@ dependencies:
   version: 0.5.4
 - name: llama-stack
   repository: https://rh-ai-quickstart.github.io/ai-architecture-charts
-  version: 0.6.0
+  version: 0.6.1
 - name: configure-pipeline
   repository: https://rh-ai-quickstart.github.io/ai-architecture-charts
   version: 0.5.2
@@ -20,5 +20,5 @@ dependencies:
 - name: oracle-db
   repository: https://rh-ai-quickstart.github.io/ai-architecture-charts
   version: 0.5.2
-digest: sha256:4ed3d491fef75d4e30bd7741ea9334a62575b9d1df8ac4b8ca7558386d85992f
-generated: "2025-11-12T17:02:54.047085877-05:00"
+digest: sha256:e59d70e7aeca1e5ef838fe8a81a54c76199d3072834597beed20ffb5673df4b6
+generated: "2025-11-13T13:13:28.912204567-05:00"

deploy/cluster/helm/Chart.yaml

@@ -34,7 +34,7 @@ dependencies:
     version: 0.5.4
     repository: https://rh-ai-quickstart.github.io/ai-architecture-charts
   - name: llama-stack
-    version: 0.6.0
+    version: 0.6.1
     repository: https://rh-ai-quickstart.github.io/ai-architecture-charts
   - name: configure-pipeline
     version: 0.5.2

frontend/src/components/chat.tsx

@@ -332,10 +332,8 @@ export function Chat({ preSelectedAgentId }: ChatProps = {}) {
     _event: React.MouseEvent<Element, MouseEvent> | undefined,
     value: string | number | undefined
   ) => {
-    console.log('onSelectAgent called with value:', value);
     if (value) {
       const agentId = value.toString();
-      console.log('Setting selected agent to:', agentId);
       setSelectedAgent(agentId);
       if (noAgentsWarning) setNoAgentsWarning(null);
     }

frontend/src/hooks/useChat.ts

@@ -211,7 +211,6 @@ export function useChat(agentId: string, options?: UseLlamaChatOptions) {
 
               try {
                 const chunk = JSON.parse(data) as StreamEvent;
-                // console.log('Received chunk:', chunk); // Disabled to prevent memory bloat
 
                 // Extract and set session ID if present
                 if (chunk.session_id && !sessionId) {