Completion for zsh seems not working on macOS
silvioprog opened this issue · comments
Silvio Clécio commented
Hi.
It seems the code completion is not working in zsh
environment.
Environment
Apple M1 Pro with macOS Monterey 12.5.1
Steps to reproduce
mkdir myapp
cd myapp
go mod init myapp
cobra-cli init
cobra-cli add serve
go build .
./myapp completion zsh > /tmp/completion
source /tmp/completion
now, try to complete:
./myapp se[TAB]
nothing is completed after hitting TAB (even multiple hits), but it should complete the serve
command.
Generated zsh code
% cat /tmp/completion
#compdef myapp
# zsh completion for myapp -*- shell-script -*-
__myapp_debug()
{
local file="$BASH_COMP_DEBUG_FILE"
if [[ -n ${file} ]]; then
echo "$*" >> "${file}"
fi
}
_myapp()
{
local shellCompDirectiveError=1
local shellCompDirectiveNoSpace=2
local shellCompDirectiveNoFileComp=4
local shellCompDirectiveFilterFileExt=8
local shellCompDirectiveFilterDirs=16
local lastParam lastChar flagPrefix requestComp out directive comp lastComp noSpace
local -a completions
__myapp_debug "\n========= starting completion logic =========="
__myapp_debug "CURRENT: ${CURRENT}, words[*]: ${words[*]}"
# The user could have moved the cursor backwards on the command-line.
# We need to trigger completion from the $CURRENT location, so we need
# to truncate the command-line ($words) up to the $CURRENT location.
# (We cannot use $CURSOR as its value does not work when a command is an alias.)
words=("${=words[1,CURRENT]}")
__myapp_debug "Truncated words[*]: ${words[*]},"
lastParam=${words[-1]}
lastChar=${lastParam[-1]}
__myapp_debug "lastParam: ${lastParam}, lastChar: ${lastChar}"
# For zsh, when completing a flag with an = (e.g., myapp -n=<TAB>)
# completions must be prefixed with the flag
setopt local_options BASH_REMATCH
if [[ "${lastParam}" =~ '-.*=' ]]; then
# We are dealing with a flag with an =
flagPrefix="-P ${BASH_REMATCH}"
fi
# Prepare the command to obtain completions
requestComp="${words[1]} __complete ${words[2,-1]}"
if [ "${lastChar}" = "" ]; then
# If the last parameter is complete (there is a space following it)
# We add an extra empty parameter so we can indicate this to the go completion code.
__myapp_debug "Adding extra empty parameter"
requestComp="${requestComp} \"\""
fi
__myapp_debug "About to call: eval ${requestComp}"
# Use eval to handle any environment variables and such
out=$(eval ${requestComp} 2>/dev/null)
__myapp_debug "completion output: ${out}"
# Extract the directive integer following a : from the last line
local lastLine
while IFS='\n' read -r line; do
lastLine=${line}
done < <(printf "%s\n" "${out[@]}")
__myapp_debug "last line: ${lastLine}"
if [ "${lastLine[1]}" = : ]; then
directive=${lastLine[2,-1]}
# Remove the directive including the : and the newline
local suffix
(( suffix=${#lastLine}+2))
out=${out[1,-$suffix]}
else
# There is no directive specified. Leave $out as is.
__myapp_debug "No directive found. Setting do default"
directive=0
fi
__myapp_debug "directive: ${directive}"
__myapp_debug "completions: ${out}"
__myapp_debug "flagPrefix: ${flagPrefix}"
if [ $((directive & shellCompDirectiveError)) -ne 0 ]; then
__myapp_debug "Completion received error. Ignoring completions."
return
fi
local activeHelpMarker="_activeHelp_ "
local endIndex=${#activeHelpMarker}
local startIndex=$((${#activeHelpMarker}+1))
local hasActiveHelp=0
while IFS='\n' read -r comp; do
# Check if this is an activeHelp statement (i.e., prefixed with $activeHelpMarker)
if [ "${comp[1,$endIndex]}" = "$activeHelpMarker" ];then
__myapp_debug "ActiveHelp found: $comp"
comp="${comp[$startIndex,-1]}"
if [ -n "$comp" ]; then
compadd -x "${comp}"
__myapp_debug "ActiveHelp will need delimiter"
hasActiveHelp=1
fi
continue
fi
if [ -n "$comp" ]; then
# If requested, completions are returned with a description.
# The description is preceded by a TAB character.
# For zsh's _describe, we need to use a : instead of a TAB.
# We first need to escape any : as part of the completion itself.
comp=${comp//:/\\:}
local tab="$(printf '\t')"
comp=${comp//$tab/:}
__myapp_debug "Adding completion: ${comp}"
completions+=${comp}
lastComp=$comp
fi
done < <(printf "%s\n" "${out[@]}")
# Add a delimiter after the activeHelp statements, but only if:
# - there are completions following the activeHelp statements, or
# - file completion will be performed (so there will be choices after the activeHelp)
if [ $hasActiveHelp -eq 1 ]; then
if [ ${#completions} -ne 0 ] || [ $((directive & shellCompDirectiveNoFileComp)) -eq 0 ]; then
__myapp_debug "Adding activeHelp delimiter"
compadd -x "--"
hasActiveHelp=0
fi
fi
if [ $((directive & shellCompDirectiveNoSpace)) -ne 0 ]; then
__myapp_debug "Activating nospace."
noSpace="-S ''"
fi
if [ $((directive & shellCompDirectiveFilterFileExt)) -ne 0 ]; then
# File extension filtering
local filteringCmd
filteringCmd='_files'
for filter in ${completions[@]}; do
if [ ${filter[1]} != '*' ]; then
# zsh requires a glob pattern to do file filtering
filter="\*.$filter"
fi
filteringCmd+=" -g $filter"
done
filteringCmd+=" ${flagPrefix}"
__myapp_debug "File filtering command: $filteringCmd"
_arguments '*:filename:'"$filteringCmd"
elif [ $((directive & shellCompDirectiveFilterDirs)) -ne 0 ]; then
# File completion for directories only
local subdir
subdir="${completions[1]}"
if [ -n "$subdir" ]; then
__myapp_debug "Listing directories in $subdir"
pushd "${subdir}" >/dev/null 2>&1
else
__myapp_debug "Listing directories in ."
fi
local result
_arguments '*:dirname:_files -/'" ${flagPrefix}"
result=$?
if [ -n "$subdir" ]; then
popd >/dev/null 2>&1
fi
return $result
else
__myapp_debug "Calling _describe"
if eval _describe "completions" completions $flagPrefix $noSpace; then
__myapp_debug "_describe found some completions"
# Return the success of having called _describe
return 0
else
__myapp_debug "_describe did not find completions."
__myapp_debug "Checking if we should do file completion."
if [ $((directive & shellCompDirectiveNoFileComp)) -ne 0 ]; then
__myapp_debug "deactivating file completion"
# We must return an error code here to let zsh know that there were no
# completions found by _describe; this is what will trigger other
# matching algorithms to attempt to find completions.
# For example zsh can match letters in the middle of words.
return 1
else
# Perform file completion
__myapp_debug "Activating file completion"
# We must return the result of this command, so it must be the
# last command, or else we must store its result to return it.
_arguments '*:filename:_files'" ${flagPrefix}"
fi
fi
fi
}
# don't run the completion function when being source-ed or eval-ed
if [ "$funcstack[1]" = "_myapp" ]; then
_myapp
fi
Is this a bug, or am I doing something wrong? 😕
TIA
Silvio Clécio commented
Additional info: same problem happening for cobra-cli
:
cobra-cli completion zsh > /tmp/completion
source /tmp/completion
cobra-cli com[TAB]
it should complete the argument completion
, but nothing is completed.
Silvio Clécio commented
Fixed by doing the following:
mkdir -p .zsh/completion/
cobra-cli completion zsh > .zsh/completion/_cobra-cli
echo 'fpath=(~/.zsh/completion $fpath)' >> .zshrc
echo 'autoload -Uz compinit && compinit -i' >> .zshrc
source .zshrc
now, if you try cobra-cli com[TAB]
, it completes automatically to cobra-cli completion
. 😄
Silvio Clécio commented
Lastly, for the myapp
:
mkdir myapp
cd myapp
go mod init myapp
cobra-cli init
cobra-cli add serve
go build .
./myapp completion zsh > ~/.zsh/completion/_myapp
source ~/.zshrc
now, to auto complete:
./myapp s[TAB]
it will auto complete to ./myapp serve
. 👍