fix(ui): restore output views without normal commands

[jensenojs]

Summary

Remove Normal-mode view commands from automatic output view maintenance.

The previous fix in #427 tried to protect the IME-sensitive output auto-scroll path by suppressing focus-related window/buffer events around normal! zb. With the view restore moved to winsaveview() / winrestview(), that wrapper no longer has a job.

This patch removes the source instead:

• output auto-scroll no longer uses normal! zb;
• lazy-render top-anchor restore no longer uses normal! zt;
• the fix: keep input focused during output auto-scroll #427 eventignore wrapper is removed;
• the behavior is covered directly: input focus stays put, and auto-scroll does not emit WinEnter, WinLeave, BufEnter, or ModeChanged.

Other normal! call sites are left unchanged because they are user navigation / picker / selection paths, not automatic output view maintenance.

Tests

• ./run_tests.sh

[phanen]

Here's another way to avoid emit ModeChanged

local function scroll_view(target)
  local view = vim.fn.winsaveview()
  view.topline = view.lnum - target + 1
  vim.fn.winrestview(view)
end

vim.keymap.set('n', '<leader>zb', function()
  scroll_view(vim.api.nvim_win_get_height(0))
end)
vim.keymap.set('n', '<leader>zz', function()
  scroll_view(math.ceil(vim.api.nvim_win_get_height(0) / 2))
end)

vim.keymap.set('n', '<leader>zt', function()
  scroll_view(1)
end)

[phanen]

All of this if-else branch should be included if we want to suppressed ModeChanged.

opencode.nvim/lua/opencode/ui/renderer/scroll.lua

Lines 128 to 138 in af18aec

	if windows and vim.api.nvim_get_current_win() == windows.input_win then
	with_window_event_autocmds_ignored(function()
	vim.api.nvim_win_call(win, function()
	vim.cmd('normal! zb')
	end)
	end)
	else
	vim.api.nvim_win_call(win, function()
	vim.cmd('normal! zb')
	end)
	end

Also there're many other call site not sure if all of them related.

[jensenojs]

Thanks, your suggestion is the right direction. I revised the PR to use winsaveview() / winrestview() for output view restore instead of suppressing ModeChanged.

This removes the normal! zb / normal! zt calls from automatic output maintenance, and also removes the old eventignore wrapper.

Could you give the updated branch a try? @phanen

[phanen]

yeah, I'm using this branch now.

[phanen]

Other thing lgtm, I've been on this branch a day and didn't reproduce input switch issue.

[sudo-tee]

Thanks for the PR.

I've tested and it seems pretty stable for me to.