refactor(ui): route output window gg through keymap config

Replace hardcoded gg handling in output_window.setup_keymaps with a config-driven first_message command. gg now loads full history and jumps to line 1 while preserving custom or window-specific mappings, and can be disabled with output_window.gg = false.

Author: sudo-tee

lua/opencode/commands/handlers/workflow.lua

@@ -388,6 +388,10 @@ M.actions.run_user_command = Promise.async(function(name, args)
   end) --[[@as Promise<void> ]]
 end)
 
+function M.actions.first_message()
+  require('opencode.ui.navigation').goto_first_message()
+end
+
 function M.actions.next_message()
   require('opencode.ui.navigation').goto_next_message()
 end
@@ -603,6 +607,10 @@ M.command_defs = {
     desc = 'Open context items picker in input window',
     execute = M.actions.context_items,
   },
+  first_message = {
+    desc = 'Load history and go to the first message',
+    execute = M.actions.first_message,
+  },
   next_message = {
     desc = 'Navigate to next message in output window',
     execute = M.actions.next_message,

lua/opencode/config.lua

@@ -82,6 +82,7 @@ M.defaults = {
 
     },
     output_window = {
+      ['gg'] =         { 'first_message',                                     desc = 'Load history and go to the first message' },
       ['<esc>'] =       { 'close',                                             desc = 'Close Opencode windows' },
       ['<C-c>'] =       { 'cancel',                                            desc = 'Cancel running request' },
       [']]']   =        { 'next_message',                                      desc = 'Go to next message' },

lua/opencode/ui/navigation.lua

@@ -11,6 +11,12 @@ local function mark_jump_position(win)
   end)
 end
 
+function M.goto_first_message()
+  renderer.load_all_messages()
+  mark_jump_position(vim.api.nvim_get_current_win())
+  vim.api.nvim_win_set_cursor(0, { 1, 0 })
+end
+
 function M.goto_message_by_id(message_id)
   require('opencode.ui.ui').focus_output()
   local windows = state.windows or {}
@@ -84,7 +90,7 @@ function M.goto_next_user_message()
     return
   end
 
-  -- Mirror `gg` in output_window.setup_keymaps: under lazy render the target
+  -- Like `gg`, under lazy render the target
   -- message may not yet have a line_start, so force a full render first.
   renderer.load_all_messages()
 

lua/opencode/ui/output_window.lua

@@ -707,30 +707,6 @@ function M.close()
   pcall(vim.api.nvim_buf_delete, state.windows.output_buf, { force = true })
 end
 
----@param windows OpencodeWindowState
----@param preserve_existing? boolean
-function M.setup_keymaps(windows, preserve_existing)
-  -- When lazy-render is active, gg only reaches the top of rendered content.
-  -- Load all messages first so gg reaches the true start of history.
-  local has_gg = false
-  if preserve_existing then
-    for _, mapping in ipairs(vim.api.nvim_buf_get_keymap(windows.output_buf, 'n')) do
-      if mapping.lhs == 'gg' then
-        has_gg = true
-        break
-      end
-    end
-  end
-  if not has_gg then
-    vim.keymap.set('n', 'gg', function()
-      local renderer = require('opencode.ui.renderer')
-      renderer.load_all_messages()
-      pcall(vim.cmd, [[noau normal! m']])
-      vim.api.nvim_win_set_cursor(0, { 1, 0 })
-    end, { buffer = windows.output_buf })
-  end
-end
-
 ---Clear the output buffer and all namespaces.
 function M.clear()
   if M.mounted() then
@@ -749,10 +725,4 @@ function M.get_buf()
   return state.windows and state.windows.output_buf
 end
 
----Trigger a re-render by calling the renderer
-function M.render()
-  local renderer = require('opencode.ui.renderer')
-  renderer._render_all_messages()
-end
-
 return M

lua/opencode/ui/ui.lua

@@ -276,7 +276,6 @@ function M.restore_hidden_windows()
 
   input_window.setup(windows)
   output_window.setup(windows)
-  output_window.setup_keymaps(windows, true)
   footer.setup(windows)
   session_tab_strip.setup(windows)
   topbar.setup()
@@ -486,7 +485,6 @@ function M.create_windows()
 
   input_window.setup(windows)
   output_window.setup(windows)
-  output_window.setup_keymaps(windows)
   footer.setup(windows)
   session_tab_strip.setup(windows)
   topbar.setup()

tests/unit/keymap_spec.lua

@@ -1,5 +1,6 @@
 local assert = require('luassert')
 local store = require('opencode.state.store')
+local default_gg = require('opencode.config').defaults.keymap.output_window.gg
 
 describe('opencode.keymap', function()
   local set_keymaps = {}
@@ -65,6 +66,7 @@ describe('opencode.keymap', function()
     mock_commands = {
       get_commands = function()
         return {
+          first_message = { desc = 'Load history and go to the first message', execute = function() end },
           open_input = { desc = 'Open input window', execute = function() end },
           toggle = { desc = 'Toggle opencode windows', execute = function() end },
           submit_input_prompt = { desc = 'Submit input prompt', execute = function() end },
@@ -155,6 +157,40 @@ describe('opencode.keymap', function()
       end
     end
 
+    it('installs the default gg action and restores it without replacing a custom mapping', function()
+      vim.keymap.set = original_keymap_set
+      keymap.setup({ output_window = { gg = default_gg } })
+      local windows = panel()
+      store.set('windows', windows)
+      assert.is_true(vim.wait(200, function()
+        return mapping(windows.output_buf, 'gg') ~= nil
+      end))
+      mapping(windows.output_buf, 'gg').callback()
+      assert.equals('first_message', executed_parsed[1].intent.name)
+
+      vim.keymap.del('n', 'gg', { buffer = windows.output_buf })
+      store.set('windows', nil)
+      store.set('windows', windows)
+      assert.is_true(vim.wait(200, function()
+        return mapping(windows.output_buf, 'gg') ~= nil
+      end))
+      original_keymap_set('n', 'gg', function() end, { buffer = windows.output_buf, desc = 'Custom gg' })
+      store.set('windows', nil)
+      store.set('windows', windows)
+      local drained = false
+      vim.schedule(function() drained = true end)
+      assert.is_true(vim.wait(200, function() return drained end))
+      assert.equals('Custom gg', mapping(windows.output_buf, 'gg').desc)
+    end)
+
+    it('does not install gg when disabled', function()
+      vim.keymap.set = original_keymap_set
+      local windows = panel()
+      store.set_raw('windows', windows)
+      keymap.setup({ output_window = { gg = false } })
+      assert.is_nil(mapping(windows.output_buf, 'gg'))
+    end)
+
     it('binds new panels and restores missing mappings without replacing custom or window mappings', function()
       vim.keymap.set = original_keymap_set
       keymap.setup({

tests/unit/navigation_user_message_spec.lua

@@ -57,6 +57,22 @@ describe('navigation user message jumps', function()
     end
   end)
 
+  it('loads history before jumping to the first line', function()
+    vim.api.nvim_win_set_cursor(output_win, { 100, 0 })
+    local load_history = stub(renderer, 'load_all_messages').invokes(function()
+      assert.equals(100, vim.api.nvim_win_get_cursor(output_win)[1])
+    end)
+    local ok, err = pcall(function()
+      navigation.goto_first_message()
+      assert.stub(load_history).was_called(1)
+      assert.same({ 1, 0 }, vim.api.nvim_win_get_cursor(output_win))
+    end)
+    load_history:revert()
+    if not ok then
+      error(err)
+    end
+  end)
+
   describe('renderer.get_prev_user_message', function()
     it('skips assistant messages and returns previous user message before cursor', function()
       seed({