fix: Replace prompt guard timer with observer on opencode window focus

Author: guillaumeboehm

lua/opencode/ui/output_window.lua

@@ -1,7 +1,5 @@
 local state = require('opencode.state')
 local config = require('opencode.config')
-local Timer = require('opencode.ui.timer')
-local topbar = require('opencode.ui.topbar')
 
 local M = {}
 M.namespace = vim.api.nvim_create_namespace('opencode_output')
@@ -157,36 +155,6 @@ function M.setup_keymaps(windows)
 end
 
 function M.setup_autocmds(windows, group)
-  local guard_timer = nil
-
-  local function start_guard_timer()
-    if guard_timer then
-      return -- Timer already running
-    end
-    guard_timer = Timer.new({
-      interval = 1000, -- 1 second
-      repeat_timer = true,
-      on_tick = function()
-        if state.windows and vim.api.nvim_win_is_valid(state.windows.output_win) then
-          topbar.render()
-          return true
-        end
-        return false
-      end,
-    })
-    guard_timer:start_and_tick()
-  end
-
-  local function stop_guard_timer()
-    if guard_timer then
-      guard_timer:stop()
-      guard_timer = nil
-    end
-  end
-
-  -- Start the timer when the window is created (visible) and tick immediately
-  start_guard_timer()
-
   vim.api.nvim_create_autocmd('WinEnter', {
     group = group,
     buffer = windows.output_buf,
@@ -207,15 +175,6 @@ function M.setup_autocmds(windows, group)
     end,
   })
 
-  -- Stop the timer when the window is closed
-  vim.api.nvim_create_autocmd('WinClosed', {
-    group = group,
-    pattern = tostring(windows.output_win),
-    callback = function()
-      stop_guard_timer()
-    end,
-  })
-
   state.subscribe('current_permission', function()
     require('opencode.keymap').toggle_permission_keymap(windows.output_buf)
   end)

lua/opencode/ui/prompt_guard_indicator.lua

@@ -1,6 +1,5 @@
 local M = {}
 
-local state = require('opencode.state')
 local config = require('opencode.config')
 local util = require('opencode.util')
 local icons = require('opencode.ui.icons')

lua/opencode/ui/topbar.lua

@@ -137,12 +137,14 @@ function M.setup()
   state.subscribe('current_mode', on_change)
   state.subscribe('current_model', on_change)
   state.subscribe('active_session', on_change)
+  state.subscribe('is_opencode_focused', on_change)
   M.render()
 end
 
 function M.close()
   state.unsubscribe('current_mode', on_change)
   state.unsubscribe('current_model', on_change)
   state.unsubscribe('active_session', on_change)
+  state.unsubscribe('is_opencode_focused', on_change)
 end
 return M