show content summary instead of argumentsText to display tool calls

Author: joaopluigi

lua/eca/sidebar.lua

@@ -1223,27 +1223,35 @@ function M:_handle_server_content(params)
     -- IMPORTANT: Return immediately - do NOT display anything for toolCallPrepare
     return
   elseif content.type == "toolCalled" then
+    local tool_text = ""
+
     -- Add diff to current tool call if present in toolCalled content
     if self._current_tool_call and content.details then
       self._current_tool_call.details = content.details
     end
 
     -- Show the final accumulated tool call if we have one
     if self._is_tool_call_streaming and self._current_tool_call then
-      self:_display_tool_call()
+      tool_text = self:_display_tool_call()
     end
 
     -- Show the tool result
-    local tool_text = string.format("✅ **Tool Result**: %s", content.name or "unknown")
+    local tool_log = string.format("**Tool Result**: %s", content.name or "unknown")
     if content.outputs and #content.outputs > 0 then
       for _, output in ipairs(content.outputs) do
         if output.type == "text" and output.content then
-          tool_text = tool_text .. "\n" .. output.content
+          tool_log = tool_log .. "\n" .. output.content
         end
       end
     end
-    self:_add_message("assistant", tool_text)
-    
+    Logger.debug(tool_log)
+
+    local tool_text_completed = "✅ " .. (content.summary or content.name or "Tool call completed")
+
+    if not self:_replace_message(tool_text, tool_text_completed) then
+      self:_add_message("assistant", tool_text_completed)
+    end
+
     -- Clean up tool call state
     self:_finalize_tool_call()
   end
@@ -1519,15 +1527,20 @@ function M:_handle_tool_call_prepare(content)
     self._is_tool_call_streaming = true
     self._current_tool_call = {
       name = "",
-      arguments = ""
+      arguments = "",
+      summary = ""
     }
   end
-  
+
   -- Accumulate tool call data
   if content.name then
     self._current_tool_call.name = content.name
   end
-  
+
+  if content.summary then
+    self._current_tool_call.summary = content.summary
+  end
+
   if content.argumentsText then
     self._current_tool_call.arguments = (self._current_tool_call.arguments or "") .. content.argumentsText
   end
@@ -1537,26 +1550,72 @@ function M:_handle_tool_call_prepare(content)
   end
 end
 
+---@return string tool text
 function M:_display_tool_call()
-  if not self._current_tool_call then return end
-  
-  local tool_text = string.format("🔧 **Tool Call**: %s", self._current_tool_call.name or "unknown")
-  
+  if not self._current_tool_call then return "" end
+
+  local tool_text = "🔧 " .. (self._current_tool_call.summary or "Tool call prepared")
+  local tool_log = string.format("**Tool Call**: %s", self._current_tool_call.name or "unknown")
+
   if self._current_tool_call.arguments and self._current_tool_call.arguments ~= "" then
-    tool_text = tool_text .. "\n```json\n" .. self._current_tool_call.arguments .. "\n```"
+    tool_log = tool_log .. "\n```json\n" .. self._current_tool_call.arguments .. "\n```"
   end
-  
+
 
   if self._current_tool_call.details and self._current_tool_call.details.diff then
-    tool_text = tool_text .. "\n\n**Diff**:\n```diff\n" .. self._current_tool_call.details.diff .. "\n```"
+    tool_log = tool_log .. "\n\n**Diff**:\n```diff\n" .. self._current_tool_call.details.diff .. "\n```"
   end
 
+  Logger.debug(tool_log)
   self:_add_message("assistant", tool_text)
+
+  return tool_text
 end
 
 function M:_finalize_tool_call()
   self._current_tool_call = nil
   self._is_tool_call_streaming = false
 end
 
+---@param target string
+---@param replacement string
+---@return boolean changed True if any replacement was made
+function M:_replace_message(target, replacement)
+  local chat = self.containers.chat
+
+  if not chat or not vim.api.nvim_buf_is_valid(chat.bufnr) then
+    Logger.warn("Cannot replace message: chat buffer not available")
+    return false
+  end
+
+  if not target or target == "" then
+    Logger.warn("Cannot replace message: empty target")
+    return false
+  end
+
+  if not replacement then
+    Logger.warn("Cannot replace message: empty replacement")
+    return false
+  end
+
+  local changed = false
+
+  self:_safe_buffer_update(chat.bufnr, function()
+    local lines = vim.api.nvim_buf_get_lines(chat.bufnr, 0, -1, false)
+
+    for i = #lines, 1, -1 do
+      local line = lines[i] or ""
+      local s_idx, e_idx = line:find(target, 1, true)
+      if s_idx then
+        local new_line = (line:sub(1, s_idx - 1)) .. replacement .. (line:sub(e_idx + 1))
+        vim.api.nvim_buf_set_lines(chat.bufnr, i - 1, i, false, { new_line })
+        changed = true
+        break
+      end
+    end
+  end)
+
+  return changed
+end
+
 return M