platformos-check 0.4.14 → 0.5.0

data/data/platformos_liquid/documentation/latest.json

@@ -1,2 +1,2 @@
 
-{"revision":"57d415b98a26ca47bb6449b1433e8f009a3ca3b9"}
+{"revision":"1a08535d872a31f9df129d72df20919ca47309a6"}

data/lib/platformos_check/analyzer.rb

@@ -53,13 +53,11 @@ module PlatformosCheck
       liquid_visitor = LiquidVisitor.new(@liquid_checks, @disabled_checks)
       html_visitor = HtmlVisitor.new(@html_checks)
 
-      PlatformosCheck.with_liquid_c_disabled do
-        @platformos_app.liquid.each_with_index do |liquid_file, i|
-          yield(liquid_file.relative_path.to_s, i, total_file_count) if block_given?
-          liquid_visitor.visit_liquid_file(liquid_file)
-          html_visitor.visit_liquid_file(liquid_file)
-          @warnings[liquid_file.path] = liquid_file.warnings if @store_warnings && !liquid_file.warnings&.empty?
-        end
+      @platformos_app.liquid.each_with_index do |liquid_file, i|
+        yield(liquid_file.relative_path.to_s, i, total_file_count) if block_given?
+        liquid_visitor.visit_liquid_file(liquid_file)
+        html_visitor.visit_liquid_file(liquid_file)
+        @warnings[liquid_file.path] = liquid_file.warnings if @store_warnings && !liquid_file.warnings&.empty?
       end
 
       @platformos_app.yaml.each_with_index do |yaml_file, i|
@@ -87,39 +85,37 @@ module PlatformosCheck
     def analyze_files(files, only_single_file: false)
       reset
 
-      PlatformosCheck.with_liquid_c_disabled do
-        total = files.size
-        offset = 0
-
-        unless only_single_file
-          # Call all checks that run on the whole platformos_app
-          liquid_visitor = LiquidVisitor.new(@liquid_checks.whole_platformos_app, @disabled_checks)
-          html_visitor = HtmlVisitor.new(@html_checks.whole_platformos_app)
-          total += total_file_count
-          offset = total_file_count
-          @platformos_app.liquid.each_with_index do |liquid_file, i|
-            yield(liquid_file.relative_path.to_s, i, total) if block_given?
-            liquid_visitor.visit_liquid_file(liquid_file)
-            html_visitor.visit_liquid_file(liquid_file)
-          end
-
-          @platformos_app.yaml.each_with_index do |yaml_file, i|
-            yield(yaml_file.relative_path.to_s, liquid_file_count + i, total) if block_given?
-            @yaml_checks.whole_platformos_app.call(:on_file, yaml_file)
-          end
+      total = files.size
+      offset = 0
+
+      unless only_single_file
+        # Call all checks that run on the whole platformos_app
+        liquid_visitor = LiquidVisitor.new(@liquid_checks.whole_platformos_app, @disabled_checks)
+        html_visitor = HtmlVisitor.new(@html_checks.whole_platformos_app)
+        total += total_file_count
+        offset = total_file_count
+        @platformos_app.liquid.each_with_index do |liquid_file, i|
+          yield(liquid_file.relative_path.to_s, i, total) if block_given?
+          liquid_visitor.visit_liquid_file(liquid_file)
+          html_visitor.visit_liquid_file(liquid_file)
         end
 
-        # Call checks that run on a single files, only on specified file
-        liquid_visitor = LiquidVisitor.new(@liquid_checks.single_file, @disabled_checks, only_single_file:)
-        html_visitor = HtmlVisitor.new(@html_checks.single_file)
-        files.each_with_index do |app_file, i|
-          yield(app_file.relative_path.to_s, offset + i, total) if block_given?
-          if app_file.liquid?
-            liquid_visitor.visit_liquid_file(app_file)
-            html_visitor.visit_liquid_file(app_file)
-          elsif app_file.yaml?
-            @yaml_checks.single_file.call(:on_file, app_file)
-          end
+        @platformos_app.yaml.each_with_index do |yaml_file, i|
+          yield(yaml_file.relative_path.to_s, liquid_file_count + i, total) if block_given?
+          @yaml_checks.whole_platformos_app.call(:on_file, yaml_file)
+        end
+      end
+
+      # Call checks that run on a single files, only on specified file
+      liquid_visitor = LiquidVisitor.new(@liquid_checks.single_file, @disabled_checks, only_single_file:)
+      html_visitor = HtmlVisitor.new(@html_checks.single_file)
+      files.each_with_index do |app_file, i|
+        yield(app_file.relative_path.to_s, offset + i, total) if block_given?
+        if app_file.liquid?
+          liquid_visitor.visit_liquid_file(app_file)
+          html_visitor.visit_liquid_file(app_file)
+        elsif app_file.yaml?
+          @yaml_checks.single_file.call(:on_file, app_file)
         end
       end
 

data/lib/platformos_check/disabled_checks.rb

@@ -70,7 +70,7 @@ module PlatformosCheck
     def comment_text(node)
       case node.type_name
       when :comment
-        node.value.nodelist.join
+        node.inner_markup
       when :inline_comment
         node.markup.sub(/\s*#+\s*/, '')
       end

data/lib/platformos_check/language_server/variable_lookup_finder/tolerant_parser.rb

@@ -11,34 +11,25 @@ module PlatformosCheck
               # The tolerant parser relies on a tolerant custom parse
               # context to creates a new 'Template' object, even when
               # a block is not closed.
-              Liquid::Template.parse(content, custom_parse_context)
+              Liquid::Template.parse(content, environment: environment)
             end
 
             private
 
-            def custom_parse_context
-              ParseContext.new
+            def environment
+              @environment ||= Liquid::Environment.build(tags: Liquid::Environment.default.tags.merge(
+                {
+                  'case' => Tags::Case,
+                  'for' => Tags::For,
+                  'if' => Tags::If,
+                  'tablerow' => Tags::TableRow,
+                  'unless' => Tags::Unless
+                }
+              ))
             end
           end
         end
 
-        class ParseContext < Liquid::ParseContext
-          def new_block_body
-            BlockBody.new
-          end
-        end
-
-        class BlockBody < Liquid::BlockBody
-          ##
-          # The tags are statically defined and referenced at the
-          # 'Liquid::Template', so the TolerantParser just uses the
-          # redefined tags at this custom block body. Thus, there's
-          # no side-effects between the regular and the tolerant parsers.
-          def registered_tags
-            Tags.new(super)
-          end
-        end
-
         class Tags
           module TolerantBlockBody
             ##
@@ -72,21 +63,6 @@ module PlatformosCheck
           class Unless < Liquid::Unless
             include TolerantBlockBody
           end
-
-          def initialize(standard_tags)
-            @standard_tags = standard_tags
-            @tolerant_tags = {
-              'case' => Case,
-              'for' => For,
-              'if' => If,
-              'tablerow' => TableRow,
-              'unless' => Unless
-            }
-          end
-
-          def [](key)
-            @tolerant_tags[key] || @standard_tags[key]
-          end
         end
       end
     end

data/lib/platformos_check/tags/rollback.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module Tags
+    class Rollback < Liquid::Tag
+    end
+  end
+end

data/lib/platformos_check/tags/transaction.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module PlatformosCheck
+  module Tags
+    class Transaction < Liquid::Block
+    end
+  end
+end

data/lib/platformos_check/tags.rb

@@ -2,8 +2,6 @@
 
 module PlatformosCheck
   module Tags
-    # Copied tags parsing code from storefront-renderer
-
     class ContentFor < BaseBlock; end
 
     class Yield < Base; end
@@ -26,40 +24,42 @@ module PlatformosCheck
       end
 
       def register_tag(name, klass)
-        Liquid::Template.register_tag(name, klass)
+        Liquid::Environment.default.register_tag(name, klass)
       end
 
       def register_tags!
         return if !register_tags? || (defined?(@registered_tags) && @registered_tags)
 
         @registered_tags = true
+        register_tag('background', Background)
+        register_tag('cache', Cache)
+        register_tag('content_for', ContentFor)
+        register_tag('context', Context)
+        register_tag('context_rc', Context)
+        register_tag('export', Export)
         register_tag('form', Form)
+        register_tag('function', Function)
+        register_tag('graphql', Graphql)
+        register_tag('hash_assign', HashAssign)
         register_tag('include_form', IncludeForm)
-        register_tag('render', Render)
-        register_tag('theme_render', ThemeRender)
-        register_tag('theme_render_rc', ThemeRender)
         register_tag('log', Log)
-        register_tag('cache', Cache)
-        register_tag('print', Print)
         register_tag('parse_json', ParseJson)
-        register_tag('try_rc', Try)
-        register_tag('try', Try)
-        register_tag('export', Export)
-        register_tag('return', Return)
+        register_tag('print', Print)
         register_tag('redirect_to', RedirectTo)
+        register_tag('render', Render)
         register_tag('response_headers', ResponseHeaders)
         register_tag('response_status', ResponseStatus)
-        register_tag('hash_assign', HashAssign)
-        register_tag('background', Background)
-        register_tag('content_for', ContentFor)
+        register_tag('return', Return)
+        register_tag('rollback', Rollback)
         register_tag('session', Session)
-        register_tag('context', Context)
-        register_tag('context_rc', Context)
         register_tag('sign_in', SignIn)
-        register_tag('yield', Yield)
-        register_tag('graphql', Graphql)
-        register_tag('function', Function)
         register_tag('spam_protection', SpamProtection)
+        register_tag('theme_render', ThemeRender)
+        register_tag('theme_render_rc', ThemeRender)
+        register_tag('transaction', Transaction)
+        register_tag('try', Try)
+        register_tag('try_rc', Try)
+        register_tag('yield', Yield)
       end
     end
     self.register_tags = true

data/lib/platformos_check/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PlatformosCheck
-  VERSION = "0.4.14"
+  VERSION = "0.5.0"
 end

data/lib/platformos_check.rb

@@ -62,10 +62,12 @@ require_relative "platformos_check/tags/render"
 require_relative "platformos_check/tags/response_headers"
 require_relative "platformos_check/tags/response_status"
 require_relative "platformos_check/tags/return"
+require_relative "platformos_check/tags/rollback"
 require_relative "platformos_check/tags/session"
 require_relative "platformos_check/tags/sign_in"
 require_relative "platformos_check/tags/spam_protection"
 require_relative "platformos_check/tags/theme_render"
+require_relative "platformos_check/tags/transaction"
 require_relative "platformos_check/tags/try"
 require_relative "platformos_check/tags"
 require_relative "platformos_check/liquid_node"
@@ -86,9 +88,6 @@ require_relative "platformos_check/language_server"
 
 Dir[__dir__ + "/platformos_check/checks/*.rb"].each { |file| require file }
 
-Encoding.default_external = Encoding::UTF_8
-Encoding.default_internal = Encoding::UTF_8
-
 module PlatformosCheck
   def self.debug?
     ENV["PLATFORMOS_CHECK_DEBUG"] == "true"
@@ -98,16 +97,6 @@ module PlatformosCheck
     ENV.fetch("PLATFORMOS_CHECK_DEBUG_LOG_FILE", nil)
   end
 
-  def self.with_liquid_c_disabled
-    if defined?(Liquid::C)
-      was_enabled = Liquid::C.enabled
-      Liquid::C.enabled = false if was_enabled
-    end
-    yield
-  ensure
-    Liquid::C.enabled = true if defined?(Liquid::C) && was_enabled
-  end
-
   def self.log(message)
     bridge = LanguageServer::Bridge.new(LanguageServer::IOMessenger.new)
     bridge.log("###############\n #{message}\n##################")

data/platformos-check.gemspec

@@ -28,7 +28,7 @@ Gem::Specification.new do |spec|
   spec.require_paths = ["lib"]
 
   spec.add_dependency('graphql', '~> 2.0.0')
-  spec.add_dependency('liquid', '5.4.0')
+  spec.add_dependency('liquid', '5.8.6')
   spec.add_dependency('nokogiri', '>= 1.12')
   spec.add_dependency('parser', '~> 3')
 

metadata

@@ -1,16 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: platformos-check
 version: !ruby/object:Gem::Version
-  version: 0.4.14
+  version: 0.5.0
 platform: ruby
 authors:
 - Piotr Bliszczyk
 - Dariusz Gorzęba
 - Maciej Krajowski-Kukiel
-autorequire: 
 bindir: exe
 cert_chain: []
-date: 2024-10-11 00:00:00.000000000 Z
+date: 2025-08-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -32,14 +31,14 @@ dependencies:
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.4.0
+        version: 5.8.6
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '='
       - !ruby/object:Gem::Version
-        version: 5.4.0
+        version: 5.8.6
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
@@ -68,7 +67,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3'
-description: 
 email:
 - support@platformos.com
 executables:
@@ -341,10 +339,12 @@ files:
 - lib/platformos_check/tags/response_headers.rb
 - lib/platformos_check/tags/response_status.rb
 - lib/platformos_check/tags/return.rb
+- lib/platformos_check/tags/rollback.rb
 - lib/platformos_check/tags/session.rb
 - lib/platformos_check/tags/sign_in.rb
 - lib/platformos_check/tags/spam_protection.rb
 - lib/platformos_check/tags/theme_render.rb
+- lib/platformos_check/tags/transaction.rb
 - lib/platformos_check/tags/try.rb
 - lib/platformos_check/translation_file.rb
 - lib/platformos_check/user_schema_file.rb
@@ -358,7 +358,6 @@ licenses:
 metadata:
   allowed_push_host: https://rubygems.org
   rubygems_mfa_required: 'true'
-post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
@@ -373,8 +372,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.5.18
-signing_key: 
+rubygems_version: 3.6.5
 specification_version: 4
 summary: A platformOS App Linter
 test_files: []