refactor: Move output_at_bottom boolean from state to output_window

Author: guillaumeboehm

lua/opencode/state.lua

@@ -14,7 +14,6 @@
 ---@field last_input_window_position integer[]|nil
 ---@field last_output_window_position integer[]|nil
 ---@field last_code_win_before_opencode integer|nil
----@field output_window_at_bottom boolean
 ---@field current_code_buf number|nil
 ---@field display_route any|nil
 ---@field current_mode string
@@ -56,7 +55,6 @@ local _state = {
   last_focused_opencode_window = nil,
   last_input_window_position = nil,
   last_output_window_position = nil,
-  output_window_at_bottom = true,
   last_code_win_before_opencode = nil,
   current_code_buf = nil,
   display_route = nil,

lua/opencode/ui/output_window.lua

@@ -3,6 +3,7 @@ local config = require('opencode.config')
 
 local M = {}
 M.namespace = vim.api.nvim_create_namespace('opencode_output')
+M.viewport_at_bottom = true
 
 function M.create_buf()
   local output_buf = vim.api.nvim_create_buf(false, true)
@@ -215,7 +216,7 @@ function M.setup_autocmds(windows, group)
     buffer = windows.output_buf,
     callback = function()
       -- Update state to track if user is at bottom
-      state.output_window_at_bottom = M.is_at_bottom(windows.output_win)
+      M.viewport_at_bottom = M.is_at_bottom(windows.output_win)
     end,
   })
 end
@@ -225,6 +226,7 @@ function M.clear()
   -- clear extmarks in all namespaces as I've seen RenderMarkdown leave some
   -- extmarks behind
   M.clear_extmarks(0, -1, true)
+  M.viewport_at_bottom = true
 end
 
 return M

lua/opencode/ui/renderer.lua

@@ -51,9 +51,6 @@ function M.reset()
   end
   state.current_permission = nil
 
-  -- Reset scroll state when session changes
-  state.output_window_at_bottom = true
-
   trigger_on_data_rendered()
 end
 
@@ -242,16 +239,16 @@ function M.scroll_to_bottom()
   if prev_line_count == 0 then
     should_scroll = true
   -- Scroll if user is at bottom (respects manual scroll position)
-  elseif state.output_window_at_bottom then
+  elseif output_window.viewport_at_bottom then
     should_scroll = true
   end
 
   if should_scroll then
     vim.api.nvim_win_set_cursor(state.windows.output_win, { line_count, 0 })
-    state.output_window_at_bottom = true
+    output_window.viewport_at_bottom = true
   else
     -- User has scrolled up, don't scroll
-    state.output_window_at_bottom = false
+    output_window.viewport_at_bottom = false
   end
 end