refactor: add render_state WIP

Building on MessageMap idea, encapsulate the rendered state into
render_state, along with easy look ups

Author: cameronr

lua/opencode/server_job.lua

@@ -17,6 +17,20 @@ local function handle_api_response(response, cb)
   end
 end
 
+M.requests = {}
+
+function M.get_unresolved_requests()
+  local unresolved = {}
+
+  for _, data in ipairs(M.requests) do
+    if data[2]._resolved ~= true then
+      table.insert(unresolved, data)
+    end
+  end
+
+  return unresolved
+end
+
 --- Make an HTTP API call to the opencode server.
 --- @generic T
 --- @param url string The API endpoint URL
@@ -61,6 +75,9 @@ function M.call_api(url, method, body)
     opts.body = body and vim.json.encode(body) or '{}'
   end
 
+  -- FIXME: remove tracking code when thinking bug is fixed
+  table.insert(M.requests, { opts, call_promise })
+
   curl.request(opts)
   return call_promise
 end

lua/opencode/ui/debug_helper.lua

@@ -26,10 +26,14 @@ function M.debug_output()
 end
 
 function M.debug_message()
-  local session_formatter = require('opencode.ui.formatter')
+  local renderer = require('opencode.ui.renderer')
   local current_line = vim.api.nvim_win_get_cursor(state.windows.output_win)[1]
-  local metadata = session_formatter.get_message_at_line(current_line) or {}
-  M.open_json_file(metadata.message)
+  local message_data = renderer._render_state:get_message_at_line(current_line)
+  if message_data and message_data.message then
+    M.open_json_file(message_data.message)
+  else
+    vim.notify('No message found at current line', vim.log.levels.WARN)
+  end
 end
 
 function M.debug_session()

lua/opencode/ui/formatter.lua

@@ -53,24 +53,9 @@ end
 ---@param line number Buffer line number
 ---@param output Output Output object to query
 ---@return {message: MessageInfo, part: MessagePart, msg_idx: number, part_idx: number}|nil
+---@deprecated Use RenderState:get_message_at_line() instead
 function M.get_message_at_line(line, output)
-  local metadata = output:get_nearest_metadata(line)
-  if metadata and metadata.msg_idx and metadata.part_idx then
-    local msg = state.messages and state.messages[metadata.msg_idx]
-    if not msg or not msg.parts then
-      return nil
-    end
-    local part = msg.parts[metadata.part_idx]
-    if not part then
-      return nil
-    end
-    return {
-      message = msg,
-      part = part,
-      msg_idx = metadata.msg_idx,
-      part_idx = metadata.part_idx,
-    }
-  end
+  return nil
 end
 
 ---Calculate statistics for reverted messages and tool calls
@@ -667,22 +652,10 @@ end
 ---Formats a single message part and returns the resulting output object
 ---@param part MessagePart The part to format
 ---@param role 'user'|'assistant'|'system' The role, user or assistant, that created this part
----@param msg_idx integer The index of the message in state.messages
----@param part_idx integer The index of the part in state.messages[msg_idx].parts
 ---@return Output
-function M.format_part(part, role, msg_idx, part_idx)
+function M.format_part(part, role)
   local output = Output.new()
 
-  -- FIXME: do we need metadata? it looks like maybe only for snapshots?
-  local metadata = {
-    msg_idx = msg_idx,
-    part_idx = part_idx,
-    role = role,
-    type = part.type,
-    snapshot = part.snapshot,
-  }
-  output:add_metadata(metadata)
-
   local content_added = false
 
   if role == 'user' then