heartml 1.0.0.beta2 → 1.0.0.beta4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c4ce5c2d26f4d668a6267c0d1ac9bcdfa39c8dfc9bc82ea4ee812842a0cd3778
-  data.tar.gz: ab81276b47264ccf1dba10b72a3ae75f406d4f13da26ea81c56f98f385f9bf25
+  metadata.gz: 76684cce6c5e1c52ce68f4b0b0770a60377aefc908ce38c469ee540b576eac8d
+  data.tar.gz: 388c37ecfc4829cbe1c0635ff301d4957b8eca52b6f75603d83b9184c36accc8
 SHA512:
-  metadata.gz: d97478938f30b82e275ec3ae79943b2def87c452ce28c4475d2e1fa68a66434842ddbe5195b877aa4de6ad0c735b58e61a34df69d7b75e0bea1f43a79e764f26
-  data.tar.gz: 757de7a3889c304655d7439e92386ded89bc04815a407acdd261d10aa6456f7380f8666d628a4c7e25d47150bc20fe36e353d5bceee3ad6b7d7cc58d18cb14a1
+  metadata.gz: 531c6b5411396f98d55ef33e3b2f8abb8c12fabd251c9fdfc2bfed032d36f188fc9d5f5721c7262bdb8adaa1e63c94f4744e1a4b5354f895579a97f359013406
+  data.tar.gz: dd55039932a06904f0fd911b9d3d2ddca7ee696fd3a085a5a3e931bac7ec1b31d9004092cacf9ea02b8b4048b463eed35480d68f563e01cf8778bcbd1f057afa

data/CHANGELOG.md

@@ -1,5 +1,17 @@
 ## [Unreleased]
 
+## [1.0.0.beta4] - 2023-08-27
+
+- Refactor and improve Bridgetown plugin and simplify context handling for template rendering
+
+## [1.0.0.beta3] - 2023-08-12
+
+- Ensure attributes are processed on component node before it's rendered as component
+
+## [1.0.0.beta2] - 2023-08-12
+
+- Fix Bridgetown issues and process component "light DOM" children
+
 ## [1.0.0.beta1] - 2023-08-12
 
 - Major refactor as part of the new Heartml project

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    heartml (1.0.0.beta2)
+    heartml (1.0.0.beta4)
       concurrent-ruby (~> 1.2)
       nokolexbor (>= 0.4.2)
 
@@ -38,9 +38,7 @@ GEM
       racc (~> 1.4)
     nokogiri (1.14.2-x86_64-linux)
       racc (~> 1.4)
-    nokolexbor (0.5.0)
     nokolexbor (0.5.0-arm64-darwin)
-    nokolexbor (0.5.0-x86_64-linux)
     parallel (1.22.1)
     parser (3.2.2.0)
       ast (~> 2.4.1)

data/lib/heartml/bridgetown_renderer.rb

@@ -0,0 +1,40 @@
+# frozen_string_literal: true
+
+module Heartml
+  class BridgetownRenderer < Bridgetown::Builder
+    def build
+      render_heart_modules
+    end
+
+    def render_heart_modules
+      inspect_html do |doc, resource|
+        view_context = Bridgetown::ERBView.new(resource)
+
+        rdr = Heartml::TemplateRenderer.new(body: doc.at_css("body"), context: view_context)
+        #        rdr.define_singleton_method(:view_context) { view_context }
+        rdr.call
+      end
+    end
+  end
+end
+
+Bridgetown.initializer :heartml do |config|
+  Bridgetown::Component.extend ActiveSupport::DescendantsTracker
+
+  Heartml.module_eval do
+    def render_in(view_context, rendering_mode: :string, &block)
+      self.rendering_mode = rendering_mode
+      super(view_context, &block)
+    end
+  end
+
+  # Eager load all components
+  config.hook :site, :after_reset do |site|
+    unless site.config.eager_load_paths.find { _1.end_with?(site.config.components_dir) }
+      site.config.eager_load_paths << site.config.autoload_paths.find { _1.end_with?(site.config.components_dir) }
+    end
+  end
+
+  config.html_inspector_parser "nokolexbor"
+  config.builder Heartml::BridgetownRenderer
+end

data/lib/heartml/fragment.rb

@@ -20,6 +20,8 @@ module Heartml
 
     def process(fragment = @fragment) # rubocop:disable Metrics
       traverse(fragment) do |node| # rubocop:disable Metrics/BlockLength
+        process_attribute_bindings(node)
+
         component = Heartml.registered_elements.find { _1.tag_name == node.name }
         if component
           attrs = node.attributes.dup
@@ -36,27 +38,29 @@ module Heartml
 
             params.each do |param|
               new_key, v2 = param.split(":").map(&:strip)
+              v2 = new_key unless v2
+
               new_attrs[new_key] = @component.evaluate_attribute_expression(attr, v2)
             end
             attrs.delete(k)
           end
           attrs.merge!(new_attrs)
           attrs.reject! { |k| k.start_with?("server-") || k.start_with?("iso-") || k.start_with?("host-") }
-          attrs.transform_keys!(&:to_sym)
+          attrs.transform_keys! { _1.tr("-", "_").to_sym }
 
           obj = component.new(**attrs)
           render_output = if obj.respond_to?(:render_in)
-                            obj.render_in(@component.view_context, rendering_mode: :node) do
+                            obj.render_in(@component.context, rendering_mode: :node) do
                               process(fragamatize(node.children))
                             end
                           else
-                            obj.render_element(content: process(fragamatize(node.children)))
+                            obj.render_element(
+                              content: process(fragamatize(node.children)), context: @component.context
+                            )
                           end
 
           node.replace(render_output)
         end
-
-        process_attribute_bindings(node)
       end
 
       fragment

data/lib/heartml/version.rb

@@ -2,5 +2,5 @@
 
 module Heartml
   # @return [String]
-  VERSION = "1.0.0.beta2"
+  VERSION = "1.0.0.beta4"
 end

data/lib/heartml.rb

@@ -149,6 +149,14 @@ module Heartml
     @_replaced_content = new_content
   end
 
+  def context
+    return @context if @context
+
+    return unless respond_to?(:view_context)
+
+    view_context # compatibility with other Ruby component systems
+  end
+
   # Override in component
   #
   # @return [Hash]
@@ -165,9 +173,10 @@ module Heartml
 
   # @param attributes [Hash]
   # @param content [String, Nokolexbor::Element]
-  def render_element(attributes: self.attributes, content: self.content) # rubocop:disable Metrics
+  def render_element(attributes: self.attributes, content: self.content, context: self.context) # rubocop:disable Metrics
     doc = self.class.doc.clone
     @_content = content
+    @context = context
 
     tmpl_el = doc.css("> template").find do |node|
       node.attributes.empty? ||
@@ -184,33 +193,6 @@ module Heartml
     # Process all the template bits
     process_fragment(tmpl_el)
 
-    # Heartml.registered_elements.each do |component|
-    #   tmpl_el.children[0].css(component.tag_name).reverse.each do |node|
-    #     if node["server-ignore"]
-    #       node.remove_attribute("server-ignore")
-    #       next
-    #     end
-
-    #     attrs = node.attributes.transform_values(&:value)
-    #     attrs.reject! { |k| k.start_with?("server-") }
-    #     new_attrs = {}
-    #     attrs.each do |k, v|
-    #       next unless k.start_with?("arg:")
-
-    #       new_key = k.delete_prefix("arg:")
-    #       attrs.delete(k)
-    #       new_attrs[new_key] = instance_eval(v, self.class.heart_module, self.class.line_number_of_node(node))
-    #     end
-    #     attrs.merge!(new_attrs)
-    #     attrs.transform_keys!(&:to_sym)
-
-    #     new_node = node.replace(
-    #       component.new(**attrs).render_element(content: node.children)
-    #     )
-    #     new_node.remove_attribute("server-ignore")
-    #   end
-    # end
-
     # Set attributes on the custom element
     attributes.each { |k, v| doc[k.to_s.tr("_", "-")] = value_to_attribute(v) if v }
 
@@ -381,26 +363,6 @@ module Heartml
     end
   end
 
-  # def _server_replace_binding(attribute:, node:)
-  #   if node.name == "template"
-  #     node.children[0].inner_html = node_or_string(evaluate_attribute_expression(attribute))
-  #     node.replace(node.children[0].children)
-  #   else
-  #     node.inner_html = node_or_string(evaluate_attribute_expression(attribute))
-  #     node.replace(node.children)
-  #   end
-  # end
-
-  # def _server_expr_binding(attribute:, node:)
-  #   if attribute.name.end_with?(":text")
-  #     node.content = node_or_string(evaluate_attribute_expression(attribute))
-  #     attribute.parent.delete(attribute.name)
-  #   elsif attribute.name.end_with?(":html")
-  #     node.inner_html = node_or_string(evaluate_attribute_expression(attribute))
-  #     attribute.parent.delete(attribute.name)
-  #   end
-  # end
-
   class ServerComponent
     def self.inherited(klass)
       super
@@ -409,7 +371,7 @@ module Heartml
     end
   end
 
-  class FragmentRenderComponent < ServerComponent
+  class TemplateRenderer < ServerComponent
     def self.heart_module
       "eval"
     end
@@ -419,9 +381,9 @@ module Heartml
       0
     end
 
-    def initialize(body:, scope:) # rubocop:disable Lint/MissingSuper
+    def initialize(body:, context:) # rubocop:disable Lint/MissingSuper
       @body = body.is_a?(String) ? Nokolexbor::DocumentFragment.parse(body) : body
-      @scope = scope
+      @context = context
     end
 
     def call
@@ -430,36 +392,14 @@ module Heartml
     end
 
     def respond_to_missing?(key)
-      @scope.respond_to?(key)
+      context.respond_to?(key)
     end
 
     # TODO: delegate instead?
     def method_missing(key, *args, **kwargs)
-      @scope.send(key, *args, **kwargs)
+      context.send(key, *args, **kwargs)
     end
   end
 end
 
-if defined?(Bridgetown)
-  Bridgetown.initializer :heartml do |config|
-    Bridgetown::Component.extend ActiveSupport::DescendantsTracker
-
-    Heartml.module_eval do
-      def render_in(view_context, rendering_mode = :string, &block)
-        self.rendering_mode = rendering_mode
-        super(view_context, &block)
-      end
-    end
-
-    # Eager load all components
-    config.hook :site, :after_reset do |site|
-      unless site.config.eager_load_paths.find { _1.end_with?(site.config.components_dir) }
-        site.config.eager_load_paths << site.config.autoload_paths.find { _1.end_with?(site.config.components_dir) }
-      end
-    end
-
-    config.html_inspector_parser "nokolexbor"
-    require_relative "heartml/component_renderer"
-    config.builder Heartml::ComponentRenderer
-  end
-end
+require_relative "heartml/bridgetown_renderer" if defined?(Bridgetown)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: heartml
 version: !ruby/object:Gem::Version
-  version: 1.0.0.beta2
+  version: 1.0.0.beta4
 platform: ruby
 authors:
 - Jared White
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-08-13 00:00:00.000000000 Z
+date: 2023-08-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: concurrent-ruby
@@ -70,7 +70,7 @@ files:
 - benchmark.rb
 - heartml.gemspec
 - lib/heartml.rb
-- lib/heartml/component_renderer.rb
+- lib/heartml/bridgetown_renderer.rb
 - lib/heartml/fragment.rb
 - lib/heartml/petite.rb
 - lib/heartml/query_selection.rb

data/lib/heartml/component_renderer.rb

@@ -1,52 +0,0 @@
-# frozen_string_literal: true
-
-module Heartml
-  class ComponentRenderer < Bridgetown::Builder
-    def build
-      render_heart_modules
-    end
-
-    # TODO: rework this using new server effects and component context!
-    def render_heart_modules
-      inspect_html do |doc, resource|
-        view_context = Bridgetown::ERBView.new(resource)
-
-        rdr = FragmentRenderComponent.new(body: doc.at_css("body"), scope: view_context)
-        rdr.define_singleton_method(:view_context) { view_context }
-        rdr.call
-
-        # Heartml.registered_elements.each do |component|
-        #   tag_name = component.tag_name
-        #   doc.xpath("//#{tag_name}").reverse.each do |node|
-        #     if node["server-ignore"]
-        #       node.remove_attribute("server-ignore")
-        #       next
-        #     end
-
-        #     attrs = node.attributes.transform_values(&:value)
-        #     attrs.reject! { |k| k.start_with?("server-") }
-
-        #     new_attrs = {}
-        #     attrs.each do |k, v|
-        #       next unless k.start_with?("arg:")
-
-        #       new_key = k.delete_prefix("arg:")
-        #       attrs.delete(k)
-        #       new_attrs[new_key] = resource.instance_eval(v)
-        #     end
-        #     attrs.merge!(new_attrs)
-        #     attrs.transform_keys!(&:to_sym)
-
-        #     new_node = node.replace(
-        #       component.new(**attrs).render_in(view_context, rendering_mode: :node) { node.children }
-        #     )
-        #     new_node.remove_attribute("server-ignore")
-        #   end
-        # rescue StandardError => e
-        #   Bridgetown.logger.error "Unable to render <#{tag_name}> (#{component}) in #{resource.path}"
-        #   raise e
-        # end
-      end
-    end
-  end
-end