Added completion for ollama

[faluciano]

Added completions for ollama.

[daanturo]

Can we additionally supply model names to relevant operations such as run?

This works for me, although I haven’t yet determined if it meets Fish’s conventions:

function __fish_complete_ollama_list
    ollama list 2>/dev/null | tail -n +2 | string replace --regex "\s.*" ""
end

complete -c ollama -f -a "$(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from show' -d "Show information for a model"
complete -c ollama -f -a "$(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from run' -d "Run a model"
complete -c ollama -f -a "$(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from cp' -d "Copy a model"
complete -c ollama -f -a "$(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from rm' -d "Remove a model"

[faluciano]

Added the proposed change to include model names

[daanturo]

Added the proposed change to include model names

Thank you, but I just realized some errors, now:

complete -c ollama -f -a "(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from show'
complete -c ollama -f -a "(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from run'
complete -c ollama -f -a "(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from cp'
complete -c ollama -f -a "(__fish_complete_ollama_list)" --condition '__fish_seen_subcommand_from rm'

"$(__fish_complete_ollama_list)" -> "(__fish_complete_ollama_list)", previously it was eager evaluation
and remove erroneous descriptions (command descriptions for model names).

Can you update them to fix?

[faluciano]

Good catch, will update accordingly