kislyuk / argcomplete

Python and tab completion, better together.
https://kislyuk.github.io/argcomplete/
Apache License 2.0
1.42k stars 135 forks source link

zsh support, including option descriptions #10

Closed nickl- closed 1 year ago

nickl- commented 11 years ago

Let me start by saying well done with all the improvements since v2.9 the README looks awesome I must make a point to look at the docs... but who reads manuals anyway? =)

Being the devils advocate again, even though I am well aware that the project clearly states BASH completion I know there are many who use zsh for whatever reasons unbeknown to myself, your friendly devil and scope-boundary-shifter extraordinaire.

From what I can source the differences are not that huge but I am not well versed on things that make you go Oh My at all, maybe someone with such astonishment afflictions would care to step up and explain yourselves or just enlighten us on the peculiarities of zsh completion as both would be fascinating to hear.

What I can add: There is something called bashcompinit as explained in man zshcompsys and reads as follows:

The function bashcompinit compatibility with bash's programmable completion system. When run it will define the functions, compgen and complete which correspond to the bash builtins with the same names. It will then be possible to use completion specifications and functions written for bash.

Would that suggest our work is done here or do we need something other (zstyle, compctl, worshiping of false devils perhaps)? None of which would make me jump with joy, being a jealous devil and all, just thought I'd stir the pot and see what's cooking.

nJoy!

kislyuk commented 11 years ago

I don't really care about zsh, but it is within the scope of this package and it's certainly #2 on the list of shells to support.

Does bashcompinit support complete -D?

nickl- commented 11 years ago

I do not have the foggiest - merely mentioned what I found on google as a starting point but I won't be much more use I am afraid. Had zsh support in aero before I yanked the custom implementation in favour of supporting argcomplete instead, for the greater good and all that. Would be nice to get it back but I won't be loosing any sleep over it just yet.

Anyone we can pass this ball to?

myint commented 11 years ago

I tested bashcompinit out of curiousity. It works.

$ autoload bashcompinit
$ bashcompinit
$ autoload compinit
$ compinit
$ eval "$(register-python-argcomplete docformatter)"
$ docformatter -<tab>
--help                 --recursive            -h
--in-place             --version              -i
--no-blank             --wrap-descriptions    -r
--pre-summary-newline  --wrap-summaries
kislyuk commented 11 years ago

Glad to hear that! If you want, you can write a test suite and add it to tests (please configure it to only run if zsh and bashcompinit are found).

redstreet commented 11 years ago

While argcomplete does indeed run fine on zsh, zsh's auto-complete is far more advanced. argcomplete doesn't take advantage of any of this. Here are two example features that I'd love to see argcomplete handle:

1) mini-documentation: zsh is capable of listing each possible argument with a mini documentation about it. When I type "git /tab/", I get:

add                 -- add paths to the index
am                  -- apply patches from a mailbox (cooler than applymbox)
annotate            -- annotate file lines with commit info

2) aliases: zsh is aware of aliases. For example, it can be told that '-h' is the same as '--help', and will display both of them on the same line, and won't show the --help if -h is already typed (and vice versa). In addition, for the mini-doc display, it will display them on the same line. For example, if I type "p4 /tab/", I get:

changelist    change   -- Create or edit a changelist description                                                                                                                                                        

They're shown together because "change" is an alias of the "changelist" subcommand.

Since argcomplete has all the necessary info to implement both these features, I'd strongly vote for doing this as a feature request. If I get around to doing it, I'll submit a patch. Meanwhile, thanks for argcomplete, it's awesome!

kislyuk commented 11 years ago

Sounds great, I'm not a big zsh user and don't have a lot of time for this, but I'd love to see this.

nickl- commented 11 years ago

I found the commit that yanked out the custom autocomplete which worked for zsh as well, and got replaced by argcomplete. Aeronautics/aero@06de51e6c5d and this was the eval code Aeronautics/aero@5ec10de5057 which did the magic.

Along the lines of:

function _aero_completion {
  local words cword
  read -Ac words
  read -cn cword
  reply=( $( COMP_WORDS="$words[*]" \\
             COMP_CWORD=$(( cword-1 )) \\
             AERO_AUTO_COMPLETE=1 $words[1] ) )
}
compctl -K _aero_completion aero

Does that shed any lights in the otherwise dark tunnel?

redstreet commented 11 years ago

The current bash autocomplete works fine in zsh, since zsh can accept bash autocomplete scripts. However, this doesn't use the much more advanced features for autocomplete that zsh has. Taking advantage of these features would likely require a bunch of new code (i.e., it's not a simple fix, it requires some design and development).

tony commented 11 years ago

@redstreet @nickl- @kislyuk

Another example from aws-cli, which uses a wrapper similar to the one mentioned by @nickl- in Aero.

aws-cli approach to wrapping zsh: https://github.com/aws/aws-cli/blob/develop/bin/aws_zsh_completer.sh

$ source bin/aws_zsh_completer.sh

Related: tcsh on #49

kislyuk commented 11 years ago

@tony, thanks for the pointer to https://github.com/aws/aws-cli/blob/develop/bin/aws_zsh_completer.sh, there's some interesting/crazy code in there.

Does zsh have a convention for installing completion hooks globally or per user? And is anyone interested in finding out what should be done to make use of the advanced features that @redstreet is referring to?

redstreet commented 11 years ago

As I mentioned earlier, the current bash autocomplete that argcomplete does works fine in zsh, since zsh can accept bash autocomplete scripts. The code at https://github.com/aws/aws-cli/blob/develop/bin/aws_zsh_completer.sh seems to help with getting this to work better, but doesn't use the advanced zsh autocompletion features themselves.

Here's an example of one of the features: showing brief descriptions next to parameters or subcommands: http://joshparnham.com/2012/10/nanoc-plus-zsh-equals-awesomeness/

I've written small zsh autcomplete scripts, and I'd be happy to work on this, but won't have the time in the near future. If anyone is interested, here's a very basic tutorial to get you started: http://askql.wordpress.com/2011/01/11/zsh-writing-own-completion/

anntzer commented 10 years ago

Does argcomplete currently support global installation with zsh? I haven't managed to get it to work (while it works fine with zsh), and I don't think this is clearly specified in the docs.

kislyuk commented 10 years ago

I don't use zsh, so I don't have a clear idea of what level of compatibility its bash completion compatibility layer provides. From the zsh documentation, it seems like this should work:

autoload bashcompinit
bashcompinit
source argcomplete/bash_completion.d/python-argcomplete.sh

But cursory testing shows that it's not working.

anntzer commented 10 years ago

Yes, that is what I tried. I understand fixing this is not a priority for you but can you please document this limitation?

kislyuk commented 10 years ago

In https://github.com/kislyuk/argcomplete/commit/b38c1aaec131871238a68ca9291db572fa7138c7.

sotte commented 10 years ago

:+1: for zsh support.

kislyuk commented 10 years ago

Pull requests are welcome. I would love to add it, but I don't use zsh on a daily basis and don't currently have the time to write this. Need a zsh expert to take a look.

yac commented 9 years ago

FYI: I tested with zsh's bashcompinit and it indeed seems to work fine.

As already stated, bash completion is overly simple (read: crap) and using zsh native completion would yield much better results... However, supporting multiple output formats in a code that was written with only one in mind sounds like more work than I'm willing to do. But a man can dream, right? :)

segevfiner commented 6 years ago

If completion doesn't work for you using bashcompinit, note https://github.com/zsh-users/zsh/commit/e2f793e7df7214cc3d80e2fcfe961ed087c860ab which I had to apply to my system's bashcompinit to get this to work.

atyshka commented 4 years ago

Will global support work in zsh the same way it does in bash? I.e. not having to individually register files? The portion of the README addressing zsh only describes the individual file method.

dan1994 commented 4 years ago

When switching to zsh and finding out that argcomplete is not fully compatible (via this issue), I decided to try my hand at writing my own solution to this problem. I have just released my first version of it, and thought to share for all of the people who'll reach this thread like I did. The project is called pyzshcomplete and can be found here. Any feedback would be highly appreciated.

kislyuk commented 1 year ago

argcomplete now supports zsh directly (without the bashcompinit compatibility layer) in the develop branch.

The practical impact of this support is currently limited to emitting description strings in zsh, but by directly emitting completions to zsh instead of using the bash compatibility layer, we have unlocked the potential for using other, fancier zsh features as well.

Global completion is not supported yet (I am still looking into how to support it in zsh), so commands have to be individually registered with register-python-argcomplete.

I will be releasing this functionality in a new release shortly.

kislyuk commented 1 year ago

OK, I was able to activate global completion for zsh using compdef _python_argcomplete_global -P '*', so zsh is now fully and officially supported by argcomplete.

kislyuk commented 1 year ago

Released in v3.0.0.