fix: cleanup nui Split autocmds on toggle and encapsulate EcaChatClear logic

Author: tamercuba

lua/eca/commands.lua

@@ -550,12 +550,7 @@ function M.setup()
   vim.api.nvim_create_user_command("EcaChatClear", function()
     local sidebar = require("eca").get()
     if sidebar then
-      sidebar._welcome_message_applied = true
-      sidebar._force_welcome = false
-      local chat = sidebar.containers and sidebar.containers.chat
-      if chat and chat.bufnr and vim.api.nvim_buf_is_valid(chat.bufnr) then
-        vim.api.nvim_buf_set_lines(chat.bufnr, 0, -1, false, {})
-      end
+      sidebar:clear_chat()
     end
   end, {
     desc = "Clear ECA chat buffer",

lua/eca/sidebar.lua

@@ -253,6 +253,15 @@ function M:reset()
   end
 end
 
+function M:clear_chat()
+  local chat = self.containers and self.containers.chat
+  if chat and chat.bufnr and vim.api.nvim_buf_is_valid(chat.bufnr) then
+    self._welcome_message_applied = true
+    self._force_welcome = false
+    vim.api.nvim_buf_set_lines(chat.bufnr, 0, -1, false, {})
+  end
+end
+
 function M:new_chat()
   self:reset()
   self._force_welcome = true
@@ -334,6 +343,14 @@ function M:_create_containers()
       and self.containers.chat.bufnr
     or nil
 
+  -- Clean up the old chat Split's autocmds before creating a new one.
+  -- Detach the buffer first so that unmount() does not delete it.
+  if existing_chat_bufnr then
+    local old_chat = self.containers.chat
+    old_chat.bufnr = nil
+    pcall(old_chat.unmount, old_chat)
+  end
+
   --  Create and mount main chat container first
   self.containers.chat = Split({
     relative = "editor",