Don't pull this, I'm going to amend it

Author: tomgeorge

lua/eca/api.lua

@@ -1,4 +1,5 @@
 local Utils = require("eca.utils")
+local Logger = require("eca.logger")
 
 -- Load nui.nvim components for floating windows
 local Popup = require("nui.popup")
@@ -395,6 +396,48 @@ end
 local logs_popup = nil
 
 function M.show_logs()
+  -- Check if file logging is enabled
+  if Logger.is_file_logging_enabled() then
+    M._show_logs_in_buffer()
+  else
+    M._show_logs_in_popup()
+  end
+end
+
+--- Show logs in a regular Neovim buffer (when file logging is enabled)
+function M._show_logs_in_buffer()
+  local log_path = Logger.get_log_path()
+  
+  if not log_path then
+    Utils.warn("File logging is enabled but no log path is available")
+    return
+  end
+  
+  -- Check if log file exists
+  if vim.fn.filereadable(log_path) ~= 1 then
+    Utils.info("Log file does not exist yet: " .. log_path)
+    return
+  end
+  
+  -- Open the log file in a new buffer
+  vim.cmd("edit " .. vim.fn.fnameescape(log_path))
+  
+  -- Set buffer options for log viewing
+  vim.bo.readonly = true
+  vim.bo.modifiable = false
+  vim.bo.filetype = "log"
+  vim.wo.wrap = false
+  vim.wo.number = true
+  vim.wo.cursorline = true
+  
+  -- Jump to the end of the file to show latest logs
+  vim.cmd("normal! G")
+  
+  Utils.info("Opened ECA log file: " .. log_path)
+end
+
+--- Show logs in a popup (when file logging is disabled)
+function M._show_logs_in_popup()
   local eca = require("eca")
 
   if not eca.server then

lua/eca/commands.lua

@@ -2,6 +2,7 @@ local Utils = require("eca.utils")
 
 local M = {}
 
+--- Setup ECA commands
 function M.setup()
   -- Define ECA commands
   vim.api.nvim_create_user_command("EcaChat", function(opts)
@@ -43,7 +44,7 @@ function M.setup()
 
   vim.api.nvim_create_user_command("EcaAddSelection", function()
     -- Force exit visual mode and set marks
-    vim.cmd('normal! \\<Esc>')
+    vim.cmd("normal! \\<Esc>")
     vim.defer_fn(function()
       require("eca.api").add_selection_context()
     end, 50) -- Small delay to ensure marks are set
@@ -83,7 +84,7 @@ function M.setup()
   })
 
   -- ===== Selected Code Commands =====
-  
+
   vim.api.nvim_create_user_command("EcaShowSelection", function()
     require("eca.api").show_selected_code()
   end, {
@@ -97,7 +98,7 @@ function M.setup()
   })
 
   -- ===== TODOs Commands =====
-  
+
   vim.api.nvim_create_user_command("EcaAddTodo", function(opts)
     if opts.args and opts.args ~= "" then
       require("eca.api").add_todo(opts.args)
@@ -159,17 +160,17 @@ function M.setup()
     local eca = require("eca")
     local status = eca.server and eca.server:status() or "Not initialized"
     local status_icon = "○"
-    
+
     if status == "Running" then
       status_icon = "✓"
     elseif status == "Starting" then
       status_icon = "⋯"
     elseif status == "Failed" then
       status_icon = "✗"
     end
-    
+
     Utils.info("ECA Server Status: " .. status .. " " .. status_icon)
-    
+
     -- Also show path info if available
     if eca.server and eca.server._path_finder then
       local config = require("eca.config")
@@ -183,10 +184,48 @@ function M.setup()
     desc = "Show ECA server status with details",
   })
 
-  vim.api.nvim_create_user_command("EcaLogs", function()
-    require("eca.api").show_logs()
+  vim.api.nvim_create_user_command("EcaLogs", function(opts)
+    local Logger = require("eca.logger")
+    local Api = require("eca.api")
+    local subcommand = opts.args and opts.args:match("%S+") or "show"
+
+    if subcommand == "show" then
+      Api.show_logs()
+    elseif subcommand == "log_path" then
+      local log_path = Logger.get_log_path()
+      if log_path then
+        Utils.info("ECA log file: " .. log_path, { nofity_only = true })
+      else
+        Utils.info("File logging is disabled")
+      end
+    elseif subcommand == "clear" then
+      if Logger.clear_log() then
+        Utils.info("ECA log file cleared")
+      else
+        Utils.warn("Failed to clear log file (file logging may be disabled)")
+      end
+    elseif subcommand == "stats" then
+      local stats = Logger.get_log_stats()
+      if stats then
+        Utils.info(string.format("Log file: %s", stats.path))
+        Utils.info(string.format("Size: %.2fMB (%d bytes)", stats.size_mb, stats.size))
+        Utils.info(string.format("Last modified: %s", stats.modified))
+      else
+        Utils.info("No log file stats available (file logging may be disabled)")
+      end
+    else
+      Utils.warn("Unknown EcaLogs subcommand: " .. subcommand)
+      Utils.info("Available subcommands: show, log_path, clear, stats")
+    end
   end, {
-    desc = "Open ECA server logs in a new buffer",
+    desc = "ECA logging commands (show, log_path, clear, stats)",
+    nargs = "?",
+    complete = function(ArgLead, CmdLine, CursorPos)
+      local subcommands = { "show", "log_path", "clear", "stats" }
+      return vim.tbl_filter(function(cmd)
+        return cmd:match("^" .. ArgLead)
+      end, subcommands)
+    end,
   })
 
   vim.api.nvim_create_user_command("EcaSend", function(opts)
@@ -213,23 +252,23 @@ function M.setup()
   vim.api.nvim_create_user_command("EcaRedownload", function()
     local eca = require("eca")
     local Utils = require("eca.utils")
-    
+
     if eca.server and eca.server:is_running() then
       Utils.info("Stopping server before re-download...")
       eca.server:stop()
     end
-    
+
     -- Remove existing version file to force re-download
     local cache_dir = Utils.get_cache_dir()
     local version_file = cache_dir .. "/eca-version"
     os.remove(version_file)
-    
+
     -- Remove existing binary
     local server_binary = cache_dir .. "/eca"
     os.remove(server_binary)
-    
+
     Utils.info("Removed cached ECA server. Will re-download on next start.")
-    
+
     -- Restart server
     vim.defer_fn(function()
       if eca.server then
@@ -243,7 +282,7 @@ function M.setup()
   vim.api.nvim_create_user_command("EcaStopResponse", function()
     local eca = require("eca")
     local Utils = require("eca.utils")
-    
+
     -- Force stop any ongoing streaming response
     if eca.sidebar then
       eca.sidebar:_finalize_streaming_response()
@@ -257,7 +296,7 @@ function M.setup()
 
   vim.api.nvim_create_user_command("EcaFixTreesitter", function()
     local Utils = require("eca.utils")
-    
+
     -- Emergency treesitter fix for chat buffer
     vim.schedule(function()
       local eca = require("eca")
@@ -266,15 +305,15 @@ function M.setup()
         if bufnr and vim.api.nvim_buf_is_valid(bufnr) then
           -- Disable all highlighting for this buffer
           pcall(vim.api.nvim_set_option_value, "syntax", "off", { buf = bufnr })
-          
+
           -- Destroy treesitter highlighter if it exists
           pcall(function()
             if vim.treesitter.highlighter.active[bufnr] then
               vim.treesitter.highlighter.active[bufnr]:destroy()
               vim.treesitter.highlighter.active[bufnr] = nil
             end
           end)
-          
+
           Utils.info("Disabled treesitter highlighting for ECA chat buffer")
           Utils.info("Buffer " .. bufnr .. " highlighting disabled")
         else

lua/eca/config.lua

@@ -10,6 +10,11 @@ M._defaults = {
   server_args = "", -- Extra args for the eca start command
   ---@type string
   usage_string_format = "{messageCost} / {sessionCost}",
+  log = {
+    level = "info", -- Log level as string: "trace", "debug", "info", "warn", "error"
+    notify = true, -- Enable vim.notify logging
+    file = false, -- false = no file logging, true = default XDG path, string = custom path
+  },
   behaviour = {
     auto_set_keymaps = true,
     auto_focus_sidebar = true,

lua/eca/init.lua

@@ -208,6 +208,9 @@ function M.setup(opts)
 
   if M.did_setup then return end
 
+  -- Initialize logger with configuration
+  require("eca.logger").setup(Config.options)
+
   require("eca.highlights").setup()
   require("eca.commands").setup()