low_node 0.3.1 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 75eb52cd5e606c086a9672c8675710979878a5b240428a619b3ab454ec8534f0
-  data.tar.gz: 476ce5d261f9034066b21223a31be47d137cf64d6ec3921605bcf5060b594ea4
+  metadata.gz: 3306c75f526286a539e171e60afec93be837c7e25047aa129b206351d61ef0ec
+  data.tar.gz: adca6941b2fec856c5cfeeee62bc19522073cc4a19369977715af2476df424e5
 SHA512:
-  metadata.gz: 9712d807495ee5299d83fc6fcd17355d009978ba9c6bbc8709da9098decb858ff4d52e035505213bc28c745561537a161b24412d6175552618cd0c8ef1f96d4e
-  data.tar.gz: 889af9b4f7caca1f61a559743f3a2ba2792d177320a37d9d9a2a8258af5713d4ca088cdf07991e8eb11415bb85defa2a839fa438450f331be4b33bd85fbb0c75
+  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,30 +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: response_body(node:, event:))
-          Low::Events::ResponseEvent.new(response:).tap { it.branch }
-        end
-
         private
 
-        def response_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.1'
+    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.1
+  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: []