wting / autojump

A cd command that learns - easily navigate directories from the command line
Other
16.08k stars 703 forks source link

Paths no longer existing still displayed with tab/autocomplete #600

Open bitwombat opened 4 years ago

bitwombat commented 4 years ago
$ j feeds

feeds__1__/info/programming/my_software/feeds  feeds__5__/opt/feeds
feeds__2__/opt/feeds                           feeds__6__/usr/share/sounds/freedesktop
feeds__3__/info/programming/my_software/feeds  feeds__7__/info/programming/my_software/feeds
feeds__4__/1_EIT/feedback                      feeds__8__/opt/feeds

$ j feeds_1[ENTER]

EXPECTED BEHAVIOR: cd'ing to /info/programming/my_software/feeds

ACTUAL BEHAVIOR: cd'ing to /1_EIT/feedback

REASON: /info/programming/my_software/feeds actually doesn't exist any more (ifilter with path_exists predicate returns False)

SUGGESTED FIX: Autocomplete list should be filtered by path_exists `

vrubiolo commented 3 years ago

I am affected by this behavior too as I have multiple directories which do not exist anymore. This is pretty annoying since it defeats the predictability of the autojump completion completely: the directory I select via its numbered entry ends up not being the one autojump goes to.

I am looking into writing a script that goes over $HOME/.local/share/autojump/autojump.txt and pruning those empty directory entries but maybe his has already been done?

Thank you very much for this wonderful too otherwise, it has improved productivity greatly for me over the years !