When `ext_cmdline` is enabled, the event `grid_cursor_goto` will not be sent during incsearch about neovim

[luukvbaal]

I forgot that the current issue is that ext_cmdline is causing event inconsistencies, which should not be the expected behavior.

Why not? This event was intentionally removed in d41b8d4 (#27858). All it did is send an event placing the cursor in the current window. Which is unhelpful, because the cursor should be in the cmdline which is unknown to Nvim. Unlike when ext_cmdline is disabled where this event does place the cursor in the cmdline(message grid).

[xiyaowong]

The potential issues may be related to the way UI events are handled. vscode-neovim currently only can adjust viewport based on cursor position rather than drawing.

However, the example I provided is indeed a special case, and the logic can be adjusted to solve it.

[xiyaowong]

==== Disable ext_cmdline ===
### Start incsearch ###
['grid_cursor_goto', [1, 19, 2]]
['grid_scroll', [1, 3, 18, 3, 33, 11, 0]]
['win_viewport', [4, <Window(handle=1001)>, 11, 27, 25, 1, 27, 11]]
['grid_cursor_goto', [1, 19, 2]]
==== Enable ext_cmdline ===
### Start incsearch ###
['grid_scroll', [1, 3, 18, 3, 33, 11, 0]]
['win_viewport', [4, <Window(handle=1001)>, 11, 27, 25, 1, 27, 11]]

I printed out events related to scroll and viewport. It seems like this behavior is an intentional change in version 0.10? incsearch scrolls the window but doesn't move the cursor.

There should be relevant commits in the commit log.

I forgot that the current issue is that ext_cmdline is causing event inconsistencies, which should not be the expected behavior.

[xiyaowong]

But the cursor is actually changed:

==== Disable ext_cmdline ===
cursor (1, 0)
### Start incsearch ###
grid_cursor_goto
grid_cursor_goto
cursor (26, 1)
==== Enable ext_cmdline ===
cursor (1, 0)
### Start incsearch ###
cursor (26, 1)

    def incsearch():
        nonlocal start_incsearch
        start_incsearch = True
        print('cursor', win.cursor)
        print("### Start incsearch ###")
        nvim.input("gg/e")
        print('cursor', win.cursor)

[luukvbaal]

Well yes the cursor position in the current window changes for incsearch but the cursor position on the grid does not and should not(implicitly). What problem does this cause?