ruby-lsp 0.26.2 → 0.26.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a3c596f1104438061fe9ee2b890b980a7048e2137c10d34783d55defa0f17354
-  data.tar.gz: b9f328146d02d36436a48385f1156a3a94ce9f321123220a2316a794ab253695
+  metadata.gz: 90c12f93d4750216ef84e046fe970f0ccc8edd7567943b9f4be02dc3828fd151
+  data.tar.gz: ef810d7d1599f9539474e87067c5972ef2aa3a6143452494e9d03857d64771e7
 SHA512:
-  metadata.gz: 21c6b94baa433cf5c944a96f2ef8829f36ec81b0a093fb1e6fb5c53537698df6adc936f71ff2856bc68411d9ccefbd143cb75ee703d8d2b5a171e940faada191
-  data.tar.gz: 7e84f101f485088c24e2afccaaa779f8cfdae3bb91798ec24f30199402bc3bbd5e142ece40a86de7845cb342b31d68be5c953383b0a0b081bf388b0574b25ab9
+  metadata.gz: 4a6ff236ecaabfd5e927a6f57f47d97ad6822dc647ce3531c57565adfc05fd592167decbabfa2ac3420d8294f193188103c7aa3a468b5f92dca7b18a92bb1b87
+  data.tar.gz: e34f78b58b73536faecf0bafb0f7a620bbbc3122c8fb8d1b526112b000055b78fe6d153ff0bf1bbd2300aaaffb3f5a6327fd7ef4248332b39ebb5a7c13fe94d5

data/VERSION

@@ -1 +1 @@
-0.26.2
+0.26.3

data/lib/ruby_indexer/lib/ruby_indexer/index.rb

@@ -195,12 +195,13 @@ module RubyIndexer
     end
 
     # Fuzzy searches index entries based on Jaro-Winkler similarity. If no query is provided, all entries are returned
-    #: (String? query) -> Array[Entry]
-    def fuzzy_search(query)
+    #: (String? query) ?{ (Entry) -> bool? } -> Array[Entry]
+    def fuzzy_search(query, &condition)
       unless query
         entries = @entries.filter_map do |_name, entries|
           next if entries.first.is_a?(Entry::SingletonClass)
 
+          entries = entries.select(&condition) if condition
           entries
         end
 
@@ -212,6 +213,9 @@ module RubyIndexer
       results = @entries.filter_map do |name, entries|
         next if entries.first.is_a?(Entry::SingletonClass)
 
+        entries = entries.select(&condition) if condition
+        next if entries.empty?
+
         similarity = DidYouMean::JaroWinkler.distance(name.gsub("::", "").downcase, normalized_query)
         [entries, -similarity] if similarity > ENTRY_SIMILARITY_THRESHOLD
       end

data/lib/ruby_lsp/base_server.rb

@@ -83,7 +83,7 @@ module RubyLsp
         # The following requests need to be executed in the main thread directly to avoid concurrency issues. Everything
         # else is pushed into the incoming queue
         case method
-        when "initialize", "initialized", "rubyLsp/diagnoseState"
+        when "initialize", "initialized", "rubyLsp/diagnoseState", "$/cancelRequest"
           process_message(message)
         when "shutdown"
           @global_state.synchronize do
@@ -154,20 +154,29 @@ module RubyLsp
     def new_worker
       Thread.new do
         while (message = @incoming_queue.pop)
-          id = message[:id]
-
-          # Check if the request was cancelled before trying to process it
-          @global_state.synchronize do
-            if id && @cancelled_requests.include?(id)
-              send_message(Result.new(id: id, response: nil))
-              @cancelled_requests.delete(id)
-              next
-            end
-          end
+          handle_incoming_message(message)
+        end
+      end
+    end
 
-          process_message(message)
+    #: (Hash[Symbol, untyped]) -> void
+    def handle_incoming_message(message)
+      id = message[:id]
+
+      # Check if the request was cancelled before trying to process it
+      @global_state.synchronize do
+        if id && @cancelled_requests.include?(id)
+          send_message(Error.new(
+            id: id,
+            code: Constant::ErrorCodes::REQUEST_CANCELLED,
+            message: "Request #{id} was cancelled",
+          ))
+          @cancelled_requests.delete(id)
+          return
         end
       end
+
+      process_message(message)
     end
 
     #: ((Result | Error | Notification | Request) message) -> void

data/lib/ruby_lsp/listeners/document_link.rb

@@ -2,6 +2,7 @@
 # frozen_string_literal: true
 
 require "ruby_lsp/requests/support/source_uri"
+require "ruby_lsp/requests/support/package_url"
 
 module RubyLsp
   module Listeners
@@ -102,19 +103,58 @@ module RubyLsp
         comment = @lines_to_comments[node.location.start_line - 1]
         return unless comment
 
-        match = comment.location.slice.match(%r{source://.*#\d+$})
+        match = comment.location.slice.match(%r{(source://.*#\d+|pkg:gem/.*#.*)$})
         return unless match
 
-        uri =  begin
-          URI(
-            match[0], #: as !nil
-          )
+        uri_string = match[0] #: as !nil
+
+        file_path, line_number = if uri_string.start_with?("pkg:gem/")
+          parse_package_url(uri_string)
+        else
+          parse_source_uri(uri_string)
+        end
+
+        return unless file_path
+
+        @response_builder << Interface::DocumentLink.new(
+          range: range_from_location(comment.location),
+          target: "file://#{file_path}##{line_number}",
+          tooltip: "Jump to #{file_path}##{line_number}",
+        )
+      end
+
+      #: (String uri_string) -> [String, String]?
+      def parse_package_url(uri_string)
+        purl = PackageURL.parse(uri_string) #: as PackageURL?
+        return unless purl
+
+        gem_version = resolve_version(purl.version, purl.name)
+        return if gem_version.nil?
+
+        path, line_number = purl.subpath.split(":", 2)
+        return unless path
+
+        gem_name = purl.name
+        return unless gem_name
+
+        file_path = self.class.gem_paths.dig(gem_name, gem_version, CGI.unescape(path))
+        return if file_path.nil?
+
+        [file_path, line_number]
+      rescue PackageURL::InvalidPackageURL
+        nil
+      end
+
+      #: (String uri_string) -> [String, String]?
+      def parse_source_uri(uri_string)
+        uri = begin
+          URI(uri_string)
         rescue URI::Error
           nil
         end #: as URI::Source?
         return unless uri
 
-        gem_version = resolve_version(uri)
+        gem_version = resolve_version(uri.gem_version, uri.gem_name)
         return if gem_version.nil?
 
         path = uri.path
@@ -126,28 +166,20 @@ module RubyLsp
         file_path = self.class.gem_paths.dig(gem_name, gem_version, CGI.unescape(path))
         return if file_path.nil?
 
-        @response_builder << Interface::DocumentLink.new(
-          range: range_from_location(comment.location),
-          target: "file://#{file_path}##{uri.line_number}",
-          tooltip: "Jump to #{file_path}##{uri.line_number}",
-        )
+        [file_path, uri.line_number || "0"]
       end
 
       # Try to figure out the gem version for a source:// link. The order of precedence is:
       # 1. The version in the URI
       # 2. The version in the RBI file name
       # 3. The version from the gemspec
-      #: (URI::Source uri) -> String?
-      def resolve_version(uri)
-        version = uri.gem_version
+      #: (String? version, String? gem_name) -> String?
+      def resolve_version(version, gem_name)
         return version unless version.nil? || version.empty?
 
         return @gem_version unless @gem_version.nil? || @gem_version.empty?
 
-        gem_name = uri.gem_name
-        return unless gem_name
-
-        GEM_TO_VERSION_MAP[gem_name]
+        GEM_TO_VERSION_MAP[gem_name.to_s]
       end
     end
   end

data/lib/ruby_lsp/requests/support/package_url.rb

@@ -0,0 +1,414 @@
+# typed: false
+# frozen_string_literal: true
+
+# This is a copy of the implementation from the `package_url` gem with the
+# following license. Original source can be found at:
+# https://github.com/package-url/packageurl-ruby/blob/main/lib/package_url.rb
+
+# MIT License
+#
+# Copyright (c) 2021 package-url
+#
+# Permission is hereby granted, free of charge, to any person obtaining a copy
+# of this software and associated documentation files (the "Software"), to deal
+# in the Software without restriction, including without limitation the rights
+# to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+# copies of the Software, and to permit persons to whom the Software is
+# furnished to do so, subject to the following conditions:
+#
+# The above copyright notice and this permission notice shall be included in all
+# copies or substantial portions of the Software.
+#
+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+# SOFTWARE.
+
+require "uri"
+
+# A package URL, or _purl_, is a URL string used to
+# identify and locate a software package in a mostly universal and uniform way
+# across programing languages, package managers, packaging conventions, tools,
+# APIs and databases.
+#
+# A purl is a URL composed of seven components:
+#
+# ```
+# scheme:type/namespace/name@version?qualifiers#subpath
+# ```
+#
+# For example,
+# the package URL for this Ruby package at version 0.1.0 is
+# `pkg:ruby/mattt/packageurl-ruby@0.1.0`.
+module RubyLsp
+  class PackageURL
+    # Raised when attempting to parse an invalid package URL string.
+    # @see #parse
+    class InvalidPackageURL < ArgumentError; end
+
+    # The URL scheme, which has a constant value of `"pkg"`.
+    def scheme
+      "pkg"
+    end
+
+    # The package type or protocol, such as `"gem"`, `"npm"`, and `"github"`.
+    attr_reader :type
+
+    # A name prefix, specific to the type of package.
+    # For example, an npm scope, a Docker image owner, or a GitHub user.
+    attr_reader :namespace
+
+    # The name of the package.
+    attr_reader :name
+
+    # The version of the package.
+    attr_reader :version
+
+    # Extra qualifying data for a package, specific to the type of package.
+    # For example, the operating system or architecture.
+    attr_reader :qualifiers
+
+    # An extra subpath within a package, relative to the package root.
+    attr_reader :subpath
+
+    # Constructs a package URL from its components
+    # @param type [String] The package type or protocol.
+    # @param namespace [String] A name prefix, specific to the type of package.
+    # @param name [String] The name of the package.
+    # @param version [String] The version of the package.
+    # @param qualifiers [Hash] Extra qualifying data for a package, specific to the type of package.
+    # @param subpath [String] An extra subpath within a package, relative to the package root.
+    def initialize(type:, name:, namespace: nil, version: nil, qualifiers: nil, subpath: nil)
+      raise ArgumentError, "type is required" if type.nil? || type.empty?
+      raise ArgumentError, "name is required" if name.nil? || name.empty?
+
+      @type = type.downcase
+      @namespace = namespace
+      @name = name
+      @version = version
+      @qualifiers = qualifiers
+      @subpath = subpath
+    end
+
+    # Creates a new PackageURL from a string.
+    # @param [String] string The package URL string.
+    # @raise [InvalidPackageURL] If the string is not a valid package URL.
+    # @return [PackageURL]
+    def self.parse(string)
+      components = {
+        type: nil,
+        namespace: nil,
+        name: nil,
+        version: nil,
+        qualifiers: nil,
+        subpath: nil,
+      }
+
+      # Split the purl string once from right on '#'
+      # - The left side is the remainder
+      # - Strip the right side from leading and trailing '/'
+      # - Split this on '/'
+      # - Discard any empty string segment from that split
+      # - Discard any '.' or '..' segment from that split
+      # - Percent-decode each segment
+      # - UTF-8-decode each segment if needed in your programming language
+      # - Join segments back with a '/'
+      # - This is the subpath
+      case string.rpartition("#")
+      in String => remainder, separator, String => subpath unless separator.empty?
+        subpath_components = []
+        subpath.split("/").each do |segment|
+          next if segment.empty? || segment == "." || segment == ".."
+
+          subpath_components << URI.decode_www_form_component(segment)
+        end
+
+        components[:subpath] = subpath_components.compact.join("/")
+
+        string = remainder
+      else
+        components[:subpath] = nil
+      end
+
+      # Split the remainder once from right on '?'
+      # - The left side is the remainder
+      # - The right side is the qualifiers string
+      # - Split the qualifiers on '&'. Each part is a key=value pair
+      # - For each pair, split the key=value once from left on '=':
+      # - The key is the lowercase left side
+      # - The value is the percent-decoded right side
+      # - UTF-8-decode the value if needed in your programming language
+      # - Discard any key/value pairs where the value is empty
+      # - If the key is checksums,
+      #   split the value on ',' to create a list of checksums
+      # - This list of key/value is the qualifiers object
+      case string.rpartition("?")
+      in String => remainder, separator, String => qualifiers unless separator.empty?
+        components[:qualifiers] = {}
+
+        qualifiers.split("&").each do |pair|
+          case pair.partition("=")
+          in String => key, separator, String => value unless separator.empty?
+            key = key.downcase
+            value = URI.decode_www_form_component(value)
+            next if value.empty?
+
+            components[:qualifiers][key] = case key
+            when "checksums"
+              value.split(",")
+            else
+              value
+            end
+          else
+            next
+          end
+        end
+
+        string = remainder
+      else
+        components[:qualifiers] = nil
+      end
+
+      # Split the remainder once from left on ':'
+      # - The left side lowercased is the scheme
+      # - The right side is the remainder
+      case string.partition(":")
+      in "pkg", separator, String => remainder unless separator.empty?
+        string = remainder
+      else
+        raise InvalidPackageURL, 'invalid or missing "pkg:" URL scheme'
+      end
+
+      # Strip the remainder from leading and trailing '/'
+      # Use gsub to remove ALL leading slashes instead of just one
+      string = string.gsub(%r{^/+}, "").delete_suffix("/")
+      # - Split this once from left on '/'
+      # - The left side lowercased is the type
+      # - The right side is the remainder
+      case string.partition("/")
+      in String => type, separator, remainder unless separator.empty?
+        components[:type] = type
+
+        string = remainder
+      else
+        raise InvalidPackageURL, "invalid or missing package type"
+      end
+
+      # Split the remainder once from right on '@'
+      # - The left side is the remainder
+      # - Percent-decode the right side. This is the version.
+      # - UTF-8-decode the version if needed in your programming language
+      # - This is the version
+      case string.rpartition("@")
+      in String => remainder, separator, String => version unless separator.empty?
+        components[:version] = URI.decode_www_form_component(version)
+
+        string = remainder
+      else
+        components[:version] = nil
+      end
+
+      # Split the remainder once from right on '/'
+      # - The left side is the remainder
+      # - Percent-decode the right side. This is the name
+      # - UTF-8-decode this name if needed in your programming language
+      # - Apply type-specific normalization to the name if needed
+      # - This is the name
+      case string.rpartition("/")
+      in String => remainder, separator, String => name unless separator.empty?
+        components[:name] = URI.decode_www_form_component(name)
+
+        # Split the remainder on '/'
+        # - Discard any empty segment from that split
+        # - Percent-decode each segment
+        # - UTF-8-decode the each segment if needed in your programming language
+        # - Apply type-specific normalization to each segment if needed
+        # - Join segments back with a '/'
+        # - This is the namespace
+        components[:namespace] = remainder.split("/").map { |s| URI.decode_www_form_component(s) }.compact.join("/")
+      in _, _, String => name
+        components[:name] = URI.decode_www_form_component(name)
+        components[:namespace] = nil
+      end
+
+      # Ensure type and name are not nil before creating the PackageURL instance
+      raise InvalidPackageURL, "missing package type" if components[:type].nil?
+      raise InvalidPackageURL, "missing package name" if components[:name].nil?
+
+      # Create a new PackageURL with validated components
+      type = components[:type] || ""  # This ensures type is never nil
+      name = components[:name] || ""  # This ensures name is never nil
+
+      new(
+        type: type,
+        name: name,
+        namespace: components[:namespace],
+        version: components[:version],
+        qualifiers: components[:qualifiers],
+        subpath: components[:subpath],
+      )
+    end
+
+    # Returns a hash containing the
+    # scheme, type, namespace, name, version, qualifiers, and subpath components
+    # of the package URL.
+    def to_h
+      {
+        scheme: scheme,
+        type: @type,
+        namespace: @namespace,
+        name: @name,
+        version: @version,
+        qualifiers: @qualifiers,
+        subpath: @subpath,
+      }
+    end
+
+    # Returns a string representation of the package URL.
+    # Package URL representations are created according to the instructions from
+    # https://github.com/package-url/purl-spec/blob/0b1559f76b79829e789c4f20e6d832c7314762c5/PURL-SPECIFICATION.rst#how-to-build-purl-string-from-its-components.
+    def to_s
+      # Start a purl string with the "pkg:" scheme as a lowercase ASCII string
+      purl = "pkg:"
+
+      # Append the type string to the purl as a lowercase ASCII string
+      # Append '/' to the purl
+
+      purl += @type
+      purl += "/"
+
+      # If the namespace is not empty:
+      # - Strip the namespace from leading and trailing '/'
+      # - Split on '/' as segments
+      # - Apply type-specific normalization to each segment if needed
+      # - UTF-8-encode each segment if needed in your programming language
+      # - Percent-encode each segment
+      # - Join the segments with '/'
+      # - Append this to the purl
+      # - Append '/' to the purl
+      # - Strip the name from leading and trailing '/'
+      # - Apply type-specific normalization to the name if needed
+      # - UTF-8-encode the name if needed in your programming language
+      # - Append the percent-encoded name to the purl
+      #
+      # If the namespace is empty:
+      # - Apply type-specific normalization to the name if needed
+      # - UTF-8-encode the name if needed in your programming language
+      # - Append the percent-encoded name to the purl
+      case @namespace
+      in String => namespace unless namespace.empty?
+        segments = []
+        @namespace.delete_prefix("/").delete_suffix("/").split("/").each do |segment|
+          next if segment.empty?
+
+          segments << URI.encode_www_form_component(segment)
+        end
+        purl += segments.join("/")
+
+        purl += "/"
+        purl += URI.encode_www_form_component(@name.delete_prefix("/").delete_suffix("/"))
+      else
+        purl += URI.encode_www_form_component(@name)
+      end
+
+      # If the version is not empty:
+      # - Append '@' to the purl
+      # - UTF-8-encode the version if needed in your programming language
+      # - Append the percent-encoded version to the purl
+      case @version
+      in String => version unless version.empty?
+        purl += "@"
+        purl += URI.encode_www_form_component(@version)
+      else
+        nil
+      end
+
+      # If the qualifiers are not empty and not composed only of key/value pairs
+      # where the value is empty:
+      # - Append '?' to the purl
+      # - Build a list from all key/value pair:
+      # - discard any pair where the value is empty.
+      # - UTF-8-encode each value if needed in your programming language
+      # - If the key is checksums and this is a list of checksums
+      #   join this list with a ',' to create this qualifier value
+      # - create a string by joining the lowercased key,
+      #   the equal '=' sign and the percent-encoded value to create a qualifier
+      # - sort this list of qualifier strings lexicographically
+      # - join this list of qualifier strings with a '&' ampersand
+      # - Append this string to the purl
+      case @qualifiers
+      in Hash => qualifiers unless qualifiers.empty?
+        list = []
+        qualifiers.each do |key, value|
+          next if value.empty?
+
+          list << case [key, value]
+          in "checksums", Array => checksums
+            "#{key.downcase}=#{checksums.join(",")}"
+          else
+            "#{key.downcase}=#{URI.encode_www_form_component(value)}"
+          end
+        end
+
+        unless list.empty?
+          purl += "?"
+          purl += list.sort.join("&")
+        end
+      else
+        nil
+      end
+
+      # If the subpath is not empty and not composed only of
+      # empty, '.' and '..' segments:
+      # - Append '#' to the purl
+      # - Strip the subpath from leading and trailing '/'
+      # - Split this on '/' as segments
+      # - Discard empty, '.' and '..' segments
+      # - Percent-encode each segment
+      # - UTF-8-encode each segment if needed in your programming language
+      # - Join the segments with '/'
+      # - Append this to the purl
+      case @subpath
+      in String => subpath unless subpath.empty?
+        segments = []
+        subpath.delete_prefix("/").delete_suffix("/").split("/").each do |segment|
+          next if segment.empty? || segment == "." || segment == ".."
+
+          # Custom encoding for URL fragment segments:
+          # 1. Explicitly encode % as %25 to prevent double-encoding issues
+          # 2. Percent-encode special characters according to URL fragment rules
+          # 3. This ensures proper round-trip encoding/decoding with the parse method
+          segments << segment.gsub(/%|[^A-Za-z0-9\-\._~]/) do |m|
+            m == "%" ? "%25" : format("%%%02X", m.ord)
+          end
+        end
+
+        unless segments.empty?
+          purl += "#"
+          purl += segments.join("/")
+        end
+      else
+        nil
+      end
+
+      purl
+    end
+
+    # Returns an array containing the
+    # scheme, type, namespace, name, version, qualifiers, and subpath components
+    # of the package URL.
+    def deconstruct
+      [scheme, @type, @namespace, @name, @version, @qualifiers, @subpath]
+    end
+
+    # Returns a hash containing the
+    # scheme, type, namespace, name, version, qualifiers, and subpath components
+    # of the package URL.
+    def deconstruct_keys(_keys)
+      to_h
+    end
+  end
+end

data/lib/ruby_lsp/requests/workspace_symbol.rb

@@ -20,17 +20,7 @@ module RubyLsp
       # @override
       #: -> Array[Interface::WorkspaceSymbol]
       def perform
-        @index.fuzzy_search(@query).filter_map do |entry|
-          uri = entry.uri
-          file_path = uri.full_path
-
-          # We only show symbols declared in the workspace
-          in_dependencies = file_path && !not_in_dependencies?(file_path)
-          next if in_dependencies
-
-          # We should never show private symbols when searching the entire workspace
-          next if entry.private?
-
+        fuzzy_search.filter_map do |entry|
           kind = kind_for_entry(entry)
           loc = entry.location
 
@@ -44,7 +34,7 @@ module RubyLsp
             container_name: container.join("::"),
             kind: kind,
             location: Interface::Location.new(
-              uri: uri.to_s,
+              uri: entry.uri.to_s,
               range:  Interface::Range.new(
                 start: Interface::Position.new(line: loc.start_line - 1, character: loc.start_column),
                 end: Interface::Position.new(line: loc.end_line - 1, character: loc.end_column),
@@ -53,6 +43,24 @@ module RubyLsp
           )
         end
       end
+
+      private
+
+      #: -> Array[RubyIndexer::Entry]
+      def fuzzy_search
+        @index.fuzzy_search(@query) do |entry|
+          file_path = entry.uri.full_path
+
+          # We only show symbols declared in the workspace
+          in_dependencies = file_path && !not_in_dependencies?(file_path)
+          next if in_dependencies
+
+          # We should never show private symbols when searching the entire workspace
+          next if entry.private?
+
+          true
+        end
+      end
     end
   end
 end

data/lib/ruby_lsp/setup_bundler.rb

@@ -270,12 +270,21 @@ module RubyLsp
     #: (Hash[String, String] env, ?force_install: bool) -> Hash[String, String]
     def run_bundle_install_directly(env, force_install: false)
       RubyVM::YJIT.enable if defined?(RubyVM::YJIT.enable)
+
+      # The should_bundle_update? check needs to run on the original Bundler environment, but everything else (like
+      # updating or running install) requires the modified environment. Here we compute the check ahead of time and
+      # merge the environment to ensure correct results.
+      #
+      # The symptoms of having these operations in the wrong order is seeing unwanted modifications in the application's
+      # main lockfile because we accidentally run update on the main bundle instead of the composed one.
+      needs_update = should_bundle_update?
+      ENV.merge!(env)
+
       return update(env) if @needs_update_path.exist?
 
       # The ENV can only be merged after checking if an update is required because we depend on the original value of
       # ENV["BUNDLE_GEMFILE"], which gets overridden after the merge
-      FileUtils.touch(@needs_update_path) if should_bundle_update?
-      ENV.merge!(env)
+      FileUtils.touch(@needs_update_path) if needs_update
 
       $stderr.puts("Ruby LSP> Checking if the composed bundle is satisfied...")
       missing_gems = bundle_check

data/lib/ruby_lsp/test_reporters/lsp_reporter.rb

@@ -35,10 +35,10 @@ module RubyLsp
       @io = begin
         # The environment variable is only used for tests. The extension always writes to the temporary file
         if port
-          TCPSocket.new("localhost", port)
+          socket(port)
         elsif File.exist?(port_db_path)
           db = JSON.load_file(port_db_path)
-          TCPSocket.new("localhost", db[Dir.pwd])
+          socket(db[Dir.pwd])
         else
           # For tests that don't spawn the TCP server
           require "stringio"
@@ -209,6 +209,14 @@ module RubyLsp
 
     private
 
+    #: (String) -> TCPSocket
+    def socket(port)
+      socket = TCPSocket.new("localhost", port)
+      socket.binmode
+      socket.sync = true
+      socket
+    end
+
     #: (String?, **untyped) -> void
     def send_message(method_name, **params)
       json_message = { method: method_name, params: params }.to_json

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ruby-lsp
 version: !ruby/object:Gem::Version
-  version: 0.26.2
+  version: 0.26.3
 platform: ruby
 authors:
 - Shopify
@@ -163,6 +163,7 @@ files:
 - lib/ruby_lsp/requests/support/annotation.rb
 - lib/ruby_lsp/requests/support/common.rb
 - lib/ruby_lsp/requests/support/formatter.rb
+- lib/ruby_lsp/requests/support/package_url.rb
 - lib/ruby_lsp/requests/support/rubocop_diagnostic.rb
 - lib/ruby_lsp/requests/support/rubocop_formatter.rb
 - lib/ruby_lsp/requests/support/rubocop_runner.rb