Fish shell integration automatically completes currently shown completion about kitty

[kovidgoyal]

Doesnt reproduce for me with fish version 3.7.0 and empty fish config, and I dont see how
prompt marking can possibly affect completion. Probably some interaction
with something else in your fish config, bisect it to find out.

[martonperei]

I've tested with no fish config, thats’s why I included the shell command with -N
It doesn't happen in iTerm2 with shell integration enabled.

Simplified the reproduction:

1. kitty --config NONE /opt/homebrew/bin/fish -N --login --interactive
2. Run printf "\033]133;A\007"; printf "\033]133;B\007"
3. Type g -> g++ will be shown as a completion
4. Click the mouse in the window (you don't need to, but it seems to trigger it. I've tested with the trackpad too)
5. The completion will be triggered, but randomly the cursor goes back to before the first character. (and the completion isn't triggered)

It kinda looks like an issue with selection and/or focus.
It does not seem to reproduce with -o clear_all_mouse_actions=yes. Any ideas what mouse mappings in kitty could cause it? I disabled all apps that could interfere with input.

Thanks for any pointers!

[kovidgoyal]

clicking the mouse will send fake arrow key press events to move the
cursor. These will trigger completion depending on how you have
completion setup. IIRC in fish the right arrow completes. There isnt
anything that can be done to avoid this, as far as I know.

[martonperei]

Hm, that makes sense. The cursor doesn't move on click without prompt marks, interesting, so that explains why it only happens with them.

Also, my best guess, that something in my setup must be sending phantom mouse presses to the kitty window periodically then, that's why it sometimes autocompletes without mouse presses.

Thanks a lot for the fast response!

[kovidgoyal]

Ideally, fish would have some input function that moves the cursor right
without accepting an auto-suggestion. Then the kitty fish shell integration
could bind that function to some otherwise unused escape sequence and we
would not have this issue. However, there is no such function:
https://fishshell.com/docs/current/cmds/bind.html that I can see.

Even more ideally, fish would add a function that has semantics of
"move cursor to the cell number X on line number Y, where Y is counted
from the line that starts the prompt".

You can try requesting something along these lines, maybe the fish
developers will be kind enough to provide such a function.

[martonperei]

As a band-aid, I've disabled the left click prompt event, while keeping the prompt marks working (which is what I needed)
Kitty could send commandline -f suppress-autosuggestion before moving the cursor, but that would mean the completion would vanish. Though I think that's a little bit better behavior than the current one.
However, after some more debugging, this behavior exists in iTerm2 too, but you have to press alt to move the mouse.

[martonperei]

Actually, there is commandline -C <cursor_position> which sets the cursor position in the current buffer without accepting the suggestion.

[kovidgoyal]

On Wed, Feb 28, 2024 at 09:18:03PM -0800, martonperei wrote: Actually, there is `commandline -C <cursor_position>` which sets the cursor position in the current buffer without accepting the suggestion.

That's relative to the command buffer as a string, we need a way to set it relative to a screen position.