Resolve symlinks before editing.

Author: moll

CHANGELOG.md

@@ -10,6 +10,8 @@
 - Updates URLs to use <https://github.com/nodejs/node> instead of the previous
   Joyent repo.  
   Thanks, [Jacky Alciné](https://jacky.wtf), for the help!
+- Resolves symlinks in paths before editing a file.  
+  Shows correct paths when using the relative symlink trick in `node_modules` (`ln -s .. node_modules/root`) and using `gf` on `require("root/lib/foo")`.
 
 ## 0.8.1 (Apr 15, 2014)
 - Updates the URL from which Node.vim downloads Node core module source files.  

autoload/node/lib.vim

@@ -22,7 +22,8 @@ function! node#lib#find(name, from)
 		return s:CORE_URL_PREFIX ."/". l:version ."/". l:dir ."/". a:name .".js"
 	endif
 
-	return s:resolve(s:absolutize(a:name, a:from))
+	let l:path = s:resolve(s:absolutize(a:name, a:from))
+	if !empty(path) | return resolve(path) | endif
 endfunction
 
 function! node#lib#version()

test/autoload_test.rb

@@ -145,6 +145,17 @@
       $vim.echo(%(bufname("%"))).must_equal target
     end
 
+    it "must edit ./foo.js given root/foo with root as symlink" do
+      touch File.join(@dir, "index.js"), %(require("root/foo"))
+      target = touch File.join(@dir, "foo.js")
+      FileUtils.mkpath File.join(@dir, "node_modules")
+      File.symlink "..", File.join(@dir, "node_modules", "root")
+
+      $vim.edit File.join(@dir, "index.js")
+      $vim.feedkeys "$hhgf"
+      $vim.echo(%(bufname("%"))).must_equal target
+    end
+
     it "must not show an error when searching for nothing" do
       touch File.join(@dir, "index.js"), %("")