ruby-lsp-rails 0.3.28 → 0.3.30

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 231b488bf88047c642f2556980d6c0ce96edbeafbf2ded65a4104515584a5100
-  data.tar.gz: 207b91a283859f74804b4e825d4ca71ee645b0f73ac84e5d7e7348b382bf2745
+  metadata.gz: 49634ac9d3fa4dd22f14f87ba41520ac0edbdee999157a46b354c1c6da36b855
+  data.tar.gz: 462413c2420e98671bee2aaadd5bb843e6b209a011fd375f6ef59f871cbb75d1
 SHA512:
-  metadata.gz: 8ef4e5363a31a1821da8d43738c5049239e827d609ffa8c8214f9e8fae148739770614a4f06af082da98a755bfd3522426560936d1ac57f0c14092ace8ced86b
-  data.tar.gz: 348fab1dc3ada6a6a2672c4c9b54e0ae0f67f2f056023929af7c76dac5d66583c32c9fbdcc1347fc3913cb59a3e30ae7e495977b762228aad6a52f502254a930
+  metadata.gz: 83f71eb4c8622c3652b561e0c5f4a4d3fa9de34b853e6fa5877435256b12b580c08a10ff8b1593d0224c48c0f9fec2ea633e4bdc9d6c2ae45cb292f6e681f67b
+  data.tar.gz: f4c7425425794e1e1b2cd3a24a5283397b702837a6383a2a617f3cffe97495b5afae44c0a96f6806ad17452c67bf6b2bd8aed84c23d6cab0b596f52ef3ee6fc7

data/lib/ruby_lsp/ruby_lsp_rails/runner_client.rb

@@ -105,6 +105,8 @@ module RubyLsp
             while (content = @stderr.gets("\n"))
               log_message(content, type: RubyLsp::Constant::MessageType::LOG)
             end
+          rescue IOError
+            # The server was shutdown and stderr is already closed
           end,
           Thread,
         )

data/lib/ruby_lsp/ruby_lsp_rails/server.rb

@@ -4,9 +4,6 @@
 require "json"
 require "open3"
 
-# NOTE: We should avoid printing to stderr since it causes problems. We never read the standard error pipe from the
-# client, so it will become full and eventually hang or crash. Instead, return a response with an `error` key.
-
 module RubyLsp
   module Rails
     module Common
@@ -125,6 +122,7 @@ module RubyLsp
 
       def start
         load_routes
+        clear_file_system_resolver_hooks
         send_result({ message: "ok", root: ::Rails.root.to_s })
 
         while @running
@@ -302,6 +300,18 @@ module RubyLsp
           routes_reloader.execute_unless_loaded if routes_reloader&.respond_to?(:execute_unless_loaded)
         end
       end
+
+      # File system resolver hooks spawn file watcher threads which introduce unnecessary overhead since the LSP already
+      # watches files. Since the Rails application is already booted by the time we reach this script, we can't no-op
+      # the file watcher implementation. Instead, we clear the hooks to prevent the registered file watchers from being
+      # instantiated
+      def clear_file_system_resolver_hooks
+        return unless defined?(::ActionView::PathRegistry)
+
+        with_notification_error_handling("clear_file_system_resolver_hooks") do
+          ::ActionView::PathRegistry.file_system_resolver_hooks.clear
+        end
+      end
     end
   end
 end

data/lib/ruby_lsp_rails/version.rb

@@ -3,6 +3,6 @@
 
 module RubyLsp
   module Rails
-    VERSION = "0.3.28"
+    VERSION = "0.3.30"
   end
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp-rails
 version: !ruby/object:Gem::Version
-  version: 0.3.28
+  version: 0.3.30
 platform: ruby
 authors:
 - Shopify
 bindir: bin
 cert_chain: []
-date: 2025-01-07 00:00:00.000000000 Z
+date: 2025-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: ruby-lsp