theme-check 1.8.0 → 1.9.0

data/lib/theme_check/version.rb

@@ -1,4 +1,4 @@
 # frozen_string_literal: true
 module ThemeCheck
-  VERSION = "1.8.0"
+  VERSION = "1.9.0"
 end

data/lib/theme_check.rb

@@ -4,6 +4,7 @@ require "liquid"
 require_relative "theme_check/version"
 require_relative "theme_check/bug"
 require_relative "theme_check/exceptions"
+require_relative "theme_check/schema_helper"
 require_relative "theme_check/theme_file_rewriter"
 require_relative "theme_check/theme_file"
 require_relative "theme_check/liquid_file"
@@ -24,10 +25,10 @@ require_relative "theme_check/regex_helpers"
 require_relative "theme_check/json_helpers"
 require_relative "theme_check/position_helper"
 require_relative "theme_check/position"
-require_relative "theme_check/language_server"
 require_relative "theme_check/checks"
 require_relative "theme_check/config"
 require_relative "theme_check/node"
+require_relative "theme_check/tags"
 require_relative "theme_check/liquid_node"
 require_relative "theme_check/html_node"
 require_relative "theme_check/offense"
@@ -38,11 +39,11 @@ require_relative "theme_check/string_helpers"
 require_relative "theme_check/storage"
 require_relative "theme_check/file_system_storage"
 require_relative "theme_check/in_memory_storage"
-require_relative "theme_check/tags"
 require_relative "theme_check/theme"
 require_relative "theme_check/corrector"
 require_relative "theme_check/liquid_visitor"
 require_relative "theme_check/html_visitor"
+require_relative "theme_check/language_server"
 
 Dir[__dir__ + "/theme_check/checks/*.rb"].each { |file| require file }
 
@@ -55,6 +56,10 @@ module ThemeCheck
     ENV["THEME_CHECK_DEBUG"] == "true"
   end
 
+  def self.debug_log_file
+    ENV["THEME_CHECK_DEBUG_LOG_FILE"]
+  end
+
   def self.with_liquid_c_disabled
     if defined?(Liquid::C)
       was_enabled = Liquid::C.enabled

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: theme-check
 version: !ruby/object:Gem::Version
-  version: 1.8.0
+  version: 1.9.0
 platform: ruby
 authors:
 - Marc-André Cournoyer
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-11-09 00:00:00.000000000 Z
+date: 2021-12-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: liquid
@@ -121,6 +121,7 @@ files:
 - docs/checks/remote_asset.md
 - docs/checks/required_directories.md
 - docs/checks/required_layout_theme_object.md
+- docs/checks/schema_json_format.md
 - docs/checks/space_inside_braces.md
 - docs/checks/syntax_error.md
 - docs/checks/template_length.md
@@ -133,6 +134,13 @@ files:
 - docs/checks/valid_json.md
 - docs/checks/valid_schema.md
 - docs/flamegraph.svg
+- docs/language_server/code-action-command-palette.png
+- docs/language_server/code-action-flow.png
+- docs/language_server/code-action-keyboard.png
+- docs/language_server/code-action-light-bulb.png
+- docs/language_server/code-action-problem.png
+- docs/language_server/code-action-quickfix.png
+- docs/language_server/how_to_correct_code_with_code_actions_and_execute_command.md
 - docs/preview.png
 - exe/theme-check
 - exe/theme-check-language-server
@@ -173,6 +181,7 @@ files:
 - lib/theme_check/checks/remote_asset.rb
 - lib/theme_check/checks/required_directories.rb
 - lib/theme_check/checks/required_layout_theme_object.rb
+- lib/theme_check/checks/schema_json_format.rb
 - lib/theme_check/checks/space_inside_braces.rb
 - lib/theme_check/checks/syntax_error.rb
 - lib/theme_check/checks/template_length.rb
@@ -203,6 +212,11 @@ files:
 - lib/theme_check/language_server.rb
 - lib/theme_check/language_server/bridge.rb
 - lib/theme_check/language_server/channel.rb
+- lib/theme_check/language_server/client_capabilities.rb
+- lib/theme_check/language_server/code_action_engine.rb
+- lib/theme_check/language_server/code_action_provider.rb
+- lib/theme_check/language_server/code_action_providers/quickfix_code_action_provider.rb
+- lib/theme_check/language_server/code_action_providers/source_fix_all_code_action_provider.rb
 - lib/theme_check/language_server/completion_engine.rb
 - lib/theme_check/language_server/completion_helper.rb
 - lib/theme_check/language_server/completion_provider.rb
@@ -210,15 +224,22 @@ files:
 - lib/theme_check/language_server/completion_providers/object_completion_provider.rb
 - lib/theme_check/language_server/completion_providers/render_snippet_completion_provider.rb
 - lib/theme_check/language_server/completion_providers/tag_completion_provider.rb
+- lib/theme_check/language_server/configuration.rb
 - lib/theme_check/language_server/constants.rb
+- lib/theme_check/language_server/diagnostic.rb
 - lib/theme_check/language_server/diagnostics_engine.rb
-- lib/theme_check/language_server/diagnostics_tracker.rb
+- lib/theme_check/language_server/diagnostics_manager.rb
+- lib/theme_check/language_server/document_change_corrector.rb
 - lib/theme_check/language_server/document_link_engine.rb
 - lib/theme_check/language_server/document_link_provider.rb
 - lib/theme_check/language_server/document_link_providers/asset_document_link_provider.rb
 - lib/theme_check/language_server/document_link_providers/include_document_link_provider.rb
 - lib/theme_check/language_server/document_link_providers/render_document_link_provider.rb
 - lib/theme_check/language_server/document_link_providers/section_document_link_provider.rb
+- lib/theme_check/language_server/execute_command_engine.rb
+- lib/theme_check/language_server/execute_command_provider.rb
+- lib/theme_check/language_server/execute_command_providers/correction_execute_command_provider.rb
+- lib/theme_check/language_server/execute_command_providers/run_checks_execute_command_provider.rb
 - lib/theme_check/language_server/handler.rb
 - lib/theme_check/language_server/io_messenger.rb
 - lib/theme_check/language_server/messenger.rb
@@ -227,6 +248,7 @@ files:
 - lib/theme_check/language_server/tokens.rb
 - lib/theme_check/language_server/uri_helper.rb
 - lib/theme_check/language_server/variable_lookup_finder.rb
+- lib/theme_check/language_server/versioned_in_memory_storage.rb
 - lib/theme_check/liquid_check.rb
 - lib/theme_check/liquid_file.rb
 - lib/theme_check/liquid_node.rb
@@ -242,6 +264,7 @@ files:
 - lib/theme_check/regex_helpers.rb
 - lib/theme_check/releaser.rb
 - lib/theme_check/remote_asset_file.rb
+- lib/theme_check/schema_helper.rb
 - lib/theme_check/shopify_liquid.rb
 - lib/theme_check/shopify_liquid/deprecated_filter.rb
 - lib/theme_check/shopify_liquid/filter.rb

data/lib/theme_check/language_server/diagnostics_tracker.rb

@@ -1,66 +0,0 @@
-# frozen_string_literal: true
-require "logger"
-
-module ThemeCheck
-  module LanguageServer
-    class DiagnosticsTracker
-      def initialize
-        @previously_reported_files = Set.new
-        @single_files_offenses = {}
-        @first_run = true
-      end
-
-      def first_run?
-        @first_run
-      end
-
-      def build_diagnostics(offenses, analyzed_files: nil)
-        reported_files = Set.new
-        new_single_file_offenses = {}
-        analyzed_files = analyzed_files.map { |path| Pathname.new(path) } if analyzed_files
-
-        offenses.group_by(&:theme_file).each do |theme_file, template_offenses|
-          next unless theme_file
-          reported_offenses = template_offenses
-          previous_offenses = @single_files_offenses[theme_file.path]
-          if analyzed_files.nil? || analyzed_files.include?(theme_file.path)
-            # We re-analyzed the file, so we know the template_offenses are update to date.
-            reported_single_file_offenses = reported_offenses.select(&:single_file?)
-            if reported_single_file_offenses.any?
-              new_single_file_offenses[theme_file.path] = reported_single_file_offenses
-            end
-          elsif previous_offenses
-            # Merge in the previous ones, if some
-            reported_offenses |= previous_offenses
-          end
-          yield theme_file.path, reported_offenses
-          reported_files << theme_file.path
-        end
-
-        @single_files_offenses.each do |path, _|
-          # Already reported above, skip
-          next if reported_files.include?(path)
-
-          if analyzed_files.nil? || analyzed_files.include?(path)
-            # We re-analyzed this file, if it was not reported, all offenses in it got fixed
-            yield path, []
-            new_single_file_offenses[path] = nil
-          end
-          # NOTE: No need to re-report previous offenses as LSP should keep them around until
-          # we clear them.
-          reported_files << path
-        end
-
-        # Publish diagnostics with empty array if all issues on a previously reported theme_file
-        # have been fixed.
-        (@previously_reported_files - reported_files).each do |path|
-          yield path, []
-        end
-
-        @previously_reported_files = reported_files
-        @single_files_offenses.merge!(new_single_file_offenses)
-        @first_run = false
-      end
-    end
-  end
-end