fix(event_manager): log errors in callbacks

We were swallowing some errors (e.g. vim.filetype.match). None of them
seemed important but it's possible there will be errors in the future
that we need to surface.

Added util.pcall_trace that will return a full stacktrace in result,
rather than just the error.

Author: cameronr

lua/opencode/event_manager.lua

@@ -1,6 +1,7 @@
 local state = require('opencode.state')
 local config = require('opencode.config')
 local ThrottlingEmitter = require('opencode.throttling_emitter')
+local util = require('opencode.util')
 
 --- @class EventInstallationUpdated
 --- @field type "installation.updated"
@@ -278,12 +279,16 @@ function EventManager:emit(event_name, data)
 
   local event = { type = event_name, properties = data }
 
-  if require('opencode.config').debug.capture_streamed_events then
+  if config.debug.capture_streamed_events then
     table.insert(self.captured_events, vim.deepcopy(event))
   end
 
   for _, callback in ipairs(listeners) do
-    pcall(callback, data)
+    local ok, result = util.pcall_trace(callback, data)
+
+    if not ok then
+      vim.notify('Error calling ' .. event_name .. ' listener: ' .. result, vim.log.levels.ERROR)
+    end
   end
 end
 

lua/opencode/types.lua

@@ -125,6 +125,7 @@
 
 ---@class OpencodeDebugConfig
 ---@field enabled boolean
+---@field capture_streamed_events any[]
 
 --- @class OpencodeProviders
 --- @field [string] string[]

lua/opencode/util.lua

@@ -198,10 +198,13 @@ function M.format_time(timestamp)
   if timestamp > 1e12 then
     timestamp = math.floor(timestamp / 1000)
   end
-  
+
   local now = os.time()
-  local today_start = os.time(os.date('*t', now)) - os.date('*t', now).hour * 3600 - os.date('*t', now).min * 60 - os.date('*t', now).sec
-  
+  local today_start = os.time(os.date('*t', now))
+    - os.date('*t', now).hour * 3600
+    - os.date('*t', now).min * 60
+    - os.date('*t', now).sec
+
   if timestamp >= today_start then
     return os.date('%I:%M %p', timestamp)
   else
@@ -416,6 +419,10 @@ end
 --- @param filename string filename, possibly including path
 --- @return string markdown_filetype
 function M.get_markdown_filetype(filename)
+  if not filename or filename == '' then
+    return ''
+  end
+
   local file_type_overrides = {
     javascriptreact = 'jsx',
     typescriptreact = 'tsx',
@@ -476,4 +483,11 @@ function M.parse_run_args(args)
   return opts, prompt
 end
 
+---pcall but returns a full stacktrace on error
+function M.pcall_trace(fn, ...)
+  return xpcall(fn, function(err)
+    return debug.traceback(err, 2)
+  end, ...)
+end
+
 return M