vident 1.0.0.alpha1 → 1.0.0.alpha2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d59f73664008e87455784726ec60622da719f758449339ac3904f99c7b04b1b6
-  data.tar.gz: d10178e8d4cfe43d441d8dde5b7540d847ecad01fea060024362d0e6c74735e6
+  metadata.gz: 7144408178f3fedee324dc23604af7baca5db589520d3cb4ea1413a88e4c93c0
+  data.tar.gz: 742e8db9df1d0f72e701ccec290693be799408b70263fd5a7c75a1c1987a0acb
 SHA512:
-  metadata.gz: 4a605e928c5749975f0f4127c49192f6676ee349e979b3d9531013a2cd988228516a7b6a592155ea175e60e30648417c7c76bb3ec4535a1b26008baa27eb3eaf
-  data.tar.gz: 1057b3d13ef990e098d0011f5b00ff0e020c605fae330c88baf0677f85fffa46ec1ff604e97276d6934e23ff1aaad03ddbd87856c1ca7bdcd555663d81b3cb06
+  metadata.gz: c1ccac1a695a9a915e87675973c7a09143582cb7775d6fd9a89d24c426a35cad5aa2d0e7adc424d680ae02d4cb767645a11394c05981cff2724c01482ccf44f2
+  data.tar.gz: 315951c114e8ecb5db720d45dc8236aabb34b7442eec520e47d8eb4b9472728b50c0b6a430e9ead6e85c292640eb6cafe03f034bf7ca6606e422e3c0c837c544

data/CHANGELOG.md

@@ -5,7 +5,17 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
-## [1.0.0.alpha1] - 2025-07-04
+
+## [1.0.0.alpha2] - 2025-07-08
+
+### Breaking
+- `nil` values in Stimulus values are ok, but `nil` for an action/target/outlet makes no sense so is ignored.
+
+### Fixed
+
+- `stimulus_scoped_event` must return a Symbol to work with `stimulus_action` and `stimulus_target` methods etc.
+
+## [1.0.0.alpha1] - 2025-07-08
 
 This release is a major overhaul of the Vident library, and introduces a new API for defining components and Stimulus attributes. The new API is designed to be more consistent and easier to use.
 

data/README.md

@@ -90,8 +90,10 @@ class ButtonComponent < Vident::ViewComponent::Base
   end
 
   def call
-    root_element do
-      @text
+    root_element do |component|
+      component.tag(:span, stimulus_target: :status) do
+        @text
+      end
     end
   end
 
@@ -130,14 +132,19 @@ export default class extends Controller {
     loadingDuration: Number 
   }
   static classes = ["loading"]
+  static targets = ["status"]
   
   handleClick(event) {
     // Increment counter
     this.clickedCountValue++
     
+    // Store original text
+    const originalText = this.statusTarget.textContent
+    
     // Add loading state
     this.element.classList.add(this.loadingClass)
     this.element.disabled = true
+    this.statusTarget.textContent = "Loading..."
     
     // Use the loading duration from the component
     setTimeout(() => {
@@ -145,7 +152,7 @@ export default class extends Controller {
       this.element.disabled = false
       
       // Update text to show count
-      this.element.textContent = `${this.element.textContent} (${this.clickedCountValue})`
+      this.statusTarget.textContent = `${originalText} (${this.clickedCountValue})`
     }, this.loadingDurationValue)
   }
 }
@@ -178,7 +185,7 @@ The rendered HTML includes all Stimulus data attributes:
         data-button-component-loading-duration-value="1000"
         data-button-component-loading-class="opacity-50 cursor-wait"
         id="button-component-123">
-  Save
+  <span data-button-component-target="status">Save</span>
 </button>
 
 <!-- Second button with pre-set count -->
@@ -189,7 +196,7 @@ The rendered HTML includes all Stimulus data attributes:
         data-button-component-loading-duration-value="1000"
         data-button-component-loading-class="opacity-50 cursor-wait"
         id="button-component-456">
-  Submit
+  <span data-button-component-target="status">Submit</span>
 </button>
 ```
 
@@ -218,6 +225,29 @@ class CardComponent < Vident::ViewComponent::Base
 end
 ```
 
+### Post-Initialization Hooks
+
+Vident provides a hook for performing actions after component initialization:
+
+```ruby
+class MyComponent < Vident::ViewComponent::Base
+  prop :data, Hash, default: -> { {} }
+  
+  def after_component_initialize
+    @processed_data = process_data(@data)
+  end
+  
+  private
+  
+  def process_data(data)
+    # Your initialization logic here
+    data.transform_values(&:upcase)
+  end
+end
+```
+
+**Important**: If you decide to override Literal's `after_initialize`, you **must** call `super` first to ensure Vident's initialization completes properly. Alternatively, use `after_component_initialize` which doesn't require calling `super`.
+
 ### Built-in Properties
 
 Every Vident component includes these properties:
@@ -389,7 +419,7 @@ Vident provides helper methods to generate scoped event names for dispatching cu
 class MyComponent < Vident::ViewComponent::Base
   stimulus do
     # Define an action that responds to a scoped event
-    actions [stimulus_scoped_event_on_window(:data_loaded), :handle_data_loaded]
+    actions -> { [stimulus_scoped_event_on_window(:data_loaded), :handle_data_loaded] }
   end
   
   def handle_click

data/lib/vident/component.rb

@@ -22,7 +22,7 @@ module Vident
 
     include TagHelper
     include Tailwind
-    include StimulusDSL
+    include StimulusHelper
 
     # Override this method to perform any initialisation after attributes are set
     def after_component_initialize

data/lib/vident/component_attribute_resolver.rb

@@ -11,7 +11,7 @@ module Vident
       # Add stimulus attributes from DSL first (lower precedence)
       add_stimulus_attributes_from_dsl
 
-      # Process root_element_attributes (higher precedence) 
+      # Process root_element_attributes (higher precedence)
       extra = root_element_attributes
       @html_options = (extra[:html_options] || {}).merge(@html_options) if extra.key?(:html_options)
       @html_options[:class] = render_classes(extra[:classes])
@@ -36,20 +36,19 @@ module Vident
       add_stimulus_actions(dsl_attrs[:stimulus_actions]) if dsl_attrs[:stimulus_actions]
       add_stimulus_targets(dsl_attrs[:stimulus_targets]) if dsl_attrs[:stimulus_targets]
       add_stimulus_outlets(dsl_attrs[:stimulus_outlets]) if dsl_attrs[:stimulus_outlets]
-      
+
       # Add static values (now includes resolved proc values)
       add_stimulus_values(dsl_attrs[:stimulus_values]) if dsl_attrs[:stimulus_values]
-      
+
       # Resolve and add values from props
       if dsl_attrs[:stimulus_values_from_props]
         resolved_values = resolve_values_from_props(dsl_attrs[:stimulus_values_from_props])
         add_stimulus_values(resolved_values) unless resolved_values.empty?
       end
-      
+
       add_stimulus_classes(dsl_attrs[:stimulus_classes]) if dsl_attrs[:stimulus_classes]
     end
 
-
     # Prepare stimulus collections and implied controller path from the given attributes, called after initialization
     def prepare_stimulus_collections # Convert raw attributes to stimulus attribute collections
       @stimulus_controllers_collection = stimulus_controllers(*Array.wrap(@stimulus_controllers))

data/lib/vident/stimulus_attributes.rb

@@ -5,13 +5,14 @@ module Vident
     extend ActiveSupport::Concern
 
     class_methods do
-      # Class methods for generating scoped event names
+      # Class methods for generating scoped event names. Returns a symbol
+      # so that the action parser will see it as a Stimulus event.
       def stimulus_scoped_event(event)
-        "#{component_name}:#{stimulus_js_name(event)}"
+        :"#{component_name}:#{stimulus_js_name(event)}"
       end
 
       def stimulus_scoped_event_on_window(event)
-        "#{component_name}:#{stimulus_js_name(event)}@window"
+        :"#{component_name}:#{stimulus_js_name(event)}@window"
       end
 
       private

data/lib/vident/stimulus_builder.rb

@@ -35,7 +35,7 @@ module Vident
       @values.merge!(value_hash) unless value_hash.empty?
       self
     end
-    
+
     def values_from_props(*prop_names)
       @values_from_props.concat(prop_names)
       self
@@ -53,11 +53,11 @@ module Vident
 
     def to_attributes(component_instance)
       attrs = {}
-      attrs[:stimulus_actions] = resolve_values(@actions, component_instance) unless @actions.empty?
-      attrs[:stimulus_targets] = resolve_values(@targets, component_instance) unless @targets.empty?
-      attrs[:stimulus_values] = resolve_hash_values(@values, component_instance) unless @values.empty?
+      attrs[:stimulus_actions] = resolve_attributes_filtering_nil(@actions, component_instance) unless @actions.empty?
+      attrs[:stimulus_targets] = resolve_attributes_filtering_nil(@targets, component_instance) unless @targets.empty?
+      attrs[:stimulus_values] = resolve_hash_values_allowing_nil(@values, component_instance) unless @values.empty?
       attrs[:stimulus_values_from_props] = @values_from_props.dup unless @values_from_props.empty?
-      attrs[:stimulus_classes] = resolve_hash_values(@classes, component_instance) unless @classes.empty?
+      attrs[:stimulus_classes] = resolve_hash_classes_filtering_nil(@classes, component_instance) unless @classes.empty?
       attrs[:stimulus_outlets] = @outlets.dup unless @outlets.empty?
       attrs
     end
@@ -95,22 +95,45 @@ module Vident
 
     private
 
-    def resolve_values(array, component_instance)
-      array.map do |value|
+    # For actions, targets - filter out nil values from procs AND static
+    def resolve_attributes_filtering_nil(array, component_instance)
+      result = []
+      array.each do |value|
         if callable?(value)
-          component_instance.instance_exec(&value)
+          resolved_value = component_instance.instance_exec(&value)
+          # Exclude nil from procs (nil is not valid for actions/targets)
+          result << resolved_value unless resolved_value.nil?
         else
-          value
+          # Exclude static nil values (nil is not valid for actions/targets)
+          result << value unless value.nil?
         end
       end
+      result
     end
 
-    def resolve_hash_values(hash, component_instance)
+    # For values - allow nil values from procs and static (will become "null" in JavaScript)
+    def resolve_hash_values_allowing_nil(hash, component_instance)
       hash.transform_values { |value| callable?(value) ? component_instance.instance_exec(&value) : value }
     end
 
+    # For classes - filter out nil values from procs AND static
+    def resolve_hash_classes_filtering_nil(hash, component_instance)
+      result = {}
+      hash.each do |key, value|
+        if callable?(value)
+          resolved_value = component_instance.instance_exec(&value)
+          # Exclude nil from procs (nil is not valid for classes)
+          result[key] = resolved_value unless resolved_value.nil?
+        else
+          # Exclude static nil values (nil is not valid for classes)
+          result[key] = value unless value.nil?
+        end
+      end
+      result
+    end
+
     def callable?(value)
       value.respond_to?(:call)
     end
   end
-end
+end

data/lib/vident/{stimulus_dsl.rb → stimulus_helper.rb}

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Vident
-  module StimulusDSL
+  module StimulusHelper
     extend ActiveSupport::Concern
 
     class_methods do
@@ -11,10 +11,10 @@ module Vident
           @stimulus_builder = StimulusBuilder.new
           @inheritance_merged = false
         end
-        
+
         # Ensure inheritance is applied
         ensure_inheritance_merged
-        
+
         # Execute the new block to add/merge new attributes
         @stimulus_builder.instance_eval(&block)
       end
@@ -24,10 +24,10 @@ module Vident
         if @stimulus_builder.nil? && superclass.respond_to?(:stimulus_dsl_attributes)
           return superclass.stimulus_dsl_attributes(component_instance)
         end
-        
+
         # Ensure inheritance is applied at access time
         ensure_inheritance_merged
-        
+
         @stimulus_builder&.to_attributes(component_instance) || {}
       end
 
@@ -35,7 +35,7 @@ module Vident
 
       def ensure_inheritance_merged
         return if @inheritance_merged || @stimulus_builder.nil?
-        
+
         if superclass.respond_to?(:stimulus_dsl_builder, true)
           parent_builder = superclass.send(:stimulus_dsl_builder)
           if parent_builder
@@ -71,4 +71,4 @@ module Vident
       resolved
     end
   end
-end
+end

data/lib/vident/version.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Vident
-  VERSION = "1.0.0.alpha1"
+  VERSION = "1.0.0.alpha2"
 
   # Shared version for all vident gems
   def self.version

data/lib/vident.rb

@@ -46,7 +46,7 @@ require "vident/stimulus_component"
 require "vident/component_class_lists"
 require "vident/component_attribute_resolver"
 require "vident/stimulus_builder"
-require "vident/stimulus_dsl"
+require "vident/stimulus_helper"
 
 require "vident/component"
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: vident
 version: !ruby/object:Gem::Version
-  version: 1.0.0.alpha1
+  version: 1.0.0.alpha2
 platform: ruby
 authors:
 - Stephen Ierodiaconou
@@ -102,7 +102,7 @@ files:
 - lib/vident/stimulus_controller.rb
 - lib/vident/stimulus_controller_collection.rb
 - lib/vident/stimulus_data_attribute_builder.rb
-- lib/vident/stimulus_dsl.rb
+- lib/vident/stimulus_helper.rb
 - lib/vident/stimulus_outlet.rb
 - lib/vident/stimulus_outlet_collection.rb
 - lib/vident/stimulus_target.rb