fix: properly schedule vim.notify and nvim_echo calls

Author: fkalny-groupon

lua/claudecode/logger.lua

@@ -68,18 +68,18 @@ local function log(level, component, message_parts)
     end
   end
 
-  if level == M.levels.ERROR then
-    vim.notify(prefix .. " " .. message, vim.log.levels.ERROR, { title = "ClaudeCode Error" })
-  elseif level == M.levels.WARN then
-    vim.notify(prefix .. " " .. message, vim.log.levels.WARN, { title = "ClaudeCode Warning" })
-  else
-    -- For INFO, DEBUG, TRACE, use nvim_echo to avoid flooding notifications,
-    -- to make them appear in :messages, and wrap in vim.schedule
-    -- to avoid "nvim_echo must not be called in a fast event context".
-    vim.schedule(function()
+  -- Wrap all vim.notify and nvim_echo calls in vim.schedule to avoid
+  -- "nvim_echo must not be called in a fast event context" errors
+  vim.schedule(function()
+    if level == M.levels.ERROR then
+      vim.notify(prefix .. " " .. message, vim.log.levels.ERROR, { title = "ClaudeCode Error" })
+    elseif level == M.levels.WARN then
+      vim.notify(prefix .. " " .. message, vim.log.levels.WARN, { title = "ClaudeCode Warning" })
+    else
+      -- For INFO, DEBUG, TRACE, use nvim_echo to avoid flooding notifications
       vim.api.nvim_echo({ { prefix .. " " .. message, "Normal" } }, true, {})
-    end)
-  end
+    end
+  end)
 end
 
 --- @param component string|nil Optional component/module name.
@@ -147,3 +147,4 @@ local default_config_for_initial_setup = require("claudecode.config").defaults
 M.setup(default_config_for_initial_setup)
 
 return M
+