style_capsule 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9ca5bf9bf535558147384d22380ad4b54c6b21f85054e578a84f74d35f1d6296
-  data.tar.gz: 93c5a155435df7d39f6a2a739d0e6c0be1b99274bf8ebf658e2aab76f065bda1
+  metadata.gz: d1fcbe6c03f9675593d6dca7b3d7d70de2b7d471d2d938993b59877614332ff4
+  data.tar.gz: 6aa5feb7ff25ef755084e7754f328e4340a0d4dc620e1ea8298badbb0387f254
 SHA512:
-  metadata.gz: 34c0a4456e242d881d0c74ca58a726ad66f953ef2fddb41ba5d38fd98219588b6ed85cd07afa2fe352b34d3c75666496af28b6c2e94bbc8b24edcc65823738b3
-  data.tar.gz: 8cac6b0eb300d2030883313d70d69fc16063630bc9dd4efac6ed4396049814187aa83abe56b14d753baabb9afdc3eb3a0c7ce14aba8c49543aa9b7e4ce7ca381
+  metadata.gz: f06af1e47282aa0220b5ff28e5e0a197a8d76eaa9f3c00639958ca1f9d672c7ad1dd4f672b9de83cc7c521af6a49b9c9f9c9bd0ce476575211b1fc7a7aeb1da2
+  data.tar.gz: 3844dd9da6608b1eff6e495e19b92e3f90d4b41df323118f5fa7c6216603d3b33620c80df8aeb13c2e2506e133e3ab6e424c3dfd76feb16dbbc68b37fac72636

data/CHANGELOG.md

@@ -1,5 +1,14 @@
 # CHANGELOG
 
+## 1.2.0 (2025-11-24)
+
+- Added `StyleCapsule::ClassRegistry` for Rails-friendly class tracking without ObjectSpace iteration
+- Fixed development environment bug where `ObjectSpace.each_object(Class)` could trigger errors with gems that override `Class#name` (e.g., Faker)
+- Improved `ComponentBuilder` to use ClassRegistry first, with ObjectSpace fallback for compatibility
+- Classes are now automatically registered when including `StyleCapsule::Component` or `StyleCapsule::ViewComponent`
+- Better performance in development mode by tracking only relevant classes instead of iterating all classes
+- Enhanced error handling for classes that cause issues during iteration
+
 ## 1.1.0 (2025-11-21)
 
 - Made Rails dependencies optional: `railties` and `activesupport` moved to development dependencies

data/README.md

@@ -1,6 +1,6 @@
 # style_capsule
 
-[![Gem Version](https://badge.fury.io/rb/style_capsule.svg?v=1.1.0)](https://badge.fury.io/rb/style_capsule) [![Test Status](https://github.com/amkisko/style_capsule.rb/actions/workflows/test.yml/badge.svg)](https://github.com/amkisko/style_capsule.rb/actions/workflows/test.yml) [![codecov](https://codecov.io/gh/amkisko/style_capsule.rb/graph/badge.svg?token=2U6NXJOVVM)](https://codecov.io/gh/amkisko/style_capsule.rb)
+[![Gem Version](https://badge.fury.io/rb/style_capsule.svg?v=1.2.0)](https://badge.fury.io/rb/style_capsule) [![Test Status](https://github.com/amkisko/style_capsule.rb/actions/workflows/test.yml/badge.svg)](https://github.com/amkisko/style_capsule.rb/actions/workflows/test.yml) [![codecov](https://codecov.io/gh/amkisko/style_capsule.rb/graph/badge.svg?token=2U6NXJOVVM)](https://codecov.io/gh/amkisko/style_capsule.rb)
 
 CSS scoping extension for Ruby components. Provides attribute-based style encapsulation for Phlex, ViewComponent, and ERB templates to prevent style leakage between components. Works with Rails and can be used standalone in other Ruby frameworks (Sinatra, Hanami, etc.) or plain Ruby scripts. Includes configurable caching strategies for optimal performance.
 

data/lib/style_capsule/class_registry.rb

@@ -0,0 +1,101 @@
+# frozen_string_literal: true
+
+module StyleCapsule
+  # Registry for tracking classes that include StyleCapsule modules
+  #
+  # This provides a Rails-friendly way to track classes without using ObjectSpace,
+  # which can be problematic with certain gems (e.g., Faker) that override Class#name.
+  #
+  # Classes are automatically registered when they include StyleCapsule::Component
+  # or StyleCapsule::ViewComponent.
+  #
+  # @example
+  #   # Classes are automatically registered when they include modules
+  #   class MyComponent < ApplicationComponent
+  #     include StyleCapsule::Component  # Automatically registered
+  #   end
+  #
+  #   # Iterate over registered classes
+  #   StyleCapsule::ClassRegistry.each do |klass|
+  #     klass.clear_css_cache if klass.respond_to?(:clear_css_cache)
+  #   end
+  #
+  #   # Clear registry (useful in development when classes are reloaded)
+  #   StyleCapsule::ClassRegistry.clear
+  class ClassRegistry
+    # Use Set for O(1) lookups instead of O(n) with Array#include?
+    @classes = Set.new
+
+    class << self
+      # Register a class that includes a StyleCapsule module
+      #
+      # @param klass [Class] The class to register
+      # @return [void]
+      def register(klass)
+        return if klass.nil?
+        return if klass.singleton_class?
+
+        # Only register classes with names (skip anonymous classes)
+        begin
+          name = klass.name
+          return if name.nil? || name.to_s.strip.empty?
+        rescue
+          # Skip classes that cause errors when calling name (e.g., ArgumentError, NoMethodError, NameError)
+          return
+        end
+
+        @classes.add(klass)
+      end
+
+      # Remove a class from the registry
+      #
+      # @param klass [Class] The class to unregister
+      # @return [void]
+      def unregister(klass)
+        @classes.delete(klass)
+      end
+
+      # Iterate over all registered classes
+      #
+      # @yield [Class] Each registered class
+      # @return [void]
+      def each(&block)
+        # Filter out classes that no longer exist or have been unloaded
+        # Use delete_if for Set (equivalent to reject! for Array)
+        @classes.delete_if do |klass|
+          # Check if class still exists and is valid
+          klass.name.nil? || klass.singleton_class?
+        rescue
+          # Class has been unloaded or causes errors - remove from registry
+          true
+        end
+
+        @classes.each(&block)
+      end
+
+      # Get all registered classes
+      #
+      # @return [Array<Class>] Array of registered classes
+      def all
+        # Filter out invalid classes
+        each.to_a
+      end
+
+      # Clear the registry
+      #
+      # Useful in development when classes are reloaded.
+      #
+      # @return [void]
+      def clear
+        @classes.clear
+      end
+
+      # Get the number of registered classes
+      #
+      # @return [Integer]
+      def count
+        each.count
+      end
+    end
+  end
+end

data/lib/style_capsule/component.rb

@@ -89,6 +89,9 @@ module StyleCapsule
 
       # Use prepend to wrap view_template method
       base.prepend(ViewTemplateWrapper)
+
+      # Register class for Rails-friendly tracking
+      ClassRegistry.register(base)
     end
 
     module ClassMethods

data/lib/style_capsule/component_builder.rb

@@ -21,34 +21,47 @@ module StyleCapsule
 
       # Find all Phlex components that use StyleCapsule
       #
+      # Uses ClassRegistry to find registered components (Rails-friendly, avoids expensive ObjectSpace scanning).
+      # Classes are automatically registered when they include StyleCapsule::Component.
+      #
       # @return [Array<Class>] Array of component classes
       def find_phlex_components
         return [] unless phlex_available?
 
         components = []
-        ObjectSpace.each_object(Class) do |klass|
+
+        # Use the registry (Rails-friendly, avoids expensive ObjectSpace scanning)
+        ClassRegistry.each do |klass|
           if klass < Phlex::HTML && klass.included_modules.include?(StyleCapsule::Component)
             components << klass
           end
+        rescue
+          # Skip classes that cause errors (inheritance checks, etc.)
+          next
         end
+
         components
       end
 
       # Find all ViewComponent components that use StyleCapsule
       #
+      # Uses ClassRegistry to find registered components (Rails-friendly, avoids expensive ObjectSpace scanning).
+      # Classes are automatically registered when they include StyleCapsule::ViewComponent.
+      #
       # @return [Array<Class>] Array of component classes
       def find_view_components
         return [] unless view_component_available?
 
         components = []
+
         begin
-          ObjectSpace.each_object(Class) do |klass|
+          # Use the registry (Rails-friendly, avoids expensive ObjectSpace scanning)
+          ClassRegistry.each do |klass|
             if klass < ViewComponent::Base && klass.included_modules.include?(StyleCapsule::ViewComponent)
               components << klass
             end
           rescue
-            # Skip this class if checking inheritance triggers ViewComponent loading errors
-            # (e.g., ViewComponent 2.83.0 has a bug with Gem::Version#to_f)
+            # Skip classes that cause errors (inheritance checks, etc.)
             next
           end
         rescue
@@ -56,6 +69,7 @@ module StyleCapsule
           # Silently skip ViewComponent components if there's an error
           # This allows the rake task to continue with Phlex components
         end
+
         components
       end
 

data/lib/style_capsule/railtie.rb

@@ -31,26 +31,16 @@ module StyleCapsule
       # This prevents memory leaks from stale cache entries during code reloading
       if Rails.env.development?
         config.to_prepare do
-          # Clear CSS caches for all classes that include StyleCapsule modules
-          # Skip singleton classes and handle errors gracefully
-          ObjectSpace.each_object(Class) do |klass|
-            # Skip singleton classes and classes without names (they can cause errors)
-            # Singleton classes don't have proper names and can't be safely checked
-            # Use fallback for blank? if ActiveSupport not available
-            name = klass.name
-            next if name.nil? || (name.respond_to?(:blank?) ? name.blank? : name.to_s.strip.empty?)
-            next if klass.singleton_class?
-
-            # Use method_defined? instead of respond_to? to avoid triggering delegation
-            # that might cause errors with singleton classes or other edge cases
-            begin
-              if klass.method_defined?(:clear_css_cache, false) || klass.private_method_defined?(:clear_css_cache)
-                klass.clear_css_cache
-              end
-            rescue
-              # Skip classes that cause errors (singleton classes, delegation issues, etc.)
-              next
+          # Use Rails-friendly class registry instead of ObjectSpace iteration
+          # This avoids issues with gems that override Class#name (e.g., Faker)
+          StyleCapsule::ClassRegistry.each do |klass|
+            # Clear CSS cache if the class has this method
+            if klass.method_defined?(:clear_css_cache, false) || klass.private_method_defined?(:clear_css_cache)
+              klass.clear_css_cache
             end
+          rescue
+            # Skip classes that cause errors (unloaded classes, etc.)
+            next
           end
 
           # Clear stylesheet manifest to allow re-registration during code reload

data/lib/style_capsule/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module StyleCapsule
-  VERSION = "1.1.0"
+  VERSION = "1.2.0"
 end

data/lib/style_capsule/view_component.rb

@@ -89,6 +89,9 @@ module StyleCapsule
 
       # Use prepend to wrap call method
       base.prepend(CallWrapper)
+
+      # Register class for Rails-friendly tracking
+      ClassRegistry.register(base)
     end
 
     module ClassMethods

data/lib/style_capsule.rb

@@ -92,6 +92,7 @@ module StyleCapsule
   require_relative "style_capsule/css_processor"
   require_relative "style_capsule/css_file_writer"
   require_relative "style_capsule/stylesheet_registry"
+  require_relative "style_capsule/class_registry"
   require_relative "style_capsule/component_styles_support"
   require_relative "style_capsule/component"
   require_relative "style_capsule/standalone_helper"

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: style_capsule
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Andrei Makarov
@@ -261,6 +261,7 @@ files:
 - README.md
 - SECURITY.md
 - lib/style_capsule.rb
+- lib/style_capsule/class_registry.rb
 - lib/style_capsule/component.rb
 - lib/style_capsule/component_builder.rb
 - lib/style_capsule/component_styles_support.rb