From 70517a6dfd52e416ba5fcc88d279097b794a1800 Mon Sep 17 00:00:00 2001 From: Eric Freese Date: Fri, 26 May 2023 16:50:02 -0600 Subject: [PATCH] Fix issues with LASTWIDGET caused by async suggestions There are a number of widgets that rely on LASTWIDGET so that they can chain their behavior together on subsequent invocations. When a suggestion was fetched asynchronously, the widget showing the suggestion would overwrite LASTWIDGET and break these widgets that rely on it. Relatively recently, a flag was added to the `zle` builtin to opt out of setting LASTWIDGET when calling a widget. We can use that flag to avoid setting LASTWIDGET when displaying suggestions that were fetched asynchronously. --- src/async.zsh | 13 ++++++++++++- zsh-autosuggestions.zsh | 13 ++++++++++++- 2 files changed, 24 insertions(+), 2 deletions(-) diff --git a/src/async.zsh b/src/async.zsh index b07fcc3..d15403f 100644 --- a/src/async.zsh +++ b/src/async.zsh @@ -64,7 +64,7 @@ _zsh_autosuggest_async_response() { if [[ -z "$2" || "$2" == "hup" ]]; then # Read everything from the fd and give it as a suggestion IFS='' read -rd '' -u $1 suggestion - zle autosuggest-suggest -- "$suggestion" + _zsh_autosuggest_async_suggest "$suggestion" # Close the fd exec {1}<&- @@ -73,3 +73,14 @@ _zsh_autosuggest_async_response() { # Always remove the handler zle -F "$1" } + +_zsh_autosuggest_async_suggest() { + # Before 5.9, async suggestions break widgets that rely on LASTWIDGET + # such as copy-earlier-word and {up,down}-line-or-beginning-search. In + # 5.9, a flag was added to `zle` that will skip setting LASTWIDGET so + # that those widgets that depend on it will continue to work + # See https://www.zsh.org/mla/workers/2020/msg00824.html + local nolast + is-at-least 5.9 && nolast=supported + zle autosuggest-suggest ${=nolast:+-f nolast} -- "$1" +} diff --git a/zsh-autosuggestions.zsh b/zsh-autosuggestions.zsh index c76c232..2f466cc 100644 --- a/zsh-autosuggestions.zsh +++ b/zsh-autosuggestions.zsh @@ -819,7 +819,7 @@ _zsh_autosuggest_async_response() { if [[ -z "$2" || "$2" == "hup" ]]; then # Read everything from the fd and give it as a suggestion IFS='' read -rd '' -u $1 suggestion - zle autosuggest-suggest -- "$suggestion" + _zsh_autosuggest_async_suggest "$suggestion" # Close the fd exec {1}<&- @@ -829,6 +829,17 @@ _zsh_autosuggest_async_response() { zle -F "$1" } +_zsh_autosuggest_async_suggest() { + # Before 5.9, async suggestions break widgets that rely on LASTWIDGET + # such as copy-earlier-word and {up,down}-line-or-beginning-search. In + # 5.9, a flag was added to `zle` that will skip setting LASTWIDGET so + # that those widgets that depend on it will continue to work + # See https://www.zsh.org/mla/workers/2020/msg00824.html + local nolast + is-at-least 5.9 && nolast=supported + zle autosuggest-suggest ${=nolast:+-f nolast} -- "$1" +} + #--------------------------------------------------------------------# # Start # #--------------------------------------------------------------------#