curlybars 1.16.1.pre.3 → 1.16.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e8152eb9d4e5789fce89e656283b0ae7be78799bafcd9755c99d793674a01d9a
-  data.tar.gz: 57102435a47d352a31c16d994e36bd61436b777e0e98409355bc31161045a44d
+  metadata.gz: 746a0625058bd9a2bca9ebf655c36f7cbd7e11e43000a051f4a76516ce73150b
+  data.tar.gz: f631902fba5a77cce57a36472636b99a5c03450dfbe5943e058c92b7c09e1d07
 SHA512:
-  metadata.gz: 797b775e410fff90804ecfad939a0f31add15dc199a4b3221dba6ab4d9ee165134b07c40b90d6e16ea87f331ee80226d9ab0ae28823c8508a0dd7c8c5671a57f
-  data.tar.gz: 5b2f6719689b4c047d3fa8a0b31dccd92457d5147eb40a0092331eb7b2f349297bdebd0b15b04702d0808bf154b6bbe69f9c2a7a83ddc71df80aae2fa3dbce0b
+  metadata.gz: c0fb5fb5398bf4643f679fdb07403de92892ee84efb70bae6a689297251e1ff9e79311b1c7711a6ba6a247a84eac945d735f72deacd8a555c9cd8ebfd75e9d81
+  data.tar.gz: ac273a2a866a2634c6d17ebf4a4bfd32b656c765e896214e2f3d6d2d49069232503e96bdc04a23b698d8f569cf5d368f5ab736699e280e404b9765bb370b4132

data/exe/curlybars-indent-lint

@@ -178,4 +178,6 @@ def main
   end
 end
 
+# simplecov:disable
 main if __FILE__ == $PROGRAM_NAME
+# simplecov:enable

data/lib/curlybars/configuration.rb

@@ -13,6 +13,7 @@ module Curlybars
 
   def self.reset
     @configuration = Configuration.new
+    @global_helpers_dependency_tree = nil
   end
 
   class Configuration

data/lib/curlybars/node/each_else.rb

@@ -12,6 +12,7 @@ module Curlybars
             template_cache_key = '#{each_template.cache_key}'
 
             collection = rendering.coerce_to_hash!(collection, #{collection_path.path.inspect}, position)
+            rendering.check_nesting_not_too_deep(position)
             collection.each.with_index.map do |key_and_presenter, index|
               rendering.check_timeout!
               rendering.optional_presenter_cache(key_and_presenter[1], template_cache_key, buffer) do |buffer|

data/lib/curlybars/node/path.rb

@@ -168,11 +168,6 @@ module Curlybars
 
           message = "`#{path}` cannot resolve to a helper"
           raise Curlybars::Error::Validate.new('not_a_helper', message, position)
-        when :not_helper
-          return unless helper?(branches)
-
-          message = "`#{path}` resolves to a helper"
-          raise Curlybars::Error::Validate.new('is_a_helper', message, position)
         when :anything
         else
           raise "invalid type `#{check_type}`"

data/lib/curlybars/node/sub_expression.rb

@@ -63,22 +63,19 @@ module Curlybars
 
         type = node.resolve_and_check!(branches, check_type: check_type)
 
-        if helper?(type)
-          if generic_helper?(type)
-            is_collection = type.last.is_a?(Array)
-            return infer_generic_helper_type!(branches, is_collection: is_collection)
-          end
-
-          return type.last
+        if generic_helper?(type)
+          is_collection = type.last.is_a?(Array)
+          infer_generic_helper_type!(branches, is_collection: is_collection)
+        elsif helper?(type)
+          type.last
+        else
+          type
         end
-
-        type
       end
 
       def generic_helper?(type)
-        return false unless type.is_a?(Array)
+        return false unless helper?(type)
         return false unless type.size == 2
-        return false unless type.first == :helper
 
         type.last == [{}] || type.last == {}
       end

data/lib/curlybars/node/template.rb

@@ -9,11 +9,6 @@ module Curlybars
         <<-RUBY
           ::Module.new do
             def self.exec(contexts, rendering, variables, buffer)
-              unless contexts.length < ::Curlybars.configuration.nesting_limit
-                message = "Nesting too deep"
-                position = rendering.position(#{position.line_number}, #{position.line_offset})
-                raise ::Curlybars::Error::Render.new('nesting_too_deep', message, position)
-              end
               #{compiled_items}
             end
           end.exec(contexts, rendering, variables, buffer)

data/lib/curlybars/node/with_else.rb

@@ -10,6 +10,7 @@ module Curlybars
           if rendering.to_bool(compiled_path)
             position = rendering.position(#{position.line_number}, #{position.line_offset})
             rendering.check_context_is_presenter(compiled_path, #{presenter_path.path.inspect}, position)
+            rendering.check_nesting_not_too_deep(position)
 
             contexts.push(compiled_path)
             begin

data/lib/curlybars/path_finder.rb

@@ -82,8 +82,7 @@ module Curlybars
         stack.pop unless stack.empty?
       end
 
-      remaining_segments = full_path.empty? ? [] : full_path.split('.')
-      stack + remaining_segments
+      stack + full_path.split('.')
     end
 
     def handle_block_helper(node, context_stack)

data/lib/curlybars/rendering_support.rb

@@ -31,6 +31,16 @@ module Curlybars
       raise ::Curlybars::Error::Render.new('timeout', message, nil)
     end
 
+    # Called before a #with or #each block pushes its context. The contexts
+    # stack starts with the root presenter, so its length is one more than
+    # the number of enclosing #with and #each blocks.
+    def check_nesting_not_too_deep(position)
+      return if contexts.length <= Curlybars.configuration.nesting_limit
+
+      message = "Nesting too deep (> #{Curlybars.configuration.nesting_limit} levels)"
+      raise Curlybars::Error::Render.new('nesting_too_deep', message, position)
+    end
+
     def check_context_is_presenter(context, path, position)
       return if presenter?(context)
 
@@ -257,8 +267,11 @@ module Curlybars
       raise Curlybars::Error::Render.new('unallowed_path', message, position)
     end
 
+    # Counts the dots after the final `/` in the path. Leading `../` segments
+    # move up the contexts stack rather than traversing, so they don't count.
     def check_traverse_not_too_deep(traverse, position)
-      return unless traverse.count('.') > Curlybars.configuration.traversing_limit
+      dots = traverse.split('/').last.count('.')
+      return unless dots > Curlybars.configuration.traversing_limit
 
       message = "`#{traverse}` too deep"
       raise Curlybars::Error::Render.new('traverse_too_deep', message, position)

data/lib/curlybars/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Curlybars
-  VERSION = '1.16.1.pre.3'
+  VERSION = '1.16.1'
 end

data/lib/curlybars.rb

@@ -29,7 +29,7 @@ module Curlybars
       cache_key = ["Curlybars.compile", Curlybars::VERSION, identifier, Digest::SHA256.hexdigest(source)]
 
       cache.fetch(cache_key) do
-        ast(transformed_source(source), identifier, run_processors: true).compile
+        ast(transformed_source(source, identifier), identifier, run_processors: true).compile
       end
     end
 
@@ -76,7 +76,7 @@ module Curlybars
     # source - The source HBS String used to generate an AST.
     # identifier - The the file name of the template being checked (defaults to `nil`).
     def visit(visitor, source, identifier = nil)
-      tree = ast(transformed_source(source), identifier, run_processors: true)
+      tree = ast(transformed_source(source, identifier), identifier, run_processors: true)
       visitor.accept(tree)
     end
 
@@ -99,7 +99,7 @@ module Curlybars
     #
     # Returns an Array of Curlybars::Node::Path instances that match the target path.
     def find(target_path, source, identifier = nil, role: nil)
-      tree = ast(transformed_source(source), identifier, run_processors: true)
+      tree = ast(transformed_source(source, identifier), identifier, run_processors: true)
       finder = Curlybars::PathFinder.new(tree)
       finder.find(target_path, role: role)
     end
@@ -127,7 +127,7 @@ module Curlybars
 
     private
 
-    def transformed_source(source)
+    def transformed_source(source, identifier)
       transformers = Curlybars.configuration.compiler_transformers
       transformers.inject(source) do |memo, transformer|
         transformer.transform(memo, identifier)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: curlybars
 version: !ruby/object:Gem::Version
-  version: 1.16.1.pre.3
+  version: 1.16.1
 platform: ruby
 authors:
 - Libo Cannici
@@ -96,7 +96,6 @@ files:
 - lib/curlybars/lexer.rb
 - lib/curlybars/method_whitelist.rb
 - lib/curlybars/node/block_helper_else.rb
-- lib/curlybars/node/boolean.rb
 - lib/curlybars/node/each_else.rb
 - lib/curlybars/node/if_else.rb
 - lib/curlybars/node/item.rb
@@ -106,7 +105,6 @@ files:
 - lib/curlybars/node/partial.rb
 - lib/curlybars/node/path.rb
 - lib/curlybars/node/root.rb
-- lib/curlybars/node/string.rb
 - lib/curlybars/node/sub_expression.rb
 - lib/curlybars/node/template.rb
 - lib/curlybars/node/text.rb
@@ -140,7 +138,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '3.2'
+      version: '3.3'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/lib/curlybars/node/boolean.rb

@@ -1,24 +0,0 @@
-module Curlybars
-  module Node
-    Boolean = Struct.new(:boolean) do
-      def compile
-        # NOTE: the following is a heredoc string, representing the ruby code fragment
-        # outputted by this node.
-        <<-RUBY
-          ->() { #{boolean} }
-        RUBY
-      end
-
-      def validate(branches, context: nil)
-        # Nothing to validate here.
-      end
-
-      def cache_key
-        [
-          boolean.to_s,
-          self.class.name
-        ].join("/")
-      end
-    end
-  end
-end

data/lib/curlybars/node/string.rb

@@ -1,24 +0,0 @@
-module Curlybars
-  module Node
-    String = Struct.new(:string) do
-      def compile
-        # NOTE: the following is a heredoc string, representing the ruby code fragment
-        # outputted by this node.
-        <<-RUBY
-          ->() { #{string.inspect} }
-        RUBY
-      end
-
-      def validate(branches, context: nil)
-        # Nothing to validate here.
-      end
-
-      def cache_key
-        [
-          string,
-          self.class.name
-        ].join("/")
-      end
-    end
-  end
-end