Skip non file: URI scheme notifications (#475)

file watcher receives them in some cases
those notifications should not trigger build
we are also unable to update dirty flag
Fixes #474

Author: lukaszsamson

apps/language_server/lib/language_server/server.ex

@@ -390,9 +390,13 @@ defmodule ElixirLS.LanguageServer.Server do
   end
 
   defp handle_notification(did_change_watched_files(changes), state) do
+    changes = Enum.filter(changes, &match?(%{"uri" => "file:" <> _}, &1))
+
     needs_build =
-      Enum.any?(changes, fn %{"uri" => uri, "type" => type} ->
-        Path.extname(uri) in @watched_extensions and
+      Enum.any?(changes, fn %{"uri" => uri = "file:" <> _, "type" => type} ->
+        path = SourceFile.path_from_uri(uri)
+
+        Path.extname(path) in @watched_extensions and
           (type in [1, 3] or not Map.has_key?(state.source_files, uri) or
              state.source_files[uri].dirty?)
       end)
@@ -422,6 +426,7 @@ defmodule ElixirLS.LanguageServer.Server do
               end
 
             _ ->
+              # file not open or not dirty
               acc
           end
       end)

apps/language_server/test/server_test.exs

@@ -629,6 +629,18 @@ defmodule ElixirLS.LanguageServer.ServerTest do
       state = :sys.get_state(server)
       assert state.needs_build?
     end
+
+    test "gracefully skip not supported URI scheme", %{server: server} do
+      uri = "git://github.com/user/repo.git"
+      fake_initialize(server)
+
+      Server.receive_packet(
+        server,
+        did_change_watched_files([%{"uri" => uri, "type" => 2}])
+      )
+
+      :sys.get_state(server)
+    end
   end
 
   test "hover", %{server: server} do