theme-check 1.11.0 → 1.12.1

data/lib/theme_check/language_server.rb

@@ -1,4 +1,5 @@
 # frozen_string_literal: true
+
 require_relative "language_server/protocol"
 require_relative "language_server/constants"
 require_relative "language_server/configuration"
@@ -7,9 +8,19 @@ require_relative "language_server/messenger"
 require_relative "language_server/io_messenger"
 require_relative "language_server/bridge"
 require_relative "language_server/uri_helper"
+require_relative "language_server/type_helper"
 require_relative "language_server/server"
 require_relative "language_server/tokens"
+require_relative "language_server/variable_lookup_finder/potential_lookup"
+require_relative "language_server/variable_lookup_finder/tolerant_parser"
+require_relative "language_server/variable_lookup_finder/assignments_finder/node_handler"
+require_relative "language_server/variable_lookup_finder/assignments_finder/scope_visitor"
+require_relative "language_server/variable_lookup_finder/assignments_finder/scope"
+require_relative "language_server/variable_lookup_finder/assignments_finder"
+require_relative "language_server/variable_lookup_finder/constants"
+require_relative "language_server/variable_lookup_finder/liquid_fixer"
 require_relative "language_server/variable_lookup_finder"
+require_relative "language_server/variable_lookup_traverser"
 require_relative "language_server/diagnostic"
 require_relative "language_server/diagnostics_manager"
 require_relative "language_server/diagnostics_engine"
@@ -17,6 +28,7 @@ require_relative "language_server/document_change_corrector"
 require_relative "language_server/versioned_in_memory_storage"
 require_relative "language_server/client_capabilities"
 
+require_relative "language_server/completion_context"
 require_relative "language_server/completion_helper"
 require_relative "language_server/completion_provider"
 require_relative "language_server/completion_engine"

data/lib/theme_check/remote_asset_file.rb

@@ -31,13 +31,7 @@ module ThemeCheck
       return if @uri.nil?
       return @content unless @content.nil?
 
-      res = Net::HTTP.start(@uri.hostname, @uri.port, use_ssl: @uri.scheme == 'https') do |http|
-        req = Net::HTTP::Get.new(@uri)
-        req['Accept-Encoding'] = 'gzip, deflate, br'
-        http.request(req)
-      end
-
-      @content = res.body
+      @content = request(@uri)
 
     rescue OpenSSL::SSL::SSLError, Zlib::StreamError, *NET_HTTP_EXCEPTIONS
       @contents = ''
@@ -47,5 +41,17 @@ module ThemeCheck
       return if @uri.nil?
       @gzipped_size ||= content.bytesize
     end
+
+    private
+
+    def request(uri)
+      res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == 'https') do |http|
+        req = Net::HTTP::Get.new(uri)
+        req['Accept-Encoding'] = 'gzip, deflate, br'
+        http.request(req)
+      end
+
+      res.body
+    end
   end
 end

data/lib/theme_check/shopify_liquid/deprecated_filter.rb

@@ -17,7 +17,7 @@ module ThemeCheck
       private
 
       def all
-        @all ||= YAML.load(File.read("#{__dir__}/../../../data/shopify_liquid/deprecated_filters.yml"))
+        @all ||= SourceIndex.deprecated_filters
           .values
           .each_with_object({}) do |filters, acc|
           filters.each do |(filter, alternatives)|

data/lib/theme_check/shopify_liquid/documentation/markdown_template.rb

@@ -0,0 +1,51 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class Documentation
+      class MarkdownTemplate
+        MARKDOWN_RELATIVE_LINK = %r{(\[([^\[]+)\]\((/[^\)]+)\))*}
+
+        def render(entry)
+          [
+            title(entry),
+            body(entry),
+          ].reject(&:empty?).join("\n")
+        end
+
+        private
+
+        def title(entry)
+          "### #{entry.name}"
+        end
+
+        def body(entry)
+          [entry.deprecation_reason, entry.summary, entry.description]
+            .reject(&:nil?)
+            .reject(&:empty?)
+            .join(horizontal_rule)
+            .tap { |body| break(patch_urls!(body)) }
+        end
+
+        def horizontal_rule
+          "\n\n---\n\n"
+        end
+
+        def patch_urls!(body)
+          body.gsub(MARKDOWN_RELATIVE_LINK) do |original_link|
+            match = Regexp.last_match
+
+            text = match[2]
+            path = match[3]
+
+            if text && path
+              "[#{text}](https://shopify.dev#{path})"
+            else
+              original_link
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/documentation.rb

@@ -0,0 +1,44 @@
+# frozen_string_literal: true
+
+require_relative 'documentation/markdown_template'
+
+module ThemeCheck
+  module ShopifyLiquid
+    class Documentation
+      class << self
+        def filter_doc(filter_name)
+          render_doc(SourceIndex.filters.find { |entry| entry.name == filter_name })
+        end
+
+        def object_doc(object_name)
+          render_doc(SourceIndex.objects.find { |entry| entry.name == object_name })
+        end
+
+        def tag_doc(tag_name)
+          render_doc(SourceIndex.tags.find { |entry| entry.name == tag_name })
+        end
+
+        def object_property_doc(object_name, property_name)
+          property_entry = SourceIndex
+            .objects
+            .find { |entry| entry.name == object_name }
+            &.properties
+            &.find { |prop| prop.name == property_name }
+
+          render_doc(property_entry)
+        end
+
+        def render_doc(entry)
+          return nil unless entry
+          markdown_template.render(entry)
+        end
+
+        private
+
+        def markdown_template
+          @markdown_template ||= MarkdownTemplate.new
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/filter.rb

@@ -6,10 +6,42 @@ module ThemeCheck
     module Filter
       extend self
 
+      LABELS_NOT_IN_SOURCE_INDEX = [
+        "h",
+        "installments_pricing",
+        "sentence",
+        "t",
+        "app_block_path_for",
+        "dev_shop?",
+        "app_extension_path_for",
+        "global_block_type?",
+        "app_block_path?",
+        "app_extension_path?",
+        "app_snippet_path?",
+        "registration_uuid_from",
+        "handle_from",
+        "camelcase",
+        "format_code",
+        "handle",
+        "encode_url_component",
+        "recover_password_link",
+        "delete_customer_address_link",
+        "edit_customer_address_link",
+        "cancel_customer_order_link",
+        "unit",
+        "weight",
+        "paragraphize",
+        "excerpt",
+        "pad_spaces",
+        "distance_from",
+        "theme_url",
+        "link_to_theme",
+        "_online_store_editor_live_setting",
+        "debug",
+      ]
+
       def labels
-        @labels ||= YAML.load(File.read("#{__dir__}/../../../data/shopify_liquid/filters.yml"))
-          .values
-          .flatten
+        @labels ||= SourceIndex.filters.map(&:name) + LABELS_NOT_IN_SOURCE_INDEX
       end
     end
   end

data/lib/theme_check/shopify_liquid/object.rb

@@ -6,16 +6,21 @@ module ThemeCheck
     module Object
       extend self
 
+      LABELS_NOT_IN_SOURCE_INDEX = [
+        "customer_address",
+        "product_variant",
+      ].freeze
+
       def labels
-        @labels ||= YAML.load(File.read("#{__dir__}/../../../data/shopify_liquid/objects.yml"))
+        @labels ||= SourceIndex.objects.map(&:name) + LABELS_NOT_IN_SOURCE_INDEX
       end
 
       def plus_labels
-        @plus_labels ||= YAML.load(File.read("#{__dir__}/../../../data/shopify_liquid/plus_objects.yml"))
+        @plus_labels ||= SourceIndex.plus_labels
       end
 
       def theme_app_extension_labels
-        @theme_app_extension_labels ||= YAML.load(File.read("#{__dir__}/../../../data/shopify_liquid/theme_app_extension_objects.yml"))
+        @theme_app_extension_labels ||= SourceIndex.theme_app_extension_labels
       end
     end
   end

data/lib/theme_check/shopify_liquid/source_index/base_entry.rb

@@ -0,0 +1,60 @@
+# frozen_string_literal: true
+
+require "forwardable"
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class BaseEntry
+        extend Forwardable
+
+        attr_reader :hash
+
+        def_delegators :return_type_instance, :generic_type?, :array_type?, :array_type, :to_s
+
+        def initialize(hash = {})
+          @hash = hash || {}
+          @return_type = nil
+        end
+
+        def name
+          hash['name']
+        end
+
+        def summary
+          hash['summary'] || ''
+        end
+
+        def description
+          hash['description'] || ''
+        end
+
+        def deprecated?
+          hash['deprecated']
+        end
+
+        def deprecation_reason
+          return nil unless deprecated?
+
+          hash['deprecation_reason'] || nil
+        end
+
+        attr_writer :return_type
+
+        def return_type
+          @return_type || to_s
+        end
+
+        def return_type_instance
+          ReturnTypeEntry.new(return_type_hash)
+        end
+
+        private
+
+        def return_type_hash
+          hash['return_type']&.first
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/base_state.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class BaseState
+        class << self
+          def mark_outdated
+            @up_to_date = false
+          end
+
+          def mark_up_to_date
+            @up_to_date = true
+          end
+
+          def outdated?
+            @up_to_date == false
+          end
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/filter_entry.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class FilterEntry < BaseEntry
+        def parameters
+          (hash['parameters'] || [])
+            .map { |hash| ParameterEntry.new(hash) }
+        end
+
+        def input_type
+          hash['syntax'].split(' | ')[0]
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/filter_state.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class FilterState < BaseState
+        @up_to_date = false
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/object_entry.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class ObjectEntry < BaseEntry
+        def properties
+          (hash['properties'] || [])
+            .map { |hash| PropertyEntry.new(hash) }
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/object_state.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class ObjectState < BaseState
+        @up_to_date = false
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/parameter_entry.rb

@@ -0,0 +1,21 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class ParameterEntry < BaseEntry
+        def summary
+          nil
+        end
+
+        private
+
+        def return_type_hash
+          {
+            'type' => (hash['types'] || ['untyped']).first,
+          }
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/property_entry.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class PropertyEntry < BaseEntry; end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/return_type_entry.rb

@@ -0,0 +1,37 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class ReturnTypeEntry < BaseEntry
+        def summary
+          nil
+        end
+
+        def to_s
+          hash['type']
+        end
+
+        def generic_type?
+          hash['type'] == 'generic'
+        end
+
+        def array_type?
+          !array_type.nil? && !array_type.empty?
+        end
+
+        def array_type
+          hash['array_value']
+        end
+
+        private
+
+        def return_type_hash
+          {
+            'type' => "type<#{self}>",
+          }
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/tag_entry.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class TagEntry < BaseEntry
+        def parameters
+          (hash['parameters'] || [])
+            .map { |hash| ParameterEntry.new(hash) }
+        end
+
+        def return_type_hash
+          {
+            'type' => "tag<#{name}>",
+          }
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index/tag_state.rb

@@ -0,0 +1,11 @@
+# frozen_string_literal: true
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class TagState < BaseState
+        @up_to_date = false
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_index.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+require 'json'
+require 'pathname'
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceIndex
+      class << self
+        def filters
+          @filters = nil if FilterState.outdated?
+
+          @filters ||= FilterState.mark_up_to_date &&
+            load_file(:filters)
+              .map { |hash| FilterEntry.new(hash) }
+        end
+
+        def objects
+          @objects = nil if ObjectState.outdated?
+
+          @objects ||= ObjectState.mark_up_to_date &&
+            load_file(:objects)
+              .concat(built_in_objects)
+              .filter_map do |hash|
+                next if (theme_app_extension_labels + labels_only_exposed_in_certain_contexts).include?(hash['name'])
+
+                ObjectEntry.new(hash)
+              end
+        end
+
+        def tags
+          @tags = nil if TagState.outdated?
+
+          @tags ||= TagState.mark_up_to_date &&
+            load_file(:tags)
+              .map { |hash| TagEntry.new(hash) }
+        end
+
+        def plus_labels
+          @plus_objects ||= load_file("../plus_labels")
+        end
+
+        def theme_app_extension_labels
+          @theme_app_extension_labels ||= load_file("../theme_app_extension_labels")
+        end
+
+        def labels_only_exposed_in_certain_contexts
+          ['robots'].freeze
+        end
+
+        def deprecated_filters
+          @deprecated_filters ||= load_file("../deprecated_filters")
+        end
+
+        private
+
+        def load_file(file_name)
+          read_json(local_path!(file_name))
+        end
+
+        def local_path!(file_name)
+          SourceManager.download unless SourceManager.has_required_files?
+          SourceManager.local_path(file_name)
+        end
+
+        def read_json(path)
+          JSON.parse(path.read)
+        end
+
+        def built_in_objects
+          load_file('../built_in_liquid_objects')
+        end
+      end
+    end
+  end
+end

data/lib/theme_check/shopify_liquid/source_manager.rb

@@ -0,0 +1,111 @@
+# frozen_string_literal: true
+
+require 'net/http'
+require 'pathname'
+require 'tmpdir'
+
+module ThemeCheck
+  module ShopifyLiquid
+    class SourceManager
+      REQUIRED_FILE_NAMES = [:filters, :objects, :tags, :latest].freeze
+
+      class << self
+        def download_or_refresh_files(destination = default_destination)
+          if has_required_files?(destination)
+            refresh(destination)
+          else
+            download(destination)
+          end
+        end
+
+        def download(destination = default_destination)
+          Dir.mkdir(destination) unless destination.exist?
+
+          REQUIRED_FILE_NAMES.each do |file_name|
+            download_file(local_path(file_name, destination), remote_path(file_name))
+          end
+        end
+
+        def refresh(destination = default_destination)
+          refresh_threads << Thread.new { refresh_thread(destination) }
+        end
+
+        def local_path(file_name, destination = default_destination)
+          destination + "#{file_name}.json"
+        end
+
+        def has_required_files?(destination = default_destination)
+          REQUIRED_FILE_NAMES.all? { |file_name| local_path(file_name, destination).exist? }
+        end
+
+        def wait_downloads
+          refresh_threads.each(&:join)
+        end
+
+        private
+
+        def refresh_thread(destination)
+          return unless refresh_needed?(destination)
+
+          Dir.mktmpdir do |tmp_dir|
+            download(Pathname.new(tmp_dir))
+
+            FileUtils.cp_r("#{tmp_dir}/.", destination)
+
+            mark_all_indexes_outdated
+          end
+        end
+
+        def refresh_needed?(destination)
+          local_latest_content = local_path(:latest, destination).read
+          remote_latest_content = open_uri(remote_path(:latest))
+
+          revision(local_latest_content) != revision(remote_latest_content)
+        end
+
+        def revision(json_content)
+          # Raise an error if revision isn't found to avoid returning nil
+          JSON.parse(json_content).fetch('revision')
+        end
+
+        def remote_path(file_name)
+          "https://raw.githubusercontent.com/Shopify/theme-liquid-docs/main/data/#{file_name}.json"
+        end
+
+        def download_file(local_path, remote_uri)
+          File.open(local_path, "wb") do |file|
+            content = open_uri(remote_uri)
+            file.write(content)
+          end
+        end
+
+        def mark_all_indexes_outdated
+          SourceIndex::FilterState.mark_outdated
+          SourceIndex::ObjectState.mark_outdated
+          SourceIndex::TagState.mark_outdated
+        end
+
+        # State
+
+        def default_destination
+          @default_destination ||= Pathname.new("#{__dir__}/../../../data/shopify_liquid/documentation")
+        end
+
+        def refresh_threads
+          @refresh_threads ||= []
+        end
+
+        def open_uri(uri_str)
+          uri = URI.parse(uri_str)
+
+          res = Net::HTTP.start(uri.hostname, uri.port, use_ssl: uri.scheme == 'https') do |http|
+            req = Net::HTTP::Get.new(uri)
+            http.request(req)
+          end
+
+          res.body
+        end
+      end
+    end
+  end
+end