theme-check 0.2.2 → 0.4.0

data/lib/theme_check/template.rb

@@ -3,16 +3,29 @@ require "pathname"
 
 module ThemeCheck
   class Template
-    attr_reader :path
+    def initialize(relative_path, storage)
+      @storage = storage
+      @relative_path = relative_path
+    end
 
-    def initialize(path, root)
-      @path = Pathname(path)
-      @root = Pathname(root)
-      @updated = false
+    def path
+      @storage.path(@relative_path)
     end
 
     def relative_path
-      @path.relative_path_from(@root)
+      @relative_pathname ||= Pathname.new(@relative_path)
+    end
+
+    def source
+      @source ||= @storage.read(@relative_path)
+    end
+
+    def write
+      content = updated_content
+      if source != content
+        @storage.write(@relative_path, content)
+        @source = content
+      end
     end
 
     def name
@@ -31,15 +44,17 @@ module ThemeCheck
       name.start_with?('snippets')
     end
 
-    def source
-      @source ||= @path.read
-    end
-
     def lines
       # Retain trailing newline character
       @lines ||= source.split("\n", -1)
     end
 
+    # Not entirely obvious but lines is mutable, corrections are to be
+    # applied on @lines.
+    def updated_content
+      lines.join("\n")
+    end
+
     def excerpt(line)
       lines[line - 1].strip
     end
@@ -65,18 +80,8 @@ module ThemeCheck
       parse.root
     end
 
-    def update!
-      @updated = true
-    end
-
-    def write
-      if @updated
-        @path.write(lines.join("\n"))
-      end
-    end
-
     def ==(other)
-      other.is_a?(Template) && @path == other.path
+      other.is_a?(Template) && relative_path == other.relative_path
     end
 
     def self.parse(source)

data/lib/theme_check/theme.rb

@@ -4,18 +4,27 @@ require "pathname"
 module ThemeCheck
   class Theme
     DEFAULT_LOCALE_REGEXP = %r{^locales/(.*)\.default$}
-    attr_reader :root
+    LIQUID_REGEX = /\.liquid$/i
+    JSON_REGEX = /\.json$/i
 
-    def initialize(root)
-      @root = Pathname.new(root)
+    def initialize(storage)
+      @storage = storage
     end
 
     def liquid
-      @liquid ||= @root.glob("**/*.liquid").map { |path| Template.new(path, @root) }
+      @liquid ||= @storage.files
+        .select { |path| LIQUID_REGEX.match?(path) }
+        .map { |path| Template.new(path, @storage) }
     end
 
     def json
-      @json ||= @root.glob("**/*.json").map { |path| JsonFile.new(path, @root) }
+      @json ||= @storage.files
+        .select { |path| JSON_REGEX.match?(path) }
+        .map { |path| JsonFile.new(path, @storage) }
+    end
+
+    def directories
+      @storage.directories
     end
 
     def default_locale_json
@@ -52,9 +61,5 @@ module ThemeCheck
     def snippets
       liquid.select(&:snippet?)
     end
-
-    def directories
-      @directories ||= @root.glob('*').select { |f| File.directory?(f) }.map { |f| f.relative_path_from(@root) }
-    end
   end
 end

data/lib/theme_check/version.rb

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

data/lib/theme_check/visitor.rb

@@ -6,12 +6,15 @@ module ThemeCheck
     end
 
     def visit_template(template)
+      @disabled_checks = DisabledChecks.new
       visit(Node.new(template.root, nil, template))
     rescue Liquid::Error => exception
       exception.template_name = template.name
       call_checks(:on_error, exception)
     end
 
+    private
+
     def visit(node)
       call_checks(:on_node, node)
       call_checks(:on_tag, node) if node.tag?
@@ -22,16 +25,24 @@ module ThemeCheck
         call_checks(:after_tag, node) if node.tag?
         call_checks(:after_node, node)
       end
-    end
 
-    private
+      @disabled_checks.update(node) if node.comment?
+    end
 
     def visit_children(node)
       node.children.each { |child| visit(child) }
     end
 
     def call_checks(method, *args)
-      @checks.call(method, *args)
+      checks.call(method, *args)
+    end
+
+    def checks
+      return @checks unless @disabled_checks.any?
+
+      return @checks.always_enabled if @disabled_checks.all_disabled?
+
+      @checks.except_for(@disabled_checks)
     end
   end
 end

data/packaging/homebrew/theme_check.base.rb

@@ -61,12 +61,16 @@ class ThemeCheck < Formula
       ENV['PATH'] = ENV['PATH'].sub(HOMEBREW_SHIMS_PATH.to_s, '/usr/local/bin')
     end
 
-    system("#{ruby_bin}/gem", "install", cached_download,
-             "--no-document",
-             "--no-wrapper",
-             "--no-user-install",
-             "--install-dir", prefix,
-             "--bindir", bin)
+    system(
+      "#{ruby_bin}/gem",
+      "install",
+      cached_download,
+      "--no-document",
+      "--no-wrapper",
+      "--no-user-install",
+      "--install-dir", prefix,
+      "--bindir", bin
+    )
 
     raise "gem install 'theme-check' failed with status #{$CHILD_STATUS.exitstatus}" unless $CHILD_STATUS.success?
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: theme-check
 version: !ruby/object:Gem::Version
-  version: 0.2.2
+  version: 0.4.0
 platform: ruby
 authors:
 - Marc-André Cournoyer
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-01-22 00:00:00.000000000 Z
+date: 2021-02-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: liquid
@@ -65,18 +65,22 @@ files:
 - ".github/workflows/theme-check.yml"
 - ".gitignore"
 - ".rubocop.yml"
+- CHANGELOG.md
 - CODE_OF_CONDUCT.md
 - CONTRIBUTING.md
 - Gemfile
 - Guardfile
 - LICENSE.md
 - README.md
+- RELEASING.md
 - Rakefile
 - bin/liquid-server
 - config/default.yml
 - data/shopify_liquid/deprecated_filters.yml
 - data/shopify_liquid/filters.yml
 - data/shopify_liquid/objects.yml
+- data/shopify_liquid/plus_objects.yml
+- data/shopify_liquid/tags.yml
 - dev.yml
 - docs/preview.png
 - exe/theme-check
@@ -91,9 +95,11 @@ files:
 - lib/theme_check/checks/liquid_tag.rb
 - lib/theme_check/checks/matching_schema_translations.rb
 - lib/theme_check/checks/matching_translations.rb
+- lib/theme_check/checks/missing_enable_comment.rb
 - lib/theme_check/checks/missing_required_template_files.rb
 - lib/theme_check/checks/missing_template.rb
 - lib/theme_check/checks/nested_snippet.rb
+- lib/theme_check/checks/parser_blocking_javascript.rb
 - lib/theme_check/checks/required_directories.rb
 - lib/theme_check/checks/required_layout_theme_object.rb
 - lib/theme_check/checks/space_inside_braces.rb
@@ -111,12 +117,24 @@ files:
 - lib/theme_check/cli.rb
 - lib/theme_check/config.rb
 - lib/theme_check/corrector.rb
+- lib/theme_check/disabled_checks.rb
+- lib/theme_check/file_system_storage.rb
+- lib/theme_check/in_memory_storage.rb
 - lib/theme_check/json_check.rb
 - lib/theme_check/json_file.rb
 - lib/theme_check/json_helpers.rb
 - lib/theme_check/language_server.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
+- lib/theme_check/language_server/completion_providers/filter_completion_provider.rb
+- lib/theme_check/language_server/completion_providers/object_completion_provider.rb
+- lib/theme_check/language_server/completion_providers/tag_completion_provider.rb
 - lib/theme_check/language_server/handler.rb
+- lib/theme_check/language_server/position_helper.rb
+- lib/theme_check/language_server/protocol.rb
 - lib/theme_check/language_server/server.rb
+- lib/theme_check/language_server/tokens.rb
 - lib/theme_check/liquid_check.rb
 - lib/theme_check/locale_diff.rb
 - lib/theme_check/node.rb
@@ -128,6 +146,8 @@ files:
 - lib/theme_check/shopify_liquid/deprecated_filter.rb
 - lib/theme_check/shopify_liquid/filter.rb
 - lib/theme_check/shopify_liquid/object.rb
+- lib/theme_check/shopify_liquid/tag.rb
+- lib/theme_check/storage.rb
 - lib/theme_check/tags.rb
 - lib/theme_check/template.rb
 - lib/theme_check/theme.rb