Move document fetching to the first document link request call

Shopify · Sep 6, 2022 · 613812f · 613812f
1 parent 37dfaf2
commit 613812f
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/lib/ruby_lsp/handler.rb b/lib/ruby_lsp/handler.rb
@@ -61,11 +61,6 @@ def initialize
     def start
       $stderr.puts "Starting Ruby LSP..."
 
-      if Requests::DocumentLink::RAILTIES_VERSION
-        $stderr.puts "Fetching Rails Documents..."
-        Requests::DocumentLink.fetch_rails_documents
-      end
-
       @reader.read do |request|
         handler = @handlers[request[:method]]
         next if handler.nil?

diff --git a/lib/ruby_lsp/requests/document_link.rb b/lib/ruby_lsp/requests/document_link.rb
@@ -196,6 +196,11 @@ def add_rails_document_link(node, name)
         # ignore non-Ruby files
         return unless @uri.match?(/\.rb$/)
 
+        if RAILTIES_VERSION
+          $stderr.puts "Fetching Rails Documents..."
+          self.class.fetch_rails_documents
+        end
+
         doc_path = self.class.rails_documents[name]
 
         if doc_path