fix(tools): additional resiliency (#1987)

Co-authored-by: Oli Morris <olimorris@users.noreply.github.com>

Author: olimorris

lua/codecompanion/strategies/chat/tools/init.lua

@@ -130,10 +130,6 @@ function Tools:execute(chat, tools)
     ---@param orchestrator CodeCompanion.Tools.Orchestrator The orchestrator instance
     ---@param tool table The tool to run
     local function enqueue_tool(orchestrator, tool)
-      if self.status == CONSTANTS.STATUS_ERROR then
-        return
-      end
-
       local name = tool["function"].name
       local tool_config = self.tools_config[name]
       local function handle_missing_tool(tool_call, err_message)

lua/codecompanion/strategies/chat/tools/orchestrator.lua

@@ -339,10 +339,11 @@ end
 ---Close the execution of the tool
 ---@return nil
 function Orchestrator:close()
-  --TODO: This is a workaround that avoids the close method being called more than once
   if self.tool then
     log:debug("Orchestrator:close")
-    self.handlers.on_exit()
+    pcall(function()
+      self.handlers.on_exit()
+    end)
     utils.fire("ToolFinished", { id = self.id, name = self.tool.name, bufnr = self.tools.bufnr })
     self.tool = nil
   end