fix(uninstall): fix uninstall completions

[jdx]

Fixes #1861

[codacy-production]

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation	Diff coverage
✅ +0.00%	✅ ∅

Coverage variation details

	Coverable lines	Covered lines	Coverage
Common ancestor commit (9138e2e)	16632	13846	83.25%
Head commit (41d81ec)	16632 (+0)	13846 (+0)	83.25% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details

	Coverable lines	Covered lines	Diff coverage
Pull request (#1869)	0	0	∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

_{You may notice some variations in coverage metrics with the latest Coverage engine update. For more details, visit the documentation}

[roele]

Was also playing around with this. Does this work? ls returns another format than ls-remote. Also it potentially list missing versions.

My approach would have been

complete "tool@version" run=r#"
prev="{{words[PREV]}}"
cur="{{words[CURRENT]}}"
declare -a rm=("uninstall" "remove" "rm")
case $cur in
  *@*)
    tool="$(echo "$cur" | cut -d'@' -f1)"
    prefix="$(echo "$cur" | cut -d'@' -f2)"

    if [[ " ${rm[@]} " =~ " ${prev} " ]]; then
      versions=$(mise ls --no-header $tool | grep -v '(missing)' | awk '{print $2}' | sed '1!G;h;$!d')
    else
      versions=$(mise ls-remote $tool $prefix | sed '1!G;h;$!d')
    fi

    for version in $versions; do
      echo "$tool@$version"
    done
    ;;
  *)
    if [[ " ${rm[@]} " =~ " ${prev} " ]]; then
      plugins=$(mise plugins --core --user)
    else
      plugins=$(mise plugins --all)
    fi

    for plugin in $plugins; do
      echo "$plugin@"
    done
    ;;
esac
"#

[jdx]

it would work, but I'd be hesitant to hardcode the aliases in the extras file. One problem with my solution is it shows plugins that have no installed versions but I feel it's at least an improvement.

[roele]

ah i see, the --installed flag filters the missing already. agree that hardcoding aliases is indeed not nice.

[jdx]

another note is you never need to pass--no-header, the header is omitted when it isn't a tty