Commit Graph

173 Commits (4a82ff1ead68db25645b2e920f17774a66e44e31)

Author SHA1 Message Date
romkatv 4a82ff1ead speed up widget rebinding by removing redundant array subscripts
Eric Freese f76472272e cleanup: Remove unnecessary braces
Eric Freese 50579b3371 Move widget definitions inside anonymous function
dana aee1b10db6 Avoid warn_create_global warnings
Eric Freese e937e89267 Respect user's set options when running original widget
Fixes GitHub 
Eric Freese 11251d97ca Merge branch 'master' into develop
Eric Freese 41657e3565 Revert async process substitution & completion strategy
They're not quite ready. Keep them on a feature branch for now.
Eric Freese 286656635c v0.5.0
Eric Nielsen 9e4d3c337f Don't overwrite config with default values
otherwise users are obliged to set the config values *after* sourcing
the plugin. They're not able to do it before. Also, re-sourcing the
plugin will reset the values to the defaults again.

See zimfw/zimfw#301

Fixes 
Eric Freese a78ea16c50 Support fallback strategies by setting array in config
Eric Freese 62f5f14f2f default => history
Eric Freese 4540304fa0 Reset opts in some functions affected by GLOB_SUBST
Should fix GitHub 
Eric Freese 4097d6e0ae Support widgets starting with dashes (ex: `-a-widget`)
Fixes 
Eric Freese 0f80f9511d Fix handling of newline + carriage return in async pty ()
Eric Nielsen e61442161e Don't overwrite config with default values
otherwise users are obliged to set the config values *after* sourcing
the plugin. They're not able to do it before. Also, re-sourcing the
plugin will reset the values to the defaults again.

See zimfw/zimfw#301

Fixes 
Eric Freese 681ffc7b28 Reset opts in some functions affected by GLOB_SUBST
Should fix GitHub 
Eric Freese b8bf86f6ab
Merge pull request from zsh-users/fixes/async-bad-file-descriptor
Fixes/async bad file descriptor
Eric Freese 7ab2124904 Kill async process by id when job control disabled
Eric Freese 88fe824ddf Add some error handling to async response handler
We only want to read data in case of POLLIN or POLLHUP. Not POLLNVAL or
select error.

We always want to remove the handler, so it doesn't get called in an
infinite loop when error is nval or err.

In zsh source, see main zle event loop in zle_main.c raw_getbyte
function.
Eric Freese 93877f6b76 We also need to remove the handler when cancelling async request
Should fix GitHub 
Eric Freese bd1fd97738 Cleanup unused async pty name
Eric Freese f1c3b98774 Only capture completions at the end of the buffer.
To prevent the suggestion from not starting with the buffer string.

Example:

`ls / /[cursor left][cursor left]b`

Before the patch, suggests `ls /b /ls /bin/ /`

After the patch, suggests `ls /b /bin/`.

https://github.com/zsh-users/zsh-autosuggestions/issues/343#issuecomment-401675712
Eric Freese 4869a757c8 Ensure we always destroy the zpty
If running in sync mode and a completion takes a long time, the user can
^C out of it. Without this patch, the pty will not be destroyed in this
case and the next time we go to create it, it will fail, making the
shell unusable.
Eric Freese 302bd7c059 Setup zshexit hook immediately in both sync/async cases
Eric Freese 245f5d2ba2 Improve completion suggestions
Just insert the first completion directly into the buffer and read the
whole buffer from the zpty.
Martin Väth 8ae0283c90 Do not rely on implicit NULLCMD=cat
Eric Freese c0315e96d8 Don't use `-s` option to `zmodload`
It is not available in zsh versions older than 5.3
Eric Freese 5529102afc zpty module is only needed for `completion` strategy
Eric Freese dad6be4d5e Remove unused feature detection
Not needed after move away from zpty for async
Eric Freese 7d968869e3 Return if no completion found
Eric Freese 8593624996
Merge pull request from zsh-users/features/completion-suggestions
Features/completion suggestions
Eric Freese 1ec43c7291 Fix error when single quote entered into buffer
Error looked something like:

```
% echo 'f(zpty):8: unmatched '
_zsh_autosuggest_capture_completion:zpty:9: no such pty command: zsh_autosuggest_completion_pty
_zsh_autosuggest_capture_completion:zpty:14: no such pty command: zsh_autosuggest_completion_pty
_zsh_autosuggest_capture_completion:zpty:21: no such pty command: zsh_autosuggest_completion_pty
```

According to `man zshmodules`, the args to `zpty` are "concatenated with
spaces between, then executed as a command, as if passed to the eval
builtin." So we need to escape the `$` so that `$1` is passed to eval
instead of the value of `$1`.
Martin Väth 43a011026f Do not leak global variables REPLY and strategy
https://github.com/zsh-users/zsh-autosuggestions/issues/341
Eric Freese cd81522b30 Attempt to kill async worker process when new request comes in
See http://www.zsh.org/mla/users/2018/msg00432.html
Eric Freese d7171232c3 Merge branch 'develop' into features/completion-suggestions
Eric Freese 9cb0101512 Refactor async mode to no longer use zpty
See technique used in `fast-syntax-highlighting`:
- ca2e18bbc9
- http://www.zsh.org/mla/users/2018/msg00424.html
Eric Freese b0ffc34fb8 completion should be a local var
Eric Freese 4e466f0e4e Support widgets starting with dashes (ex: `-a-widget`)
Fixes 
Eric Freese bcbdad83e9 Support fallback strategies by setting array in config
Eric Freese 949c374195 Fix `completion` strategy on older versions of zsh
`zpty -r` with a pattern seems to have some funky behavior on older
versions, giving unpredictable results
Eric Freese 973205005c Add spec for `completion` strategy
Eric Freese cf458d2a3b Fix completion suggestions when compinit is not enabled
Need to make sure compinit is called in the pty or the shell hangs
Eric Freese 3dbd9afaec Fix completion strategy killing other pty's
Only a problem in synchronous mode
Eric Freese 6ffaec725a Allow completion suggestions from current shell
The `zsh -f` running in the PTY doesn't know about the non-exported
variables and functions defined in the original shell, thus can't make
suggestions for them. Run local functions in the PTY instead of a new
`zsh` process.

We have to handle things differently based on whether zle is active or
not (async vs. sync mode).
Eric Freese 0a548c62f4 Forgot to make after small tweak
Eric Freese 4cca26ec84 Modify completion code to better fit our needs
Only need the first completion result
Eric Freese c5551daabc Default strategy now tries history first and falls back to completion
Eric Freese 82b08e2dc8 First pass at getting suggestions from completion engine ()
Uses https://github.com/Valodim/zsh-capture-completion with some slight
modifications.
Eric Freese 63789e96b5 Fix handling of newline + carriage return in async pty ()
Eric Freese aa0b10db44 v0.4.3