refactor(ui): expose find_message_in_state and avoid circular dependency

Export find_message_in_state as M.find_message_in_state in lua/opencode/ui/renderer.lua and update events.lua to call it via a lazy require through a new local helper find_message_by_id. This removes duplicated lookup logic and prevents the circular dependency between renderer.lua and events.lua without changing runtime behavior.

Author: sudo-tee

lua/opencode/ui/renderer.lua

@@ -107,7 +107,7 @@ end
 
 ---@param message_id string
 ---@return OpencodeMessage|nil
-local function find_message_in_state(message_id)
+M.find_message_in_state = function(message_id)
   for _, message in ipairs(state.messages or {}) do
     if message.info and message.info.id == message_id then
       return message
@@ -137,7 +137,7 @@ end
 ---@param message_id string
 local function hide_rendered_message(message_id)
   local rendered_message = ctx.render_state:get_message(message_id)
-  local message = rendered_message and rendered_message.message or find_message_in_state(message_id)
+  local message = rendered_message and rendered_message.message or M.find_message_in_state(message_id)
   if not message then
     return
   end

lua/opencode/ui/renderer/events.lua

@@ -33,23 +33,17 @@ local function find_text_part_for_message(message)
   return nil
 end
 
+-- Lazy require to avoid circular dependency: renderer.lua <-> events.lua
 ---@param message_id string|nil
 ---@return OpencodeMessage|nil
-local function find_message_in_state(message_id)
+local function find_message_by_id(message_id)
   if not message_id then
     return nil
   end
 
-  for _, message in ipairs(state.messages or {}) do
-    if message.info and message.info.id == message_id then
-      return message
-    end
-  end
-
-  return nil
+  return require('opencode.ui.renderer').find_message_in_state(message_id)
 end
 
--- Lazy require to avoid circular dependency: renderer.lua <-> events.lua
 ---@param force? boolean
 local function scroll(force)
   require('opencode.ui.renderer').scroll_to_bottom(force)
@@ -187,7 +181,7 @@ function M.on_message_updated(message, revert_index)
   end
 
   local rendered_message = ctx.render_state:get_message(msg.info.id)
-  local found_msg = rendered_message and rendered_message.message or find_message_in_state(msg.info.id)
+  local found_msg = rendered_message and rendered_message.message or find_message_by_id(msg.info.id)
 
   if revert_index then
     if not found_msg then
@@ -250,7 +244,7 @@ function M.on_message_removed(properties)
   end
 
   local rendered_message = ctx.render_state:get_message(message_id)
-  local message = rendered_message and rendered_message.message or find_message_in_state(message_id)
+  local message = rendered_message and rendered_message.message or find_message_by_id(message_id)
   ctx.render_state:clear_orphan_parts(message_id)
   if not message then
     return
@@ -303,7 +297,7 @@ function M.on_part_updated(properties, revert_index)
 
   local rendered_message = ctx.render_state:get_message(part.messageID)
   if not rendered_message then
-    local existing_message = find_message_in_state(part.messageID)
+    local existing_message = find_message_by_id(part.messageID)
     if existing_message then
       ctx.render_state:set_message(existing_message)
       rendered_message = ctx.render_state:get_message(part.messageID)