low_node 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ac55ce01d35523d45910268c02ce723e5baac95880791bc2ba776be62ec42896
-  data.tar.gz: 845688a5770bcb13e48c00e31b41fc9f6f77ea5f23e30b7fe7c6515675732b27
+  metadata.gz: 3306c75f526286a539e171e60afec93be837c7e25047aa129b206351d61ef0ec
+  data.tar.gz: adca6941b2fec856c5cfeeee62bc19522073cc4a19369977715af2476df424e5
 SHA512:
-  metadata.gz: cae79c698dc5a1986ad920782fc2ca4fc445ed51e3eed3661dd5c501c3791a648c04d53264b01197404e3a53b682bf31f28985ddf4d206f7f202707c5b790ba5
-  data.tar.gz: f6eeca786d6dc6bab237b56ed8a260fe2a0efbd5ffbbb58bfa5e88949ee76524dff3c02460ba5535fe92584e3cccf77b4436e32733b981665ff0278d6318c2a6
+  metadata.gz: c33df65970b91273bf362a342ad2f055479a34fab25025cf8583420552587bb93726a9344d2fe50ea971690f5d11b455c0daa0fc859fe966b874c5251bc0a467
+  data.tar.gz: 1a8596590dbd4c8b033297b70d631f4f580e4d9ef342af20d937cd58d65e4f8e6a5e0bb4c58c7c519551e6f0d04fc4dfe28229d1146b0b55100745b65350dc73

data/lib/low_node.rb

@@ -17,17 +17,18 @@ class LowNode
 
   attr_reader :event
 
-  def initialize(event:)
+  def initialize(event: nil)
     @event = event
   end
 
+  # TODO: Rename "handle" method/action to more specific "route" and maybe split out into module too.
   def handle(event:)
     nil
   end
 
   class << self
     def handle(event:)
-      self.new(event:).handle(event:)
+      new(event:).handle(event:)
     end
 
     def inherited(child)

data/lib/templates/renderer.rb

@@ -4,26 +4,22 @@ require_relative 'template'
 
 module Low
   module Templates
+    # Use the Method Factory pattern to .render -> .new -> #initialize -> #render a low node.
     module Renderer
-      def render(event: nil)
-        nil
-      end
-
       # When render() contains RBX/Antlers then LowLoad builds a template to render with instead.
-      def render_template(event:, parent_binding: nil, slot_node: nil, props: {})
+      def render_template(parent_binding: nil, slot_node: nil, props: {}, **kwargs)
         template = self.class.template
 
         # NOTE: We currently set local variables on this local binding, not the receiver's binding.
         # TODO: Should we create an "instance_binding" method on LowNode and use that binding instead?
         current_binding = binding
-        current_binding.local_variable_set(:event, event)
 
-        # Pass in props from parent component as keyword arguments.
         template.params.each do |param|
-          current_binding.local_variable_set(param, props[param]) if props[param]
+          variable = props[param] || kwargs[param] || raise(ArgumentError, "Missing argument: '#{param}'")
+          current_binding.local_variable_set(param, variable)
         end
 
-        template.engine.render(ast: template.ast, current_binding:, parent_binding:, slot_node:, namespace: template.namespace)
+        template.engine.render(ast: template.ast, current_binding:, parent_binding:, slot_node:)
       end
 
       def self.included(base)
@@ -31,34 +27,30 @@ module Low
       end
 
       module ClassMethods
+        def render(**)
+          # GOAL: Make return value configurable; ResponseEvent, Response, or body.
+          response = Low::Factories::ResponseFactory.html(body: response_body(**))
+          Low::Events::ResponseEvent.new(response:).tap { it.branch }
+        end
+
+        # LowLoad hook.
         def template
           @template
         end
 
+        # LowLoad hook.
         def build_template(template:, params:, engine:, namespace:)
           @template = Template.new(template:, params:, engine:, namespace:)
         end
 
-        # TODO: Handle situation where node is tested in a unit test and args come in here; expose them to the template.
-        def render(event: nil)
-          node = self.new(event:)
-
-          # GOAL: Make return value configurable; ResponseEvent, Response, or body.
-          response = Low::Factories::ResponseFactory.html(body: body(node:, event:))
-          Low::Events::ResponseEvent.new(response:)
-        end
-
-        def render_template(event:)
-          self.new(event:).render_template(event:)
-        end
-
         private
 
-        def body(node:, event:)
-          return node.render_template(event:) if @template
-          return node.render(event:) if node.method(:render).arity > 0
+        def response_body(**)
+          node = new(**)
+
+          return node.render_template(**) if @template
 
-          node.render
+          node.render(**)
         end
       end
     end

data/lib/templates/template.rb

@@ -11,7 +11,7 @@ module Low
         @namespace = namespace
 
         @engine = engine
-        @ast = engine.ast(template)
+        @ast = engine.ast(template:, namespace:)
       end
     end
   end

data/lib/version.rb

@@ -2,6 +2,6 @@
 
 module Low
   module Node
-    VERSION = '0.3.0'
+    VERSION = '0.4.0'
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: low_node
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - maedi
@@ -108,7 +108,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.10
+rubygems_version: 4.0.6
 specification_version: 4
 summary: Flexible building blocks
 test_files: []