refactor: clean up PR sudo-tee#42 based on maintainer feedback

- Remove unnecessary inline type annotations in session_formatter.lua
- Remove deprecated assistant_mode field from Message type
- Simplify mode field documentation
- Remove assistant_mode fallback logic in header formatting
- Restore accidentally deleted keymap and context config fields
- Fix indentation inconsistencies

This addresses all review feedback from @sudo-tee while preserving
the core feature: using CLI-persisted mode field to display agent
names (BUILD, PLAN, etc.) in assistant message headers.

Author: Danilo Verde Ribeiro

lua/opencode/types.lua

@@ -57,6 +57,11 @@
 ---@field diff_close string
 ---@field diff_revert_all_last_prompt string
 ---@field diff_revert_this_last_prompt string
+---@field diff_revert_all string
+---@field diff_revert_this string
+---@field diff_restore_snapshot_file string
+---@field diff_restore_snapshot_all string
+---@field open_configuration_file string
 ---@field swap_position string # Swap Opencode pane left/right
 
 ---@class OpencodeKeymapWindow
@@ -72,12 +77,9 @@
 ---@field toggle_pane string
 ---@field prev_prompt_history string
 ---@field next_prompt_history string
----@field focus_input string
----@field debug_message string
----@field debug_session string
----@field debug_output string
 ---@field switch_mode string
----@field select_child_session string
+---@field focus_input string
+---@field select_child_session string\n---@field debug_message string\n---@field debug_output string\n---@field debug_session string
 ---@class OpencodeKeymap
 ---@field global OpencodeKeymapGlobal
 ---@field window OpencodeKeymapWindow
@@ -109,17 +111,52 @@
 
 ---@class OpencodeContextConfig
 ---@field enabled boolean
+---@field plugin_versions { enabled: boolean, limit: number }
 ---@field cursor_data { enabled: boolean }
 ---@field diagnostics { info: boolean, warning: boolean, error: boolean }
----@field current_file { enabled: boolean }
+---@field current_file { enabled: boolean, show_full_path: boolean }
 ---@field selection { enabled: boolean }
+---@field marks { enabled: boolean, limit: number }
+---@field jumplist { enabled: boolean, limit: number }
+---@field recent_buffers { enabled: boolean, limit: number, symbols_only: boolean }
+---@field undo_history { enabled: boolean, limit: number }
+---@field windows_tabs { enabled: boolean }
+---@field highlights { enabled: boolean }
+---@field session_info { enabled: boolean }
+---@field registers { enabled: boolean, include: string[] }
+---@field command_history { enabled: boolean, limit: number }
+---@field search_history { enabled: boolean, limit: number }
+---@field debug_data { enabled: boolean }
+---@field lsp_context { enabled: boolean, diagnostics_limit: number, code_actions: boolean }
+---@field git_info { enabled: boolean, diff_limit: number, changes_limit: number }
+---@field fold_info { enabled: boolean }
+---@field cursor_surrounding { enabled: boolean, lines_above: number, lines_below: number }
+---@field quickfix_loclist { enabled: boolean, limit: number }
+---@field macros { enabled: boolean, register: string }
+---@field terminal_buffers { enabled: boolean }
+---@field session_duration { enabled: boolean }
 
 ---@class OpencodeDebugConfig
 ---@field enabled boolean
 
 --- @class OpencodeProviders
 --- @field [string] string[]
 
+---@class OpencodeConfigModule
+---@field defaults OpencodeConfig
+---@field values OpencodeConfig
+---@field setup fun(opts?: OpencodeConfig): nil
+---@overload fun(key: nil): OpencodeConfig
+---@overload fun(key: "preferred_picker"): 'mini.pick' | 'telescope' | 'fzf' | 'snacks' | nil
+---@overload fun(key: "preferred_completion"): 'blink' | 'nvim-cmp' | 'vim_complete' | nil
+---@overload fun(key: "default_mode"): 'build' | 'plan'
+---@overload fun(key: "default_global_keymaps"): boolean
+---@overload fun(key: "keymap"): OpencodeKeymap
+---@overload fun(key: "ui"): OpencodeUIConfig
+---@overload fun(key: "providers"): OpencodeProviders
+---@overload fun(key: "context"): OpencodeContextConfig
+---@overload fun(key: "debug"): OpencodeDebugConfig
+
 ---@class OpencodeConfig
 ---@field preferred_picker 'telescope' | 'fzf' | 'mini.pick' | 'snacks' | nil
 ---@field preferred_completion 'blink' | 'nvim-cmp' | 'vim_complete' | nil -- Preferred completion strategy for mentons and commands
@@ -256,8 +293,7 @@
 ---@field providerID string Provider identifier
 ---@field role 'user'|'assistant'|'system' Role of the message sender
 ---@field system_role string|nil Role defined in system messages
----@field mode string|nil Agent/mode used to create this message (from CLI)
----@field assistant_mode string|nil Assistant mode active when message was created (deprecated)
+---@field mode string|nil Agent or mode identifier
 ---@field error table
 
 ---@class RestorePoint

lua/opencode/ui/session_formatter.lua

@@ -192,10 +192,10 @@ function M._format_revert_message(stats)
       end
       if #file_diff > 0 then
         local line_str = string.format(icons.get('file') .. '%s: %s', file, table.concat(file_diff, ' '))
-        local line_idx = M.output:add_line(line_str) ---@type number
-        local col = #('  ' .. file .. ': ') ---@type number
+        local line_idx = M.output:add_line(line_str)
+        local col = #('  ' .. file .. ': ')
         for _, diff in ipairs(file_diff) do
-           local hl_group = diff:sub(1, 1) == '+' and 'OpencodeDiffAddText' or 'OpencodeDiffDeleteText' ---@type string
+          local hl_group = diff:sub(1, 1) == '+' and 'OpencodeDiffAddText' or 'OpencodeDiffDeleteText'
           M.output:add_extmark(line_idx, {
             virt_text = { { diff, hl_group } },
             virt_text_pos = 'inline',
@@ -251,7 +251,7 @@ function M._format_patch(part)
           util.time_ago(restore_point.created_at)
         )
       )
-      local restore_line = M.output:get_line_count() ---@type number
+      local restore_line = M.output:get_line_count()
       M.output:add_action({
         text = 'Restore [A]ll',
         type = 'diff_restore_snapshot_all',
@@ -293,10 +293,9 @@ function M._format_message_header(message, msg_idx)
   M.output:add_empty_line()
   M.output:add_metadata({ msg_idx = msg_idx, part_idx = 1, role = role, type = 'header' })
 
-  -- Use the mode field from the message (stable label from CLI)
   local display_name
   if role == 'assistant' then
-    local mode = message.mode or message.assistant_mode
+    local mode = message.mode
     if mode and mode ~= '' then
       display_name = mode:upper()
     else
@@ -360,7 +359,7 @@ function M._format_user_message(text, message)
     context = context_module.extract_from_opencode_message(message)
   end
 
-   local start_line = M.output:get_line_count() - 1 ---@type number
+  local start_line = M.output:get_line_count() - 1
 
   M.output:add_empty_line()
   M.output:add_lines(vim.split(context.prompt, '\n'))
@@ -378,7 +377,7 @@ function M._format_user_message(text, message)
     M.output:add_line(string.format('[%s](%s)', path, context.current_file))
   end
 
-     local end_line = M.output:get_line_count() ---@type number
+  local end_line = M.output:get_line_count()
 
   M._add_vertical_border(start_line, end_line, 'OpencodeMessageRoleUser', -3)
 end
@@ -519,12 +518,9 @@ function M._format_tool(part)
   end
 
   local start_line = M.output:get_line_count() + 1
-   ---@type TaskToolInput|BashToolInput|FileToolInput|TodoToolInput|GlobToolInput|GrepToolInput|WebFetchToolInput|ListToolInput
-   local input = (part.state and part.state.input) or {}
-   ---@type ToolMetadataBase|TaskToolMetadata|WebFetchToolMetadata|BashToolMetadata|FileToolMetadata|GlobToolMetadata|GrepToolMetadata|ListToolMetadata
-   local metadata = (part.state and part.state.metadata) or {}
-   ---@type string
-   local output = (part.state and part.state.output) or ''
+  local input = (part.state and part.state.input) or {}
+  local metadata = (part.state and part.state.metadata) or {}
+  local output = (part.state and part.state.output) or ''
 
   if tool == 'bash' then
     M._format_bash_tool(input --[[@as BashToolInput]], metadata --[[@as BashToolMetadata]])
@@ -552,7 +548,7 @@ function M._format_tool(part)
 
   M.output:add_empty_line()
 
-     local end_line = M.output:get_line_count() ---@type number
+  local end_line = M.output:get_line_count()
   if end_line - start_line > 1 then
     M._add_vertical_border(start_line, end_line - 1, 'OpencodeToolBorder', -1)
   end
@@ -581,7 +577,7 @@ function M._format_task_tool(input, metadata, output)
     end
   end
 
-     local end_line = M.output:get_line_count() ---@type number
+  local end_line = M.output:get_line_count()
   M.output:add_action({
     text = '[S]elect Child Session',
     type = 'select_child_session',