kkoomen / gfi.vim

A zero-configuration and improved version of the "gf" command in Vim, but now smarter.

Hi there 👋

My name is Kim. Learning Chinese since August 2017. Software engineer since 2013.

Since 2022 back at university studying Artificial Intelligence, not because of the hype, but just because it's one of the few things I haven't learned yet that I would really like to learn.

On the road to a PhD:

• 📕 Webdevelopment (4y)
• 📗 Software Engineering foundation year (1y)
• 📘 Artificial Intelligence BSc (3y)
• 📙 Pattern Recognition and Intelligent Systems MSc (2.5y)
• 🎓 PhD

gfi.vim's Issues

[BUG] This breaks `gf` in python files

Describe the bug

When opening python files, the behaviour of gf changes slightly, but this plugin doesn't handle that, which ends up breaking gf.

The changes I'm aware of are:

• separator: import paths have . between the components, but those match to folder separators on the file system.
• suffix: imported paths need to have .py appended if it's a file, and /__init__.py if it's a folder (btw, now that I think about it, I don't know what the precedence between the two should be, in case both foo.py and foo/__init.py exist)

To Reproduce
Steps to reproduce the behavior:

$ cat > foo.py <<EOF
import asd.qwe
EOF

$ mkdir asd
$ touch asd/qwe.py

Open foo.py, navigate to the second word (the import path) and hit gf.

Expected behavior

asd/qwe.py should get opened; instead the following error is printed:

Opening "asd.qwe"...
Could not open "asd.qwe"

This shows clearly that the separators are not properly converted, and that the suffix is missing.

**Output of `vim --version`**

I'm using the standard `vim` package in fedora right now:

VIM - Vi IMproved 8.2 (2019 Dec 12, compiled Mar 15 2021 00:00:00)
Included patches: 1-2607
Modified by <[email protected]>
Compiled by <[email protected]>
Huge version without GUI.  Features included (+) or not (-):
+acl               -farsi             +mouse_sgr         +tag_binary
+arabic            +file_in_path      -mouse_sysmouse    -tag_old_static
+autocmd           +find_in_path      +mouse_urxvt       -tag_any_white
+autochdir         +float             +mouse_xterm       -tcl
-autoservername    +folding           +multi_byte        +termguicolors
-balloon_eval      -footer            +multi_lang        +terminal
+balloon_eval_term +fork()            -mzscheme          +terminfo
-browse            +gettext           +netbeans_intg     +termresponse
++builtin_terms    -hangul_input      +num64             +textobjects
+byte_offset       +iconv             +packages          +textprop
+channel           +insert_expand     +path_extra        +timers
+cindent           +ipv6              +perl/dyn          +title
-clientserver      +job               +persistent_undo   -toolbar
-clipboard         +jumplist          +popupwin          +user_commands
+cmdline_compl     +keymap            +postscript        +vartabs
+cmdline_hist      +lambda            +printer           +vertsplit
+cmdline_info      +langmap           +profile           +virtualedit
+comments          +libcall           -python            +visual
+conceal           +linebreak         +python3/dyn       +visualextra
+cryptv            +lispindent        +quickfix          +viminfo
+cscope            +listcmds          +reltime           +vreplace
+cursorbind        +localmap          +rightleft         +wildignore
+cursorshape       +lua/dyn           +ruby/dyn          +wildmenu
+dialog_con        +menu              +scrollbind        +windows
+diff              +mksession         +signs             +writebackup
+digraphs          +modify_fname      +smartindent       -X11
-dnd               +mouse             -sound             -xfontset
-ebcdic            -mouseshape        +spell             -xim
+emacs_tags        +mouse_dec         +startuptime       -xpm
+eval              +mouse_gpm         +statusline        -xsmp
+ex_extra          -mouse_jsbterm     -sun_workshop      -xterm_clipboard
+extra_search      +mouse_netterm     +syntax            -xterm_save
   system vimrc file: "/etc/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/etc"
 f-b for $VIMRUNTIME: "/usr/share/vim/vim82"
Compilation: gcc -c -I. -Iproto -DHAVE_CONFIG_H -O2 -flto=auto -ffat-lto-objects -fexceptions -g -grecord-gcc-switches -pipe -Wall -Werror=format-security -Wp,-D_GLIBCXX_ASSERTIONS -specs=/usr/lib/rpm/redhat/redhat-hardened-cc1 -fstack-protector-strong -specs=/usr/lib/rpm/redhat/redhat-annobin-cc1 -m64 -mtune=generic -fasynchronous-unwind-tables -fstack-clash-protection -fcf-protection -D_GNU_SOURCE -D_FILE_OFFSET_BITS=64 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1 
Linking: gcc -L. -Wl,-z,relro -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -fstack-protector-strong -rdynamic -Wl,-export-dynamic -Wl,--enable-new-dtags -Wl,-z,relro -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -Wl,-z,relro -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -L/usr/local/lib -Wl,--as-needed -o vim -lm -lselinux -lncurses -lacl -lattr -lgpm -ldl -Wl,--enable-new-dtags -Wl,-z,relro -Wl,--as-needed -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -Wl,-z,relro -Wl,--as-needed -Wl,-z,now -specs=/usr/lib/rpm/redhat/redhat-hardened-ld -fstack-protector-strong -L/usr/local/lib -L/usr/lib64/perl5/CORE -lperl -lpthread -lresolv -ldl -lm -lcrypt -lutil -lc 

[FEATURE] check for index file

Is your feature request related to a problem? Please describe.
Hi! Great plugin but if I try to load a path that is a folder, for example "../routes/home" it won't load as the exact file path is "../routes/home/index.js"

Describe the solution you'd like
I'd like to add the index route in the paths list with something like

call add(l:paths, simplify(getcwd() . '/' . l:cfile . '/index'))

It works for me but there might be something to take into account which I'm not aware of.

I'd be happy to send a PR but I've got no permissions