Remove client/server logging distinction

Author: tomgeorge

lua/eca/logger.lua

@@ -88,10 +88,7 @@ end
 --- Log a message to the log file
 ---@param message string
 ---@param level integer
----@param opts? {title?: string}
-function M.log(message, level, opts)
-  opts = opts or {}
-
+function M.log(message, level)
   if not M.config then
     return
   end
@@ -105,8 +102,7 @@ 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.server and "[SERVER]" or ""
-  local formatted = string.format("[%s] %-5s %s %s\n", timestamp, level_name, prefix, message)
+  local formatted = string.format("[%s] %-5s %s\n", timestamp, level_name, message)
 
   uv.fs_open(log_path, "a", 420, function(err, fd)
     if err or not fd then
@@ -120,49 +116,43 @@ end
 
 --- Log debug message
 ---@param message string
----@param opts? {title?: string}
-function M.debug(message, opts)
-  M.log(message, vim.log.levels.DEBUG, opts)
+function M.debug(message)
+  M.log(message, vim.log.levels.DEBUG)
 end
 
 --- Log info message
 ---@param message string
----@param opts? {title?: string}
-function M.info(message, opts)
-  M.log(message, vim.log.levels.INFO, opts)
+function M.info(message)
+  M.log(message, vim.log.levels.INFO)
 end
 
 --- Log warn message
 ---@param message string
----@param opts? {title?: string}
-function M.warn(message, opts)
-  M.log(message, vim.log.levels.WARN, opts)
+function M.warn(message)
+  M.log(message, vim.log.levels.WARN)
 end
 
 --- Log error message
 ---@param message string
----@param opts? {title?: string}
-function M.error(message, opts)
-  M.log(message, vim.log.levels.ERROR, opts)
+function M.error(message)
+  M.log(message, vim.log.levels.ERROR)
 end
 
 --- Send notification to user via vim.notify
 ---@param message string
 ---@param level? integer vim.log.levels (default: INFO)
----@param opts? {title?: string, once?: boolean}
+---@param opts? {title?: string}
 function M.notify(message, level, opts)
   level = level or vim.log.levels.INFO
   opts = opts or {}
 
-  M.log(message, level, opts)
+  M.log(message, level)
 
   vim.notify(message, level, {
     title = opts.title or "ECA",
-    once = opts.once,
   })
 end
 
-
 --- Get log file statistics
 ---@return table|nil
 function M.get_log_stats()

lua/eca/server.lua

@@ -103,7 +103,7 @@ function M:start()
       if data and self._rpc then
         local output = table.concat(data, "\n")
         if output and output ~= "" and output ~= "\n" then
-          Logger.log(output, vim.log.levels.INFO, { server = true })
+          Logger.log(output, vim.log.levels.INFO)
           self._rpc:_handle_stdout(output)
         end
       end
@@ -121,7 +121,7 @@ function M:start()
 
         if #meaningful_lines > 0 then
           local error_output = table.concat(meaningful_lines, "\n")
-          Logger.log(error_output, vim.log.levels.WARN, { server = true })
+          Logger.log(error_output, vim.log.levels.WARN)
         end
       end
     end,

tests/test_logging.lua

@@ -378,22 +378,6 @@ T["integration"]["logger functions work correctly"] = function()
   expect_match(contents, "error message")
 end
 
-T["integration"]["server logging with prefix"] = function()
-  local contents = log_and_read(
-    "server_integration.log",
-    [[
-    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 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()
   setup_logger_with_file("eca_logs_test.log")