Hook into the server on_stdout and on_stderr to log server logs.

This is still a little clunky but I think it is valuable to get the server logs as well as client-side logs

Author: tomgeorge

lua/eca/logger.lua

@@ -105,8 +105,8 @@ function M.log(message, level, opts)
 
   local timestamp = os.date("%Y-%m-%d %H:%M:%S")
   local level_name = get_level_name(level)
-  local prefix = opts.title == "SERVER" and "SERVER" or "CLIENT"
-  local formatted = string.format("[%s] %-5s [%s] %s\n", timestamp, level_name, prefix, message)
+  local prefix = opts.server and "[SERVER]" or ""
+  local formatted = string.format("[%s] %-5s %s %s\n", timestamp, level_name, prefix, message)
 
   uv.fs_open(log_path, "a", 420, function(err, fd)
     if err or not fd then
@@ -162,6 +162,7 @@ function M.notify(message, level, opts)
   })
 end
 
+
 --- Get log file statistics
 ---@return table|nil
 function M.get_log_stats()

lua/eca/server.lua

@@ -100,11 +100,10 @@ function M:start()
   -- Use vim.fn.jobstart for interactive processes
   local job_opts = {
     on_stdout = function(_, data, _)
-      Logger.debug("Server stdout received: " .. vim.inspect(data))
       if data and self._rpc then
         local output = table.concat(data, "\n")
         if output and output ~= "" and output ~= "\n" then
-          Logger.debug("Processing stdout: " .. output)
+          Logger.log(output, vim.log.levels.INFO, { server = true })
           self._rpc:_handle_stdout(output)
         end
       end
@@ -122,8 +121,7 @@ function M:start()
 
         if #meaningful_lines > 0 then
           local error_output = table.concat(meaningful_lines, "\n")
-          Logger.debug("ECA server stderr: " .. error_output)
-          -- Capture logs for the logs buffer
+          Logger.log(error_output, vim.log.levels.WARN, { server = true })
         end
       end
     end,

tests/test_logging.lua

@@ -228,7 +228,6 @@ T["file_logging"]["writes to file with correct format"] = function()
 
   expect_match(contents, "%[%d%d%d%d%-%d%d%-%d%d %d%d:%d%d:%d%d%]")
   expect_match(contents, "INFO")
-  expect_match(contents, "%[CLIENT%]")
   expect_match(contents, "test message")
 end
 
@@ -303,7 +302,6 @@ T["notifications"]["Logger.notify writes to log file"] = function()
 
   expect_match(contents, "WARN")
   expect_match(contents, "test notification message")
-  expect_match(contents, "%[CLIENT%]")
 
   local notifications = child.lua_get("_G.captured_notifications")
   eq(#notifications, 1)
@@ -380,22 +378,20 @@ T["integration"]["logger functions work correctly"] = function()
   expect_match(contents, "error message")
 end
 
-T["integration"]["server logging with SERVER prefix"] = function()
+T["integration"]["server logging with prefix"] = function()
   local contents = log_and_read(
     "server_integration.log",
     [[
-    Logger.info('server message 1', { title = 'SERVER' })
-    Logger.warn('server warning', { title = 'SERVER' })
-    Logger.error('server error', { title = 'SERVER' })
-
+    Logger.log('server stdout message', vim.log.levels.INFO, { server = true })
+    Logger.log('server stderr message', vim.log.levels.WARN, { server = true })
     Logger.info('client message')
   ]]
   )
 
-  expect_match(contents, "%[SERVER%] server message 1")
-  expect_match(contents, "%[SERVER%] server warning")
-  expect_match(contents, "%[SERVER%] server error")
-  expect_match(contents, "%[CLIENT%] client message")
+  expect_match(contents, "%[SERVER%] server stdout message")
+  expect_match(contents, "%[SERVER%] server stderr message") 
+  expect_match(contents, "client message")
+  expect_no_match(contents, "%[SERVER%] client message")
 end
 
 T["integration"]["EcaLogs command behavior"] = function()