Rename {% doc %} constant strictly validates the abscense of args

History.md

@@ -1,34 +1,22 @@
 # Liquid Change Log
 
-## 5.8.1 (unreleased)
-
-## 5.8.1
-
-* Fix `{% doc %}` tag to be visitable [Guilherme Carreiro]
-
-## 5.8.0
-
-* Introduce the new `{% doc %}` tag [Guilherme Carreiro]
-
-## 5.7.3
-
-* Raise Liquid::SyntaxError when parsing invalidly encoded strings [Chris AtLee]
+## 5.8.0 (unreleased)
 
 ## 5.7.2 2025-01-31
 
-* Fix array filters to not support nested properties [Guilherme Carreiro]
+* Fix array filters to not support nested properties
 
 ## 5.7.1 2025-01-24
 
-* Fix the `find` and `find_index`filters to return `nil` when filtering empty arrays [Guilherme Carreiro]
-* Fix the `has` filter to return `false` when filtering empty arrays [Guilherme Carreiro]
+* Fix the `find` and `find_index`filters to return `nil` when filtering empty arrays
+* Fix the `has` filter to return `false` when filtering empty arrays
 
 ## 5.7.0 2025-01-16
 
 ### Features
 
-* Add `find`, `find_index`, `has`, and `reject` filters to arrays [Guilherme Carreiro]
-* Compatibility with Ruby 3.4 [Ian Ker-Seymer]
+* Add `find`, `find_index`, `has`, and `reject` filters to arrays
+* Compatibility with Ruby 3.4
 
 ## 5.6.4 2025-01-14
 

lib/liquid/lexer.rb

@@ -161,12 +161,6 @@ module Liquid
         end
         # rubocop:enable Metrics/BlockNesting
         output << EOS
-      rescue ::ArgumentError => e
-        if e.message == "invalid byte sequence in #{ss.string.encoding}"
-          raise SyntaxError, "Invalid byte sequence in #{ss.string.encoding}"
-        else
-          raise
-        end
       end
 
       def raise_syntax_error(start_pos, ss)

lib/liquid/standardfilters.rb

@@ -456,7 +456,7 @@ module Liquid
     #   Tests if any item in an array has a specific property value.
     # @liquid_description
     #   This requires you to provide both the property name and the associated value.
-    # @liquid_syntax array | has: string, string
+    # @liquid_syntax array | some: string, string
     # @liquid_return [boolean]
     def has(input, property, target_value = nil)
       filter_array(input, property, target_value, false) { |ary, &block| ary.any?(&block) }

lib/liquid/tags/doc.rb

@@ -55,10 +55,6 @@ module Liquid
       true
     end
 
-    def nodelist
-      []
-    end
-
     private
 
     def ensure_valid_markup(tag_name, markup, parse_context)

lib/liquid/tokenizer.rb

@@ -103,12 +103,6 @@ module Liquid
 
       pos = @ss.pos -= 2
       @source.byteslice(start, pos - start)
-    rescue ::ArgumentError => e
-      if e.message == "invalid byte sequence in #{@ss.string.encoding}"
-        raise SyntaxError, "Invalid byte sequence in #{@ss.string.encoding}"
-      else
-        raise
-      end
     end
 
     def next_variable_token

lib/liquid/version.rb

@@ -2,5 +2,5 @@
 # frozen_string_literal: true
 
 module Liquid
-  VERSION = "5.8.1"
+  VERSION = "5.7.2"
 end

test/unit/lexer_unit_test.rb

@@ -131,16 +131,6 @@ class LexerUnitTest < Minitest::Test
     assert_equal([[:id, "false"], [:number, "1"], [:end_of_string]], tokenize("false 1"))
   end
 
-  def test_error_with_invalid_utf8
-    error = assert_raises(SyntaxError) do
-      tokenize("\x00\xff")
-    end
-    assert_equal(
-      'Liquid syntax error: Invalid byte sequence in UTF-8',
-      error.message,
-    )
-  end
-
   private
 
   def tokenize(input)

test/unit/tags/doc_tag_unit_test.rb

@@ -157,28 +157,4 @@ class DocTagUnitTest < Minitest::Test
     assert_template_result('', "{% doc %}123{% enddoc\nxyz %}")
     assert_template_result('', "{% doc %}123{% enddoc\n   xyz  enddoc %}")
   end
-
-  def test_doc_tag_visitor
-    template_source = '{% doc %}{% enddoc %}'
-
-    assert_equal(
-      [Liquid::Doc],
-      visit(template_source),
-    )
-  end
-
-  private
-
-  def traversal(template)
-    ParseTreeVisitor
-      .for(Template.parse(template).root)
-      .add_callback_for(Liquid::Doc) do |tag|
-        tag_class = tag.class
-        tag_class
-      end
-  end
-
-  def visit(template)
-    traversal(template).visit.flatten.compact
-  end
 end

test/unit/template_unit_test.rb

@@ -35,15 +35,4 @@ class TemplateUnitTest < Minitest::Test
   def test_template_inheritance
     assert_equal("foo", TemplateSubclass.parse("foo").render)
   end
-
-  def test_invalid_utf8
-    input = "\xff\x00"
-    error = assert_raises(SyntaxError) do
-      Liquid::Tokenizer.new(source: input, string_scanner: StringScanner.new(input))
-    end
-    assert_equal(
-      'Liquid syntax error: Invalid byte sequence in UTF-8',
-      error.message,
-    )
-  end
 end